Adapter always stops

Hello guys,
unfortunately my judoisoft adapter always stops due to an error you see in my log below. Can you help me with this?
Thanks in advance!
Daniel
`

judoisoft.0
2022-04-25 11:34:10.785
warn
Terminated (UNCAUGHT_EXCEPTION): Without reason

judoisoft.0
2022-04-25 11:34:10.782
info
terminating

judoisoft.0
2022-04-25 11:34:10.778
info
cleaned everything up...

judoisoft.0
2022-04-25 11:34:10.747
error
Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle

judoisoft.0
2022-04-25 11:34:10.745
error
TypeError: Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle at JSON.stringify () at judoisoftControll.getInfosCloud (/opt/iobroker/node_modules/iobroker.judoisoft/main.js:276:58) at processTicksAndRejections (internal/process/task_queues.js:95:5)

judoisoft.0
2022-04-25 11:34:10.742
error
unhandled promise rejection: Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle

judoisoft.0
2022-04-25 11:34:10.739
error
Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().

judoisoft.0
2022-04-25 11:33:05.610
info
State value to set for "judoisoft.0.Maintenance" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:33:05.590
info
State value to set for "judoisoft.0.ServiceDays" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:32:04.438
info
State value to set for "judoisoft.0.Maintenance" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:32:04.418
info
State value to set for "judoisoft.0.ServiceDays" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:31:00.272
info
State value to set for "judoisoft.0.Maintenance" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:31:00.252
info
State value to set for "judoisoft.0.ServiceDays" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:29:59.654
info
State value to set for "judoisoft.0.Maintenance" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:29:59.634
info
State value to set for "judoisoft.0.ServiceDays" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:28:58.924
info
State value to set for "judoisoft.0.Maintenance" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:28:58.903
info
State value to set for "judoisoft.0.ServiceDays" has to be type "number" but received type "string"

judoisoft.0
2022-04-25 11:28:56.518
info
starting. Version 1.0.10 in /opt/iobroker/node_modules/iobroker.judoisoft, node: v14.19.1, js-controller: 4.0.21

judoisoft.0
2022-04-25 11:28:18.614
warn
Terminated (UNCAUGHT_EXCEPTION): Without reason

judoisoft.0
2022-04-25 11:28:18.611
info
terminating

judoisoft.0
2022-04-25 11:28:18.607
info
cleaned everything up...

judoisoft.0
2022-04-25 11:28:18.582
error
Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle

judoisoft.0
2022-04-25 11:28:18.580
error
TypeError: Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle at JSON.stringify () at judoisoftControll.getInfosCloud (/opt/iobroker/node_modules/iobroker.judoisoft/main.js:276:58) at runMicrotasks () at processTicksAndRejections (internal/process/task_queues.js:95:5)

judoisoft.0
2022-04-25 11:28:18.577
error
unhandled promise rejection: Converting circular structure to JSON --> starting at object with constructor 'TLSSocket' | property 'parser' -> object with constructor 'HTTPParser' --- property 'socket' closes the circle

judoisoft.0
2022-04-25 11:28:18.575
error
Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().

`

stop the adapter and del all object ..
then restart

Leider funktioniert mein Adapter nach einem Umzug auf den Pi4 wieder nicht. Habe sowohl Cloud login als auch über IP Adresse probiert. Adapter geht kurz auf grün und dann auf rot. Mit der IP kommt folgender Log
`

judoisoft.0
2022-05-18 22:04:57.675
warn
Terminated (UNCAUGHT_EXCEPTION): Without reason

judoisoft.0
2022-05-18 22:04:57.673
info
terminating

judoisoft.0
2022-05-18 22:04:57.671
info
cleaned everything up...

judoisoft.0
2022-05-18 22:04:57.663
error
Converting circular structure to JSON --> starting at object with constructor 'Object' | property 'httpsAgent' -> object with constructor 'Agent' | property 'sockets' -> object with constructor 'Object' | ... | property 'errored' -> object with constructor 'Object' --- property 'config' closes the circle

judoisoft.0
2022-05-18 22:04:57.662
error
TypeError: Converting circular structure to JSON --> starting at object with constructor 'Object' | property 'httpsAgent' -> object with constructor 'Agent' | property 'sockets' -> object with constructor 'Object' | ... | property 'errored' -> object with constructor 'Object' --- property 'config' closes the circle at JSON.stringify () at judoisoftControll.getInfoStaticLocal (/opt/iobroker/node_modules/iobroker.judoisoft/main.js:147:62) at processTicksAndRejections (internal/process/task_queues.js:95:5) at async judoisoftControll.onReady (/opt/iobroker/node_modules/iobroker.judoisoft/main.js:68:13)

judoisoft.0
2022-05-18 22:04:57.660
error
unhandled promise rejection: Converting circular structure to JSON --> starting at object with constructor 'Object' | property 'httpsAgent' -> object with constructor 'Agent' | property 'sockets' -> object with constructor 'Object' | ... | property 'errored' -> object with constructor 'Object' --- property 'config' closes the circle

judoisoft.0
2022-05-18 22:04:57.659
error
Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().

judoisoft.0
2022-05-18 22:04:54.547
warn
Read-only state "judoisoft.0.info.connection" has been written without ack-flag with value "false"

judoisoft.0
2022-05-18 22:04:54.539
debug
get Information Static Local

judoisoft.0
2022-05-18 22:04:51.394
debug
get statusURL for local

judoisoft.0
2022-05-18 22:04:50.941
debug
create state

judoisoft.0
2022-05-18 22:04:50.939
debug
base url https://<IP_ADDRESS>:8124/?group=

judoisoft.0
2022-05-18 22:04:50.904
info
starting. Version 1.0.10 in /opt/iobroker/node_modules/iobroker.judoisoft, node: v14.19.2, js-controller: 4.0.23

judoisoft.0
2022-05-18 22:04:50.658
debug
States connected to redis: <IP_ADDRESS>:9000

judoisoft.0
2022-05-18 22:04:50.634
debug
States create User PubSub Client

judoisoft.0
2022-05-18 22:04:50.632
debug
States create System PubSub Client

judoisoft.0
2022-05-18 22:04:50.617
debug
Redis States: Use Redis connection: <IP_ADDRESS>:9000

judoisoft.0
2022-05-18 22:04:50.578
debug
Objects connected to redis: <IP_ADDRESS>:9001

judoisoft.0
2022-05-18 22:04:50.570
debug
Objects client initialize lua scripts

judoisoft.0
2022-05-18 22:04:50.525
debug
Objects create User PubSub Client

judoisoft.0
2022-05-18 22:04:50.523
debug
Objects create System PubSub Client

judoisoft.0
2022-05-18 22:04:50.521
debug
Objects client ready ... initialize now

judoisoft.0
2022-05-18 22:04:50.449
debug
Redis Objects: Use Redis connection: <IP_ADDRESS>:9001

judoisoft.0
2022-05-18 22:04:38.682
warn
Terminated (NO_ADAPTER_CONFIG_FOUND): Without reason

judoisoft.0
2022-05-18 22:04:38.672
error
adapter disabled

`
Ein Löschen des Objektbaums nach kurzzeitigem Stop brachte keine Abhilfe.
Liegt es eventuell daran, dass ich nebenzu ein pi-hole am laufen habe?
Herzlichen Dank für deine Hilfe arteck!

der Adapter hat kein zugrff mehr..und ja es kann am pi-hole liegen


XS✔ ◾ Downgrading Minimum Agent Version

Summary
Motivation
As per issue #175, Azure DevOps Server (i.e., the non-cloud version of Azure DevOps) comes pre-packaged with agent version 2.181.2 and not the latest verson used within the current task. This means that the agent continually needs to be updated, which results in poor runtime performance that is not in keeping with the design goals of the task.
Technical
Downgrading the dependency to 2.181.2 within the appropriate JSON files.
Testing
Test Types

[X] Unit tests
[X] Manual tests

Unit Test Coverage
100%

This is great, looking forward to trying it out.


Log Rotation Example

This is a work in progress for log rotation. There is also a flask integration in progress, but I wanted to get this out for the launch. Right now it's really rough cause I'm still trying to figure out how using the logger work. I'm getting streaming data from the cpu.

[X] I have reviewed the Guidelines for Contributing and the Code of Conduct.

@FelipeAdachi and @jamie256 here is an update. There are two major places that need fixed before it's ready, the Google Collab won't let me add in a gist for it, if someone could show me how that would be great. Second, how do we merge from the point of reading in .bin? Can it only be the views?
I addressed most of the feedback I believe

If you run poetry run pre-commit run --all-files locally before pushing the branch pre-commit can fix up some of these issues e.g. the end-of-file-fixer that appears to be failing the ci above: https://github.com/whylabs/whylogs/runs/6910942736?check_suite_focus=true

Btw

Nice Melly!
One last minor thing:
We don't want to have 15 seconds on the code, because if people copy the code as is, it will flood the platform with profiles.
Maybe put 15 minutes, as described in the text, and add a comment that the user can change it for testing purposes.

Good catch, that was just to run it for the output! Forgot to move it back


Error while testing packages

When testing packages by running:
meteor test-packages --velocity --driver-package velocity:console-reporter

