Mint SQL Database

[ ] SQLite
[ ] postgress

Have you give any though about what crate should this project use handle the db stuff?
AFAIK there are two major crates for that: sqlx and diesel. Both support SQLlite and Postgres.
I'm more familiar with diesel, that's what I use in my daytime job. Diesel is an ORM, db queries look like this:
let versions = Version::belonging_to(krate)
  .select(id)
  .order(num.desc())
  .limit(5);
let downloads = version_downloads
  .filter(date.gt(now - 90.days()))
  .filter(version_id.eq_any(versions))
  .order(date)
  .load::<Download>(&mut conn)?;

It's also very fast compared to other crates (see: https://github.com/diesel-rs/metrics). By the way, this metrics repo also has comparisons between sqlite and postgres.
The downsides are the error messages, which sometimes are honestly disgusting.
I can't talk much about sqlx. All that I known is that it has compile-time verification for queries and it's not an ORM, so queries look like normal sql queries written inside macros:
let countries = sqlx::query!(
       "
SELECT country, COUNT(*) as count
FROM users
GROUP BY country
WHERE organization = ?
       ",
       organization
   )
   .fetch_all(&pool) // -> Vec<{ country: String, count: i64 }>
   .await?;

I'm obviously biased towards diesel, since I only have xp with it. I'm interested to read your thoughts on this.
Also, I think this is more important of a decision and will affect the project more than what db should we use. In that matter tho, I think it's probably best to start with sqlite because its simpler. There's no reason to not support both in the future if needed.

Thanks for posting this is a good thing to discuss that I've thought a bit about but don't have direct experience with either.
In general I think ORMs like diesel, while they provide some nice conveniences they're an unnecessary abstraction, over something like SQLx where SQL can be used. Using an ORM means using the ORM's specific syntax and hiding the true SQL that is generated and run.
Though the syntax of diesel is a bit nicer and the the speed of the crate is compelling at least on those metrics.
To avoid the abstraction i lean towards not using an ORM and using sqlx but certainly open to hearing more thoughts on it.

That's a fair point. SQLx is fine by me


[Snyk] Security upgrade node-sass from 7.0.1 to 7.0.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

479/1000  Why? Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-SCSSTOKENIZER-2339884
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: node-sass
    The new version differs by 16 commits.

3b556c1 7.0.2
c716359 Bump<EMAIL_ADDRESS>(#3292)
24741b3 docs(readme): fix docpad plugin link
1523330 feat: Drop Node 12
365d357 update https://registry.npm.taobao.org to https://registry.npmmirror.com
1456114 build(deps): bump actions/upload-artifact from 2 to 3
b465b69 chore: bump GitHub Actions to Windows 2019 (#3254)
e6194b1 build(deps): bump make-fetch-happen from 9.1.0 to 10.0.4
4edf594 build(deps): bump node-gyp from 8.4.1 to 9.0.0
29e2344 build(deps): bump actions/checkout from 2 to 3
85b0d22 build(deps): bump actions/setup-node from 2 to 3
3bb51da Use make-fetch-happen instead of request (#3193)
adc2f8b build(deps): bump true-case-path from 1.0.3 to 2.2.1 (#3000)
77d12f0 chore: disable Apline for Node 16/17 builds
308d533 ci: use Python 3 for Node 12
c818907 ci: unpin actions/setup-node to v2

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

 Quality gate: NA
Details


Turning off linkstatic = 1 does not produce dynamic libraries.

I followed the guidance listed here (https://github.com/google/tcmalloc/issues/27) on how to build a static library. In the various BUILD files, I commented out the linkstatic = 1 lines, however, I can't find any .so files produced:
find tcmalloc/ -name *.so
Since I'm on a SLES 15 HPC cluster, I can't install packages through zypper/apt/yum (no root access) so I am not sure how I can generate a .so file.

Bazel places binaries in bazel-bin.
$ git diff | cat
diff --git a/tcmalloc/BUILD b/tcmalloc/BUILD
index 97e2f72..523d742 100644
--- a/tcmalloc/BUILD
+++ b/tcmalloc/BUILD
@@ -88,7 +88,6 @@ cc_library(
         "tcmalloc.h",
     ],
     copts = TCMALLOC_DEFAULT_COPTS,
-    linkstatic = 1,
     visibility = ["//visibility:public"],
     deps = tcmalloc_deps + [
         ":common",
$ bazel build //tcmalloc:tcmalloc
Starting local Bazel server and connecting to it...
INFO: Analyzed target //tcmalloc:tcmalloc (32 packages loaded, 442 targets configured).
INFO: Found 1 target...
Target //tcmalloc:tcmalloc up-to-date:
  bazel-bin/tcmalloc/libtcmalloc.lo
  bazel-bin/tcmalloc/libtcmalloc.so
INFO: Elapsed time: 19.190s, Critical Path: 5.23s
INFO: 108 processes: 108 processwrapper-sandbox.
INFO: Build completed successfully, 111 total actions
$ find bazel-bin/tcmalloc/ -name '*.so'
bazel-bin/tcmalloc/libtcmalloc.so

That said, the libtcmalloc.so will have further deps (like on Abseil), so those will need to be findable by the loader.

How to link abseil in tcmalloc.so?


💡 [FEATURE] - Changing Navbar Style

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
I would like to change Navbar style as its not visible while on scrolling when background gets white.

Explain your solution
I would like to decrease the transparency of the navbar background to make it more visible and also add more space between the start of About content and the Navbar.

Any alternative approaches/features
Alternative approach could be to modify About Style instead to suit the dark mode by changing the background to dark color and changing font color to lighter colors.

We;re currently redesigning the site #998


Fixed #27 support glide package management.

Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling 210cefd7491e51ce02709a7165e979e8c6a69da7 on glide into 894418533ad2690ed3577b0ed2d59fd97f5ee311 on master.


Question about running simulator and generating offline training data

Hello, thank you very much for your excellent research work. However, I have encountered some problems while running your code recently, so I would like to ask for your help.

When I run the "collect_data.py" file located in the "code" folder under the  "Simulator" , I get an error: Traceback (most recent call last):

Traceback (most recent call last):
  File "collect_data.py", line 435, in <module>
    loop_cnt += 1
NameError: name 'loop_cnt' is not defined

I tried to locate the error and found that changing the loop_cnt variable in the collect_data.py file from line 413-437 to robotloop_cnt allowed the code to run normally. I was wondering if there is a better solution to this problem?
The original code is as follows. Could you tell me how this problem occurred?

2, When I run the scripts/run_gen_offline_data.sh script, all of my output data is "cmd fail [-1]". I wonder how this happened? I have downloaded and properly stored the data according to the instructions in the data folder's readme file.

I would greatly appreciate it if you could reply in a timely manner. Thank you again for your wonderful work.

Hi Xinyu,
Thank you for reaching out and for your interest in our work. We appreciate your feedback.

Thanks for pointing out the bug in variableloop_cnt. The collect_data.py has been updated.

cmd fail means that this particular data point encounters object clipping in the scene. Our scene configuration is implemented by putting objects one by one, which may clip with each other. When this happens, we disgard such a data point and print cmd fail, see #1 .

Thanks and hope your experiments go well!


Syntax Error in PDKs

Description
Our team encountered syntax error while executing GLS on the verilog file using "FUNCTIONAL" directive in the path OpenLane/pdks/sky130A/libs.ref/sky130_fd_sc_hd/verilog/sky130_fd_sc_hd.v in line number 49162. We should be commenting this SKY130_FD_SC_HD__LPFLOW_BLEEDER_FUNCTIONAL_V in that line then its giving accurate result.
Environment

Kernel: Linux v3.10.0-1160.59.1.el7.x86_64
Distribution: centos 7
Python: v3.6.8 (OK)
OpenLane Git Version: 0c611a3422f0a9691c10776f951efd32887e85e5
pip: INSTALLED
pip:venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

0c611a3 2022-05-09T08:39:57-07:00 Update to use the new C++ pdngen in OR (#1059) - Matt Liberty -  (HEAD -> master, tag: 2022.05.11_01.53.47, origin/master, origin/HEAD)
47c9f22 2022-05-06T14:32:57+02:00 Remove Hard-coded Paths in Sample Designs (#1083) - Balint Cristian -  (tag: 2022.05.07_01.50.30)
0c2d66a 2022-05-04T11:11:24+02:00 Global Placement Fixes (#1039) - Anton Blanchard -  (tag: 2022.05.05_02.01.30)

PDKs version:- open_pdks 41c0908b47130d5675ff8484255b43f66463a7d6
Expected behavior
We should be commenting this SKY130_FD_SC_HD__LPFLOW_BLEEDER_FUNCTIONAL_V in line number 49162, then its giving accurate result.

Logs
We are getting syntax error at OpenLane/pdks/sky130A/libs.ref/sky130_fd_sc_hd/verilog/sky130_fd_sc_hd.v in line number 49162

Thanks,
Eswar

@RTimothyEdwards
FYI

@vijayank88 We thought we fixed this a long time ago, tbh

@donn
I checked that file. That particular line was not commented.

Fixed


Internal Server Error

Hello, can you please suggest me in this?
I have followed the instructions properly (found the dll and run regasm successfully, too)
The only thing I had to modify was setting the startup project to Translator. The program starts, Open file dialog opens, the context menu is there on my rvt file, but after selecting it I got this unhandled exception:
System.Reflection.TargetInvocationException was unhandled   HResult=-2146232828   Message=Exception has been thrown by the target of an invocation.   Source=mscorlib   StackTrace:        at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)        at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)        at System.Delegate.DynamicInvokeImpl(Object[] args)        at System.Windows.Forms.Control.InvokeMarshaledCallbackDo(ThreadMethodEntry tme)        at System.Windows.Forms.Control.InvokeMarshaledCallbackHelper(Object obj)        at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)        at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)        at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)        at System.Windows.Forms.Control.InvokeMarshaledCallback(ThreadMethodEntry tme)        at System.Windows.Forms.Control.InvokeMarshaledCallbacks()        at System.Windows.Forms.Control.WndProc(Message& m)        at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)        at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)        at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)        at System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg)        at System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop(IntPtr dwComponentID, Int32 reason, Int32 pvLoopData)        at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)        at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)        at System.Windows.Forms.Application.Run(Form mainForm)        at Translator.Program.Main(String[] args) in C:\Users\My\work\fiverr\revit\Translator\Program.cs:line 42        at System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, String[] args)        at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args)        at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()        at System.Threading.ThreadHelper.ThreadStart_Context(Object state)        at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)        at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)        at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)        at System.Threading.ThreadHelper.ThreadStart()   InnerException:         HResult=-2146233088        Message=Error uploading file: InternalServerError        Source=Translator        StackTrace:             at Translator.Server.<UploadFile>d__1.MoveNext() in C:\Users\My\work\Translator\Server.cs:line 53          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)             at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)             at Translator.Progress.<Progress_Load>d__5.MoveNext() in C:\Users\My\work\Translator\Progress.cs:line 48          --- End of stack trace from previous location where exception was thrown ---             at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.<ThrowAsync>b__6_0(Object state)        InnerException:

Despite the error happens on the desktop app, it's actually throwing an "Internal Server Error", meaning the server is the one with problems. Are you running it locally? If so, check a breakpoint at TranslatorController.cs - UploadObject method. If is running on a host, check if you enable write access to AppData folder.

Thanks for answering!
Yes, I'm running it locally. The UploadObject method will not be hit, the debugger says.
Anyway, was it correct to set the startup project to Translator? Shouldn't the TranslatorServer also be started (see the result in attachment)? And I don't have the context menu on any rtv files outside the program. Is it intended to work like this?
Plus, I need to create an addin to Revit from this (without the context menu, the exporting is the point), how should I start that?

To debug you should start the TranslatorServer, then use the right-click to launch the desktop app (this upload the .RVT, check for status and download .XLS).
It can work on a Revit plugin, just move the code from Translator.exe into your plugin, meaning it will get the file and upload it from inside Revit, just make sure the file is read-enabled.
The translator server don't require directory listing as there is nothing there, just the API endpoints.

OK, now I'm aware of the right workflow, but there is something with TranslatorServer at my end :(
I omitted the directory listing as it is not needed, still when I start TranslatorServer, a browser opens, but with the error (dirlisting) from previous comment. When I'm debugging, it just ends after WebApiConfig' Register method (I mean the http://localhost:3000/api/forge/callback/oauth loads with error, then the debugger options are unavailable, I have to stop it manually)
Many thanks for your help!

On the project settings, ensure that TranslatorServer is configurated to do not launch page on debug, there is no need. If you have a Revti plugin on the same solution as the TranslatorServer, go to solution properties and configure to start 2 projects on debug. That way you can debug it property.

OK, thanks, I set Start Action to "Don't open a page. Wait for....".
I have no plugin code yet, I'd like to make your project work at my end first.
But still, when I start the TranslatorServer it seems to run, but the context menu doesn't appear on any rvt file... So I guess the desktop app won't start. I have no clue where to put any breakpoints, because any of them seems to be hit...

the desktop app will not start manually, the context menu starts it. The context menu needs to be registered with regasm, have you perform this step? Any error?

I remember setting it without errors. Is it needed to set more than once? Anyway, I did it again, the result is in attachment (same as first time). Does debug or release version matter? (I registered the one in obj/Debug first, but now the Release, too)

Well, I've skipped the deployment part so far, because I thought it wasn't needed for local runs. Now I tried to follow the steps here: http://adndevblog.typepad.com/cloud_and_mobile/2017/01/deploying-forge-aspnet-samples-to-appharbor.html , but I have to admit, I have no idea of deploying such things (nor building it into a plugin, unfortunately).
If you have more time to suggest me on this, I appreciate it much, otherwise I've got to get familiar with the javascript version.

I'll prepare a video explaining this sample, that might help.
Meanwhile, as you mentioned a Revit plugin, why not write it using Revit APIs? There is no need to use Forge if you have the .RVT project open on Revit. Please correct if I missed anything here.

No, you didn't miss anything, I'm the new and incompetent in the area right now, my idea was to convert this somehow to export the actual rvt in Revit via plugin. I don't know it yet how to achieve without this project (and the javascript version), I found those two as the only existing samples, so I thought they can be a good starting point to create the plugin.

See video demo: https://www.youtube.com/watch?v=Vu8yAhmZUb0

Thank you so much!
I did exactly as you explained, but after starting the server, the context menu won't appear on any rvt file.
I have no idea what's different on my side...

you may need to remove it and add again, just replace the /codebase with /u to unregister, then try again. On the video I'm using the Framework64 folder for Windows 64 bit, so if you have a 32 bit OS, use the Framework folder instead.


Not working

09.06 10:27:33 [Server] Server thread/ERROR [Sponge]: Could not pass FMLInitializationEvent to Plugin{id=stackban, name=Stack Ban, version=0.3.0, description=Ban a variety of item actions, including crafting a configured item, placing or breaking and more, url=http://github.com/TrenTech/StackBan, authors=[TrenTech], source=/mods/stackban-6.1.0-0.3.0.jar}

nothing working

Partial log shows me nothing. Need full log

Okay what I did wrong was I had the wrong mod . Now everything works on banning a item from being craft that was the only thing wanted done but  once they for example use it on there pokemon and let say put it back on there inventory it takes the item and say this item is ban how do I prevent that from happening as the only thing I want is them from crafting the item

https://pastebin.com/GkYUnNvj

How can I stop  the item from being remove once they put it back in there inventory after it's being used?  The server is  a pixelmon based server, all I am trying to do is prevent a user from crafting it only but if they find the item, or get it from a reward it doesn't get deleted.
Here on my config files
https://pastebin.com/m9eEeKQj

hello?

Due to Sponges half finished Inventory API, and lack of a working recipe registry, there's not much I can do about it right now. The current method for banning crafting is the only reliable way to make sure it's consistent and doesn't cause item duping, but also has the unfortunate side effect of falsely detecting items in all inventories as just being crafted. Been trying to come up with a work around but not having a lot of luck. Blame Sponge for taking almost 3 years to develop a working Inventory API

Oh rip :(


Update CONTRIBUTING and README #1808

Summary of the discussion
Made changes to CONTRIBUTING.md and README.md in order to make it more accessable to new contributors. See Issue #1808
Type of change (CHANGELOG.md)
Update

rework contributing and readme

Workflow checklist
Automation
Closes #1808
PR-Assignee

[x] 🐙 Follow the Pull Request Workflow
[x] 📝 Update the CHANGELOG.md
[x] 📙 Add #'s to term tracker item (this doesn't apply as no changes are made to the oeo)

Reviewer

[ ] 🐙 Follow the Reviewer Guide
[ ] 🐙 Provided feedback and show sufficient appreciation for the work done

@LillyG901 please take a look at the Reviewer Guide and review this PR as 2nd reviewer.

@LillyG901
Since this pull request was already merged we will need a new issue for these changes. I'd be fine with implementing them, they all sound reasonable. I'd only like to talk about it in the meeting on monday first - maybe someone can figure out why the link to contributing doesn't show properly. In my editor it does but here in GitHub it is as you describe.

Alright sounds good to me :)


Dragging the color slider does not work until a click is registered in the large canvas

Testing #33295

With further testing, this might be a generic problem with shades of gray ?
I would also not expect the large canvas to change shades while I'm dragging inside of it. Notice how it turns from green to yellow and then changes to red:

Achromatic grays (rgb values are the same) have no hue (the hue code is indicated with a dash) so changing the hue will not change anything. We can probably disable hue strip in this case.

The reason that changing the Hue value for achromatic grays, whose /rgb values are the same/saturation is 0/, doesn't change the background of the large canvas is the same as color changing while dragging inside the canvas: the loss of Hue value.
We internally convert HSLA to RBGA and only stored the RGBA value. However, if the saturation of HSVA is 0, no matter what the Hue value is, the result RGBA will be the same, e.g. (Hue: 100, Saturation: 0, Luminosity).toRGBA() === (Hue: 25, Saturation: 0, Luminosity).toRGBA().


Would you mind to release rustler-0.30.1 which includes #570 ?

#570 fixed #569. If there will be the new release, users can get the rustler which resolves #569, dialyzer issue.

Released v0.31.0 which includes this and a few more changes.


no explanation

how exactly are you supposed to use profiles? You can't assign profiles to users as the boxes are greyed out and those belonging to multiple groups seem to get a profile at random.

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

ID: cbc54780-19ba-4d44-cd54-3435966b1293
Version Independent ID: 1417b68c-68fd-9f06-fafa-854edc3c1e75
Content: Manage users and roles - Business Central
Content Source: business-central/admin-users-profiles-roles.md
Service: dynamics365-business-central
GitHub Login: @edupont04
Microsoft Alias: edupont

Thank you for your feedback! As the note states, in the current version of Business Central, you cannot add, edit, or delete profiles. We have chosen to show the profiles so that you can see which users have access to which Role Centres, and we expect to be able to enhance this experience at a later point.
In the current version, access to different areas is controlled by user groups and permisssions. For more information, see this article: https://docs.microsoft.com/en-gb/dynamics365/business-central/ui-how-users-permissions
I hope this answers your concern. Otherwise, you can submit a request on the Ideas site at https://experience.dynamics.com/ideas/list/?forum=e288ef32-82ed-e611-8101-5065f38b21f1


[refactor] Remove duration, use timecode instead

Timecode can be used to for partial previews and renders by specifying any start or end time in the animation.

Previously the duration parameter assumed every render and preview started at time zero.

framerate has been moved from encoderSettings to timecode.
filename has been moved out of encoderSettings as well.
lengthMs has been removed from DeskScene. Just supply timecode instead.

Pull Request Test Coverage Report for Build 923624377

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

Totals

Change from base Build 922794558:
0.0%

Covered Lines:
2

Relevant Lines:
2

💛  - Coveralls


Handle sharing of options and arguments between subcommands

I'd like to use mow cli to implement sub commands that share a core of common options and arguments
My understanding is that it is not possible right now in mow cli without  redeclaring options and args in every sub (otherwise why would the reddit example duplicate the account declaration).
Please add a way to reuse option/arg between subs to avoid the usual drift when cut and parting
If I'm wrong and it is possible today, please clarify the documentation on that point.
Lastly thanks a lot for a nicely documented project. Good documentation is the main reason I chose mow cli over alternatives.

I would need to confirm and test, but maybe this can be done by assigning the same handler to many subcommands? Pass some value to the handler so it knows what subcommand it should be handling at any point. Hope I didn't misunderstood.

I have an initial positive result. I'm writing a CLI interface to a rest service, and this code appears to do the trick:
func main() {
	restClient := cli.App("restclient", "rest client")

....(toplevel configuration)

	restClient.Command("GET", "issue GET", func(cmd *cli.Cmd) { cmdMethod("get", cmd) })
	restClient.Command("POST", "issue POST", func(cmd *cli.Cmd) { cmdMethod("post", cmd) })
	restClient.Command("PUT", "issue PUT", func(cmd *cli.Cmd) { cmdMethod("put", cmd) })
	restClient.Command("DELETE", "issue DELETE", func(cmd *cli.Cmd) { cmdMethod("delete", cmd) })

	restClient.Run(os.Args)
}

func cmdMethod(method string, cmd *cli.Cmd) {
	cmd.Spec = "ENDPOINT ARGS..."

	var (
		endpoint = cmd.StringArg("ENDPOINT", "", "endpoint for the request")
		args     = cmd.StringsArg("ARGS", []string{}, "arguments for the request")
	)

	cmd.Action = func() {
		fmt.Printf("Method: %s\nEndpoint %s\nArgs: %v", method, *endpoint, *args)
		cli.Exit(0)
	}
}


Close, RSI, TALIB indicator don't display real values, depending on when you start the bot

I'm submitting a ...
[x] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
Start bot with the following indicators:

close
talib RSI
talib moving average1
talib moving average2

All history IS available according to the bot
The trading method requests 4021 minutes of historic data. Checking availablity..
gekko-mainbot_1
gekko-mainbot_1  |
gekko-mainbot_1  | 	Usable local data available, trying to match with exchange data..
gekko-mainbot_1  | 	Full history locally available. Seeding the trading method with all required historical candles.

Expected result (what you hoped would happen)
Numbers should match those shown on the exchange (Bitfinex, data seen through TradingView.com chart)
candleTimestamp: 	 Mon Jul 01 2019 12:57:00 GMT+0000 (does this mean it got data from the 12:00:00 tick????)
gekko-mainbot_1  |       			  closePrice: 		 $283.00
gekko-mainbot_1  |       			  rsi: 			 37.76
gekko-mainbot_1  |       			  fastMA: 	   	 297.22
gekko-mainbot_1  |       			  slowMA: 		 299.93

Actual result (unexpected outcome)
Close, RSI, and TALib Moving Averages are completely wrong. The numbers don't match at all on any tick.
Actual exchange numbers (seen on tradingview.com for Mon Jul 01 2019 12:00:00 GMT+0000 for BITFINEX:ETHUSD)
close: 	 		 $285.41
rsi: 			 37.32
fastMA: 		 298.56008
slowMA: 		 299.81649

Other information
This problem does not happen so dramatically on a shorter timeframe like 1 minute. The numbers are within a small tolerance of accuracy. But on the 3 hours timeframe, these numbers are way off.
I wonder if the indicators aren't properly warming up, because even though it says full history is available, and I did import the amount of ticks required for the strategy, its possible that the history isn't being properly used to calculate the numbers. However, what worries me is the price is wrong, which is not based on any indicator but direct exchange data.
If not this, then why would my numbers be so far off? Thanks for the help in advance!

UPDATE: My initial suspicion about the reasoning was correct. If I start the bot right on the same interval (within a few seconds before the next tick), then the bot numbers correspond exactly to the exchange numbers as viewed on tradingview.com
This means that if the bot is started in between the interval, it will still run at 3 or 4 hours candles, but the data will not match any exchange on the same hour candle due to the start of each candle differing.
I suggest we add a method into gekko config which would instruct the bot to "standby" but not actually start "ticking" the data feed intervals until X specified time in the future. We could say, make sure it starts on any 3 hour interval starting counting from 00:00 UTC. What do you guys think?

@askmike

Hey, i have the same problem! Were you able to find the solution?

Bot! Don't close the issue. It's still a real problem.

@Claudio-F the only solution I found is to schedule the Gekko CLI to start at the next candle tick using the Unix at command.
Example:
echo "docker-compose up | at 16:00"


System.MissingMethodException BulkExecutor..ctor when referencing .NET Standard from .NET Framework

Describe the bug
I am using Microsoft.Azure.CosmosDB.BulkExecutor 2.3.0-preview2.

To Reproduce
My .NET Framework 4.7.2 console app.
namespace NetFrameworkConsole
{
    class Program
    {
        static void Main(string[] args)
        {
            NetStandardLibrary.Class1.CreateBulkExecutor();
        }
    }
}

.NET Framework console app's csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net472</TargetFramework>
    <OutputType>Exe</OutputType>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Azure.DocumentDB.Core" Version="2.3.0" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\NetStandardLibrary\NetStandardLibrary.csproj" />
  </ItemGroup>

</Project>

My .NET Standard 2.0 library.
using Microsoft.Azure.CosmosDB.BulkExecutor;
using Microsoft.Azure.Documents;
using Microsoft.Azure.Documents.Client;
using System;
using System.Linq;

namespace NetStandardLibrary
{
    public class Class1
    {
        public static void CreateBulkExecutor()
        {
            var client = new DocumentClient(
                new Uri("https://documentdb.com"),
                "documentkey",
                ConnectionPolicy.Default,
                ConsistencyLevel.Session);
            DocumentCollection documentCollection = client.CreateDocumentCollectionQuery(UriFactory.CreateDatabaseUri("mydatabase"))
                .Where(c => c.Id == "mycollectionid").AsEnumerable().FirstOrDefault();
            var bulkExecutor = new BulkExecutor(client, documentCollection);
        }
    }
}

.NET Standard 2.0 library's csproj.
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Azure.CosmosDB.BulkExecutor" Version="2.3.0-preview2" />
  </ItemGroup>

</Project>

Expected behavior
Should be able to call the BulkExecutor constructor.
Actual behavior
The MissingMethodException was thrown at runtime when I try to step into NetStandardLibrary.Class1.CreateBulkExecutor(); in Visual Studio 2019.
Environment summary
SDK Version:
Microsoft.Azure.CosmosDB.BulkExecutor 2.3.0-preview2.
OS Version (e.g. Windows, Linux, MacOSX):
Windows 10 Enterprise 10.0.18362 Build 18362
Additional context
I have to add nuget package Microsoft.Azure.DocumentDB.Core to my .NET Framework 4.7.2 console app. Else, I will hit System.IO.FileNotFoundException: 'Could not load file or assembly 'Microsoft.Azure.DocumentDB.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file specified.'

I posted this issue in Azure/azure-cosmos-dotnet-v2 repository as well.
https://github.com/Azure/azure-cosmos-dotnet-v2/issues/701
Not sure where is the right place.

I saw this a long time ago.  Double check the version of DocumentDB vs the installed version of Service Fabric SDK.  I was running an old SDK with a newer DocumentDB version.  Updating the SDK fixed my problem.
It would be nice if there was a better error message :)

I saw this a long time ago. Double check the version of DocumentDB vs the installed version of Service Fabric SDK. I was running an old SDK with a newer DocumentDB version. Updating the SDK fixed my problem.
It would be nice if there was a better error message :)

