Fixed typo (aplied)

Docs Build status updates of commit 223aaaa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

en/UIKit/UIImage.xml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Process data

Downloads the zip from url.
Stores the zip on host computer
Loads csv training and csv validation straight from the zip

Add dependencies:

Pandas

Issue #4
I performed the following prior to filing this pull request:

[X] Tested that my change does not break the analysis pipeline
[X] Ran a linter through my code
[X] Update environment dependencies if my code introduces a new package

4. The current file `0process-data/main.py` is the code used to download the data. So anytime someone wanted access to the csv files, they would have to re-download. This module should be run only once by an external user. For example, the user will run some command that calls `main.py` and the data will be placed in appropriate folders (see #5)

about point 4. Why is unzipping needed? The zipped files can be loaded directly into python and be unzipped over there the same way it is done with mnist dataset
It will not re-download. There are checks to prevent that from happening

Thanks for the important PR @wagenrace - I will go ahead and merge so we can continue with the project :+1:


Suggestion: add flag to avoid running the typst command

I would like to use this tool together with the VSCode extension which does the compilation and watching already. I would only need this tool for the webserver with auto-refresh.
Would it be possible to add an option to only monitor for file changes but not run typst in the background?
In this case, one could pass the path to the PDF file instead of the source file.

Implemented it, it should work now using -R <pdfname>

Thanks!


updated vue-fela link in ecosystem docs

In reference to this PR, I have now updated the Ecosystem doc with a link to the latest version of vue-fela.
Special thanks to @dustin-H and @rofrischmann for making this happen 😄

Thanks for merging @rofrischmann—really stoked about this 😄
Could you kindly let me know when you've updated the docs so I can let my team know?

Of course I will! Just updating some parts of the docs at the moment.
Btw. would you want to write a Usage with Vue documentation to be added here: http://fela.js.org/docs/UsageGuides.html
Apart from that, what do you think about moving vue-fela to the official Fela multirepo to benefit from dependency updates / integrated tests?


Logging

Really needs some proper logging. My test environment isn't working and I'm not sure why. Do you have a preference on library?

In my opinion env_logger is the best choice at the moment. Mature enough and issue to use by virtue of not suffering from the pedantic design choices of slog/slogger.
I will probably add it and use it to replace some of the current logging on Wednesday, if you needed it before hand feel free to add it.
If you think another logger would work better, feel free to suggest one.

I've only used slog. It's neat and powerful, but definitely a little complicated. Also the logger is an object you have to pass around, which isn't the nicest thing for our use case. I'd be interested in seeing how env_logger works across crate boundaries, but otherwise it works for me.

Hmh, env_logger is, well, env based, so the same settings should apply no matter where you call it from (though I think there's ways to specify what to log and what to not log in the crate).
Overall I find it more convenient to use, and definitely easier to replace than something like slog, so for the early stages of software I think it's the best solution.
I will try to get it into the project to add some debug/error/info logs that I'd want to have there anyway and I think you will find it does the job quite alright.

Agent logging isn't done yet

Hmh, I closed the issue because I intorduced env_loger as the logger and assumed that this issue was more about "introducing a logger" rather than "Getting some proper logs", since the second will always be a moving goal. But feel free to close it whenever you are satisifed with the situation logging-wise or open a new open with more clear goals as to what should be logged and at what levels.


How to get PNS Handle?

This article mentions that a PNS Handle is needed. I would like a link on how to get the PNS Handle for Android in .NET MAUI.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 1fe12c8b-5f7b-be6f-9db0-1009a27ceef8
Version Independent ID: 5bb28d30-3240-7354-9f58-e9e51da7f3ae
Content: Registration Management
Content Source: articles/notification-hubs/notification-hubs-push-notification-registration-management.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@blanthor Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

@blanthor Very sorry for the slow response here. If this is still an issue and you need assistance, please contact Microsoft Support or open a support ticket from the Azure portal. #please-close


Desktop mode not working

After the update my PC is stuck in a mobile browser version of the page.
Appending /Material/?layout=desktop does not help.
Distressingly the settings button dropped the item for server and I was unable to return to the default skin.
My iPad using Chrome did not have that issue and was able to swap to default.
I have changed back and forwarth since then but still only a mobile version.
Setup
LMS on a Ubuntu Server VM - ProxMox host
Windows 11 + Chrome
Many thanks for an excellent product.

Sorry but this is an issue on your system, I use desktop layout on my Laptop and can confirm it still works.
Perhaps you need to refresh your cache?

i also have the same problem, lms opened in dark theme, there is no Server item in the settings menu, and this js error
material.min.js?r=5.0.0:75 ReferenceError: defaultTheme is not defined
    at s.initUiSettings (material.min.js?r=5.0.0:741:268)
    at material.min.js?r=5.0.0:87:5905
    at material.min.js?r=5.0.0:87:7403
    at Array.forEach (<anonymous>)
    at material.min.js?r=5.0.0:87:7382
    at s._withCommit (material.min.js?r=5.0.0:87:9209)
    at s.commit (material.min.js?r=5.0.0:87:7356)
    at s.commit (material.min.js?r=5.0.0:87:3371)
    at kr.created (material.min.js?r=5.0.0:756:339)
    at pn (material.min.js?r=5.0.0:75:22030)

Perhaps you need to refresh your cache?

This worked for me perfectly. Give it a go.


Adding avoid_multicast to PDP and WLP [6467]

This will improve the communications between a participant without the avoid_multicast flag set and another participant with it.

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows

Run the failing test locally 100 iterations and didn't fail. Merging.


The same sentence appears in loop

Hi,  Foldl.
I am learning about your program.
After testing, I found that it will start to appear the same sentence after a few answers and keep looping. But this problem does not occur on llama.cpp.
I can provide the testing ENV and how can I contact you.
Thanks you.

Which model?

I've tested qwen, gemma, and llama, all models have this issue, but it doesn't happen every time.

chatllm.cpp does not have repetition penalty yet. You can try greedy or tfs sampling by --temp 0 or --sampling tfs.


Add Remainder of Timeless Isles Armor Tokens

What is the reason for this PR? Were some of the items not working as intended?
There should be no need to add them 1 by 1, as the spell-tracking used here checks the item spell name, e.g. "Create Chestpiece", which is identical for all the items.

Oh my bad, hadn't realised. I know it wasn't working for the Timeless Curio, not sure about the Timeless Lavalliere.

I was indeed missing some items: Cloak, Signet, Lavalliere and Curio.
Thank you for your contribution.


Extruding a path containing an ellipse along  path also containing an ellipse doesn't work.

The following piece of code is supposed to extrude a shape along a curved path. The straight part of the shape is extruded, but the curved part is not.
var project = Project.CreateSimpleProject();
CADability.GeoObject.Path path = Path.Construct();
path.Add(Line.MakeLine(new GeoPoint(0, 0), new GeoPoint(0, 100)));
var ellipse = Ellipse.Construct();
ellipse.SetArc3Points(new GeoPoint(0, 100), new GeoPoint(50, 50), new GeoPoint(0, 0),Plane.XYPlane);
path.Add(ellipse);
var extrudePath = Path.Construct();
var extEllipse = Ellipse.Construct();
extEllipse.SetArc3Points(new GeoPoint(0,0), new GeoPoint(0, 0, 100),new GeoPoint(0,60,50),Plane.YZPlane);
extrudePath.Add(extEllipse);
var pipe = Make3D.MakePipe(path, extrudePath, project);
project.GetActiveModel().Add(pipe);
project.ExportToSTL("c:/temp/e.stl");

Currently extruding ellipses along another ellipse is only implemented for the perpendicular case, i.e. when the extrusion starts  perpendicular to the plane of the ellipse to be extruded (and results in a torus). If this is not the case, the extruded surface might be self intersection or have a strange shape. It could be implemented as a SurfaceOfRevolution , but I think you will rarely find such surfaces in real models.

Thank you for the answer. You might consider throwing a notimplementedexception in this case.


[Issue #50] create_config selects runtime based on local python version.

Addressing https://github.com/ericmjonas/pywren/issues/50.
Testing for this PR (and the cli script in general) should be added as separate PR in the future.
@ericmjonas @shivaram

Closing this now that we have #52 .


Lack confirmation message when a developer opened/imported a TSML file into αW

A developer has to go to advanced settings to see if the new file is recognised and in use - if the new file has the same name as one already in use there will be no clue if the import/open is successful.
currently:

open a tsml file (e.g. received from Telegram) with AlphaWallet
AlphaWallet launches innocently as αWallet but secretly takes the file in.

expected:

open a tsml file (e.g. received from Telegram) with AlphaWallet
AlphaWallet launches with a message like "xxx file imported with no error" or give some error.

@AW-STJ is following the process.

A developer has to go to advanced settings to see if the new file is recognised and in use

@James-Sangalli how could i determine what file currentry in use? i need to check if AssetDefinitionStoreCoordinator.overrides contains this file?

A developer has to go to advanced settings to see if the new file is recognised and in use ...

@vladyslav-iosdev probably look at how the console in settings displays files that override. Maybe ConsoleViewController.

@vladyslav-iosdev probably look at how the console in settings displays files that override. Maybe ConsoleViewController.

@hboon am i understand right, TSML its a token script file, that i can share with aw app and can get one from https://github.com/AlphaWallet/TokenScript-Examples, right?

@vladyslav-iosdev probably look at how the console in settings displays files that override. Maybe ConsoleViewController. ...

@vladyslav-iosdev yes. It’s these files that define what the action buttons do (the one that are alongside the send/receive buttons).


changes made to signup

On adding new dependencies use --save npm install --save xyz
The warning is removed and no need to add the component in the App.js after adding its route and component in route.js this was making the form appear twice.
Also, try styling the form in center alignment in a box kinda structure. Nice work with validation!


Handle charset-conditional setup of ContentMetaAttributeValue following Pydantic conversion

Part of #3 in bisque.element.attributes

Bug diagnosed: the encode method is not called, whereas in campbells it is
breakpointing ContentMetaAttributeValue.encode reaches the following stack:
  /home/louis/dev/campbells/src/campbells/tests/pageelement_test.py(227)test_encoding_substitution_correct()
-> utf_8 = soup.encode("utf-8")
  /home/louis/dev/campbells/src/campbells/element/tag_core.py(1636)encode()
-> u = self.decode(indent_level, encoding, formatter)
  /home/louis/dev/campbells/src/campbells/main.py(841)decode()
-> return prefix + super().decode(
  /home/louis/dev/campbells/src/campbells/element/tag_core.py(1666)decode()
-> piece = element._format_tag(eventual_encoding, formatter, opening=True)
  /home/louis/dev/campbells/src/campbells/element/tag_core.py(1834)_format_tag()
-> val = val.encode(eventual_encoding)
> /home/louis/dev/campbells/src/campbells/element/attributes.py(87)encode()
-> if encoding in PYTHON_SPECIFIC_ENCODINGS:

this suggests to fix it I need to diagnose where the stack goes instead when soup.encode("utf-8") is called in the test


Customisation

Modified package names & pointed travis badge to matthewdavidson/create-react-app

I assume you were intending to make these changes to your own fork :P ?

You assume correctly! I was attempting to do something along the lines of this:
https://github.com/facebookincubator/create-react-app/issues/682
Hopefully I’ll be able to send a real PR in the future!
From: Facebook Community Bot<EMAIL_ADDRESS>Reply-To: facebookincubator/create-react-app<EMAIL_ADDRESS>Date: Wednesday, 9 November 2016 23:10
To: facebookincubator/create-react-app<EMAIL_ADDRESS>Cc: Matthew Davidson<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [facebookincubator/create-react-app] Customisation (#1027)
I assume you were intending to make these changes to your own fork :P ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/facebookincubator/create-react-app/pull/1027#issuecomment-259553649, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABX5WrAwqQKW9S9IDpryojvlzXVWyFogks5q8lL-gaJpZM4Kt82M.


Published to NPM

Installalation step says npm install @cungminh2710/graphql-depth-limit, but seems it's not been published yet?

Thanks @ecker00, it has been published just now. let me know by raising another issue if you have any problem 👍


Add some things to docs in order to avoid confusion

Hey Gajus!
Thank you for this module! Looks like you have really explored the bowels of webpack and have put together something very useful for setting up isomorphic apps.
I finally managed to get this to work and would like to let you know about some of the issues I found while attempting to use this.

I had been using relative paths to context which is quite valid for webpack-dev-middleware to serve.
You must use arrays to wrap your entry point paths
Again this is not being as flexible as webpack if this is a requirement it would be great to have some kind of reminder or documentation
You can't use a fancy libraryTarget setting
Entry point output.libraryTarget must be set to the default (which I think is "var") or at least certainly not "umd" like I had it set to from the config I was branching off.
No example for a production build
I know this is only meant to be to enable a dev server but in reading the readme it takes a while to actually grok the usecase.
It would make it quicker to understand and implement if there was a build script included in the demo. I understand that this would have nothing to do with the library and that you want to present a minimal example to focus on how to use the lib but honestly a bit of context would really help out someone with new eyes to the space. Perhaps you could provide a couple of examples?
Personally I knew didn't want to use this in production to serve my routes as it is poor practice however this gets integrated a little deeply when you try to create a real project and setting it up is kind of involved and without an example of a build step I couldn't tell 100% if you were actually using this in production. Anyway a build script would help remove the confusion.

Now I was not sure how you wanted to present these things in docs but wanted to document somewhere for other users that are trying to get started with this which is why I opened this issue.
Again thank you very much and great work!
Rudi

I just want to add that the README should highlight that context is required in webpack.config. I was missing it and isomorphic-webpack was complaining:
TypeError: Path must be a string. Received undefined
    at assertPath (path.js:28:11)
    at Object.resolve (path.js:1167:7)
    at exports.default (./node_modules/isomorphic-webpack/dist/factories/createCompilerConfiguration.js:28:37)
    at exports.default (./node_modules/isomorphic-webpack/dist/factories/createIsomorphicWebpack.js:72:73)
    ...

I am starting to use isomorphic-webpack more and more. Will eventually come back to update the docs.


Feature/274 - Transactions List should be able to clear all transactions

Might need a new design in place. For now I used the generic Airswap style button
https://github.com/airswap/airswap-web/assets/90465420/a9dff5a1-7038-4044-9a56-e22dd82815c1

Video demo shows clearing localStorage + clearing the UI
https://github.com/airswap/airswap-web/assets/90465420/1df0cece-8581-48a6-8fd4-eb1a663c8af0

This issue should be closed and not merged. The same code (and more) can be seen in PR #363 . 363 will contain this code, and also code to clear all failed transactions. It'll also have the design that 2x posted into issue #799


Seeding database for testing

Hi,
at first i want to thank you for that awesome package, i use it in an social network for user 2 user communication.
I want to do some penetration testing so i need to seed my threads and messages table.
It's really hard for me to figure out how to do that with threads and messages. I already seeded the user table with 50 random users. now i want to create threats and messages from every of that random user to one specific user wich i had defined seperate in the user table seeder.
had anyone did this before and can tell me how it works?
I'm really new to laravel
thanks in advance

Hi @Evoolo, thanks for using the package! Please take a look at my testing repos for a few examples of how you can seed thread data. Just pick the repo that corresponds with the version of Laravel you're using. If you have any further questions, or need additional help, just let me know.

Oh i should open my eyes. You are Awesome. Thanks, that schould do the trick.


fix: Conda venv is not used

Fixes https://github.com/sublimelsp/LSP-pyright/issues/348

As I noted on issue #348, these changes seem to fix the conda environment issues I have on macOS.

As I noted on issue #348, these changes seem to fix the conda environment issues I have on macOS.

Thanks.


Adding Whack-a-mole game

Closes #816
Adding Whack-a-mole game.

Please Create new PRs for this because it contains conflict issue.


[SPARK-46767][PYTHON][DOCS] Refine docstring of abs/acos/acosh

What changes were proposed in this pull request?
This pr refine docstring of  abs/acos/acosh and add some new examples.
Why are the changes needed?
To improve PySpark documentation
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass Github Actions
Was this patch authored or co-authored using generative AI tooling?
No

Merged into master. Thanks @zhengruifeng


TAJO-2071: Supporting DATE type in Parquet format

Tested to read in Hive with Parquet file created from Tajo

+1 LGTM!


Rocks'n'Diamonds

see about getting a newer version working.
the newest version will not work with the old version of SDL2
fix from dev
a workaround would be to simply remove that line "SDL_OpenURL(url);" from the R'n'D code, which would mean that the game does nothing when it should open a URL, but this is currently not used by the standard R'n'D program anyway (but only by very few third-party level packages and custom game editions).
he might look into making a check so it will know how to install for older systems

testing latest version

works great


Unable to decode CODE39

Hi, zxingcpp.read_barcodes() has been able to decode multiple barcodes in a figure. However, the code didn't work well for this figure. Kindly assist me with this. Thank you.
code (with python wrapper v1.3.0):
import zxingcpp
import numpy as np
import cv2
img_gray = cv2.cvtColor(img,cv2.COLOR_BGR2GRAY)
[results.text for results in zxingcpp.read_barcodes(img_gray)]
Expected result: 2 barcodes decoded
Actual result: ['M8195-69502']

Your code is correct. The problem is the detection of the lower barcode. And that is caused by a limitation of the GlobalHistogramBinarizer that fails to properly distinguish between black and white here due to the greyish background that interferes. It is basically the same issue as in #82. If you crop away the metallic background it works. I hope to sometime be able to overcome this limitation.

Your code is correct. The problem is the detection of the lower barcode. And that is caused by a limitation of the GlobalHistogramBinarizer that fails to properly distinguish between black and white here due to the greyish background that interferes. It is basically the same issue as in #82. If you crop away the metallic background it works. I hope to sometime be able to overcome this limitation.

Thank you for the help. Your suggestion worked.


sdist tests not runnable

The source distribution isnt testable as-is.
Ideally one of the following should work.
python setup.py test
python `which django-admin` test

I see this project uses tox and runtests.py.
The files to support that are not included in the sdist.

I'll happily review pull requests for this.

I'll be happy to help with the packaging, but currently that isnt helpful as https://github.com/antonagestam/collectfast/issues/132 means all is broken for me, so fixing packaging isnt useful.

Agreed. I probably won't find the time to fix it for at least a few weeks though.

This should now be more feasible to implement as tests that require a working backend are marked and skipped when SKIP_LIVE_TESTS=true is set.
However, I noticed python setup.py test is now deprecated (noticed your comments in the discussion too). Do you have any opinions on on the issue or guidance on how and if we should strive to achieve a testable source distribution?

Closing this for now, feel to re-open if there's interest for this.

Distros still test from source distributions, using setup.py test by default.  Nothing has changed there - deprecation notices had zero effect.  When they actually break it, there are several setuptools plugins which provide the test command, and distros will start using one of those.


Backup won't run…

Describe The Bug
Unfortunately, the Homebridge backup is not running. The path I specified exists but no backups are created there. It is an external drive that is correctly mounted as other services backup there too.

Logs
[16/03/2024, 00:31:49] [Homebridge UI] Could not run scheduled backup: Custom instance backup path is not writable / readable by service: EACCES: permission denied, access '/media/pi/Pi-Data/Backups/Homebridge'
Config
No response
Homebridge UI Version
latest
Homebridge Version
latest
Node.js Version
latest
Operating System
Raspberry Pi OS / Raspbian
Environment Info
None
Raspberry Pi Model
Raspberry Pi 3 B+

The error / problem is here - [16/03/2024, 00:31:49] [Homebridge UI] Could not run scheduled backup: Custom instance backup path is not writable / readable by service: EACCES: permission denied, access '/media/pi/Pi-Data/Backups/Homebridge'
You need to change ownership of the directory to the user account that is running the homebridge process.   Current ownership is root based on your directory listing.

you're right


20w49a

I ported the mod to 20w49a
Great mod btw, very helpful :+1:

Well, I expanded the semver range myself, this PR is no longer relevant.


add json output --format option

Adds a new flag --format, which takes either 'json' or
'term' (default). if 'json' returns a json structure of the results
normally printed to the terminal
resolves #142, resolves #119

@zmc I made some changes to pass around the terminal object to make it easier to pass around some state, and the runner object as well, which required some changes in the tests to account for the new structure. Also added the results to the metadata in order to output as json. We could dump the whole json structure, but i was going for terminal output compatibility/consistency


Fix video/video.c:1458:52: error: implicit conversion from enumeration type 'enum v4l2_buf_type' to different enumeration type 'imgsensor_stream_type_t' (aka 'enum imgsensor_stream_type_e') [-Werror,-Wenum-conversion]

Summary
ret = g_video_sensor_ops->validate_frame_setting(type, nr_fmt, sf, &si);
       ~~~~~~~~~~~~~~~~~~                         ^~~~

Impact
video validation
Testing
Pass CI

@jerpelea could you review the change?

https://github.com/apache/incubator-nuttx/pull/7391


Flow tests

Introduce Tests for
o flow/table_test.go
o pcap2sflow-replay (functional testing)

CI tests are failing since 3 patches it seems

@safchain : Travis seems still failed with "Connection timeout reached" on topology_test
I tested locally on master and the flow-tests branch, and have the same result every time.
Previous failed was due to a slow fork/exec on Travis platform, as he can't inject the packets within 3 secs .... I change it to 10 second, and now it's pass.

@safchain : After a couplesss of test, the only way to fix it is to stop the global agent, as the instance is keep between tests (topology_test and agent_sflow_test)
So we will merge this PR after #45


Change persist strategy

Immediate persist was causing more problems than it solved. Backing that out in favor of just running the poller more frequently

🚢


plane.fromPoints() implementation buggy

Expected Behavior
For new testcase
const obs5 = fromPoints(obs1, [-1, -1, -1], [1, -1, -1], [-1, 1, -1], [1, 1, -1])

all pointes lie on plane z==-1, and the normalized normal vector returned should be either [0,0,-1] or [0,0,1].
Actual Behavior
Instead [0,0,0] is returned which cannot be a normal vector. Reason is that two times normal vector computed in loop is [0,0,1], and two times it is [0,0,-1], and summation cancels them 0 as [0,0,0]. Summaion is just wrong.
Steps to Reproduce the Problem

add this test to OpenJSCAD.org/packages/modeling/src/maths/plane/fromPoints.test.js:

pi@raspberrypi5:~/old/OpenJSCAD.org/packages/modeling/src/maths/plane $ git diff fromPoints.test.js
diff --git a/packages/modeling/src/maths/plane/fromPoints.test.js b/packages/modeling/src/maths/plane/fromPoints.test.js
index 7e924d3b..b4c9297a 100644
--- a/packages/modeling/src/maths/plane/fromPoints.test.js
+++ b/packages/modeling/src/maths/plane/fromPoints.test.js
@@ -17,4 +17,7 @@ test('plane: fromPoints() should return a new plane with correct values', (t) =>
// polygon with co-linear vertices
const obs4 = fromPoints(obs1, [0, 0, 0], [1, 0, 0], [2, 0, 0], [0, 1, 0])
t.true(compareVectors(obs4, [0, 0, 1, 0]))
+
+  const obs5 = fromPoints(obs1, [-1, -1, -1], [1, -1, -1], [-1, 1, -1], [1, 1, -1])
+  t.true(compareVectors(obs5, [0, 0, 1, -1]))
})
pi@raspberrypi5:~/old/OpenJSCAD.org/packages/modeling/src/maths/plane $

run

pi@raspberrypi5:~/old/OpenJSCAD.org/packages/modeling/src/maths/plane $ ../../../../../node_modules/.bin/ava fromPoints.test.js

  plane: fromPoints() should return a new plane with correct values

  src/maths/plane/fromPoints.test.js:22

   21:   const obs5 = fromPoints(obs1, [-1, -1, -1], [1, -1, -1], [-1, 1, -1]…
   22:   t.true(compareVectors(obs5, [0, 0, 1, -1]))
   23: })

  Value is not `true`:

  false

  › fromPoints.test.js:22:9

  ─

  1 test failed
pi@raspberrypi5:~/old/OpenJSCAD.org/packages/modeling/src/maths/plane $

use "t.is(obs5, [0, 0, 1, -1])" instead to see that indeed invalid [0,0,0] is returned as normal

Difference:

  [
    0,
    0,
-   0,
-   -0,
+   1,
+   -1,
  ]

Specifications

Version:
Platform: Debian based 64bit Raspberry PiOS, same error on Intel Ubuntu browser
Environment: browser, local ava test

Fix is available on my fork:
https://github.com/jscad/OpenJSCAD.org/compare/master...Hermann-SW:OpenJSCAD.org:master
Only problem is that this single testcase fails after fix, but that is likely because because of incorrect plane.fromPoints() a point too much was returned, and now is not. So likely 863 is correct:
pi@raspberrypi5:~/OpenJSCAD.org/packages/modeling/src/operations/expansions $ ../../../../../node_modules/.bin/ava expand.test.js

  expand: expanding of a geom3 produces expected changes to polygons

  src/operations/expansions/expand.test.js:133

   132:   t.notThrows.skip(() => geom3.validate(obs2))
   133:   t.is(pts2.length, 864)
   134: })

  Difference:

  - 863
  + 864

  › expand.test.js:133:5

  ─

  1 test failed
pi@raspberrypi5:~/OpenJSCAD.org/packages/modeling/src/operations/expansions $

It is not only that single point difference, I did outpus pts2 in both cases, and there are much more (minimal) differences:
$ diff out  ~/OpenJSCAD.org/packages/modeling/src/operations/expansions/out  | head
20c20
<   -       -1.7870337216829664,
---
>   -       -1.7870337216829657,
25c25
<   -       -1.7870337216829664,
---
>   -       -1.7870337216829657,
30c30
<   -       -5.322567627615704,
$ diff out  ~/OpenJSCAD.org/packages/modeling/src/operations/expansions/out | wc --lines
23983
pi@raspberrypi5:~/old/OpenJSCAD.org/packages/modeling/src/operations/expansions $

Yeah. This is the issue that I mentioned about the order of the points. The points have to create a valid polygon, i.e. non-crossing edges.

Yeah. This is the issue that I mentioned about the ordering of the points. The points have to create a valid polygon, i.e. non-crossing edges.

Interesting, and you are right, my example above in this issue was with intersection of 2 of the 4 edges:
fromPoints(obs1, [-1, -1, -1], [1, -1, -1], [-1, 1, -1], [1, 1, -1])
*--*
 \/
 /\
*--*

I did rewrite for no intersection ...
console.log(plane.fromPoints([], [-1, -1, -1], [1, -1, -1], [1, 1, -1], [-1, 1, -1]))
*--*
|  |
|  |
*--*

... and indeed non-null normal gets returned in first three components:
[
    0,
    0,
    1,
    -1
]

With your new function for the general case a wontfix for this function would be OK.
But only with updating doc stating that no "intersections" in the plane are allowed:
https://jscad.app/docs/module-modeling_maths_plane.html#.fromPoints

@Hermann-SW thanks again for the feedback, and great discussion.


Add parse for TRUNCATE

Hi, I've implemented parsing for TRUNCATE [TABLE] MySQL query in accordance with official MySQL documentation:
http://dev.mysql.com/doc/refman/5.7/en/truncate-table.html

thanks!


The matched value type 'T?' can never be equal to this constant of type 'Null'.

Hi!
I write a small dart program and put it in a package directory (lib/sandbox.dart):
void someMethod<T extends Object>(T? value) {
  switch (value) {
    case null: // gives a warning: constant_pattern_never_matches_value_type
      print('matched');
  }
}

void main() {
  someMethod<int>(null); // prints "matched"
}

When I run dart analyze on this package I get the following:
% dart analyze
Analyzing sandbox...                   0.5s

warning • lib/sandbox.dart:3:10 • The matched value type 'T?' can never be equal to this constant of type 'Null'. Try a constant of the same type as the matched value type. • constant_pattern_never_matches_value_type

1 issue found.

So it warns me that the null constant will never be matched. Running the program prints "matched" so it does get matched anyway. Removing the extends Object constraint makes the warning message go away.
pubspec.yaml is minimal:
name: sandbox
version: 1.0.0
publish_to: none
environment:
  sdk: ^3.0.6

% dart --version
Dart SDK version: 3.0.6 (stable) (Tue Jul 11 18:49:07 2023 +0000) on "macos_arm64"

Cheers,
Marton

This seems to have been fixed: DartPad, beta channel and master channel do not emit this warning.

@eernstg
Thanks Erik! Also for pointing me to DartPad, never really tried it before. Looks useful.


OpenXR initialization errors

I'm trying to switch from WMR plugin to Microsoft OpenXR. Tried this version and UE Marketplace version in both 4.26.2 and 4.27.1. Always get the same warnings and, of course, features don't work. In particular, FHolographicRemotingPlugin isn't loaded and I can't use Remoting Connect.
Do I miss something in setup? I disabled all other VR and AR plugins in UE4 except Hololens, bult-in OpenXR's and Microsoft OpenXR.
LogHMD: Optional extension XR_KHR_opengl_enable is not available
LogHMD: Optional extension XR_KHR_vulkan_enable is not available
LogHMD: Optional extension XR_VARJO_quad_views is not available
LogHMD: Warning: Required extension XR_MSFT_holographic_remoting is not available
LogHMD: Could not enable all required OpenXR extensions for OpenXRExtensionPlugin on current system. This plugin will be loaded but ignored, but will be enabled on a target platform that supports the required extension.
LogHMD: Warning: Required extension XR_MSFT_scene_understanding is not available
LogHMD: Could not enable all required OpenXR extensions for OpenXRExtensionPlugin on current system. This plugin will be loaded but ignored, but will be enabled on a target platform that supports the required extension.
LogHMD: Warning: Required extension XR_MSFT_scene_understanding is not available
LogHMD: Could not enable all required OpenXR extensions for OpenXRExtensionPlugin on current system. This plugin will be loaded but ignored, but will be enabled on a target platform that supports the required extension.
LogHMD: Optional extension XR_MSFT_holographic_remoting_speech is not available
LogHMD: Warning: Required extension XR_MSFT_secondary_view_configuration is not available
LogHMD: Could not enable all required OpenXR extensions for OpenXRExtensionPlugin on current system. This plugin will be loaded but ignored, but will be enabled on a target platform that supports the required extension.
LogHMD: Failed to get an OpenXR system, result is XR_ERROR_FORM_FACTOR_UNAVAILABLE. Please check that your runtime supports VR headsets.

Did you enable remoting in Project Settings > Windows Mixed Reality and restart?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/941 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit f8e559c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-07-07-02-00-44 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add RwStreamSink

For secio, and probably for other things in the future, it is useful to have a way to turn a Stream + Sink that produces/takes bytes into a AsyncRead + AsyncWrite.

Closing and reopening because of github issues :-/


multiple layouts on devices after commit  "[7ce781e87cf7cf789a54d37af7d78f1c11d66dbc] keybinds: better follow xkb translation state" causes Hyprland to crash on startup

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit b2c34404777ea0850f90f4c38f64484f03599d14 dirty (animations: Refactor AnimatedVariable (4911)).
Date: Sat Mar 2 02:35:17 2024
Tag: v0.36.0-24-gb2c3440

flags: (if any)

System Information:
System name: Linux
Node name: ELARCH-F15
Release: 6.7.4-arch1-1.1-g14
Version: #1 SMP PREEMPT_DYNAMIC Wed, 07 Feb 2024 13:01:49 +0000

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation Alder Lake-P GT1 [UHD Graphics] [8086:46a3] (rev 0c) (prog-if 00 [VGA controller])
0000:01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA107M [GeForce RTX 3050 Ti Mobile] [10de:25a0] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
Hyprland crashes on startup when devices have multiple layouts configured,
I have git bisected and first bad commit is 7ce781e87cf7cf789a54d37af7d78f1c11d66dbc which has to do with xkb translation state, so something must have happened there
How to reproduce
Have multiple layouts on devices like:
device {
name = at-translated-set-2-keyboard
kb_layout = us,gr # here having both the layouts causes the crash while having just  1 works
kb_variant = qwerty
kb_options = grp:alt_shift_toggle,caps:escape
numlock_by_default=true
}
Crash reports, logs, images, videos
hyprlandCrashReport7786.txt

segmentation fault (core dumped)


Enable glow for compatibility.

Removing borders.

Godot Engine v4.0.2.stable.official.7a0977ce2 - https://godotengine.org
OpenGL API 4.1 ATI-4.8.101 - Compatibility - Using Device: ATI Technologies Inc. - AMD Radeon R9 M370X OpenGL Engine

Forward on old GPU

Compatibility on old GPU

Enable glow

@clemens-tolboom just wondering, what does glow actually do?


Wind + Dust

I don't know if this is a bug or a feature, but dust explodes when it comes into contact with heavy wind.

working as intended! many explosive are pressure-volatile


Lock the Side Panel so that it doesnt let the user just screw up the view entierly

Work In Progress.   Will find someone else to help.

Actually, it's fixed after the dashboard redesign.

Woohoo.  Closing this


struct/union/enum

[ ] struct
[ ] union
[x] enum

Error is different since https://github.com/jyn514/rcc/commit/ae2c94e904c86286269a6d2747d2da641c105c54:
struct s;
thread 'main' panicked at 'not yet implemented: structs, unions, and enums', src/parse/decl.rs:371:9

Enums are done since https://github.com/jyn514/rcc/commit/ddc165ccd2f792be5cdeae28b5969d6b131f4f1a!

Struct and union declarations are done. Need to implement expressions and codegen.
struct s { int i; } s;
int main() {
        return s.i;
}
thread 'main' panicked at 'not yet implemented: struct and union members are very much not implemented', src/parse/expr.rs:706:33

Possible ways to implement this:
Have a cache in ir::LLVMCompiler that stores offsets into structs. For unions, the offset is always zero. The size of a struct is the greatest offset + sizeof(member at that offset). Accessing a struct member is fetching *(s + offset).
Ways to calculate the offset: For now, just do sizeof(member) + (8-sizeof(member))%8.
In the future, sort members from largest size to smallest, excluding arrays. Put chars, shorts in spare bytes.

Done in https://github.com/jyn514/rcc/commit/01f53f224c43a9ba9a66be3d90935dd8cab1f1ba!!!


Shared dependencies installed to conda base

Should instead be installed to each target environment. The issue is that the following command is not working correctly, at least on Windows:
conda run -n [env_name] pip install [dependency]

Patched installer. pip must be installed in conda environment for conda run -n to work correctly. Otherwise, the system pip is used and packages install to the system.
Added ability to install whatever is possible with conda to hard link and conserve space. Made pip inside environment install method into a fallback.
Added transformers (Generative NLP AI), seaborn and a few more shared dependencies.


Add user defined operators

Closes #4152

TODO:

[ ] Add more robust error test cases (e.g., op cycles)
[ ] Add test for call user defined ops within user defined ops
[ ] Add support for calling user-defined ops with no arguments.


Typo fix in How to Use This

Corrected typo in How to Use This section

Congratulation on your first PR merged! Thanks


docs/devel: add 'deploy to minikube' docs

What this PR does / why we need it:
I felt that the steps to deploy/run/test cert-manager in minikube were non-trivial enough that it was worth documenting.
Let me know if this should live somewhere else in the repo or if you have suggestions for how to better document this.
NONE

I also tossed on a typo fix I had laying around

Great, thanks for this 😄
In future it'd be awesome to expand this out with info on how to run e2e tests. There are a couple of nuances in how DNS needs to be configured to get it right.
/lgtm
/approve

/ok-to-test


Is shuttle tied to Rocket, or are other servers possible?

The initial paragarph doesn't mention Rocket, but all the examples use it, and it is not clear if we can use another server architecture.
Even if it is not yet supported, but is intended to be supported, it would be good to hear about it.

Indeed! We chose Rocket as a first supported target because we were familiar with it. The key is the IntoService trait, not the web framework specifically.
We're already working on implementing support for axum #83!

Axum is done! And Tide is in the roadmap. Closing this issue for now!


Libtomcrypt

Update #8787 to support mac and linux.  Address comments there

What does your PR fix? Fixes issue #10947, #10948

Which triplets are supported/not supported? Have you updated the CI baseline?

Does your PR follow the maintainer guide?

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

@crmoore Thanks for your PR. Please sign the cla.

Thanks for the PR!


Remove CosmosPlugin env variables on Fleek and CloudFlare

Just a reminder so we make sure that it will be done when @0xdef1cafe comes back!
If anyone else has the right to do it, don't hesitate!

done!


Newest files first?

Is it possible to have the file list sorted by newest first?

This would be so useful to me.
It would be awesome to have the option to change that in settings


Cut down PPP length

Codecov Report

Merging #316 (0b7ccca) into master (55c7ba5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #316   +/-   ##
=========================================
  Coverage     78.07%   78.07%
  Complexity      993      993
=========================================
  Files           119      119
  Lines          3033     3033
  Branches        417      417
=========================================
  Hits           2368     2368
  Misses          540      540
  Partials        125      125

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 55c7ba5...0b7ccca. Read the comment docs.


64-Bit binary

I guess we should provide an x86 (current) + an 64-bit binary.

Hi this makes sense and your other issue. However I don't have Windows anymore, so if you have any drive to make changes please be my guest!

+1 for x86 + x64 in https://github.com/DominicTobias/SecurePad/releases


Add commit hash

Extend jest + webpack config of craco

Add envHelper incl. tests

Footer shows a "branch" icon linked to latest commit at GitHub

Close #57

LGTM!


fixed the gst l4 notebook

Fixed a bug created by an Alphalens change. Related to #2787

LGTM.
Is the semicolon at EOL intentional?

I believe the semicolon hides a print of the plot name. I'm probably explaining that poorly but it cleans up the output a bit!

I believe the semicolon hides a print of the plot name. I'm probably explaining that incorrectly but it cleans up the output a bit!

Ok, cool. I'm vaguely aware of that.


Improve linting and line endings config

Adds lint-staged/husky so that eslint and prettier run before commit
Adds .editorconfig  for line endings and indent style (we seem to be using 2 so I made it explicit)
Adds .gitattributes to set the line endings mode on the git repository
Adds more things to .gitignore for other editors, log files and result output

This adds on top of the existing open PR: https://github.com/sitespeedio/plugin-lighthouse/pull/16
Resolves https://github.com/sitespeedio/plugin-lighthouse/issues/15

Ah cool thanks @gidztech !

@gidztech this means we can close https://github.com/sitespeedio/plugin-lighthouse/pull/16? Or something else needed? Sorry been focusing on other stuff.


Mutually Exclusive Options?

Would there be an easy way for either me or you and myself to implement mutually exclusive options so that either a default_value: true can be disabled when using an MEO, or another flag parameter that you list the options that are not to be used with it.

Can you show an example of what you mean? I'm not 100% sure I follow.

https://github.com/skorks/escort/blob/d0c021b92ea41128807a37854ec5486d0aa7150c/lib/escort/option_parser.rb#L69-L79
https://github.com/skorks/escort/blob/master/lib/escort/option_dependency_validator.rb
https://github.com/skorks/escort/blob/master/lib/escort/trollop.rb#L252-L264
And then https://github.com/skorks/escort#option-dependencies from the anchor down

Sorry, I still don't understand.  What is the behavior you would like to see? Can you show some command-line invocations of a hypothetical app that had this feature and how that app would behave?

Well first off, either way, I'd want it to be added onto the flag invocation, but something like this out of my project would qualify as a option I don't want to have to do a --no-* on
$ cloudflare_cli zones list

outputed is a ascii table (code mostly irrelavent to how options are parsed)
(--table is because  default_value: true)
$ cloudflare_cli zones --output 'result.name' --output 'result.status' --output-sep newline

which if table was a conflicted option and disabled itself when another conflicting option was used, should only output the following
ken@kenny:~/everygit/cloudflare_cli$ bundle exec bin/cloudflare_cli zones list \
  --output "result.first.name" \
  --output "result.first.status" \
  --output-sep newline
e-code.in
active

If there were a 'depends_on' then I could make --output-sep require at least one --output
$ cloudflare_cli --output-sep newline
error: flag 'output-sep' requires the option 'output' to be used (in tandem or similar word)

Then what was actually used
(--table put in asterisks for clarity)

bundle exec bin/cloudflare_cli zones list \
  **--no-table** \
    --output 'result.name' \
    --output 'result.status' \
    --output-sep newline

What actually gets outputted is then
ken@kenny:~/everygit/cloudflare_cli$ bundle exec bin/cloudflare_cli zones list --output "result.first.name" --output "result.first.status" --output-sep newline
+-----------------------+------------------------------------------------------------------+
|                                          Zones                                           |
+-----------------------+------------------------------------------------------------------+
|                                        e-code.in                                         |
+-----------------------+------------------------------------------------------------------+
.
. table contents
.
+-----------------------+------------------------------------------------------------------+
|                                     electrocode.net                                      |
+-----------------------+------------------------------------------------------------------+
.
. table contents
.
+-----------------------+------------------------------------------------------------------+
|                                      iotaspencer.me                                      |
+-----------------------+------------------------------------------------------------------+
.
. table contents
.
+-----------------------+------------------------------------------------------------------+
| success               | true                                                             |
| errors                | none                                                             |
| messages              | none                                                             |
+-----------------------+------------------------------------------------------------------+
e-code.in
active

OK, so it sounds like you are wanting a few things:

be able to specify that two options may not be specified together and if so, it's an error.
If an option has a default, and one of it's mutually exclusive options is set, don't set the default (set nil?)
be able to specify that two options are required together (opposite of mutually exclusive), i.e. if one is given, the other must be given as well.

Much of GLI's option parsing is deferred to the standard Ruby OptionParser, so to implement something like this, we'd need to:

Agree on what the DSL/API is like for the developer
Write code that happens after all options are parsed that can examine them and generate the requisite errors.
Probably changes in the help output to indicate options that are mutually exclusive or mutually required are that way

GLI has the concept of a pre hook that is given all the parsed options and can decide to execute the command or not, based on what's in there.  That's how you could implement this in your app directly, but I would think to make this a GLI feature, we could have some sort of built-in pre blocks to handle these checks.

Yes, and I can show you what I'm currently having to do to check all this, though I have made the default_value for table currently false so I don't have to deal with that as much

This being from here
as you can see its a bit cumbersome
and hopefully I can do something like
        lm.flag :'output-sep', desc: 'choose character to separate output fields', must_match: [:newline, :crlf, :space], required: false, default_value: :space, depends_on: [:output]
        lm.flag :output, desc: 'output fields, use multiple times to output multiple fields', multiple: true, conflicts_with: [:table, :json
        lm.switch :table, desc: 'Put some of the output into a table', default_value: false, conflicts_with: [:output, :json]
        lm.switch :json, desc: 'output raw json', default_value: false, conflicts_with: [:table, :output]

considering output is a multiple value and does exist in the options no matter what, unlike something being nil or not in the hash, this may cause some problems, do to most likely having to check if --output was given rather than just being empty
          if options[:json] and options[:table] and options[:output].empty?
            puts js.to_json
          elsif options[:table] and not options[:json] and options[:output].empty?
            table = CloudflareCli::Methods::Tables::Memberships.new('Memberships', js)
            table.make_it_so
            table.beam_me_out
          elsif not options[:output].empty? and options[:'output-sep'] and not options[:table] and not options[:json]
            want = CloudflareCli::Methods::Output.new(options[:output], options, js)
            want.put
          else
            if options[:json] and options[:table]
              raise FlagSwitchConflictError.new('json', 'table')
            end
            if options[:output].empty? and options[:'output-sep']
              raise FlagSwitchDependencyError.new('option-sep', 'output')
            end
          end

Well easy stuff first, help output:
ken@kenny:~/everygit/cloudflare_cli$ bundle exec bin/cloudflare_cli help memberships list
NAME
    list - list memberships

SYNOPSIS
    cloudflare_cli [global options] memberships list [command options]

COMMAND OPTIONS
    --direction, --dir=arg - direction to order results (default: asc)
    --[no-]json            - output raw json (conflicts with: --table, --output) # <---
    --order=arg            - field to order memberships by (default: none)
    --output=arg           - output fields, use multiple times to output multiple fields (may be used more than once, default: none, conflicts with: --table, --json) # <---
    --output-sep=arg       - choose character to separate output fields (default: space, depends on: --output) # <---
    --page=arg             - page to query (default: 1)
    --per_page=arg         - how many accounts listed per page (default: 20)
    --[no-]table           - Put some of the output into a table (conflicts with: --json, --output) # <---

something simple like this would also work
https://github.com/ManageIQ/optimist/wiki/Medium-Example#dependencies-and-conflicts

Closing as old.  Given that it can work with pre I hope that is working for you, but please re-open if not


Feature/obd feeder example

@SebastianSchildt Pls sync your master with eclipse master. And then you can see my changes.

Hmm merged master into my obd feeder branch. But still this shows millions of changes... Maybe Git needs a moment to render this again.... I will wait to see if it magically gets better :)
In the mid-term we should male a seperate repo with feeder only...

Hmm merged master into my obd feeder branch. But still this shows millions of changes... Maybe Git needs a moment to render this again.... I will wait to see if it magically gets better :)
In the mid-term we should male a seperate repo with feeder only...

