Replace ad-hoc envar "protocol" with something more structured

Currently we utilise a random set of environment variables such as FTL_CONFIG, FTL_ENDPOINT, LOG_LEVEL, etc. to propagate configuration state between the runner and the module it is executing. We should come up with something less gross.
One option would be to double down on using envars for this, but have a data structure that encapsulates it all. Another option would be to push some of this back to the Runner over gRPC. That is to read a config key or secret key, the user code calls a gRPC method on the Runner.

Just adding triage back to this one because someone needs to take this, so we should discuss next week.

Stupid bot.

design doc
have a single get into ftl from user code that returns structured data for all of this

Doc: https://hackmd.io/@ftl/SyynEK2eC

@matt2e is this issue related to this work here? Or something else https://github.com/TBD54566975/ftl/issues/919

That one is more related to pulling config/secrets from different sources in production - ie. a DB table for config, and AWS Secrets Manager for secrets. The plumbing is there in the config resolvers, we just need implementations for them.


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.2.4
Time: Thu May 19 2016 20:47:05 GMT 0100 (GMT Summer Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\bjusz\AppData\Local\GPMDP_3\app-3.2.4\resources\app\build\inject\GPMWebView\interface\customNavigation\mouseButtonNavigation.js:27:9.
    at BrowserWindow.callIntoRenderer (C:\Users\bjusz\AppData\Local\GPMDP_3\app-3.2.4\resources\electron.asar\browser\rpc-server.js:193:19)
    at emitTwo (events.js:105:20)
    at BrowserWindow.emit (events.js:185:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #1045


update documentation about vsecm sentinel init file usage

Closing. There are use case docs about it already.


nimble install: system module needs string

Hi there,
I wanted again to try Karax with Nim 0.17.0 and a "nimble install" at the project root gives me
$ nimble install                                                           *[master]
     Info Hint: used config file '/etc/nim.cfg' [Conf]
     Error: Could not read package info file in /home/vince/bacasable/karax/karax.nimble;
        ...   Reading as ini file failed with:
        ...     Invalid section: .
        ...   Evaluating as NimScript file failed with:
        ...     Error: system module needs 'string'.

If it is not necessary to do that (as discussed in https://github.com/pragmagic/karax/issues/2), then I have the same errors as discussed here https://github.com/pragmagic/karax/issues/6#issuecomment-299133321
meaning with nim js todoapp.nim:
/home/vince/bacasable/karax/src/components.nim(82, 13) Error: type mismatch: got (NimNode, proc (bl: typed, op: string): NimNode{.noSideEffect.}, void)
but expected one of:
proc add(x: var cstring; y: cstring)
proc add[T](x: var seq[T]; y: openArray[T])
proc add(x: var string; y: cstring)
proc add[T](x: var seq[T]; y: T)
proc add(x: var string; y: string)
proc add(x: var string; y: char)
proc add(parent, kid: VNode)
proc add(c: ClassList; class: cstring)
proc add[A, B](t: var OrderedTable[A, B]; key: A; val: B)
proc add(father, child: NimNode): NimNode
proc add[A, B](t: TableRef[A, B]; key: A; val: B)
proc add(father: NimNode; children: varargs[NimNode]): NimNode
proc add[A, B](t: var Table[A, B]; key: A; val: B)
proc add[A, B](t: OrderedTableRef[A, B]; key: A; val: B)

Regarding the second error: Are you running this from within the Karax repo or from a different place? It looks like for the time being you need a .nims file which points --path to your local karax/src directory. Otherwise I'm getting this error as well just from a plain include karaxprelude.

I am running this from within the Karax repo (karax/examples/todoapp or just karax/).

Strange, it works for from within the repo (using latest Nim devel). Maybe if you're on Windows, you have to change the path separator in the nims file?

I'm on Debian.
(btw if you want me to try something with a .nims file and the --path please explain, your precedent comment is cryptic to me :] )

The .nims allows to specify an additional search path for the compiler. I think this was needed because of the dom naming conflict of the dom module in the Nim standard library and Karax's extended dom module. Now that Karax's version is renamed to kdom I think this is no longer needed.
You are using the latest master of Karax and latest devel of Nim and there is still the issue from above?

I got the latest master and it's beautiful now :) Very exciting.


[JENKINS-37422] fixed compatibility with Ruby Runtime Plugin

@sheehan I did some changes on the API viewer in https://github.com/jenkinsci/job-dsl-plugin/pull/906/commits/03d8d70cb886dfa5689fbeb537648fa6fb3985f7 to support multiple plugins per signature. Can you have a look?

👍 looks good to me

@sheehan thanks for the review!

Any chance of getting a release with this fix in?
Cheers


Error: unsupported macvlan network option mtu

When running the e2e tests with netavark, we get a failure with Error: unsupported macvlan network option mtu.  See https://github.com/containers/podman/blob/main/test/e2e/network_test.go#L574

https://github.com/containers/common/blob/97814789b1bb4fac1a2ddaebcaf9873c457f4625/libnetwork/netavark/config.go#L127-L137

I will open a PR to fix it in c/common if you add support to netavark.

@Luap99 Sure I'll add it to netavark. We have to stop blocking it from c/common/ thanks.

https://github.com/containers/common/pull/881


Rails 5 - undefined method 'define_index'

I am not sure if the project is still alive. I try to upgrade my old Rails3 app to Rails 5. And I have a problem with undefined "define_index" method. :(
$ rails ts:configure
Generating configuration to /Users/hipertracker/apps/myapp/config/development.sphinx.conf
rails aborted!
NoMethodError: undefined method `define_index' for Verse (call 'Verse.connection' to establish a connection):Class
Did you mean?  defined_enums
/Users/hipertracker/apps/myapp/app/models/verse.rb:8:in `<class:Verse>'
/Users/hipertracker/apps/myapp/app/models/verse.rb:1:in `<top (required)>'
/Users/hipertracker/apps/myapp/app/indices/verse_index.rb:2:in `block in <top (required)>'
/Users/hipertracker/apps/myapp/bin/rails:9:in `require'
/Users/hipertracker/apps/myapp/bin/rails:9:in `<top (required)>'
/Users/hipertracker/apps/myapp/bin/spring:13:in `<top (required)>'
bin/rails:3:in `load'
bin/rails:3:in `<main>'
Tasks: TOP => ts:configure
(See full trace by running task with --trace)

Gemfile:
gem 'rails', '~> 5.0.0'
gem 'mysql2', '>= 0.3.18', '< 0.5'
gem 'thinking-sphinx', github: 'pat/thinking-sphinx'
#...

app/models/verse.rb:
class Verse < ActiveRecord::Base
  belongs_to :lang
  define_index do
    indexes verse
    has :lang_id
  end
end

app/indices/verse_index.rb
ThinkingSphinx::Index.define :verse, :with => :active_record do
  indexes text
  has :lang_id
end

config/thinking_sphinx.yml
development: &shared
  mem_limit: 512M
  min_word_len: 2
  #enable_star: true
  min_infix_len: 3
  html_strip: true
  charset_table: 0..9, A..Z->a..z, a..z, U+0143->U+0144, U+0104->U+0105, U+0106->U+0107, U+0118->U+0119, U+0141->U+0142, U+00D3->U+00F3, U+015A->U+015B, U+0179->U+017A, U+017B->U+017C, U+0105, U+0107, U+0119, U+0142, U+00F3, U+015B, U+017A, U+017C, U+0144, U+41..U+5a->U+61..U+7a, U+61..U+7a, U+aa, U+b5, U+ba, U+c0..U+d6->U+e0..U+f6, U+d8..U+de->U+f8..U+fe, U+df..U+f6, U+f8..U+ff, U+100..U+12f/2, U+130->U+69, U+131, U+132..U+137/2, U+138, U+139..U+148/2, U+149, U+14a..U+177/2, U+178->U+ff, U+179..U+17e/2, U+17f..U+180, U+181->U+253, U+182..U+185/2, U+186->U+254, U+187..U+188/2, U+189..U+18a->U+256..U+257, U+18b..U+18c/2, U+18d, U+18e->U+1dd, U+18f->U+259, U+190->U+25b, U+191..U+192/2, U+193->U+260, U+194->U+263, U+195, U+196->U+269, U+197->U+268, U+198..U+199/2, U+19a..U+19b, U+19c->U+26f, U+19d->U+272, U+19e, U+19f->U+275, U+1a0..U+1a5/2, U+1a6->U+280, U+1a7..U+1a8/2, U+1a9->U+283, U+1aa..U+1ab, U+1ac..U+1ad/2, U+1ae->U+288, U+1af..U+1b0/2, U+1b1..U+1b2->U+28a..U+28b, U+1b3..U+1b6/2, U+1b7->U+292, U+1b8..U+1b9/2, U+1ba..U+1bb, U+1bc..U+1bd/2, U+1be..U+1c3, U+1c4->U+1c6, U+1c5..U+1c6/2, U+1c7->U+1c9, U+1c8..U+1c9/2, U+1ca->U+1cc, U+1cb..U+1dc/2, U+1dd, U+1de..U+1ef/2, U+1f0, U+1f1->U+1f3, U+1f2..U+1f5/2, U+1f6->U+195, U+1f7->U+1bf, U+1f8..U+21f/2, U+220->U+19e, U+221, U+222..U+233/2, U+234..U+23a, U+23b..U+23c/2, U+23d->U+19a, U+23e..U+240, U+241->U+294, U+250..U+2c1, U+2c6..U+2d1, U+2e0..U+2e4, U+2ee, U+1d00..U+1dbf, U+1e00..U+1e95/2, U+1e96..U+1e9b, U+1ea0..U+1ef9/2, U+37a, U+386..U+389->U+3ac..U+3af, U+38c..U+38e->U+3cc..U+3ce, U+390, U+391..U+3a1->U+3b1..U+3c1, U+3a3..U+3ab->U+3c3..U+3cb, U+3ac..U+3ce, U+3d0..U+3d7, U+3d8..U+3ef/2, U+3f0..U+3f3, U+3f4->U+3b8, U+3f5, U+3f7..U+3f8/2, U+3f9->U+3f2, U+3fa..U+3fb/2, U+3fc..U+3ff, U+400..U+40f->U+450..U+45f, U+410..U+42f->U+430..U+44f, U+430..U+45f, U+460..U+481/2, U+48a..U+4bf/2, U+4c0, U+4c1..U+4ce/2, U+4d0..U+4f9/2, U+500..U+50f/2, U+5D0..U+5EA, U+5F0..U+5F2, U+5BE, U+5C0, U+5C3, U+5F3, U+5F4, U+621..U+63a, U+640..U+64a, U+66e..U+66f, U+671..U+6d3, U+6d5, U+6e5..U+6e6, U+6ee..U+6ef, U+6fa..U+6fc, U+6ff
  indexed_models:
    - Verse

test:
  <<: *shared

production:
  <<: *shared

stage:
  <<: *shared

All is OK. define_index is gone and replaced by indices/verse_index.rb file. I had a typo in the column (verse vs text).

Good to know you've got it working :) Yes, define_index is only part of TS v1/v2, all v3 releases use the app/indices approach instead. And yup, the project's still active, though new features aren't too common these days.


Update REMOVE_ALL_MODULES to handle ''

When defining a class inside a function or method, the qualified name of that class includes <locals>. This PR adds this case to the tests, and it updates the REMOVE_ALL_MODULES regex to remove it.

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

Project coverage is 98.4%. Comparing base (9d939a3) to head (0c62283).

Additional details and impacted files
@@          Coverage Diff          @@
##            main      #4   +/-   ##
=====================================
  Coverage   98.4%   98.4%
=====================================
  Files          1       1
  Lines        199     199
=====================================
  Hits         196     196
  Misses         3       3

Files
Coverage Δ

typenames.py
98.4% <100.0%> (ø)


added output version of migration for more details in logs

It would be nice to get the version outputted to the logs. But before this request can be merged in, all the tests have to pass. This change breaks 10 tests in MigrationRunnerTests. The test migration classes do not have migration attributes which the new code checks for and throws an exception. Can you change that, please.

Ok, I'll look tests.

@tabushi You still on this ?
I like this feature :)

There were no free time.
I'll see the tests in nearest day

@tabushi thanks :)

This is not logical to transfer the method migration and version migration, which is already contained in the attribute of the first argument.

@tabushi It doesn't seem DRY to me, IMigration does not contain a version property, you need to reflect again to get the version, the migrationloader seems to only class that is responsible of this
It's just my oppinion

I suggest closing this pull

Do whatever you want.

@tabushi Are you ok with changes done to add version number to output #230 ?


Alternate method using inline css to hide regular html output in an untrusted notebook

When using xarray in a jupyterlab notebook environment with lab extensions loading external css libraries (e.g., bootstrap), the hidden html5 attribute applied to the 'xr-wrap' div element is actually respected and cannot be escaped using a display change in the xarray included css.
The html hidden attribute was weakly enforced in some browsers, but bootstrap added this rule to ensure it took effect as early as v4.0:
https://github.com/twbs/bootstrap/blob/v4.0.0/scss/_reboot.scss#L480-L482
This means that not only does the normal html content inside the xr-wrap div not display in a trusted notebook environment that has this type of css rule applied from an external library, the fallback text display mechanism in the pre element does not kick in because the css block is present and sets that content to 'display: none' and so then you just get blank output in the cell when trying to display a Dataset, which is not ideal.
This PR addresses the above problem and maintains a fallback display mechanism when in an untrusted notebook.  In a trusted environment, the css will be loaded and the inline css for 'xr-wrap' of 'display:none' will be overridden by the embedded css style block of 'display: block !important', displaying the expected html.  In an untrusted environment, the css block will be removed (not hiding the pre element) but the inline style will remain (hiding the 'xr-wrap' div element), leaving the pre text to load instead.

[x] Passes pre-commit run --all-files
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

I looked at the other issues, and this should resolve the sphinx-book issue in https://github.com/executablebooks/sphinx-book-theme/issues/238#issuecomment-714784321 without needing a special exception for that codebase.
For https://github.com/pydata/xarray/pull/4053:
Github rendered notebooks will still not look great because it will strip the inline css of the xr-wrap div (and all css).  All of the actual styling (placement, color, etc) for that html block is in the css style block that gets stripped.  Fixing that is a different problem from the untrusted notebook case, which is less aggressive, although that may not be the case in the future.  You would need to render the content in a different format (maybe an image) for the saved notebook to get around losing css entirely.

Hi @mlhenderson — I don't know this well — if you are confident that @benbovy 's concern is not pressing, then we could merge? (unless @benbovy you are confident that it is a concern!)

Unfortunately, I can't predict what the Jupyter project could do in the
future in terms of stripping inline CSS or the html hidden attribute.  I
don't get the sense that this would be happening anytime soon, but I can't
promise it would never happen.
I sent in the PR because I am an author of a JupyterLab extension that we
plan to use at NERSC, which includes Bootstrap, and we ran into the problem
of the xarray repr() returning a hidden div (blank output) in a regular
notebook (trusted).  Any JupyterLab extension that includes Bootstrap or
has a css rule that affects how the browser treats the html hidden
attribute will have similar problems with xarray in notebooks.  It seemed
appropriate to flag it and send in a fix before it becomes a bigger problem.
There is an existing test in xarray (which passes) that checks for the css
content.  If you wanted to add another test that actually spins up a
Jupyter instance to test it further, that is possible but definitely more
involved and may not be a fast test to run.
I think this could be merged as-is, but feedback is always welcome.  If
there are any suggestions for improving this PR further to resolve
the existing issue please let me know.
Longer-term, embedding html/css may not be the way to go, but I think that
falls outside the scope of this PR.  We can arrange to chat with some
Jupyter devs if there is interest in a discussion about longer-term
solutions for the rendering in notebooks and/or JupyterLab, just let me
know if you want to do that. @benbovy
On Fri, Nov 12, 2021 at 2:26 PM Maximilian Roos @.***>
wrote:

Hi @mlhenderson https://github.com/mlhenderson — I don't know this well
— if you are confident that @benbovy https://github.com/benbovy 's
concern is not pressing, then we could merge? (unless @benbovy
https://github.com/benbovy you are confident that it is a concern!)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/pull/5880#issuecomment-967701381, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVNKLNE6NK653KHEUYDX5LULWICVANCNFSM5GMKPTKA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Many thanks for your thoughtful reply @mlhenderson . It seems like the proposed code is better than the current code, so I would suggest we merge, and we can iterate further if others have thoughts.
And thanks for your first contribution !

Thanks @mlhenderson!

Longer-term, embedding html/css may not be the way to go, but I think that
falls outside the scope of this PR.  We can arrange to chat with some
Jupyter devs if there is interest in a discussion about longer-term
solutions for the rendering in notebooks and/or JupyterLab, just let me
know if you want to do that. @benbovy

Yeah, now that (since v3) Jupyterlab extensions can simply be installed using conda or pip, we may consider eventually porting this rich-repr feature into a lightweight extension instead. That would be a more robust and extensible solution. The main problem, however, is how to support the variety of notebook front-ends. There seems to be a recent agreement on refactoring Jupyter "official" front-ends (e.g., "classic" notebook) so that they all use Jupyterlab's current machinery, which will certainly help here. Not sure for the other front-ends (e.g., VSCode, GitHub, GitLab, Sphinx themes, etc.), though. Maybe we could have three repr alternatives:

The full-featured, interactive version: compatible with all official Jupyter front-ends, maintained in a 3rd-party Jupyter extension
A basic, html-based version (as fallback): less interactive but should work (and be nicely integrated) with all notebook front-ends without relying on ugly hacks (no css, just basic html tags)
The plain text version

I agree that it would be worth discussing this general issue with Jupyter devs. There's more and more projects in the Python/PyData ecosystem that provide advanced, html-based reprs.


There is almost no client unit test coverage.

Fix it. Mock the http server and do round trip like tests for the http client. Similar for nats.

There is some now! It could get better.


List of Formats, Libraries and plugins

Post links here for (open source) libraries and plugins that could be used in Unity (runtime) CAD Import and Export
Preferred solutions (in this order)

c# compatible libraries / frameworks (as sources or dll's)
native dll plugins with c# bindings (or possible pinvoke could be used)
external simple commandline converter tools (that could be used to convert from cad format(s) into some common formats (fbx, obj..)
source from other languages, that could be converted into c#

Common CAD formats that would need runtime importers (see more formats)

STEP (i'd consider this high priority, as its one of the most common interchange formats)
DWG
DXF (would be nice to have 2D layout imports too)
STL
IFC (would be nice to import meta data also)
IGES

I can't help much with libraries. I only remember that PIXYZ was one of the importers that Unity integrated for CAD. Check in the UnityCommunity repository, inside Importers/Exporters section.
Anyway, which is the purpose of this repository?

this would be for an open source runtime multiple cad format importers (and later some export formats could be added also).
Pixyz is definitely best solution at the moment (but of course its paid only).

stp

import https://github.com/anycad/StpViewer
import/export https://github.com/ixmilia/step
export https://github.com/drGsus/simple-step-exporter

iges, stp, stl, wrl, 3dxml

import https://github.com/i2e-haw-hamburg/cad-in-unity (quite old repo though)

ifc

https://github.com/xBimTeam/XbimEssentials

fbx export

https://github.com/KellanHiggins/UnityFBXExporter
https://github.com/unity3d-jp/FbxExporter
https://github.com/Ymiku/UnityFBXExporter

fbx import

https://github.com/mpreble/UnityFBXImporter
https://github.com/izrik/FbxSharp

fbx import/export

https://github.com/hamish-milne/FbxWriter
https://github.com/Unity-Technologies/com.unity.formats.fbx
https://github.com/Hoetan/FBXImporterExporterForUnity_20181015

fbx, maya export

https://github.com/newyellow/Unity-Runtime-Animation-Recorder

Currently working to convert IFC++ (http://www.ifcquery.com/) ( Github: https://github.com/ifcquery/ifcplusplus)
Over to C# May take a while.  Spent 6 hours last night on it.  Apparently one of the ways that he made it efficient was using pointers, so having to connect it back to regular variables is meh.  Will try to do pointers as well if @unitycoder can help with that once done.  Ill have to label which ones to make as pointers.  It def helps to speed it up.
Also reattaching these:
stp
import https://github.com/anycad/StpViewer
import/export https://github.com/ixmilia/step
export https://github.com/drGsus/simple-step-exporter
iges, stp, stl, wrl, 3dxml
import https://github.com/i2e-haw-hamburg/cad-in-unity (quite old repo though)
ifc
https://github.com/xBimTeam/XbimEssentials
fbx export
https://github.com/KellanHiggins/UnityFBXExporter
https://github.com/unity3d-jp/FbxExporter
https://github.com/Ymiku/UnityFBXExporter
fbx import
https://github.com/mpreble/UnityFBXImporter
https://github.com/izrik/FbxSharp
fbx import/export
https://github.com/hamish-milne/FbxWriter
https://github.com/Unity-Technologies/com.unity.formats.fbx
https://github.com/Hoetan/FBXImporterExporterForUnity_20181015
fbx, maya export
https://github.com/newyellow/Unity-Runtime-Animation-Recorder

unity assimp wrapper (40+ mesh formats, JT, IFC, STL..)
https://github.com/eastskykang/UnityMeshImporter

ifc dotnet
https://github.com/xBimTeam/XbimEssentials

tested step importing: (their sample file, or regular small step file)

(not working in unity) https://github.com/i2e-haw-hamburg/cad-in-unity
(not working in unity) https://github.com/ixmilia/step
(works in winform build, but requires license) https://github.com/anycad/StpViewer

STL import (using Rust plugin)
https://github.com/unitycoder/UnityRustStl

GLTF importer (exporter not implemented yet)
https://github.com/Siccity/GLTFUtility

FBX glTF importer/exporter plug-in and converter
https://github.com/cyrillef/FBX-glTF

I'm exploring runtime import and export options for WebGL builds in unity. I've reviewed options like Trilib and various glTF libraries, but they don't meet my requirements. I'm also looking into using Assimp in Unity WebGL. Could you suggest a straightforward and efficient solution? @unitycoder I have one more question does Pixyz supports run time import/export?

Pixyz supports run time import/export?

from what i've read, it doesn't work in webgl and that gltf would be the recommended for that..
(but not sure whats current status)
recently tested this https://github.com/eastskykang/UnityMeshImporter
for runtime fbx, but the mesh was broken in some parts..
then tried this https://github.com/atteneder/glTFast *this is apparently best glft importer, and it does read quite fast.
but the gltf mesh had more parts than the original fbx file and fails on large +3gb files)

Thank you so much


How to find the hospitals nearby along with the insurance card available in that hospital,Just like the search widget in cmv Template,how can i develop an app based on this requirement

Having  hospitals data along with insurance card category is available ,I would like to develop the app to find nearby locations to the current location based on the insurance card category  available over it.
i.e. Nearby Hospitals with condition of insurance category for current location

@pnh10 This app does not support nearby search capabilities.  Please post questions like these to our GeoNet forum for broader ArcGIS Android questions.


User export fails, actual error not exposed due to potential issue in error handling

We are seeing errors at different points when exporting users and only the following information is provided:
Error exporting Users: The property 'Message' cannot be found on this object. Verify  that the property exists.
This error seems to originate here. I think the error handling code might need to be more defensive in order to expose the relevant issue.

Can you try running with the -Debug parameter? This should show the exact Graph API call that is failing. Once you do that can you try running that same command using Graph Explorer and let me know if you are still seeing the issue?

Hey @merill , thank you for the swift reply. We run the export from a azure devops pipeline and usually it fails after an hour or two (the tenant has a lot of users to export) but when adding the Debug-switch it fails almost immediately (with the initially reported error) so it seems that there might be issues with setting Debug here (non-interactive shell?).
Any other ideas or does the error handling code need to check if the Message property exists?

Unfortunately I cannot use Graph Explorer yet in GCC-High. (Any idea when that will change?)

This issue is still open and I am curious if the root was solved or at least a reason was found.
I am facing the same issue that the exception handling is not able to give me a clear error while trying to get all users from an AAD tenant. Running the export with -Verbose only shows me
VERBOSE: GET https://graph.microsoft.com/beta/users?expand=extensions&$count=true&$skiptoken=m~AQAnO2U5Z4OzswOzA7 with 0-byte payload
VERBOSE: received 542616-byte response of content type application/json
So I don't know the reason why it is failing after ~300 GET commands.

I have fixed the error handling to provide more detailed messages.
Can you please retry and let me know if you are still seeing this issue?


Allow Cross-Linking for Dual-Role Users in Patient-Caregiver Relationships

Description:
When two users each have both PATIENT and CAREGIVER roles, CareLink allows only a one-directional link (e.g., linking John Smith as a patient to Sam Tan as a caregiver). However, attempting to create the reverse link (Sam Tan as a patient and John Smith as a caregiver) results in an error. This restriction prevents mutual caregiving relationships, which are often needed in real-world care settings.
Steps to Reproduce:

Add two users with both roles:
add n/John Smith nric/S9841776A p/91234567<EMAIL_ADDRESS>a/123 Bishan Street 12 #05-45 role/patient role/caregiver
add n/Sam Tan nric/S7382917H p/91029382<EMAIL_ADDRESS>a/Blk 123 Bukit Timah Street 32 #12-12 role/patient role/caregiver

Link John Smith to Sam Tan in a patient-caregiver relationship:
link patient/S9841776A caregiver/S7382917H

Attempt to create the reverse link (Sam Tan as patient, John Smith as caregiver):
link patient/S7382917H caregiver/S9841776A

Expected Behavior:
The system should allow the reverse link, supporting mutual caregiving relationships between two dual-role users.
Actual Behavior:
CareLink denies the second link attempt with an error, blocking reciprocal patient-caregiver relationships.
Impact:
This restriction limits CareLink’s flexibility in managing real-life scenarios where two individuals may support each other reciprocally in caregiving roles.
Suggested Solution:
Allow dual-role users to be linked in both directions as patient-caregiver and caregiver-patient without restriction.

This is actually intentional because I believed we could not have the patient and caregiver having dual roles for each other, especially since this is for elderly patients. However, I can still allow this behaviour if we believe it could be possible?


Queue Jobs lifecylce is different when run with supervisor

Laravel Version: 5.4.18
PHP Version: 7.0
Database Driver & Version: mysql 5.7

Description:
Queues that call other queues have a different lifecycle when run from command line and from supervisor. This results in the Parent queue being restarted too when the child queue fails if managed by supervisor.
I detected this behavior on a database backend for the queues, running with supervisor on the production server and from the command line on a local homestead environment.
Steps To Reproduce:
Create 2 Jobs (Parent and child), both implementing the ShouldQueue class on a database backend.
Dispatch the Parent job which dispatches the Child job itself.
//call the parent job somewhere in your code
dispatch(new Parent());

//in Parent.php:
public function handle() {
    //some code
    dispatch(new Child());
}

//in Child.php
public function handle() {
    throw new Exception('something went wrong', 500);
}

What happens:
taken from my logs you can see that the lifecycle is different when run:
Supervisor (command=php /www/my_project/artisan queue:work database --queue=my_queue):
[2017-06-26 17:14:07] Processing: App\Jobs\Parent
[2017-06-26 17:14:08] Processing: App\Jobs\Child
[2017-06-26 17:14:09] Processed:  App\Jobs\Child
[2017-06-26 17:14:09] Processed:  App\Jobs\Parent

From command line (php artisan queue:work database --queue my_queue):
[2017-06-26 17:14:20] Processed: App\Jobs\Parent
[2017-06-26 17:14:21] Processed: App\Jobs\Child

So what I understand out of this is that when run with supervisor the Child job lives inside the Parent job - meaning that if I fail the Child job, on supervisor the Parent job will also restart, executing the code twice.
What have I tried already:

Tried the DispatchesJobs trait on the Parent job to dispatch the Child job
Tried running the Child job on a different queue
Adding a delay to the child job

but these attempts did not give me any joy...

My first thought; are you running several queue workers with supervisor? Do you have numprocs=1? Your output looks like two queue workers both outputting to the same log file.

meaning that if I fail the Child job, on supervisor the Parent job will also restart, executing the code twice.

Did this happen to you already or you're just assuming this would happen? I don't seem to find an issue I can investigate here, things look ok to me from the details you shared.

@themsaid This did happen to me.
If I find the time later today I will check the numprocs attribute in the supervisor config suggested by @sisve - I do have numprocs=2 right now, so I want to make sure if that is causing this behavior or not.

I don't think so, dispatching a job from within another doesn't link the two jobs together in anyway, they're treaded as two separate jobs.
However, if the parent failed after it dispatched the child job, the child job will be dispatched again on retry.

Closing for lack of activity, please feel free to ping me if you have further findings.

Hi @themsaid
I just updated the supervisor config file with numprocs=1, but the behaviour seems to stay the same.

[2017-07-14 16:27:18] Processing: App\Jobs\Parent
[2017-07-14 16:27:19] Processing: App\Jobs\Child
[2017-07-14 16:27:25] Processed:  App\Jobs\Child
[2017-07-14 16:27:25] Processed:  App\Jobs\Parent

I added sleep(5); to the Child job to confirm that the flow remains the same since it was stated that the parent and child jobs aren't linked together in any way, but as you can see in the above output the child job waits 5 seconds before finishing and the parent job still waits for the child job to finish too.
Maybe to avoid all of this behavior it could be a good idea to add some sort of 'after' or 'complete' method to the Job class to make sure that the Child Job is only started after the Parent Job completed.

That output only makes sense if you're running with the sync queue driver.

Ok, @sisve You were right!
After a whole lot of local testing, sandboxing, doubleckecking and comparing with the live system, I found that in the .env file the queue driver was set to sync.
Strange that everything worked as expected with the database.
This totally restored my faith in the supervisor system :)

@sisve is the man :)


Remove row from Nav. Table

Hello dears ,
I have an a little issue with the Nav Menu
Suppose i have two viewControllers A & B
how to remove A VC from table when i'm in a B vc , and how to update the table to hide it from table

I do not really understand the question...


Add example angular factory

Moved to https://github.com/antonsamper/gulp-boilerplate-with-angular/issues/2


[BLAS] --offload-arch required in both compilation and linking

Description
Building SYCL-BLAS backend with AMD gpu support requires a slight tweak to config, since a different command-line argument is needed for both compilation and linking.
Checklist
All Submissions

[x] Do all unit tests pass locally? Log attached: amd.txt

[x] Have you formatted the code using clang-format?

Do you know why almost all of the tests are skipped in your test log? It seems there is a missing - in the target, not sure if this is related.

Good question - I know we skip a lot of tests because some things aren't implemented but I didn't pay attention to how many. Let me take a look.

Based on very rough counts with grep, there are about 32 APIs implemented in src/blas/backends/syclblas. They run for CT and RT linking and float and double precisions (only column major layout, no complex precisions). So around 128 permutations, and I see 118 passed tests in the log, which seems pretty close to what we would expect?

Ah yes that makes sense. I thought more permutations were supported but haven't followed the progress on SYCL-BLAS recently. Thanks for checking!


hackgreenville-com-436 Update laravel and add composer dev command

added composer dev

@allella, sorry I didn't see the message until now! It's a new command from Taylor Otwell, included in the latest version of laravel
You can see the new command here.
https://laravel-news.com/laravel-11-28-0

@zach2825 alright.
Should we update our docs to encourage the use of composer dev over the existing yarn dev & php artisan serve?

Thank you, @allella. I missed that! I also moved the key:generate above the composer dev section since it is a prerequisite.


Pull requests?

Thank you for your repository, it is helping me to transition from EasyMapping to Codable.
I bumped into an issue and made a fix.
Do you accept pull requests on this repo?

Sure, just send me a pull request and I’ll review it! :)

Thank you, see #4 .

Merged :)


v12.6.0 Cloud Function Authentication Triggers - no longer working

[REQUIRED] Step 2: Describe your environment

Operating System version: OS Name	Microsoft Windows 11 Home Version 10.0.22631 Build 22631
Firebase SDK version: _____
Firebase Product: cloud functions (auth, database, storage, etc)
Node.js version:  20
NPM version: 10.8.2

[REQUIRED] Step 3: Describe the problem
The documentation for the auth triggers is no longer valid
https://firebase.google.com/docs/auth/extend-with-functions
src/onUserCreated.ts:8:45 - error TS2339: Property 'user' does not exist on type '(app?: App | undefined) => Auth'.

8 export const onUserCreated = functions.auth.user().onCreate((user:any) => {

Steps to reproduce:
Using v12.6 of the firebase-functions package the below trigger is invalid:
// The Cloud Functions for Firebase SDK to create Cloud Functions and set up triggers.
import * as functions from 'firebase-functions';
// Start writing functions
// https://firebase.google.com/docs/functions/typescript

export const onUserCreated = functions.auth.user().onCreate((user:any) => {
 //TODO: Create avatar on user create and send verification email
  console.info(new Date())
  console.info(user)
});

I am currently refactoring to use a blocking function - only because the project is new and I can afford to do this

// The Cloud Functions for Firebase SDK to create Cloud Functions and set up triggers.
import {
  beforeUserCreated,
} from "firebase-functions/v2/identity";
// Start writing functions
// https://firebase.google.com/docs/functions/typescript

export const onUserCreated = beforeUserCreated((event:any) => {
  //TODO: Create avatar on user create and send verification email
  console.info(new Date())
  console.info(event)
});

What happened? How can we make the problem occur?
Run firebase init and try to set up a cloud function with an auth trigger as described in the documentation.
Relevant Code:
package.json

{
  "name": "functions",
  "scripts": {
    "lint": "eslint --ext .js,.ts .",
    "build": "tsc",
    "build:watch": "tsc --watch",
    "serve": "npm run build && firebase emulators:start --only functions",
    "shell": "npm run build && firebase functions:shell",
    "start": "npm run shell",
    "deploy": "firebase deploy --only functions",
    "logs": "firebase functions:log"
  },
  "engines": {
    "node": "20"
  },
  "main": "lib/index.js",
  "dependencies": {
    "firebase-admin": "^12.6.0",
    "firebase-functions": "^6.0.1"
  },
  "devDependencies": {
    "@typescript-eslint/eslint-plugin": "^5.12.0",
    "@typescript-eslint/parser": "^5.12.0",
    "eslint": "^8.9.0",
    "eslint-config-google": "^0.14.0",
    "eslint-plugin-import": "^2.25.4",
    "firebase-functions-test": "^3.1.0",
    "typescript": "^4.9.0"
  },
  "private": true
}

Used //@ts-expect-error and it cleared away - not sure what's the issue there

That has just led to this error:
TypeError: Cannot read properties of undefined (reading 'user')

Found a fix here:
https://github.com/firebase/firebase-functions/issues/1622#issuecomment-2373170560
const functions = require('firebase-functions/v1');

or
import functions from 'firebase-functions/v1'


Adding auth0/angular-jwt to the seed

I'm trying to add auth0 support to this seed: basically, this should imply adding the NPM package angular2-jwt, yet as a newbie I'm probably missing something as I get errors when launching the app. Could anyone give some guidance? Maybe such questions are somewhat useful for writing entry-level docs about packages configuration. Here are my steps:

(1) npm install --save angular2-jwt to install the package.
(2) in seed.config.ts, add to SystemJS config this entry:

    'angular2-jwt': `${this.APP_BASE}node_modules/angular2-jwt/angular2-jwt`,

(3) in main.ts, add DI for AuthHttp:

    import {Http} from 'angular2/http';
    import {AuthHttp, AuthConfig} from 'angular2-jwt';
    //...
    bootstrap(AppComponent, [
      ROUTER_PROVIDERS,
      provide(APP_BASE_HREF, { useValue: '<%= APP_BASE %>' }),
      provide(AuthHttp, {
          useFactory: (http: Http) => {
            return new AuthHttp(new AuthConfig(), http);
          },
          deps: [Http]
      }),
    ]);

If now I try using AuthHttp e.g. in app.ts, I get the angular2 EXCEPTION: No provider for Http! (AppComponent -> AuthHttp -> Http). Http is a dependency of AuthHttp, but I thought this should be taken care of by my changes in bootstrap.

import {AuthHttp, tokenNotExpired, JwtHelper} from 'angular2-jwt/angular2-jwt';

And don't add things to the seed-config, extending projects should be done in the project config

Sorry, I edited my comment before getting your replies (my connection has troubles). I fixed the typo but I'm getting some errors in the AuthHttp dependencies.

Thanks again, as for d3viant0ne's comment, I removed the line from seed.config.ts and added it in project.config.ts at the end of its constructor, like:
    (<any>this.SYSTEM_CONFIG_DEV.paths)['angular2-jwt'] =
        `${this.APP_BASE}node_modules/angular2-jwt/angular2-jwt`;

Is this the intended way of handling such cases in the seed? Apart from this, I'm still getting the Http injection error.

Ok, found it, I had forgotten to move the HTTP_PROVIDERS "macro" before the provide in bootstrap. Sorry to bother. Anyway, this discussion helped me getting more in touch with the intended seed usage, thanks to all again.

Not sure you actually need anything in any of the config files. I implemented Auth0 with the seed myself but on an older version and I didn't touch the config.

The config are required to import the angular2-jwt modules, even if there seems to be some errors in TS, probably the angular2 version used by the JWT library is not synched with that used by the seed (see below). Anyway this does not seem to harm. To recap, as this might be useful to someone else:

(1) npm install --save angular2-jwt to install the package.

(2) in project.config.ts, add to the end of the constructor code:

(<any>this.SYSTEM_CONFIG_DEV.paths)['angular2-jwt'] =
        `${this.APP_BASE}node_modules/angular2-jwt/angular2-jwt`;

(3) in main.ts, add DI for AuthHttp (remember to import HTTP_PROVIDERS too!):

    import {Http, HTTP_PROVIDERS} from 'angular2/http';
    import {AuthHttp, AuthConfig} from 'angular2-jwt';
    //...
    bootstrap(AppComponent, [
      ROUTER_PROVIDERS,
      HTTP_PROVIDERS,
      provide(APP_BASE_HREF, { useValue: '<%= APP_BASE %>' }),
      provide(AuthHttp, {
          useFactory: (http: Http) => {
            return new AuthHttp(new AuthConfig(), http);
          },
          deps: [Http]
      }),
    ]);

(4) usage example:

import {AuthHttp} from 'angular2-jwt/angular2-jwt';
//... use in your code ...

The TS errors are like:

main.ts: error TS2345: Argument of type 'Http' is not assignable to parameter of type 'Http'.
Property '_backend' is protected but type 'Http' is not a class derived from 'Http'.
when using AuthHttp: error TS2322: Type 'Observable' is not assignable to type 'Observable',  and Property 'source' is protected in type 'Observable' but public in type 'Observable'.

@Myrmex thanks for the great explanation! I didn't get any TS errors with the latest version of the seed.
Would you want to update the wiki page?

Did it right now, I'm happy to give my 2 cents to this wonderful seed.

Thank you!

@mgechev thanks for your seed.
and this question's answer.
and follow this issue, i can import angular2-jwt in the project.
but what i want to ask is, could i use SystemJS import Auth0Lock.
i copy some code from auth0-samples/auth0-angularjs2-systemjs-sample,
and notice that, it use Auth0Lock by script in index.html.
i read the SystemJS's docs, and i try to include it like the angular2-jwt,
but alway EXCEPTION: Error in :0:0 caused by: Auth0Lock is not defined.
could you tell me how to import it?

i can use it like
{
  map: {
    'auth0-lock': 'cdn link'
  }
}

or tons of deps
{
  map: {
    'auth0-lock': 'npm:auth0-lock',
    'npm:auth0-lock': {
      ...
    }
  }
}

is there any simple method to import it?


No EDIFACT tracking schemas?

I was looking over the content for Tracking schemas but did not see any details about EDIFACT tracking schemas. Do these or exist or this is a current feature gap?

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

ID: e3ec51a1-59dc-3739-9ef4-8356e42aee0e
Version Independent ID: c4c0730a-59f2-01ff-f347-451dd0c20f5b
Content: X12 tracking schemas for B2B messages - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-track-integration-account-x12-tracking-schema.md
Service: logic-apps
GitHub Login: @divyaswarnkar
Microsoft Alias: divswa

@clineer Thanks for the question! We're investigating this and will get back to you shortly.

@clineer Just checking if you have gone through this doc and see if it helps.

I was trying to find out how to setup the OMS portal or an app insights report for EDIFACT messages. Not setup of an agreement.
Thanks,
Ben
Get Outlook for iOShttps://aka.ms/o0ukef

From: DixitArora-MSFT<EMAIL_ADDRESS>Sent: Tuesday, August 27, 2019 1:40:04 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ben Cline<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] No EDIFACT tracking schemas? (#37730)
@clineerhttps://github.com/clineer Just checking if you have gone through this dochttps://docs.microsoft.com/en-us/azure/logic-apps/logic-apps-enterprise-integration-edifact#schemas and see if it helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/37730?email_source=notifications&email_token=AJ2OIRI7GYOE65BA7LNKXM3QGTD4JA5CNFSM4IPGMJKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5GVILQ#issuecomment-525161518, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJ2OIRONTOODAGP34XLUBDDQGTD4JANCNFSM4IPGMJKA.

Thanks, my issue was just the documentation gap.
Ben
From: Divya Swarnkar<EMAIL_ADDRESS>Sent: Tuesday, August 27, 2019 11:09 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ben Cline<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] No EDIFACT tracking schemas? (#37730)
Hi Ben,
We do publish tracking data for EDIFACT connector and you can enable it in the same way as you do for AS2/X12 through Diagnostics settings. Thanks for the note on doc gap, we have added the workitem to publish EDIFACT schemas as well.
Thanks - Divya
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/37730?email_source=notifications&email_token=AJ2OIRMXOZVURRG4HYRA5DLQGVGSXA5CNFSM4IPGMJKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5IIVTY#issuecomment-525372111, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJ2OIRI2HA6IOOQGWMS2FUTQGVGSXANCNFSM4IPGMJKA.

@clineer As Divya mentioned work item has been added for this doc gap and will be address. So for now I will proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Test AlaSQL with require()

We need to test this mode of AlaSQL., because we have few tests for it, and probably it is a source of this error (see https://github.com/agershun/alasql/issues/388)

Can you put more words to what you mean with require()?
All tests are using require() to include the lib...


Fix file encoding issues preventing successful pip installation on Windows

This PR should fix issue #10. Sorry about the duplicate--read my comments on PR #40 to see what happened, I'm completely new to contributing to open source!
The README file is now read correctly by codecs.open with utf-8 encoding into the setup's long_description.
After this fix, I was able to successfully build and install a wheel in Windows cmd with python setup.py bdist_wheel and pip install dist\maya-0.1.4-py3-none-any.whl :)

✨🍰✨


UICollectionView nested in UIScrollView

I have a collection view (horizontal scrolling) nested inside of a scroll view in the floating panel and scrolling does not work.
Any suggestion on a fix?

Thanks for your report. It's possible this is a problem in a gesture handling of this library. Now I'm investigating.

Could you try to use fix-gesture-handling branch?

Perfect!  Thank you very much!


[FR] Compatibility with MLflow 2.0

Thank you for submitting a feature request. Before proceeding, please review MLflow's Issue Policy for feature requests and the MLflow Contributing Guide.
Please fill in this feature request template to ensure a timely and thorough response.
Willingness to contribute
The MLflow Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature (as an enhancement to the MLflow TorchServe Deployment plugin code base)?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from the MLflow community.
[X] No. I cannot contribute this feature at this time.

Proposal Summary
In MLflow 2.0 (scheduled for release on Nov. 14), we will be making small modifications to the MLflow Model Server's RESTful scoring protocol (documented here: https://output.circle-artifacts.com/output/job/bb07270e-1101-421c-901c-01e72bc7b6df/artifacts/0/docs/build/html/models.html#deploy-mlflow-models) and the MLflow Deployment Client predict() API (documented here:  https://output.circle-artifacts.com/output/job/bb07270e-1101-421c-901c-01e72bc7b6df/artifacts/0/docs/build/html/python_api/mlflow.deployments.html#mlflow.deployments.BaseDeploymentClient.predict).
For compatibility with MLflow 2.0, the mlflow-torchserve plugin will need to be updated to conform to the new scoring protocol and Deployment Client interface. The MLflow maintainers are happy to assist with this process, and we apologize for the short notice.
Motivation

What is the use case for this feature? Provide a richer, more extensible scoring protocol and broaden the deployment client prediction interface beyond dataframe inputs.
Why is this use case valuable to support for MLflow TorchServe Deployment plugin users in general? Necessary for compatibility for MLflow 2.0
Why is this use case valuable to support for your project(s) or organization? ^
Why is it currently difficult to achieve this use case? Without these changes, the mlflow-torchserve plugin will break in MLflow 2.0.

What component(s) does this feature affect?
Components

[X] area/deploy: Main deployment plugin logic
[ ] area/build: Build and test infrastructure for MLflow TorchServe Deployment Plugin
[ ] area/docs: MLflow TorchServe Deployment Plugin documentation pages
[ ] area/examples: Example code

@shrinath-suresh @chauhang Do you have bandwidth on your end to migrate the mlflow-torchserve plugin to the updated scoring protocol and adjust the Deployment Client predict() API? By my estimates, it should only take a few hours of work at most.

@dbczumar I am testing the changes with MLflow 2.0.
From MLflow 2.0, is it mandatory to return predict response of type  pd.DataFrame or pd.Series ? . Getting error while converting text/json due to result.to_json() convertion done here
Till MLflow 1.30.0, we are returning the result as a string (code link) and mlflow cli converts it to json (code link).
Most of the ML examples (bert, mnist, iris classification), the output of the model will be either json or text.
If the changes are intentional, we need to change the mlflow_torchserve plugin's predict method to  convert the user response to dataframe and return it to mlflow cli

@dbczumar @harupy

@shrinath-suresh So sorry for the delay here. The deployment client predict() method always needs to return a PredictionsResponse object (https://mlflow.org/docs/latest/python_api/mlflow.deployments.html#mlflow.deployments.PredictionsResponse). For your use case, feel free to subclass PredictionsResponse and define to_json() as desired :)

@dbczumar Can you please review this PR when you find time ? https://github.com/mlflow/mlflow-torchserve/pull/97

@shrinath-suresh Approved! Happy New Year!

Has this been merged/solved and we are good to go with the latest version?
Looks like it by #97 but just wanted to confirm if there/s anything outstanding

Is this plugin still maintained?


Scripting for Web Development: PHP vs. JavaScript

Title Of Blog Post : Scripting for Web Development: PHP vs. JavaScript
Short Description of what is expected : PHP and JavaScript are popular scripting languages, compare PHP and JavaScript in the context of web development.
Category : PHP JavaScript Languages WebDevelopment
Tags :PHP JavaScript

hello bro can you assign this to me

Hi,
Can you please assign me to this?


Create juxiaoshuo.js

Chapter list broken please check tried fetching chapter list but no success, everything else good to go.
Site: https://www.juxiaoshuo.net/

Okey I got it 😀

Content is combined tried spliting but not working 😞

Should I merged it even through content is not being split? I tried using split or replace but it gives error like not a function at watch.


Modified quaternion2euler function to cap the arcsin's argument by +-1

Description

Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References

Adresses #2525 .
How has this been tested?

Checked that returned tuple of angles doesn't contain nan, and doesn't return a warning anymore.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas

I think this approach of not using the additional capper function looks good. As @goanpeca some tests would be great. It looks like quaternion2euler didn't have any of it owns dedicated tests yet, so adding some both for the general functionality and then this specific bug fix would be much appreciated. We can help advise more around the best way to do that too if needed.
Also it looks like our black formatting test is failing. There are some instructions on getting black working with a pre-commit hook here that you might find helpful https://napari.org/developers/contributing.html#setting-up-a-development-installation.
Thanks again for the PR!!

Thanks again for your advices ; I feel I'm getting closer to the right track ! I began using pre-commit so the code is ok with this regard, and I rewrote a part of the pre-existing unittest for quaternion2euler. I made it so that it also check at least one problematic set of angles, so right now the function doesn't pass the test as I raised it in #2525 . I'll work on how to rewrite the quaternion to Euler angles conversion !
By the way, the quaternion2euler unittest was in a file named test_utils ; should I rename it to test_quaternion2euler, or was the function meant to be added in the utils file ?
Thanks again for your help !

As Euler angles can be degenerate, I changed the test so that the corresponding quaternions are compared.
For a regular user, it means that setting viewer.camera.angles and then getting the same attribute might lead to different values (even though it describes the same view), and that is the expected behaviour. Should I specify this somewhere in the documentation ?
Side question : I'm new to contributing and to testing, and I wanted to ask ; here I modified the test according to the expected behaviour of quaternion2euler, but the function itself wouldn't pass this test in its actual state. Will it bother the community of contributors if I leave it like that while I modify the quaternion2euler function ? Thanks again for your advices, it's helping me a lot ! :)

For a regular user, it means that setting viewer.camera.angles and then getting the same attribute might lead to different values (even though it describes the same view), and that is the expected behaviour. Should I specify this somewhere in the documentation ?

Yeah, I'd add that to the documentation, certainly to the quaternion2euler doc string, and possibly to the viewer.camera.angles doc string too.

Side question : I'm new to contributing and to testing, and I wanted to ask ; here I modified the test according to the expected behaviour of quaternion2euler, but the function itself wouldn't pass this test in its actual state. Will it bother the community of contributors if I leave it like that while I modify the quaternion2euler function ? Thanks again for your advices, it's helping me a lot ! :)

Good question! It's fine to have broken tests in this PR as you do your development, but it will be important to get all the tests passing this PR before we merge it. So either we'll have to figure out a way to update the old tests, or the function to get things working with the new behavior. Having all the tests passing before merge is important as then when others are developing they know they are building on a clean slate. Let me know if you have additional questions!!

I've modified quaternion2euler to take into account the singularity, simplified the test as suggested (I think) and added some docstrings info ! Now tests are passing :)


Filter bubbles too big + not inline with text

this is a picture of what it looks like on mine

Need to constrain height and width => then make the flexbox wrap


Improve

Expected Behavior
Link to the documentation or explain the expected outcome of following the documentation.
Actual Behavior
Explain what actually happens when you follow the documentation.
Section that requires update
Provide a link to the page that needs an update and be specific about which section requires the update.
Additional information
Any other details or screenshots you think are relevant.

Thank you for the feedback. I'm going to close it now, but feel free to open a new issue or ask any questions.


[feat] custom .desktop file template

Describe the problem
I'm trying to implement a custom URL scheme in my app, and as demonstrated by https://github.com/tauri-apps/tauri/issues/323#issuecomment-920928947 this is already possible on Windows using a custom wix template.
It would be pretty easy to do on Linux as well, but it requires some changes to the .desktop file: namely adding a MimeType field and appending %u after Exec:
[Desktop Entry]
Categories=Development;
Exec=app-name %u
Icon=app-name
Name=app-name
Terminal=false
Type=Application
MimeType=x-scheme-handler/customprotocol

Describe the solution you'd like
Use a handlebars template for .desktop file generation, the same way it works for wix:
"deb": {
    "depends": [],
    "desktopTemplate": ""
},

[Desktop Entry]
Categories={{{categories}}}
Exec={{{exec}}}
Icon={{{icon}}}
Name={{{name}}}
Terminal=false
Type=Application

I can make a PR later if you aggree on this change
Alternatives considered
No response
Additional context
No response

A PR would be greatly appreciated, thanks.


Fix npm lint errors preventing publish of new version

@fjs21 Thanks for the note - hopefully this should fix the issues that were preventing you from publishing a new version to NPM.


Some alignment optimizations

[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

May fix some performance regressions, e.g., see https://github.com/pydata/xarray/issues/7376#issuecomment-1352989233.
@ravwojdyla with this PR ds.assign(foo=~ds["d3"]) in your example should be much faster (on par with version 2022.3.0).

Quick benchmark taking the example in #7376 (it seems even much faster than in version 2022.3.0!)
# version 2022.3.0
%timeit ds.assign(foo=~ds["d3"])
# 22.5 ms ± 1.96 ms per loop (mean ± std. dev. of 7 runs, 100 loops each)

# main branch
%timeit ds.assign(foo=~ds["d3"])
# 193 ms ± 1.35 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

# this PR
%timeit ds.assign(foo=~ds["d3"])
# 1.01 ms ± 10.7 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)

No obenchmark is catching this? Maybe we can add a small one in https://github.com/pydata/xarray/blob/main/asv_bench/benchmarks/indexing.py ?

I don't know if the optimizations added here will benefit a large set of use cases (it took 6 months before seeing an issue report), but it is worth for at least a few of them. This is ready I think (added some benchmarks).

Thanks, @benbovy !


Add support for execution within Node.js to generated code

(Optionally per plugin config) for #27

probably generate within macros code conforming to nbind

Would also need to add support for storing what transition WebGME node IDs were taken when a transition occurs as well as the WebGME node ID is for the current state.


Improve sorting in positional arg context

Purpose

Positional args are not sorted properly when the required context is a typedesc.

Fixes #36349
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

closing this pr and send a new one to 2201.2.x branch.


No 'Access-Control-Allow-Origin' header is present on the requested resource.

I am using ember for front end and laravel for api back end. The problem i am facing is i have two api say "a" and "b". I solved the problem " No 'Access-Control-Allow-Origin' header is present on the requested resource." with my api "a" by adding cors to my project and adding the cors middleware to my route. But my another api "b" is also having the same issue eventhough i added the cors middleware.

Is this issue fixed ?

Yeah i fixed this issue
On Apr 18, 2017 11:42 AM, "AbdurRehman26"<EMAIL_ADDRESS>wrote:

Is this issue fixed ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/barryvdh/laravel-cors/issues/125#issuecomment-294697059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AITk_9WNb2dbfn7wuUYUQXztOFptiwKqks5rxFQxgaJpZM4JZRiI
.

how did you do it  ??? my get requests are working fine but cant seem to post anything

Are u in facebook?
My name is vijay sundarapandiyan
With dragon ball super goku dp
Text me there and explain what is the issue.
On Apr 18, 2017 12:30 PM, "AbdurRehman26"<EMAIL_ADDRESS>wrote:

how did you do it ??? my get requests are working fine but cant seem to
post anything
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/barryvdh/laravel-cors/issues/125#issuecomment-294706772,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AITk_6nDqXrDnfxNYzYWKrSF07jq9zFTks5rxF-PgaJpZM4JZRiI
.

do u have skype ? I am at the office and we are not allowed to use facebook

Use my mailid to search.
<EMAIL_ADDRESS>On Apr 18, 2017 12:34 PM, "AbdurRehman26"<EMAIL_ADDRESS>wrote:

do u have skype ? I am at the office and we are not allowed to use facebook
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/barryvdh/laravel-cors/issues/125#issuecomment-294707501,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AITk_2SbJr21_kgUURohHbHSqM3nqqq9ks5rxGCGgaJpZM4JZRiI
.

added u on skype

Text me in that.
On Apr 18, 2017 12:39 PM, "AbdurRehman26"<EMAIL_ADDRESS>wrote:

added u on skype
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/barryvdh/laravel-cors/issues/125#issuecomment-294708302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AITk__hMt3MGhR4Y1IgWvyedEyh3KOYaks5rxGGVgaJpZM4JZRiI
.

Accept my request.. cant message you without that

Can't you post the solution here? I would like to know as well.

I created a Custom CORS middleware and applied it to the routes.  I will share the code here tomorrow.

 'POST, GET, OPTIONS, PUT, DELETE',
            'Access-Control-Allow-Headers'=> 'Content-Type, X-Auth-Token, Origin'
        ];
        if($request->getMethod() == "OPTIONS") {
            // The client-side application can set only headers allowed in Access-Control-Allow-Headers
            return Response::make('OK', 200, $headers);
        }

        $response = $next($request);
        foreach($headers as $key => $value)
            $response->header($key, $value);
        return $response;
    }

}

This is my middleware. Instead of * in header("Access-Control-Allow-Origin: *"); you can use you frontend host:port

use Closure;

class CORS {

    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {

        header("Access-Control-Allow-Origin: *");

        // ALLOW OPTIONS METHOD
        $headers = [
            'Access-Control-Allow-Methods'=> 'POST, GET, OPTIONS, PUT, DELETE',
            'Access-Control-Allow-Headers'=> 'Content-Type, X-Auth-Token, Origin'
        ];
        if($request->getMethod() == "OPTIONS") {
            // The client-side application can set only headers allowed in Access-Control-Allow-Headers
            return Response::make('OK', 200, $headers);
        }

        $response = $next($request);
        foreach($headers as $key => $value)
            $response->header($key, $value);
        return $response;
    }

}

Can you try with v0.10.1? Please re-open if the issue this persists.


Fixed link in README

Codecov Report

Merging #64 (560fdd1) into main (c7d7566) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   91.27%   91.27%
=======================================
  Files           8        8
  Lines        1868     1868
  Branches      277      277
=======================================
  Hits         1705     1705
  Misses        161      161
  Partials        2        2

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


Initial implementation

what

Initial implementation of terraform-aws-transit-gateway module

why
Terraform module to provision:

AWS Transit Gateway
AWS Resource Access Manager (AWS RAM) Resource Share to share the Transit Gateway with
the Organization or another AWS Account (configurable via the variables ram_resource_share_enabled and ram_principal)
Transit Gateway route table
Transit Gateway VPC attachments to connect multiple VPCs via the Transit Gateway
Transit Gateway route table propagations to create propagated routes and allow traffic from the Transit Gateway to the VPC attachments
Transit Gateway route table associations to allow traffic from the VPC attachments to the Transit Gateway
Transit Gateway static routes (static routes have a higher precedence than propagated routes)
Subnet routes to route traffic from the subnets in each VPC to the other Transit Gateway VPC attachments

/test all

/test all

/test all

/test all

/test all

/test all


NI 6251 - simultaneous analog input and analog output

Historically, I've heard that the card takes a 'very long time' to switch between input and output mode, but further reading makes me think that it may be possible to allocate physical channels / samples to output without sacrificing our wonderful input parameterization. This would be powerful for obvious reasons, although as a rule I still prefer scan hardware to run separately if possible (things like Arduino make that possible over usb for homebrew hardware).
I'm opening this issue to serve as a placeholder / location to discuss how we might use analog output features from our 6251 cards. I have no plans to add this functionality in the near future.

This is a beautiful idea that could allow us to more easily create multi-triggered sample and hold devices for our pyros etc. I agree that developing this is should not be a priority for you, but if you could compile the sources from which you have gleaned this information here, that would be wonderful.

May be abstractable without hardware now, but since it is phrased as NI specific, adding the tag

Closing as all NI daq code was exported by #291


Bunch of code improvements

Removed unwanted code.
Removed redundant else.
Converted code to expression body for better understanding.
Removed unused variable.
Simplified if block code.
Removed redundant class constructor.
Removed unused using directives.
Removed one redundant String.Format call in ConsoleAccessException class.
Added LINQ expression query instead of foreach loop in ExtensionMethods class.
Used new object initializer way to initialize object.
Added new local functions to code instead of Func<> delegate.
Removed redundant this keyword.

C# 7.1 for the win!!!
I had a quick scan and it LGTM. @tomakita can confirm.

@MCCshreyas Thanks for this great contribution!  I've looked through this PR, and have cloned your branch on my machine for some testing.  Everything looks great, with one small exception: I'm not quite able to understand the changes to the ColorExtensions (src/Colorful.Console/ColorExtensions.cs) class.  Can you walk me through the changes to this class?
@RehanSaeed Thanks for looking through these changes Muhammad!

The Math.Abs(sum) will round off the value of sum variable.
It is recommended that you should round off the double value before comparing it to float which is 0.0 in our case. That's why that change has been made. Not important one by the way, you can discard it.

That's a good catch -- we should be checking for equality in a more robust way.  I might be wrong, but I'm not sure if comparing Math.Abs of the sum to zero will do this, though, because an absolute value can't be less than zero.  Instead, we can compare the sum to a value that is small enough that we can regard it as being equivalent to zero ( for the purposes of our problem, anyway).  Let's call this value epsilon.  If the sum is smaller than epsilon, then we can regard the sum itself as zero.  Also, we can avoid taking the absolute value of sum, because we know that it must be positive (it is the sum of 3 squares, and a value squared must always be positive).  So what if, instead of this:
if (Math.Abs(sum) < 0.0)
{
    return consoleColor;
}

We did this:
double epsilon = 0.001;
if (sum < epsilon)
{
    return consoleColor;
}

What do you think of this approach?

Yes. It is correct.

My pleasure. : )
Have a great day ahead.


fix(website): changelog plugin leads to CI bugs on release

Motivation
Will attempt to fix changelog bug happening after each new release: https://github.com/facebook/docusaurus/pull/6829#issuecomment-1058302603
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
ci

Seems it's because the action builds the branch first, and then builds the branch's base. However, when running git reset, the newly generated changelog entry file is not removed, but the authors list is re-generated. I think the fix is to read authors.json first and only append extra entries.

I believe it works but unfortunately, I can't verify this in current PR as it seems it executes the clean script of the main branch 😅
Will merge this assuming it works and open another PR to verify


Feat/new components

:tada: This PR is included in version 4.7.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Graph C API] Enable plugin optimizers and configs.

This is the 3rd PR(and the last one) of Graph C API, following RFC Modular TensorFlow Graph C API.
This PR is mainly to add plugin optimizer and config in meta_optimizer, which includes:

Run plugin optimizer in meta_optimizer.
Change config if plugin optimizer is enabled, otherwise leave it as default. Print it when plugin config is conflicit with user config.
Call InitGraphPlugin in RegisterPluggableDevicePlugin.

Hi @penpornk @ezhulenev, please help to have a review. Thanks very much!

@penpornk  This PR is the 3rd Graph C API PR. Without this PR, users won't be able to use graph plugin.

@ezhulenev Thanks for your review! I have addressed all your comments.

@ShengYang1 Can you please resolve conflicts? Thanks!

@ShengYang1 Can you please resolve conflicts? Thanks!

I have already rebased, please review. Thanks!

#45784 just got merged. Could you please resolve conflicts again? Thank you!

@penpornk Already rebased.
Meanwhile I have slightly changed the logic in RegisterPluggableDevicePlugin. In the previous design, it will directly return error if SE_InitPlugin is not found. However, plugin with graph only should also be allowed, so I changed it to "return error if neither device nor graph is found". Suggestions are welcome. Thanks.

So I have good news and bad news.
Good news: This PR is merged, and the main PluggableDevice PR is now confirmed safe (based on last night's tests)! 🎉
Bad news: There has been some issues adding the new RewriterConfig option (use_plugin_optimizers). I added it separately here but the change got reverted. So I made this PR use kUsePluginOptimizers = RewriterConfig::ON; in metaoptimizer.cc and removed the changes in eager/context.py for now. I will change them back once I can get the RewriterConfig change back in.

@penpornk Thanks for your effort to get this PR merged. It's really good news🎉.


API for downloading json files from s3

@azurebrd good catch! the fastapi needs aiofiles and starlette to be able to work with FileResponse properly.

@valearna Good suggestion!! I just added 'BUCKET_NAME' to .env and fixed the download.py to get bucketname from ENV.

Cool, thanks for explaining @sweng66 !   @valearna Passing the bucketname sounds fine to me, but it's always going to be agr-literature for the literature service, right ?  I guess this allows for in the future having a different root-level bucket that isn't for literature where the literature service can deposit and get files from ?

The bucket name could be defined in the code and doesn't need to be in the .env file if it's always going to be the same, but we could define it once (maybe in the api/config/GlobalConfig class?) and import it from the scripts instead of defining it inside each scripts.

That makes sense

Thanks both for the explanation! @valearna @azurebrd
Everything makes sense to me. The current fix is to use "agr-literature" as default bucket name if nothing is passed in from ENV for this variable so I will leave as is. Later we can 'globally' fix every program/module (that is using s3) to get the bucket name from a config file at the same time. Does this sound reasonable?

Cool, sounds good @sweng66 !  Let me try to revert main before merging this, so we probably shouldn't update this branch with main.


Fix links in readme

The web site emr.cs.iit.edu/~reingold no longer seems to answer (timeout). I replaced the link with one to reingold.co. Also change some broken internal links:

node_size → cluster_nodeSize
node_size → tree_nodeSize
resquarify_ratio → squarify_ratio

Thanks!


[Bug] D3.js not found

Steps to reproduce
ember new nfgraph-test; cd nfgraph-test
ember install ember-nf-graph
ember s

package.json
 "ember-nf-graph": "1.0.0-beta.8",

This is apparently already fixed. The issue was that the blueprint folder was still named ember-cli-nf-graph.
Running ember-cli-nf-graph will solve this issue for now, this will not be present in the next release (later today).


feat(build): fix bug of generating incorrect ADD_SECRETSTORE_TOKENS value

Makefile with make gen dev asc-http-s asc-mqtt-s now generates the correct ADD_SECRETSTORE_TOKENS value in secure mode.
Fixes: #376
Signed-off-by: Jim Wang<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/developer-scripts/blob/master/.github/Contributing.md.
What is the current behavior?

ADD_SECRETSTORE_TOKENS not generated correctly when both asc-http-s and asc-mqtt-s were specified.
Issue Number: #376
What is the new behavior?
ADD_SECRETSTORE_TOKENS now is generated correctly when both asc-http-s and asc-mqtt-s are specified.
Does this PR introduce a breaking change?

[ ] Yes
[x ] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

@jim-wang-intel  app-service-http-export and app-service-mqtt-export are needed for TAF tests, so the smoke tests fail. Why removing them?

@jim-wang-intel  app-service-http-export and app-service-mqtt-export are needed for TAF tests, so the smoke tests fail. Why removing them?

@jim-wang-intel app-service-http-export and app-service-mqtt-export are needed for TAF tests, so the smoke tests fail. Why removing them?

@jinlinGuan:  Ok, I didn't know that these two are must for TAF-security testing cases. Thanks for the info. I'll put them back.

@jim-wang-intel app-service-http-export and app-service-mqtt-export are needed for TAF tests, so the smoke tests fail. Why removing them?

@jinlinGuan:  Ok, I didn't know that these two are must for TAF-security testing cases. Thanks for the info. I'll put them back.


OpenAPI: Add missing required flag in getRoomKeys

Added missing required flag in getRoomKeys in matrix-doc/data/api/client-server/key_backup.yaml.
Fixes: #3485
Signed-off-by: ankur12-1610<EMAIL_ADDRESS>
Preview: https://pr3509--matrix-org-previews.netlify.app

@Dominaezzz can you review the changes?

https://pr3509--matrix-org-previews.netlify.app/client-server-api/#get_matrixclientv3room_keyskeys
Doesn't look like it's been reflected in the spec?

https://pr3509--matrix-org-previews.netlify.app/client-server-api/#get_matrixclientv3room_keyskeys Doesn't look like it's been reflected in the spec?

I'll look into it :sweat_smile:
Did I place it in wrong place? :')

Was on my phone so I couldn't check easily.
It's almost in the right place, it's just indented too far. Removing 4 spaces should do the trick I think.

Was on my phone so I couldn't check easily.
It's almost in the right place, it's just indented too far. Removing 4 spaces should do the trick I think.

@Dominaezzz, Done :)


[docs] Update docs theme and update pyproject.toml

Description
Update docs theme and update pyproject.toml
Backporting
This change should be backported to the previous releases, please add the relevant labels.
Refer here to resolve backport conflicts if any.

[x] 2016
[x] 2017
[x] 2018
[x] 2019
[x] 2020
[x] 2021
[x] 2022

Type of change
Please check the options that are relevant, the corresponding labels will be added automatically.

[ ] Bug Fix (non-breaking change which fixes an issue)

[ ] bug (fixes an issue)
[ ] test (adds/updates test)
[ ] typo (fixes typo)
[ ] refactor (refactors code)
[ ] reformat with black (check this to reformat the code with black)

[ ] New Feature (non-breaking change which adds functionality)

[ ] typing (adds/updates typing annotations)

[ ] Documentation Update

[ ] docs (adds/updates documentation)
[ ] docs preview (check this to preview docs)

[ ] Automation/Translation/Release

[ ] workflow (adds/updates workflow)
[ ] release (adds/updates release)
[ ] translation (adds/updates translation)

Codecov Report

Merging #3880 (2a0ebcf) into 2023 (3e52215) will decrease coverage by 0.01%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             2023    #3880      +/-   ##
==========================================
- Coverage   88.49%   88.49%   -0.01%
==========================================
  Files         931      931
  Lines       27320    27320
==========================================
- Hits        24178    24177       -1
- Misses       3142     3143       +1

Flag
Coverage Δ

2023
88.49% <ø> (-0.01%)
:arrow_down:

Python-3.10
?

Python-3.11
88.49% <ø> (-0.01%)
:arrow_down:

Python-3.7
88.04% <ø> (-0.01%)
:arrow_down:

Python-3.8
88.49% <ø> (-0.01%)
:arrow_down:

Python-3.9
88.49% <ø> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/abqpy/abaqus.py
68.42% <0.00%> (-2.64%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


(fix) routers - update url search params according to validate search

Should fix: #1268
With this fix, it should update the url search params according to the validateSearch zod schema so the visible url should match the router search state (except if masked is activated).
Update on packages/react-router/src/RouterProvider.tsx is a "revert" from changes made between 1.18.3 & 1.18.4.
I don't know why this was deleted, maybe it's part of something I miss. If it's the case, ignore this PR.
If you need more precisions, let me know.

did you test it locally?


Problem white android

when i build my app, this message : JNI DETECTED ERROR IN APPLICATION: JNI GetMethodID called with pending exception android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.
what is the problem?

Which device and Android version? The StatusBar implementation is using QtAndroid::runOnAndroidThread() to execute the JNI calls, so I can't say off the top of my head what might go wrong there. Is there any multi-threading involved in the app's side? Where is the StatusBar instance created?

Are you able to reproduce the problem with a minimal Qt Quick app?
// main.qml
import QtQuick 2.7
import QtQuick.Window 2.2
import StatusBar 0.1

Window {
    visible: true
    StatusBar {
        color: "red"
    }
}

Thanks the problem was, quickcontrols2 and in the main.cpp write  QQuickStyle::setStyle("Material");
and this:
QtAndroid::runOnAndroidThread(= {
QAndroidJniObject window = QtAndroid::androidActivity().callObjectMethod("getWindow", "()Landroid/view/Window;");
window.callMethod("addFlags", "(I)V", FLAG_DRAWS_SYSTEM_BAR_BACKGROUNDS);
window.callMethod("clearFlags", "(I)V", FLAG_TRANSLUCENT_STATUS);
window.callMethod("setStatusBarColor", "(I)V", color.rgba());
});
i don't include   QtAndroid::runOnAndroidThread
thanks


Does not seem to work with latest version of svelte kit? (SvelteKit v1.0.0-next.88)

I've just tried to create a fresh project via:
npm init svelte@next testapp

answered all questions with "yes"
"npm i"
"npx svelte-add tailwindcss"
getting "sveltePreprocess is not defined" error

Full error:
sveltePreprocess is not defined
ReferenceError: sveltePreprocess is not defined
    at Object.<anonymous> (/home/aaa/dev/testapp/svelte.config.cjs:5:14)
    at Module._compile (node:internal/modules/cjs/loader:1108:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
    at Module.load (node:internal/modules/cjs/loader:988:32)
    at Function.Module._load (node:internal/modules/cjs/loader:828:14)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:201:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:175:25)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Loader.import (node:internal/modules/esm/loader:178:24)
    at async load_config (file:///home/aaa/dev/testapp/node_modules/@sveltejs/kit/dist/cli.js:557:17)

Tracking in https://github.com/svelte-add/postcss/issues/20


Update lambda function to lowercase the node prefix in buckets

💡 Description
Right now data is being pushed to buckets like for SBN, /SBN/my/data/here. the /SBN seems a bit redundant, but we can leave it for now. However, definitely want this lowercase.
⚔ Parent Epic / Related Tickets
No response

@collinss-jpl Updated ticket to simple lowercase the prefix for now. Sorry. Pet peeve of mine...


Make util for url

so far a lot of code has forcefully hardcoded rmrk but it will fail for other impl like statemine or bsx.
It would be nice to have a util for url
function toUrl(prefix: string, route: 'user' | 'nft' | 'col', isDetail = false);

Continue in https://github.com/kodadot/packages/issues/90


Improve error message when LDFLAG "-Wl,-z,cet-report=error" is set

The LDFLAG -Wl,-z,cet-report=error can help debug why software is not built with SHSTK/IBT support (which could be done via the CFLAG "-fcf-protection").
When I enabled -Wl,-z,cet-report=error, however, I could no longer configure OpenARC. The error was:
checking for libidn2... yes
checking for libmilter/mfapi.h... yes
checking for smfi_register in -lmilter... no
configure: error: Unable to find libmilter
==> ERROR: A failure occurred in build().
    Aborting...

The full config.log is posted online.
It turned out that libmilter was not built with SHSTK/IBT support on my system. As a result, OpenARC could not be built with SHSTK/IBT, either, as all dependencies need to be built with that support first. Either everything has SHSTK/IBT support, or it won't show.
I presume that -Wl,-z,cet-report=error threw an error for libmilter, and as a result it ~somehow~ was no longer usable/visible for the OpenARC configure.
Would it be possible to write a Test that checks whether a LDFLAG -Wl,-z,cet-report=error is set, and if so, if there is an error along the line of configure: error: Unable to find libmilter, that a different error message to the current message is given?
Since I am not sure what exactly happens when -Wl,-z,cet-report=error spots an issue, I am not sure what the message should be. But perhaps it can contain a hint that it might be related to that flag? error: Unable to find libmilter was definitely not helpful in debugging.
Here is a link to a related discussion on the Archlinux forums.

That's what config.log is for.
mold: error: /usr/lib/libmilter.a(main.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(main.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(engine.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(engine.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(listener.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(listener.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(worker.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(worker.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(comm.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(comm.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(signal.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(signal.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(sm_gethost.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(sm_gethost.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(errstring.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(errstring.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK
mold: error: /usr/lib/libmilter.a(strl.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_IBT
mold: error: /usr/lib/libmilter.a(strl.o): -cet-report=error: missing GNU_PROPERTY_X86_FEATURE_1_SHSTK

Correct. I did find the actual error, and was able to fix it.
However, because OpenARC could find libmilter just find without the -Wl,-z,cet-report=error flag enabled, I was confused at first. That is why I thought a different error message might be more helpful.
If you don't think it's worth it, I understand. Not many people will enable that flag, and if one does, one should be on the lookout for its results anyway.

There are a lot of compilation flags you can provide that will break various configure checks, I don't think trying to special-case any particular one is worthwhile.

Okay, thank you for your reply.


Upgrade from Ruby 3.1.3 to Ruby 3.3.0

🔗 Issue

✍ Description

📷 Screenshots/Demos

Hey @timrogers is this still draft for a reason? Thanks.

Looks like 3.3.1 has some security fixes https://www.ruby-lang.org/en/news/2024/04/23/ruby-3-3-1-released/

@kasugaijin @timrogers are you ok if we merge this now?
(and then I'll work on a 3.3.1 PR)

@maebeale yes let’s merge this one


For last sibling, "Next" goes to first sibling instead of next sibling of parent

Fixes #2
The logic was much easier to conceive of and implement recursively. Now we recursively look upward for the next siblings of parents until we find one or else we hit the root page, in which case we're on the last page of the wiki.

Link to commit comment that closes this issue


add element_id

Since ID should be unique, it would be better to use elem_classes instead, or add the tab mode to the id


sched/sched: Appease nxstyle errors

The remaining errors ("Missing space before closing C comment")
don't make sense. Maybe a bug in nxstyle.

Maybe a bug in nxstyle.

https://github.com/apache/incubator-nuttx/issues/540

Let's wait for #542 to be merged and let the checks run again.

Let's wait for #542 to be merged and let the checks run again.

@yamt you can rebase this PR now.

Now I see that some lone line fixes that I did in sched/sched/sched_mergepending.c are conflicting with something else.  The version from #542 with the shorter lines is the one that is correct.


add secrets access to the role

Allow jenkins to access kubernetes secrets so that plugins like kubernetes-credentials-provider can work

Can we implement it in a way that users of the chart need to explicitly enable it? roles are by intention limited. We could for example have a separate role just for accessing secrets and only bind it when a certain property is set.

Makes sense. I will try to add that PR.
On Sat., Oct. 26, 2019, 04:06 Torsten Walter<EMAIL_ADDRESS>wrote:

Can we implement it in a way that users of the chart need to explicitly
enable it? roles are by intention limited. We could for example have a
separate role just for accessing secrets and only bind it when a certain
property is set.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helm/charts/pull/18361?email_source=notifications&email_token=AB6KUURJN53MDMJ6ISOIIFTQQP3ANA5CNFSM4JFHSS42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKCTWA#issuecomment-546580952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6KUUU6I2GC5Z3CB6553Z3QQP3ANANCNFSM4JFHSS4Q
.

@musabmasood
Could you sign DCO and bump the chart version to latest please?


Add toggle status button to entry page.

Closes #87

Refactored EntryHandler.toggleEntryStatus() to handle toggling unread/read from both list and entry pages. Also minor tweaks to the associated JS methods that call toggleEntryStatus().
Removed EntryHandler.toggleLinkStatus(); it has been merged into toggleEntryStatus() and a grep didn't turn up any uses elsewhere.
Updated the relevant controllers to set the actual entry status in the entry object after all processing (right before calling the view); needed so the button shows the correct state when the page is loaded.
Existing functionality related to toggling the status is working in my testing, but feedback is welcome.

Note: the heavy checkmark unicode looks out of place on the entry page because its colour is not matching the other button/links (blue). Found this Stack Overflow answer that explains how the checkmark is drawn as a full-colour bitmap with its own colouring. This is less-obvious on the list pages. There is a solution workaround there, but I left it as-is for consistency. Feedback welcome.

I implemented the hack/workaround regarding the checkmark symbol in the next commit. Seems to work on my iPhone.
Thanks for the PR!

I think the text-based one looks better as it more closely matches the x (unread) counterpart.


Infinite loop when inserting Array into RBTree

Somewhat of an edge case, but can lead to a bad state in applications using this library -- hanging and high CPU usage.
Repro case:
let rbtree = new RBTree(function(a, b) { return a.value - b.value; });
rbtree.insert( [ ] );
rbtree.insert( [ ] );  // <-- infinite loop

This works ok with BinTree though.

Found in package version 1.0.1

I can confirm this bug is still present in v1.0.2.
The obvious workaround is to create a child class of RBTree and override the insert method with Array.isArray(item) and abort.
Are there any plans/suggestions for an official fix?


Bug: Mergify configuration is invalid

Expected Behaviour
When creating a new PR no mergify configuration errors should be shown
Current Behaviour

Code snippet
N/A

Possible Solution
Nothing obvious without debug log access.
Steps to Reproduce
Create a new PR
AWS Lambda Powertools for Python version
latest
AWS Lambda function runtime
3.6
Packaging format used
Lambda Layers
Debugging logs
N/A

Seems to have fixed itself @sthulb  ?


Add specific finding codes to a block list to give greater control

Description of changes:
Certain suggestions, warnings, etc might be considered "blocking" internally whereas Access Analyzer only considers them suggestions.
I don't want to block on all SUGGESTION findings but there are some SUGGESTION codes that I do want to block on, specifically: SUGGESTION.ALLOW_WITH_UNSUPPORTED_TAG_CONDITION_KEY_FOR_SERVICE
Many teams try to build policies with condition tags that don't work with the service and do something like this pseudo policy:
Action: kinesis:*
Resource: *
Condition: Tags Must Include = 123456
This gets flagged as a suggestion but actually grants full Kinesis:* to the policy, despite the user wanting to scope permissions down.
By blocking on this specific suggestion code, the user is protected from over permissive policy actions that they didn't intend on granting themselves.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi James, I am Nina from the team maintaining this repo. Thanks for your contribution. It might not make much sense to reuse --treat-finding-type-as-blocking. Can we instead create a separate flag --treat-finding-code-as-blocking?  Finding codes are unique so we wouldn’t need to do something like "SUGGESTION.FINDING_CODE"


lodash: changed _.set() method

https://lodash.com/docs#set
https://lodash.com/docs#_ (description of the chainable wrapper)

@chrootsu thanks mate!


cyclops-streams [reactive-streams] : forEachWithError / forEachEvent operators

Two new reactive-streams based operators that accept multiple consumers to forward reactive streams events to. E.g. forEachWithError
     <X extends Throwable> void forEachWithError(Consumer<? super T> consumerElement,
			Consumer<? super Throwable> consumerError);

    SequenceM.of(1,2,3)
                       .map(this::loadById)
                      .forEachWithError(t->process(t),error->log(error));

forEachEvent  (like forEachWithError but also recieves onComplete events
<X extends Throwable> void forEachWithEvent(Consumer<? super T> consumerElement,
			Consumer<? super Throwable> consumerError,
			Runnable onComplete);

    SequenceM.of(1,2,3)
                      .map(this::loadById)
                      .forEachEvent(t->process(t),error->log(error),()->complete());

Make these operators available as asynchronous 'futureOperations' also https://github.com/aol/cyclops/wiki/cyclops-streams-:-Asynchronous-Terminal-Operations

merged to master


Generating SQLite preprocessed files from gnomAD

Hi
I'm looking at using your tool for querying a specific set of genes from gnomAD, and to change to a new gnomeAD version as needed. I'd like to create my own filtered variant set from around 400 genes to reuse as a SQLite file in my pipeline. How did you go about generating your preprocessed SQLlite files you provided in the download link here: https://zenodo.org/record/6818606/files/gnomad_db_v3.1.2.sqlite3.gz?download=1
Ultimately I'd like a variant summary TSV from the output in a small pandas dataframe of 400 genes from gnomAD, to cross check with our internal database and also cross check against Clinvar.
Thanks.

Please don't hesitate to reopen this GitHub issue if you have any more questions or need further assistance.


[Bug]: Invalid links in web browser block show a blank page instead of an error.

Current Behavior
When visiting an invalid link in the web browser block, it displays a blank white page
Visiting a non existent link on the web browser block and you will get a white blank page ,  -- Example visit https://test.waveterm.dev/ from the web browser block and it will show you a white blank page instead of showing a error.

Expected Behavior
When visiting an invalid link, an error message should appear explaining why the link cannot be loaded.

Steps To Reproduce

Open the Web browser block
Paste the url https://test.waveterm.devon the url bar and hit enter key.

Wave Version
0.8.11<PHONE_NUMBER>34)
OS
macOS Sonoma Version 14.7 (23H124)
Architecture
arm64
Anything else?
Running Wave with this configuration (settings.json)
  "ai:baseurl": "http://<IP_ADDRESS>:11434/v1",
  "ai:model": "llama3.1:latest",
  "term:fontsize": 16,
  "term:fontfamily": "JetBrainsMono Nerd Font",
  "term:disablewebgl": false,
  "editor:minimapenabled": false,
  "editor:stickscrollenabled": true,
  "web:openlinksinternally": false,
  "web:defaulturl": "https://shadowarcanist.com",
  "web:defaultsearch": "https://duckduckgo.com/search?q={query}",
  "autoupdate:enabled": true,
  "autoupdate:channel": "latest",
  "widget:showhelp": false,
  "window:transparent": true,
  "window:blur": false,
  "window:opacity": 0.1,
  "window:bgcolor": "#000000",
  "window:reducedmotion": false,
  "window:tilegapsize": 4,
  "window:disablehardwareacceleration": false,
  "telemetry:enabled": false
}

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Good catch!