Thanks for trying to help. However in my case I don't have Service Fabric SDK installed though. Are you saying that there is version conflict between Microsoft.Azure.CosmosDB.BulkExecutor 2.3.0-preview2 and some other packages that I am using?

Thanks for trying to help. However in my case I don't have Service Fabric SDK installed though. Are you saying that there is version conflict between Microsoft.Azure.CosmosDB.BulkExecutor 2.3.0-preview2 and some other packages that I am using?

This was just my experience and I was running locally.

Closing as this is not related to Cosmos DB SDK V3, please see linked Cosmos DB V2 SDK issue.


[wrt] Update pyunit TIZEN_USER for default value

Impacted tests(approved): new 0, update 264, delete 0
Unit test platform: [Tizen]
Unit test result summary: pass 264, fail 0, block 0

Because our target will be Tizen Common now, so i just suggest setting default user to "guest". It's only the initial value, we should get final value from testkit-lite anyway.
Do you agree?

@yinhaichao Please update the default value to "app" , i don't have strong opinion about it.

LGTM


HC-SR04 Sensor with micro-ROS on the Raspberry Pi Pico

Connect an HC-SR04 range sensor to a Raspberry Pi Pico and use micro-ROS to post its values to the ROS 2 graph.

The above-provided issue has been completed. Please refer to the file below for a more detailed tutorial on How to connect an HC-SR04 range sensor to a Raspberry Pi Pico and use micro-ROS to post its values to the ROS 2 graph.
HC-SR04 Sensor with micro-ROS on the Raspberry Pi Pico.md


Added Synthical (https://synthical.com)

Hey @mixeden, thanks for the suggestion! I am not quite sure how this fits in the research summary or directions specific to NLP?

It actually creates summaries of papers, see https://synthical.com/article/4d4f86a2-ffaa-11ed-9b54-72eb57fa10b3

Aah, got it. This looks great as a reading experience. We'd love to have a great curation here which is useful to a reasonable number of wide readers, and not machine generated or curated topics.
I still might be missing something or not seeing it, please feel free to re-open if I've misinterpreted something there.


Buspirate improvements: allow changing I2C speed, fix i2c_write

This is great. I haven't been able to test it since i've blown up my buspirate but the change looks fine.


Solo preprocess

I am sorry.. Bad click. Please close this pull request.

 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.radek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @RadekPazderka ！We are grateful for your efforts in helping improve this open-source project during your personal time.
Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have a WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


mongodb connection

Repro steps:

Action: cosmosDB.attachDatabaseAccount
Error type: Error
Error Message: failed to connect to server [<IP_ADDRESS>:27017] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:27017]
Version: 0.7.1
OS: win32