No matter the tests pass or fail, I always get:
error calling testsComplete function { [Error: Match error: Expected string, got undefined]

Please see this repo for reproduction: https://github.com/sungwoncho/meteor-mocha-issue

@sungwoncho thanks for reporting, and thanks for a clear reproduction repo.
You shouldn't need to use the console reporter when testing in this way. I was able to successfully run your test at the command line with just
meteor test-packages --velocity
And within the browser using
meteor test-packages --velocity --driver-package respondly:test-reporter
Hope this helps.


Some resource_queue.gd issues in 3.0

Operating system or device, Godot version, GPU Model and driver (if graphics related):
3.0 latest compiled
Issue description:
I'm using the latest resource_queue.gd from the latest docs:
http://docs.godotengine.org/en/latest/learning/features/misc/background_loading.html
W/o touching anything and following the step by step guide, you get:

Parser Error: Identifier not found: VS
However, if you remove that line, the thing actually works. Except..when you do get_progress it will return:
-nan(ind)
So prob not a good idea to do that lol, and I don't know if that line is a correlation to the -nan(ind) issue

Previously, Godot had shortcuts for servers. AS for AudioServer. VS for VisualServer. etc. This is deprecated. VS.flush() probably is an obsolete version of VisualServer.force_draw()?
BTW, that tutorial is very old and not up to date. New tutorials are being written.

@Noshyaar do you have an up-to-date tutorial on background loading, then?

No. I think @NathanLovato is working on the new tutorials. Not sure if Background Loading is in the list.

Tutorial was re-written and the newly modified resources_queue.gd was updated in July though, so I figured it'd be okay to follow (they changed extends to is) in the if conditions
I used VisualServer.flush() and it works but the -nan(ind) is still there
Also. What's the difference between this and the 3.0 thread example? This resource_queue.gd gives us the ability to use get_progress is the only main difference? I think there should be a doc about maybe the differentiality between this if I'm mistaken, not really sure

Background Loading's not on the list for now - it'd make for a short tut, maybe somebody else can take care of it in an upcoming sprint. Here's what we're working on with Chris: https://trello.com/b/BxdbFCzL/godot-30-docs

Bug triage note: The original issue reporter closed all their issues before deleting their account. If you are affected by a similar issue/want a similar feature, please file a new ticket (referencing this one if it contains useful information) so that it can be further looked at and eventually resolved.


template codegen need to respect the value of go_package option in the template file

Codegen is not piping the go_package name to the augmented proto and not using it to create the generated code. Need to fix that.
@geeknoid @douglas-reid FYI

We cannot land this for 0.2. @mandarjog @geeknoid . You all okay if I move this to 0.3 ?


includeTagsInMessage and includeUrlInMessage added to chatConfiguration

options added to enable/disable includeTagsInMessage and includeUrlInMessage
if includeTagsInMessage is enabled, we loop through the tags and create a slug (with underscore) and append this string to the message
if includeUrlInMessage is enabled, we append the current url from document.location.href to the message
slugify npm package added to package.json

I hope my editor didn't change the code style too much or something not in your sense. That should make the inclusion of tags and also the current url optional.

I just remembered that I didn't check what happens when data-include-tags-in-message is not set. I wonder if that doesn't lead to errors. I'll have a look.

How can I build the bundle.js? I always get an error when I try to run rollup -c.
Or is it another command?
[!] RollupError: Could not resolve "./QRCode" from "node_modules/svelte-qr/src/QR.svelte"
node_modules/svelte-qr/src/QR.svelte
    at error (/home/fsociety/.nvm/versions/node/v16.18.1/lib/node_modules/rollup/dist/shared/rollup.js:271:30)


Should use response caching

If i am requesting a resource and response comes . Again I am requesting same resource which is not updated yet by POST/PUT/DELETE request. So In these type of cases  we can use response caching for reducing our MongoDB effort.

@amitk-shangariglobal, we are implementing the server side caching by implementing the CDN and Nginx.
And we can't cache the PUT, POST and DELETE request, since every time we need to give the updated values.


Correctly set inner fragment TypeCondition when formatting nested fragment spreads

The previous fix (https://github.com/movio/bramble/pull/58) would only preserve the type information of the outer fragment, and any inner fragments would incorrectly inherit the parent type. This corrects that and will set the right TypeCondition for inner fragment spreads.
Also converted the whitespace to tabs in the new tests for consistency.

Codecov Report

Merging #60 (02a0db6) into main (8c553f3) will decrease coverage by 0.11%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   66.45%   66.33%   -0.12%
==========================================
  Files          24       24
  Lines        2519     2513       -6
==========================================
- Hits         1674     1667       -7
- Misses        726      727       +1
  Partials      119      119

Impacted Files
Coverage Δ

format.go
77.87% <71.42%> (-1.01%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c553f3...02a0db6. Read the comment docs.


Error on building nextjs project with auth helpers

Failed to compile.

<EMAIL_ADDRESS>Module parse failed: Unexpected token (22:21)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
| let networkRetries = 0;
> let refreshTokenTimer: ReturnType<typeof setTimeout>;
|
| const UserContext = createContext<UserState | undefined>(undefined);

Import trace for requested module:
<EMAIL_ADDRESS>

Please provide more information like the version of nextjs you are using and which version of the auth helpers you are using.

"next": "12.3.1",
"react": "18.2.0",
"react-dom": "18.2.0",
"@supabase/supabase-js": "^1.35.7",
"@supabase/auth-helpers-nextjs": "^0.2.8",
"@supabase/auth-helpers-react": "^0.2.4",

Could you try installing the next release npm install @supabase/auth-helpers-nextjs@next and see if that works for you. You can see the migration guide here: https://github.com/supabase/auth-helpers/blob/next/packages/nextjs/MIGRATION_GUIDE.md#migrating-to-03x


Add FAQ link to Data browser home page

After the intro sentence on the homepage, add a comment about FAQs
"For questions about the HMDA Data Browser, visit out FAQ page" (the FAQ page should link to https://hmda4.demo.cfpb.gov/documentation/2018/faq/)

@wpears our is spelled wrong. Sorry!
Can we also remove the period after FAQ
For questions about the HMDA Data Browser, visit our FAQ page.

Closed by 2625d821fc612a3cc9e33e36c6d6806c384686cc There actually isn't a period after FAQ, it's just part of the underline :)


Change Cocoaheads to "Learn Swift Louisville"

We recently "rebranded" to align with the growing community interest in Swift. We've also started meeting more regularly and have an updated URL.
CC @jwillis

Looks good to me! I'm going to leave this open until either you or I get a chance to update the icon, then I'll merge and deploy.

@ericlathrop How do I update the icon?

@ericlathrop just kidding, I figured it out. Should be ready to merge now.

Deployed!


entry dependOn doesn't support array syntax

Feature request

There's entry new syntax that looks like this:
module.exports = {
  entry: {
    app: { import: './app.js', dependOn: 'react-vendors' },
    'react-vendors': ['react', 'react-dom', 'prop-types']
  }
};

In my use case with webpack-plugin-serve, I have to inject a client script to each of the entries. With the usual object syntax I rewrite the entries as arrays of entry objects ([{ app: '...path...' }, 'wps-client']).
The array syntax doesn't appear to work with the new addition.
What is the expected behavior?
I would expect the following to work:
module.exports = {
  entry: {
    app: [{ import: './app.js', dependOn: 'react-vendors' }, 'wps-client'],
    'react-vendors': ['react', 'react-dom', 'prop-types']
  }
};

Currently using an array like this yields a configuration validation error. Instead it should pass and then attach wps-client to the entry.
What is motivation or use case for adding/changing the behavior?
The motivation is to support use cases like script injection (wps-client in the demonstration).
How should this be implemented in your opinion?
By adding support for array syntax for the new case.
Are you willing to work on this yourself?
Possibly.

I found out the array syntax works within import. That's enough. 👍


Add interactive HQMTool and testing

Add testing of HQMTool. No coverage on interactive project notifications or on plotting.

I attempted to "properly" include the HQMTool functionality such that one only needs to type using HOHQMesh to load everything. This now gives errors because GLMakie is not in the dependencies although there is the following inclusion of Makie in src/HOHQMesh.jl https://github.com/trixi-framework/HOHQMesh.jl/blob/e3b5b5c16f59d804792c9e79ab871f0dbdba6cce/src/HOHQMesh.jl#L15
with the comment #Enable features that depend on the availability of the Makie package. So this
should handle the GLMakie too right? I am not sure exactly the issue @ranocha @sloede .
Locally it works but I get a warning
┌ Warning: Package HOHQMesh does not have GLMakie in its dependencies:
│ - If you have HOHQMesh checked out for development and have
│   added GLMakie as a dependency but haven't updated your primary
│   environment's manifest file, try `Pkg.resolve()`.
│ - Otherwise you may need to report an issue with HOHQMesh
└ Loading GLMakie into HOHQMesh from project dependency, future warnings for HOHQMesh are suppressed.

Also, this way of including would mean that one would always need to preface the HQMTool commands with HOHQMesh. to call it, e.g.,
p = HOHQMesh.newProject("projektnamn", "out_folder")

To avoid this would we need to export all the functions?

should handle the GLMakie too right?

No, Makie.jl is re-exported by GLMakie.jl, not the other way round. If you depend on GLMakie, you need to update the statement using Requires.jl appropriately.

You need to decide first whether you want to have GLMakie.jl as explicit dependency (in Project.toml) or only load the interactive stuff conditionally.

GLMakie.jl as dependency in Project.toml

Everybody installs GLMakie when installing HOHQMesh.jl, even if you just want to use it as thin wrapper of HOHQMesh
You don't need to worry about the availability of GLMakie

GLMakie.jl via Requires

Only available if a user explicitly uses GLMakie, too
Less dependencies for everybody not using this feature
Needs an additional logic to determine whether GLMakie.jl is loaded
Does not work well with precompilation

GLMakie.jl via Requires

Doing it this way is analogous to what is currently in there for Makie, right? Which uuid would I use for GLMakie because I see that GLMakie.jl has been deprecated

I think it should be
e9467ef8-e4e7-5192-8a1a-b1aee30e663a

see https://github.com/JuliaPlots/Makie.jl/blob/master/GLMakie/Project.toml

I think I know how to adapt the tests that @DavidAKopriva wrote and port them into the testing folder for coverage. However, my current understanding of giving HOHQMesh.jl access to all the HQMTool functions means that I would need to add HOHQMesh. to the front of all function calls. @ranocha @sloede Is this what we want for this functionality? Is there a way to not have to add this HOHQMesh preface?

I think I know how to adapt the tests that @DavidAKopriva wrote and port them into the testing folder for coverage. However, my current understanding of giving HOHQMesh.jl access to all the HQMTool functions means that I would need to add HOHQMesh. to the front of all function calls. @ranocha @sloede Is this what we want for this functionality? Is there a way to not have to add this HOHQMesh preface?

We can have a look at this tomorrow as well!

You need to export stuff if you do not want to use the prefix explicitly.

I've gone through the changes to the code and have the following comments.

UnRegisterForNotification is not currently used, but is a standard function for the observer pattern. For now it's probably OK to leave it commented out, but not to delete it.
removeInnerBoundaryCurveAtIndex currently doesn't have undo if there is only one curve. [Note: All @index routines are for program(er) use only, e.g. for undo purposes, not user facing]
The generate mesh call is inconsistent.
There is no way to visualize 3D at this point. Will have to be added in the future by someone.

Other than that, 98.XX% coverage is pretty darn good.

Further comment: What is the vision on this tool? Is it "HOHQMesh.jl" or a front end and separate tool, HQMTool.jl" to HOHQMesh. The way it is now organized is that the tool is now the HOHQMesh.jl. You don't need the took if you simply want to use a text editor, write/edit a control file and generate a mesh. The tool basically gives a graphical way to write a control file. Just a question of making a decision of what HOHQMesh is. For example if they are not separate, then the HONQMesh manual should be re-written from the user API point of view, not the control file point of view.

What is the vision on this tool?

My general vision is that HQMTool is a Julia written feature as a front end to interface with the power of the Fortran code. (@sloede @ranocha feel free to hop in and correct my vision if it is wrong).

You don't need the tool if you simply want to use a text editor, write/edit a control file and generate a mesh. The tool basically gives a graphical way to write a control file. We should make sure we have that vision before making a final merge.

That is the idea. HQMTool functions are there if you want it and provides a graphical interface to develop you 2D mesh / curves and then creates the control, mesh, and TecPlot files. However, if you only require the control file way and editing that directly HQMTool does not interfere with that and you can still use generate_mesh albeit with a different call signature.

For example if they are not separate, then the HOHQMesh manual should be re-written from the user API point of view, not the control file point of view.

That is very true. The docs will need a bit of restructuring / rewriting. Especially in the introduction part we need to make it clear that there are these two strategies and what HQMTool can help you with.

UnRegisterForNotification is not currently used, but is a standard function for the observer pattern. For now it's probably OK to leave it commented out, but not to delete it.

Now that I understand the code better I plan to uncomment this function and add it to the testing.

removeInnerBoundaryCurveAtIndex currently doesn't have undo if there is only one curve. [Note: All @Index routines are for program(er) use only, e.g. for undo purposes, not user facing]

True, I can add this undo ability. This is probably something a user will want because often the inner boundary is a single curve, e.g., a spline.

The generate mesh call is inconsistent.

I attempted to use multiple dispatch to make the generate_mesh function more generic such that it works for either the classic control file way or with a HQMTool Project. Is that what you mean with the inconsistency?

There is no way to visualize 3D at this point. Will have to be added in the future by someone.

Yeah, adapting this feature to 3D is "straightforward" in the sense that the bones are there. But one needs to write new plotting routines and an ExtrusionAPI.jl (or something similar) and some corresponding control file read-in routines for that HOHQMeshfeature.
That is a fairly nice (possibly large) student project. For now I think we can simply mention in the docs that 3D is not yet available.

What is the vision on this tool?

My general vision is that HQMTool is a Julia written feature as a front end to interface with the power of the Fortran code. (@sloede @ranocha feel free to hop in and correct my vision if it is wrong).

This is why I think they should be separate. HOHQMesh.jl should be the Julia front end to the fortran code. It takes the path to a control file and executes. HQMTool is an interactive front end that lets you build the model and then mesh while being able to view the process. So my view is that there should be two entry points: HOHQMesh.jl and HQMTool.jl (Or a variation or the name) rather than the combined that is there now at the top level. The first would be what Michael originally set up. The second is what is essentially there now, mostly just re-named. This way the documentation flows as it stands.

I agree with @DavidAKopriva that it would be nice to have different packages for the thin wrapper and the interactive Julia interface of HOHQMesh. For example, people who want to use Julia as a kind of package manager could just install HOHQMesh via HOHQMesh.jl without all the additional dependencies. HQMTool.jl (or InteractiveHOHQMesh.jl or something like that) could be another package, depending on the thin wrapper HOHQMesh.jl.

I would like to offer a slightly different point of view:
From a consistency point of view, I agree that having two separate names (and packages) has some benefits as to what is available and documented where. On the other hand, now that I've seen in action multiple times the new interactive functionality added in this PR, I seriously doubt that anyone is ever going to use HOHQMesh.jl  and its generate_mesh function only as a thin wrapper around HOHQMesh. Instead, everyone is going use the HQMTool, even if it is just to not having to write a control file by hand - your new creation is just too convenient 🙂
Therefore, in my head there's really only one tool - HOHQMesh - that has a backend written in Fortran and an interactive frontend written in Julia. Anyone who's happy to create their control file by hand is welcome to use the backend directly, but most new users are going to use the more convenient frontend anyways. Even if you exclude the really interactive part, it is - to me at least - much easier to write a "HOHQMesh elixir" using the add! commands etc. than directly editing the control file. Thus, considering the "user story", using a single name makes more sense to me, as it is just conceptually simpler.
In the end, this is not a decision for life, though. From a developer's point of view, having a single Julia package to maintain means generally less overhead. We could thus just start out with the frontend/backend storyline, and if we see down the road that it confuses users or has other downsides, we can always spin of the interactive functionality in a separate package.
Finally, I'd like to point out that this is more of an "outsider's" point of view, since I am not a core developer and thus my opinion should only be taken as a friendly suggestion. I might also be biased, since I find the idea of having a "hybrid" Fortran/Julia tool, combining the best of both worlds, very exciting and thus in my head this is really just "HOHQMesh".

Need a new figure for the docs now that the boundary labels are different


When Launching TEOC from Teams or Teams Web header and Name show up but blank body

teams.microsoft.com-1672759470218.log
Not sure what I did wrong worked well in the lab but in production getting what others have I tried redploying and going over the steps again.  attached is the console logs from my web teams session.

Hi @rsnyder-feld ,
Thanks for sharing the logs. I see the issue is with the SharePoint Site Name (/sites/feld_teoc) that was provided while creating a resource group.  Please update the REACT_APP_SHAREPOINT_SITE_NAME setting to feld_teoc in the resource group and try again.
To update the configuration, navigate to Azure Portal --> Resource group --> App service --> Configuration --> Edit the REACT_APP_SHAREPOINT_SITE_NAME setting --> Update the value to feld_teoc --> Click OK --> Click Save

I made the change and hit saved and the service restarted but even thought it just feld_teoc it still showing up wrong in the console.

its only been like 5 minutes so I may not be being patient enough.

Hi @rsnyder-feld ,
Can you please sync the code again after updating the above-mentioned configuration?

In Azure portal, navigate to TEOC resource group, open app service and click on deployment center.
Click on sync.
Wait until you see status as success for sync.
Then try to launch the application in Teams again.


4.0 beta 1: superEncoder is missing/fatal errors

Ran into this problem trying to save a model/content that contains child that is itself model/content.
In PostgresDataEncoder.swift line 93, the function
mutating func superEncoder(forKey key: Key) -> Encoder {
      fatalError()
}

blows up, because (I think) in FluentKit's Model.swift line 46:
public func encode(to encoder: Encoder) throws {
        var container = encoder.container(keyedBy: _ModelCodingKey.self)
        try self.properties.forEach { label, property in
            let encoder = LazyEncoder { container.superEncoder(forKey: .string(label)) }
            try property.encode(to: encoder)
        }
}

It calls superEncoder and sure enough you hit that fatal error.

FluentKit beta.2 no longer uses superEncoded, so this isn't as critical. PostgresDataEncoder/Decoder really should implement that method properly though.


Chinese documents?

Describe the feature
Do you have any Chinese documents?

There is no need for this， all apis is so clear that everyone can understand it.

This project is undergoing a major revamp; closing out old issues as part of the prep process.
The chatgpt package is pretty outdated at this point. I recommend that you use the openai package or the openai-fetch package instead.


IMobileServiceTableOperation.ExecuteAsync hides actual error information

Here is typical code in my TableController:
public class EventController : TableController<Event>
{
        public async Task<Event> PatchEvent(string id, Delta<Event> patch)
        {
            var current = Lookup(id);
            if (current == null)
            {
                // problem: error information is not delivered to IMobileServiceSyncHandler
                throw new HttpResponseException(this.Request.CreateNotFoundResponse());
            }

            var userId = await IdentityHelper.FindSidAsync(User, Request).ConfigureAwait(false);

            if (current?.Queryable?.FirstOrDefault()?.UserId == userId)
            {
                return await UpdateAsync(id, patch).ConfigureAwait(false);
            }
            else
            {
                return null;
            }
        }
...
}

Then on the client side I am trying to handle sync conflicts as follows:

    internal class MobileServiceSyncHandler : IMobileServiceSyncHandler
    {
        public async Task<JObject> ExecuteTableOperationAsync(IMobileServiceTableOperation operation)
        {
            MobileServiceInvalidOperationException error = null;
            Func<Task<JObject>> tryExecuteAsync = operation.ExecuteAsync;

            int retries = 0;
            do
            {
                try
                {
                    error = null;
                    var result = await tryExecuteAsync().ConfigureAwait(false);
                    return result;
                }
                catch (MobileServiceConflictException e)
                {
                    error = e;
                }
                catch (MobileServicePreconditionFailedException e)
                {
                    error = e;
                }
                catch (MobileServiceInvalidOperationException e)
                {
                    error = e;
                }
                catch (Exception e)
                {
                    throw e;
                }

                if (error != null)
                {
                    // problem: StatusCode is never "NotFound"
                    if (error.Response?.StatusCode == System.Net.HttpStatusCode.NotFound && operation.Kind == MobileServiceTableOperationKind.Update)
                    {
                        // tried to update an item that does not exists on server, change to insert operation
                        tryExecuteAsync = async () => await operation.Table.InsertAsync(operation.Item).ConfigureAwait(false) as JObject;
                        continue;
                    }
                    else
                    {
						...
                    }
                }
            }
            while (error != null && ++retries < 2);

            if (error != null)
            {
                operation.AbortPush();
            }

            return null;
        }
    }

So if I throw "HttpResponseException(this.Request.CreateNotFoundResponse())" in my TableController, the exception thrown from  IMobileServiceTableOperation.ExecuteAsync is "MobileServiceInvalidOperationException" with a message "Mobile Service table operation returned an unexpected response." and there is no other useful information available from the exception object. I guess MobileServiceInvalidOperationException may be thrown for several other reasons too, so I cannot assume it is always the "NotFound" case.
In other words it is now very difficult to handle a conflict where we are trying to update an item that does not exist on the server.

There is an embedded exception in MobileServiceInvalidOperationException.  Have you checked that?

Yes I checked everything that is passed to the method. If I remember correctly InnerException is null.

Thanks for using Azure Mobile Apps. I'm closing this question as it seems to be over a year old. Please feel free to reach out to us in case you have any questions.


Add more ContentApp cluster handlers

Problem

ContentApp clusters do not route messages to the ContentApp

Change overview

Add cluster handlers for all ContentApp clusters

Testing

chip-tool commands
TODO: integration tests

// launch tv-app
./out/host/chip-tv-app --secured-device-port 5640 --secured-commissioner-port 5552
// commission it from chip-tool
./out/host/chip-tool pairing onnetwork-long 1234 20202021 3840
// launch app named "App3" which is a dummy app on linux, which puts it at endpoint 6
./out/host/chip-tool applicationlauncher launch-app foo1 1 App3 1234 1
// read the application launcher list
./out/host/chip-tool applicationlauncher read application-launcher-list 1234 6
// read the accepts header for content launcher
./out/host/chip-tool contentlauncher read accepts-header-list 1234 6
// use content launcher to launch a url
./out/host/chip-tool contentlauncher launch-url http://www.yahoo.com foo2 1234 6
// use content launcher to launch content
./out/host/chip-tool contentlauncher launch-content 1 data1 1234 6
// use target navigator to read list of targets
./out/host/chip-tool targetnavigator read target-navigator-list 1234 6
// use target navigator to navigate to target 2
./out/host/chip-tool targetnavigator navigate-target 2 foo1 1234 6

/rebase


All Entities derive from an KindProvider interface

#2819

Awesome - does this mean the branch can now be merged to master ?

Yes, i think this about covers it.  I didn't implement EntityKindProvider for every possible EntityKind as it might be a bit heavyweight for some like the flows etc.


Question about L3 and L2 CAT and MBA

Hi,
I saw in README that both L2 and L3 CAT are supported in the software.
(1) Is L2 shared among all cores or only between the hardware threads?
(2) Is there any Intel CPU that supports both L2 and L3 CAT?
As to the Memory Bandwidth Allocation,
(3) Is there any Intel CPU that support Memory Bandwidth Allocation now? When should we expect to purchase it on the market?
Thank you very much for your help!
Meng

Hi Meng,
Thanks for getting in contact and showing an interest in RDT. We would love to hear how you are using these tools and would be happy to receive feedback on your experience.
To answer your questions:

It depends on the product type. L2 is designed a different way depending on the product you have. Have a look into Intel's SDM for more information: https://software.intel.com/en-us/articles/intel-sdm
This user space software enables L2 /L3 CAT and MBA based on the SDM linked above. I cannot advice on products implementing these technologies apart from the table in the README.
Same as above.

Best,
Aaron

Hi Meng,
Apologies that we cannot be more helpful here.
For validating L2 CAT software implementation we used one of these parts:
http://ark.intel.com/products/codename/63508/Denverton?q=denverton
and this product family doesn't have L3 cache.
I hope it helps
Tomasz

Hi Tomasz,
Thank you very much for your information! :-)
Meng

Let me close this issue. Feel free to open a new one if you have further questions.
Tomasz


How should I embed goby into my application

I have an existing golang application , How should I embed goby into my application?

Every component of Goby like parser, code generator and the VM is a go package, so you can import and use them in your application like the goby.go does (not exactly the same though, but you should get the idea).
However, I personally don't think there's any practical reason to use Goby in a Go application, as it'll become an unnecessary abstraction layer for the application. Would you mind share the motivation behind this idea, I'm curious about it 😄

Like a groovy ，Make a  pipline DSL

hmm, I never thought about this before 🤔
do you mean, for example, having something like config.gb in a Go application that uses Goby-defined dsl for easier configuration?

yes

that's really an interesting idea, thanks for sharing!
I believe you can do that under the current implementation, but I can't guarantee it'd be easy though as I didn't have this usage in mind when building it 😂  let me know if you find any issue or have any suggestion when trying Goby
btw, I just recalled that I did use pure Goby to build a dsl-based test framework (inspired by a popular Ruby testing framework called rspec) and load it like this. maybe you can do it in a similar fashion?

tks

thank you for your help

np, happy to learn the new idea from you. thank you too 😄


made option in Menubar to upload Doc and render handwriting

Added the option in Menubar to upload document , with same functionality as before
the issue of: 100+ word was because of text area, which has been taken care.(i tried with 1100+ word file)

Closes: #186
If 100+ words is till a issue please can i get a sample file.

many merges are made and so i will have to close this PR and make a New one.

Added the option in Menubar to upload document , with same functionality as before
the issue of: 100+ word was because of text area, which has been taken care.(i tried with 1100+ word file)

Closes: #186
If 100+ words is till a issue please can i get a sample file.

I just saw your commits. Your code works fine for a txt file but not a doc file. You can add this for now


adding a chapter on Sort and merge

Signed-off-by: ahmedEid1<EMAIL_ADDRESS>This PR adds a chapter on SORT and MERGE Verbs
Currently, there are no labs for the chapter.

@MikeBauerCA
this is the same PR, I kind of broke my Git history, so I recreate the PRs on new branches.
sorry, if this caused any problems.

I added all the comments that were on the original PR

@MikeBauerCA

Please review how other chapters are set up with a brief introduction followed by a table of contents for the chapter. Try to be consistent with that format.

DATA DEVISION >> DATA DIVISION

Was the following resource leveraged to construct this content? http://www.csis.ul.ie/cobol/course/SortMerge.htm

Was the following resource leveraged to construct this content? http://www.csis.ul.ie/cobol/course/SortMerge.htm

Yes, the structure of the chapter and some of the content is from it

I got the problem.
I thought it was a blog post or something so I didn't check for Copyrights (which is a big mistake).
will remove all the parts from this resource, and rewrite the chapter

Yes or alternatively, you could seek approval from the author and see if they would like to be involved in this initiative.

Didn't get a response back from the author, so I rewrote the chapter.


Add feature to have bot add a reaction when it forwards a message from the user

Adds a positive indicator that the bot saw a message sent by the user.  The way you customize the emoji is pretty awkward but I haven't been able to figure out how to allow unicode codepoints without it being escaped by the ini module.

This is merged to the dev branch, but GitHub doesn't allow me to change the base anymore for some reason so it's still showing up as "Open." Should be marked as "Merged" once I merge the dev branch into master.


Normalize.css bug when using Aurelia UX with Webpack on Cordova

I'm submitting a bug report

Library Version:
0.10.1

Please tell us about your environment:

Operating System:
ios web app built with Cordova

JSPM OR Webpack AND Version
webpack 4.8.3

Browser:
iOS XX UIWebView

Language:
TypeScript X.X

Current behavior:
When building the app, the AureliaUX styles are not taken into account (all the root CSS vars). This comes from the fact that AureliaUX creates a wrong line [object Object] in the styles section (see below).

When building the app for the web au run --watch the styles work properly (no problem). I've tested in Chrome and Safari (MacOS) and it works correctly.
However, when running the same web app inside a cordova project, I've got this line coming in the styles sections. What's strange is that the following  tag contains the proper normalize.css code, but for some reason, only in this environment, AureliaUX writes this wrong object line in the AurliaUX styles tag.
Note also that I've used the same code source in an AureliaCLI with RequireJS project and here also: no problem. Only when building with AureliaCLI and webpack  I have the problem.
I've looked a little bit at the code but have no idea where this come from. Any idea?

I currently found a temporary fix by deactivating the normalize.css script when calling @aurelia-ux/core library:
.plugin(PLATFORM.moduleName('@aurelia-ux/core'), (config) => {
  // do nothing, so no call to defaultConfiguration() nor cssNormalize()
})

This has been fixed by https://github.com/aurelia/ux/commit/1a38f1439f88d30e036a74f40019da1eda317425. Thanks to @MaximBalaganskiy


branch: interoperable

The "together" branch contains the brand for interoperable PhET sims. It was migrated from phetbrand/brand 'together' branch.

Renamed this branch to "interoperable".

This branch should be deleted when all brands are moved to master.

All brands got moved to master yesterday, but I'd like to look around the other branch issues before deleting these branches.  Reassigning to myself.

I can't think of anything else we will need this branch for.  Deleting and closing.


Could you release the code for CIFAR100 dataset?

Really appreciate your work. It is really impressive.
I'd like to replicate the results using CIFAR-100 as the in-distribution dataset. Is it possible to access the code for the CIFAR-100 dataset or receive guidance on image transformations and dataset splitting, such as using CIFAR-10 as the out-of-distribution dataset?
Thank you very much.

Hi,
Thanks for your interest in our work. All experiments on Cifar are conducted based on CLIPN with hand-crafted prompts.
The original plan is to release those codes this month. However, I found it hard to free up my time to reorganize them until mid-November due to several very important things, such as preparing for my PhD PQE. I will release them before the end of November. When the codes are released, I will contact you!
Bests,
Hualiang

OK! Thank you very much!

Sorry to bother you ...
I am interested in CLIPN and am eagerly awaiting the release of the code using CIFAR-10 and CIFAR-100.
When will it be released?

Hi, the code is coming. I am writing some necessary code comments. I will upload it tomorrow.

Hi, codes are released. Although there are still some hard codes that need to be optimized further, the main body of the code is ready to run.


SolveFileLocation

The content manager should try .xcf in case no extension was provided before querying the disk.

https://github.com/ThuCommix/Sharpex2D/commit/01bdb5ffe07fdb850166bc530187baf0cf59313b


Change SSL_CERT_DIRS to SSL_CERT_DIR

Description
This is the bug to populate wrong environment variable name for SSL_CERT_DIR. To unblock this bug, user can set variable name for ssl cert location like below.
rad install kubernetes --reinstall --set global.rootCA.sslCertDirEnvVar=SSL_CERT_DIR --set-file global.rootCA.cert=<CERT FILE>

Type of change

This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #6955
Auto-generated summary

copilot:all

I will not merge until user confirmed the workaround solution verifies.


H2 client protocol adjustments

a9e74d3 is about the changes we discussed in the latest meeting
499dd49 is mostly unrelated, fixing some (optional) type hints and adding missing ones

@elacuesta Changes looks good -- but no change to connectionLost method? In a way the stream still pops the Stream instance from streams dict

Right, I forgot about https://github.com/scrapy/scrapy/pull/4610#discussion_r447240521. I'll update shortly, thanks for mentioning it.


Pkg: Add section and panel content containers

Was hoping to avoid an additional container but it will solve problems with bottom margins. The problem now is that if a section has a footer, then the content above will NOT be the last-child and will retain bottom margin. The system relies on last-child block elements NOT having bottom margins so that the parent container padding will supply the spacing.
Add new container .tn-section__content. It is a wrapper only, it should have no styles.
.tn-section
    .tn-section__header
        .tn-section__title
        .tn-section__actions
    .tn-section__content
    .tn-section__footer

Do the same with the panel.
Also, revert the negative bottom margin of card-group to the last child only.

I found a different approach to this. The overhead of requiring an extra container is bad because it forces two containers when using a section, even without a header or footer. And the only purpose of that content is to control margins of the content.
Instead, it is better to set bottom margins of all the block components — tn-table, tn-tree, tn-card-group — with the :last-child bottom margin removed. And to remove the top margin of the footer elements. That way, if the footer is present the content above will provide the margin (since it won't be the last child).
The only side effect is that the margin between the content and the footer will now be 20px instead of 16px. A small price to pay to avoid the overhead of an additional presentation-only container.


Installation fault

I was trying to install GraphBlast on Ubuntu 16.04, checked software versions (CUDA 9.1, Boost 1.58, g++ 4.9.3) and followed the instruction step by step, but got the errors.
Also, the flags recommended in this issue didn't help.

Looking at the error log, this line says you are using g++ 5.x:
 /usr/lib/gcc/x86_64-linux-gnu/5/include/mwaitxintrin.h(36): error: identifier "__builtin_ia32_monitorx" is undefined
Could you verify you are using g++ 4.9.3 by doing g++ -v?
If you are changing between g++ versions, you need to delete the CMakeCache.txt and CMakeFiles directory before doing cmake .. again.

Yes, you are right, the compiler identification was 5.x. I checked all versions again, fixed this problem, but there are still errors.

Sorry, I was busy for the last little while. Are you still getting the same error? I have not encountered your error before, but I would perhaps try adding the flags recommended in this issue, clearing the CMakeCache.txt and CMakeFiles directory before redoing cmake ...


Day305

Resolves #622.

Pull Request Test Coverage Report for Build 1163

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1153:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Show hours instead of exact date and time of comment on post creation

It can be just adjusted as we are updating the post feed time and date!

Fixed in #9


GEODE-7683: introduce BR.cmnClearRegion

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I'm not 100% sure I understand what this is implementing. Clearing a BucketRegion still fails the same way it has in the past.
I wrote a function that attempts to clear a PartitionedRegion by clearing all of its primary BucketRegions. To do that, it tried these API:
br.basicClear(regionEvent);
br.basicClear(regionEvent, true);
br.cmnClearRegion(regionEvent, true, true);

No matter which of those it called, this exception occurs:
Caused by: org.apache.geode.InternalGemFireError: Trying to clear a bucket region that was not destroyed or in initialization.
	at org.apache.geode.internal.cache.BucketRegion.updateSizeOnClearRegion(BucketRegion.java:2128)
	at org.apache.geode.internal.cache.AbstractRegionMap.clear(AbstractRegionMap.java:460)
	at org.apache.geode.internal.cache.BucketRegion.clearEntries(BucketRegion.java:2479)
	at org.apache.geode.internal.cache.LocalRegion.clearRegionLocally(LocalRegion.java:8544)
	at org.apache.geode.internal.cache.BucketRegion.cmnClearRegion(BucketRegion.java:578)
	at org.apache.geode.internal.cache.BasicClearBucketRegionFunction.basicClear(BasicClearBucketRegionFunction.java:32)
	at org.apache.geode.internal.cache.BasicClearBucketRegionFunction.execute(BasicClearBucketRegionFunction.java:21)
	at org.apache.geode.internal.cache.execute.AbstractExecution.executeFunctionLocally(AbstractExecution.java:328)
	at org.apache.geode.internal.cache.execute.AbstractExecution.lambda$executeFunctionOnLocalPRNode$0(AbstractExecution.java:273)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at org.apache.geode.distributed.internal.ClusterOperationExecutors.runUntilShutdown(ClusterOperationExecutors.java:449)
	at org.apache.geode.distributed.internal.ClusterOperationExecutors.doFunctionExecutionThread(ClusterOperationExecutors.java:379)
	at org.apache.geode.logging.internal.executors.LoggingThreadFactory.lambda$newThread$0(LoggingThreadFactory.java:119)
	at java.lang.Thread.run(Thread.java:745)

AFAICT, a BucketRegion still can't be cleared.
When I hack-implemented this feature, I changed BucketRegion.updateSizeOnClearRegion to do this instead of throwing the InternalGemFireError. I'm not sure if this is valid, though.
oldMemValue = this.bytesInMemory.getAndSet(0);

This looks like a loose-end I did not found earlier. I will do that fix. And I will add this dunit test case in PR.clear.

there's some dunit tests fail from time to time on different test. I think it's unstable in base branch i.e feature/GEODE-7665


Add support for best/ being the new default sort

This is super quick and dirty and I haven't written any tests.

Thanks for the PR! Overall the changes look pretty straightforward and I will try to get around to cleaning this up a bit and merging soon. The praw changes will need to be merged into my fork @ https://github.com/michael-lazar/praw3

Sorry, I have been busy focusing on other endeavors and haven't gotten around to it yet. Thanks for the bump though.


Sorry you are not qualified to join

Hi, I live in France
Phone number ok
3 devices ok
But... Airdrop is not following.
My account is rabo69
Thank you in advance

I live in France
Phone number ok
3 devices ok
But... Airdrop is not following.
My account is aneesh
Thank you

I live in Germany
Phone number ok
3 devices ok
But... Airdrop is not following.
My account is nilkos
Thank you

I'm in an approved country, Poland with an approved phone number.
Airdrop says I am qualified with 3 devices and "A beautiful smile"
Then after selecting the "Become a lucky Airdropee" button it switches to
"Sorry you are not qualified to join"
My account is quickly123
Thanks for your review

I live in Italy
Phone number ok
4 devices ok
But... Airdrop is not following.
My account is unica_design
Thank you


Adding precommit with black and pyupgr

As suggested in #109, this PR adds pre-commit hooks using the pre-commit package. To start off with, it has the yaml-fixer hook, the black hook, and the pyupgrade hook. I think it's nice to run pyupgrade on tests too, so I didn't exclude the tests from it, but I can exclude them if you feel otherwise.

Codecov Report

Merging #119 (f2d67a6) into main (2d06a87) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   88.94%   88.94%
=======================================
  Files          11       11
  Lines        2098     2098
=======================================
  Hits         1866     1866
  Misses        232      232

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


Prometheus rework

Fix
After #939, team members have agreed in the following:

remove deploy active metric
add clean up to metrics endpoint and increase scrapping time interval (no need to be every 15s)
add time window to exporters (only accept data with timestamp as value that is higher than exporter instance creation, but it not more than a threshold) (except committime exporter, because it only collects commit after a deployment)
explain this new behavior in the docs to user

Requirements

[ ] remove deploy active metric
[ ] OPTIONAL Test if it is possible to deploytime metric do not use exactly timestamp (prometheus would use timestamp it received (because of ~3horus receive problem) and use exactly timestamp as value)
[ ] add clean up to deploytime, committime and failure exporter
[ ] add clean up to webhook exporter (seems harder than on the old exporters)
[ ] add time window to exporters (except committime exporter)
[ ] explain this new behavior in the docs to user

[ ]  remove deploy active metric

This is resolved by https://github.com/dora-metrics/pelorus/pull/943

[ ]  OPTIONAL Test if it is possible to deploytime metric do not use exactly timestamp (prometheus would use timestamp it received (because of ~3horus receive problem) and use exactly timestamp as value)

agree, additional tests:

check scenario where we deploy test application let's call it TestApp

deploy TestApp it 10 times every 30min (can be a simple shell script with sleep)
wait 2h before going into grafana
After that test number of deploymets over the following periods:

last 2 hours, should give 0
previous 2h (so 4h ago -> 2h ago) should give 4 deployments;
last week - should give 10 deployments.

check scenario where we create 10 deployments using webhook exporter and then restart webhook exporter (just delete running webhook instance).

wait 10 min
check how many deployments happened in the last week
check how many deployments happened in the last 5 minutes

[ ]  add clean up to deploytime, committime and failure exporter

This I think is't really required as the scrape always get the new Gauge object, so we could have this for free. Just need to check if the caching in committime requires some cleanup.

[ ]  add clean up to webhook exporter (seems harder than on the old exporters)

Agree. There should be a lifetime of metric after which it is removed. The way I see it:

Collect metrics not older than X (as implemented in #943)
Add function to check if the time of event where it happened is not greater then Y and remove it if it's true.

[ ]  add time window to exporters (except committime exporter)

As @weshayutin reworded above.

[ ]  explain this new behavior in the docs to user

Agree.

RE: cleanup functionality
I think we should create a new class for the metrics that inherits from GaugeMetricFamily, let's call it PelorusGaugeMetricFamily.
This class should have at least three additional methods:
def clear_all_metrics(self) -> int:
"""
Clears out all the in memory metrics from the GaugeMetricFamily.

Returns:
  - number of metrics removed
"""

def clear_metrics(self, threshold_sec: int) -> int:
"""
Clears out the in memory metrics from the GaugeMetricFamily, which are
above certain threshold

Arg:
  - threshold_sec (int): The time threshold in seconds. All metrics that occurred `threshold_sec` seconds ago or earlier will be removed.

Returns:
  - number of metrics removed.
"""

def remove_metric(self, labels: Sequence[str]) -> int:
"""
Remove particular metric from the GaugeMetricFamily.
Reverse function to GaugeMetricFamily.add_metric().

Arg:
  - labels Sequence[str]: The labels representing metric to be removed.

Returns:
  - number of metrics removed.
"""


New bw

Change BLDFR in Table, to be consistent with mean SO12 ranges
Remove BW from Table
Derive new BW, based on BLDFR_URB2D,  FRC_URB2D and SW
Update JOSS draft accordingly

This change is done now, addressing #26.


Rewrite dumper parameters

One last change to make things clearer I hope:

Separate choices for SQL and other files: --dump_sql --dump_all_files or --dump_selection
List of files and their numbers is in one simple map

Looks good to me. 👍
The only comment (that I have noticed now) is that it may be wiser to declare dump_selection as param.dump_selection and use it directly in the subworkflow instead of passing it, as I have found this "passing a variable instead of a channel" sometimes dangerous and leading to hard-to-notice bugs.

I'm kind of thinking the opposite? I'm trying to avoid using too many params values as global parameters, i.e. we should be able to use the subworkflows by providing all info necessary directly instead of -- sometimes hard to find -- params values.
(One example where that would cause a problem is when we call a subworkflow several times in the same workflow).


setFlash not working

Quick explainer:
a have a +page.svelte @ '/' from there I POST to '/other' where form actions handle the request and call setFlash. But +page.svelte @ '/other' does not get the flash message.
/ --POST--> /other
here's a repord repo:
https://github.com/philippviereck/flash-message-reprod
I didn't dig deep, but if here event.cookies would be used instead of the header, it works. 🤷♂
  const header = event.request.headers.get('cookie') || '';
  if (!header.includes(cookieName + '=')) {
    //d('No flash cookie found.');
    return { [cookieName]: undefined };
  }

  const cookies = parse(header) as Record<string, string>;
  const dataString = cookies[cookieName];

Should be fixed now in 2.2.1, can you confirm that it is?

I can confirm

Thank you, and thanks for figuring out the problem. :)


User Customization

Fixes #514, fixes #432, fixes #508, fixes #538, fixes #581, fixes #598, references #212
Changes proposed:

[x] Add
[ ] Prepare
[ ] Delete
[x] Fix

Things I've done:

[x] My pull request fixes an issue, I referenced the issue.

This introduces user customizations. I've also made changes to the project structure and refactored some parts of the code like iconGenerator tests and others.
The Docs are still a draft version but we could go with that for the moment.

@jens1o what do you mean by that?

The svg is hardcoded everywhere, and you can't change other files... https://github.com/robertohuertasm/vscode-icons/pull/596/commits/fc8e8f837fe0602bd7fa774ece70ee1e6b3a1809#diff-ec3153353ded9d38b2283de751d804ceR32
I would like to have it responsive, so it could even work with png.

@jens1o This is something that I'll change once @robertohuertasm merges the PR.

@jens1o this is only for the default schema which it's controlled by us. If the user wants to change the icon she can use whatever extension we decide to support.

@robertohuertasm He means the file type association of the default icons.

@jens1o @JimiC you're right.

Thanks @JimiC, so I'm approving, when the bug has been fixed Jimi pointed out.

I didn't have the time to end default icons customization functionality. I have prepared the code to end it tonight.

Still WIP. Validations and tests pending.

I think it's done. For sure there will be things that can be improved  and maybe docs will have to be extended. I've also provided a new preset for hiding all the folders (like in the old times) 😉 and new tests.
As usual, feel free to comment or suggest anything.

@robertohuertasm Great work there. Just wondering if we can override the Angular2Preset icons?

Regarding the absolute path issue, we have a getRelativePath function, can't we use it to support absolute path -> relative path?

Is there any mechanism that ensures that in case of a custom file/folder declaration, the custom folder and the file for the icon exists (assuming it follows the vscode-icons file naming convention)?

@JimiC, about the angular2 preset... it can not be overriden at the moment, you're right. It's tightly coupled to the icon names. Maybe we can use some kind of convention... let's say, that it will enable/disable all icons starting with ng2_. I will rework it.

@JimiC, about the mechanism to ensure the existence of an icon, it's not present. In the case the icon is not there it just won't be shown. This could be a nice to have that we could introduce in another PR.

Angular2 renamed to Angular to follow AngularJS naming style and new convention about Angular icons: They must start with ng_. This way, anyone will be able to customize them and take advantage of the preset.

@JimiC, finally the relative approach is working! It's already pushed! Thanks!

Permission issues arising in Mac... as usual... this time is package.json modification by the iconGenerator. I'll make this part optional as it's only needed for us when we publish the extension.

@robertohuertasm custom_removal has been rebased.


This link does not work - https://azure.github.io/Vision-AI-DevKit-Pages/docs/Wi-Fi_issues/

No working - https://azure.github.io/Vision-AI-DevKit-Pages/docs/Wi-Fi_issues/

Fixed this. The correct link is https://azure.github.io/Vision-AI-DevKit-Pages/docs/wifi_issues/

Thanks
Sent from my iPhone
On Sep 17, 2019, at 2:20 PM, Jussi Niemelä <notifications<EMAIL_ADDRESS>wrote:
Closed #241https://github.com/microsoft/vision-ai-developer-kit/issues/241.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vision-ai-developer-kit/issues/241?email_source=notifications&email_token=AMGC6HEC7VZXDNTYBOK45P3QKEUY7A5CNFSM4IXFPHWKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTVVREBQ#event-2641039878, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMGC6HH7FQW3TXSWNR6WOM3QKEUY7ANCNFSM4IXFPHWA.


Cerbos compile failed

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Error :
Missing Imports
resource_channel.yaml: Import 'user_access' not found
Load failures
role_user_access.yaml: failed to unmarshal JSON: proto: syntax error (line 1:486): unexpected token [
cerbos: error: failed to compile
Expected Behavior
Expected to compile without any error
Steps To Reproduce

config:

---
server:
  httpListenAddr: ":3592"
storage:
  driver: "disk"
  disk:
    directory: "./tutorial/policies"

policies:
role_user_access.yaml

---
apiVersion: api.cerbos.dev/v1
description: |-
  Only accessed by author, admin or owner of the resource
derivedRoles:
  name: user_access
  definitions:
    - name: access
      parentRoles: ["user"]
      condition:
        match:
          any:
            of:
              - expr: request.resource.attr.ownerId == request.principal.id
              - expr: request.principal.id in request.resource.attr.authorIds

    - name: visible
      parentRoles: ["user"]
      condition:
        match:
          any:
            of:
              - expr: request.resource.attr.visibility == "PUBLIC"
              - expr: request.resource.attr.payed == true

    - name: private
      parentRoles: ["user"]
      condition:
        match:
          - expr: request.resource.attr.visibility == "PRIVATE"

resource_user.yaml
---
apiVersion: api.cerbos.dev/v1
resourcePolicy:
  version: "default"
  resource: "user"
  rules:
    - actions:
        - create
      effect: EFFECT_ALLOW
      roles:
        - user
    - actions:
        - update
        - read
      effect: EFFECT_ALLOW
      roles:
        - user
      condition:
        match:
          expr: request.resource.id == request.principal.id
    - actions:
        - "*"
      effect: EFFECT_ALLOW
      roles:
        - admin

resource_channel.yaml
---
apiVersion: api.cerbos.dev/v1
resourcePolicy:
  version: "default"
  importDerivedRoles:
    - user_access
  resource: "channel"
  rules:
    - actions:
        - create
        - update
        - delete
        - read
      effect: EFFECT_ALLOW
      derivedRoles:
        - access
    - actions:
        - read
      effect: EFFECT_ALLOW
      derivedRoles:
        - visible
    - actions:
        - "*"
      effect: EFFECT_ALLOW
      roles:
        - admin
    - actions:
        - "*"
      effect: EFFECT_DENY
      derivedRoles:
        - private

compile command:

docker run --rm --name cerbos -t \
  -v "$(pwd)/tutorial:/tutorial" \
  ghcr.io/cerbos/cerbos:latest compile "./tutorial/policies"

Environment
- OS: Windows 11
- Cerbos version: 0.23.1
- Docker version: 4.15.0
- Go version: go1.19.3

Anything else?
No response

Hi @mohamadnosratian,
The problems seems to occur here (role_user_access.yaml:L30);
            - expr: request.resource.attr.visibility == "PRIVATE"

The line above should be like this because match doesn't expect list;
            expr: request.resource.attr.visibility == "PRIVATE"

Let us know if this doesn't solve your problem.


cmake: XMLRPC not found

We are using ros:noetic
Recently (1 week ago from now), it seems that cmake was not able to find xmlrpc package
We first run rosdep install --from-paths src --ignore-src -y.
We can see that xmlrpc is installed in the logs:
"The following additional packages will be installed:
libxmlrpc-c++8v5 libxmlrpc-core-c3 libxmlrpc-core-c3-dev
Suggested packages:
xmlrpc-api-utils
The following NEW packages will be installed:
libxmlrpc-c++8-dev libxmlrpc-c++8v5 libxmlrpc-core-c3 libxmlrpc-core-c3-dev
0 upgraded, 4 newly installed, 0 to remove and 0 not upgraded.
The following additional packages will be installed:
libxmlrpc-c++8v5 libxmlrpc-core-c3 libxmlrpc-core-c3-dev
Suggested packages:
xmlrpc-api-utils
The following NEW packages will be installed:
libxmlrpc-c++8-dev libxmlrpc-c++8v5 libxmlrpc-core-c3 libxmlrpc-core-c3-dev
0 upgraded, 4 newly installed, 0 to remove and 0 not upgraded."
We then run catkin_make and then cmake reports that it cannot find XMLRPC that:
"CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
XMLRPC was not found.  Make sure the entries XMLRPC_* are set.  (missing:
XMLRPC_C_FOUND)
Checking At CMakeErrors.log, it seems that it cannot find libpthread:
Performing C SOURCE FILE Test CMAKE_HAVE_LIBC_PTHREAD failed with the following output:
Change Dir: /root/ws/build/CMakeFiles/CMakeTmp
Run Build Command(s):/usr/bin/make cmTC_0de78/fast && /usr/bin/make -f CMakeFiles/cmTC_0de78.dir/build.make CMakeFiles/cmTC_0de78.dir/build
make[1]: Entering directory '/root/ws/build/CMakeFiles/CMakeTmp'
Building C object CMakeFiles/cmTC_0de78.dir/src.c.o
/usr/bin/cc   -DCMAKE_HAVE_LIBC_PTHREAD   -o CMakeFiles/cmTC_0de78.dir/src.c.o   -c /root/ws/build/CMakeFiles/CMakeTmp/src.c
Linking C executable cmTC_0de78
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_0de78.dir/link.txt --verbose=1
/usr/bin/cc  -DCMAKE_HAVE_LIBC_PTHREAD    -rdynamic CMakeFiles/cmTC_0de78.dir/src.c.o  -o cmTC_0de78
/usr/bin/ld: CMakeFiles/cmTC_0de78.dir/src.c.o: in function main': src.c:(.text+0x46): undefined reference to pthread_create'
/usr/bin/ld: src.c:(.text+0x52): undefined reference to pthread_detach' /usr/bin/ld: src.c:(.text+0x63): undefined reference to pthread_join'
collect2: error: ld returned 1 exit status
make[1]: *** [CMakeFiles/cmTC_0de78.dir/build.make:87: cmTC_0de78] Error 1
make[1]: Leaving directory '/root/ws/build/CMakeFiles/CMakeTmp'
make: *** [Makefile:121: cmTC_0de78/fast] Error 2
Do you happen to have a clue about what is happening please?

I think this is a problem with your system. Can you try in a clean environment?
Do you have a minimal, reproducible example?

I pulled the latest ros:noetic docker image and ran it.
I created a small CMakeLists.txt and added :
find_package (XMLRPC REQUIRED c++2 client)
THe output is the following:
No project() command is present.  The top-level CMakeLists.txt file must
contain a literal, direct call to the project() command.  Add a line of
code such as
project(ProjectName)

near the top of the file, but after cmake_minimum_required().
CMake is pretending there is a "project(Project)" command on the first
line.
This warning is for project developers.  Use -Wno-dev to suppress it.
-- The C compiler identification is GNU 9.3.0
-- The CXX compiler identification is GNU 9.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
XMLRPC was not found.  Make sure the entries XMLRPC_* are set.  (missing:
XMLRPC_C_FOUND XMLRPC_LIBRARIES)
Call Stack (most recent call first):
usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:393 (_FPHSA_FAILURE_MESSAGE)
usr/share/cmake-3.16/Modules/FindXMLRPC.cmake:126 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
//CMakeLists.txt:1 (find_package)
CMake Warning (dev) in //CMakeLists.txt:
No cmake_minimum_required command is present.  A line of code such as
cmake_minimum_required(VERSION 3.16)

should be added at the top of the file.  The version specified may be lower
if you wish to support older CMake versions for this project.  For more
information run "cmake --help-policy CMP0000".
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Configuring incomplete, errors occurred!
See also "//CMakeFiles/CMakeOutput.log".
I had a look at the CMakeOutputs.log and there is no error inside...
I tried different cmake version, but it does not work
Can you reproduce the error on your side ?
cmake version 3.16.3.

Additonally, i also installed XML-RPC-C separatly:
apt-get install libxmlrpc-core-c3-dev
But i still have the same error

I also checked which xmlrpc packages i have:
apt list --installed | grep -i xmlrpc
WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
libxmlrpc-c++8-dev/focal,now 1.33.14-8build2 amd64 [installed]
libxmlrpc-c++8v5/focal,now 1.33.14-8build2 amd64 [installed,automatic]
libxmlrpc-core-c3-dev/focal,now 1.33.14-8build2 amd64 [installed]
libxmlrpc-core-c3/focal,now 1.33.14-8build2 amd64 [installed,automatic]
ros-noetic-xmlrpcpp/focal,now 1.15.14-1focal.20220106.232911 amd64 [installed,automatic]

So i updated : find_package (XMLRPC REQUIRED c++2 client) to find_package (XMLRPC REQUIRED) on my docker image and relaunch catkin_make
It has found XMLRPC packages. But later on, it crashed when  linking. It gave the following log:
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function xmlrpc_c::value_string::~value_string()': psu_driver.cpp:(.text._ZN8xmlrpc_c12value_stringD2Ev[_ZN8xmlrpc_c12value_stringD5Ev]+0x18): undefined reference to xmlrpc_c::value::~value()'
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function xmlrpc_c::value_int::~value_int()': psu_driver.cpp:(.text._ZN8xmlrpc_c9value_intD2Ev[_ZN8xmlrpc_c9value_intD5Ev]+0x18): undefined reference to xmlrpc_c::value::~value()'
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function xmlrpc_c::rpcOutcome::~rpcOutcome()': psu_driver.cpp:(.text._ZN8xmlrpc_c10rpcOutcomeD2Ev[_ZN8xmlrpc_c10rpcOutcomeD5Ev]+0x2c): undefined reference to xmlrpc_c::value::~value()'
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function xmlrpc_c::rpcOutcome::rpcOutcome(xmlrpc_c::rpcOutcome&&)': psu_driver.cpp:(.text._ZN8xmlrpc_c10rpcOutcomeC2EOS0_[_ZN8xmlrpc_c10rpcOutcomeC5EOS0_]+0x47): undefined reference to xmlrpc_c::value::value(xmlrpc_c::value const&)'
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function rosasio::Connection::on_read(boost::system::error_code)': psu_driver.cpp:(.text._ZN7rosasio10Connection7on_readEN5boost6system10error_codeE[_ZN7rosasio10Connection7on_readEN5boost6system10error_codeE]+0x109): undefined reference to xmlrpc_c::paramList::paramList(unsigned int)'
/usr/bin/ld: psu_driver.cpp:(.text._ZN7rosasio10Connection7on_readEN5boost6system10error_codeE[_ZN7rosasio10Connection7on_readEN5boost6system10error_codeE]+0x129): undefined reference to xmlrpc_c::xml::parseCall(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >*, xmlrpc_c::paramList*)' /usr/bin/ld: psu_driver.cpp:(.text._ZN7rosasio10Connection7on_readEN5boost6system10error_codeE[_ZN7rosasio10Connection7on_readEN5boost6system10error_codeE]+0x1e2): undefined reference to xmlrpc_c::xml::generateResponse(xmlrpc_c::rpcOutcome const&, std::__cxx11::basic_string<char, std::char_traits, std::allocator >*)'
/usr/bin/ld: CMakeFiles/psud.dir/src/psu_driver.cpp.o: in function `auto rosasio::Node::Node(boost::asio::io_context&, std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&)::{lambda(auto:1&)#1}::operator()<xmlrpc_c::paramList const>(xmlrpc_c::paramList const&)
I think methods have not been compiled with c++ flag, therefore it cannot find them. Would it mean consequently xmlrpc c++ are not installed on ros:noetic docker image ?
THe weird thing is that  (XMLRPC REQUIRED c++2 client) works fine on a classic Linux Ubuntu  20.04 desktop

Found the issue: libcurl4-gnutls-dev is missing from ros image
apt install libcurl4-gnutls-dev solves the issue


[stable/velero] add resourceGroup to VolumeSnapshotLocation

Signed-off-by: Previtera Roberto<EMAIL_ADDRESS>What this PR does / why we need it:
add resourceGroup to VolumeSnapshotLocation (optional, Azure only)
Which issue this PR fixes
None
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] title of the PR contains starts with chart name e.g. [stable/chart]

/ok-to-test

/lgtm


FieldInfosFormat translation should be independent of VectorSimilartyFunction enum

This commit updates the FieldInfosFormat translation of vector similarity functions to be independent of the VectorSimilartyFunction enum.
The VectorSimilartyFunction enum lives outside of the codec format, and the format should not inadvertently depend upon the declaration order or values in VectorSimilartyFunction. The format should be in charge of the translation of similarity function to format ordinal (and visa versa). In reality, and for now, the translation remains the same as the declaration order, but this may not be the case in the future.
Note: did we introduce a potential index corruption issue when adding maximum inner product in 9.8.0? since the format was not updated when the enum value was added - the ordinal for maximum inner product is unknown to Lucene 9.7.0, which uses the same format.

This PR is a prerequisite for future work to make the similarity function symbolic and lookup-able, see https://github.com/apache/lucene/pull/13076#issuecomment-1930363479.

Hi, as stated in the other issue: I am not really happy to have that enum at all! The similarity/distance functions should be pluggable using NamedSPILoader. To implement that, the ordinals must removed in a new file format version and instead names be written using the Codec utility classes.

Agreed on where we wanna get to. Just trying to get there incrementally, since format changes are quite noisy.

As a first step this PR is fine as it does not change file format and just decouples the ordinals from the enum. In future, when we have SPI, we can use the current code of the ordinals

Exactly, this is just a first step. It (for the most part) encapsulates the translation in the format. When we add a new format and/or evolve VectorSimilarityFunction, this format should be largely immune to the change.

In my opinion, the strings as lookup keys are not needed: Just define it as List<VectorSimilarityFunction> to get the link between them. At a later stage the backwards layer could then fallback to the list with SPI instances to lookup the legacy ordinals. The coec and the enum are still enough decoupled.

Yeah, that's probably good enough for now. Updated.

Index format wise, I think the index corruption can occur when reading a Lucene 9.8.0 index with Lucene 9.7.0, as the format would allow that, but I am not sure this is an expected scenario.

This is perfectly fine.

Ok cool. I was worried for nothing then.

Hi, as stated in the other issue: I am not really happy to have that enum at all! The similarity/distance functions should be pluggable using NamedSPILoader. To implement that, the ordinals must removed in a new file format version and instead names be written using the Codec utility classes.

Agreed on where we wanna get to. Just trying to get there incrementally, since format changes are quite noisy.

As a first step this PR is fine as it does not change file format and just decouples the ordinals from the enum. In future, when we have SPI, we can use the current code of the ordinals

Exactly, this is just a first step. It (for the most part) encapsulates the translation in the format. When we add a new format and/or evolve VectorSimilarityFunction, this format should be largely immune to the change.

In my opinion, the strings as lookup keys are not needed: Just define it as List<VectorSimilarityFunction> to get the link between them. At a later stage the backwards layer could then fallback to the list with SPI instances to lookup the legacy ordinals. The coec and the enum are still enough decoupled.

Yeah, that's probably good enough for now. Updated.

The comment is a bit outdated. I was thinking of making it even more vebose by having 2 maps for the lookup...
+1 looks fine.

I see now that we have a similar dependency in Lucene99HnswVectorsReader. I'll update in a similar way.

Thanks for the reviews. All comments have been addressed.

OK, this is really weird to me. For some reason, we are writing the dimension & similarity into the vector metdata but that information is retained in the field info already.

@benwtrent honestly don't remember, but I do know that early on we tried things a few different ways. There was some discussion about whether the similarity function and dimensions should be in the codec vs in the field info. I suspect it evolved and we did not end up removing the redundant version?

@msokolov thanks for clarifying. I just wanted to make sure there wasn't an important reason that I missed.


audit: Check for invocations of cctools.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Now that ruby-macho is available, cctools should be invoked as little
as possible.

This shouldn't be merged until I revendor ruby-macho at least one more time, since the changes that add lipo(1)-like functionality aren't in the currently vendored version.

This shouldn't be merged until I revendor ruby-macho at least one more time, since the changes that add lipo(1)-like functionality aren't in the currently vendored version.

@woodruffw Any news on this?

Yeah, I implemented the necessary features in ruby-macho a few months ago - I just haven't done a 1.1 release, which when revendored would include the changes.
I'm currently in the mountains with limited internet, but I can do one next week :smile:

@woodruffw Cool, thanks!

Just bumped ruby-macho to 1.1.0, I'll have a revendor PR ready soon and then this'll be good to go :tada:

This should now be good to go. I'll merge it tonight, unless someone else feels like doing it first :smile:

:confetti_ball:

@woodruffw looks like this needs some kind of exception for cctools itself:
iMac-TMP:~ joe$ brew audit --strict --online cctools
cctools:
  * Use ruby-macho instead of calling
Error: 1 problem in 1 formula
iMac-TMP:~ joe$

@woodruffw also the tool name not being printed isn't specific to cctools:
chicken:
  * Use ruby-macho instead of calling
Error: 1 problem in 1 formula
iMac-TMP:~ joe$

Yeah, the check should be something like :
if line =~ /system\s+['"](otool|install_name_tool|lipo)/ && formula.name != "cctools"

to keep to capture all in $1 and exclude cctools. Should I revert-and-fix or just update it in another PR?

No need for revert.

wine calls lipo like this:
        system "lipo", "-create", "#{dirs.first}/openssl",
                                  "#{dirs.last}/openssl",
                       "-output", "#{libexec}/bin/openssl"

What would be the ruby-macho equivalent?

I think you want
          MachO::Tools.merge_machos("#{libexec}/bin/openssl",
                                    "#{dirs.first}/openssl",
                                    "#{dirs.last}/openssl")

Yep, merge_machos is what you want. The docs are here: http://www.rubydoc.info/gems/ruby-macho/MachO/Tools#merge_machos-class_method, although I just realized there's an error: the files are strings like @ilovezfs showed, not MachOFile/FatFile instances like the doc says.

@woodruffw can you confirm that it will work the same way for dylibs and executables?

@fxcoudert Yes.  It doesn't care about what kind of Mach-O they are (dylib, executable, object file, etc).

@woodruffw thanks!

No problem :+1:

@woodruffw Feature request: a lipo drop-in written in Ruby 😉

@MikeMcQuaid That would actually be pretty easy to do, although having Homebrew use it would probably be a pretty ugly hack onto $PATH when the real lipo isn't available. We also benefit a fair bit in terms of performance from not having to exec in the current form :smile:

I'm not sure you two are on the same page. @MikeMcQuaid could you be more explicit about what you're asking for please.

This isn't a feature request for Homebrew but just in general for Ruby MachO having command-line tools that can be used instead of e.g. lipo, install_name_tool, otool but produce identical output could make the tool more widely useful.

Yeah, that's something I can do. I recently broke https://github.com/woodruffw/ruby-macho-utils off from the main repo to experiment with command-line tools (since I wanted to add extra dependencies that ruby-macho itself doesn't require), so I can implement lipo, etc over there.


Update Mean-Teacher and FixMatch Self-Training Scheme(s)

Mean-Teacher training parameters updated
AdaMT training parameters updated

Thanks for all the feedback! The expected changes have been made (in both the self-training approaches).

The forward_context has been updated for generating the Pseudo Labels. Thanks!


Clarification on batched NUFFT

Hi,
I wanted to ask some clarification about the batched NUFFT mode, seemingly supported by finufft/cufinufft and torchkbnufft (cpu/gpu) backends. Specifically, I am focusing on the torchkbnufft backend at the moment.
According to Torchkbnufft documentation, a batch of trajectories is a stack of small k-space trajectories which is processed in parallel, e.g., for dynamic imaging. To enable parallel computation for torchkbnufft, batch dimension should be included in the trajectory tensor, i.e., (n_batchs, n_dim, n_samples) in their notation. However, this does not seems to be used in the mri-nufft interface:
            samples.astype(np.float32, copy=False), normalize="pi"
        )
        self.samples = torch.tensor(samples).to(self.device)

which produce a flattened (nsamples * n_batchs, n_dim) trajectory used as is in both "op" and "adj_op". Before working on a modification of the interface (which I would be glad to!), I wanted to make sure I did not get this completely wrong, and that "batch" axis does not have a different meaning in this context.
Thanks!
Matteo

Hi Matteo,
the "batched" dimension in MRI-NUFFT is only used for the image domain.
In other words, you have a single k-space sampling trajectory per operator. However, all operators can take an array/tensor of shape (B, <1 or C>, XYZ)  (you don't need the coil dimension if you have coil sensitivities map in the operator) and will return you a kspace of shape (B, C, K) (and the other way around for adjoint case)
For doing batched of trajectories applied to a batch of images, you would have to create different NUFFT operators.
On a side note, I would advise to use other backend than torchkbnufft, like gpunufft or cufinuff,  that are faster and more memory efficient (see here for first results )

Many thanks, this is super clear!


automation: split explorer tests into two buckets

Summary
Fixes #https://github.com/rancher/qa-tasks/issues/1423
Currently, the explorer test run for admin takes 27 minutes to complete so we need to split explorer tests into two buckets for efficiency (based on PR review https://github.com/rancher/dashboard/pull/11374#pullrequestreview-2171608065)

Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[ ] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

Explorer test runs after fix:

Tests / e2e-test (admin, @adminUser, @explorer) (pull_request) Successful in 15m

Tests / e2e-test (admin, @adminUser, @explorer2) (pull_request) Successful in 13m


feat: add service api GetSchemaTypeMappingUnderPath

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y #1546

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

1 of 55 (1.82%)  changed or added relevant lines in 3 files are covered.
16 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.08%) to 70.873%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/api/src/service/jsonrpc.rs
0
11
0.0%

kclvm/api/src/service/capi.rs
1
17
5.88%

kclvm/api/src/service/service_impl.rs
0
27
0.0%

Files with Coverage Reduction
New Missed Lines
%

kclvm/tools/src/LSP/src/request.rs
1
66.67%

kclvm/sema/src/core/symbol.rs
3
53.65%

kclvm/tools/src/LSP/src/state.rs
12
82.65%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.08%

Covered Lines:
52896

Relevant Lines:
74635

💛  - Coveralls


FFI for cryptographic primitives

In GitLab by @fahree on Jan 28, 2020, 18:16
To support each blockchain, we need to add FFI to the underlying primitives. On Ethereum, the digest function is keccak256: it is the function used by sha3, with the parameters originally chosen by the designers rather than by the standard body. The signing primitive is secp256k2.
Neither primitive is in OpenSSL, though OpenSSL has some support for secp256k1 that could be leverages somehow

In GitLab by @fahree on Nov 5, 2020, 01:23
assigned to @fahree and unassigned @drewc

In GitLab by @fahree on Nov 5, 2020, 02:38
The Ethereum primitives are in https://github.com/fare/gerbil-crypto already
The Cardano primitives are in https://github.com/vyzo/gerbil/pull/579 waiting for upstream merge.

In GitLab by @fahree on Nov 9, 2020, 04:52
#579 made it upstream to nixpkgs: https://github.com/NixOS/nixpkgs/pull/102936


line graph

graph = pg.graphs.Sensor(5, k=2, seed=10)
line_graph = pg.graphs.LineGraph(graph)
fig, ax = plt.subplots()
fig, ax = graph.plot(vertex_color='blue', edge_color='blue', indices=True, ax=ax)
fig, ax = line_graph.plot(vertex_color='red', edge_color='red', indices=True, ax=ax)
_ = ax.set_title('graph and its line graph')

Todo:

Check for directed graphs.
Any way to set the edge weight?

Should be consistent with NetworkX's line_graph.

Coverage increased (+0.04%) to 88.101% when pulling c24f37bd35f3ce10fd67680b84bf0cc21685d592 on line-graph into 8ce5bde39206129287375af24fdbcd7edddca8c5 on master.

For the edge weight, I think we should think about the properties we want the graph to have. So let us do some math. For now we could raise a warning if the original graph is weighted.

Agreed. Warning added. For now, it creates a binary line graph in all cases (as if the original graph was unweighted).

This looks fine for me...


Implement UI component for PersistentVolumeClaims

Is your feature request related to a problem? Please describe.
Some K8s components are supported to have a custom UI component, and those can be found here. The ccope of this issue is to support PersistantVolumeClaims. The new PVC component should be similar to other components we already have, so feel free to check the existing ones out. You can fetch the data about a PVC by calling the backend on
/api/resources?group=&version=v1&kind=PersistentVolumeClaim&name=<name>&namespace=<namespace>

To be able to test this, you will need to use a template that supports PVCs. You should add the following template and create a Module from it:
repo: https://github.com/cyclops-ui/templates
path: redis
version: test-pvc-ui

Hi @petar-cvit, I'd like to contribute to this project, could you assign this issue to me? Thanks!

Hey @doncicuto, sure thing! Let us know if you have any questions


U/talal/fix math binary pyspark

This PR makes fixes the following:

The pyspark wrapper for math binary did not handle defaults correctly. You can see it in the PR that introduces the pyspark wrapper for mathbinary: https://github.com/combust/mleap/pull/672/ . The PR fixes this issue and brings the pyspark wrapper on par with base spark. Tests have been added to ensure correctness.

Fixes the following serialization direction: python serialize -> scala deserialize. This is scala deserialization reading defaults as 0 instead of None. I've fixed this by changing the python code to use the scala None instead of the python None.
I've inspected manually to make sure that this works but since I am not aware of any python serialization -> scala deserialization tests in mleap, I've not added any unit tests to address this.

@lucagiovagnoli @voganrc for review

@ancasarb it seems my MathBinary is currently broken on scala deserialization on 0.16.0, do you think it's possible to publish 0.16.1 to include this fix ?

hey @lucagiovagnoli sounds good, i'll release 0.16.1 early this week.


Install Zip Archive by default

This extension is required by OctoberCMS, which is build around Laravel. So Laradock does not support OctoberCMS with Zip Archive disabled out-of-the-box.

I think you can change this setting by yourself. Laravel doesn't need to enable zip extension by default.


ENH: Improve CI.

Improve the module's CI:

Add Circle, Travis and AppVeyor CI configuration *.yml files.
Add the CTestConfig.cmake file to submit builds to the Insight
project in the open.cdash.org dashboard.
Add the Python setup setup.py file
Add the Python wrapping files.

Not sure whether this requires any wrapping/whether it makes sense here: I added an empty CMakeLists.txt file. @phcerdan @thewtex please comment on this. Thanks !

Thanks, @jhlegarreta
We can skip wrapping for this module.

Done in c189d20.

File ITKKWStyleOverwrite.txt added in #5.