I also need a merge. But now you can see the changes. now is the obd feeder much powerful


[helm/charts] [incubator/wave] Adds pusher/wave to helm incubator charts

Signed-off-by: Marcos Estevez<EMAIL_ADDRESS>This is a new chart under incubator.
What this PR does / why we need it:

wave an opensource tool maintained by pusher
wave is a Kubernetes controller to watch changes in ConfigMap and Secrets and then restart pods for Deployment, StatefulSet, and DaemonSet
Added to the incubator repository because it needs proper testing and more contributions before going to stable

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 starts with chart name (e.g. [stable/chart])

/ok-to-test

/verify-owners

@marcostvz
Do you guys mind to host your own chart? As per discussion with @maorfr :
https://github.com/helm/charts/pull/5071#issuecomment-520707994


Added URL provider

See issue #46
Two parts:

some minor cleanups I came across in the code
I added an URL provider. The URL provider is a barebone Provider class, which just returns the url when get_genome_download_link is called. I changed the get_localname function to get a localname from an url. I am not sure however if my 'heuristic' approach to spotting an url always works (checking for / . and : ). I could for instance change it so that it checks if urlopen(name) works.

You can now install through:
genomepy install https://research.nhgri.nih.gov/hydra/download/assembly/Hm105_Dovetail_Assembly_1.0.fa.gz URL
Is URL a good provider name? Capitals or not? Should we be afraid this breaks something somewhere? Should I add tests? The current test already fails at the build stage, does this indicate something is already wrong? Should I update the readme?

