FIRRTL generating broken bit-masked sequential memories

Here's my Chisel code:
val h_table = SeqMem(num_entries, Vec(fetch_width, Bool()))
...

   val waddr = hwq.io.deq.bits.index
   val wmask = hwq.io.deq.bits.executed
   val wdata = hwq.io.deq.bits.takens
   h_table.write(waddr, wdata, wmask)
...

Which turns into perfectly acceptable FIRRTL:
smem h_table : UInt<1>[2][32768] @[2bc-table.scala 164:24]
...
  write mport T_99 = h_table[hwq.io.deq.bits.index], hwq.clock
  when hwq.io.deq.bits.executed[0] :
    T_99[0] <= hwq.io.deq.bits.takens[0]
    skip
  when hwq.io.deq.bits.executed[1] :
    T_99[1] <= hwq.io.deq.bits.takens[1]
    skip
  skip @[2bc-table.scala 172:4]

But then gets turned into the broken Verilog code in which it swaps the bit-mask bits and the write-data bits:
module h_table(
  input  [14:0] RW0_addr,
  input   RW0_en,
  input   RW0_clk,
  input   RW0_wmode,
  input   RW0_wdata_0,
  input   RW0_wdata_1,
  output  RW0_rdata_0,
  output  RW0_rdata_1,
  input   RW0_wmask_0,
  input   RW0_wmask_1
);
  wire [14:0] h_table_ext_RW0_addr;
  wire  h_table_ext_RW0_en;
  wire  h_table_ext_RW0_clk;
  wire  h_table_ext_RW0_wmode;
  wire [1:0] h_table_ext_RW0_wdata;
  wire [1:0] h_table_ext_RW0_rdata;
  wire [1:0] h_table_ext_RW0_wmask;
  h_table_ext h_table_ext (
    .RW0_addr(h_table_ext_RW0_addr),
    .RW0_en(h_table_ext_RW0_en),
    .RW0_clk(h_table_ext_RW0_clk),
    .RW0_wmode(h_table_ext_RW0_wmode),
    .RW0_wdata(h_table_ext_RW0_wdata),
    .RW0_rdata(h_table_ext_RW0_rdata),
    .RW0_wmask(h_table_ext_RW0_wmask)
  );
  assign RW0_rdata_0 = h_table_ext_RW0_rdata[1];
  assign RW0_rdata_1 = h_table_ext_RW0_rdata[0];
  assign h_table_ext_RW0_addr = RW0_addr;
  assign h_table_ext_RW0_en = RW0_en;
  assign h_table_ext_RW0_clk = RW0_clk;
  assign h_table_ext_RW0_wmode = RW0_wmode;
  assign h_table_ext_RW0_wdata = {RW0_wdata_0,RW0_wdata_1}; /// <<---- !!!
  assign h_table_ext_RW0_wmask = {RW0_wmask_1,RW0_wmask_0}; /// <<---- !!!
endmodule

This kills the processor.

Whoa I assume this wasn't there before? I can fix it tonight w/ some other refactoring I intend to do.

Thanks, I really appreciate it.

Whoa I assume this wasn't there before?

I don't really pay attention to when my FIRRTL version gets bumped, so I don't know how long this has been around (BOOM gets bumped whenever Rocket-chip gets bumped).  It shows up in the branch predictor, so I only noticed this by analyzing the performance and seeing a regression.

lol weird broken edge case:
circuit Top:
  module Top:
    input clk: Clock
    input ptr : UInt<5>
    input mask: UInt<1>[2]
    input data: UInt<1>[2]

    smem h_table : UInt<1>[2][32]
    write mport T_99 = h_table[ptr], clk
    when mask[0] :
      T_99[0] <= data[0]
      skip
    when mask[1] :
      T_99[1] <= data[1]
      skip
    skip

Generates
  assign h_table_ext_W0_data = {W0_data_0,W0_data_1};
  assign h_table_ext_W0_mask = {W0_mask_1,W0_mask_0};

Whereas
circuit Top:
  module Top:
    input clk: Clock
    input ptr : UInt<5>
    input mask: UInt<1>[2]
    input data: UInt<5>[2]

    smem h_table : UInt<5>[2][32]
    write mport T_99 = h_table[ptr], clk
    when mask[0] :
      T_99[0] <= data[0]
      skip
    when mask[1] :
      T_99[1] <= data[1]
      skip
    skip

Generates the correct:
  assign h_table_ext_W0_data = {W0_data_0,W0_data_1};
  assign h_table_ext_W0_mask = {W0_mask_0,W0_mask_1};

Should be a bug in MemUtils. Will fix.

Thanks for working on this.
Unfortunately, I'm having trouble testing this.   I've checked out the fix_bitmask branch on my firrtl submodule in rocketchip, and executed "sbt assembly" within the top directory, but I'm getting errors on trying to build rocket-chip with it:
Error: Unknown option --inferRW
Error: Unknown argument 'TestHarness'
Error: Unknown option --replSeqMem
Error: Unknown option --inferRW
Error: Unknown argument 'TestHarness'
Error: Unknown option --replSeqMem
Error: Unknown option -c:TestHarness:-o:/scratch/celio/boom-chip/vsim/generated-src/rocketchip.BOOMConfig.conf
Try --help for more information.

Manually applying your fix to the hiermask function to my copy of FIRRTL does indeed generate the correct code.  ^^
assign h_table_ext_RW0_wdata = {RW0_wdata_0,RW0_wdata_1};
assign h_table_ext_RW0_wmask = {RW0_wmask_0,RW0_wmask_1};

Haha ok... yeah, I think rocket-chip firrtl is too far behind master (what with all of the new execute harness changes...)

you should look at rocket-chip/#409 for how to fix it.

by #346, #349


The oxygen bar decreases when you are treated with a poultice

@Craluminum2413 Thanks for the report. Version 0.2.4 fixes it.


Fix Android behaviour with Android Gradle Plugin >=4.2.0

React Native 0.65 upgraded the Android Gradle Plugin to 4.2.1, which causes anything using react-native-config to fail with the following error:
The value for property 'buildConfigFields' is final and cannot be changed any further.