@sandyksit You're attempting to connect to a local mongodb instance.  Please make sure you actually have the local server running.  For instance, on Windows, you would run (changing path as necessary):
"C:\Program Files\MongoDB\Server\3.6\bin\mongod.exe"

in a command prompt window and leave it open (see https://docs.mongodb.com/manual/tutorial/install-mongodb-on-windows/ for more information).
If you already have started MongoDB, then in a different command window try running the MongoDB shell, for instance:
"C:\Program Files\MongoDB\Server\3.6\bin\mongo.exe"

and see if it is able to connect to the MongoDB.exe server.
You can find more information about getting started with MongoDB here: https://docs.mongodb.com/manual/installation/.  Please let us know if this resolves the issue for you.
Alternatively, if you would like to connect to an Azure instance of MongoDB, then you should sign in to Azure and browse to or create an Azure MongoDB database. In that case you don't need to use the "Attach Database Account" command.
thx,
Stephen

I'm closing for now.  We are planning to catch this error and provide better information.  Thanks again for reporting.


Add Apache as the vagrant user

This is a huge security concern on a server, but since this is all local dev it should be okay.
This will allow for things like Wordpress to automatically add files and uploads :)
http://askubuntu.com/questions/97810/how-to-make-apache-run-as-current-user

I used the 'sudo nano ...../envars' solution and it worked!


Configuring Payment Automation: Payment Refunds

https://help.salesforce.com/s/articleView?id=sfdo.NPSP_Configure_Opportunity_Payments.htm&type=5
https://docs.google.com/document/d/1tj37ss_k94ExdInkmP0jaR6MFsihZ7NOkemBeypauSw/edit?usp=sharing

@barbarachristensen I need your expert advice on this one. A permission set is assigned to Juan, so the rest of the video is written as 'Juan does X...' can you take a look at whether this is okay, if not it might need a rewrite, but some help on how we make the transition from giving Juan a perm set, to then it being 'we' would be appreciated :)

@sarikabollampally I think you said you finished review and I just need to move this, but my brain was mush at the end there :) Just let me know where you are in the process, please. Thanks!

@barbarachristensen I reviewed and added my comments to the script. Thanks!


Slack connector: even more debug around non threaded ingestion

Description
Follow-up to https://github.com/dust-tt/dust/pull/6839
Risk
N/A
Deploy Plan

deploy connectors

👍


[language] eliminating usage of tx builtins in epilogue

Moving toward removing all of the tx builtins as opcodes. This commit makes the transaction epilogue builtin-clean.

Motivation
Continuing the line of work in https://github.com/libra/libra/commit/adb891b171cdaa097eedcac1ff7a2cf887e9f5e8.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
Existing uses of epilogue continue to work as before, should be pure refactoring.

:umbrella: The latest upstream changes (presumably 7c74389b0725de6a8b161e78db16b1b7aeecc5b2) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=runtian-zhou

:pushpin: Commit 1962b10 has been approved by runtian-zhou

:hourglass: Testing commit 1962b1030022398a68587799f3f2319523fe6a9e with merge 386666642c2f8a265b275e72f8d36cec4be7af4c...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: runtian-zhou
Pushing 386666642c2f8a265b275e72f8d36cec4be7af4c to master...


hint at common error of non-existing file path

closes https://github.com/bioimage-io/spec-bioimage-io/issues/453