Please add a test, yes. Can be basic, just as long as this provider works.
A solution for the name issue could be that with a URL a localname should always be provided? On the  other hand, this is probably fine as is.
URL is fine. We should probably make the provider name case insensitive anyway. This could easily be done in ProviderBase.create().
Please update the README, yes, as well as CHANGELOG.md.

The test is failing due to some weird issue with conda.

Travis config is fixed in the develop branch, c50d920.

@simonvh

I added some simple tests
I changed that urllib checks if an url is a valid url
Providers are case insensitive now
README and changelog are updated


Implemented Logic and improved code

Implementing Logic and Improving code
Summary
This PR aims to implement the changes done by @XlokolokiX in our main brach, development, so everything is in one place.
Details

The logic of our system has been stablished and optimized. It is explained through our State Machine Diagram
Our system takes advantage of Systick updating it periodically

Fixed mi docs mistakes.


EPS files corrupted in german language environment

We have a german php environment (PHP7.2) with the following setting in php.ini:
intl.default_locale = de
Now the EPS files created with some errors.
In environment without the german settings the EPS started with ...
%!PS-Adobe-3.0 EPSF-3.0
%%BoundingBox: 0 0 320 320
/F { rectfill } def
2.55 2.55 2.55 setrgbcolor
0 0 320 320 F
0 0 0 setrgbcolor