Gradle has multiple phases: https://docs.gradle.org/current/userguide/build_lifecycle.html. The reason for this error is that Gradle executes the contents of task.doFirst during the execution phase. However, during execution then buildConfigFields is final so cannot be modified. It can only be modified during the configure phase.
This PR removes the task.doFirst callback, and simply executes the code inside the callback immediately during the configure phase. This has been tested by myself and some other users (see #567) and still works with multiple build types and multiple flavors, both with gradle plugin 4.1.x and 4.2.x.
My assumption as to why this upgrade broke react-native-config is that Gradle decided to strictly enforce that buildConfigFields is final during the execution phase, whereas in previous versions it probably wasn't enforced but should have been. Google are aware of this change in behaviour (https://issuetracker.google.com/issues/172657565) and closed the issue with a Won't fix (intended behaviour) response. So it's up to react-native-config to change to be compatible with this new behaviour.
Fixes #567
Fixes #578
Fixes #608

May I know which version of gradle plugin you use for this fixing? For RN-0.65.1 they used 4.2.1. I try to implement but still got some errors, Need help, thanks

May I know which version of gradle plugin you use for this fixing? For RN-0.65.1 they used 4.2.1. I try to implement but still got some errors, Need help, thanks

I've verified this with classpath 'com.android.tools.build:gradle:4.2.1', distributionUrl=https\://services.gradle.org/distributions/gradle-6.9-all.zip and worked fine for me.

@octsiri I'm using the same as @mlazari is using. It's the same versions as RN-0.65.1 uses. You also get errors when updating to 4.2.x if you're using an old version of Android Studio, make sure you're on the latest. I expect your errors are likely due to something different. If you're using the "quick fix" way of editing a file in node_modules to implement this PR just to see if it works, make sure you clean your build to ensure the changes are definitely picked up as well.

@sanduluca Thanks for approving - what needs to be done to merge this?


psar() parameters

Apologies for this very basic question.  I'm developing an FX trading system, and learning to use Python + Pandas-TA along the way.
With this statement:
psardf = ta.psar(high=psar_df["high"], low=psar_df["low"], close=psar_df["close"], start=0.01, step=0.01, max_step=0.2, fillna=True)
I've used 'start', 'step', 'max_step' parameters to initialise the psar().  Is this correct?
I've run it, and it seems to work without error, but the only documentation I can find is reference to parameters ap0, ap, max_ap.
Cheers.

Hello @keith5551,
First if you are not using the development version, you should. (Better performance, documentation, et al...)
$ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development

Second, for help on an indicator:
import pandas_ta as ta
help(ta.psar)

I've run it, and it seems to work without error, but the only documentation I can find is reference to parameters ap0, ap, max_ap.

Those are the correct parameters that help(ta.psar) describes. However start, step and max_step have no effect; they are not valid psar arguments in Pandas TA. Given your code above, I would replace start with af0, step with af, max_step with max_af,
Also the fillna argument fills the na values with what you set it equal to. Given your code above, it will fill the na values with True. Not sure that is your intention since it's a non-numerical value. 🤷🏼♂ See Pandas fillna for more info.
import pandas_ta as ta

df = # your ohlcv data
psardf = df.ta.psar(af0=0.01, af=0.01, max_af=0.2)

Also here is an example of the resultant psardf (with default parameters):

Note: The first 2 columns show long (green) and short (red) lines. If you need one line, you can combine them like so:
# Combine Long and Short SAR"s into one SAR value
psar = psardf[psardf.columns[:2]].fillna(0)
psar = psar[psar.columns[0]] + psar[psar.columns[1]]
psar.iloc[0] = np.nan

The second chart keeps track of af. The last chart shows when a reversal happens; psar changes from long to short or short to long. If you do not find those last two columns useful, just drop or exclude them.
Hope this helps!
KJ

Thanks Kevin for your comprehensive response.  Very much appreciated.  This has helped me greatly.
Cheers. Keith


proposing change

seems reasonable

Hi @4c74356b41, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@4c74356b41 : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@NarayanAnnamalai - Hi Narayan, please review this change. If it's okay, please update this note format to Markdown. Thanks! Katie
#hold-off

I reviewed the change and it looks good.
#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.

@NarayanAnnamalai Hi Narayan. Please resolve the conflicts to enable a merge. Usually, authors of the files are able to fix merge conflicts using the process here on the [wiki](https://microsoft.sharepoint.com/teams/azurecontentguidance/wiki/Pages/Resolve a local merge conflict yourself.aspx). Thanks.

Just a ping ;)

@4c74356b41 thanks for your contribution. Due to the merge conflict, the changes have been incorporated in this PR.

#please-close


abstract class doesn't parse in expression position

TypeScript Version:  2.4.2 / nightly (2.5.0-dev.20170725)
Code
function f1() {
  return abstract class {}
}

function f2() {
  const x = abstract class {}
  return x
}

function f3() {
  abstract class x {}
  return x
}

Expected behavior:
No errors reported by the compiler.
Actual behavior:
some.ts(2,19): error TS1005: ';' expected.
some.ts(6,22): error TS1005: ',' expected.

We aren't looking to support them (see #4578), but it wouldn't hurt to parse these and provide a useful error.

Please implement this, simplifies notation, when using mixins.


endorsed not exist in JDK 12 anymore

Description:

Hi, I am trying to run sh wso2server.sh but I have an error "-Djava.endorsed.dirs=/Library/WSO2/IdentityServer/5.7.0/lib/endorsed:/Library/Java/JavaVirtualMachines/jdk-12.0.1.jdk/Contents/Home/jre/lib/endorsed:/Library/Java/JavaVirtualMachines/jdk-12.0.1.jdk/Contents/Home/lib/endorsed is not supported. Endorsed standards and standalone APIs
in modular form will be supported via the concept of upgradeable modules.
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit."
Since JDK 12 does not have endorsed.dirs, how can I overcome this problem? Thank you in advance!!!
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Hi @mbgazi ,
WSO2 Identity server 5.7.0 does not support java version more than 8. The latest released WSO2 Identity server 5.9.0 will support JDK11 (which is the latest java LTS version). You can try out with 5.9.0 version. This issue should be fixed. But yet we haven't tested the 5.9.0 with JDK 12.

I'm having the same issue with JDK 11, does anyone knows how to fix it?
JAVA_HOME environment variable is set to C:\Program Files\Java\jdk-11.0.14
CARBON_HOME environment variable is set to C:\PRD0B2~1\NEKI\R2D2\WSO2AM~1.0\bin..
Endorsed standards and standalone APIs in modular form will be supported via the concept of upgradeable modules.
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Change test data names and improve docs.

Change header in test data files to reflect new naming convention.
Change tests, doc notebooks to run with new names in test data.
Clean up docs.

Resolves #39, #40 and #46.


WebEngineView & SecComp

At the moment we've disabled seccomp for web engine on linux because of this bug:
https://bugreports.qt.io/browse/QTBUG-96214
Before we open apps for public upload need

build custom patched Qt 5.15

OR

move to Qt 6.x

Moved to 6.4. Delay approved by Romanov.


pkgs/misc/mosquitto-go-auth: init at 0.2.0

Motivation for this change
Things done

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

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

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

Closing as OP doesn't respond to review requests.


Incomplete information [Sys reqs, limits, etc.]

There are other limitations like camera control image and resolution size, max size of media and Storage limits for offline apps etc are missing.

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

ID: 6fdf0872-929b-e44a-dedd-eb6a759915f2
Version Independent ID: 0b65b025-62f8-fc2a-0026-3cb96b7b4554
Content: System requirements, limits, and configuration values for canvas apps - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/limits-and-config.md
Service: powerapps
GitHub Login: @gregli-msft
Microsoft Alias: gregli

@JS2Code, we've added a new article on data types that covers size limitations.  In short, there are no built in limitations, however the memory available to an app especially on older devices can be problematic, requiring apps to be tested with those configurations if there is a concern.

@aftowen, please close this issue.

@JS2Code We've also added a link from this topic to the new information. Cheers.
@gregli-msft FYI.


KAFKA-10295: Wait for connector recovery in test_bounce

Signed-off-by: Greg Harris<EMAIL_ADDRESS>This additional waiting is for:

The scheduled.rebalance.max.time.ms to expire and for tasks to be reassigned
The source connector to start and commit offsets at least once beyond it's last commit
The sink connector to have adequate time to read any of the messages written by the source

Prevents flakey test failures with messages like "Found sink sequence number greater than any generated sink sequence number for task %d: %d > %d" when test_bounce is running with hard bounces and Incremental Cooperative Rebalancing enabled.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @rhauch for review

@gharris1727 Thanks for the proposed fix. I've started a build for this branch: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/4046/

All 69 Connect system tests passed: http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2020-07-20--001.1595250965--gharris1727--bounce-recovery--c00de2d82/report.html


hello_world.scala

Added any language

Are in scala have filename convention? I think I need to know about this too.

You're right, corrected the filename to reflect Scala conventions


How to figure out which native proxy events to use with derived events?

On my architecture, papi_avail shows that -- for example -- PAPI_L1_DCA is available but derived.  So when I try to run:
mpirun -np 1 hpcrun -e PAPI_TOT_CYC -e PAPI_TOT_INS -e PAPI_L1_DCA <myprogram.exe>
I get the error:
HPCToolkit fatal error: PAPI event PAPI_L1_DCA is a derived event and thus cannot be profiled.
The docs mention that a native event may be required for proxy sampling a derived event.  How can I find out which native event I need to use for a particular derived event?
Thank you for any help!
Note: I am using IBM Power9

Use papi_avail -d to see the definitions of the derived events. The definition of a derived event will tell you what collection of native events you can sample. You can compute the derived event in HPCToolkit's hpcviewer user interface as an expression computed from native event measurements.


focusState property in MenuItem can't be set

Problem description
When I set the property focusState in a MenuItem component it doesn't cae about it and it is replaced for other values.
Steps to reproduce
Create many MenuItem in a Menu and set the property focusState with different values.
Versions

Material-UI: 0.15.4
React: 15.3.2
Browser: Chrome 53.0.2785.143 (64-bit)

I kinda managed to get around this bug ... using Menu.props.disableAutoFocus, set it to :

true for normal mouse focus events
false to allow keyboard-focused events

Just forget about MenuItem.focusState
Took me some hours to get the thing working tough....

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


Please test/review Python for issue 22490

In bpo-22490, a bugfix is now in place for an issue where PYVENV_LAUNCHER would leak to the interpreter and child processes. The fix was applied today to the master, 3.8, and 3.7 branches.
One action item that remained was to test the change against homebrew. Please consider testing one of these branches against homebrew as time permits. Thanks.

Hi. We may need a little bit of time to test this. The initial issue was reported in 2014 and there is a lot of discussion in that ticket. @tdsmith may give us some help if he is still around :)


F51-214, F51-222 Anonymous Shopping

Summary of changes:

When app.constants.json > anonymous is set to true, the ocRefreshToken service will attempt to authorize anonymously using the current clientid and scope constants.
An error will be logged to the console if the clientid is not configured to work with anonymous authentication.
Do not store anon refresh tokens: storing these tokens will trigger the refresh token workflow which does not correspond to an anonymous authentication.
Comments have been added in the code explaining why we do not store the refresh tokens from the anonymous auth response.
The /register component has been added. Anonymous users can access this at any time to profile themselves.
The login.controller.js has been updated to merge anon orders with profiled users most recent unsubmitted order
Checkout state has a resolve for opening up the ocAnonymous.Identify() modal which will force anon users to either login or register before continuing to checking out.
ocAnonymous has a "Redirect" function for sending the newly profiled user to the state that was passed into the ocAnonymous.Identify() modal (usually "cart" but it could be anything)
Remove the redirect cookie after a successful ocAnonymous.Redirect()
Block the registration state from non-anonymous users & applications
Fix undefined error blocking ocStateLoading from completing
Add a redirectState argument to ocRefreshToken(redirectState) for controlling where the user is sent to after the new token is retrieved.
Registration form validation for username conflicts.

See the commit messages for more details.

I confirmed the login unit tests are still passing. I will create tasks for unit testing the common/services/oc-anonymous directory.


Proposal: canonical path for icon files.

Some applications may display icon files for Helm charts (for example, service catalog and marketplace web UIs). At one point, we had an icon: directive in the Chart.yaml, but removed it because it was never particularly well-defined. However, there is a clear need to associate icons with charts.
There are three possibilities for doing this:

Add a icon: URL directive in the Chart.yaml file, and require the value to be a URL to a remote resource.
Specify a canonical file name for icons (icon.svg or icon.png), and suggest that packagers put the icon inside of the chart.
Add an icon: PATH_OR_URL directive in Chart.yaml

For the sake of simplicity and (network) security, I am in favor of option 2. I'd suggest that we support both PNG and SVG formats, and look in the chart root for icon.svg or icon.png. I suggest that we also provide guidelines on sizing for PNG images.
Since neither Helm nor Tiller directly use images, this would likely be more a matter of documentation than anything else.

/cc @runseb

The problem with 2. is that if you read the index.yaml file you will need to extract the icon from the packaged chart to build up your marketplace display. Hence download the entire chart.
So I am actually in favor of 1.
PS: What we have done so far is to have our own map from the chart name to a local icon . It is not that bad after all.

Oh, I suppose we could have an option 4, which would be to place the icon file in the repo alongside the tgz file.
I see what you mean about 2. Requiring apps to fetch and unpack the tarball in order to get the image is maybe a bit of a burden.

So right now, momentum is leaning toward Option 1: Add a URL field (icon: FULL_URL).

By going with 1: we make it easy for all readers of chart index.yaml / Chart.yaml to have a URL they can use without any magic/scanning logic.
For chart creators, the tool which packages and publishes the chart (or the repository) could take care of uploading the icon somewhere and filling in the full URL? So it might be for publishes it may look like 2: its just the tool fills in 1: for you?