#1591 Sample product list

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
What is the current behavior?
I have added a sample product list.
Issue Number: [#1591]
What is the new behavior?
Newly added products displayed on the homepage
Other information
@nikohoffren  Thanks for giving the opportunity to new contributors.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Updated level prefabs

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description

updated small and large level prefabs
removed desert and desert cube areas
moved walls to be at edge of the standard ground areas
fixed spawn point in tile 12

Tests
works in editor
Documentation
none
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Merged as part of:
https://github.com/spatialos/gdk-for-unity-fps-starter-project/pull/9


Add custom labels to ServiceMonitor

Allow customization of labels on ServiceMonitor object in Helm chart

Done in #34


Cant join channel

Describe the bug
When i join a channel with a temp token or token provided by my server
Despite having a valid token i get
( 6350): Access denied finding property "net.dns1"

I had the same issue, sometimes is working fine but in other attempts just don't let join to the channels. It's a weird case. :(

you can get the error code from the Error event.

you can get the error code from the Error event.

ErrorCode.JoinChannelRejected.
I've readed the docs about that enum. But is jut called one time. About the test I don't know what means. 😬

have you joined the channel already?

pls provide the sdk log.

Agora is working fine but sometimes that error occurs. Is weird. IDK. Let me do it again and if happens again I'll tell you. Thanks dude.

same issue with me


Geolocator.getPositionStream steals control of position update configuration.

I believe this is not maybe an issue but more like architectural decision on the part of the author.
By creating its own instance of position stream via Geolocator.getPositionStream, the plugin basically removes ability for developer to change configuration of the stream.
Please see https://github.com/Baseflow/flutter-geolocator/issues/939#issuecomment-101199093 for issues related to this.

unfortunately the problem here is with the native implementation on
the platforms (specifically on iOS and macOS) which will cancel any existing
location stream when you open another. Meaning it is impossible to listen to
multiple location streams (within the sandbox of the App) at the same time.

So it means whoever makes getPositionStream call first wins since the cached stream is returned. You could say that I could create instance of the plugin only after I created my own getPositionStream which is fine. However if I want to change the stream configuration (by providing distanceFilter for example), first all listeners have to be canceled. That is not possible since the plugin doesn't expose the stream subscription (_positionStreamSubscription).
I think one solution would be to create optional position argument to the plugin. Whenever the position would be changed, the plugin could detect it in didUpdateWidget method and move the pin on the map appropriately. This would allow me to retain full control over position stream and use the plugin basically only as a presentational widget with some extra features (like camera centering).
If position argument would be present, the plugin wouldn't register listener to Geolocator.getPositionStream. The default behaviour could stay the same for people who want to use the plugin easily.

I ended up creating a PR which adds optional argument of Position type. It avoids the problem of multiple position streams thus you can safely use Geolocator within your own code.
There's an example page in the sample application where I demonstrate how you can change the stream during runtime and how it affects the rendering of the marker while still allowing to listen to the changes in your own app with the ability to recreate the stream any time.

I just uploaded version 3.0.0 which can accept streams from application as the source. Hope that solve this problem.

I just uploaded version 3.0.0 which can accept streams from application as the source. Hope that solve this problem.

Yes this should fix #15 and #18 for people on this new version but not on v2. For v2 you could accept https://github.com/tlserver/flutter_map_location_marker/pull/16 which is backwards compatible and only extends the functionality.
My opinion is that the positionStream could be Stream<Position>, not sure why you're creating your own class LocationMarkerPosition, it's kind of redundant[^1] in my opinion. But that's just a nitpick, I'm glad you improved ergonomics
[^1]: Since the plugin depends on geolocator anyway, you could have used Position class. I just apply stream transformer that converts between the two and it seems like extra step. I know there are ppl who might not even use geolocator in which case it'd be better for the plugin to not rely on it at all.

If I expose the streams to API, I think it should accept data from other plugin. It may confuse people if I still using Position in geolocator or CompassEvent in flutter_compass, because it is too many unused field. There is always a class LocationMarkerDataStreamFactory to help converting. I am considering to split the class LocationMarkerDataStreamFactory to separate package to make this does not depend on geolocator anymore. So that, the only widget of this package will be a presentation widget only.

Come from #14, Would you please explain more about the different of this issue & #13? I think #14 should solve this but seem it does not.
If I understand correctly, you want to change the LocationSettings while you have your own geolocator's position stream outside this package. Assume #14 is applied, is it possible to first cancel all the geolocator's position stream outside the package, then change the locationSettings in LocationMarkerPlugin and subscript again to the new position stream outside the package?
I just want to make the issue clear. You say canceling all listeners of the position stream is not possible. Isn't that #14 create a chance to re-create the position stream by replacing the locationSettings in LocationMarkerPlugin?
Sorry for not accepting #16 yet and thank a lot for helping improving this package.

On Wed Feb 2, 2022 at 12:32 AM CET, 6y wrote:

If I expose the streams to API, I think it should accept data from other plugin. It may confuse people if I still using Position in geolocator or CompassEvent in flutter_compass, because it is too many unused field. There is always a class LocationMarkerDataStreamFactory to help converting. I am considering to split the class LocationMarkerDataStreamFactory to separate package to make this does not depend on geolocator anymore. So that, the only widget of this package will be a presentation widget only.

Yeah that sounds good!

If I understand correctly, you want to change the LocationSettings while you have your own geolocator's position stream outside this package.

Yes

Assume #14 is applied, is it possible to first cancel all the geolocator's position stream outside the package, then change the locationSettings in LocationMarkerPlugin and subscribe again to the new position stream outside the package?

I don't think that's the way it works. Geolocator.getPositionStream always returns single position stream, that's a decision the authors of the plugin made to make it consistent between platforms.
Even if I cancel the stream in my app, your plugin is still using it, I think it'd have to be other way around, cancel stream in the plugin and then recreating it in application. When I tried doing what you proposed it did not work for me.
It doesn't matter now that much though, with v3 you added the ability to provide stream from outside so it fixes this problem.

Ok, then I close this


Where is the random background color set?

and how do i stop it / make it white!!??

method setup() in CardAnimation class
replace this code:
Random rnd = new Random();
int color = Color.argb(255, rnd.nextInt(256), rnd.nextInt(256), rnd.nextInt(256));
 v.setBackgroundColor(color);

with this:
v.setBackgroundColor(Color.WHITE);

As a work around, you can put a white background view in your custom layout.


MaxElementsTilln'th.cpp Added

Read the Contributor's Guide. It says you have to update the README with your program.


Create abstraction to clean up logic relation to canncelation channels

blocked by #976

Please remember about assigning team labels @tkumor3


EventStore.RecordedEvent does not implement the Access behaviour

(EventStore.RecordedEvent does not implement the Access behaviour)
when i use get_in to access fields.
https://github.com/planswell/kitchen-sink/blob/master/lib/kitchen_sink/struct_access.ex
here is an example of how to implement this...
would you be interested in a PR?

@boxxxie Does that require just adding fetch/2 and get_and_update/3 functions? Then happy to take a pull request to add.

yes, it does.
ok. i'll give you a PR soon.

Fixed by #37.


Roll bar seems to be logging Delayed Job errors twice

Hi,
We are using delayed job with a max retry of 5 after which the delayed job is marked as failed. When the job errors for the 5th time the exception is reported twice once in the injoke_job hook and once in the failure hook. https://github.com/rollbar/rollbar-gem/blob/ffd4278332cc63b6dc29cde5fc3716793c455bec/lib/rollbar/plugins/delayed_job/plugin.rb#L13-L15
Is there are way to disable error reporting on failed jobs, as this second report is not helpful to us?
Thanks
Rob

Just noticed the ::Rollbar.configuration.report_dj_data config setting, ignore me.
Thanks

We'll take a look into this.

Will address in the upcoming release

@RobWillison this can be achieved by setting
config.dj_threshold = 6

in your config/initializers/rollbar.rb
This way Rollbar will only report the error on :failure hook and not report on :invoke_job hook.

Is there a way to set the threshold on a per job class basis or per queue?  For example, some jobs we would want the error reported right away and others we would want to set a higher threshold.

Hi @drewB, can you open a new issue for this, and describe your use case? This will also make it easier for others to comment on the proposed change.
Thank you!


[Core] refactor package/module structure before release

bagri-xdm -> bagri-core
get rid of .xdm. packages

done


Hotfix migrations

Add migrations to revision control

Coverage remained the same at 60.759% when pulling f2b1e7859ecb194707f1acd6e2d9c773c160cbe0 on hotfix_waitress into ffcaa1a1d0311a2d1a22c36e2b0962f28d8bbc7f on master.


Snyk-fix-a94a41f571dfc80b04a19ba6fb660c75

The following vulnerabilities are fixed by pinning transitive dependencies:

https://snyk.io/vuln/SNYK-PYTHON-DNSPYTHON-6241713

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Oh interesting looks like Snyk is working with our current pip-compile-multi where dependabot isn't.
Good hint, now I'm wondering if we can set it up to watch our repo and submit PR the way dependabot is supposed to. Maybe they have a "free for open source project" program.

Oh you'll want to rebase as the base seems old and has conflicts.

@matrix-compute to address any security vulnerability you'll have to update the bounds in setup.py.


Expected failures

I have a situation where I expect an update to fail and I'd like to assert that it does. Unfortunately errors are always treated as fatal:
https://github.com/pulumi/providertest/blob/385bd7f4a916c5134093d99b138992ea9b10eaf6/pulumitest/up.go#L19-L22
It would be really nice if we returned result, err here instead, although that would be a breaking change.
We should at least be able to have an option ExpectFailure and then use that to inform the t.fatal? Although that will need to update the auto API's options.
// Ideal
up, err := test.Up()
assert.Error(t, err)

// Also OK
up := test.Up(optup.ExpectFailure())

You need to call test.CurrentStack().Up() - pulumitest just wraps the automation API and asserts the failure didn't happen here.

Suggestion: add an ExpectFailure flag to the Up options, and fail accordingly.

Returning result, err would reduce the clarity of tests as they'd have to now contain error handling for every step. This would be equivilent to just using the AutomationAPI directly without the test framework.
Another option instead of disabling the assert would be to present an alternative method for operations we expect might fail - perhaps something like upResult, err := test.TryUp(). I'm not sure if Try... is a usual convention in Go, perhaps there's another ideom that would do the same job.


Trust referee email

In gitlab by @A-Lawrence on Aug 20, 2016, 19:28
When contacting a referee, trust the given email address.
This is particularly important for those accounts that we don't yet have on file

In gitlab by @A-Lawrence on Aug 20, 2016, 19:29
mentioned in issue #449

In gitlab by @A-Lawrence on Aug 20, 2016, 19:35
Removed ~98471 label

In gitlab by @A-Lawrence on Aug 20, 2016, 23:27
mentioned in commit 834c1e83457e482b78ee835938de077c891bf6a6


Panstwowa Uczelnia Zawodowa im. Ignacego Moscickiego w Ciechanowie

Add upper-level domain: Panstwowa Uczelnia Zawodowa im. Ignacego Moscickiego w Ciechanowie

@pmlodzianowski I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host puzim.puzim.edu.pl not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Feature/edit mode

Hi,
this is rather simple: the edit mode just hides the input and the delete button for a "read only" view of the tags.
I would update the docs on gh-pages as soon as you give me the ok ;)

Coverage remained the same at 100.0% when pulling 6cda153dccb130238183ca46b384432519a15cf8 on urecio:feature/edit-mode into 214b84b8b4ba0905ab53f54a35ec382f05492b76 on mbenford:master.


Have the iPad-Toolbar have the navigationBars color

Add the navigationBars tintColor to the newly created UIToolbar as
well.

This is a little troublesome because depending on your tint, the toolbar icons might not show up.  There isn't a mechanism in place that will colorize the toolbar button images to contrast with your toolbar tint.  However, the UIColor-Expanded repo can do contrasting/complimentary colors, and UIImage+OverlayColor will colorize the icon images for you.


Only working in Edge & Chrome

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Windows 10
@angular/cli: 1.0.0
node: 6.10.0
os: win32 x64
@angular/animations: 4.0.0
@angular/common: 4.0.0
@angular/compiler: 4.0.0
@angular/compiler-cli: 4.0.0
@angular/core: 4.0.0
@angular/forms: 4.0.0
@angular/http: 4.0.0
@angular/platform-browser: 4.0.0
@angular/platform-browser-dynamic: 4.0.0
@angular/platform-server: 4.0.0
@angular/router: 4.0.0
@angular/cli: 1.0.0
Repro steps.
When building my project with ng build --prod it only works in edge & chrome.
The log given by the failure.
The are no errors in console, the site just doesn't load
Desired functionality.
App to load in all browsers on supported list.
Mention any other details that might be useful.
I am using the web animation api so I have run npm install --save web-animations-js and also uncommented import 'web-animations-js'; in polyfills.ts.
App works fine in latest chrome and edge but I tried in IE11 and it doesn't work.  Just a blank screen with background colour

Additional polyfills are required for IE9, IE10 and IE11.  Try uncommenting the relevant lines in polyfills.ts.

Thanks, I had missed them.

@GuerrillaCoder: I'm having the same symptoms but for all browsers on iOS + Safari on MacOS (works on Firefox & Chrome). Have you by any chance tested your app on those configurations? I'm asking by despair: I have spent 6 hours already on this and you're the only one I found yet with similar symptoms.

@guenam I am not on mac so not sure.  You can check my url though: https://moboz.co.uk

I just tested your website on my iPhone 4s (old... safari 9): everything seems fine. At least my website passed your mobile tests! ;-)
Thanks for your super quick reply!

@guenam Did you uncomment everything in polyfils.ts?  My issue was that there was some individual modules I hadn't uncommented.

Yes, I did and even tried adding others... Nothing is the log makes it super frustrating. Thanks again.

@guenam all I can suggest is to make a new default project and then test it.  Then re-add elements one by one until it breaks.  Super annoying but it should isolate the issue.

@GuerrillaCoder I did that and found out that an "url slugifier" package called limax was the source of the issue. Quite a good package though, I'll henceforth use it only on server side.
Thanks for your good counsel!


Statistic update

Separate Statistic tab from the Connection tab.
Automatically change the Monitoring page to the Statistic tab on connection and to the Connection tab on disconnection.

bugfix on the graph tab

Works well, thanks!

Works well, thanks!


1.55.0 - Incorrect behavior for gotoLocation.multipleDeclarations

VS Code Version:

Version: 1.55.0
Commit: c185983a683d14c396952dd432459097bc7f757f
Date: 2021-03-30T16:01:05.981Z (6 days ago)
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.3.0

Version: 1.56.0-insider (Universal)
Commit:<PHONE_NUMBER>07c15252e581e8abe1b820b9d311ee
Date: 2021-04-02T05:13:17.592Z (4 days ago)
Electron: 11.4.1
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.3.0

OS Version:
Big Sur 11.2.3

Does this issue occur when all extensions are disabled?: Yes

Hello,
with editor.gotoLocation.multipleDeclarations sets to "goto"
In a React/Typescript project, prior to 1.55.0, when you ⌘-click a Tag, you were sent to the component, now, you are sent to the type defintion of React. But if you ⌘-click on the closing tag, you are sent to the correct component.
You can see the problem here

Duplicate of #121003. If you still see this in VS code insiders, please share a self contained example that demonstrates the issue (as text)


Resolve containerdisk after initialization in template tests

What this PR does / why we need it:
BeforeEach got the containerdisk as parameter before commandline
arguments were parsed. Therefore the tag of the containerdisk was empty
in the template task.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Fixes issues like this in some test environments:
Failed to apply default image tag \"/fedora-cloud-container-disk-demo:\": couldn't parse image reference \"/fedora-cloud-container-disk-demo:\": invalid reference format"

Release note:
NONE

/cc @enp0s3

/cc @acardace

nice work!
/lgtm

/cherrypick release-0.41


Internal Server Error when registering the reMarkable in OneNote