In environment with the german settings this looks like:
%!PS-Adobe-3.0 EPSF-3.0
%%BoundingBox: 0 0 320 320
/F { rectfill } def
2,55 2,55 2,55 setrgbcolor
0 0 320 320 F
0 0 0 setrgbcolor

The problems are the decimal separators - so the EPS files are corrupted and can not opened by any graphic software.

Hi @LinkingYou I was unable to reproduce the issue, but I now use number_format to create the floats as string. Can you update to 3.2.8 and check if this solves the issue for you?

Great work. It works perfect now. Thanks you the quick respond.


File extension when saving vector graphics

When using VectorGraphicsEncoder.saveVectorGraphic the file extension is always added.
When using BitmapEncoder.saveBitmap the file extension is added only if it is not present already.
It would be nice if both methods would behave the same.
Thanks for the great work.

please test

Tested, works like a charm, thanks.
I tested the addFileExtension method using JUnit as well as the output using my application.
JUnit-Test in branch feature/test of https://github.com/ekleinod/XChart

Thank you!


Larger network

Now that we have an end-to-end pipeline working and initial results, we should think about adding in automated hyper-parameter tuning to allow easy determination of optimal parameters like learning rate, network size etc.
RayTune has been mentioned as a possible option. I also recall wanting to implement our own version to allow easier interaction with condor.

