Idea/Discussion: implicit decimal value on whole number returns type conflict error

the value type for a metric is stored as decimal (money 50.99). Sometimes the value to write will be a whole number (60).
Influxdb will respond with BadRequestError: write failed: field type conflict: input field "value" is type int64, already exists as type %!s(influxql.DataType=1)
In javascript, even if I explicitly set a number variable as 60.0, it is set as 60. I can easily enough convert toString / toFixed(2) on write, but I would hope that if the metric field type exists as decimal, the database would implicitly save a whole number (60) as (60.0). Thoughts?

i can completely feel the pain from the JS side of things.  if it's not a burden to do this within the db i agree it would be helpful.  maybe the parser for the line protocol could treat all numeric values as floats?  interacting more directly with the go api would still allow for other numeric types.

This would likely be mitigated by a fix for #3006, but that's not going to happen in the short term.
In the short term, perhaps we can automatically cast incoming int64 as float64 if the underlying target field value is already cast as a float64. The only downside is int64s that exceed float64s capacity, but that's an unlikely edge case.
@pauldix @corylanou @benbjohnson, thoughts?

Has there been any progress on this or #3006? Inquiring minds would love to know.

yeah, I think automatic casing should be done. However, we should catch an overflow error and return an error if that occurs.
This seems like a good usability enhancement so I've renamed the issue and assigned it to the 0.9.3 release.

However, I will say that this can create a problem if the first value you write is a whole number and future values are floats. That will result in an error.

Yes. Catching overflow errors on int64 -> float64 sounds reasonable. Why not truncate a float64 to an int64 if the series expects int64?

@j-rock what if the user expected to be writing floats, but the very first value was a whole number so it gets set to ints. From that point forward we'll be dropping precision while the user expects us to be keeping it.
This whole int/float thing is making me think that it might be a good idea to force everything to be a float like Prometheus does.

@pauldix presumably we would keep time as int64, and just reduce field values to boolean, float64, and string?
What's the math performance hit for doing floats vs ints?

@beckettsean not sure, would require testing. Although for it to really matter they'd have to be churning through millions of points.

What if choosing only [boolean, string, float64] as field values was a config parameter?

from a coding/support standpoint that won't make our job any easier. The goal with forcing float64 is that it makes everything simpler and makes it so the user is less likely to shoot themselves in the foot.