feat!: v7 [WIP]

docs: https://uploadthing-docs-twui.vercel.app/
migration guide: https://uploadthing-docs-twui.vercel.app/migration-guides/v7
release post somewhere?

reminder for self: update test tokens


Materials for new belt drive system

1045 Carbon Steel Keyed Rotary Shaft Fully Keyed, 14mm Diameter, 200mm Long - $16

https://www.mcmaster.com/1439K26/

Quick Detachable Bushing, QT Series, 1/2" Bore Dia., 1.250" Length - $14 x 2?

https://www.grainger.com/product/TB-WOOD-S-Quick-Detachable-Bushing-5UHZ6?_

Flanged Ball Bearing ID 1/2" x OD 1-3/8" - $2 x 2

https://www.amazon.com/XiKe-Bearing-Wheelbarrows-Suitable-Replacement/dp/B01J67N29I/ref=sr_1_6?dchild=1&keywords=flanged+ball+bearings+.5in&qid=1595640159&sr=8-6_

Motor: RMD-X7

https://www.aliexpress.com/item/10000397235320.html?spm=2114.12010612.8148356.3.66fd22a06keXEs

Pulleys:

https://www.aliexpress.com/item/4000768114918.html?spm=2114.12010612.8148356.11.3b70e29dVvoN7N
https://www.aliexpress.com/item/4000499698920.html?spm=2114.12010612.8148356.13.3b70e29dVvoN7N

Belt:

https://www.aliexpress.com/item/32791799628.html

@wilton-lee did you verify that the keyway, pulley width and everything else match? Because if you copied this from my presentation, I did not do all of the due diligence to verify that everything works. I just needed the materials to calculate approximate cost and weight.

I was using this list more to organize the links of potential parts to
choose from so I have quicker access. I will add a note to my main post to
clarify this.
On Sun, Aug 2, 2020, 9:21 AM Khalil Estell<EMAIL_ADDRESS>wrote:

@wilton-lee https://github.com/wilton-lee did you verify that the
keyway, pulley width and everything else match? Because if you copied this
from my presentation, I did not do all of the due diligence to verify that
everything works. I just needed the materials to calculate approximate cost
and weight.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SJSURoboticsTeam/urc-central-2021/issues/93#issuecomment-667694479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7AWV6VUCFYVOFXEC2NDHDR6WHBFANCNFSM4PHTXT7Q
.

Looking for 12mm keyed shaft ~200mm long OR 14mm flanged ball bearing OR compatible shaft+flanged bearing that can support loads

ready for purchase


CSDID implementation error : (number of observations=0)

I have five variables[ (fid or firm id - Panel identifier),  (year- time ), (group variable - first_treat constructed in which year a firm received  the treatment), depvar- Log(loans+1) and indepvar (relationship with a private bank or not)] in the dataset for now. The only problem in the data is that there are few fid which never got treated and there are many firms which got the treatment later on.
When I run the csdid command in the stata as
csdid ln_loans priv, ivar(fid) time (year) gvar(first_treat) method(reg)
It shows that number of observations are 0 and everything is absorbed (basically omitted).
I dropped the firms in the dataset which never received the treatment and rerun the same. It did not work with that as well.
Kindly help.
Thank you!!

I am trying to solve the same problem. Did you have any progress?


404 dojo-list-index-controller.js

When loading https://zen.coderdojo.com/dojo/861
I get a 404-SyntaxError for https://zen.coderdojo.com/js/controllers/dojo-list-index-controller.js

Hi @niccokunzmann - could you send a screenshot and let us know what browser you are using please? I can't see this error myself

Firefox 43.0.4

Looks like a file got renamed, fix here; https://github.com/CoderDojo/cp-zen-platform/pull/824


bug(ui): reset anchors also resets camera rotation

When Reset anchors is pressed, only reset the anchor positions and leave the camera rotation unchanged.

@Lucaszw please add a reference to the commit where this was implemented.

1fed8c36b00eedc40fe38663bfa72ddd8ce7d19c


Support ECDSA key-generation/signing in TPM2.0