Now that we have an end-to-end pipeline working and initial results, we should think about adding in automated hyper-parameter tuning to allow easy determination of optimal parameters like learning rate, network size etc.
RayTune has been mentioned as a possible option. I also recall wanting to implement our own version to allow easier interaction with condor.

Could be a duplicate of #36


release raiwidgets and responsibleai 0.26.0

Description
v0.26.0

educational materials

Update notebooks for categorical features to pass through FeatureMetadata by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/2011

new features

Responsible AI Dashboard

Adding Object Detection Fridge Data by @natalie-isak in https://github.com/microsoft/responsible-ai-toolbox/pull/1998

RAIInsights

Support forecasting in RAIInsights by @romanlutz in https://github.com/microsoft/responsible-ai-toolbox/pull/1948

Model Overview

[Model Overview] Metrics Template support with OD-specific optional args by @Advitya17 in https://github.com/microsoft/responsible-ai-toolbox/pull/2002
Object Detection Model Overview Flask template by @Advitya17 in https://github.com/microsoft/responsible-ai-toolbox/pull/2004
Model Overview: Object Detection Widgets for Aggregate Methods, Class selection, & IoU Threshold by @Advitya17 in https://github.com/microsoft/responsible-ai-toolbox/pull/1997

bug fixes and tests