Hello, I have followed the instructions for installing and configuring the OneNote reMarkable Plugin, but when registering with reMarkable I am getting an error. This is wat the log says:
23-1-2023 19:57:01	AddIn.GetCustomUI(76) - called
23-1-2023 19:57:16	SettingsForm.setRmConnectionMethod(162) - Setting connection method to RmCloud
23-1-2023 19:57:23	RmCloudDataSource.RegisterWithOneTimeCode(70) - registring with code: cxifedmh
23-1-2023 19:57:23	RmCloudDataSource.Request(173) - url is: https://webapp-production-dot-remarkable-production.appspot.com/token/json/2/device/new
23-1-2023 19:57:23	RmCloudDataSource.RegisterWithOneTimeCode(86) - response code: InternalServerError
Using Enterprise Apps for Business, Windows 10 and a 64-bit version of OneNote

Hi, I ran into same issue just now


[FEATURE]  Add used stats for cpu/mem/storage

Is your feature request related to a problem? Please describe.

Add used stats charts to the dashboard home page and node details page

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Add used stats charts to the dashboard home page and node details page

[x] dashboard home page
[ ] node details page

Test Information

Environment: qemu/KVM 3 nodes
Harvester Version: master-a67f96d0-head
ui-source Option: Auto

Verify Steps:

Install Harvester with any nodes
Login to dashboard
Capacity should display Reserved and Used of CPU/Memory/Storage
Navigate to Hosts then click any node
Monitor Data should display Reserved and Used of CPU/Memory/Storage

Tested in master-1a861fc1-head
Verification steps

Created new VM
Checked dashboard home page

Checked node details page

Added large file to VM. Verified that storage usage updated in node details and dashboard home

Notes
It's worth noting that the host list page doesn't show usage. However this wasn't in scope for this ticket so I'm closing it.


Missing state field for block actions payload

Hi again! I'm working with block actions and it seems like the state field is missing from SlackInteractionBlockActionsEvent. It's documented here: https://api.slack.com/reference/interaction-payloads/block-actions#fields
Thanks! :)

Hey, keep them coming if you find more :)
This should work now after this PR
https://github.com/abdolence/slack-morphism-rust/pull/173
Available on the master branch, let me know if it doesn't work for you.

Released in v1.6.1


Dammmmm this is when internet kids create fork project for engagement. Anyway good luck with your project Cuz it not going to be easy because developing a game is enough mental break down for devs😂😂.


Add support for the graphql-transport-ws GraphQL over WebSocket Protocol

Currently our ASGI and AIOHTTP integrations only support the "legacy" graphql-ws websocket subprotocol. Its repository states:

The subscriptions-transport-ws library is not being actively maintained. It is recommended that you use the graphql-ws library instead. For details read the GraphQL over WebSockets announcement.

Just for clarification: the legacy graphql-ws websocket subprotocols library is called "subscriptions-transport-ws", while the newer graphql-transport-wswebsocket subprotocols library is called "graphql-ws". Yep, slightly confusing.
According to NPM download stats the legacy library is still downloaded more often than the new library. Since both protocols are incompatible I propose we support both for now and give users a choice to disable them individually.
I already implemented the new protocol for our AIOHTTP integration, which went very well since the new protocol is way better documented than the old one. I'm currently working on tests and a good abstraction that allows us to reuse the code for the ASGI integration. I'll finish work on this next week.

we use graphql-transport-ws in combination with starlette & django - works really nice. I'm not sure if supporting the old library/spec makes sense, because the backend code is so much different and there is not much in common.
I think many downloads on the old library are coming from the very first tutorials with subscriptions (which ar quite popular).
A good subscription example with the new protocol in the strawberry documentation would make much more sense IMO.

@Speedy1991 out of curiosity why are you using starlette instead of Django channels? I'm assuming you're just using starlette for subscriptions only?

@jkimbo sorry for the late reply:
I use Django3 async stuff with uvicorn and an starlette app
Subscriptions just works ootb with starlette, there is no more need for django channels with async support

@jkimbo sorry for the late reply:
I use Django3 async stuff with uvicorn and an starlette app
Subscriptions just works ootb with starlette, there is no more need for django channels because of django3 async support

Interesting thanks @Speedy1991 . So you're not using Django views at all?

well maybe this was missleading:
My stack looks simelar to this:
Uvicorn -> starlette -> Handle some routes directly with starlette (e.g. fast async views which don't need the whole django boilerplate middleware, orm stuff) and yes also delivering the websockets (handling django.setup myself and mount some websocket middlewares)
Everything not matching a route in starlette is forwarded into the django app (fully async) because django will handle sync/async requests itself with the django.core.handlers.asgi import ASGIHandler - obviously you have to handle ratelimits or your django server will be easy to ddos :)
I think this is going out of scope of this issue - if you are interested in this setup we can talk in discord?

I should also mention with starlette you already get an working graphql example + endpoint with starlette

Thats a really interesting setup, thanks @Speedy1991


fix(throttleTime): leading and trailing true should only emit a single value per time frame

Fixes https://github.com/ReactiveX/rxjs/issues/2727
fix 1: throttleTime {leading: true, trailing: true}
Currently it behaves like
'a1234b1234c1234d12e--|'
'a----b1----12----2e--|'

By not scheduling a new throttle when emitting a trailing value.
this PR changes it to:
'a1234b1234c1234d12e--|'
'a----b----c----d----e|'

fix 2: throttleTime {leading: false, trailing: true}
Currently it behaves like
'a-------------a-|'
'----------------|'

First and only value in each timeframe is dropped by not storing the first value when not yet throttled. So this code doesn't console.log anything:
Rx.Observable.of(1)
    .throttleTime(100, undefined, {leading: false, trailing: true})
    .subscribe((value) => console.log(value))

This PR changes the behavior to:
'a-------------a-|'
'-----a-------------a|'

I added tests to cover those issues.

Coverage increased (+0.001%) to 97.735% when pulling 74faac5e0251f3f3f94bbc53aef8e9cabbf4d1ad on sod:throttle-time-single-emit into 69058531f8ec8ca0fd09c2e3872193e9d6c86723 on ReactiveX:master.

@benlesh it's a followup to https://github.com/ReactiveX/rxjs/pull/2465 ... what do you think? :)

I've been testing this, and the only hiccup is that the throttle call should come before the .next() emission because otherwise if the .next() is resolved synchronously it can really mess up the timing.
I suggest changing _next() to have something like:
let emit = false;
if (!this.throttled && this.leading) {
    emit = true;
} else ...

if (!this.throttled) this.throttle();
if (emit) this.destination.next(value);

And also clearThrottle to something like:
if (this.trailing && this._hasTrailingValue) {
    let value = this._trailingValue;
    this._trailingValue = null;
    this._hasTrailingValue = false;
    this.throttle();
    this.destination.next(value);
}

I have also recommended that this PR be replicated to solve #2466 but considering what I have discovered regarding the timing this may not be so simple.