One of the main results of an XTT handshake is the creation of a client's identity ("longterm") keypair (such that the server links the public key with the identity assigned to the client). The private key of this pair is very sensitive (until this client runs another XTT handshake, this private key will be used to prove the client's identity during all subsequent ENF sessions).
Currently, this project saves this identity keypair to disk, in plaintext. To improve the security of this valuable private key, we should generate it on a TPM (secure hardware) such that it never leaves the TPM (and, in fact, is defined to be incapable of leaving the TPM; cf. the TPM flags that can be set during the key's creation).
The xaptum-tpm project, which this project links to transitively via its libecdaa-tpm dependency (when the USE_TPM CMake option is on, of course), already supports ECDSA (using only secp256r1 or p256 or whatever you want to call that curve) key-generation and signing (cf. the test/tss2_sys_sign-secp256-test.c file for example usage).
A rough outline of this work could be (all the new code should be wrapped in USE_TPM ifdefs):

Add xtt_crypto_create_ecdsap256TPM_key_pair and xtt_crypto_sign_ecdsap256TPM declarations in include/xtt/crypto_wrapper.h
Create a src/xaptum_tpm_wrapper.c file

Implement  xtt_crypto_create_ecdsap256TPM_key_pair and xtt_crypto_sign_ecdsap256TPM in this file, calling the xaptum-tpm functions
The keypair-generation function will need to make sure it copies the public key out of the TPM structure and into the XTT buffer in the correct format (for later use during the handshake)

Add a longterm_sign_ecdsap256TPM function in src/internal/crypto_utils.c

This simply calls xtt_crypto_sign_ecdsap256TPM (just like sign_lrswTPM is a simple wrapper of xtt_daa_sign_lrswTPM)

Add a xtt_initialize_client_handshake_context_TPM function to context.[h,c]

This will be almost the exact same as xtt_initialize_client_handshake_context, except it will use xtt_crypto_create_ecdsap256TPM_key_pair rather than xtt_crypto_create_ecdsap256_key_pair.
There's certainly a cleaner way to support these two functionalities, but this is the way the TPM-vs-no-TPM is handled for xtt_initialize_client_group_context_lrswTPM, so it seems best to just stick to that for now

The above outline is intended to be a minimal effort plan, and as such doesn't help with the existing poorly-conceived API in this project. While we need to re-architect the crypto design, I think this is important-enough functionality that adding support for it before delving into redesign is a good idea.
Questions:

How to handle key handle?

Should this be a Xaptum-wide-constant, defined in xaptum-tpm (like the NVRAM indices, etc.)?
Or, should it be user-configurable within this project?

@drbild I'd be interested in your thoughts on this

I agree with adding support as outlined here, before re-architecting the internal crypto APIs.

How to handle key handle?
I think a Xaptum-wide-constant defined in xaptum-tpm is the most straightforward approach.  That keeps the definitions for all Xaptum-required handles in one place.  Should there ever be a need for a different handle, it would be easy to modify xtt to allow overriding that handle.


risedev psql quit when Ctrl-C once

Describe the bug

Error message/log
No response
To Reproduce
./risedev psql
Expected behavior

How did you deploy RisingWave?
No response
The version of RisingWave
No response
Additional context
No response

Interesting

I guess this is because cargo-make runs script in a child process, and thus signal handling is not considered.

+1. This is kind of annoying.

Saw that @xxchan opened an issue in the cargo-make repo. Will wait for updates on that. Thanks @xxchan!
https://github.com/sagiegurari/cargo-make/issues/954

Can use this instead https://github.com/risingwavelabs/risingwave/pull/15294 to launch psql

Any updates?

Didn't have a chance to do https://github.com/sagiegurari/cargo-make/issues/954 yet. 🤪


Support for https_redirect - or simple slash-adding redirects

Please describe your use case / problem.
We want to use ambassador as reverse proxy between several services which each should be available at a specific path. We'd like ambassador to behave like this:
GET /service1 -> redirect to /service1/
GET /service1/ -> proxy as / to service1
GET /service1/a -> proxy as /a to service1
GET /service1a -> not found

In my opinion, this is pretty standard. We always want to serve the service with a trailing slash so that relative URLs from within the service are properly resolved. If the user however enters it without slash, they should end up on the service (thus the redirect from /service1), service1a clearly isn't service1, so it should neither be proxied nor redirected to the service.
Describe the solution you'd like
Ideally, we would deploy one mapping, configure the prefix (service1) on it, and toggle some switch to enable this behavior (because obvously there are good use cases for other behaviors). Bonus points if the behavior of /service1 could be switched between a redirect and a proxy request to / on the service, for services that can cope with it.
Describe alternatives you've considered
Currently, we deploy two mappings:

one with prefix /service1/ and the regular config for the proxy call
one with prefix /service1 and prefix_regex: true so it only matches exactly. This mapping uses host_redirect and path_redirect to redirect to /service1/.

There are three issues with this approach

It's pretty complicated (requires two mappings per service) for something I'd consider best practice
We need to configure the host and complete path (ambassador might be running behind another reverse proxy, so that path needs to be added, too) on the mapping, so we need to know these values that we otherwise wouldn't have needed here
This is the main issue - it does not let us configure the HTTPS scheme. The redirect always uses a http:// Location header. Maybe ambassador could add the envoy option https_redirect?

-> The "simple" feature request is just adding https_redirect. I have elaborated our use case quite a bit for background information, and it would be great to be able to configure this more easily, but I think just https_redirect would already work for us.

Can we reopen this issue please?  We are having the same issue with https redirects


Document how to cancel a subscription stream from the client side

https://github.com/enisdenjo/graphql-ws/blob/master/PROTOCOL.md#complete
Direction: bidirectional
Server -> Client indicates that the requested operation execution has completed. If the server dispatched the Error message relative to the original Subscribe message, no Complete message will be emitted.
Client -> Server indicates that the client has stopped listening and wants to complete the subscription. No further events, relevant to the original subscription, should be sent through. Even if the client sent a Complete message for a single-result-operation before it resolved, the result should not be sent through once it does.
Note: The asynchronous nature of the full-duplex connection means that a client can send a Complete message to the server even when messages are in-flight to the client, or when the server has itself completed the operation (via a Error or Complete message). Both client and server must therefore be prepared to receive (and ignore) messages for operations that they consider already completed.
interface CompleteMessage {
  id: '<unique-operation-id>';
  type: 'complete';
}

Completing the subscription from a client side is supported. When the Flux for the stream is cancelled, we send the "complete" message to the server. How to cancel the Flux depends on how you use it. Some operators like take and timeout send the cancel signal. If subscribing to the Flux with a Subscriber you can also get a reference to the Subscription and cancel through it.
We can improve the reference documentation on this topic around here.

Thanks you very much for your reply @rstoyanchev It makes sense. If you could improve the reference documentation, it would be great help for the users of this liberary.


adding clarification about fraud alerts reports

I added some helpful information based on cases received. Thanks team

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

Thanks for the contribution, @rdiazel
#sign-off


I get lots of errors when I run with -d

Great app, but I was trying to run using the -d flag and I get TONS of errors:
cvepri -d

#    ______   ______
#   / ___/ | / / __/
#  / /__ | |/ / _/
#  \___/_|___/___/        _ __  _
#    / _ \____(_)__  ____(_) /_(_)__ ___ ____
#   / ___/ __/ / _ \/ __/ / __/ /_ // -_) __/
#  /_/  /_/ /_/\___/_/ /_/\__/_//__/\__/_/
#                                  BY TURROKS

CVE-ID            Priority
------------------------------
CVE-2023-1389     Error
CVE-2023-20869    Error
CVE-2023-29489    Error
CVE-2023-28771    Error
CVE-2023-27524    Error
CVE-2023-27350    Error
CVE-2023-21839    Error
CVE-2021-45046    Error
CVE-2023-27532    Error
CVE-2023-21932    Error

I am using python 3.9.7 and Darwin LNGRDUM-C02Z705DLVDQ 21.6.0 Darwin Kernel Version 21.6.0: Thu Mar  9 20:08:59 PST 2023; root:xnu-8<IP_ADDRESS>0.8~1/RELEASE_X86_64 x86_64
Mac Monterey 12.6.5 INTEL
Thomas

Hi Thomas, thanks for your kind words and for creating an issue. I will try to recreate the problem and get back to you soon.
Does it only happen with the -d flag? Does it work properly if you try to get the info for one of those CVEs using the normal -c function?

It seems to work fine on specific CVE's, just the -d

From: Mario Rojas @.***>
Sent: Wednesday, May 3, 2023 00:06
To: TURROKS/CVE_Prioritizer
Cc: Munn, Thomas (LNG-RDU); Author
Subject: Re: [TURROKS/CVE_Prioritizer] I get lots of errors when I run with -d (Issue #3)
*** External email: use caution ***
Hi Thomas, thanks for your kind words and for creating an issue. I will try to recreate the problem and get back to you soon.
Does it only happen with the -d flag? Does it work properly if you try to get the info for one of those CVEs using the normal -c function?
—
Reply to this email directly, view it on GitHubhttps://github.com/TURROKS/CVE_Prioritizer/issues/3#issuecomment-1532433021, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALVIUB5AQ6TVFMZIZVX6WDTXEHKT7ANCNFSM6AAAAAAXTIUTOQ.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Thomas, thanks for getting back to me, Im able to replicate now, it looks like it's an issue from NIST's API, I'll work on a solution for this issue and also add more detailed error messages.
Best,
Mario

Hi Thomas,
The errors are associated with NIST rate-limiting requests that don't provide an API key.
I have added support for doing so; after testing the tool with and without the Key, it looks like the issue is resolved.
You can request a key here https://nvd.nist.gov/developers/request-an-api-key
Once you get the Key, you can add it to the new .env file included in the project like so NIST_API=your-key-goes-here
Please make sure to install the new required libraries as well.
Best,
Mario

Hey Thomas,
Im closing this issue for now, please feel free to re-open it if the issue persists after using the API

I just ran it with a key. works fine now. Thanks for the update!


add test for multibyte unicode support

Tests for: https://github.com/groupon-testium/webdriver-http-sync/pull/24
Addresses: https://github.com/groupon-testium/testium/issues/124

published in 2.5.0


[pigeon] asserts when java platform throws NPE.

@gaaclarke
I'm using pigeon 0.10.0.
When calling a platform method that generates an NPE (on the platform side) the pigeon code is generating an assert:
2020-10-08 11:12:46.626 6638-6672/com.bsutton.sounds.sounds_example E/flutter: [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: 'package:flutter/src/services/message_codec.dart': Failed assertion: line 111 pos 15: 'code != null': is not true.
    #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:46:39)
    #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:36:5)
    #2      new PlatformException (package:flutter/src/services/message_codec.dart:111:15)
    #3      SoundsToPlatformApi.startPlayer (package:sounds_platform_interface/sounds_platform_interface.dart:868:13)

The pigeon code generates:
  }
  private static HashMap wrapError(Exception exception) {
    HashMap<String, Object> errorMap = new HashMap<>();
    errorMap.put("message", exception.toString());
    errorMap.put("code", null);
    errorMap.put("details", null);
    return errorMap;
  }

As you can see the generated code sets put("code", null) which the core messages_code class does not accept.
My suggestion would be to set 'code' to be the simple name of the exception.

thanks.


ProblemData usage is ambiguous

My understanding is that we have an abstract ConstraintBuilder which is templatized by some <ProblemData> type.
Then, the user implements concrete ConstraintBuilder such as FrictionConeConstraintBuilder, which are also templatized by some <ProblemData> type (but we know ahead of time that that type will be a user-defined FrictionConeProblemData).
Then, the FrictionConeConstraintBuilder overrides the virtual ConstraintBuilder methods, and creates the ingredients to produce a ConstraintData.
Up to here, I think I understand.
What I do not understand is how we templatize TrajectoryOpt with some < ProblemData > type. I am assuming the intent is so that we can run the concrete xxxConstraintBuilder's from within TrajectoryOpt, so that the user does not have to. This would also be convenient for updating the ConstraintData. I imagine we have something like this pseudocode:
class LeggedRobotProblemData
{
    GeneralProblemData;
    std::vector<ProblemDatas*> problem_data_vec = {FrictionConeProblemData, ContactProblemData}
}

traj_opt(LeggedRobotProblemData, std::vector<ConstraintBuilder> = {FrictionConeConstraintBuilder, ContactConstraintBuilder})

    in traj_opt:
    Gvec = {}
    i = 0
    for (auto &builder : builders)
    {
        builder->build_constraints(problem_data_vec[i]);
        Gvec.insert(builder->get_g());
        ++i
    }

However, there is not yet a base class which xxxSpecificProblemData's are derived from, so we cannot index them from a container within TrajectoryOpt. And if we do add a base class, other things begin to not make sense. Also, if we pass in a vector of * ConstraintBuilders, how do we know what type of ProblemData they are associated with? The more I play around with it, the more problems I run into.
How do we rectify this? At the end of the day, all we need is a vector of ConstraintData and some metadata about the current phase, but the current approach seems somewhat bloated and overcomplicated.

As it turns out, if we just pass LeggedRobotData as the template argument, it makes perfect sense. See 1e9f2e04793ccc3a16bdc7bc723d90a60e57490d


Update dependency: kevm-pyk/src/kevm_pyk/kproj/plugin

@dwightguth any idea what's going on here? This is also blocking other PRs I guess now (eg. https://github.com/runtimeverification/evm-semantics/pull/2092). Maybe the two PRs need to be put together, since it's updating K and blockchain-k-plugin at the same time, and they need to sync to being updated to the same K....

I've cherry pickede the blockchain update to this PR: https://github.com/runtimeverification/evm-semantics/pull/2092, so this one likely can be closed if that one goes throug.h

I mean, this is kind of what I would expect to happen if you have two different submodules or pinned commits of the same repo in different parts of the recursive dependency tree of a single repo. So the solution is I guess, maybe, don't do that? Not immediately helpful for the situation you're in, I'm aware, but this type of dependency hell is going to happen under that particular scenario. If you don't want to commit to the invariant that any given repo only depends on one version of any given other repo, this is the sort of thing you'll have to work through.

Yeah, that's obvious. Thank you.
I meant more to ask, are there changes that need to happen (in the other PR), other than cherry picking the change to blockchain-k-plugin dependency over to there?
As far as fixing the dependency diamond issue, my personal opinion is that it's not worth it, since this particular problem has happened once in, how long? Maybe a year or two? But open to suggestions.

Changes subsumed here: https://github.com/runtimeverification/evm-semantics/pull/2092


I had to add "implementation("com.google.protobuf:protobuf-java:3.24.0")"

I use kotlin. I had to add line implementation("com.google.protobuf:protobuf-java:3.24.0") to my build.gradle.kts to make qdrant java-client works.
Now I have this 2 lines:
implementation("io.qdrant:client:1.9.1")
implementation("com.google.protobuf:protobuf-java:3.24.0")

Secund think, I had to use exacly version 3.24.0, witch is pointed in https://github.com/qdrant/java-client/blob/master/build.gradle#L81
Maybe it because kotlin, but I think it is worth to say it in documentation.

com.google.protobuf:protobuf-java:3.24.0 shouldn't be required.
Could you please describe more about the error/issue you ran into?

Thank you, slazu!
I encountered the same problem. I also use Kotlin in my project with the newest qdrant-Client (1.10.0).
I was trying to do a search:
val searchResults : List<ScoredPoint> =
        client.searchAsync(
            SearchPoints.newBuilder()
                .setCollectionName("my_dataset")
                .setLimit(3)
                .addAllVector(...)
                .setWithPayload(enable(true))
                .build()
        ).get()

But then encountered the following errors in my IDE:

Adding the protobuf-Dependency fixed the errors for me.

I expected it to be bundled with the project dependencies.
It seems like, it might be useful to add this as a side note for users.
Thanks

I expected it to be bundled with the project dependencies. It seems like, it might be useful to add this as a side note for users. Thanks

As far as I know you would have to change the way your dependencies are added in your gradle.build, if you want to have these dependencies available transitively.
The libraries dependencies are defined this way:
dependencies {
	errorprone "com.uber.nullaway:nullaway:0.10.18"

	implementation "io.grpc:grpc-protobuf:${grpcVersion}"
	implementation "io.grpc:grpc-services:${grpcVersion}"
	implementation "io.grpc:grpc-stub:${grpcVersion}"
	implementation "com.google.guava:guava:30.1-jre"
	implementation "org.slf4j:slf4j-api:${slf4jVersion}"

        //...
}

note that they are imported using implementation "..." .
This means your libraries are available during development, but won't be automatically made accessible to a user of the library.
The issue is that the client libraries methods return Guava-Objects. So Guava-Classes are part of your public facing api and thus part of the library. E.g:
public ListenableFuture<Points.ScoredPoint>> searchAsync(Points.SearchPoints request) {
    // -> returns a Guava Listenable from com.google.common.util.concurrent
}

Gradle allows defining dependencies using api "..." which makes dependencies transitive:
dependencies {
	errorprone "com.uber.nullaway:nullaway:0.10.18"

	api "io.grpc:grpc-protobuf:${grpcVersion}"
	implementation "io.grpc:grpc-services:${grpcVersion}"
	implementation "io.grpc:grpc-stub:${grpcVersion}"
	api "com.google.guava:guava:30.1-jre"
	implementation "org.slf4j:slf4j-api:${slf4jVersion}"

        //...
}

A cleaner approach could be to abstract away third party dependencies on public facing api.
See: https://docs.gradle.org/current/userguide/java_library_plugin.html#sec:java_library_separation
Also note the explanation here: https://docs.gradle.org/current/userguide/java_library_plugin.html#sec:java_library_recognizing_dependencies

io.grpc:grpc-protobuf indeed needs to be included in your project.
OP mentioned about missing com.google.protobuf:protobuf-java, which for me is weird since I expected it to be available with the above.

Anyways, a note about these dependencies for consumers would be useful. Will do.
Thank you @slaszu and @Loahrs

io.grpc:grpc-protobuf indeed needs to be included in your project.
OP mentioned about missing com.google.protobuf:protobuf-java, which for me is weird since I expected it to be available with the above.

I also had to add the Guava library in my case. The missing com.google.protobuf:protobuf-java should be following the same logic as in my last comment - Because SearchPoints extends a Protobuf-Class but is part of the public API of this library.
But just adding a note about the needed dependencies might be a simpler thing to indeed :)

Using maven and java, I also have issues with class not accessible using the simple  example from the readme.
I do have the tensorflow java bindings as well, so maybe they case havoc as they also use grpc libs.

Resolved with https://github.com/qdrant/java-client#gradle


python3Packages.scrapy: 2.2.1 -> 2.3.0

Motivation for this change
https://docs.scrapy.org/en/2.3/news.html#scrapy-2-3-0-2020-08-04
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build python38Packages.itemloaders python38Packages.scrapy

@GrahamcOfBorg build python38Packages.itemloaders
@GrahamcOfBorg build python38Packages.scrapy


Documentation: sap.base.Log methods accept Error as second parameter

OpenUI5 version: 1.99.0
I'm using the ui5 typescript types for developing ui5 applications. That already works quite well, however I noticed that the different log methods fatal/error/warning/info/debug/trace in sap/base/Log currently have the following parameters documented
https://github.com/SAP/openui5/blob/e7a495d975fcd8e1105e3bfa79a60a6f157b18f4/src/sap.ui.core/src/sap/base/Log.js#L219-L222
while internally, the log() method also allows to give an Error as second parameter for sDetails, which is very useful to log error objects to the console.
https://github.com/SAP/openui5/blob/e7a495d975fcd8e1105e3bfa79a60a6f157b18f4/src/sap.ui.core/src/sap/base/Log.js#L383
As far as I know the typescript types are generated from the jsdocs. Could you adjust the jsdocs for the different log methods which are part of the public API so that Error objects are also allowed?

I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.

Fixed with a6970f7f3239172ebcbf077b62f2b3453d25555f .

Thanks @codeworrior!


docs(configuration): add a missing option

The output.globalObject is useful for UMD build, which should be documented.
Ref: https://github.com/webpack/webpack/issues/6522

@EugeneHlushko What's wrong with the Travis job?

@Munter can you check why svg throw content type mismatch?

@EugeneHlushko 	It's an undesirable side effect of a strict matching of content types based on what type of relations are pointing at an asset. I think a newer version of assetgraph (hyperlink dependency) has an improvement here.
I'm marking this specific error as TODO in the tests for now so they should pass until I can update hyperlink

Got it, thank you @Munter
I've also resolved the conflict and updated the branch

Thanks!


Missing details of permissions and class for database-scoped credentials

The documentation page for the sys.database_permissions view doesn't include any details for the "database scoped credential" object type introduced in SQL Server 2016/2017, sys.database_scoped_credentials
Could these details be added please? I can see from this page what possible permissions can be granted/denied over those objects (i.e. ALTER, CONTROL, REFERENCES, TAKE OWNERSHIP, VIEW DEFINITION), but I need to know what Class value is used to represent those permissions in the sys.database_permissions view for a report I'm writing, and I don't see a record of that anywhere online.
Kind regards,
Rob Ashton

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

ID: 78080f83-be60-8136-9a97-dee7cae49df4
Version Independent ID: 07cbb880-46a7-3da7-21ad-00b9de39eccb
Content: sys.database_permissions (Transact-SQL) - SQL Server
Content Source: docs/relational-databases/system-catalog-views/sys-database-permissions-transact-sql.md
Product: sql
Technology: system-objects
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@RobAshton-IM -- Rob, thank you for your feedback and question.
@VanMSFT -- Van, please look into this issue.

Hey @AndreasWolter - Can you please help with this?

Hi<EMAIL_ADDRESS>I think for Database Scoped Credential there is no dedicated permission
The Doc mentions “ALTER on the Credential”, but how could you have that?
For Create it is:
Requires CONTROL permission on the database.
I think the same applies to ALTER/DROP
And hence there is no special entry in permissions.
Is that right?
Thanks
Andreas
Andreas Wolter | Senior Program Manager
MCM, MCSM | Data Platform Security
Twitterhttps://twitter.com/AndreasWolter | Linkedinhttps://www.linkedin.com/in/andreaswolter/
From: Van To<EMAIL_ADDRESS>Sent: Wednesday, July 1, 2020 3:06 PM
To: MicrosoftDocs/sql-docs<EMAIL_ADDRESS>Cc: Andreas Wolter<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/sql-docs] Missing details of permissions and class for database-scoped credentials (#5075)
Hey @AndreasWolterhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAndreasWolter&data=02|01|andreas.wolter%40microsoft.com|8e4ed9b221934bbfb26308d81e0afd45|72f988bf86f141af91ab2d7cd011db47|1|0|637292379834206183&sdata=g14TO0QEgCXMgPR6XuxmGBTUAyQ3ta4576cgoJ2XkLI%3D&reserved=0 - Can you please help with this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fissues%2F5075%23issuecomment-652669344&data=02|01|andreas.wolter%40microsoft.com|8e4ed9b221934bbfb26308d81e0afd45|72f988bf86f141af91ab2d7cd011db47|1|0|637292379834216133&sdata=okT6q9ZpkS9n4HIctHEAPi1wpyuCRXaUB7IhR2Hyz7k%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALGUO7CM2P7QAQEYOHPQNKLRZOXN3ANCNFSM4ONOA2NA&data=02|01|andreas.wolter%40microsoft.com|8e4ed9b221934bbfb26308d81e0afd45|72f988bf86f141af91ab2d7cd011db47|1|0|637292379834216133&sdata=sVubNgAFTvG1%2BtPh6JJIIl0wNnEE9NHyNM4OGe4A7yE%3D&reserved=0.

Hi Andreas, while DSC permission inheritance worked the way you describe in SQL Server 2016 when they were first introduced, the latest version of the SQL Server permissions poster shows that ALTER, CONTROL, REFERENCES, TAKE OWNERSHIP, and VIEW DEFINITION permissions were added to database scoped credentials in SQL Server 2017 onwards and Azure SQL Database at the same time.
Unless DSC's have a fundamentally different way of associating permissions, grantee, grantor, objects and state (DENY/GRANT/GRANT WITH GRANT) together, I assume they are represented in sys.database_permissions with a specific Class value, however I currently don't have access to an example to confirm that myself.
Kind regards,
Rob Ashton

Also, while we're at it... :)
Could the same question be asked of Full Text Search Property Lists, or does that want raising as a separate issue?
Kind regards,
Rob Ashton

Hi @RobAshton-IM - Sorry for the delay. This fell off my radar and I apologize. Let me follow-up internally.
#reassign:AndreasWolter

Checking back with Andreas

Revisiting with Andreas

We're still following up on this.

Hi @RobAshton-IM, the class is listed on the sys.database_permissions article (32), so I'm closing this issue.


updated ssi service playground image

updated the SSI service playground image. Checked in beforehand and should be ok, but LMK if you have any questions.

LGTM!

thank you!


Added delete method, modified post to return a response

Added JoinRoom and LeaveRoom based on current api. ref #7, #8
Note that this is a breaking change to go-gitter, necessary to properly make api calls.

@anchepiece , thanks a lot! very nice update.


System-test: Temporarily disable 030-run

While investigating issue
https://github.com/containers/libpod/issues/4044 there is no sense
subjecting forward progress elsewhere.  Skip the test with a note
temporarily, until a resolution to 4044 and any other related issues
is found and fix implemented.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

Opened at the request of @edsantiago as he's headed off on PTO for several days.


Replace Multiselect (UISelectMany) with auto-suggestion box

It would be nice if UISelectMany fields could be rendered as the Dependencies input in http://wildfly-swarm.io/generator

The auto-suggestion can happen locally as all the allowed values are already returned in the json as "valueChoices"


Group configurations

Currently, in ConfigDef in GcsSinkConfig, configurations aren't grouped, however ConfigDef provides a way to do this. When the configuration info is exposed outside, having these groups will be beneficial, for example, for UIs that automatically layout configurations by tabs.

Done by https://github.com/aiven/aiven-kafka-connect-gcs/pull/7


2FA for web UI login

Is your feature request related to a problem? Please describe.
I like to access the dashboard from outside my home network, so I think a 2FA is a good way of a little extra security.
Describe the solution you'd like
2FA like Google Authenticator or 1Password 2FA token.

For later Alex when he circles back around to this https://github.com/webbingbrasil/filament-2fa

This also could be very useful https://github.com/Laragear/WebAuthn

Something else in the same vain that would be useful is integrations with SSO solutions like Authentik

Hello guys, just an idea in addition to the 2FA request. Would there also be a possibility of some IP based auto-lock in case of failed logins in a certain (short) period of time, to lock-out any brute-force attempts?


allocate training data arrays in advance

At the moment, the arrays are copied when concatenating generator and target datasets. Allocating in advance should halve the peak memory use.

Could use a numpy.memmap for the preallocated array, which should we always fit into memory.

Closed by #192.


How can I install previous nni version with pip command

Hello, guys.
I'd like to know how can I install previous nni version with pip command.
For example, I'd like to install nni v1.7 for my class instruction.
Thank you for your reading

Yes, you can use pip install nni==1.7 --user

Thank you for your reply.


Replace set-output

This PR follows a recommendation by GitHub to change the way outputs are set in GitHub Actions and Workflows.
The old way of setting the outputs will become unsupported in 2023.

@rossfarrugia Shall we use the standard admiralci workflows for this repo also?

@cicdguy not all are relevant but they do use some, see https://github.com/pharmaverse/datacutr/blob/devel/.github/workflows/common.yml

As agreed with @cicdguy: closing this PR and opening another one that will be more in line with the standard admiralci workflows.


Allow specifying custom resources

I would like to use infracost with resources outside of AWS, Azure, GCP.
For that, it would be great to be able to specify custom resources along with their prices.

@dhilgarth can you possibly share some details about what types of custom resources you're referring to?

Sure, I'm thinking about resources from other providers, e.g. Hetzner, OVH, OTC, OCI...
I'm thinking about both virtual servers as well as co-located physical servers, but also about their other resources like load balancers, volume, storage boxes etc. etc

Thanks @dhilgarth - that's helpful. Longer term, we would like to move towards a provider model where community-supported cloud vendors could also be added (similar to Terraform providers). That is the underlying work we have to do before things like this can be supported.

Yes, that sounds exactly like what I was looking for.
Is this already part of a milestone plan?

Great - thanks for confirming. When we have a roadmap/ETA we will post here for visibility (we're currently focused on AWS/Azure/GCP).


Do not override SIGINT globally

Importing this library overrides (or at least tries) the default signal handler for the UNIX signal SIGINT.
This is not just a general bad practice but breaks compatibility with various frameworks, and causes the writing of complex applications that uses this library very hard.
A few examples where this behavior of the library do cause problems:

Using any framework, that loads modules on a separate thread (i.e.: Flask application with Werkzeug as WSGI runner). Starting the application (or serving the first request) causes the following exception: ValueError: signal only works in main thread
Any code that builds upon catching KeyboardInterrupt breaks as no such exceptions would be raised. Instead sys.exit(0) is called which causes an unclean exit. This behavior requires special hacks to work around (using __exit__ method of objects, etc.) which are not nice nor reliable.

These issues are caused by the first few lines of audioTrainTest.py: https://github.com/tyiannak/pyAudioAnalysis/blob/32abdd748d9a26f1fcf5ee58860c0eb7b45df1ec/pyAudioAnalysis/audioTrainTest.py#L22-L27
I believe a library should never attempt tho handle signals. This is entirely the responsibility of the main application. Thus that signal handler should be removed at least from the beginning of the file.
If this functionality is required when individual functions run at command line, then I suggest to only override the signal, when this code is run manually at the command line (e.g.: using __name__). So that it won't break code it's imported in.
Soon I'll post a pull request that I think fixes this issue.

What you say makes perfect sense, to be honest, I dont remember how this signal handler was kept in audioTrainTest, i guess it should be from some very old version that included realtime audio recording. Anyway, you are absolutely true and thank you for the comment. I am testing the PR and approving in a while


Custom spawners and spawn-eggs

Maybe a function can be added, so that we can get the spawn egg or spawner of the specified monster through the instruction?

You can use /dcaves summon mob-type command. Just copy type from config, like dead-miner.
If you want a real spawner or spawn-egg.. I will think about it ;)

alright, thank you very much！

Hello, I love your plugin, it lacks especially the ability to spawn with your mobs. Because on one of the worlds I don't want to activate all your mobs but just some in certain places.
Another option that would be interesting would be to be able to choose the possible mobs per world.


contributors from the repo are not showing up in the front end

https://opencollective.com/vuejs#about we fetch them and store the data still but we are not showing it.

We stopped showing them in the new frontend.
Do we want to bring them back? If so, would love to have @cuiki take a stab at it.

was it too complicated or was it slowing the page too much? why did we stop?


Removed unused loopback client source

With the removal of "local" mode, this file is no longer required.
@benbjohnson

This file is already chopped in https://github.com/influxdb/influxdb/pull/1295. Closing to avoid merge.


Add international children's day to thuringia in germany

Hi @Tracer1337 , thanks for your PR and sorry for the long delay.
I'll happily merge the PR and improve it slightly afterwards - mostly that the day has only been a holiday since 2019.


Mic Disconnection Issues

Mic keeps disconnecting while using Pi-Home.
Possible Solutions

Mic Sensitivity https://developers.google.com/assistant/sdk/prototype/getting-started-pi-python/configure-audio
Card Definitions https://developers.google.com/assistant/sdk/prototype/getting-started-pi-python/configure-audio
Hardware issues with Mic

Fixed by altering the order of which the cli commands are executed


Update README.md

Hi I Done A Minor Change For The Hacktoberfest Prize. Please Accept It

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hi, @chiragj-coder the Hacktoberfest guidelines ask that contributors don't create:

Something that's clearly an attempt to simply +1 your pull request count for October.


Vector Clustering

@gberaudo I saw in these slides  that you mentioned clustering. I am currently exploring possibilities for clustering in Ol3 and Cesium. Currently for the Cesium I came across this which is quite slow because of all the pickPosition() calls which is quite expensive.. I was thinking of using Cesium.SceneTransforms.wgs84ToWindowCoordinates() instead since it is much faster.
Anyway, could you elaborate a little on your approach / views / experiences ?
Lucas
BTW. I couldnt find a google groups or similar for ol3-cesium to discuss these kind of things so ended up opening an issue..

My approach is to precompute the clusters and have the GPU shader handle the filtering about what to show according to the distance of each point to the camera.
For this I use:

the ol3-cluster-tool to add clustering information to a geojson;
some small patches to Cesium see https://github.com/camptocamp/cesium/tree/c2c_patches / e2ddd8d650a3b5f7cc2aba78723f5a43ddae52f3 90cd1c236c61cda1eeb9ee68e144b908fa509ea6 26c012b2ba17d6d6954826799f1101b862b3d747.

It is used in https://map.schweizmobil.ch/?lang=fr&bgLayer=pk&resolution=250&X=631750&Y=189000&layers=Station (click 3D).
Note that clicking a clustered point on the map is handled by a graph traversal on the cluster metadata on the client side.
I think it is a good approach, wihch work well for a reasonable number of points.
Compared to what was done for Schweizmobil, it could be improved in several ways:

use a better compressed format than geojson to store the points and metadata;
use the clusters also for 2D;
don't block Cesium when loading cluster data;
add the tool to OL3-Cesium base code so that it is more straightforward to use it.

Out of curiosity, what is the number of points you want to handle?

Sounds interesting. I don't exactly know yet how many points we have for clustering but I guess quite similar to the schweizmobil app.
The schweizmobil app is quite interesting by the way. Do you know how the terrain was created? Did they have their own .terrain writer or did they buy something from AGI ?

The data comes from Swisstopo as visible in the attribution section at the bottom of the page.
See https://github.com/geoadmin/3d-forge.

Hi there is also a pypi module just for the encode decode and data structure part.
You can find it here.
https://github.com/loicgasser/quantized-mesh-tile
3d forge has not been updated with the module yet

Great, thnx a lot!


Create version 2

I have a number of ideas for version 2 of the plugin. I feel there are a number of things that need to be improved. In order to do this, version 2 would not be backward compatible with version 1. Some things that need addressing:

change data-content and binding to the innerHTML using the word content. This prevents biding to the attribute "content" currently
change the data-something tags to be data-template-something tags, keeping everything under the data-template structure, thus avoiding potential conflicts
simplify the notation somehow. Currently data-template-bind can get rather long and complicated. Is there a better way to do this?
look at the possibility of introducing a watch on the data and updating the DOM
potentially include conditional formatting, to render template only if a condition is met

Any other ideas welcome for version 2 of the plugin, or advice around whether this is a good idea.

I would suggest adding the possibility to bind to more than a key.
I actually just figured out how to do this with a template formatter but I find it a bit unelegant.
"look at the possibility of introducing a watch on the data and updating the DOM"... maybe also some way to define a function to call so that changing data client side not only update the DOM but also the remote source of data?

Did you see way.js? https://github.com/gwendall/way.js?files=1
This is similar to your approach but has the two-way binding, maybe it's worth to combine efforts?
Way.js misses your flexibility, for example there's no way to control classes via a template.
Also you can take a look at Meteor autorun library, that can easily power that can power the data binding and observation. The challenge might be the "rerender" overhead - we should rerender only the specific DOM part when needed, not the whole list.

Thanks, I will check this out. I plan to start work on version 2 sometime soon and I am open to all ideas regarding the way forward and am more than happy to work with other contributors and libraries so please do let me know about things like this.


Activity Steams Schemas are Messy

missing titles
missing descriptions
circular references (making @ankaur's life difficult)

@majindal can you help improving this?

@theManikJindal once you've done this, I'm happy to take a PR that rejects PRs without examples

@trieloff I am just on the verge of raising a PR that fixes this issue. However, there are some places where the tooling is failing.
For example, if one defines a schema having an array at the root, the tooling throws an error. Tooling wants only objects to be at the root level.
Example schema:
{
    "type": "array",
}

Example instance:
["fails", "to", "validate", "this"]

Change the meta schema to allow arrays.


Removed the all the usage of recordName

The #705 asks to remove ALL the instances of the recordName annotations as it is unused.
fixes: #705
/claim #705
If you want to USE it specifically instead of removing it. Please specify that while closing the PR

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

I think the $250 bounty I was thinking it would be used in all the codecs, not deleted. The codecs are just ignoring it right now.


fix: update the worker script

Fixes #66

Attempts to resolve the following runtime error that, supposedly, happens when fetching the worker script:
A bad HTTP response code (404) was received when fetching the script.

To start, I'm updating the worker script to be aligned with the latest installed msw. That shouldn't affect how the script is fetched in any way, though.

The worker script is never present in the storybook-static dir, which is the server app's root. We need to respect NextJS's public directory during the build.

The build-storybook command we were using was referring to the build-storybook binary. Since we called it without any options in the root-level yarn build, it never copied the Next's public dir to its static dir.
Fixing this to refer to our own custom storybook-build command.


feat: add keymaps for floating window

closes #172
I am up for discussion on how to call these settings and where to put them
fyi I tested the changes locally
Another approach would be to architecture settings like this:
win_mappings = {
	tags = {},
	scopes = {},
	loaded_scopes = {},
}

Not really sure which one's betteer

@cbochs friendly bump :))