Responsible AI Dashboard

Forecasting: UI code adjustments to renamed datetime_features and time_series_id_features by @romanlutz in https://github.com/microsoft/responsible-ai-toolbox/pull/1993
[Refactor]1.Add DatasetCohort 2. Move FeatureFlights to core by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/2003
[Refactor]Add columnRanges and modelType to ModelAssessmentContext by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/2006

RAIInsights

Add validations for model predictions by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/2008
Add validations if target column is included in FeatureMetadata by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/2013
try pickling model instead of failing based on state functions and change to UserConfigValidationException by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/2014
update dependencies for rai-core-flask, raiwidgets and responsibleai packages by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/2016
Fix deprecated DataFrame indexer type in Error Analysis package by @JarvisG495 in https://github.com/microsoft/responsible-ai-toolbox/pull/2019

Error Analysis

fix bug in tree view where slider params are not used by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1990
fix error analysis showing float values with many zeros for some classification task nodes by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1991
release erroranalysis v0.4.2 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/2000
update raiwidgets and responsibleai to erroranalysis 0.4.2 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/2001

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

@gaugup just fixed!  I missed the version bump.


Issue on Ascent: {issue}

Hi I can not enable the compliance features anymore. It is read only. Thank you very much for any help!

@frankeisenhardt closing this as we've worked it out.


Easter Eggs

Because its fun.
Ideas

Dango Music before match starts (From "Clannad")
A Cruel Angel's Thesis before match starts (From "Neon Genesis Evangelion")

Added in 2112d2c


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4526

Questions ?
Please refer to the Backport tool documentation


Merging SimPleAC to master pt. 3

Why was this so hard...

Can one of the admins verify this patch?

add to whitelist

test models please

test models please

@bqpd I am still having trouble generating the pdf from README.md. Perhaps I can bug you for a few minutes after our meeting with Bob tomorrow?

for sure!

test models please

@bqpd I believe this is ready to rumble! Please review the pdf!

add to whitelist

test models please

test models please

test models please

@bqpd I have responded to all of your comments, and README.md.py runs.

Poop, wanted to squash merge, but it seems like it merged item by item without asking. Sorry...


Add QE identity verification to EvidenceVerifier

The EvidenceVerifier now verifies the signed QE identity data as well
as the QE report body's identity

Motivation

Current dependencies on/for this PR:

main

PR #129   👈

This comment was auto-generated by Graphite.


22 07 24 liberty square staking page

hey ansel, couple of notes on this:

not sure if the mappings.tsx changes are inclusive of what's needed to launch but i did my best
the rarity data looks missing to me or i couldn't find it. there is something called "Boost" that you'll see referenced. but i think the team still wants rarity names to be on the page for staked tokens.
the total rewards look assigned at the token level. i didn't have enough time to dig through the wallet for that info but i think they want a sub header that lists the total accrual of rewards.

i'm sure there's something else. let me know if i can clarify anything.
Thanks!

opened on accident. apologies


problem: pg_yregress doesn't support explicit connections

Implemented as a map that tracks the connection and re-use if the same connection is being used in further steps.
Also, added support for database, reset flags.

addressed the review comments. Updated test to validate session-level settings across multiple tests.
CI is failing on Extension scripts steps across all versions that it couldn't install omni_aws. I am able to build omni_aws locally. Any pointers on what is going on? Looks a recent change that you pushed? @yrashk

I thought about connection a little further. It came to my mind that there is an interplay between connection and instance (if you use both of them in a test, it should pick that name for a connection specific to that instance name) to be consistent.
This has also led me to a thought on how to eliminate introducing a dependency to UTHASH: use libfyaml to populate connection objects in individual instances objects and look them up there! The introduction of this kind of "phantom" nodes in the YAML document is already done when we provision "default" instances where none are defined.

CI is failing on Extension scripts steps across all versions that it couldn't install omni_aws

@viggy28 could you rebase your branch on to master to see if it goes away ? One of the configuration variable vars.EXT_UPGRADES_START used by a newer workflow extension scripts is empty for your PR's workflows


Verification at the APIM level of Access via an AAD group

When I visit the users list inside the APIM area in the Azure portal, it shows the users and any local groups they are a member of (local groups being developers and other groups created only on the AMS instance). Would it be possible to also show, or at least call out the method for visibility and verification that members of an AAD group also have appropriate access as users in the APIM - listing the AAD groups they are members of might help here.
Currently we have to trust that the provided group access (visible via Active Directory in Azure) is accurate, there is no way to directly correlate that access.
thanks
Derek

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

ID: 692852cd-11a6-1d6b-9540-caec9d0f14a4
Version Independent ID: 4c376561-6015-6dab-e23e-feffe74ccc8b
Content: Authorize developer accounts by using Azure Active Directory - Azure API Management
Content Source: articles/api-management/api-management-howto-aad.md
Service: api-management
GitHub Login: @miaojiang
Microsoft Alias: apimpm

@derekschauland Thank you for taking the time to provide feedback!  We want to make sure that this is seen by the product team, and we've created a channel for that very purpose:
https://feedback.azure.com
We'd be happy to submit this feature request on your behalf, but we'd like to give you the chance to submit directly in case you want to receive notifications, vote, or monitor progress more closely.  Let us know which you would prefer.  If we don't hear back from you in a day or two, we'll submit this for you and provide a link to the request.

I have it posted to the feedback site, however, if the docs team could assist in getting this seen and help it move up the backlog that would be awesome 😉

From: kobulloc-MSFT<EMAIL_ADDRESS>Sent: Wednesday, May 2, 2018 1:34:09 PM
To: MicrosoftDocs/azure-docs
Cc: Derek Schauland; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Verification at the APIM level of Access via an AAD group (#7984)
@derekschaulandhttps://github.com/derekschauland Thank you for taking the time to provide feedback! We want to make sure that this is seen by the product team, and we've created a channel for that very purpose:
https://feedback.azure.com
We'd be happy to submit this feature request on your behalf, but we'd like to give you the chance to submit directly in case you want to receive notifications, vote, or monitor progress more closely. Let us know which you would prefer. If we don't hear back from you in a day or two, we'll submit this for you and provide a link to the request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/7984#issuecomment-386077271, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AS1ioYHu79jCC2rFCX18Bw_eU4P3s4aMks5tufwhgaJpZM4Tvi3T.

@derekschauland Thank you for posting feedback!  I've voted for it and if anyone else wants to vote they can find the thread here:
https://feedback.azure.com/forums/34192--general-feedback/suggestions/34136242-verification-at-the-apim-level-of-access-via-an-aa
This is the best way to send feedback to the product team :)  Additionally, I've made a note of this and if there's anyway that we can update the documents to be clearer, we will do our best to do so.
We are going to close this thread as resolved but if there are any further questions regarding this matter, please tag me in your reply and we will be happy to continue the conversation.


bug: ionic cli add does not find bower packages

From @gpolyn on July 19, 2015 13:42
Type: bug
Platform: desktop browser
Components that a 'bower search' can find sometimes result in the following error message after executing 'ionic add'
Failed to find the bower component [...]
Are you sure it exists? (CLI v1.6.1)

Try, for example, well-known and useful packages angularfire and firebase: they appear in bower searches, but result in the error above.
Copied from original issue: driftyco/ionic#4107

This is odd, since ionic add is just an alias for bower install --save-dev
https://github.com/driftyco/ionic-cli/blob/master/lib/ionic/add.js#L16
Could you paste the out put from ionic info @gpolyn

Hi Mike,
I am also seeing this trying to run 'ionic add angularfire'
Here's my output of 'ionic info':
Your system information:
Cordova CLI: 5.0.0
Gulp version:  CLI version 3.9.0
Gulp local:
Ionic Version: 1.1.0
Ionic CLI Version: 1.4.5
Ionic App Lib Version: 0.0.22
ios-deploy version: 1.7.0
ios-sim version: 4.1.1
OS: Mac OS X Yosemite
Node Version: v0.12.4
Xcode version: Xcode 6.4 Build version 6E35b

Weirdly enough, 'bower install --save-dev angularfire' did work without hitch.