How did you test the hiccup? next is not being called when a throttle is present, so I can't see how that happens.
if (!this.throttled && this.leading) {
    this.destination.next(value);
} /* ...

Later in the same function is the call to this.throttle().  If next() takes 20ms to complete and our throttle is supposed to be 100ms then it will actually be 120ms between emissions because the setting of throttle doesn't start until after next completes.

Thanks, now I got it. I think just adding a second this.throttle() should be sufficient
if (!this.throttled && this.leading) {
    this.throttle(); // <---
    this.destination.next(value);
} /* ...

Thanks, now I understood the issue.

rebased & added your feedback

@sod can you look at what I've done with throttle in #2864 and try to mirror it as closely as possible to keep the code base consistent?


feat: add '.will-change-x' utility class

This adds the .will-change-x classes.
References

https://tailwindcss.com/docs/will-change

Codecov Report

Merging #608 (e967fd2) into main (14edb4c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #608   +/-   ##
=======================================
  Coverage   88.15%   88.16%
=======================================
  Files          56       56
  Lines        4839     4840    +1
  Branches     1034     1034
=======================================
+ Hits         4266     4267    +1
  Misses        310      310
  Partials      263      263

Impacted Files
Coverage Δ

src/lib/utilities/static.ts
100.00% <ø> (ø)

src/utils/completions.ts
100.00% <ø> (ø)

src/config/order.ts
100.00% <100.00%> (ø)

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 14edb4c...e967fd2. Read the comment docs.


docs: Interactive docs for the ColorEffect

Description
added Interactive docs for the ColorEffect
Checklist

[x] The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
[x] I have followed the [Contributor Guide] when preparing my PR.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples or docs.

Breaking Change?

[x] No, this PR is not a breaking change.

Related Issues
Closes #1958
https://user-images.githubusercontent.com/36990991/197996638-86198cd0-a4a1-4b0d-826c-b05b525a5935.mov

did the force push and it closed the pull request automatically 🙆


Issue #216 inline editor

Fixes #216
Closes #201

A nice convention I've used in another projects is to include a screenshot when the changes involves the UI, because many of us don't have the ability of rendering HTML in our heads. :stuck_out_tongue:
(ok, I could checkout your branch locally... but that decreases my will of looking at your PR)

@faloi please check the discussion in the issue. I put the screeshot there

Do you have an example of the markdown syntax?

Yes:

tablero inicial

tablero final

Great, I've used that before... always with the help of http://www.tablesgenerator.com/markdown_tables#


Allow correctly replaying Events

refactored returnDeadToQueue to allow correctly returning Events as well as Messages(commands)

Coverage decreased (-0.2%) to 76.952% when pulling 8f9bf82e52549e5642891f88c89a33203119bba0 on yuvmendel:return_dead_to_queue_with_routing_key into 660ff9a7db6fa464103f5611f91933d26ae131ad on wework:v1.x.

@yuvmendel  your PR is failing the linting check. please run golang-cli on the project and fix the issues

#191

@vladshub @adiweiss @danielwitz adding you as well to this PR so you will know what this is about and get a chance to comment.

@rhinof PR is ready for your re-review - I want to make sure you noticed a change I made in the previous commit after discussing with Witz a little.
If you remember we discussed the x-death list, and I originally decided to use the routing-key of the "latest" death. I realised this might not always be correct, as the other params we use are of the "earliest" death, meaning we use rabbit's own first-death-exchange and first-death-queue so if use them with a routing-key we should use the earliest routing key as well.
I could have gone the other way and take the exchange and queue from the "latest" death as well, but I think the assumption that when replaying we are probably replaying to the queue the message first got rejected at makes more sense.
This discussion doesnt really matter currently as our setup never creates messages with more than one death in the x-death list as we reject once to the DLQ and any replay first clears the x-death list, we never truly reject the same message twice adding on another death. However, I think for grabbit as a framework this discussion matters and we should make sure we are okay with this assumption.


Fix typos in comments for clarity and grammatical correctness

Changes:

In frontend/builder.go:

Fixed the grammatical error in the comment for the MarkBoolean method. The original comment was:

"MarkBoolean sets (but do not constraint!) v to be boolean"

Corrected to:

"MarkBoolean sets (but does not constrain!) v to be boolean"

Explanation: The word "constraint" was used incorrectly. The verb should be "constrain" as it follows the auxiliary verb "does" which requires the infinitive form.

In frontend/variable.go:

Fixed the plural form of "builder" in the comment for the IsCanonical function. The original comment was:

"by one of the constraint system builder."

Corrected to:

"by one of the constraint system builders."

Explanation: The sentence refers to multiple potential builders, so the correct form is "builders" (plural).

These changes improve the grammatical accuracy and clarity of the comments, ensuring the documentation aligns with standard English usage.

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

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


Empty context in kwargs (SAM)

Hi,
I'm trying to run the SAM backend provided in https://github.com/heartexlabs/label-studio-ml-backend/tree/master/label_studio_ml/examples/segment_anything_model
I'm using the Docker version which builds fine, but when I try to run a prediction it fails with:
the kwargs are {'login': None, 'password': None, 'context': None}
the tasks are [{'id': 451102, 'data': {'image': 's3://....jpg', 'ref_id': 3602}, 'meta': {}, 'created_at': '2023-06-20T16:29:12.201416Z', 'updated_at': '2023-06-20T16:29:12.201442Z', 'is_labeled': False, 'overlap': 1, 'inner_id': 2, 'total_annotations': 0, 'cancelled_annotations': 0, 'total_predictions': 0, 'comment_count': 0, 'unresolved_comment_count': 0, 'last_comment_updated_at': None, 'project': 218, 'updated_by': None, 'file_upload': None, 'comment_authors': [], 'annotations': [], 'predictions': []}]
[2023-06-27 14:13:42,314] [ERROR] [label_studio_ml.exceptions::exception_f::53] Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/exceptions.py", line 39, in exception_f
return f(*args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/api.py", line 51, in _predict
predictions, model = _manager.predict(tasks, project, label_config, force_reload, try_fetch, **params)
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/model.py", line 617, in predict
predictions = cls._current_model.model.predict(tasks, **kwargs)
File "/app/segment_anything_model.py", line 70, in predict
height = kwargs['context']['result'][0]['original_height']
TypeError: 'NoneType' object is not subscriptable
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/exceptions.py", line 39, in exception_f
return f(*args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/api.py", line 51, in _predict
predictions, model = _manager.predict(tasks, project, label_config, force_reload, try_fetch, **params)
File "/usr/local/lib/python3.8/site-packages/label_studio_ml/model.py", line 617, in predict
predictions = cls._current_model.model.predict(tasks, **kwargs)
File "/app/segment_anything_model.py", line 70, in predict
height = kwargs['context']['result'][0]['original_height']
TypeError: 'NoneType' object is not subscriptable

Seems like kwargs are empty, how may I fix that ?
Thanks,
Robin

Bump same issue. This seems to be a persistent issue with label-studio 1.8.0

I'm facing the same issue with 1.7.3 as well, the context object in kwargs is None

Encounter the same problem for label studio versions 1.6.0, 1.7.3, 1.8.0
It seems it's sever side (label studio) problem:
Request JSON:
{
 'tasks': [{'id': 128, 'data': {'image': '/data/local-files/?d=Documents/label_studio/dataset/coco128/000000000650.jpg'}, 'meta': {}, 'created_at': '2023-07-08T11:15:13.167604Z', 'updated_at': '2023-07-08T11:15:13.167616Z', 'is_labeled': False, 'overlap': 1, 'inner_id': 128, 'total_annotations': 0, 'cancelled_annotations': 0, 'total_predictions': 0, 'comment_count': 0, 'unresolved_comment_count': 0, 'last_comment_updated_at': None, 'project': 1, 'updated_by': None, 'file_upload': None, 'comment_authors': [], 'annotations': [], 'predictions': []}],
 'model_version': 'INITIAL',
 'project': '1.1688814855',
 'label_config': '<View>\n  <Image name="image" value="$image" zoom="true"/>\n  <BrushLabels name="tag" toName="image">\n  \t<Label value="Banana" background="#FF0000"/>\n  \t<Label value="Orange" background="#0d14d3"/>\n  </BrushLabels>\n  <KeyPointLabels name="tag2" toName="image">\n    <Label value="Banana" smart="true" background="#000000" showInline="true"/>\n    <Label value="Orange" smart="true" background="#000000" showInline="true"/>\n    <Label value="Orange Eraser" smart="true" background="#000000" showInline="true"/>\n  </KeyPointLabels>\n</View>',
 'params': {'login': None, 'password': None, 'context': None}
}

had the same issue.
Also tested it without docker. Getting the same error:
    height = kwargs['context']['result'][0]['original_height']
             ~~~~~~~~~~~~~~~~~^^^^^^^^^^
TypeError: 'NoneType' object is not subscriptable

used label studio 1.7.1

Hey all (@DaMuBo, @feicccccccc @sriramsk1999, @jackmead515, @RobinFrcd) 👋🏻
We discussed this issue during our issue triage today and are flagging this for further inspection and review.
Thank you all for your patience on this — I'll keep y'all posted on what we find.
[ see also https://github.com/heartexlabs/label-studio/issues/4493 ]

I have the same problem!!

@shondle as the developer of the SAM feature, have you seen this issue in your own testing, and do you have a workaround?

Hi everyone, the video tutorial is included in the pull request above. Again, I have been able to replicate this without issue in varying environments, so please follow the video after the installation instructions to see if this works for you.
If it doesn't, please send a video in this issue showing your settings in the Machine Learning part of Label Studio, how you are getting a prediction with the tool in Label Studio, and the output in your ML backend terminal.
I realize this video won't address all issues, but I hope it helps a good amount of you.

@shondle thanks a lot. I tried it with your fork and with the mobileSAM it works like a charm now.
Even without my GPU attached. So great work thanks 👍🏻

Hi all!
Thanks @shondle for the great adaptation and @erinmikailstaples for maintaining the framework!
I think there's gonna be a lot more people discovering label-studio through SAM's adaptation, so I believe fixing this thread bug is really important.
At the moment I was trying to change a few things in the framework to get the ML backend started, but I believe there might be a version mismatch all over the examples?
The one that I bumped into first was the init_app signature in the api.py vs the ones in used in the examples/*/_wsgi.py  files.
Checking now #304 to see if that fixes the issue :) I'm sure I might be just missing something very basic stuff, let me know if I looked over something.
Thnks,
Csabi

I was also able to get it working without problems. Was most likely facing some environment issues. I reinstalled everything in a fresh env and it worked like a charm. Thanks for the nice work on this!

i also has this problems. Is there any new solution?

@shondle thanks a lot. I tried it with your fork and with the mobileSAM it works like a charm now.
Even without my GPU attached. So great work thanks 👍🏻

did you run label-studio without EMPTY CONTXTX?

no solution


drivers: eth: e1000: Remove unused variable

The probe function had unused variable which caused compile warning.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #10612 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10612      +/-   ##
==========================================
+ Coverage   53.34%   53.37%   +0.02%
==========================================
  Files         211      211
  Lines       25828    25828
  Branches     5686     5686
==========================================
+ Hits        13779    13786       +7
+ Misses       9734     9727       -7
  Partials     2315     2315

Impacted Files
Coverage Δ

subsys/logging/log_output.c
85.45% <0%> (+1.21%)
:arrow_up:

subsys/logging/log_core.c
78.4% <0%> (+2.34%)
:arrow_up:

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 92efb09...7df62d9. Read the comment docs.


fix(tasks/ast_codegen): run cargo fmt synchronously

relates #4152

#4181  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


ClickHouse function IF treats NULL values in different way than expected

Hi Team,
Could you please clarify why ClickHouse function IF treats NULL values in different way comparing with CASE?

NULL values are enabled on CLickHouse server.
When we compare NULL value with something in CASE statement NULL with anything produces FALSE condition.
When we compare NULL value with something in IF function NULL with anything produces NULL, not a FALSE result.

Please check SQL below:
select if(1=1,    'A', 'B')                   as if_a      -- A
     , if(1=2,    'A', 'B')                   as if_b      -- B
     , if(null=1, 'A', 'B')                   as if_null   -- NULL
     , case when 1=1    then 'A' else 'B' end as case_a    -- A
     , case when 1=2    then 'A' else 'B' end as case_b    -- B
     , case when null=1 then 'A' else 'B' end as case_null -- B
     , 'dumb text'

Could you please clarify is it a bug?
Best regards,
Kostyantyn

Fixed in https://github.com/yandex/ClickHouse/pull/5238


Syntax highlighting override

Hello,
I  have tried to override the $basexx variables but it seems they are not included in the main.css file.
I have put them in the assets/css/main.scss before the imports but it does not work. If I put $link-color it works in the sense that I can override the color of the links.
Am I doing something wrong? Any hint?
Thanks a lot for your time.