@Tronikelis apologies for the late reply. I was away on vacation for the past several weeks. Will take a look here soon 🙂

@cbochs I've changed the names based on your comments, please check if I got everything right

@cbochs could we get this merged?

@cbochs 🥲


Per-project virtualenv

In readme written «Goes along really nicely with the autocomplete-python package to provide a full Python IDE experience in Atom.» But autocomplete-python support per-project virtualenv settings in format:
$PROJECT/.env/lib/python2.7/site-packages;

Where $PROJECT is placeholder for project's folder path. Seems like python-tools not used this settings and not support same format in own settings (only full qualified path, but it's not useful when you have ~50 projects with own virtualenvs). So «Go to definition» not work me at all :(

Hi @qnub sorry for the very late reply, but I must have missed my email with this notification for some reason. I'll look into this as soon as I can. Should not be too hard to implement.

It'll be great, thank you!

great idea.

I try change "path to python directory" to $PROJECT/env/lib/python2.7/bin . It's seem work fine.

Can I chose remote python (vagrant or Docker) for go-to-definition?

Also interesting in using docker

+1


Image

How can I add images to the product? Please provide source code.

Hi @willbun244,  this project is meant to provide the base only, to add image for product, you need to import this android-shoppingcart project and extend Product class. Would you like to do that on android-shoppingcart-demo (https://github.com/tonyvu2014/android-shoppingcart-demo) project and make a pull request? I am happy to merge. Thanks.

Yes of course! Ok, I will.

product images have been added to demo project


How to deal the case that the solver inputs and outputs tensors with different shapes?

How to deal the case that the solver inputs and outputs tensors with different shapes?
Suppose the solver's output shape is different from the shape of its input, which leads to grad_output_numpy's shape is different from ctx.weights, and then the following code will report error due to shape conflict in the addtion operator.
weights_prime = np.maximum(ctx.weights + ctx.lambda_val * grad_output_numpy, 0.0)
In https://github.com/martius-lab/blackbox-backprop/blob/master/blackbox_backprop/shortest_path.py#L93
This case may happen frequently, for example, the solver is a image classifier, input is 224x224x3 image and output is a classification probability vector.

The blackbox solver is defined as:

Indeed, an image classifier is not an example of a combinatorial solver.
The solver really needs to be a minimizer of an inner-product objective, as it is common in combinatorics.
Closing this.


add bloom filter implementation based on split block (sbbf) spec

Which issue does this PR close?

Part I of:

#3023

Next up:

read block data from column chunk metadata
setup reader to filter by this bloom filter

This implementation is partly based on https://github.com/jorgecarleitao/parquet2/tree/main/src/bloom_filter on the testing data part.
Rationale for this change
add bloom filter implementation based on split block spec

What changes are included in this PR?
SBBF implementation per spec

Are there any user-facing changes?

turns out i had to make the sbbf module public to make the clippy happy but either way i am going to make use of it soon later and then maybe change it to crate private?
one thing that has been bothering me is the fact that down the lane, RowGroupReader, PageReader, etc. all make use of the trait ChunkReader which requires both an offset and a length to do actual IO. For bloom filter it is different: it only contains an offset, then the length of its record is read out from the BloomFilterHeader, with only an offset, there's no way to read with ChunkReader.
I'd probably need to setup a separate trait for this purpose...

it only contains an offset, then the length of its record is read out from the BloomFilterHeader, with only an offset, there's no way to read with ChunkReader.

We may have to do something similar to what we do for the footer, guess what the length is, and then read additional data if necessary

it only contains an offset, then the length of its record is read out from the BloomFilterHeader, with only an offset, there's no way to read with ChunkReader.

We may have to do something similar to what we do for the footer, guess what the length is, and then read additional data if necessary

thanks, do happen to know the link to that example?

i intend to merge this one first before moving on to the part of parsing bitset from parquet file

I will try and review this PR later today. Thank you @Jimexist

I started (but did not finish) reviewing this PR, thank you. I need to find some dedicated time to study the bloom filter spec in more detail
So far, my analysis is that twox-hash seems reasonable -- it is widely used https://crates.io/crates/twox-hash/reverse_dependencies

@viirya  or @tustvold  any concerns about merging this?

let me try to merge this and address the issues in a subsequent pull request


Taking too long to close the node

Describe the bug
While updating the node, takes a lot of time close the node.
To Reproduce
Steps to reproduce the behavior:

Run the node
Press Ctrl+C

Expected behavior
Close the node as fast as possible with a grateful shut down.
Mon Sep  5 16:53:12 2022        INFO    apiCollector    Collector Started
^CMon Sep  5 16:53:23 2022      INFO    Shutting down Juno...
 Mon Sep  5 16:54:30 2022       INFO    apiCollector    StateUpdate collected   {"BlockNumber": 0}
Mon Sep  5 16:57:31 2022        INFO    Sync Service    Updated block info      {"Block Number": 0}
Mon Sep  5 16:57:31 2022        INFO    Shutting down JSON-RPC server...

3 min to close the node

I have seen this as well. It may be because MDBX is writing the update to disk and needs to complete the transaction when we call Close. If this is the case, then making the database faster is the best solution. A temporary solution may be to cancel current transactions on Close.


This function will generate a random number of cnh

CNH's valid throughout the national territory

Hey guys, I'm need this function in my project...
Please review as soon as possible because other companies in my group need the resource
Thank's


GPL-289 - Hi-C library type - Sequencescape half

Make the Hi-C Library Type available to 'Limber Bespoke PCR' Request Types

previously, Hi-C libraries were only received pre-prepared by faculty; now, they are going to be prepared by the Bespoke team in a new pipeline

Also ran this code, to avoid validation errors on submission upload:
lib = LibraryType.find_by(name: 'Hi-C')
rt = RequestType.find_by(name: 'Illumina-B HiSeq X Paired end sequencing')
rt.library_types << lib


WebGL usingThreads error converting draco mesh

When I try to convert a Draco to a unity mesh on a WebGL build that has threads enabled, I am faced with this error:
UnityException: BuiltinUpdate can only be called from the main thread.
Constructors and field initializers will be executed from the loading thread when loading a scene.
Don't use this function in the constructor or field initializers, instead move initialization code to the Awake or Start function.
  at UnityEngine.Experimental.Rendering.BuiltinRuntimeReflectionSystem.BuiltinUpdate () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.Experimental.Rendering.BuiltinRuntimeReflectionSystem.TickRealtimeProbes () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.Experimental.Rendering.ScriptableRuntimeReflectionSystemWrapper.Internal_ScriptableRuntimeReflectionSystemWrapper_TickRealtimeProbes (System.Boolean& result) [0x00000] in <00000000000000000000000000000000>:0

(Filename: currently not available on il2cpp Line: -1)

However if I disable threads for the WebGL build it works, but the there is a visible stutter when converting the mesh.

Unity 2020.1.0
Draco 3D Data Compression 3.03
Windows 10

To Configure Threads on WebGL I Created a Editor folder containing this script:
using UnityEditor;

[InitializeOnLoad]
public class EnableThreading
{
    static EnableThreading()
    {
        PlayerSettings.WebGL.linkerTarget = WebGLLinkerTarget.Wasm;
        PlayerSettings.WebGL.emscriptenArgs = "-s ALLOW_MEMORY_GROWTH=1";
        PlayerSettings.WebGL.threadsSupport = true;
        PlayerSettings.WebGL.memorySize = 2032;
        PlayerSettings.WebGL.exceptionSupport = WebGLExceptionSupport.FullWithStacktrace;
    }
}

And this is my script:
(error occurs at: var mesh = await draco.ConvertDracoMeshToUnity(data);)
using Draco;
using System.IO;
using System.Threading.Tasks;
using UnityEngine;
using UnityEngine.Networking;

[RequireComponent(typeof(MeshFilter))]
[RequireComponent(typeof(MeshRenderer))]
public class ThreadingTests : MonoBehaviour
{
    public string filePath;
    async Task Start()
    {
        var fullPath = Path.Combine(Application.streamingAssetsPath, filePath);
        Debug.Log(fullPath);
#if !UNITY_EDITOR
        var request = UnityWebRequest.Get(fullPath);
        request.SendWebRequest();
        while (!request.isDone)
        {
            await Task.Yield();
        }

        var data = request.downloadHandler.data;
#else
        var data = File.ReadAllBytes(fullPath);
#endif
        // Convert data to Unity mesh
        var draco = new DracoMeshLoader();
        // Async decoding has to start on the main thread and spawns multiple C# jobs.
        var mesh = await draco.ConvertDracoMeshToUnity(data);
        if (mesh != null)
        {
            // Use the resulting mesh
            GetComponent<MeshFilter>().mesh = mesh;
        }
    }
}

I followed this guide to setup my local server for the WebGL build:
https://medium.com/medialesson/so-you-want-to-use-multithreading-in-unity-webgl-5953769dd337#7ca7

Hi @laurentprevu ,
I never tried that, but it seems like you have got to make sure somehow, that your Start Task runs on the main thread.
The topic is very interesting, but I cannot offer support for an unsupported feature at the moment. I hope threading in Web will become a thing soon.
Regards

Hey @atteneder
Actually the Start Task is just the regular Start method with a different signature, so I would imagine that it runs on the main thread. I've gotten a new error since yesterday and it seems like the current Job system doesn't cooperate very well with threading activated on WebGL.
Anyway I understand that supporting this feature at the moment is not possible, but thanks for answering. I will continue investigating on my end.
Sincerely
ScheduleBatchedJobsAndIsCompleted can only be called from the main thread.
Constructors and field initializers will be executed from the loading thread when loading a scene.
Don't use this function in the constructor or field initializers, instead move initialization code to the Awake or Start function.
  at Unity.Jobs.JobHandle.ScheduleBatchedJobsAndIsCompleted (Unity.Jobs.JobHandle& job) [0x00000] in <00000000000000000000000000000000>:0
  at Unity.Jobs.JobHandle.get_IsCompleted () [0x00000] in <00000000000000000000000000000000>:0
  at Draco.DracoMeshLoader+<WaitForJobHandle>d__7.MoveNext () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.InvokeMoveNext (System.Object stateMachine) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ContextCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.Run () [0x00000] in <00000000000000000000000000000000>:0
  at System.Action.Invoke () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.YieldAwaitable+YieldAwaiter.RunAction (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.SendOrPostCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext+WorkRequest.Invoke () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext.Exec () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext.ExecuteTasks () [0x00000] in <00000000000000000000000000000000>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in <00000000000000000000000000000000>:0
  at Draco.DracoMeshLoader+<ConvertDracoMeshToUnity>d__6.MoveNext () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.InvokeMoveNext (System.Object stateMachine) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ContextCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.Run () [0x00000] in <00000000000000000000000000000000>:0
  at System.Action.Invoke () [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.AwaitTaskContinuation.InvokeAction (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ContextCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.AwaitTaskContinuation.RunCallback (System.Threading.ContextCallback callback, System.Object state, System.Threading.Tasks.Task& currentTask) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.SynchronizationContextAwaitTaskContinuation.Run (System.Threading.Tasks.Task task, System.Boolean canInlineContinuationTask) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.Task.FinishContinuations () [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.Task.FinishStageThree () [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.Task.FinishStageTwo () [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.Task.Finish (System.Boolean bUserDelegateExecuted) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.Tasks.Task`1[TResult].TrySetException (System.Object exceptionObject) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1[TResult].SetException (System.Exception exception) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncTaskMethodBuilder.SetException (System.Exception exception) [0x00000] in <00000000000000000000000000000000>:0
  at Draco.DracoMeshLoader+<WaitForJobHandle>d__7.MoveNext () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.InvokeMoveNext (System.Object stateMachine) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ContextCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.AsyncMethodBuilderCore+MoveNextRunner.Run () [0x00000] in <00000000000000000000000000000000>:0
  at System.Action.Invoke () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.YieldAwaitable+YieldAwaiter.RunAction (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at System.Threading.SendOrPostCallback.Invoke (System.Object state) [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext+WorkRequest.Invoke () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext.Exec () [0x00000] in <00000000000000000000000000000000>:0
  at UnityEngine.UnitySynchronizationContext.ExecuteTasks () [0x00000] in <00000000000000000000000000000000>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00000] in <00000000000000000000000000000000>:0
  at System.Runtime.CompilerServices.TaskAwaiter`1[TResult

Actually the Start Task is just the regular Start method with a different signature, so I would imagine that it runs on the main thread.

Me too, but I wasn't so sure anymore after reading your report :)
Anyways, hope you succeed!


my pulse detect 200 more bpm

can you help me why my pulse detect more value, i didnt touch sensor, and if i touch the bpm isnt stable, im confuse for that, im use 4.7k resistor and 10k resistor and 10mf 32v kapasitor for reduce the noise, it cant help anything -_-

Hello lukaskris,
I need more information to help you. Can you please send a photo of your setup?
What OS are you using? What arduino board are you using? what code are you using?

i use arduino uno, im use windows 10, i use code pulse sensor amped with update HRV
this is my screenshoot visualize in processing

Hi,
It does look like something has changed. What did you do besides add the blue '0 HRV'?
Also, did you do anything to the Arduino code? It looks like from the picture that your data stream is not updating regularly.

#include <LiquidCrystal_I2C.h>
/*  Pulse Sensor Amped 1.4    by Joel Murphy and Yury Gitman   http://www.pulsesensor.com
----------------------  Notes ----------------------  ----------------------
This code:

Blinks an LED to User's Live Heartbeat   PIN 13
Fades an LED to User's Live HeartBeat
Determines BPM
Prints All of the Above to Serial
Read Me:
https://github.com/WorldFamousElectronics/PulseSensor_Amped_Arduino/blob/master/README.md
Updated with HRV display by Elliot Mebane, Roguish, Inc. Feb 2016
HRV requires 1 minute of data before it's accurate.

*/
LiquidCrystal_I2C lcd(0x27, 16, 2);
//  Variables
int pulsePin = 0;                 // Pulse Sensor purple wire connected to analog pin 0
int blinkPin = 13;                // pin to blink led at each beat
int fadePin = 5;                  // pin to do fancy classy fading blink at each beat
int fadeRate = 0;                 // used to fade LED on with PWM on fadePin
// Volatile Variables, used in the interrupt service routine!
volatile int BPM;                   // int that holds raw Analog in 0. updated every 2mS
volatile int Signal;                // holds the incoming raw data
volatile int IBI = 600;             // int that holds the time interval between beats! Must be seeded!
volatile boolean Pulse = false;     // "True" when User's live heartbeat is detected. "False" when not a "live beat".
volatile boolean QS = false;        // becomes true when Arduoino finds a beat.
volatile int RRDelta = 0;
volatile double RMSSD = 0;
volatile double LN20RMSSD = 0;
volatile double SDNN = 0;
volatile double SDSD = 0;
volatile double NN50 = 0;
volatile double PNN50 = 0;
// Regards Serial OutPut  -- Set This Up to your needs
static boolean serialVisual = false;   // Set to 'false' by Default.  Re-set to 'true' to see Arduino Serial Monitor ASCII Visual Pulse
void setup(){
pinMode(blinkPin,OUTPUT);         // pin that will blink to your heartbeat!
pinMode(fadePin,OUTPUT);          // pin that will fade to your heartbeat!
Serial.begin(115200);             // we agree to talk fast!
interruptSetup();                 // sets up to read Pulse Sensor signal every 2mS
lcd.begin();
// IF YOU ARE POWERING The Pulse Sensor AT VOLTAGE LESS THAN THE BOARD VOLTAGE,
// UN-COMMENT THE NEXT LINE AND APPLY THAT VOLTAGE TO THE A-REF PIN
//   analogReference(EXTERNAL);
}
//  Where the Magic Happens
void loop(){
serialOutput() ;

if (QS == true){     // A Heartbeat Was Found
// BPM and IBI have been Determined
// Quantified Self "QS" true when arduino finds a heartbeat
fadeRate = 255;         // Makes the LED Fade Effect Happen
// Set 'fadeRate' Variable to 255 to fade LED with pulse
serialOutputWhenBeatHappens();   // A Beat Happened, Output that to serial.
lcd.clear();
lcd.print(BPM);
QS = false;                      // reset the Quantified Self flag for next time
}
ledFadeToBeat();                      // Makes the LED Fade Effect Happen
delay(500);                             //  take a break
}
void ledFadeToBeat(){
fadeRate -= 15;                         //  set LED fade value
fadeRate = constrain(fadeRate,0,255);   //  keep LED fade value from going into negative numbers!
analogWrite(fadePin,fadeRate);          //  fade LED
}

i just download and uploading into my arduino
i didnt know what the problem
1 problem sensor always output 200 more
and 2 the output when i touch the sensor if my ESP8266 is on is bad, better if the ESP8266 off

thanks it work well,
why the delay set 20? and 500 make big problem?
yes i use 4rd module, pulse+esp8266+sd+lcd
i want create heart tracker with android,
this HRV just stay 0??

I would need to see what your processing sketch is doing to know why HRV is 0.
don't use any delay in the loop longer than 20, or else the loop will not see the heartbeats.

if i use RTC DS1302 for get the real time, can i use millis() for get delay to output in my lcd screen?

You can use the Uno board millis() to time your LCD writes if you need to. You don't really need an RTC for that.
If you have your computer plugged into AC power, you can unplug it  and see a much cleaner signal...

i use my laptop, and my laptop still connected with the adaptor, but my pulse hrv still 0 point,
so if i want real time, where i must get it? from esp8266? or where?
thanks