Hmm, just gave ionic add angularfire a try and it worked fine.
Please update your ionic cli to the latest and try again.
Cordova CLI: 5.2.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.0
Ionic Version: 1.1.0
Ionic CLI Version: 1.6.4
Ionic App Lib Version: 0.3.8
ios-deploy version: 1.7.0
ios-sim version: 3.1.1
OS: Mac OS X Yosemite
Node Version: v0.10.40
Xcode version: Xcode 6.4 Build version 6E35b

ionic platform add android
worked for me instead of
node
ionic add platform android

Actually this is occuring when bower is not installed on your computer, ionic don't say it if it's the case
So, you can do:
npm install -g bower
ionic add angularfire
Think it should work now :)

Looks liked the latest beta release of the CLI handles this scenario. Closing this issue.

Simple use the command ionic start myapp [template]


Obsolescence and "Internal Use only"

Today we have 48 obsolete API in the SKD that do not have a hint to an alternative and a lot of obsolete API declared as "Internal Use Only".
Can we have a version of the SDK consolidating this issues.

hello @joern-schlingensiepen,
The CDM SDK has evolved over the years and as part of this process we've added new APIs that are easier to use, while marking those that could cause confusion to our customers as deprecated.
As you mentioned, some of these deprecated APIs are used internally by our code so we still kept them.
We also didn't completely remove the deprecated APIs because that could cause problems for customers when updating to a new version.
Is there a specific scenario where you see the use of an API that is marked as deprecated?

Dear Victor,
I like the idea of using tags like 'deprecation' to ensure a development of tools. It would be helpful to give users an advice which alternatives one has to use. In most cases this is done in CDM SDK, but for 48 APIs I do not see this.
My current issue is 'namespace'. We can mount different LocalStores for different Namespaces (which implies a relation between a namespace and a document). From my (current) understanding namespaces can help to organize different types in logical packages, e.g. to avoid interference in naming, etc.
But since the API for namespaces in Documents is 'deprecated' I am sure that I misunderstood this concept. My approach was, asking the Owner (which is a document) for the namespace. An alternative is using the namespace of a folder. But I do not see any api for getting a folder of a CdmObject (e.g. CDMEntityDefinition).
Could you give me an advice, where to find information about organizing namespaces in the docs?
Best Regards
Jörn

The namespace for a document is determined by the adapter that it originates from. I believe that it is marked as deprecated because it should not be modified by the user and that should only be used internally to determine paths for objects inside of the document. What are you trying to accomplish with the namespace? Are you just trying to keep track of what adapter each document comes from?

Hi,
we try to express data models from industrial context in CDM, so it would be nice to map their namespace structure to a namespace structure in CDM.
So, for me it is not a question, where does a ___Definition comes from but where to place it on creation.
BR
Jörn

Namespace for CDM has to do more with the location where the definitions exist. It sounds to me like you should use CdmFolderDefinitions and CdmManifestDefintinitions to replicate the namespace structure that you have. Maybe for each namespace you have, you can create a new manifest (with the name of that namespace) and that manifest would contain all of the entities that you expect in that namespace.


Fix require external registration password

Investigating #8558, I've found that REQUIRE_EXTERNAL_REGISTRATION_PASSWORD was incorrectly applied in the registration form. Unrelated REQUIRE_EXTERNAL_REGISTRATION_CAPTCHA was used instead to drive the password requirement.

:flushed: Sorry, it was not me who found this, it was @jolheiser !!! I'm just absentminded as usual. :sweat_smile:

For the record, I think this is the root of #8558 as well.
It mistakenly asks you for a password, but accounts created from GitHub don't set a password in Gitea and so when you go to delete your account or change your password, you need to use an empty password.

Codecov Report

Merging #8885 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8885      +/-   ##
==========================================
- Coverage   41.26%   41.21%   -0.05%
==========================================
  Files         544      544
  Lines       70019    70050      +31
==========================================
- Hits        28891    28872      -19
- Misses      37438    37482      +44
- Partials     3690     3696       +6

Impacted Files
Coverage Δ

routers/user/auth.go
11.93% <0%> (ø)
:arrow_up:

modules/sync/unique_queue.go
78.57% <0%> (-10.72%)
:arrow_down:

modules/indexer/indexer.go
44.73% <0%> (-10.53%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

modules/process/manager.go
78.37% <0%> (-4.06%)
:arrow_down:

models/repo_indexer.go
66.54% <0%> (-3.64%)
:arrow_down:

modules/migrations/gitea.go
8.68% <0%> (-2.97%)
:arrow_down:

routers/api/v1/repo/repo.go
68.8% <0%> (-0.61%)
:arrow_down:

models/error.go
31.34% <0%> (-0.42%)
:arrow_down:

models/repo.go
49.12% <0%> (+0.07%)
:arrow_up:

... and 1 more

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 55bdc9a...b26588d. Read the comment docs.

For the record, I think this is the root of #8558 as well.
It mistakenly asks you for a password, but accounts created from GitHub don't set a password in Gitea and so when you go to delete your account or change your password, you need to use an empty password.

@jolheiser Unfortunately, it doesn't seem to be the case. I've made all kinds of checks with and without this PR. But there's something stranger going on there. Check my latest comments.

Shouldn't LinkAccountPostSignIn also match? Currently it just has ctx.Data["DisablePassword"] = setting.Service.AllowOnlyExternalRegistration

Shouldn't LinkAccountPostSignIn also match? Currently it just has
ctx.Data["DisablePassword"] = setting.Service.AllowOnlyExternalRegistration

@jolheiser You're right. Although it's an edge case in case of some error happens before ctx.HasError() (some network timeout, perhaps). Fixed.


request to support Heat Pump Room Thermostat

Hello,
Thank you for the time invested in creating this component.
I have tried the add on and while adding the device it only promote the switch.
I have heat pump thermostat from company called BECA. There is custom firmware available but I have not tried it.
Having said that, Can you please check the possibilities to support this device in your addon?
here is a link of the device it self
My device generated the following
{
"result": {
"active_time":<PHONE_NUMBER>,
"biz_type": 18,
"category": "wk",
"create_time":<PHONE_NUMBER>,
"icon": "smart/program_category_icon/wk.png",
"id": "quoted for security",
"ip": "quoted for security",
"lat": "quoted for security",
"local_key": "quoted for security",
"lon": "quoted for security",
"model": "热泵温控器",
"name": "Room thermostat",
"online": true,
"owner_id": "quoted for security",
"product_id": "quoted for security",
"product_name": "HeatPump Thermostat",
"status": [
{
"code": "switch",
"value": true
},
{
"code": "temp_set_f",
"value": 77
},
{
"code": "temp_current_f",
"value": 78
},
{
"code": "level",
"value": "1"
},
{
"code": "eco",
"value": false
},
{
"code": "child_lock",
"value": false
}
],
"sub": false,
"time_zone": "+00:00",
"uid": "xxxxxxxxxx",
"update_time": xxxxxx,
"uuid": "xxxxxxxxx"
},
"success": true,
"t": xxxxxxxxxx
}
if any data required please let me know and I will try to provide every thing possible to me
Thanks

Please send the logs messages from Home Assistant that are output when it is detected as "Simple Switch".
That will have the list of DPS ids returned by the local protocol, which we can try to match up with the functions (switch, temp_set, temp_current, level, eco, child_lock).

I'm thinking of using the product_id in future, so if you can uncloak that (it is linked to the product, not unique to your specific device), it might help with future plans for improving detection.

Hello again,
Here is the log generated from the component
2021-07-22 16:00:04 INFO (MainThread) [custom_components.tuya_local.device] Setting protocol version for Test to 3.3.
2021-07-22 16:00:05 WARNING (MainThread) [custom_components.tuya_local.config_flow] Device matches simple_switch with quality of 14%. DPS: {'1': True, '2': 77, '3': 87, '4': '3', '5': '3', '6': False, '7': False, 'updated_at':<PHONE_NUMBER>.1256018}
2021-07-22 16:00:05 WARNING (MainThread) [custom_components.tuya_local.config_flow] Report this to https://github.com/make-all/tuya-local/issues/
2021-07-22 16:00:08 INFO (MainThread) [custom_components.tuya_local.device] Creating device: "quoted for security"
2021-07-22 16:00:08 INFO (MainThread) [custom_components.tuya_local.device] Setting protocol version for Simple Switch to 3.3.
And for the product ID as requested
"product_id": "drwezxukumr9ohuh",

It looks like most of the functions available on the control panel are not exposed via Wi-Fi. Only the functions mentioned in the web API capture you posted first, and one additional numeric field.  I see the temperature units are selectable, so I hope the extra numeric field is an indication of those.  But the current value is 3, and the units only seem to be selectable between C and F, so perhaps not. If the temperature units are not indicated by the response from the device, it might be more complex to support.
I have checked in an experimental config, please reinstall the Tuya Local integration using the "main" branch, which you can do from HACS using the menu button on the Tuya Local card, and selecting "Reinstall", then on the version, instead of the default latest release of 0.8.9, select main, which should be at the bottom of the list.
This hopefully will let you use the Thermostat card, and in Developer Tools you will be able to observe the rest of the attributes.  Most of interest to me is whether my guess about the C/F selection is correct, as getting this wrong will probably make some strange behavior.  Also, there are a number of attributes that might be swapped - current and target temperature, the child lock and eco modes, and the level and other numeric indicator.

Hello again,
I will reinstall now,
I have observed the DPS and I found the following
DPS:
{
'1': True, "back light switch bool"
'2': 77, "set temp in Fahrenheit"
'3': 87, "ambient temp in Fahrenheit"
'4': '3', "function, using schdule 1, temporary hold 2, permanent hold 3, holiday hold 4"
'5': '3', "work mode, cool 1, heat 2, off 3, emergency heat 4, auto 5"
'6': False, "fan mode bool (on = true)(auto = false)"
'7': False, "child lock bool"

After reinstalling, I'm able to use temperature card, I can change temperature but it is not behaving correctly. When I choose eco, the fan change from auto to on, and normal it will change to on. And when I change operation from heat/cool to off  the backlight on the device change from on to off and vice versa.
Thank you

If this helps the following in developer tool
also a note, I have contacted Tuya about changing from F to C and they said it is not possible to change it in app only on the physical device. So I don't think there will be DPS for that I'm afraid.
`hvac_modes:

heat_cool
'off'
min_temp: 7
max_temp: 35
target_temp_step: 1
preset_modes:
normal
eco
current_temperature: 80
temperature: 74
preset_mode: normal
level: 3
friendly_name: Beca BHP-6000 Thermostat
supported_features: 17
`

If you reinstall again, I have reorganised based on your first comment from yesterday.  I created two separate configs for F and C, so you'll need to select the _F config for your device, which is configured for Fahrenheit.  Configuration from the previous version should automatically choose the Fahrenheit version, but if it doesn't, delete the device and set it up again.

Now it is working like a charm, just like expected. Thanks a lot.
As a side note, I have checked for C as well, when I set the value "say 20 C" it will promote the correct value and it will set it as 20 in the thermostat, however after some time in the card it will change back to the F values (68) without setting them to that value in the device which is good. I'm not sure if I'm clear. Also it is the same in the ambient temp.
Other note, for the backlight when it is switched off it will close the whole unit and I think what you have done to keep it as a separate entity same as the child lock is perfect for it Thanks and appreciated
Cheers

Thanks for your checking.  The new release 0.9.0 has the support for this device now.


Unable to reproduce results

Hi @bo-miao  thanks for open sourcing the nice work!
When I run your checkpoint with the eval script on DAVIS2017 val set, the performance is indeed 70.4. Then I tried to replicate your results by running train.sh without any modifications, but I was getting an J&F of 67.65 only, with J=0.662 and F=0.691. I'm using the train_all_frames in YouTube-VOS (the valid_all_frames and test_all_frames are not used) to train.
Is this behavior (large variance in model performance, maybe depending on the random seed) normal, or am I setting up the training data incorrectly?

Hi,
We only use the training set of YouTube-VOS to train our model, the training data are sampled from consecutive frames instead of the data version that sampled at an interval of 5. We first resize the images of the whole training data of YouTube-VOS into 256x256 and store them on the server, and then train our model using the resized images.
The snapshot of our environment configuration is shown as follows. Moreover, we found that using the model of 32/33 epochs performs better than that of 35 epochs, and the released checkpoint is the model that trained for 33 epochs.
Collecting environment information...
PyTorch version: 1.9.0a0+gitc91c4a0
Is debug build: False
CUDA used to build PyTorch: 11.2
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.2 LTS (x86_64)
GCC version: (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
Clang version: Could not collect
CMake version: version 3.19.6

Python version: 3.9 (64-bit runtime)
Is CUDA available: True
CUDA runtime version: 11.2.142
GPU models and configuration: GPU 0: GeForce RTX 3090
Nvidia driver version: 460.67
cuDNN version: Probably one of the following:
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_adv_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_adv_train.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_cnn_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_cnn_train.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_ops_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_ops_train.so.8
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.19.5
[pip3] torch==1.9.0a0+gitc91c4a0
[pip3] torchgeometry==0.1.2
[pip3] torchstat==0.0.7
[pip3] torchvision==0.10.0a0+a64b54a
[conda] mkl                       2020.2                      256    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] mkl-include               2020.2                      256    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] numpy                     1.19.5                   pypi_0    pypi
[conda] torch                     1.9.0a0+gitc91c4a0          pypi_0    pypi
[conda] torchgeometry             0.1.2                    pypi_0    pypi
[conda] torchstat                 0.0.7                    pypi_0    pypi
[conda] torchvision               0.10.0a0+a64b54a          pypi_0    pypi

Hi,
We only use the training set of YouTube-VOS to train our model, the training data is from train_all_frames. We first resize the images of the whole training data of YouTube-VOS into 256x256 and store them on the server, and then train our model using the resized images.
The snapshot of our environment configuration is shown as follows. Moreover, we found that using the model of 32/33 epochs performs better than that of 35 epochs, and the released checkpoint is the model that trained for 33 epochs.
I hope this helps.
Collecting environment information...
PyTorch version: 1.9.0a0+gitc91c4a0
Is debug build: False
CUDA used to build PyTorch: 11.2
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.2 LTS (x86_64)
GCC version: (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
Clang version: Could not collect
CMake version: version 3.19.6

Python version: 3.9 (64-bit runtime)
Is CUDA available: True
CUDA runtime version: 11.2.142
GPU models and configuration: GPU 0: GeForce RTX 3090
Nvidia driver version: 460.67
cuDNN version: Probably one of the following:
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_adv_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_adv_train.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_cnn_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_cnn_train.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_ops_infer.so.8
/usr/local/cuda-11.2/targets/x86_64-linux/lib/libcudnn_ops_train.so.8
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.19.5
[pip3] torch==1.9.0a0+gitc91c4a0
[pip3] torchgeometry==0.1.2
[pip3] torchstat==0.0.7
[pip3] torchvision==0.10.0a0+a64b54a
[conda] mkl                       2020.2                      256    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] mkl-include               2020.2                      256    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] numpy                     1.19.5                   pypi_0    pypi
[conda] torch                     1.9.0a0+gitc91c4a0          pypi_0    pypi
[conda] torchgeometry             0.1.2                    pypi_0    pypi
[conda] torchstat                 0.0.7                    pypi_0    pypi
[conda] torchvision               0.10.0a0+a64b54a          pypi_0    pypi