all numbers are now floats unless explicitly declared integers during writes (with a trailing i #3526). Closing as no longer relevant.


Security Issue

Please check out the report.
https://huntr.dev/bounties/09218d3f-1f6a-48ae-981c-85e86ad5ed8b/

hi, this issue is fixed, thanks for the report

Hello Peter. The commit you sent me does not seem to be the solution.
https://www.github.com/microweber/microweber/commit/957d307178eb73e13fcfdffb364cee32bc6f9928


set rviz_DEFAULT_PLUGIN_LIBRARIES during configure time

Attempt to fix catkin_make builds.

See: https://github.com/ros-planning/moveit_ros/issues/654#issuecomment-203407388


Publish image to Docker Hub

Push the docker image to Docker Hub for people to pull and use.

Added in https://github.com/tvvignesh/node-skeleton/pull/8
DockerHub URL: https://hub.docker.com/r/tvvignesh/node-skeleton/


[FIX] Fix handling of str vs bytes in WebClientCustom

odoo.addons.web.controllers.main.contac_xml explicitly returns
a str object, not bytes, hence we should call debrand
instead of debrand_bytes.

Fixes:
[...]
File "/opt/zelo/src/main/odoo/addons/base/ir/ir_http.py", line 208, in _dispatch
  result = request.dispatch()
File "/opt/zelo/src/main/odoo/http.py", line 826, in dispatch
  r = self._call_function(**self.params)
File "/opt/zelo/src/main/odoo/http.py", line 339, in _call_function
  return checked_call(self.db, *args, **kwargs)
File "/opt/zelo/src/main/odoo/service/model.py", line 97, in wrapper
  return f(dbname, *args, **kwargs)
File "/opt/zelo/src/main/odoo/http.py", line 332, in checked_call
  result = self.endpoint(*a, **kw)
File "/opt/zelo/src/main/odoo/http.py", line 933, in __call__
  return self.method(*args, **kw)
File "/opt/zelo/src/main/odoo/http.py", line 512, in response_wrap
  response = f(*args, **kw)
File "/opt/zelo/src/addons/misc-addons/web_debranding/controllers/main.py", line 77, in qweb
  content = debrand_bytes(request.session.db and request.env or None, content)
File "/opt/zelo/src/addons/misc-addons/web_debranding/models/ir_translation.py", line 51, in debrand_bytes
 return bytes(debrand(env, source.decode('utf-8')), 'utf-8')
AttributeError: 'str' object has no attribute 'decode'

It makes sense, but I cannot reproduce the error. Do know your steps? What is your Accept-Language header value?

I mean I cannot reproduce the error on current version without your fix

It looks like there is confusion with tostring method. It actually returns bytes. I still wonder how you get the error. Are you sure that you use python3?

In [6]: from xml.etree import ElementTree

In [7]: root = ElementTree.Element('body')

In [8]: s = ElementTree.tostring(root, 'utf-8')

In [9]: type(s)
Out[9]: bytes

In [10]: s.decode
Out[10]: <function bytes.decode>

A bit of investigation log:
Running on Python 3.5.3 (default, Jan 19 2017, 14:11:04), debian stretch.
Accept-language: en-US;q=0.5
With the code you posted I cannot reproduce the problem either, somehow that is not what is really going on, since just loading the login page i find [1] in the log.
Sneaking in some temporary inspection logging, around line 75, like:
        logging.error("FILES: "+str(files))
        content, checksum = controllers_main.concat_xml(files)
        logging.error("TYPE: "+str(type(content))+"  --  "+content)

After loading teh login page the log states:
2018-03-17 09:21:43,085 919 ERROR test_zelo root: FILES: []
2018-03-17 09:21:43,085 919 ERROR test_zelo root: TYPE: <class 'str'>  --

hence i'm led to believe a more appropriate test code would be:
In [1]: from odoo.addons.web.controllers import main as controllers_main
   ...: content, checksum = controllers_main.concat_xml([])
   ...:

In [2]: type(content)
Out[2]: str

which reproduces the problem.
Now, having a closer look at contac_xml, the problem is indeed at line 202, which gets executed when files is an empty list:
return '', checksum.hexdigest()

which is indeed returning a string instead of bytes.
So you were right about being suspicios of my patch, the problem is not in web_debranding, and the tostring method of ElementTree became misleading
with the switch to python 3, since it's now returning bytes...
Thanks for the good hints!
[1]
``2018-03-17 09:00:27,504 353 ERROR test_zelo werkzeug: Error on request:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 205, in run_wsgi
execute(self.server.app)
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 193, in execute
application_iter = app(environ, start_response)
File "/opt/zelo/src/main/odoo/service/wsgi_server.py", line 166, in application
return application_unproxied(environ, start_response)
File "/opt/zelo/src/main/odoo/service/wsgi_server.py", line 154, in application_unproxied
result = handler(environ, start_response)
File "/opt/zelo/src/main/odoo/http.py", line 1304, in call
return self.dispatch(environ, start_response)
File "/opt/zelo/src/main/odoo/http.py", line 1278, in call
return self.app(environ, start_wrapped)
File "/usr/lib/python3/dist-packages/werkzeug/wsgi.py", line 599, in call
return self.app(environ, start_response)
File "/opt/zelo/src/main/odoo/http.py", line 1468, in dispatch
result = ir_http._dispatch()
File "/opt/zelo/src/main/addons/auth_signup/models/ir_http.py", line 19, in _dispatch
return super(Http, cls)._dispatch()
File "/opt/zelo/src/main/addons/web_editor/models/ir_http.py", line 21, in _dispatch
return super(IrHttp, cls)._dispatch()
File "/opt/zelo/src/main/addons/utm/models/ir_http.py", line 20, in _dispatch
response = super(IrHttp, cls)._dispatch()
File "/opt/zelo/src/main/odoo/addons/base/ir/ir_http.py", line 212, in _dispatch
return cls._handle_exception(e)
File "/opt/zelo/src/main/odoo/addons/base/ir/ir_http.py", line 182, in _handle_exception
return request._handle_exception(exception)
File "/opt/zelo/src/main/odoo/http.py", line 767, in _handle_exception
return super(HttpRequest, self)._handle_exception(exception)
File "/opt/zelo/src/main/odoo/http.py", line 307, in _handle_exception
raise pycompat.reraise(type(exception), exception, sys.exc_info()[2])
File "/opt/zelo/src/main/odoo/tools/pycompat.py", line 87, in reraise
raise value
File "/opt/zelo/src/main/odoo/addons/base/ir/ir_http.py", line 208, in _dispatch
result = request.dispatch()
File "/opt/zelo/src/main/odoo/http.py", line 826, in dispatch
r = self._call_function(**self.params)
File "/opt/zelo/src/main/odoo/http.py", line 339, in _call_function
return checked_call(self.db, *args, **kwargs)
File "/opt/zelo/src/main/odoo/service/model.py", line 97, in wrapper
return f(dbname, *args, **kwargs)
File "/opt/zelo/src/main/odoo/http.py", line 332, in checked_call
result = self.endpoint(*a, **kw)
File "/opt/zelo/src/main/odoo/http.py", line 933, in call
return self.method(*args, **kw)
File "/opt/zelo/src/main/odoo/http.py", line 512, in response_wrap
response = f(*args, **kw)
File "/opt/zelo/src/addons/misc-addons/web_debranding/controllers/main.py", line 77, in qweb
content = debrand_bytes(request.session.db and request.env or None, content)
File "/opt/zelo/src/addons/misc-addons/web_debranding/models/ir_translation.py", line 51, in debrand_bytes
return bytes(debrand(env, source.decode('utf-8')), 'utf-8')
AttributeError: 'str' object has no attribute 'decode'
`

As I understand, we can close it.


chore(mitre azure): add mapping to mitre for azure provider

Context
We need to have Azure checks mapped to MITRE ATT&CK framework as we have for AWS
Description
Azure mitre att&ck mapping for Prowler
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Yep, good catch !

Is there any way to standardise the compliance model for all the providers?

I tried to do it, but implies changing the field Attributes[{AWSService}] to Attributes[{Service}] to match all the providers which could be a breaking change for some  user integrations ...


GitHub Actions fails to push hyp3-isce2 image to GHCR with 403 Forbidden

Jira: https://asfdaac.atlassian.net/browse/TOOL-2099

The dockerize job for https://github.com/ASFHyP3/hyp3-isce2/pull/119 is failing here with:
Error: buildx failed with: ERROR: failed to solve: failed to push ghcr.io/asfhyp3/hyp3-isce2:0.6.2.dev5_g3db5604: unexpected status: 403 Forbidden

Relevant links:

https://github.com/ASFHyP3/hyp3-isce2/blob/develop/.github/workflows/test-and-build.yml
https://github.com/ASFHyP3/actions/blob/develop/.github/workflows/reusable-docker-ghcr.yml
https://github.com/asfhyp3/hyp3-isce2/pkgs/container/hyp3-isce2

I wonder if it's related to the pytest coverage report failure here on the same PR:
Error: HttpError: Resource not accessible by integration

I've confirmed that the ASFHyP3 org has "Read and write permissions" under "Workflow permissions" for the GITHUB_TOKEN:

Workflows have read and write permissions in the repository for all scopes.

I've confirmed that https://github.com/orgs/asfhyp3/packages/container/hyp3-isce2/settings shows the hyp3-isce2 repo has Write access to the package.

Given that we authenticate as tools-bot here (gets used here), I checked https://github.com/orgs/asfhyp3/packages/container/hyp3-isce2/settings and noticed that tools-bot was not listed under "Manage access". So I added it with Write access, re-ran the workflow and...it still failed with the same error.

All checks passed for a dummy PR (just adds an empty foo file) https://github.com/ASFHyP3/hyp3-isce2/pull/121 so the failures must be related to the changes made as part of https://github.com/ASFHyP3/hyp3-isce2/pull/119, which just bumps all the ASFHyP3/actions versions from 0.8.0 to 0.8.1.

OK, so replacing GITHUB_TOKEN with TOOLS_BOT_PAK in .github/workflows/test-and-build.yml made all the checks pass for https://github.com/ASFHyP3/hyp3-isce2/pull/119. If possible I would like to understand why. And we'll also need to update https://github.com/ASFHyP3/actions/#reusable-docker-ghcryml to recommend using a personal access token.

I still don’t understand why using a personal access token works while GITHUB_TOKEN does not. Someone else had the same experience here although that doesn’t explain why it was only failing for hyp3-isce2 after the actions version bump.

I was suspicious that switching to TOOLS_BOT_PAK was actually what resolved the issue, so I opened https://github.com/ASFHyP3/hyp3-isce2/pull/123 to revert the change and...all checks passed. So maybe the issue just resolved itself? I removed package access for @forrestfwilliams and @tools-bot at https://github.com/orgs/asfhyp3/packages/container/hyp3-isce2/settings and re-ran the docker workflow to see if it still works.

So, to summarize, this issue has entirely resolved itself. No individual GitHub accounts have access under https://github.com/orgs/asfhyp3/packages/container/hyp3-isce2/settings. If this issue re-occurs, we should probably wait a few days for it to resolve itself.


Prepare for and conduct Knowledge Transfer session

Acceptance Criteria

Documentation re other data sources brought up to date
Decision log brought to date
Architectural design - all technologies added, e.g. eCharts ✅
If any areas of complexity in the code are not covered by the above then specific pages written
Meeting held with at least one mentor ✅

Update: We were not sure what to do for the first two points

Notes:
Documentation re other data sources brought up to date - seems to be all up to date as the only data sources we use are CAMs and OpenAq and we have docs for both.


libbsd: fix version script with lld 17+

Description of changes
Fixes #319975

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 319976 run on x86_64-linux 1

3 packages built:

libbsd
libbsd.dev (libbsd.dev.dev ,libbsd.dev.man)
libbsd.man (libbsd.man.dev ,libbsd.man.man)

Sweet, this is ready to merge then from the looks of it.


Fix issues + Support template literal types as discriminants

Fixes #46045.
Fixes #46125.

@typescript-bot test this
@typescript-bot user test this
@typescript-bot perf test this

Tests look clean, perf is slightly improved if anything. I think this one is good to go!

Great! But will buitin string methods support this? For now, I'm forced to use these standalone functions:
const startsWith = <T extends string>(str: string, search: T): str is `${T}${string}` => str.startsWith(search);
const endsWith = <T extends string>(str: string, search: T): str is `${string}${T}` => str.endsWith(search);
const includes = <T extends string>(str: string, search: T): str is `${string}${T}${string}` => str.includes(search);


fix(voice): send speaking packet on reconnect

Summary
AudioPlayer would previously not update the SPEAKING state to a non-zero value after reconnecting (which happens when moving between channels, for instance). This results in the bot continuing to send audio packets, which aren't processed by Discord as the speaking state would still be 0.
Fixes #845.
Checklist

[x] If code changes were made, then they have been tested

[x] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running task lint
[x] I have type-checked the code by running task pyright

[x] This PR fixes an issue
[ ] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Tested locally, unfortunately still ran into an unrecoverable 1006 with the following state:

The caught exception, 1006 which per the timestamps occurred as the bot stopped functioning,

Close code 1006 unfortunately doesn't provide much information, it's an error by the local websocket implementation and just means the connection was closed abnormally. :/
This could be anything from blocking code interfering with connections to miscellaneous network issues, though it should always try to reconnect automatically (also indicated by log messages, which might be helpful in debugging).

It looks like right after we connect to a voice channel we send a packet indicating we are not speaking, followed immediately by a speaking packet.

While it may seem odd, that's intended - clients are required to send at least one op5 packet to set the SSRC, the second one is when it actually starts sending audio data.


chore: add build instructions in readme

This would have saved me 20 mins :)
Cheers.

 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.

Thanks!


fix initialization in online pg3

we were accidentally starting the search for online PG3 with the old policy, which could use outdated NSRTs

don't you instead want to reset self._current_ldl = LiftedDecisionList([]) at the start of learn_ldl?
I considered that, but we do have a "current LDL" still technically, so I think that this is better

why would you have a current LDL if you've just called learn_ldl()?

you have it from the last learning cycle


Android platform view fallback to Virtual Display doesn't trigger when SurfaceView is added dynamically

This is split off from https://github.com/flutter/flutter/issues/103630 for detailed investigation.
According to https://github.com/flutter/flutter/issues/103630#issuecomment-1218077657, the fix for https://github.com/flutter/flutter/issues/103630 does not work for https://pub.dev/packages/dyte_client. The most likely explanation is that fallback to Virtual Display isn't triggering for the plugin, although it's possible that there's something else going on.
I tried to debug locally, but the example doesn't build out of the box, and appears to require access to a non-public SDK. We'll need a minimal repro case to investigate further.

@rlee1990 you mentioned that you've already reached out to the developers; if you could ask them to follow up here that would be great.

@stuartmorgan Thank you and will do

@stuartmorgan It appears that this is now the only one having an issue from the image below in the video view there are buttons at the bottom of the video that do no show

Thanks, I was able to reproduce with https://github.com/dyte-in/flutter-sample-app
The reason that https://github.com/flutter/flutter/issues/103630 didn't work for this plugin is that at view creation time, when the view hierarchy is scanned for SurfaceViews, there isn't one (it looks like it's just an empty container view: com.facebook.react.ReactRootView), so fallback to Virtual Display doesn't trigger. Then later, a SurfaceView is added, triggering the bug.
It might be possible to monitor the view hierarchy for changes and then dynamically trigger a switch from TLHC to VD if a SurfaceView is added. (cc @bparrishMines )
In the short term, affected plugins should be able to work around this issue (for Flutter 3.3, which is currently on beta, and later) by making the view returned from their view factory contains an initial dummy SurfaceView so that fallback mode triggers.

Thanks, I was able to reproduce with https://github.com/dyte-in/flutter-sample-app
The reason that #103630 didn't work for this plugin is that at view creation time, when the view hierarchy is scanned for SurfaceViews, there isn't one (it looks like it's just an empty container view: com.facebook.react.ReactRootView), so fallback to Virtual Display doesn't trigger. Then later, a SurfaceView is added, triggering the bug.
It might be possible to monitor the view hierarchy for changes and then dynamically trigger a switch from TLHC to VD if a SurfaceView is added. (cc @bparrishMines )
In the short term, affected plugins should be able to work around this issue (for Flutter 3.3, which is currently on beta, and later) by making the view returned from their view factory contains an initial dummy SurfaceView so that fallback mode triggers.

Thanks, I will let the developers know.

https://github.com/flutter/flutter/issues/110816 is another duplicate of this (also starting as an empty com.facebook.react.ReactRootView).
@bparrishMines Do you have cycles to look into whether dynamic fallback is viable?

Oh, sorry, I'll pick that back up tomorrow. It mostly needs tests.

Any news on this? This still prevents us to upgrade to Flutter v3.
I see that https://github.com/flutter/engine/pull/35233 has been merged, so is there anything blocking this issue?

This still prevents us from upgrading to Flutter v3.

What plugin is preventing you from updating? As noted above, developers of plugins with this behavior can work around it with a minor change.

This still prevents us from upgrading to Flutter v3.

What plugin is preventing you from updating? As noted above, developers of plugins with this behavior can work around it with a minor change.

It's a plugin that's not being maintained anymore, but we already created a fork, so we might be able to add this workaround. Thanks!

this issue is the same problem that I have?
https://github.com/HMS-Core/hms-flutter-plugin/issues/253

This workaround worked for me:
    override fun onCreate(savedInstanceState: Bundle?) {
        intent.putExtra("background_mode", "transparent");//TODO Huawei fix map
        super.onCreate(savedInstanceState);
        MapsInitializer.initialize(applicationContext, Renderer.LATEST, this)
    }

    //TODO Huawei fix map
    override fun onFlutterTextureViewCreated(flutterTextureView: FlutterTextureView) {
        // We do not actually need a transparent TextureView.
        flutterTextureView.isOpaque = true;
        super.onFlutterTextureViewCreated(flutterTextureView);
    }

Thanks to @ppiatkowski-here

Patch released and fixed in the beta release. dyte_client 0.9.0-beta.1

Note, it does not appear possible to dynamically switch from Android View Hierarchy to Virtual Display or vice-versa.
If your Platform View uses a SurfaceView it must be present when the Platform View is first instantiated and then the correct behaviour will trigger.
NOTE: You should avoid using SurfaceViews if at all possible. TextureView does not suffer from this issue.


Added info on how to start a yeoman-generated project

Really simple edit to the two READMEs adding info on how to run a yeoman-generated project. I followed the instructions and got confused why it wouldn't run; I figured others probably would, too.

Whoa.  @doug-wade - Is that new?  Didn't it used to just start with npm start?

Actually, wait. This may be a mistake on my part.

Actually, wait. This may be a mistake on my part.

No, I was right the first time; got confused looking at the wrong package.json.
Looks like what the yeoman project does is: npm run compile calls gulp, which runs the JS files through react-server-gulp-module-tagger and babel with react-server preset, dumping the output in the build directory. Then npm run start runs react-server-cli with build/routes.js.
I think that after #280 is merged, the compile step can be skipped altogether. This will enable hot reloading (which currently doesn't work because of the build step) and CSS and other assets (which currently don't work because the gulp script doesn't copy them to build.

@gigabo no, you have to run compile then start because module tagging.  @aickin's summary is 100% spot-on

I see.  In that case let's get #280 out (and revert that portion of this PR when we do)!
Thanks for clearing this up @aickin.


let's fill up the pages column

Let's fill up the column, it's important
@uchitsa please, help

@yegor256 thanks, issue in progress


[WIP] Warnings upon tuning  information mismatch for Convolutions

Apart from CI this looks good.

Yes, i'll fix tidy.

Just to be clear, since there isn't a description
You're adding better warnings for the convolution operators and based on MIOpen version, or CU core count

Yes, warning are for the cases where tuning information stored in mxr file doesn't match the user's configuration.

Unit test change is to handle the extra args you added (gfx_arch, cu_count, miopen_version) from how things are serialized?

Yes, size of thecontext changes because of that.

Apart from CI this looks good.

Yes, i'll fix tidy.

Just to be clear, since there isn't a description
You're adding better warnings for the convolution operators and based on MIOpen version, or CU core count

warning are for the cases where tuning information stored in mxr file doesn't match the user's configuration. In that case, users are required to enable tuning flag to get optimal performance.

Unit test change is to handle the extra args you added (gfx_arch, cu_count, miopen_version) from how things are serialized?

Yes, size of thecontext changes because of that.

All good. Just approved looks good. Just wanted to make sure I understood everything here.

We should throw an error for different gfx arch when the model has binary code objects since the model cannot run on a different arch. We will need to check this in finalize so we aren't throwing errors when they aren't applicable.

I disagree on this one. Users should be able to share/transfer models across different gfx-arch. Erroring out would not allow that.

We could consider adding a logger similar to MIOpen with different levels. So the warnings will show by default but we can have higher log level the user can show to enable this less important info(such as CUs or MIOpen version difference).

Logging with levels is a good idea. I can work on Logging with different levels in a separate work item. But i think for now, we should keep warnings for MIOpen version and CUs mismatch.

Logging with levels is a good idea. I can work on Logging with different levels in a separate work item. But i think for now, we should keep warnings for MIOpen version and CUs mismatch.

You can still put the warnings behind an env variable. Also the warnings should probably go in finalize so it warns when MIOpen is actually used. They are completely pointless when using BERT.


Missing argument in hook_fn in model_profile.py

Hi,
I noticed a small issue in the file model_profile.py on line 67. The hook_fn function appears to be missing an argument, which prevents the code from executing correctly. Specifically, it seems the function should include the n_layer and args parameters when registering the hook.
hook = module.register_forward_hook(hook_fn(n_layer, args))

Thank you for pointing out this issue! You are correct that the ‘args’ argument was missing in model_profile.py.
The reason for this oversight is that I moved the hook_fn (originally located within model_profile) into the shared utility file. To access the global timestep, I subsequently passed the ‘args’ into the hook function. Unfortunately, I forgot to update model_profile.py accordingly.
I have updated the codes. Thanks again for bringing this to my attention!


Hardening AAD COnnect Server

Please add information about securing the configuration of the AAD Connect server, this is an important component that is not addressed in this document. The guidance exists in other places, see https://docs.microsoft.com/en-us/security/compass/overview, so it may best to just add a reference, maybe by including another step

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

ID: bad1a372-3d89-f509-6bd2-1e6d12b1d6c7
Version Independent ID: 92fb5a9a-24b8-6f17-2749-99e3b0e999fb
Content: Secure your Azure AD identity infrastructure - Azure Active Directory
Content Source: articles/security/fundamentals/steps-secure-identity.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @martincoetzer
Microsoft Alias: martinco

@DeanGross
Thank you for your feedback . We will investigate and update the thread.

Thank you for your suggestion. I added the link #please-close


feat: [#504] Route supports configure timeout

github.com/goravel/goravel/issues/504
add WithContext
📑 Description
Closes https://github.com/goravel/goravel/issues/504

@coderabbitai summary

✅ Checks

[ ] Added test cases for my code

Thanks for your updates. I'm not sure if this PR is ready to be reviewed again, just notice, a few points are still not updated.

Unfortunately, CI still failed.

You  can ask me if you have any questions for this.

Если у вас возникнут какие-либо вопросы по этому поводу, вы можете задать их мне.

Do I understand correctly that I need to update the mock files?

You can ask me if you have any questions for this .

Do I understand correctly that I need to update the mock files?

No, unit tests failed, did you run unit tests locally?

Yes, you need to optimize the test flow, there are some unexpected mockConfig assertion.

Hey @KlassnayaAfrodita Are you still working on this, please? If you feel it's difficult to fix the CI, you can add me to your fiber repo, I can solve it.

I invited you to the repository, could you then describe how this problem is solved. current branch patch-1


SFRAME problem and turicreate

Hello
I'm using turicreate as bellow but I got the errors as bellow also
transaction_data = '../notebooks/blocks_0_10000/'
if os.path.exists('../notebooks/blocks_0_10000'):
sf = gl.SFrame(transaction_data)
# TAKE SAMPLE
s = sf.sample(0.22, seed=1)
df = s.to_dataframe()
df = df[df['input_address'].notnull()]
df = df[df['output_address'].notnull()]
df = df.fillna(0)
sf_transactions = gl.SFrame(df)
g = gl.SGraph().add_edges(sf_transactions, src_field='input_address', dst_field='output_address')
print(len(sf))
print(len(df))
print(g)
else:
print('cant find data')
ERROR:
OSError                                   Traceback (most recent call last)
File ~/anaconda3/envs/py38/lib/python3.8/site-packages/turicreate/data_structures/sframe.py:834, in SFrame.init(self, data, format, _proxy)
833     url = _make_internal_url(data)
--> 834     self.proxy.load_from_sframe_index(url)
835 elif _format == "empty":
File cy_sframe.pyx:77, in turicreate._cython.cy_sframe.UnitySFrameProxy.load_from_sframe_index()
File cy_sframe.pyx:80, in turicreate._cython.cy_sframe.UnitySFrameProxy.load_from_sframe_index()
OSError: Cannot open /home/aa/Downloads/thesis-bitcoin-clustering-master/notebooks/blocks_0_10000/dir_archive.ini for read. Cannot open /home/aa/Downloads/thesis-bitcoin-clustering-master/notebooks/blocks_0_10000/dir_archive.ini for reading
During handling of the above exception, another exception occurred:
OSError                                   Traceback (most recent call last)
Input In [11], in <cell line: 5>()
3 transaction_data = '../notebooks/blocks_0_10000/'
5 if os.path.exists('../notebooks/blocks_0_10000'):
----> 6     sf = gl.SFrame(transaction_data)
7     # TAKE SAMPLE
8     s = sf.sample(0.22, seed=1)
File ~/anaconda3/envs/py38/lib/python3.8/site-packages/turicreate/data_structures/sframe.py:838, in SFrame.init(self, data, format, _proxy)
836     pass
837 else:
--> 838     raise ValueError("Unknown input type: " + format)
File ~/anaconda3/envs/py38/lib/python3.8/site-packages/turicreate/_cython/context.py:50, in debug_trace.exit(self, exc_type, exc_value, traceback)
47 if exc_type:
48     if not self.show_cython_trace:
49         # To hide cython trace, we re-raise from here
---> 50         raise exc_type(exc_value)
51     else:
52         # To show the full trace, we do nothing and let exception propagate
53
54         # In verbose mode we print the server log
55         if self.show_server_log:
OSError: Cannot open /home/aa/Downloads/thesis-bitcoin-clustering-master/notebooks/blocks_0_10000/dir_archive.ini for read. Cannot open /home/aa/Downloads/thesis-bitcoin-clustering-master/notebooks/blocks_0_10000/dir_archive.ini for reading

What is the contents of /home/aa/Downloads/thesis-bitcoin-clustering-master/notebooks/blocks_0_10000? Looks like it's not an SFrame.

Thanks for your response the blocks_0_10000 is a folder that contains below files which is the output of the (https://github.com/behas/bitcoingraph)
output of this script : bcgraph-export 0 1000 -u your_rpcuser -p your_rpcpass
addresses.csv: sorted list of Bitcoin addressed
addresses_header.csv
blocks.csv: list of blocks (hash, height, timestamp)
blocks_header.csv
transactions.csv: list of transactions (hash, coinbase/non-coinbase)
outputs.csv: list of transaction outputs (output key, id, value, script type)
rel_block_tx.csv: relationship between blocks and transactions (block_hash, tx_hash)
rel_input.csv: relationship between transactions and transaction outputs (tx_hash, output key)
rel_output_address.csv: relationship between outputs and addresses (output key, address)
rel_tx_output.csv: relationship between transactions and transaction outputs (tx_hash, output key)

Take a look at the API for SFrame. You can pass a single csv file to SFrame(...) but you can't pass a directory of csv files.


System Management IP Declarative Onboarding Conversion is non functional

Environment

F5 Extension Version: 3.14.0

Summary
Certain Sys Management configurations do not convert properly
Steps To Reproduce
Steps to reproduce the behavior:

Convert the following CONF to DO declaration by highlight and right click, convert to DO:

sys management-ip <IP_ADDRESS>/24 {   }

Observe the following error response:

{
    "schemaVersion": "1.0.0",
    "class": "Device",
    "async": true,
    "Common": {
        "class": "Tenant",
        "}": {
            "class": "ManagementIp",
            "address": "}"
        }
    }
}

Expected Behavior
Output like this:
{
    "schemaVersion": "1.0.0",
    "class": "Device",
    "async": true,
    "Common": {
        "class": "Tenant",
        "ManagementIp_IPv4": {
            "class": "ManagementIp",
            "address": "<IP_ADDRESS>/24"
        }
    }
}

this function is provided by the vscode-f5-chariot extension which is based on the ACC engine.  That engine is no longer getting any public updates.  closing


Is there anyway to delay computing of loss?

Say, I have a compute graph,
pred = <blah blah graph> # the predicted output from graph
delayed_label = mx.sym.Variable(name="delayed_label")
loss = mx.sym.MakeLoss(<blah blah expression of symbol pred and symbol delayed_label>)
loss_output = mx.sym.Group([mx.sym.BlockGrad(pred), loss])

where delayed_label can only be retrieved after the pred is got. In this setting, seems I can do
exe = loss_output.simple_bind(...)
... # copy input to input array
exe.forward(is_train=False)
... # get pred output from exe.outputs[0] and then retrieve data for delayed_label
... # copy retrieved data to delayed_label array
exe.forward(is_train=True)
exe.backward()
... # update weights

Is there anyway I can do only one forward pass in this scenario instead of two?

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


npm run dev:show blue window, but not show Home to Counter

version of the boilerplate: "0.13.3"
Windows 7
npm run start, work ok
When I package to windows (npm run package-win), work ok
But npm run dev show blue window, but not show Home to Counter
View->Reload, still not show
Thanks in advance

What are the errors in the console?

@c2rlost2r perhaps you need to run set UPGRADE_EXTENSIONS=1 && npm run dev. That will upgrade your devtools extensions.
see: https://github.com/chentsulin/electron-react-boilerplate/issues/1570

Ahhh yes I am pretty sure that is the solution. Thanks @BuckyMaler

Yes, it works, Thank you.
Great Boilerplate!!!!!!!!!!!!!!!


Speed up query results copying

This PR will fix https://github.com/microsoft/azuredatastudio/issues/25632
The other PR's needed:
sqlops-dataprotocolclient - https://github.com/microsoft/sqlops-dataprotocolclient/pull/101
sqltoolsservice - https://github.com/microsoft/sqltoolsservice/pull/2385
Copying query results from Azure Data Studio (Fast):

Copying query results from the SQL Tools Service (Slow, takes about 50 seconds to complete):

The copy behavior on Mac is also similar to windows, where letting Azure Data Studio do the copying completes quicker and allowing the query provider to do the copying is slower.
Additionally, this PR reverts changes that weren't necessary to fix this Linux query results copy issue (#24341 ). The copy issue around Linux was due to the TextCopy NuGet package, used in the SQL Tools Service, relying on xsel to do the copying. The xsel package isn't installed by default on Linux, and as a result prevented some user's from copying when xsel wasn't installed. To get around that, there's a setting that allows user's to control if they would like to copy from the SQL Tools Service process or the Azure Data Studio UI process which doesn't use TextCopy. The setting to change where copying occurs is this one:

Here's a gif where 200,000 query results are copied on Linux using the Azure Data Studio UI Process:

So this change (alongside with PRs in STS + sqlops-dataprotocolclient) will now do the two following things, right?

revert the provider side copy feature to the state Alan initially implemented.
set the default value for provider side copy config to false instead of true.

This means win/mac users will need to manually enable this config to get the fast copy feature. Can we enable this for win/mac automatically without breaking copy for linux?
Can we do something like this?
'default': isLinux ? false : true,

So this change (alongside with PRs in STS + sqlops-dataprotocolclient) will now do the two following things, right?

revert the provider side copy feature to the state Alan initially implemented.
set the default value for provider side copy config to false instead of true.

This means win/mac users will need to manually enable this config to get the fast copy feature. Can we enable this for win/mac automatically without breaking copy for linux?
Can we do something like this?
'default': isLinux ? false : true,

This change will revert the provider side copy feature to the state Alan initially implemented, and will also set the query provider copy feature to false. User's don't need to change anything because copying from the UI was much faster than copying from STS.

Win/Mac users don't need to change anything because copying from the UI was much faster than copying from the query provider.

I'm confused by this comment.  What is "query provider" referring to here, the query service in STS?  The original bug for slow copy was reported on Windows with the current implementation (https://github.com/microsoft/azuredatastudio/issues/25632). What is this PR doing to improve the copy performance on Windows?

So this change (alongside with PRs in STS + sqlops-dataprotocolclient) will now do the two following things, right?

revert the provider side copy feature to the state Alan initially implemented.
set the default value for provider side copy config to false instead of true.

This means win/mac users will need to manually enable this config to get the fast copy feature. Can we enable this for win/mac automatically without breaking copy for linux?
Can we do something like this?
'default': isLinux ? false : true,

This change will

Revert the provider side copy feature to the state Alan initially implemented
will also set the query provider copy feature to false.

Win/Mac users don't need to change anything because copying from the UI was much faster than copying from the query provider.

I was always under the impression that STS side copy is faster and not aware of the UI side copy optimization happened in https://github.com/microsoft/azuredatastudio/pull/25152. Can you update the PR description to include the whole story?

So this change (alongside with PRs in STS + sqlops-dataprotocolclient) will now do the two following things, right?

revert the provider side copy feature to the state Alan initially implemented.
set the default value for provider side copy config to false instead of true.

This means win/mac users will need to manually enable this config to get the fast copy feature. Can we enable this for win/mac automatically without breaking copy for linux?
Can we do something like this?
'default': isLinux ? false : true,

This change will

Revert the provider side copy feature to the state Alan initially implemented
will also set the query provider copy feature to false.

Win/Mac users don't need to change anything because copying from the UI was much faster than copying from the query provider.

I was always under the impression that STS side copy is faster and not aware of the UI side copy optimization happened in #25152. Can you update the PR description to include the whole story?

It would also be interesting to see this PR compared to the current implementation.  The two changes I see here for default Windows users is that the copy method is now awaited and the result string is no longer returned from STS (it's not clear that's even being called in default Windows code path).  The recent bug reports are on Windows after https://github.com/microsoft/azuredatastudio/pull/25152, so unclear how this change address this bug to me.


Fixing typo

Updated to web app down from overview.

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

@cephalin
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Jak-MS Could you help move this one along? Thanks.

@RobiladK - your best bet in the future is to submit changes via the private repo: 'azure-docs-pr' ... for this one, I can't merge it without a review.
#assign: @georgewallace
@georgewallace - can you take a look? thanks.

@Jak-MS I am no longer in the docs org. I have been out of it for quite some time. I would recommend reaching out to cephas again or someone else on his team.

@cephalin Could you take a look at this PR?

#sign-off


How to add tabs dynamically from the json obtained

Hi, How to add tabs dynamically from the json obtained. My API returns the json with tab title, icon and component,...
Thank so much.

@facebook-github-bot stackoverflow

@facebook-github-bot stack-overflow

Hey @agumack and thanks for posting this! @chirag04 tells me this issue looks like a question that would be best asked on StackOverflow. StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only. Will close this as this is really a question that should be asked on SO.


Question about the inner gradient step

Hi Jingwei,
The lines you reference are doing the meta-evaluation of the network.
Before fine-tuning for each test task, the parameters of the test network
are reset and the optimizer is re-created, in these lines
https://github.com/katerakelly/pytorch-maml/blob/master/src/maml.py#L100-L102
.
As for the meta-training, you are correct that a batch of meta-updates is
collected by fine-tuning independently on separate tasks, all starting from
the same parameters, see here
https://github.com/katerakelly/pytorch-maml/blob/master/src/maml.py#L159-L164
.
Let me know if you still see a problem!
On Sun, Feb 25, 2018 at 11:00 AM, Jingwei Zhang<EMAIL_ADDRESS>wrote:

Hi,
Nice repo! One question about src/maml.py Line 107~111: as far as I
understand, when doing the inner gradient steps, maml uses the same /theta
for computing the loss for all the training demos, whereas in your
implementation, the parameters of test_net is updated after every
training demo, then the loss for the next training demo is computed using
the updated parameters. Am I misunderstanding sth here?
Thanks a lot in advance!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/katerakelly/pytorch-maml/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADplVuNA-IaMuuiVPJCL0xgJ_ggEUEg4ks5tYa3EgaJpZM4SSaZE
.

--
Kate Rakelly
UC Berkeley EECS PhD Student
<EMAIL_ADDRESS>

Hey,
yes I found out I was reading it wrong so I deleted my question lol, but still thank you so much for the explanation:)


panic

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x124220c]

goroutine 20 [running]:
main.(*Resolver).Lookup.func1(0x1313c60, 0xc00000e420)
	/Users/huzhifeng/workspace/netroby/dnscache/resolver.go:35 +0x18c
created by main.(*Resolver).Lookup
	/Users/huzhifeng/workspace/netroby/dnscache/resolver.go:82 +0x12e
DNS server start

问题已修复


org.redisson.client.RedisConnectionException: SlaveConnectionPool no available Redis entries.  Disconnected hosts:{IPS}

Intermittent issue
org.redisson.client.RedisConnectionException: SlaveConnectionPool no available Redis entries.  Disconnected hosts:{IPS}
Expected behavior : If any ips of slave change it should adjust it.
Actual behavior Intermittently getting issue : Intermittent issue
org.redisson.client.RedisConnectionException: SlaveConnectionPool no available Redis entries.  Disconnected hosts:{IPS}
Steps to reproduce or test case
Redis version 6.x
Redisson version <IP_ADDRESS>
Redisson configuration

Can you upgrade to 3.23.2 version?

Will version 3.21.3 works? Does version 3.21.3  has resolution of this issue?

mrniko
Can you please get chance to reply on my above comment of version?

@miteshjain24
3.23.3 version should resolve the issue.


Symbol

It means exactly the same in TF, e.g. [m,n] -> [m*n]

From: Luo Mai<EMAIL_ADDRESS>Sent: Sunday, March 4, 2018 5:14:15 PM
To: lgarithm/crystalnet
Cc: LG; Author
Subject: Re: [lgarithm/crystalnet] Symbol (#9)
@luomai commented on this pull request.

In examples/c/mnist_slp_symbol.chttps://github.com/lgarithm/crystalnet/pull/9#discussion_r172041941:

@@ -0,0 +1,49 @@
+#include <assert.h>

+#include <crystalnet.h>
+
+// y = softmax(flatten(x) * w + b)
+s_model_t *slp(shape_t *image_shape, uint8_t arity)
+{

shape_t *lable_shape = make_shape(1, arity);
shape_t *weight_shape = make_shape(2, shape_dim(image_shape), arity);
shape_t *x_wrap_shape = make_shape(1, shape_dim(image_shape));

s_model_ctx_t *ctx = new_s_model_ctx();
symbol x = var(ctx, image_shape);
symbol x_ = reshape(ctx, x_wrap_shape, x);

what does reshape mean?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/lgarithm/crystalnet/pull/9#pullrequestreview-100999544, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB8ydHMbUULgmOFuWjBUuZ_B6OEzt6hxks5ta7BngaJpZM4SbPOr.

new_shape is definitely required in C, free_shape could be removed by using GC.

From: Luo Mai<EMAIL_ADDRESS>Sent: Sunday, March 4, 2018 5:16:04 PM
To: lgarithm/crystalnet
Cc: LG; Author
Subject: Re: [lgarithm/crystalnet] Symbol (#9)
@luomai commented on this pull request.

In examples/c/mnist_slp_symbol.chttps://github.com/lgarithm/crystalnet/pull/9#discussion_r172042007:

@@ -0,0 +1,49 @@
+#include <assert.h>

+#include <crystalnet.h>
+
+// y = softmax(flatten(x) * w + b)
+s_model_t *slp(shape_t *image_shape, uint8_t arity)
+{

shape_t *lable_shape = make_shape(1, arity);

Is it really necessary to ask users to new/free shape here?
In case of creating a complex neural network, this would largely complicate the code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/lgarithm/crystalnet/pull/9#pullrequestreview-100999602, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB8ydHyTJLbsobZd1fRyOyKAnDxjSZUsks5ta7DUgaJpZM4SbPOr.


chore: replace github.com/ghodss/yaml with sigs.k8s.io/yaml

The github.com/ghodss/yaml package is no longer being actively maintained. sigs.k8s.io/yaml is a permanent fork of github.com/ghodss/yaml, which is actively maintained by Kubernetes SIG and widely used in K8s projects.
The notable change is that github.com/ghodss/yaml uses gopkg.in/yaml.v2 v2.2.2, while sigs.k8s.io/yaml uses gopkg.in/yaml.v2 v2.4.0. You can see the changes between the two versions here: v2.2.2...v2.4.0, which mostly consists of bug fixes.
Note to the reviewer:
Please review and merge https://github.com/openshift/library-go/pull/1501 as well. Once that's done, I'll update github.com/openshift/library-go in this PR, allowing us to completely remove github.com/ghodss/yaml.
$ go mod why -m github.com/ghodss/yaml
# github.com/ghodss/yaml
github.com/openshift/microshift/pkg/assets
github.com/openshift/library-go/pkg/operator/resource/resourceapply
github.com/openshift/library-go/pkg/operator/v1helpers
github.com/ghodss/yaml

Thanks!
/ok-to-test

It looks like the patch we add to klog is missing. Could you run make vendor and push the results as part of this PR?

Hi @dhellmann, thank you for taking the time to review my PR. I apologize for not being aware of the make vendor command in the Makefile and instead running go mod vendor directly.
Additionally, I would like to kindly request that you review another PR https://github.com/openshift/library-go/pull/1501. If we are able to merge that PR, we can fully remove github.com/ghodss/yaml from this repository.
Thank you for your assistance and consideration :smiley: .

/retest-required

/retest

Can't seem to get the last test e2e-openshift-conformance-reduced-arm to pass

/retest

/reteset

/retest

/retest

/retest

/retest

/retest

@dhellmann @copejon @ggiguash ci/prow/e2e-openshift-conformance-reduced seems to be failing since 19 May. Can we override it?

Rebased and resolved conflicts in go.mod

/retest-required

/lgtm

/unhold


npm install -g flightplan throws error

Here is the last bit of the error:

npm ERR<EMAIL_ADDRESS>install: node build.js || nodes build.js

npm ERR! Exit status 127
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node build.js || nodejs build.js'.
npm ERR! This is most likely a problem with the fibers package,
npm ERR! not with nam itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node build.js || nodes build.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls fibers
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:

Here is the debug.log file:
npm-debug.log.zip

I had to update node on my system... - sorry for the noise.


Supporting Rails 5

As I mentioned privately to @boffbowsh -- I'm a good way along with making this gem Rails 5 compatible and am keen to keep our changes going upstream.
One outstanding issue is Rails 5 dropping support for Rubies earlier than 2.2.2, thanks to Rack 2's requirements.
I couldn't remember if you folks still maintain projects targeting earlier rubies so perhaps this is a non-issue? If you do, we could still maintain backwards compatibility for those older Ruby / Rails versions but would likely suffer a wall of deprecation messages when running in the context of Rails 5.x. There are of course workarounds we could employ -- checking the Rails version at runtime and doing the Right Thing:tm:.
Happy to take a steer on your most favoured approach. Thoughts?

Yo. Cheers for helping out on this. We're actively working towards getting rid of Ruby 2.1 on our apps, so making gds-sso support 2.2+ only is fine. If apps need to upgrade to a newer gds-sso, they can upgrade Ruby at the same time.
Almost everything is now on Rails 4.2+ so you wouldn't need to worry about incompatibility with older Railses.
Cheers again, shout via the various internet methods if you need a hand with anything.

Perfect, thanks ❤💛💚💙


Links broken

Links are pointing to 404 not found

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

ID: dd4366d9-2275-80aa-3ba6-195a234d45f4
Version Independent ID: 11726543-8d63-c7e4-00b7-c715110b6032
Content: Overview of Process Advisor (preview) - Power Automate
Content Source: articles/process-advisor-overview.md
Service: flow
GitHub Login: @nijemcevic
Microsoft Alias: tatn

@PekkaJalonen Thanks for reporting this issue. The fix will be avilable in the next 3 hours.


Job launch race condition fix.

Previous job and module state file locking mechanism placed a lock on the actual state file needed. Updated to use separate lock file as a lock. This allows the lock to be already held when state file is attempted to be opened in 'r+' and then opened in 'w' if it doesn't already exist, preventing a race condition. Additionally, added better handling of null job returned to onramppce._update_job_in_db() in the event the requested job has not finished launching.

Is there a server instance with this fix that I can test before merging?

I tested it a bit and it seems a lot better.  Thanks!


Feature request: add tracking

It'd be great to add a tracking mode e.g. see video here.
Specifically, if I enable tracking (per track) if there are no annotations in the future of the track, then attempt to track the last annotated box for all future frames. If the user goes to the next frame while one of these tracks is displayed, then this is considered as marking this as 'good', and it gets added as a key frame. Otherwise the user can edit it manually.
There are probably more UI considerations, but this would provide a lot of value. My use case is tracking people heads, and the standard interpolation is less useful (but still much better than without!) due to heads 'bobbing' while walking etc. Feature tracking would likely solve this in many situations (aside from when a head is occluded etc.).
Aside from UI considerations, this is pretty easy to implement. It can even be done in the browser with opencv.js (and suitable performance, depending on device).

Hi @kodonnell ,
I agree that in some cases it can be very useful feature. Need to add the feature in the future. If you can prepare a PR just let me know.
Previously we used tracking in other our annotation tools. The main problem that most of our annotation tasks required precise annotations which trackers could not give us. Even on your video you can see that results are very bad. Especially difficult to track non rigid objects (like human).

Thanks @nmanovic. I agree - tracking can be hard and won't work in all circumstances, especially if you need exact annotations. In our case, we're less interested in exactness, as we're more using it to say "is there a head there or not?" as opposed to the exact bounding box around the head.
FYI - I thought the results on the video were pretty good.


Same vehicles for some stops.

Is there a way to specify that some stops should be served with the same vehicle ?

Have a look at AddPickupAndDelivery, but be aware: this function merely locks both stops to the vehicle, it doesn't guarantee their order (e.g. delivery can come before pickup).
This function basicly does the following (C#):
solver.Add(solver.MakeEquality(routing.VehicleVar(constraint.Pickup), routing.VehicleVar(constraint.Delivery));


Cannot overwrite default text filter type

Describe the bug (required)
Although it is possible to overwrite the text filter type that does not overwrite the default as well so the default text filter type cannot be overwritten.
Provide an example via Codesandbox! (required)
We can see in the filtering example Codesandbox that the text filter type is overwritten (Lines 220-231) and we also see that the first name column is configured without a filter (Lines 343-346). However the custom text filter is not used.
Steps To Reproduce (required)
Steps to reproduce the behaviour:

Go to https://codesandbox.io/embed/github/tannerlinsley/react-table/tree/master/examples/filtering
Enter text into the First Name search box
Any results that include your text appear (Default behaviour rather than custom)

Expected behaviour (Recommended)
In my opinion the custom text filter should be used and, in this case, only results that start with search string should be displayed.
Desktop (please complete the following information):

OS: MacOS 10.15.4
Browser: Chrome
Version: 81.0.4044.129

Additional context
The cause seems to be that in the utils file if no filter is specified (and therefore the filter variable is presumably undefined or null) the getFilterMethod function always returns the built in text function. See: https://github.com/tannerlinsley/react-table/blob/42b41d1260fab3c6635b071ca984dce253bf23ab/src/utils.js#L264-L271

Simply set the default filter property on defaultColumn to filter: 'text'. This is how things will work in the next version anyway.

Just ran in to this too.  And when you say "this is how things will work in the next version anyway" does that mean you are going to change getFilterMethod to return user defined `text? Or just not have a default filter at all make users have to define a default column?
Just curious as we have implemented your work around, but wanted to know if any future changes would break anything.  Thanks!


Modbus in PLC Pico

Very good project. Can you grant access to variable type "M" in Modbus?

This firmware is deprecated.I suggest you to use picoPLC Arduino library instead.

Este firmware está obsoleto. Sugiro que você use a biblioteca picoPLC Arduino.

Ok, and what is the Modbus address for the "M" variables in the PicoPLC library?


Actions are not typed to be able to observe their state on creation.

Currently this does not work.

Closed via fe6615ca1bdacaad357943f5e4d3f78ab51f5f48


Update naming format of PowerShell scripts - functions part

Run all BVT cases against Azure - Pass
Pick up 2 cases run against Hyper-V - Pass

@LiliDeng @v-sijliu I don't recall we decided to rename it yet. The change is not an issue but you need to buy team's agreement.
suggest to buy team's agreement before moving forward.

@v-sijliu The changes were simple but happened many placed. Can you randomly select and run tests if changes are not breaking? As far as it does not break, I will sign off.

@lizzha Rebase latest code, run 3 BVT cases against Azure - pass, run 1 BVT case against Hyper-V - pass


Create screenshots with fastlane/snapshot

https://docs.fastlane.tools/getting-started/ios/screenshots/

not working for iMessage Extension. Otherwise done.


More documentation not only to generate the attestation for a SBOM but also to know how to verify it

Hi there 👋🏻
It would be great if there was some more documentation not only to generate the attestation for a SBOM but also to know how to verify what this action generates. Because if you are not an expert in this area it is difficult to know what to do once you launch this action.
I have been able to invoke this action for an image and for a binary but after that I don't know what to do with it 😖
Thank you so much

No matter what type of attestation you are generating (build provenance, SBOM, etc) you can use the gh attestation verify command to do the verification. More details in the docs: https://docs.github.com/en/actions/security-for-github-actions/using-artifact-attestations/using-artifact-attestations-to-establish-provenance-for-builds#verifying-artifact-attestations-with-the-github-cli


ClientX and ClientY for Drop

Many thanks for the code, saved me a lot of pain! ClientX and ClientY are not currently included for Drop. I added them into dispatchDrop and they worked as expected.
  dropEvt.clientX = touch.clientX;
  dropEvt.clientY = touch.clientY;

Thanks for using the library, and your issue report! I believe this is fixed in our rewrite, so we'll wait till that's merged.


Make TestReportFormat, specifically the junit format selectable

Context
Circleci and probably other tools as well need the Junit xml format in order to properly parse test runs.
Support for both the sonar generic test execution format and the Junit XML format were already implemented 👏🏼
This PR simply adds the option to select the junit test report formatter from the cli tool.
Default is still the sonar generic test execution format, the change is non breaking.
Summary of changes

Add new junit output format
Improve tests by checking for root element name
Update docs
Bump version

Hi Florian, thanks for your contribution. It totally slipped my attention, that, what you were after was implemented, but inaccessible due to the missing command line argument, which you now added. Great!
Also the improvement to the very sparse (as usual ;-) unit tests is very welcome.


errata state changes on History tab

Description
A junk report creates an entry both when the report generated and when it's deleted. For example, see https://datatracker.ietf.org/doc/rfc1/history/. This creates noisy histories for an RFC.
Perhaps errata state changes should not be captured on the History tab at all, or perhaps there should be lag between the time it's reported and when it's added to the History page so that there's time for a junk report to be deleted.
Code of Conduct

[x] I agree to follow the IETF's Code of Conduct

We could also record the events and filter them out by default.


can increase the network usage for reading data from cpp server by java client

Describe the bug
A clear and concise description of what the bug is.
https://github.com/apache/brpc/issues/2369
the brpc java client can read data from cpp server, but just using 30% network, so I wonder starlight can increase the network usage for such cases?
To Reproduce
Steps to reproduce the behavior:

Go to '...'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

这块没有特别的设计，我理解是starlight是可以用满网络带宽的，不过可能在StarRocks的场景还需要streaming的能力，我们目前还没支持。

@tullyliu 明白，谢谢您的回复。


“test your bus factor” promo page

A user said:

kudos for adding a list of projects for which I am the sole owner under Delete Account. I saw two in there that I immediately fixed. Very helpful! :+1:

@di suggested that this is a serendipitous "bus factor" feature and suggested we create a small page to promote it.
My spec:

This is outside of the usual project URL structure, maybe at something like pypi.org/win-the-lottery
copy would be something like:

If you won the lottery, do you have co-maintainers who could step up and take care of your packages? Check your list of projects (LINK, OR INCLUDE THIS LIST IN THE PAGE) to check.
And check https://libraries.io/experiments/bus-factor?platforms=PyPI for packages that are depended upon by many other packages but have only ever had a handful of contributors commit to the project. You can help out!

make it shareable with the usual social media buttons

I really like this idea.

related: https://github.com/pypa/warehouse/issues/3127

Since this was my idea, I'm going to close this as "so incredibly low-priority that it's not even worth keeping the issue open".


Fix #14264

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14268/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14268/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14268/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14268/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14268/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14268/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14268/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14268/merge#BCU1XR#0


SSP - Can't send reset token by email

Hi,
I'm trying to setup reset token sent by email with ssp. However, I get the following error in /www/logs/nginx/error.log when going to the Email tab, typing my user and clicking send :
2024/08/27 13:43:59 [error] 3431#3431: *5 FastCGI sent in stderr: "PHP message: Send reset URL https://:80/index.php?action=resetbytoken&token=XXXXXXXXX message:
Connection:opening to postfix-relay:25, timeout=30, options=array (
)PHP message: Connection failed. Error #2: stream_socket_client(): php_network_getaddresses: getaddrinfo failed: Name does not resolve [/www/ssp/lib/vendor/PHPMailer/class.smtp.php line 293]PHP message: Connection failed. Error #2: stream_socket_client(): unable to connect to postfix-relay:25 (php_network_getaddresses: getaddrinfo failed: Name does not resolve) [/www/ssp/lib/vendor/PHPMailer/class.smtp.php line 293]PHP message: SMTP ERROR: Failed to connect to server: php_network_getaddresses: getaddrinfo failed: Name does not resolve (0)PHP message: SMTP connect() failed. https://github.com/PHPMailer/PHPMailer/wiki/TroubleshootingPHP message: send_mail: SMTP
connect() failed. https://github.com/PHPMailer/PHPMailer/wiki/TroubleshootingPHP message: Error while sending token to<EMAIL_ADDRESS>(user my.user)PHP message: PHP Notice:  Undefined variable: obscure_failure_messages in /www/ssp/pages/sendtoken.php on line 212PHP message: PHP Warning:  in_array() expects parameter 2 to be array, null given in /www/ssp/pages/sendtoken.php on line 212" while reading response header from upstream, client: X.X.X.X, server: , request: "POST /?action=sendtoken HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "ssp.my.domain", referrer: "https://ssp.my.domain/?action=sendtoken"

My values.yml for ltb-ssp is as followed :
# Default values for ltb-passwd.
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

replicaCount: 1

image:
  repository: tiredofit/self-service-password
  tag: 5.3.1
  pullPolicy: Always
  pullSecrets: []

nameOverride: ""
fullnameOverride: ""

service:
  type: ClusterIP
  port: 80

## Configure Ingress based on the documentation here: https://kubernetes.io/docs/concepts/services-networking/ingress/
ingress:
  enabled: false
  annotations: {}
  ## See https://kubernetes.io/docs/concepts/services-networking/ingress/#ingressclass-scope
  # ingressClassName: nginx
  path: /
  tls: []

# -- Init containers to add to the pod
## Note: Supports use of custom Helm templates
initContainers:
  - name: copy-ro-certs
    image: busybox
    command: ['sh', '-c', 'cp /tmp-certs/ca.crt /path/']
    volumeMounts:
      - name: certs
        mountPath: /tmp-certs
      - name: ssp-certs
        mountPath: /path

# -- Additional volumeMounts to the application main container
volumeMounts:
  - mountPath: /path/
    name: ssp-certs

# -- Additional volumes to the application pod
volumes:
  - name: certs
    secret:
      secretName: tls-secrets
  - name: ssp-certs
    emptyDir: {}
resources: {}
nodeSelector: {}
tolerations: []
affinity: {}
ldap:
  custom:
    - name: TLS_REQCERT
      value: never
env:
- name: SECRETKEY
  value: "password"
- name: LDAP_LOGIN_ATTRIBUTE
  value: "cn"
- name: LDAP_STARTTLS
  value: "false"
- name: LDAP_CA_CERTIFICATE
  value: "/path/ca.crt"
- name: PASSWORD_MIN_LENGTH
  value: "8"
- name: PASSWORD_MIN_UPPERCASE
  value: "1"
- name: PASSWORD_MIN_SPECIAL
  value : "1"
- name: MAIL_USE_LDAP
  value: "true"
- name: DEBUG_MODE
  value: "true"
- name: LANG
  value: "en"
- name: ENABLE_RESET_LOG
  value: "true"

It seems it can't resolve postfix-relay. Not sure what to do about that.

Download
https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi @RorFis ,
You need to configure the SMTP to be able to send email.
You can find the list of variable on the docker project of ssp
You need to configure SMTP_ env variable for the host, authentication and port of your email server.

Hi @jp-gouin,
Indeed it works now.
Thanks !


Add icon to 'open lapce here' command

(This is intended to open the folder the user is in inside of lapce, but currently simply opens lapce. This is because we don't have cli argument parsing yet.)

Having the Lapce icon show rather than nothing would be good.

Fixed for some time now


[BUG] -NextUI Table - type.getCollectionNode is not a function

NextUI Version
2.1.10
Describe the bug
Whenever you wrap a table element with another container, even if it's a supported HTML table element, this error is thrown. For some reason the table enforces the children to be only components it expects, wondering if its possible to switch to an element selector approach vs expecting the direct children
Steps to Reproduce the Bug or Issue

Install NextUI
Build a table
Wrap your NextUI TableRow or the cells within a row, with a div or table HTML element and this error ill throw

Expected behavior
Be able to wrap with any component you'd like to extend the tables functionality
Operating System Version
Any
Browser
Firefox

Hey NextUI relies on react-aria colections https://react-spectrum.adobe.com/react-stately/collections.html, the collections API uses specific components to manage items, sections, etc... You can use any content inside of those items tho


refactor:using vuejs instead of jquery

Coverage remained the same at 85.625% when pulling 4f82a02e689bdbd42cfa3a683c226d65d944de1d on pnaik1:vuejs into 536fbad9ded318a57ff5cbe84dac7b0c1dbdc13f on nodeshift-starters:main.


Will this get updated to 1.16.5 soon and can we have new brick textures?

Describe the feature you'd like
I would like an update to 1.16.5 and also many different types of Bricks as this is a decoration mod and we are missing brick variety like having small bricks like bricks and large bricks like stone brick would be nice as well as having like a path texture for bricks.
Additional Context
https://github.com/Quizer9O8/Strata/issues/14 Similar to this but not changing brick textures, adding more

Sadly there's no 1.16.5 update, but in the past month @GamingCrew69 was so kind to update the mod to later versions of Minecraft. Check out Releases for updated versions.
As for the brick variety this is something I still want to introduce into the mod later down the line as well.

Thanks for your reply even after all this time. Luckily now these newer updates are incredibly useful. Thanks for the mod!


Strange behavior of OnTouched

For sprite A with code:
// sprite A
onTouched => {
    println "onTouched"
}

The callback will not be called when A collides with another sprite (referred as B).
The callback is called when A collides with B, and method Touching of B is called with A, for example:
// sprite B
onMoving => {
    if touching("A") {
        // do whatever here
    }
}

It's strange, but may be intended. @xushiwei can you offer any hint?
Related PR: https://github.com/goplus/spx/pull/68
P.S. There is no similar API in Scratch

Consider details about the process of "touching" between A & B, it can be devided into several stages:

The first time the engine found that there is an overlap between A & B, referred as "touch start"
There is a period, in which A & B keep the overlap, refered as "touching"
The first time, afterwards, the engine found that the overlap disappears, referred as "touch end"

If we define event touching, which is expected to be fired repeatedly in the 2nd stage, it causes problems like

B loses a lot of health (repeatedly checking that it was hit by a bullet)

But if we define event for "touch start", such problems can be avoided.
Actually in my opinion, in most cases developers care about touch start more than touching or touch end, because most of the effect of touching / collision should take place when the touching / collision starts.
For the other problem:

A disappears but B does not lose health

It should be considered a bug of the engine. Whatever the event is ("touch start", "touching", "touch end", etc.), when the event is detected by the engine, it should be the engine's responsibility to ensure that the event is broadcasted to all attendees. When A touches B, B should always be notified of that, even when A is notifed first and destroyed itself.


receiving more than 6 events at the same time causing events lost

https://github.com/samuel/go-zookeeper/blob/master/zk/conn.go#L297
receiving more than 6 events at the same time, the 7th event will be lost.
for example, receiving events in following order(processing events may be slow):
StateConnected
StateDisconnected
StateDisconnected
StateDisconnected
StateDisconnected
StateConnected
StateHasSession
then StateHasSession will be lost.

What happens if you change this ..
https://github.com/samuel/go-zookeeper/blob/master/zk/conn.go#L39

What happens if you change this ..
https://github.com/samuel/go-zookeeper/blob/master/zk/conn.go#L39

actually we did increase the channel len to 1024 and ran for years in production system......


How to except connection?

Hello!
How i can to except connection?
$oClient = Client::account('default');
try {
            $oClient->connect();
        } catch (\Exception $e) {
            // fail
            return '<pre>' . $e . '</pre>';
        }

Exception return nothing.

Hi @ceo-zenlix ,
Client::connect()  can throw a ConnectionFailedException exception. The exception message is identical with the original exception but contains any potential imap errors as well.
$oClient = Client::account('default');
try {
    $oClient->connect();
} catch(ConnectionFailedException $e) {
    dd($e->getMessage());
}

Best regards

dosn't work with this code
$oClient = Client::account('default'); try {     $oClient->connect(); } catch(ConnectionFailedException $e) {     dd($e->getMessage()); }

@jagoanmamah what did you get instead?


[fix][client] Cannot access message data inside ProducerInterceptor#onSendAcknowledgement

Motivation
In #22393, the payload of message get released before calling ProducerInterceptor#onSendAcknowledgement, hence access message data inside ProducerInterceptor#onSendAcknowledgement will get exception below:
io.netty.util.IllegalReferenceCountException: refCnt: 0
	at io.netty.buffer.AbstractByteBuf.ensureAccessible(AbstractByteBuf.java:1454) ~[netty-buffer-4.1.115.Final.jar:4.1.115.Final]
	at io.netty.buffer.UnpooledHeapByteBuf.array(UnpooledHeapByteBuf.java:147) ~[netty-buffer-4.1.115.Final.jar:4.1.115.Final]
	at org.apache.pulsar.client.impl.MessageImpl.getData(MessageImpl.java:371) ~[classes/:?]
	at org.apache.pulsar.client.api.InterceptorsTest$6.onSendAcknowledgement(InterceptorsTest.java:239) ~[test-classes/:?]
	at org.apache.pulsar.client.api.interceptor.ProducerInterceptorWrapper.onSendAcknowledgement(ProducerInterceptorWrapper.java:58) ~[classes/:?]
	at org.apache.pulsar.client.impl.ProducerInterceptors.onSendAcknowledgement(ProducerInterceptors.java:100) [classes/:?]
	at org.apache.pulsar.client.impl.ProducerBase.onSendAcknowledgement(ProducerBase.java:162) [classes/:?]
	at org.apache.pulsar.client.impl.ProducerImpl$DefaultSendMessageCallback.onSendComplete(ProducerImpl.java:451) [classes/:?]
	at org.apache.pulsar.client.impl.ProducerImpl$DefaultSendMessageCallback.sendComplete(ProducerImpl.java:423) [classes/:?]
	at org.apache.pulsar.client.impl.ProducerImpl$OpSendMsg.sendComplete(ProducerImpl.java:1643) [classes/:?]
	at org.apache.pulsar.client.impl.ProducerImpl.ackReceived(ProducerImpl.java:1316) [classes/:?]
	at org.apache.pulsar.client.impl.ClientCnx.handleSendReceipt(ClientCnx.java:490) [classes/:?]
	at org.apache.pulsar.common.protocol.PulsarDecoder.channelRead(PulsarDecoder.java:236) [classes/:?]

Modifications

Release the message payload after ProducerInterceptor#onSendAcknowledgement is invoked to ensure the payload is accessible during the callback
Add test case InterceptorsTest#testProducerInterceptorAccessMessageData

Verifying this change

[x] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

InterceptorsTest#testProducerInterceptorAccessMessageData

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/Shawyeok/pulsar/pull/22

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.09%. Comparing base (bbc6224) to head (ea703c8).
Report is 817 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23791      +/-   ##
============================================
+ Coverage     73.57%   74.09%   +0.51%
+ Complexity    32624    31759     -865
============================================
  Files          1877     1853      -24
  Lines        139502   143374    +3872
  Branches      15299    16276     +977
============================================
+ Hits         102638   106229    +3591
+ Misses        28908    28774     -134
- Partials       7956     8371     +415

Flag
Coverage Δ

inttests
26.74% <54.54%> (+2.16%)
:arrow_up:

systests
23.12% <100.00%> (-1.20%)
:arrow_down:

unittests
73.61% <100.00%> (+0.77%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

...va/org/apache/pulsar/client/impl/ProducerImpl.java
84.23% <100.00%> (+0.64%)
:arrow_up:

... and 1019 files with indirect coverage changes


Use Aktiv Grotesk in the Docs (#301)

Relates to issue-301.
This PR adds the Aktiv Grotesk font to the Docs font family.

Hi, @fernandofleury and @connor-baer! I've added the requested style tag to load the Aktiv Grotesk font to the docs. If there is anything else you might need, please, let me know 😄. Thank you for the time to review this PR!


module/cpufreq: Add a contextmanager for temporary governor changes

We may sometime want to temporarily use another governor, and then
restore whatever governor was used previously - for instance, RT-app
calibration (1) needs to use the performance governor, but we don't
want this to interfere with e.g. our current experiment.

Added a try/finally around the yield.


Automatic dependency upgrades

We should look into dependabot or greenkeeper as a way to keep our dependencies updated. Any tool that opens PRs using new versions of dependencies would be a good fit.

I’ve configured Travis to automatically do a build once a week so ember-try can tell us if future versions of ember break ember-sortable.


BUGFIX: Fix realtime indexing of node deletion

See https://github.com/ttreeagency/Flowpack.ElasticSearch.ContentRepositoryQueueIndexer/issues/14 for details.

See https://github.com/ttreeagency/Flowpack.ElasticSearch.ContentRepositoryQueueIndexer/issues/14#issuecomment-312013670 for the open issues with this approach…

So here we are, the fake node data are generated, currently the CLI build command is broken, but can be solved.
Not fully tested because midnight ... but at least my error log stay silent ;)
Now we need to decide what we do when the nodedata is not available in the IndexingJob, that can happens because of ordering, or during bulk indexing when the queue is huge. My current (post midnight) solution is to generate a RemovalJob for this missing NodeData so we are sure that it's removed from the index (better multiple removal, than no removal)

Test and review highly welcome

Hi @dfeyer - i want to test this PR and have a question regarding a change from @kdambekalns.
I suspect the following line, to ignore removed Nodes, as they are not found with the NodeRepository and therefore would not be instances of "NodeData". What do you think?
Did you check, debug.... this?
-> https://github.com/kdambekalns/Flowpack.ElasticSearch.ContentRepositoryQueueIndexer/blob/e3195b0e512938c08d706854978f4eccba6ce91a/Classes/IndexingJob.php#L33

@peterbucher Yes, the RemovalJob is unaffected by those lines. IMO they are still useful as a safeguard in the IndexingJob, though.

@peterbucher The RemovalJob also still has this check… It does no harm, but I'm not sure it would ever be needed (since the NodeDataRepository should return only NodeData instances.
@dfeyer did you check that?

@kdambekalns NodeDataRepository can return NULL, which is not a instance of NodeData. So i think this check does matter.
It seems to be better now in case of stability (background process) @dfeyer but, as far as i can see, the nodes (elements) and pages are not removed in the index. If i change some text within a text element, its updated. If i remove it, it will no longer be found, but it is still in the full text field of the corresponding page  node. But if i delete a page, its still in the index, but the Search result page does not display it, as there are no corresponding nodes in the database.
This leads to a wrong result which e.g. with Flowpack.SearchPlugin tells me, there is 1 Result found, but no result is displayed.

I did some cleanup in the codebase and will push a follow-up. I'll try to test this and see if I can confirm what @peterbucher observed, as well.

This leads to a wrong result which e.g. with Flowpack.SearchPlugin tells me, there is 1 Result found, but no result is displayed.

Need more debugging, will not have time this week to test this more. Hope @kdambekalns founds something, if not I try to work on it next week.

What's the current status, @kdambekalns ?

Ok, (content and document) node removal seems to still be handled by IndexingJob, not RemovalJob. That leads to action=indexing step=failed node=… message="Node data could not be loaded" and thus the index is not updated correctly. I'll dig further…

The cause is the way signals and slots are wired. When a node is removed, the NodeIndexingManager is tasked with removeNode(), that's fine. But during publishing, only the afterNodePublishing signal is emitted, wired to indexNode(). So the removal is not "seen" by the indexing code.

When adding code to divert removals from within indexNode()´ to removeNode(), the log shows removal operations being done. The fulltext is not updated correctly, though. The log shows No fulltext root found for node …` entries.
So right now I assume the "fake nodes" we build do not return their parent on getParent() and thus the update fails…

It might be that there is something else going wrong as well. When content node is removed, the log shows Could not index node with identifier …, not found in workspace live and when publishing the same identifier appears in Removed node … from index. But the related fulltext (root) is not updated…

Discussed to topic yesterday with @kdambekalns has we fake the node, this node does not have any parent ;) so we need to fake a bit more to have the full text index rebuild correctly.


MicrosoftSqlServerVersion to capture supported versions and their properties

Rationale
An enum that captures all supported versions of Microsoft SQL Server and their properties (reported version number, tested, deprecated, allowed as a primary data source, dialect factory to use, etc.) is easier to manage than version-specific if statements. See PostgreSqlVersion as the model.

Old dependencies sqlserver suite (running 2014) is all green as well: https://teamcity.labkey.org/viewLog.html?buildId=2109787&buildTypeId=LabKey_2211Release_Internal_InternalSuites_OldDependenciesSqlserver&tab=buildResultsDiv


[@azure/arm-eventhub] Error thrown when listing namespaces in a subscription with no Event Hub namespaces

Package Name: @azure/arm-eventhub (also happens on @azure/arm-servicebus)
Package Version: 0.1.0
Operating system: Ubuntu 18.04
[x] nodejs

version: 8.14.0

[x] typescript

version: 2.9.2 (minimal repro written in Javascript)

Is the bug related to documentation? - No

Describe the bug
When calling the client.namespaces.list() operation on a client pointed at a subscription with no Event Hubs namespaces, the following error is thrown, rather than an empty array being returned:
TypeError: (_response.parsedBody || []).slice is not a function    at flattenResponse (/mnt/c/Users/jepri/scratch/node_modules/@azure/ms-rest-js/dist/msRest.node.js:3518:62)
    at /mnt/c/Users/jepri/scratch/node_modules/@azure/ms-rest-js/dist/msRest.node.js:3353:47
    at <anonymous>    at process._tickCallback (internal/process/next_tick.js:189:7)

An identical error is thrown when running the equivalent operation in the @azure/arm-servicebus package.
To Reproduce
Steps to reproduce the behavior:

Save this script to a file (repro.js):
const msRestNodeAuth = require('@azure/ms-rest-nodeauth');
const { EventHubManagementClient } = require('@azure/arm-eventhub');

const subscriptionId = process.argv[2];

async function main() {
    const creds = await msRestNodeAuth.interactiveLogin();
    const client = new EventHubManagementClient(creds, subscriptionId);
    return client.namespaces.list();
}

main()
    .then(console.log)
    .catch(console.error);

Run the following, passing a subscription id for a subscription with no Event Hubs namespaces.
node repro.js <subscriptionId>

When passing a subscription with no Event Hubs namespaces, the following error is thrown:
TypeError: (_response.parsedBody || []).slice is not a function    at flattenResponse (/mnt/c/Users/jepri/scratch/node_modules/@azure/ms-rest-js/dist/msRest.node.js:3518:62)
    at /mnt/c/Users/jepri/scratch/node_modules/@azure/ms-rest-js/dist/msRest.node.js:3353:47
    at <anonymous>    at process._tickCallback (internal/process/next_tick.js:189:7)

However, when invoking the same script with a subscription that contains one or more Event Hubs namespaces, an array with the expected results is returned.

Expected behavior
The API should return an empty array.
Additional context
A similar issue was previously reported for the older versions of these packages (https://github.com/Azure/azure-sdk-for-node/issues/4453). When I looked into that problem, it appeared to be an issue with the way the mapper was working, so I'm guessing this is probably due to a similar problem.

Thanks for reporting @princjef and apologies for not responding sooner.
@daschult Can you direct this issue to the right people?

/cc @daschult

@daschult, @kpajdzik - This seems like a serialization bug in ms-rest-js

Looks like this issue has been resolved. Tried it on my box with a subscription having no eventhub namespaces and everything works fine
import { AzureCliCredentials } from "@azure/ms-rest-nodeauth";
import { EventHubManagementClient } from "@azure/arm-eventhub";

async function main(): Promise<void> {
  try {
    const subscriptionId = "00977cdb-163f-435f-9c32-39ec8ae61f4d"
    const creds = await AzureCliCredentials.create({ subscriptionIdOrName: subscriptionId });
    const client = new EventHubManagementClient(creds, subscriptionId);
    const result = await client.namespaces.list();
    console.dir(result._response, { depth: null, colors: true });
    console.log(">>>>>>>>>>>>>>>>>>>>>>");
    console.log(result);
    console.log(Array.isArray(result));
  } catch (err) {
    console.log(err);
  }
}

main();

amarz-MBP:tmp amarz$ ts-node ./eh.ts
{ request:
   WebResource {
     streamResponseBody: false,
     url:
      'https://management.azure.com/subscriptions/00977cdb-163f-435f-9c32-39ec8ae61f4d/providers/Microsoft.EventHub/namespaces?api-version=2017-04-01',
     method: 'GET',
     headers: [HttpHeaders],
     body: undefined,
     query: undefined,
     formData: undefined,
     withCredentials: false,
     abortSignal: undefined,
     timeout: 0,
     onUploadProgress: undefined,
     onDownloadProgress: undefined,
     proxySettings: undefined,
     operationSpec: [Object] },
  status: 200,
  headers: HttpHeaders { _headersMap: [Object] },
  readableStreamBody: undefined,
  bodyAsText: '{"value":[]}',
  parsedBody: [ value: [] ] }
>>>>>>>>>>>>>>>>>>>>>>
[ nextLink: undefined ]
true

package.json
{
   . . .
   "dependencies": {
      "@azure/arm-eventhub": "^3.2.0",
      "@azure/ms-rest-nodeauth": "^2.0.2",
    }
}

Thanks @amarzavery!
@princjef Please try the latest version of @azure/arm-eventhub where this issue is no longer occuring.


Modifying Footer

Feature request-
Adding newsletter feature in the footer. It will be responsive.
Is your feature request related to a problem? Please describe. (optional)
A clear and concise description of what the problem is.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Adding newsletter will make user to get all the updates at time. which will make website more beneficial
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Describe how this feature will be useful to our readers.
A clear and concise description of how this feature will be useful for our  other readers.
Readers will get regular updates through newsletter
Additional context
Add any other context or screenshots about the feature request here.

Note:

Do mention if your are working under GSSoC'21.
If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.
According to the level of the issue, respective time will be allotted under which the task needs to be completed/resolved by the assigned person. If the issue is not resolved within the given time, it will be assigned to the next person. In case of any discrepancies, the decision of the maintainer will be final.

level - 0: 2 days
level - 1: 3 days
level - 2: 4 days
level - 3: 5 days

/assign

I want to work as a GSSOC participant

Footer doesn't have that much space, maybe a separate page all about signing up for the newsletter would be great. Also can i get some more info regarding this issue like your approach to implement the newsletter feature so that I can help if you make any mistake.

repeated issue #276


Cancel button is showing up below filter boxes.

I am having trouble trying to figure out why the "x" button is showing up under these filter boxes.

There is an inner wrapper

Then there is a wrapper that is around both lines.

This is my call to the columns

And this is the result.

Any help would be appreciated.
Thanks, Justin

Looks like you forgot to include yadcf css file


Python script to generate (mostly) complete IPA steps

A python script to run oneshot_ipa benchmark for all combinations of IPA options to generate all the possible steps, sort them and print them out. The output will be used to codegen the optimized Steps implementation.
This script basically executes the following command:
RUST_LOG=ipa=DEBUG cargo bench --bench oneshot_ipa --no-default-features --features="enable-benches debug-trace step-trace" -- --num-multi-bits 1 -n 10 -c $1 -w $2 -b $3 -m $4
where options that affect IPA protocol paths are:

-c PER_USER_CAP = [1, 3]
-w ATTRIBUTION_WINDOW = [0, 86400]
-b BREAKDOWN_KEYS = [1, 33]
-m SECURITY_MODEL = ["malicious", "semi-honest"]

The output is not 100% complete because there are protocols that change the number of narrows based on the number of bits or rows that are processed. However, the limits are well bounded and will be less than 64 or lower for both (log(100B) < 37). For these missing steps, it's not realistic to generate them by actually running IPA. Instead, we'll generate them using macros, or maybe some other primitive methods during codegen time.
Output
❯ python3 ./scripts/collect_steps.py | wc -l
    Finished bench [optimized] target(s) in 1.02s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.52s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.40s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.48s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.42s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.41s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.44s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.87s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.88s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 1.05s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.41s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.71s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.42s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 1.13s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 0.39s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    Finished bench [optimized] target(s) in 1.20s
     Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)
    8740

Finished bench [optimized] target(s) in 0.52s
Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)

this looks a bit noisy, do these come from stderr? we should probably suppress them too

Finished bench [optimized] target(s) in 0.52s
Running benches/oneshot/ipa.rs (target/release/deps/oneshot_ipa-ff953e773cf9b04c)

this looks a bit noisy, do these come from stderr? we should probably suppress them too

Yes they are from stderr. I kinda like it because it let's me see the progress, and any errors if they occur.