I just tested creating /assets/css/main.scss and added the base16 variables with different colors before the other Sass imports and the colors changed for me.
Example:
---
# Only the main Sass file needs front matter (the dashes are enough)
---

@charset "utf-8";

$base00: #1d1f21;
$base01: #282a2e;
$base02: #373b41;
$base03: #969896;
$base04: #b4b7b4;
$base05: #c5c8c6;
$base06: #e0e0e0;
$base07: #ffffff;
$base08: #cc6666;
$base09: #de935f;
$base0a: #f0c674;
$base0b: #b5bd68;
$base0c: #8abeb7;
$base0d: #81a2be;
$base0e: #b294bb;
$base0f: #a3685a;

@import "minimal-mistakes/skins/{{ site.minimal_mistakes_skin | default: 'default' }}"; // skin
@import "minimal-mistakes"; // main partials

I will try again then.
Thanks for the reply.

My error, sorry, it works like a charm.

I'd like to obtain a syntax highlighting identical to either GitHub (as we see if I put code blocks right here) or Stack Overflow. How can I identify exactly what color values to set for each of my $base settings above to achieve this?

@ElectricRCAircraftGuy The colors map to the styling in this Sass partial: https://github.com/mmistakes/minimal-mistakes/blob/master/_sass/minimal-mistakes/_syntax.scss
You'll have to determine how they map, but there are plenty of base16 themes out in the wild.


Update the SFUI version in Capybara

What is the motivation for adding/enhancing this feature?

Currently, we're using 0.7.11 version of SFUI, and there's a lot of new elements and bug fixes introduced in the latest version
What are the acceptance criteria?

On upgrading the SFUI version we need to fix many small UI bugs which will be there due to change in the library

[ ] ...

Can you complete this feature request by yourself?

[X] YES
[ ] NO

Additional information

As per the discussion with @Fifciu and also in the community

Mega Menu

Home page main carousel

Home page product carousel

Share your look section

Category page navbar

Product page main image

Form in checkout section

Review section in checkout process

Footer

Here is link for full overview after upgrading

Is Capybara a dead project? should we expect any updates further on?

This theme is such a pain to work with. The documentation is not nearly as concise as it could be and the dependencies are pretty much outdated to a point where this theme is almost unusable. Calling it "prodution ready" unfortunately seems not to be possible at its current state :(

Hey @ke-real , definitely its not!
The above screenshots are after we update a specific dependency(SFUI), on which capybara heavily relies.
This issue if for upgrading a dependency.
There will be more updates and stable releases.
In case you're looking for some specific fix, you may find it in the pull requests and if not, feel free to create an issue around it.

Closing since we will jump to 0.11 which is the latest version. Will make a new issue for that.


[Bug]: Doesn't display base64 image on iOS

What happened?
Doesn't display animation where main image contains in base64 on iOS. But on Android it's work.
What Compottie version do you experience the bug on?
1.1.2
What platforms are you seeing the problem on?
iOS
Does Compottie log anythyng to the stdout / your logger set with Compottie.logger?
No response

Images are supported starting with Compottie 2.0


Slightly unintuitive eqx.nn.Sequential

My first intuition when working with eqx.nn.Sequential (e.g. for a simple MLP) is like below:
import jax
import equinox as eqx

key = jax.random.PRNGKey(0)
key1, key2, key = jax.random.split(key, 3)
module = eqx.nn.Sequential([
    eqx.nn.Linear(5, 10, key=key1),
    jax.nn.log_sigmoid,
    eqx.nn.Linear(10, 2, key=key2),
])
keyx, key = jax.random.split(key, 2)
x = jax.random.normal(keyx, (5,))
y = module(x)

However, the code above does not work because of the error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ubuntu/anaconda3/envs/py39/lib/python3.9/site-packages/equinox/nn/composed.py", line 124, in __call__
    x = layer(x, key=key)
TypeError: log_sigmoid() got an unexpected keyword argument 'key'

So the module has to be slightly modified into:
module = eqx.nn.Sequential([
    eqx.nn.Linear(5, 10, key=key1),
    lambda x, key: jax.nn.log_sigmoid(x),  # adding a thin wrapper
    eqx.nn.Linear(10, 2, key=key2),
])

Is this a deliberate choice?
Why not detecting whether a module has key in its __call__ method and then call it appropriately (whether to use key or not)?

So this was a deliberate choice, basically just for simplicity/consistency of interfaces.
Detecting that (presumably via inspect.signature) sounds like it would be quite fragile. For example what if the layer provided is some sort of *args, **kwargs-accepting operation? (Perhaps it's a wrapper for something else.)
For this kind of thing a lot of libraries have some kind of Lambda functionality, that would also do the trick here:
class Lambda(eqx.Module)
    fn: Callable[[Any], Any]

    def __call__(self, x, *, key=None):
        return self.fn(x)

Maybe it'd be worth adding eqx.nn.Lambda? WDYT?

eqx.nn.Lambda seems too thin for me. Actually having lambda x, key: fcn(x) is not that of a big problem. It could just be a bit confusing for newcomers to equinox (like me) because having additional key kwarg is unconventional. Maybe a clearer error message in the initialization eqx.nn.Sequential would help?
Something like Module {module_name} has no 'key' in its __call__ method. equinox.nn.Sequential requires the module to be callable with 'key' argument.

So it's worth noting that one big difference between lambda and Lambda is that the latter is a PyTree, and therefore we can see any leaves of fn if fn is also a PyTree. (Rather than having lambda x, key: fn(x) as a leaf.) That is, fn need not be just a Python callable.
Nonetheless I'm inclined to agree with you -- it feels quite niche to have Lambda purely as a helper for Sequential.
I'm not sure how strongly I feel about an error message. I think the current one is already pretty clear, and moreover it's easy to debug. If we start introducing cunning machinery based around inspect.signature or something then it makes the implementation for MLP a bit more convoluted. How strongly do you feel about this?

I don't feel strongly on this as well because it is just a little hiccup at the beginning, but once people get used to it, then it is fine.


Add Book Package Dashboard Feature

Goal: without opening the resource status view, indicate visually the state of the book package.
@birchamp Let's use a red/green color for discussion purposes. Could you update this to edit and/or remove question marks to affirm the correct answer?
Needs Definition/Clarification: NOTE: the below is superseded by subsequent post.

If there are no repos at all, such as might be the case for a new org with no work done yet.. use red?
If a book has no resources at all... use red? [ie, there is at least one resource repo]
If a book has no Literal Translation... use ??
If a book has no Simplified Translation... use ??
If a book has no Translation Notes... use ??
If a book has no TWL... use ??
If a book has tN, but is missing one or more tA articles... use ??
If a book has tWL, but is missing one or more tW articles... use ??

You had made mention that a missing repo would not "count against" the quality of the BP until they had created it. And, a TW or TA without the TWL or TN is an orphaned condition. How do these factors contribute to the dashboard color?

In general I think:
RED - errors - This will break our software
Yellow - You might not want this to be this way, but stuff won't break
Green - Healthy (but not necessarily complete)
Here's my initial thoughts:
No Repos - Red
Book but no resources - Red
No Literal - yellow
TN exists but no Literal - Literal Red (because TN depends on it)
No Simplified - blank? (Does this cause an error in tC Create?)
No TN, TA, TWL, TW, TQ - blank
TW with no TWL - TWL Yellow
TN with no TA - TA Yellow

Refining the dashboard rules... here are the color rules (only for illustration; icons could be used as well or both):

red: indicates that there are manifest issues (no manifest or book is not in the manifest)

yellow: indicates everything is OK, except that the book file does not exist yet.

green: indicates that all is well.

if no repos exist at all, then "red"

if some repos exist, then confine the dashboard to those repos only.

Two dashboard production rules are below. One for books of the Bible and one for Open Bible Stories.
Given a book, say Titus, and existence of one or more repos:

red: if book is missing from all the manifests
yellow: if book is present in all the manifests, but no repo has a book file
green: if book file exists in all repos and is present in all manifests

On mixed conditions:

red and green: if book is missing from at least one repo manifest; and file is present and in the manifest of at least one repo
red and yellow: if book is missing from at least one repo manifest; and book is present in the manifest of at least one repo but file is missing
yellow and green: if book is present and in the manifest of at least one repo; and book is in manifest, but file missing in at least one repo

For OBS and existence of one or more OBS repos:

red: if OBS resource is missing from all the manifests
yellow: if OBS resource is present in all the manifests, but no repo has resource file
green: if OBS resource file exists in all repos and is present in all manifests

On mixed conditions:

red and green: if OBS resource is missing from at least one repo manifest; and resource file is present and in the manifest of at least one repo
red and yellow: if OBS resource is missing from at least one repo manifest; and OBS resource is present in the manifest of at least one repo but resource file is missing
yellow and green: if OBS resource is present and in the manifest of at least one repo; and OBS resource is in manifest, but resource file missing in at least one repo

Refining the dashboard rules... here are the color rules (only for illustration; icons could be used as well or both).
Note 1: if a resource is not in the manifest, then it is not considered for the dashboard. Not present in the manifest indicates that the team has not yet started on this book in this resource and, therefore, this does not count "against" the team for the dashboard. When the resource for the book is assigned, then this will be indicated by putting the book in the manifest.
Note 2: using the manifest to indicate that work has started means that the admin app should not create the manifest pre-loaded with books.
Rules:

red: indicates that the resource file does not yet exist (but is in the manifest)
green: indicates that the resource file exists and is in the manifest

Mixed conditions:

if at least one repo is red, then the dashboard for the BP should include red.
if at least one repo is gree, then the dashboard for the BP should include green.
showing both colors (or conditions) is normal; in such a case, the admin must open the resource details to discover which are red and which are green.

NOTES on TW and TA

Neither TW nor TA can be assessed without the BP's TWL or TN resource to specify which articles apply to the BP
Therefore, TW and TA will be red when none of the articles specified in the TWL/TN exist.
It will be green when at least one of the articles exist.
It will have no color and will not be considered for the dashboard if the book's TWL or TN do not exist.