Thanks for your detailed explanation! I'll try this.


The guidance decorator should drop the first argument from the wrapped function

Is your feature request related to a problem? Please describe.
The guidance decorator returns a function with one fewer argument than the function being wrapped (namely, the first argument, e.g. lm is "popped" off).
Because the wrapper @functools.wraps the original function, the signature of the wrapped function incorrectly still has that first popped argument, making linters like Pylint complain and providing an incorrect signature when using help or inspect.signature.
import guidance
import inspect

@guidance
def func(lm, x, y, z):
   ...

# Shows <Signature (lm, x, y, z)>, when it should be <Signature (x, y, z)>
inspect.signature(func)

Describe the solution you'd like
inspect.signature looks for an attribute __signature__ before falling back to actually extracting the signature from a callable, so the fix here is rather simple.
In the guidance decorator, after the wrapped function is decorated with @functools.wraps, we can use inspect to get the signature of the wrapped function, pop off the first argument, and add the new signature to the wrapped function as __signature__
I.e.
# in guidance.__init__

def _decorator(f, ...):
    ...
    @functools.wraps(f)
    def wrapped(*args, **kwds):
        ...

    # Remove the first argument of f from the wrapped function
    signature = inspect.signature(f)
    params = list(signature.parameters.values())
    params.pop(0)
    wrapped.__signature__ = signature.replace(parameters=params)

    return wrapped

Additional context
It may also promote clarity to add some type annotations, e.g.
from guidance.models import Model
from guidance._grammar import Function
from typing import Callable
from typing_extensions import Concatenate, ParamSpec

P = ParamSpec("P")

def _decorator(f: Callable[Concatenate[Model, P], Model], ...) -> Callable[P, Function]:
    ...

@Harsha-Nori do you have any opinions on the suggested type annotation? I'm not typically in the "type annotate all the things!" camp, but the decorator is (1) public facing and (2) definitely doing something non-trivial to the signature (and return type) of the wrapped function. If you're open to it, I'll put together a tiny little PR or piggyback on https://github.com/guidance-ai/guidance/pull/634 before it's closed.
Side note: I believe that while Concatenate and ParamSpec are in typing as of python 3.10, typing_extensions backports them as far as 3.8.

Closed by https://github.com/guidance-ai/guidance/pull/634


181 moving a file to itself will no longer throw an exception

See http://stackoverflow.com/questions/40722205/should-mscorlibs-fileinfo-moveto-throw-an-exception-if-trying-to-move-a-file

Due to activity, this will be handled in #299


how to use lemmaization?

We will add example early March this year :-)

Sorry May (not March :-) )

Any news on that?


Pydantic handling of timestamps possibly incorrect

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I am passing the output from a database to a model. Snowflake timestamp_tz format outputs a date in the format 2024-09-30 01:40:10.069 +0000
When passing this as input to a model:
from pydantic import AwareDatetime, BaseModel

class Tester(BaseModel):
    created: AwareDatetime

print(Tester.model_validate({"created": "2024-09-30 01:40:10.069 +0000"}))

I get:
  Input should be a valid datetime or date, unexpected extra characters at the end of the input [type=datetime_from_date_parsing, input_value='2024-09-30 01:40:10.069 +0000', input_type=str]

However, if I remove the extra whitespace as below there is no problem in the output:
from pydantic import AwareDatetime, BaseModel

class Tester(BaseModel):
    created: AwareDatetime

print(Tester.model_validate({"created": "2024-09-30 01:40:10.069+0000"}))

Looking at a few different databases it seems that its common for the string representation to have a space before the + sign. In the absence of pydantic supporting this, we'd have to add a custom validator to transform the input before its parsed in pydantic which seems counter productive.
Example Code
from pydantic import AwareDatetime, BaseModel

class Tester(BaseModel):
    created: AwareDatetime

print(Tester.model_validate({"created": "2024-09-30 01:40:10.069 +0000"}))

Python, Pydantic & OS Version
2.7.4

Hmm, it makes sense to me that this format isn't accepted given our datetime docs.
Will chat with the team to see if we'd consider adding support for this.

Following up here - looks like that whitespace isn't RFC3339 nor ISO 8601 compliant (i.e. it's nonstandard), so we don't intend to support this. The right approach here for your use case is the custom validator 👍

Thanks @sydney-runkle appreciate you checking up on this