SSO can cause UB with string_body

If a composed operation's movable state includes a message with a string body, and that composed operation attempts to use a serializer with that message, it will cause undefined behavior when the small string optimization is in effect.

No one has reported this as an issue so we can wait until someone does.


Adonisjs has something to do with binary file uploads?

I'm doing file uploading, but our front-end standard is uploading binary pictures. I haven't found a way to handle this part of the document. Do you have a solution now?

I'm doing file uploading, but our front-end's standards is uploading binary pictures. I haven't found a way to handle this part of the document. Do you have a solution now?

Issue doesn't adhere to the guidelines of creating a proper issue.

Make sure that you are only reporting bugs or submitting feature requests on Github.
In case of feature requests, please be descriptive and explain how this feature helps a broader audience and not just you.

If you are not submitting a bug and neither a feature request, then please use the forum or discord server
In any case, issues which are not descriptive enough will be closed. Please respect our time.


Add document language to HTML element

Per WCAG 2.0, the W3C's accessibility guidelines, HTML pages should ID their document language:
3.1.1 Language of Page: The default human language of each Web page can be programmatically determined. (Level A)
This makes it easier for assistive tech to pick the right captions, scripts, screenreader voice, etc. Instead of <html>, the header should be <html lang="en-US">.
For more, http://www.w3.org/TR/WCAG20-TECHS/H57.html

Should be able to just do this in application.html.erb, will do this and submit a PR

PR submitted

Tagging #127

Now PR #167

Fixed with #167!


Fix #37: Add a option for triggering build for new branches

Currently, a branch build is always triggered after discovering new branches.
I added a option to suppress this force build.

@vernyquartara It looks like that it has not been released yet. I think that it is included in next version 0.3.1.


Bugfix

REFERENCE: #1460
CHANGES:

shutil module in use (instead of distutils ... which is gonna soon be removed)
updated requirements.txt file
removed imp module (due to deprection)

How do i build your repository version??
I wanna test it

How do i build your repository version??
I wanna test it

Would it be OK if I say I have no idea what you are talking about .... as per my inference from the main repo to develope or test new features we just install it via pip under a virtual environment...is that correct ?

How do i build your repository version??
I wanna test it

Would it be OK if I say I have no idea what you are talking about .... as per my inference from the main repo to develope or test new features we just install it via pip under a virtual environment...is that correct ?

I say
how do i convert it to a setup.exe file??


How to improve ollama performance

current model params :
FROM llama2:13b-chat
PARAMETER temperature 0.2
PARAMETER num_ctx 4096
PARAMETER num_thread 16
PARAMETER use_mmap False
System config :
Ram 108 GB
T4 graphics card 16 gb

Also hardly any ram is being used.
Using ollama python bindings to get the result but due to some params issue not getting the result as expected. What am i missing here ?

Look in the log file to see what it says about GPU detection and model layers being offloaded to GPU
https://github.com/ollama/ollama/blob/main/docs/troubleshooting.md#how-to-troubleshoot-issues

If you're still having performance problems, please share your server log so we can better understand what's going wrong.


Make smoosh use UTC for scheduling window

Overview

Smoosh's exunit tests use UTC, so they sometimes fail when run in non-GMT time zones:
  1) test in_allowed_window returns true when to < from < now (SmooshSchedulingWindowTest)
     src/smoosh/test/exunit/scheduling_window_test.exs:53
     Assertion with == failed
     code:  assert :smoosh_utils.in_allowed_window('test_channel') == true
     left:  false
     right: true
     stacktrace:
       src/smoosh/test/exunit/scheduling_window_test.exs:59: (test)

  * test in_allowed_window returns false when from < to < now (0.5ms)

 2) test in_allowed_window returns false when from < to < now (SmooshSchedulingWindowTest)
     src/smoosh/test/exunit/scheduling_window_test.exs:44
     Assertion with == failed
     code:  assert :smoosh_utils.in_allowed_window('test_channel') == false
     left:  true
     right: false
     stacktrace:
       src/smoosh/test/exunit/scheduling_window_test.exs:50: (test)

  * test in_allowed_window returns true by default (0.03ms)
  * test in_allowed_window returns false when to < now < from (0.4ms)

  3) test in_allowed_window returns false when to < now < from (SmooshSchedulingWindowTest)
     src/smoosh/test/exunit/scheduling_window_test.exs:62
     Assertion with == failed
     code:  assert :smoosh_utils.in_allowed_window('test_channel') == false
     left:  true
     right: false
     stacktrace:
       src/smoosh/test/exunit/scheduling_window_test.exs:68: (test)

  * test in_allowed_window returns true when now < to < from (0.4ms)
  * test in_allowed_window returns false when now < from < to (0.5ms)
  * test in_allowed_window returns true when from < now < to (0.4ms)

  4) test in_allowed_window returns true when from < now < to (SmooshSchedulingWindowTest)
     src/smoosh/test/exunit/scheduling_window_test.exs:35
     Assertion with == failed
     code:  assert :smoosh_utils.in_allowed_window('test_channel') == true
     left:  false
     right: true
     stacktrace:
       src/smoosh/test/exunit/scheduling_window_test.exs:41: (test)

Testing recommendations

make exunit

should pass regardless of when or in which time zone the tests are run.
Related Issues or Pull Requests

Checklist

[x] Code is written and works correctly
[x] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

@jaydoane this creates a subtle breaking change from the scheduled compaction settings in previous versions and seems like it could create some user confusion, particularly given the standard requirement to run compaction outside of peak hours which tend to follow local time rather than utc (e.g. adjust for daylight savings).
I wonder if it would be better to maintain compatibility with the previous settings and change the tests to use local time instead?

@jaydoane this creates a subtle breaking change from the scheduled compaction settings in previous versions and seems like it could create some user confusion, particularly given the standard requirement to run compaction outside of peak hours which tend to follow local time rather than utc (e.g. adjust for daylight savings).
I wonder if it would be better to maintain compatibility with the previous settings and change the tests to use local time instead?

@willholley Oh drat, I was completely unaware that the old compaction demon used local time, which is a strange decision on its face, but I can see how it might have made sense, and how it could cause problems to change it. Although I could argue that a good time to change it might be now, with the introduction of smoosh. In any case, whichever way we go, the documentation should be very clear. I'm happy to change the tests instead to local time (or approve a PR if that's easier).
@kocolosk what do you think is the best approach here?

Funny, I added this enhancement and copied over the use of erlang:time() from the old compaction daemon without even thinking about the timezone issue. Even when I was writing the documentation it seemed totally natural to include examples of suspending compactions during  business hours. Sort of crazy in hindsight.
ISO8601 says to assume local time if no offset is given, so I guess that's what we should do for servers that set a timezone to something other than UTC.
It seems like it'd also be a good idea to enhance smoosh to understand ISO8601-formatted timestamps and recommend their usage to avoid ambiguity.


How to send image to vision model

Please tell me the correct way to send an image to the vision model.
this is my function:
def generate_image_description(image_path):
prompt = f"Describe the content of this image: {image_path}."
response = client.chat(model='llava-phi3:3.8b', messages=[
{
'role': 'user',
'content': prompt,
},
])
return response['message']['content']

Please refer to the definition of a "chat message" in the python code Message Type Dict.
The image can be passed in using the "images" key in your message dictionary. The "images" key is a sequence of "bytes" or "path-like str".
Here is an example:
import ollama

response = ollama.chat(
    model="moondream",
    messages=[
        {"role": "user", "content": "Describe the image", "images": ["./cat.jpeg"]}
    ],
)

print(response["message"]['role'])
print(response["message"]['content'])


Prune /libraries out of urls?

I wonder if we should just drop libraries from url. That's the only resource type we serve so now it seems redundant. We could do this change in v2.

Given #62, we're now planning to serve more than just the libraries resource (e.g. /jsdelivr/analytics) - can we close this issue?

Yes, I think it makes sense. Our API got bigger


Date field not displaying moment.js format

Describe the bug
Database Table contains a date field and needs to be displayed in local format. Date field "format" selected and displays formatted date in the preview page but not in the actual view.
To Reproduce
Steps to reproduce the behavior:

'Create a Table with a date field and enter a UTC datetime'
'Create a List View to display date and select Field (Date Field) -> Format -> enter LLLL in format filed'
'Finish List View creation'
See error -> Date shows "LLLL" not formatted Date

Expected behavior
Date field should be displayed in this format -> Mon, Jul 22, 2024 4:00 PM
Screenshots

Additional context
Saltcorn version	0.9.6-beta.15

The problem here is that we are not actually using moment but day.js which does not support all of the moment formats. This is to run it on the client side, so it can be localized for the user. I think the right thing to do here is a help popup with the supported format options

implmented in #2700


File uploaded, but UI not refreshed

When uploading an image, sometimes the progressbar does not update and is stuck at 0%, even though the file was successfully uploaded to the server.
In some cases, the UI eventually refreshes after waiting a very long time...
It is unpredictable to me when this happens, what's always is printed in the client console is following warning:
cfs_upload-http.js:138: The provided value 'undefined' is not a valid enum value of type XMLHttpRequestResponseType.
I've setup my schema as described in the example: https://github.com/yogiben/meteor-autoform-file#multiple-images
I'm using Meteor 1.2.1 and autoform-file 0.4.2
Does anyone have a solution to this problem?
Thanks in advance

same issue :(

Ah, i guess this is the same issue as mine https://github.com/yogiben/meteor-autoform-file/issues/100, only my file actually gets uploaded (receives a response from the server) after ~ 2 mins, UI refresh part is instant i suppose. I do get the same warning
The provided value 'undefined' is not a valid enum value of type XMLHttpRequestResponseType.
Thanks for noting that, i didn't notice that, i'll try to dig deeper.

Same issue here

check the last pull request guys

cool, i'll try if it works in the next couple of days.
but the fix will only work for files with size > 2mb?
can't this be tweaked so that the filesize doesn't matter?

No I meant this fixes the issue for files under 2MB so it will show all progress for all files...
Try it and tell me thanks

thanks for your contribution, i've tested the pull request, and it's almost working ;)
there is still an issue in autoform-file.coffee line 78.
the line should be:
(if doc then doc.isUploaded() else undefined) and doc._id
doc != null in coffee script is equivalent doc !== null in JS, which will resolve to true, because doc is undefined....
after this modification, the file upload works like a charm :)
this will fix the following error in the js console:
Exception in template helper: TypeError: Cannot read property 'isUploaded' of undefined
at Object.Template.afFileUpload.helpers.value (http://localhost:3000/packages/yogiben_autoform-file.js?92f4b34a1c115acdcda913bcbd0c149214fee28f:306:31)
at http://localhost:3000/packages/blaze.js?9391df93ba5076c2cfc61ee68724eb79b65f00d9:2986:16
at http://localhost:3000/packages/blaze.js?9391df93ba5076c2cfc61ee68724eb79b65f00d9:1650:16
at http://localhost:3000/packages/blaze.js?9391df93ba5076c2cfc61ee68724eb79b65f00d9:3038:66
at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?9391df93ba5076c2cfc61ee68724eb79b65f00d9:3671:12)
at http://localhost:3000/packages/blaze.js?9391df93ba5076c2cfc61ee68724eb79b65f00d9:3037:27
at Spacebars.call (http://localhost:3000/packages/spacebars.js?1aedcc2aa3ae9ff5d860d73516110cedd77c033e:167:18)
at Spacebars.mustacheImpl (http://localhost:3000/packages/spacebars.js?1aedcc2aa3ae9ff5d860d73516110cedd77c033e:104:25)
at Object.Spacebars.mustache (http://localhost:3000/packages/spacebars.js?1aedcc2aa3ae9ff5d860d73516110cedd77c033e:108:39)
at Template.afFileUpload.HTML.DIV.HTML.INPUT.value (http://localhost:3000/packages/yogiben_autoform-file.js?92f4b34a1c115acdcda913bcbd0c149214fee28f:97:24)


Rename Referenced Web Pages to Canonical Form

Rename the following web pages to canonical form and update links to them:

docs/home/11g-ocm/11g-ocm-performance-management/11g-ocm-set-up-and-configure-resource-manager/ASH_report_1327656054538.html?attredirects=0&d=1.html
docs/home/11g-ocm/11g-ocm-performance-management/11g-ocm-replay-a-captured-workload/workload_report_1329885258683.html?attredirects=0&d=1.html
docs/home/11g-ocm/11g-ocm-performance-management/11g-ocm-replay-a-captured-workload/workload_report_1329872485962.html?attredirects=0&d=1.html

This issue is a duplicate of #45 which has a better description.


Error loading syntax file

Hello, been using your Improved shell for a couple of days now :-) Good work.
When sublime updated to the new version though, ShellScript Improved stopped to work.
It's like a 1-3 days back. I've seen it today for the first time. Collegue on Monday.
Error loading syntax file "Packages/ShellScriptImproved/Shell-Unix-Generic.sublime-syntax": Error in regex: empty range in char class in regex \+{1,2}|-{1,2}|!|~|\*{1,2}|/|%|([=!*/%+-&^|]|<<?|>>?)?=|^|\|{1,2}|&{1,2}|[?:,<>]

I just started using Sublime and SSI, so I'm at a disadvantage 'cause I've never seen SSI at work... subscribing here to keep updated when a fix comes out! =)

I guess there are some changes for ST's regex engine. This can be reproduced in 3103 but not 3110. This will be fixed in 1.0.21.


2.1 Southampton Crystal Palace: Wilfried Zaha nets winner for Eagles

Goals in each half from Joel Ward and Wilfried Zaha have been enough to secure victory against the opposition out of shape, whose only response came from Oriol Romeu.cheap FIFA coins corresponding to each platform has a comprehensive sales, the players do not miss.
Palacio, largely inspired by former Southampton striker Jason Puncheon scored the first goal in the 29th minute. Puncheon former teammate Steven Davis dispossessed in midfield, dribbled to give from the center line of the penalty area and the armies of the square, a living routine ends just outside the door.
Low confidence of Southampton began clearly deteriorate further and director Ronald Koeman responded tirelessly introducing surprise Dusan Tadic and Juanmi for Maya Yoshida and Davis, back from a back from five to four.
His reply was almost instantaneous. If Sadio Mane tested Wayne Hennessey with a header 51 minutes, Romeu said to recovery, and produces a narrow range of routine to even finish.
Playing with more conviction, the hosts looked the more likely winner until 68 minutes, Palace won the leadership and victory thus closed.This may affect the FIFA 16 game on sale at a certain extent, the impact of cheap FIFA 16 coins sales.

Bollocks.


Portfolio Accessibility

Hi there, we are sending this PR for our code revision! 📝
We have check all page & items to ensure we comply with accessibility standard requirements.
We have prepared this project by doing mob programming.
The result of the check and changes was the following:

Page Title: changed, we did it more descriptive (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051L9-R9)
Images: corrected Alt, giving a more descriptive alternative text: Snapshot Images, logos, item. Hand icon set to null (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051L9-R337)
Headings: H1 – H2 - Changed h4 as subheading in cards, and replaced it with a h3, to respect hierarchy (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051L86-R240)
Color-contrasts: Future Features to be included
Resize: correct background size & position in "Contact Form" image (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-380b7b38760dd442e897eb0164c58f6a17da966ccaca6318017a468c163979b1L639-R660)
Interaction: Add labels in the form for voice-over reader using “text-indent:-999em” (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-0eb547304658805aad788d320f10bf1f292797b5e6d745a3bf617584da017051R355-R361) & (https://github.com/CarlosIgreda/Portfolio-setup-and-mobile-first/compare/main...accessibility#diff-380b7b38760dd442e897eb0164c58f6a17da966ccaca6318017a468c163979b1R752-R757)
Moving:  not an issue
Multimedia: not an issue
Basic structure of Page: no issues. Check done using Web Developer tool

Thank you in advance for your feedback! 🚀

Great @didierganthier, thanks for your review 🤓


[DC-2641] Verify whether or not all COPE surveys have one ID in the concept ID field

Fixed some of the inline comments in the SQL from -- comments to -- comments -- to comply with JINJA2 format.
Removed the check for DC-2146 from this notebook. This check was removed by DC-2346 but unintendetly added back by DC-2488
See the comment in DC-2641 for how the variable cope_question_concept_ids was generated.

@molly-giddings I am closing this PR without merging since the merge conflict got a bit messy. Can you please review again with https://github.com/all-of-us/curation/pull/1396? Sorry about the trouble!


[test] enable prefix/suffix test for the native runtime

un-skips the prefix/suffix test of String in the absence of ObjC, and verifies the matches without relying on an NSString method.

@swift-ci please smoke test

Lost the failure
@swift-ci please smoke test macOS platform

@swift-ci please smoke test Linux platform

This looks good to me, but cc @milseman for a second opinion.

Looks like we can’t quite un-XFAIL this test.

LGTM

Can one of the admins verify this patch?

@swift-ci smoke test Linux

Huh, okay. Well, I guess we can't take this after all.
Thanks for your efforts here, but I believe we'll need a deeper change to get this to pass.

Sorry: I haven't been able to build on Linux lately, so I couldn't do anything about this PR.


Url link in data

Hi,
Your package is very usefull, but is it possible to add a href link in data section to click on slider's picture and been redirect to a specific url or react-dom route ?
Regards !

I think this can be a good feature


[FEATURE] Add Interactable pooling/loading events

Introduction
Add Interactable pooling/loading events and potentially a "safe reference class".
Description of the feature
Lib.Interactable.OnBeforeLoadedOrRespawned(SafeInteractableRef )
Lib.Interactable.OnAfterLoadedOrRespawned(SafeInteractableRef , bool wasPooled)
Lib.Interactable.OnBeforePooled(SafeInteractableRef )
Lib.Interactable.OnAfterPooled(SafeInteractableRef )
Lib.Interactable.OnBeforeDeleted(SafeInteractableRef , bool isSafeDelete)
// potentially an extension like this:
public static SafeInteractableRef GetSafeInteractableRef(this Interactable interactable)
public class SafeInteractableRef {
  InteractablePreset Preset { get; } // never deleted
  bool TryGetInteractable(out Interactable) // can be explicitly deleted
  bool TryGetController(out InteractableController) // deleted when interactable despawns
  bool TryGetSpawnedObject(out GameObject) // deleted when interactable despawns
}

Feature usage
The pooling/loading events allow mod authors to respond if an interactable is pooled or re-loaded. This is important when a mod needs to change something about one particular instance of a Interactable, rather than the preset, because interactables are made to match their preset when they are re-loaded. The extension method is intended to allow mod authors to safely access (or be denied access to) Interactable data regardless of what is happening to the Interactable.
Dependencies and Prerequisites
Does this feature require any new dependencies? No
Implementation details
As a Lib helper with patches, and as an extension method and class definition (SafeInteractableRef).

I'm not sure if this is needed now that we have InteractableInstanceData.

I suppose it is still useful to have a feature to detect if an interactable is being pooled or spawned from the pool.
As we currently only have access to starting/completing/cancelling actions.


conflict with azerty keyboard

Hello,
Just to tell you alt+n is a bit risky as command, because it's the way to have "~" on an azerty keyboard.

And there doesn’t seem to be a clean way to restore its usage with the way Sublime works now. Adding another keybinding for fm_create while working does not overwrite the alt+n binding. Using something like "keys": ["alt+n"], "command": "" } is also not an option since the binding will still be intercepted. This is partly an issue with Sublime itself too but maybe it would be better not to put any keybindings by default? Or do you know any good solution to this issue?

I'm planning on changing this shortcut, maybe even removing it altogether (but explaining how to the user how to bind something themselves to fm_create).
To fix your issue now, you can add this to your keybindings:
{
	"keys": ["alt+n"],
	"command": "insert",
	"args": {
		"characters": "~"
	}
}

What shortcut are you going to be using instead? I'm asking because I don't really have any other idea as a good default for this command... 🙂

Thanks for your answer, I used this fix before but couldn’t find it anymore. It’s not ideal though but maybe the issue should be solved on Sublime side? Overriding global shortcuts without any easy way to tell what‘s going on or reverting behavior is not ideal…
Otherwise I think it is better to even remove the the binding altogether considering the situation and adding a message in the next update to explain why. I’m using super+alt+n for the fm_create_ personally but it might not be ideal either.

Yep, I think that's what I will end up doing.
Here's another idea: add a setting to enable default keybindings (this would be a boolean setting in FileManager.sublime-settings using package_setting_context)
Basically, I would like things to work "out of the box", whilst not being a pain if it doesn't quite work for someone.

Oh wait, I've just realized this that I had already implemented that feature 3 years ago: https://github.com/math2001/FileManager/commit/c9dd7b1efabec450ed8f4249b22f152d2027ad37
The setting is called "create_keybinding_enabled". Preferences: FileManager Settings, and set add "create_keybinding_enabled": false.
The FileManager is somewhat extensive, but not discoverable and it doesn't include everything either. There should be a single document (probably the readme) listing common tips and tricks #52.


Fix tutorial (#5093)

Close #5093

Oops I merged without linking to the hacktoberfest issue - not sure if it counts in this case
sorry about that..

It should :) (the hacktoberfest repo topic is all that matters)


Clarify how to handle web socket back pressure

The docs should be clearer about how to handle back pressure. Right now, the explanation is
def ws_message(ws, message, opcode):
    #Ok is false if backpressure was built up, wait for drain
    ok = ws.send(message, opcode)

But how should I wait for drain? Should I busy wait before returning from the call or wait in the next call or something else?

You should queue this in some way and use the ws_drain handler to try send it again


Add 'undelete' after record deletion

Partially implemented for one step back only of person delete. There is problem tracing back relations if persons id's get changed after undelete.


No error indication when trying to log in with wrong password

At site: https://moduleregistry.azurewebsites.net/
I try to log in and enter wrong password.
Actual: when pressing log in with a non-valid password and press login, nothing happens the page stays the same
Expected: It would be good if some pop-up or notification was shown indicating that the username or passord combination was not valid.

You get a positive response when providing correct username and password - you are logged in. If not, I think its a good idea to have no response at all, because this does not reveal anything when someone is trying to log in with without correct password.


time.h included but not used

I have difficulties to compile KalmanFilter, the error is file time.h not found.
I checked the file KalmanFilter.cpp and I think despite you included it, you are not using time.h.
Maybe you should remove the line ?
Maybe I am wrong but maybe have a look at it just to be sure.
regards,
TTAK

I fully agree with TTAK. I was unable to compile my sketches the last time I updated my arduino IDE. Same conclusion: one include line to remove.

Thanks guys, it's fixed.


[QUESTION] Is it possible to specify the name of the key when response model is set to dict?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
class Something(BaseModel):
    a: str
    b: int

@app.get("/something", response_model=dict[str, Something])
async def get_something():
    some_dict = {}
    return some_dict[Something.a] = Something

Description
Open the browser and navigate to /doc
In the schema for this route it will have {"additionalProp": Something}.
Is there a way to rename the additionalProp to something else on a per route basis, for example so it will say {"a": Something} instead?
Operating System
macOS
Operating System Details
No response
FastAPI Version
0.72
Python Version
3.9.7
Additional Context
No response

You can always override the openapi method

Closing for now as it's more to do with OpenAPI!
WIll update with my solution when I get time ✨


Namespace not specified. Specify a namespace in the module's build file

Describe the bug
After trying to upgrade my Flutter app's Android port to AGP v8.0+ I get an error message:
A problem occurred configuring project ':uni_links'.
> Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
   > Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

I depend on uni_links 0.5.1 in my app. I just tried to upgrade the Android side of the Flutter app to AGP 8.0+ (to be precise it's AGP 8.1 now), because Android Studio brings up the upgrade assistant for the last half year about this. The upgrade fails because some of the plugins don't have the namespace in their build.gradle. On of the failing plugins is the uni_links.
To Reproduce
Steps to reproduce the behavior:

Have a Flutter app with Android port and AGP 7.4 (or any AGP version below 8.0)
Open the Android folder of the project with Android Studio
The AGP upgrade assistant will pop up
Let the AGP upgrading assistant try the upgrade and it gonna fail

Expected behavior
I'd be able to upgrade to AGP 8.0+.
Smartphone (please complete the following information):

Device: OnePlus Nord EU (AC2003), Android 12
OS: Devuan GNU/Linux 6 (excalibur/ceres) 6.4.0-1-amd64, locale en_US.UTF-8
Flutter version: Channel beta, 3.13.0-0.2.pre

Additional context
The way Flutter Community projects solve this is adding a conditional clause to the build.gradle:
android {
    // Conditional for compatibility with AGP <4.2.
    if (project.android.hasProperty("namespace")) {
        namespace 'name.avioli.unilinks'
    }

    compileSdkVersion 28

    defaultConfig {
        minSdkVersion 16
        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
    }
    lintOptions {
        disable 'InvalidPackage'
    }
}

Do you need a PR?

Here is  to get it to work until PR is merged:
uni_links:
git:
url: https://github.com/DanMossa/uni_links.git
path: uni_links/

This package is now marked as discontinued and app_links recommended replacement.


arm64-v8a version of libyuv_static.a

Hi, I am working on a arm64-v8a version of your great tutorial. Since the official Caffe2 AICamera project has not been updated for several months, this repo really helps me a lot.
My question is that how could I generate the libyuv_static.a for arm64-v8a? Should I compile it based on other project? Thanks!

Hi, @meegoStar  You can check https://chromium.googlesource.com/libyuv/libyuv/+/master

Hi, @wangnamu , thanks for your reply!
Based on the "Getting started" instructions in the link you provide, I build libyuv_internal.a and libyuv_neon.a for arm64-v8a successfully, and finally manage to make the arm64-v8a version of this AICamera project work!!! Great thanks again!
p.s. For any others interested in building libyuv_*.a for arm64-v8a, this link in the "Getting started" page (https://code.google.com/p/chromium/wiki/AndroidBuildInstructions) is outdated and should be changed to this one (https://chromium.googlesource.com/chromium/src/+/master/docs/android_build_instructions.md#install-additional-build-dependencies).

@meegoStar Fine, thank you


Cannot read property 'protocol' of undefined

C:\Users\Candyseul\Desktop\WADark-Windows\whatsapp-desktop-dark (master<EMAIL_ADDRESS>λ npm install

><EMAIL_ADDRESS>postinstall C:\Users\Candyseul\Desktop\WADark-Windows\whatsapp-desktop-dark\node_modules\resourcehacker
> node scripts/install.js

Downloading ResourceHacker by Angus Johnson...
_http_client.js:131
  var expectedProtocol = defaultAgent.protocol;
                                      ^

TypeError: Cannot read property 'protocol' of undefined
    at new ClientRequest (_http_client.js:131:39)
    at request (http.js:44:10)
    at Object.get (http.js:48:15)
    at Object.<anonymous> (C:\Users\Candyseul\Desktop\WADark-Windows\whatsapp-desktop-dark\node_modules\resourcehacker\scripts\install.js:23:20)
    at Module._compile (internal/modules/cjs/loader.js:776:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
    at Module.load (internal/modules/cjs/loader.js:643:32)
    at Function.Module._load (internal/modules/cjs/loader.js:556:12)
    at Function.Module.runMain (internal/modules/cjs/loader.js:839:10)
    at internal/main/run_main_module.js:17:11
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `node scripts/install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

I can't reproduce here.
Can you please give me more information about your setup ?

Version of windows
Version of nodejs
The shell you're using (cmd, powershell, git-bash, ...)
Is any of the following env variables are set ? npm_config_proxy http_proxy HTTP_PROXY

OS: Windows 10 1903
node: 12.10.0
npm: 6.10.3
yarn: 1.17.3
Shell: cmder also I tried cmd too.
env variables: Nothing I'm not using anything.


Added ability to filter IP addresses

Added ability to filter IP addresses for Geolocation Security. Propagated the remote IP address using HTTP's X-Forwarded-For header. Updated test scripts to handle X-Forwarded-For header.

Looking very good, @sanjay-saxena. :smile:


given proper spacing for image count

Given proper spacing for the image count as it was not looking proper before
Before:

After:

Developer Certificate of Origin Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

#609 @brenoferreira

now the text will also been aligned

@akmadian i have done the changes you can have a look sir.

Changed px to rem as 10px = 0.6rem and 20px = 1.3rem
You can check now @brenoferreira

@brenoferreira do i have to do more changes or it's ok?


Changed the onclick state of button

@raka334 Create a new pull request. This pr cannot be accepted. Please take a look at what you are uploading before raising a pull request.


[OPTIONAL BUGFIX] Add Y2030 bugfix

We're 7 years away from the year 2030, and since we're reverse engineering the game now would be a great time to implement a Y2030 fix. Because we need to stop people from capitalizing on it and selling 40-dollar Y2030 survival kits. Nobody has studied the effects, so it could be much like the Y2K problem. We should fix it now while we still have time.

The repo can't currently achieve that yet, since the timein submenú overlay (aka the date changer before loading your player) isn't implemented yet, and other changes in the repo too that clamp the year to 2030
Regardless anything, I think it would be good to do that whenever it's done, but at the moment it's not possible with the decomp at this current state

So, we should have this as somewhat of a priority. Besides, we're 7 years away from 2030, so this really isn't an issue right now. We nearly got burned with Y2K, and we might get burned with Y2038. When it becomes possible (hopefully soon within a 7-year period), we should add it. And chances are other Animal Crossing games may share the same mechanism for managing the clock; so, we may be able to fix CF, NL, and NH in one sweep of the rug. Nintendo's programmers should have had foresight. We're doing their job for them

I think the main problem is that a lot of things (such as the event calendar) are all hardcoded through 2030. The most ideal way of implementing this would be to go through and create some sort of algorithm for generating the event calendar so that it can go beyond 2030. And since the decomp is still in progress, it's just not possible right now.

The repo can't currently achieve that yet, since the timein submenú overlay (aka the date changer before loading your player) isn't implemented yet, and other changes in the repo too that clamp the year to 2030
Regardless anything, I think it would be good to do that whenever it's done, but at the moment it's not possible with the decomp at this current state

m_timeIn_ovl is linked, https://github.com/Prakxo/ac-decomp/blob/master/src/m_timeIn_ovl.c
But yeah there're still a lot of other constraints preventing us from adding something like that. The biggest is that the luni-solar based events cannot be algorithmically calculated due to how it works. You can approximate the dates but it's not going to be exact unless you hardcode them. So we would either have to ditch luni-solar event calendar or add in N years we want to support.

The repo can't currently achieve that yet, since the timein submenú overlay (aka the date changer before loading your player) isn't implemented yet, and other changes in the repo too that clamp the year to 2030
Regardless anything, I think it would be good to do that whenever it's done, but at the moment it's not possible with the decomp at this current state

m_timeIn_ovl is linked, https://github.com/Prakxo/ac-decomp/blob/master/src/m_timeIn_ovl.c
But yeah there're still a lot of other constraints preventing us from adding something like that. The biggest is that the luni-solar based events cannot be algorithmically calculated due to how the calendar works. You can approximate the dates but it's not going to be exact unless you hardcode them. So we would either have to ditch the luni-solar event calendar or add in N years we want to support.

The best solution is to hardcode 2099 as the date rather than removing it. Besides, 2099 is the maximum value that the GameCube's internal real-time clock will accept. This has the bonus of preventing people from jumping on the fear bandwagon and selling $40 survival kits

This will be considered but in a mod repo. The 2030 year limit is not a bug. The only optional features added to this decomp repo at the moment are bugfixes and minor code flow improvements. I will be increasing the maximum year to 2100 in my 'deluxe' mod but that's separate from decomp.
https://github.com/Cuyler36/ac-decomp/tree/deluxe

This will be considered but in a mod repo. The 2030 year limit is not a bug. The only optional features added to this decomp repo at the moment are bugfixes and minor code flow improvements. I will be increasing the maximum year to 2100 in my 'deluxe' mod but that's separate from decomp.
https://github.com/Cuyler36/ac-decomp/tree/deluxe

Still better than no bugfix at all. And having to worry about people trying to leverage this for monetary gain

Still better than no bugfix at all. And having to worry about people trying to leverage this for monetary gain

The thing is it's not necessarily a bug. The devs intended it to go through 2030. Optional bug fixes are only present for mistakes made by the devs, such as oranges being unobtainable in your town unless you have peaches. As for someone leveraging it for monetary gain, I don't think you have to worry about that. Nothing like that has happened for other decomp projects like Super Mario 64, Zelda OoT, etc.

So it really isn’t a bug, rather it was done to make programming easy and to save a dozen bytes of memory. This really looks like Y2K to me. Also, by people leveraging it for monetary gain; I mean fear mongers leveraging the fact the game only goes up to 2030 and trying to sell $40 “survival kits” much like they did in the late 90s when Y2K was near. Nintendo’s programmers should have had a little bit more foresight about that, as tons of people still play the original Animal Crossing and most GameCubes are still working nearly 20 years later.

tbf no programmer for a game 20 years ago would have expected people to still be playing 2 decades later


⚠ Bofrost.at has degraded performance

In a1db29d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1386 ms

Resolved: Bofrost.at performance has improved in b40258f.


cuda aligner API to take in max available memory and max ref/query sizes

each aligner batch to take in max memory and max ref/query sizes and determine how many how alignments can be performed in the batch.
provide api to check max alignments possible
actual max alignments may me larger based on inputs processed so far, and actual max can be determined by continually adding and checking return value of add alignment api call

Done for banded Myers implementation, but Hirschberg still uses old API that doesn't cap memory from the API itself

Done for banded Myers implementation, but Hirschberg still uses old API that doesn't cap memory from the API itself


Clean separation between SQL and PL/SQL.

The eunit and common tests of dderl and imem went through without any problems.
Most of the changes can be found in the regression test environment, since we now have statements that are only allowed in PL/SQL, or only in SQL, or both.

Coverage decreased (-0.3%) to 98.833% when pulling 96d83d0210d321526acb1a2698fe1304cd4c510a on wwe_issue_155 into c109be1367b515f06620a645d81bfe3a42ce7efd on master.


[Map] DM-Coke

Hash: c93ec6a5704d18d01cf65b939552c99086f1d081
Current name: DM-Coke
According to the readme, the author is simon 'birdieman' medium
This author also sometimes uses just the name Birdieman

Done and added aliases


Code explanation?

Can You explain why in yaml file defaults are this --
target: ldm.modules.embedding_manager.EmbeddingManager
params:
placeholder_strings: ["*"]
initializer_words: ["sculpture"]
per_image_tokens: false
num_vectors_per_token: 1
progressive_words: False
So that means i am trainig a sculpture even if my --init word is "mydog sketch" ? I dont quite get it, should i remove the "sculpture from there ? Where are instructions about all this ? Was it skipped cause its not important ? Id prefere to know before i spend days on training and i realise i trained a sculpture not a sketch drawing
Ooooor , these are just defaults if you dont use --init word and theyre ignored once you use init word ?
So next question---------------------------------
I want to train Stablediffusion model on images of he-man from filmation cartoon( cause now he-man doesnt look like him much in stable diffusion) , sooo, i have --init word "he-man filmation" and img folder with images from cartoon and now questions....
Will this make other versions of he-man to improve in any way ? ( when i prompt he-man filmation photorealistic, film still )  Or only this he-man filmation cartoon version ? What if i prompt only he-man without filmation ? WIll embedding checkpoint affect my results too ?  I want to eventually train on more realistic images of he-man , should i do it separately ? Shouuld i just bring all he-man images and train on them at once  with realistic, cartoony ? I cant find info on this anywyere.I want to basically force out current bad looking he-man images and bring in good looking images by embedding, how i can do that so it wont generate cartoon and realistic he-man images from original stable diffusion model but rather from finetuned checkpoints where he looks like he should ?
So when i train with --init word he-man filmation, should i adjust something in yaml file ? like initializer words or something else ? Or is the command line with --init word enough ?
Next question ------------------------------
I can see some empty white images in preview during training  with text on them "a photo of my *" , why theyre white ? I dont get this part at all, am i training this wrong way ? Should they show me he-man images ? I have to add that i can see he-man images as well in this folder ( on reconstruction gs images and input gs images ) but im concerned about the white ones named "conditioning" , what are they ?
What are images called "reconstruction gs" ????
What are images called "inputs gs" ????
What are images called "samples" ?? Are those actual results from the model with finetuned data being used when generating ?
I know that samples are made with makesamples.py but it does not contain any info on what samples are and how they might guide you.
Sorry lot of questions but i want to get this right and the paper does not have any info about all that, where is that info so i can read what each image means for finetuning ?

The basis for a lot of your questions are addressed at this issue.

Thanks so its what i suspected, , --init word just overwrites whats in  yaml so thats good that im not training a sculpture, im wondering about other questions tho, ill be training just cartoon version for now and see if it will affect other non cartoon styles.

Config file:
Most of these extra parameters are outlined in the actual paper. They are just flags for enabling our baselines.
Here's what they do:

placeholder_strings: The actual string that will represent the concept in future prompts. This must be a single token string, so don't use things like "myplaceholderword".
initializer_words: The words used as the initialization for the 'new word' embedding. This is not the placeholder, and will be overwritten by whatever you provide with --init_word when running main.py. It's just a starting point for the optimization. It should be some rough description of your concept's main class, for example 'dog', 'face', 'painting' etc.
per_image_tokens: This gives each image in your training set (up to ~20ish images) its own unique token, in addition to the shared placeholder. This way, the model can try to capture the shared information in "*", and the image-specific information (background etc.) in the specific tokens. For LDM this led to worse results (some information about the shared concept was instead stored in the per-image tokens).
num_vectors_per_token: This is the number of embedding vectors used to represent the concept. Basically how many 'words' is the concept represented as behind the scenes. More words = more expressivity, but also more overfitting so its hard to later edit images.
progressive_words: If you are using more than one vector per token, you can enable this to increase the number of vectors progressively over training. So you'd start with 1 word which will capture the concept as best as it can, and after a set number of training iterations, the model will move to using more and more vectors. This is meant to give the same flexibility as the multi-vector approach, but possibly reduce overfitting. This also didn't work well with LDM, so its off by default.

Training on He-man images:
We didn't train on mixed modalities. You're free to give it a try and see what works best for you.
Don't use 'he-man filmation' as your init word because it's not a single token word. Try something like 'cartoon'.
You don't need to adjust the initializer words in the config if you use the --init_word flag. It's there exactly for this reason - to let you avoid editing the config every time.

Output folder images:
gs stands for global step, it's the step of the training where the images were produced.

inputs: These are just the input images provided to the model at this iteration
reconstruction: If you read the LDM/SD paper, you'll see that part of the model is a network that knows how to compress and uncompress the images into some latent space, where the actual diffusion model is performed. The reconstructions show the input images after this compression and uncompression step, so you can make sure its loaded correctly / see what small details might be lost at this step.
conditioning: These are the conditioning sentences used when sampling images for validation. It literally just prints out the text that was used to produce each of the samples in the same step.
samples: Samples produced at this step with the conditioning texts but without classifier-free guidance (e.g. the guidance scale is 1.0). These will be mostly noise.
samples_scaled: Samples produced at this step with the conditioning texts and with classifier-free guidance (guidance scale is 5.0). These should look like your concept.

The output you want to track is samples_scaled. Everything else is mostly for debugging purposes.

Thanks a lot , those are very helpful , so if i dont have to specify that its he-man on images by --init word at all - does the model know that its him ? How come ? This part is confusing, also can i somehow force out trained data from original 4 GB ckpt weights , i think they tagged a lot of images as he-man in dataset but it wasnt him and it corrupts the results too much.
I tried to resume the training of the model but it just starts over, how do you resume , whats the command line to do it properly ?

We don't currently have resume support. There's a couple of requests for this so we'll have it done over the weekend.
We're learning a new word that represents he-man, using the images you're providing. So it shouldn't matter if the pre-trained model saw unrelated images tied to this text. What you'll have at the end of the day is a new word, '*', which represents your images, and you can use in prompts like "a photo of *" or "an oil painting of * hanging on the wall".
If you provide images of some random bearded man, it will learn the random bearded man, not Chuck Norris.
I don't think it will work very well if you have too many different viewpoint images. I'd try to focus on frontal stuff.

So the best way to train he-man in my case is to just use --init word "cartoon" and dont even name images in a specific way at all ?
Cause original  LAION dataset that stablediff was trained on had images and also a text describing whats on the image, isnt this supported or its not needed ?
https://huggingface.co/datasets/laion/laion2B-en

Sorry, completely missed this followup.
You don't need to name your images. Using just "cartoon" for the init word is good, yes.
What we're doing is learning a 'word' that describes the object. You don't need to give us this description.
25k iterations seems a bit much. I wouldn't expect it to keep improving that far. Most of the time, ~5k iterations is enough.
Identical results to inputs - it's possible if you do things like increasing the number of vectors per token in the config, but this will also cause a sort of overfitting where its harder to change the subject with new text.
You can certainly put your init_words in the config instead of the --init_word argument, yes. I don't think the tokenizer knows filmation and he-man so those will likely be multi-token words and will cause problems. Using "cartoon" is absolutely fine.
We didn't try training on different body parts and merging them, but I'm not sure it will work. Using multiple subjects in one prompt tends to merge them semantically, not place them side-by-side.

Wouldnt using "male" be better even tho its a cartoon version? im training a character not a cartoon style

Wouldnt using "male" or "likeness" or "identity" be better even tho its a cartoon version? im training a character not a cartoon style, anyone figured out a best universal way to train a likeness while staying editable into other styles ?
I did that stallone test and rsults were great but its 50 vectors and this is way too much, with 20 vectrs likeness got worse, what is other way to improve likeness ? https://github.com/rinongal/textual_inversion/issues/35
Are there any ways to separate subject from the style when finetuning  so its not that heavilyu embedded into photo realistic ?
Is there a way to tell how many vectors for their likeness take lets say trump or zuckerberg in SD weights ? i tried with 2 but its getting there very slowly

Sorry, completely missed this followup.
You don't need to name your images. Using just "cartoon" for the init word is good, yes. What we're doing is learning a 'word' that describes the object. You don't need to give us this description.
25k iterations seems a bit much. I wouldn't expect it to keep improving that far. Most of the time, ~5k iterations is enough. Identical results to inputs - it's possible if you do things like increasing the number of vectors per token in the config, but this will also cause a sort of overfitting where its harder to change the subject with new text.
You can certainly put your init_words in the config instead of the --init_word argument, yes. I don't think the tokenizer knows filmation and he-man so those will likely be multi-token words and will cause problems. Using "cartoon" is absolutely fine. We didn't try training on different body parts and merging them, but I'm not sure it will work. Using multiple subjects in one prompt tends to merge them semantically, not place them side-by-side.

Wouldnt using "male" be better even tho its a cartoon version? im training a character not a cartoon style, anyone figured out a best universal way to train a likeness while staying editable into other styles ?
I did that stallone test and rsults were great but its 50 vectors and this is way too much, with 20 vectrs likeness got worse, what is other way to improve likeness ? https://github.com/rinongal/textual_inversion/issues/35
Are there any ways to separate subject from the style when finetuning  so its not that heavilyu embedded into photo realistic ?
Is there a way to tell how many vectors for their likeness take lets say trump or zuckerberg in SD weights ? i tried with 2 but its getting there very slowly

You can combat the overfitting by using the weighted prompts approach from the webui repo. I'm not sure how well it would work when you're using 50 vectors, but you can try giving the rest of the prompt a very high score compared to the part with the placeholder ("*"). Regarding the initial words: I don't have a better tip than just to try. In practice your results are probably more sensitive to your initial seed than they are to this choice of an initial word.

i compared 2 vectors vs 50 vectors, it takes the same amount of overwhelming it with words representing style, theres practically no difference , just takes longer to train 2 vector one to get similar likeness as 50 vector one.
I did managed to change from photo to oil painting but when changing to anime, stallone mixed with japanese man.
I dont know but i think it randomy takes away from identity tokens when you force the style, and you lose identity but style transfer is more possible, thats not always the case tho, i could get better identity with ilya repin oil painting.
All in all is this a final version of this approach or some new ones are coming to battle overfitting and identity preservation issue ?


erro on compute_milan_descriptions.py

I faced an error when computing Milan descriptions with running code below:
! python3 -m scripts.compute_milan_descriptions biggan imagenet --device cuda
the error is located on line 71 of compute_milan_descriptions.py.
with results_csv_file.open('r') as handle:
I think it should be:
with results_csv_file.open('w') as handle:

Good catch, fixed!


US 04.02.01

Shopping List

Allow meal planner to see each ingredient on the shopping list with its description, amount, unit, and ingredient category

Done


Debug submission

Coverage increased (+0.06%) to 73.441% when pulling 106628526c498b3961f9b922a71b0a17994c83da on debug-submission into b951f5d6f8db41cb49d176b1821edbf824b78acf on master.


Change the changelog link in /README.md to CHANGELOG/README.md

Signed-off-by: Jianjun Shen<EMAIL_ADDRESS>

Codecov Report

Merging #2296 (7e338d2) into main (14f58f0) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head 7e338d2 differs from pull request most recent head 8536f5a. Consider uploading reports for the commit 8536f5a to get more accurate results

@@            Coverage Diff             @@
##             main    #2296      +/-   ##
==========================================
- Coverage   41.23%   41.22%   -0.02%
==========================================
  Files         141      141
  Lines       17385    17385
==========================================
- Hits         7169     7167       -2
  Misses       9583     9583
- Partials      633      635       +2

Flag
Coverage Δ

unit-tests
41.22% <ø> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/apiserver/storage/ram/store.go
77.44% <0.00%> (-1.51%)
:arrow_down:

pkg/controller/traceflow/controller.go
66.26% <0.00%> (ø)

/skip-all


buildRustCrate: adding the description field

Motivation for this change
The CARGO_PKG_DESCRIPTION was ignored by buildRustCrate. This PR changes that situation, and adds documentation.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build buildRustCrateTests


In the Query Result, unable to select multiple cells

Type: Feature Request
When i run the Oracle SQL query, results are displayed in the grid (Query Result grid). I want to select multiple cells in the column but unable to do so. Same feature is available in many grids (e.g. oracle SQL Developer).
Currently i have to export the data in the CSV and from then i need to select required cells.
VS Code version: Code 1.95.1 (Universal) (65edc4939843c90c34d61f4ce11704f09d3e5cb6, 2024-10-31T05:14:54.222Z)
OS version: Darwin arm64 23.6.0
Modes:

Hi @kartik1581,
This issue appears to be a support question specific to the extension which is best addressed by the extension owner.  We recommend visiting the extension page on https://marketplace.visualstudio.com/ and posting a question there. Let us know if we can help you with anything else.
Regards,
VS Marketplace Team


Windows Hello PIN/Smartcard issue resolved?

Can somebody please check if the issue mentioned in the README below Known limitation is resolved by enabling Shared User Certificates on package.appxmanifest's Capabilities tab?

Yes, please use WAM. I'll make an effort to update this sample, but see https://docs.microsoft.com/en-us/azure/active-directory/develop/scenario-desktop-acquire-token-wam


node 20 support

Node 20 is LTS now. We are unable to upgrade to Node 20 because the package allows upto Node 18 only. It would be great to have this done.

Please consider moving to the in-tree provider of strapi, they included all functionality of this plugin over the last iterations, notably baseUrl, prefix (which they call rootUrl), and the use of TypeScript. If you run strapi with Node 20, i can assume your version is fairly recent and you should be able to just move the configuration over.
Closing this issue with regards to ab743a132624cc58415654f69476d7e686b2c271


MIL-6: Summary of Work

What's up?
As stated in #226, our Milestone Group Review should include the following deliverables:

Overall description of the responsibilities that are assigned to each member.
A Summary of work performed by each team member (in tabular format).

To Do
Please add your summary of work to the md file and update the list below:

[x] Sena Mumcu
[x] Karahan Saritas
[x] Serdar Akol
[x] Elif Bayraktar
[x] Mustafa Cihan
[x] Metehan Dundar
[ ] Furkan Keskin
[x] Sinem Kocoglu
[ ] Serhat Hebun Simsek
[x] Dogukan Turksoy
[x] Mustafa Emre Erengül

Deadline
04.11.2022 @23.55
Additional Information
No response
Reviewers
I'll close the issue when the deadline is reached.

Deadline has passed, no further action is taken.


not working with webpack and es6

import Big from './big.mjs'; this gives unable to resolve path
and require seems to import it but throws an invalid number error even when a basic number is given as argument

If I pass a hard coded number directly it works but if I pass a number from variable I get [big.js] Invalid number

How was this resolved?


In Parameter window, default values should not have ↪ marker

Minor issue in parameter window

change a parameter
-> ↪ is shown, which nicely allows to go back to default value with one click
edit parameter, so that the value is default
-> ↪ is still shown, click on it is somewhat useless because it has no effect (besides of removing the marker)

If fixed, ↪ would also serve the purpose to indicate that a value is non-default

This is by design: The default value is NOT just a value. It means "use the value that is defined by the Symbol for that parameter". This means, that you can, e.g. change the default Blend mode for all [Layer2d] instances.
And sometimes is even makes sense to say "yes, I explicitly set this parameter 0.0, although it's the default, but I want o make sure that it will not be affected by changes to the default value".
Although advanced this feature is incredibly valuable for refining Operators and their behaviour.


Store eligibility and character decisions

@KateMJAC
You need to flesh this out but the idea moving forward is that the system will store SCC decisions and auto-populate if the same candidate.

This Epic supersedes #248


ARROW-12249: [R] [CI] Fix test-r-install-local nightlies [WIP]

@github-actions crossbow submit test-r-install-local


fix: remoe unnecessary code

I just remove a unnecessary code, please check App.tsx file

:tada: This PR is included in version 1.1.1-beta.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add support for randomizing the enable_hashjoin setting in tests using @UseHashJoin

@andreidan Nice refactoring for the annotations!

Codecov Report

:exclamation: No coverage uploaded for pull request base (hash-join@0aaefad). Click here to learn what that means.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             hash-join    #6832   +/-   ##
============================================
  Coverage             ?   83.26%
  Complexity           ?    13455
============================================
  Files                ?     1536
  Lines                ?    49966
  Branches             ?     5996
============================================
  Hits                 ?    41605
  Misses               ?     5556
  Partials             ?     2805

Impacted Files
Coverage Δ
Complexity Δ

...adata/settings/session/SessionSettingRegistry.java
96.42% <100%> (ø)
8 <2> (?)

.../main/java/io/crate/action/sql/SessionContext.java
100% <100%> (ø)
16 <2> (?)

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 0aaefad...52b426a. Read the comment docs.


Toposort build env

This PR works locally, it's ready for merging into master to build+upload the docker image.

Looks like travis doesn't like the /anaconda dir for OSX build. We may have to go back to /tmp/anaconda. I will change that back over, and also remove the crossmap recipe for now since that appears to also be failing now.


Bug, nameof fix doesn't work well with operators

For  "a"  in method Add CodeFix returns nameof(a), but in operator+  returns nameof(MyClass):
    public class MyClass
    {
        public MyClass Add(MyClass a, MyClass b)
        {
            var str = "a"; //Fix to: nameof(a)
            return null;
        }
        public static string operator +(MyClass a, MyClass b)
        {
            var str = "a"; //Fix to: nameof(MyClass)
            return null;
        }
    }

@SergeyZhurikhin Bug confirmed.

This is a duplicate of #594, and it is already being fixed. I just verified the PR and it fixes this case as well.


Retrieve song suggestions api is not working

I attached screenshot for reference

Every page has the same result

I think it is related to this issue

SUGGESTIONS try this example
https://saavn.dev/api/search/artists?query=Linkin+Park/suggestions
Domain / api / search / artists?query= / suggestions

Where can I get the song suggestion IDs for the suggestion API?

https://jiosavan-api-with-playlist.vercel.app/api/songs/id=BeXBcbVK/suggestions 500 (Internal Server Error)
https://jiosavan-api-with-playlist.vercel.app/api/songs/id=yDeAS8Eh/suggestions 500(Internal Server Error)


GitHub Page doesn’t include link to repo

Hi, just a small note: I landed on the GitHub Page and wasn’t able to find a quick link to the GitHub repo. I was on mobile and wanted to star the gem for later. I ended up cracking a new tab to search for the gem. Right, I know. First world problem. But, I just wanted to share it with you as it’s easy enough, and it might help other users get on their way.
Cool gem, thanks!

Just added a link to the main body. Thanks for the note!


ILO Theme Foundation

Foundation
The PR introduces the foundation file mentioned in #999. The foundation file has 2 primary sources, the first one is the Figma foundation file, and the second one is the old token system or actual source code. There is also a third-ish one I named it Mixed, part from source code and part from Figma components
Foundation file
✅ - Implemented
🚧 - Partially
⛔ - Not Implemented

Scope
Status
Soure

Colors
✅
Figma

Typography
🚧
Mixed

Sizing
✅
Figma

Spacing
✅
Figma

Breakpoints
✅
Source Code

Transitions
✅
Source Code

Borders
✅
Source Code

Typography
Typograph is marked as not implemented because we need a Figma update for #896, but it has a main port of the current token system
Mixins
Two widely used mixins were refactored

spacing mixin now is fully powered by a new foundation
breakpoint mixin was a bit problematic because referencing CSS variables as a media query value isn't possible without js interaction so I have used SCSS variables to define the breakpoints(for mixing usage) and also composed CSS variables from those, SCSSversion of variables aren't directly referenced by any property it's only used for breakpoint mixin.

Found Figma Problems
We have 2 green colors, so I made the second one, green-light

Prefixes
All foundation variables are prefixed as proposed in #999. Prefixes are static for now, but they can be implemented with concatenation if we really want to make it "dynamic".  I still suggest to keep it static for now because it's better for visual feedback, referencing and general readability, after we agree about "composability" of the designsystem we can always do plain old find and replace for --ilo

Hey @GGKapanadze I love the direction here, this all looks fine, I'm still undecided about the prefixes. However, I was noticing that the main css bundle has gone from 271kb in the current release to 421kb in this one. I didn't have the chance to dig in and figure out why that was happening, do you have any ideas?

Hey @GGKapanadze actually I had a look and it seems like the root variables are all getting prepended to each class for some reason

I'll try to fix it, I think it's about import resolution

Hey @GGKapanadze before I look at this again, did you have a second to look at the import resolution?

@justintemps The issue was caused by the scss variable import inside the mixin, which importing the full :root definition for every file that was using the mixins.scss

Old

New

Hey @GGKapanadze , I was trying to test this out in bb8, but then I hit this error: https://github.com/international-labour-organization/designsystem-site-builder/issues/6
Could you have a look?

Fixed the e2e test for tooltip. :')
Cc :- @justintemps @GGKapanadze


amplify env add: Cannot read property 'idToken' of undefined

Describe the bug
I updated my CLI from 4.36.2 to 4.37.0 and ran amplify env add to create a new environment. It threw an error: amplify Cannot read property 'idToken' of undefined. When reverting back to 4.36.2, the same command worked.
Amplify CLI Version
4.37.0
To Reproduce
Attempt to run amplify env add to a new environment name ("N" to would you like to use an existing env)
Expected behavior
It shouldn't throw an error.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Mac OS Big Sur

Node Version. You can use node -v to check the node version on your system
v12.16.3

Additional context
Add any other context about the problem here.

Hey @blydewright, I've been unable to reproduce so any additional information is appreciated.

Can you show me the full output in the terminal?
Have you enabled AdminUI in the console?
Are you using an AWS profile?

Hi @jhockett,

I'll finish what I'm busy with and upgrade again to see if I can reproduce that output.
Yes, as a matter of fact I enabled that for the first time today
Yes

@blydewright thanks for the information! I was able to reproduce and I marked the issue as a bug. The Amplify CLI is seeing that AdminUI is enabled and is expecting the new authentication method.
As a workaround until we can roll out a fix, you can use the new authentication method:

cd to your project's root directory
amplify console -> choose Amplify console
from the side bar click on "Admin UI management"
send yourself an invite and select "Full access"
run amplify configure --appId <YourAmplifyAppId> --envName <AnEnvName>,
login with the temporary credentials that were sent to the email address you picked above
click "yes" in the browser when it asks to authenticate with the Amplify CLI

Then you should be able to create an environment like you normally would with amplify env add. Sorry to make you jump through hoops just to get unblocked. Please let me know if you have any questions about any of the steps above.

Thanks for the workaround @jhockett :) You can also advise people to revert back to 4.36.2 - that seems to work as the admin UI feature is probably not recognised anyway. I'll keep my eye out for the fix. Also keen to see what this Admin UI is all about ;)

@jhockett  no dice, A valid backend environment was not specified. in the ui, and I get the same   Failed to fetch Amplify Admin credentials init failed Error: TypeError: Cannot read property 'idToken' of undefined at Object.getAwsConfig (/Users/amlcodes/.nvm/versions/node/v15.3.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/src/configuration-manager.js:761:13) error still when running amplify env add

@amlcodes I have the same issue with my Admin UI interface in Console. Doesn't work, busy dealing with support on this. Have a feeling it isn't ready for use yet ;)
I reverted back to 4.36.2 for now and all works fine.

Unfortunately @jhockett 's workaround didn't work for me (I always get a Failed to communnicate with the Amplify CLI. error in the Amplify Console). (BTW: that typo in the error message really is there and might need fixing as well)
But revering to 4.36.2 solved it.


2 fixes which I need: E2506 from Delphi 2010 compiler, Indy makes up useless authentication data.

Doesn't compile in Delphi 2010 anymore. Using RestJsonUtils from SuperObject even in the implementation section breaks D2010's Generics leading to a E2506 pointing to RestJsonGenerics: Result := ctx.AsType(SuperObject.SO(text));
One can't supply (basic) authentication data by SetHeaders in a way that would survive a 40x error from the server using the Indy implementation. Indy tries to use it's user name and password fields to construct an authentication object to retry the connection  and stores that. This data supersedes every authentication header set by SetHeaders if it exists. As user name and password are both '' a (hopefully) invalid basic auth header is sent instead of the one supllied with SetHeaders.

Now I have merge permissions on this repository. I will start to merge/close all pull requests.
As I commented before, will be better send only your changes.


Lint/NonLocalExitFromIterator false positive

I think I'm get a false positive from Lint/NonLocalExitFromIterator in the below code:
VCR.configure do |c|
  # ...

  def server_cookies(response)
    raw_cookies = response.headers['Set-Cookie']
    return if raw_cookies.nil? || raw_cookies.empty?

    raw_cookies.map { |ck| parse_server_cookie(ck) }
  end
end

Expected behavior
No flags should be raised.
Actual behavior
It's flagged with:
W: Non-local exit from iterator, without return value. next, break, Array#find, Array#any?, etc. is preferred.

Curiously, this only seems to happen when there's a block argument specified in the VCR.configure block – try omitting |c| to see what I mean.
Steps to reproduce the problem

Create a file with the above code snippet as its contents
Run Rubocop on it

RuboCop version
Include the output of rubocop -V. Here's an example:
bundle exec rubocop -V
0.46.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-linux)

Nice find @zachlatta! I think the reason we've missed this is that it's a pretty uncommon (at least to me) edge case. But there's no reason why we couldn't account for this, that I can see. 😀

Yeah, that should be easy to fix.


Question: How to get qty of "end" symbol in a market order?

When I sell XRP to ETH (or BTC, BNB...) with a market order and get the result back that order succeeded I get amount of my start symbol (in this case XRP), but what I need is to get end symbol. Say, I sold 100 XRP to ETH, what's the amount of ETH I got from 100 XRP?
I checked Trades API as well but still I miss that info. In Binance - Order History I see that amount, ex. 0.04502440 ETH (for 40 XRP).

@betimd, the returned Order should have one or more Fill instances in the Order.Fills property with Fill.Quantity.
Reference: Official Binance API


SphereUV -> SphereUv

This PR renames SphereUV as SphereUv in order to adhere the Rust naming conventions.

bors r+


Support a #[serde(validate = "some_function")] attribute on fields

I occasionally run into fields in a serde-deserializable type that require some extra validation after deserialization, for example "a u32 >= 2". The way to do this now is to make a custom deserialize function that deserializes and then does the check:
#[derive(Deserialize)]
struct Thing {
    #[serde(deserialize_with = "validate_foo")]
    foo: u32,
}

fn validate_foo<'de, D>(d: D) -> Result<u32, D::Error>
    where D: de::Deserializer<'de>
{

    let value = u32::deserialize(d)?;

    if value < 2 {
        return Err(de::Error::invalid_value(de::Unexpected::Unsigned(value as u64),
                                            &"a value at least 2"));
    }

    Ok(value)
}

However, it'd be nice if it were possible to specifically tell serde to use the standard deserializer but apply a validation function after the field is deserialized:
#[derive(Deserialize)]
struct Thing {
    #[serde(validate = "validate_foo")]
    foo: u32,
}

fn validate_foo<E>(v: &u32) -> Result<(), E>
    where E: de::Error
{
    if value < 2 {
        Err(de::Error::invalid_value(de::Unexpected::Unsigned(value as u64),
                                     &"a value at least 2"));
    } else {
        Ok(())
    }
}

That's very similar to one of my crate (https://github.com/Keats/validator) with a wip new version at https://github.com/Keats/validator/pull/27

I guess this is a duplicate of https://github.com/serde-rs/serde/issues/642.

I don't think this is a duplicated of #642. This is for a field attribute and that one is for a container attribute. They are useful in different situations.

@sfackler, how would you feel about this:
#[derive(Deserialize)]
struct Thing {
    #[serde(and_then = "validate_foo")]
    foo: u32,
}

fn validate_foo<E>(v: u32) -> Result<u32, E>
    where E: de::Error
{
    if value < 2 {
        Err(de::Error::invalid_value(de::Unexpected::Unsigned(value as u64),
                                     &"a value at least 2"));
    } else {
        Ok(value)
    }
}

That would enable both validation and post-deserialize transformation, which is something I've found myself reaching for in the past.

I think this one is superseded by #1550 and can be closed in preference to it

I think this issue should be closed as it seems there is a solution with and_then. Otherwise, it makes the impression it requires work....

@avkonst it appears that #1858 was closed without being merged, meaning this particular issue probably will need to instead reference a documentation update.
@dtolnay your logic for individual fields in this comment makes sense; the place where I've found myself reaching for and_then is with complex objects such as the OASIS CACAO specification. That has requirements such as:

If both valid_from and valid_until are specified, then valid_from MUST be less than valid_until
All variables mentioned in a workflow step MUST be defined either in the playbook's variables or in the step's variables

If I'm also providing some other avenue for making these objects, then I likely already have a function with the signature validate(Playbook) -> Result<Playbook, AnErrSerdeCanWorkWith> in existence, and I'd like to have serde run the struct through that.
An alternative would be a custom Deserialize impl on the Playbook struct, like this:
pub struct Playbook {
    // fields
}

impl<'de> Deserialize<'de> for Playbook {
    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
    where
        D: serde::Deserializer<'de>,
    {
        #[derive(Deserialize)]
        struct UncheckedPlaybook {
            // fields...
        }

        impl TryFrom<UncheckedPlaybook> for Playbook {
            type Error = AnErrorThatImplsDisplay;

            fn try_from(_: UncheckedPlaybook) -> Result<Self, Self::Error> {
                // elided...
            }
        }

        UncheckedPlaybook::deserialize(deserializer)?.try_into().map_err(D::Error::custom)
    }
}

That works, but when the struct in question has a lot of fields that's code drift waiting to happen. I've considered making a crate to solve this that would allow the generation of a replica struct with different serde attributes, but that seems like overkill for the common case.

elided

But this check is not lazy! Meaning, it will be performed only when you finish to serialize UncheckedPlaybook.  One might want a check to accrue in the middle of the serialization. For example, ELF magic for elf binaries or checksum.
pub struct Data{
data: T
}

My serialized Data object has a MAGIC, md5sum and a total_size virtual fields before the data field.
I want my decerializer to check the MAGIC and the md5sum before it serializes anything.

@dtolnay 's comment https://github.com/serde-rs/serde/pull/1858#pullrequestreview-575089757 provides a quick example of validation using deserialize_with. I feel the need to link it here to help those who's looking for a solution.
Maybe this should be summed up and added to the documentation.

Maybe this should be summed up and added to the documentation.

Yes, I agree. I feel there are several issues and prs that we could close out as "good enough" with some existing solutions documented.

I just realized that one can use const generics for integer validators; a bit much to declare but pretty nice to use.

I feel like conflating parsing (aka deserialization, what serde was built for) with value validation is not the right way to go. I know it's tempting to combine the two operations, it'd be convenient, but convenience invites complexity. Validation could easily be done as a second step, on the serde deserialized data structure. @Keats's validator crate seems nice (my understanding is you call .validate() on the struct after serde successfully deserialized).

Personally, I think that what people are asking for as "validation" is more a subset of parsing.
Parsing a sequence of UTF-8 bytes into an XML DOM doesn't become "validation" just because < or > or & are only valid in certain positions, and an argument can be made that one shouldn't need to newtype everything just to enforce that an integer named week_of_year only has a certain valid range.
...that said, my approach generally is to newtype everything since that both lets me push validation into Serde and allows the compiler to prevent conflation of values.

Parse, don't validate is probably one of my top 3 favorite blog posts of all time
Another reason I favor newtypes over a second "validation" step is that without newtypes you can write this code:
let x = Stuff {
    between_1_and_3: 200,
};

which is clearly wrong. But with my example above, you can do either of:
between_1_and_3: BoundedUsize::clamp(200),

between_1_and_3: BoundedUsize::new(200).expect("out of bounds"),

And you could feasibly write a macro like this that would cause a compile error instead of a runtime one:
between_1_and_3: bounded_usize!(200),

I think the common name for this pattern is "correct by construction", which seems more in line with the way the rest of Rust works.

"Parse, don't validate" is ideal in many situations, but if you need serde with validation, check out SerdeV ( https://crates.io/crates/serdev ), essentially serde with #[serde(validate = "some_function")] container attribute. This allows you to validate automatically in deserialzing with no boilerplate.


Web UI fix missing units psm2

Description
This PR will add the units (W) and the correct minimum and maximum values to the power threshold, lower threshold and higher threshold textboxes in the channel 7 for HmIP-PSM-2 and HmIP-PSM-2 QHJ devices.
Related Issue
This should solve #2746.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Possible Drawbacks
There might be a problem with the values. It could be that they require conversion (see #2746). During my tests, there was no need for conversion.
Verification Process
I have used the lower and higher values to values around the current power consumption. I have not run in any problems.
Release Notes

Fixed an issue where incorrect min/max values and no units where displayed in channel 7 for HmIP-PSM-2 and HmIP-PSM-2 QHJ devices.

Contributing checklist

[ ] My code follows the code style of this project.
[X] I have read the CONTRIBUTING and LICENSE document.
[X] I fully agree to distribute my changes under Apache 2.0 license.

@jens-maus: thank you and please excuse my mistake. I will be more careful next time.

No problem. The procedure how to create a proper WebUI patch is unfortunately quite cumbersome and also undocumented...


Run CI on generated project

We should run formatters, linters and other CI tools on generated project, because at now it's just failing on invalid syntax.
This also add possibility to add tests for template.

CI was removed as workaround in e09ff32acdc644f3d12d78d86d3bf21d2c526776.


Support VS2013/VS2015 mixed environment

Currently, Prig installs fixedly self to the latest version if plural Visual Studios are found in the same environment. Thus, in the environment that Visual Studio 2013 and 2015 are installed, Prig cannot be used in Visual Studio 2013.

Extent of Impact
ChocolateyUninstall.ps1
https://github.com/urasandesu/Prig/blob/ded24d8cfeb24ce88ef3f4c1e79f110c6639344b/Chocolatey/tools/ChocolateyUninstall.ps1#L32
This is the processing that calls vsixinstaller of the latest installed Visual Studio to uninstall VSIX.
DRAFT CORRECTION
Won't fix. Because vsixinstaller will uninstall from all products that contain the extension with the corresponding identifier if /skuName and /skuVersion are not specified.
MachineWideInstaller.cs
https://github.com/urasandesu/Prig/blob/3225ad45737b6b9f53d5c989ce68e3b2261c5f9d/Urasandesu.Prig.VSPackage/Models/MachineWideInstaller.cs#L172
This is the processing that specifies the Test Explorer of the latest installed Visual Studio as the target package of Prig.
DRAFT CORRECTION
Execute prig install against the Test Explorer of all installed Visual Studio.
Add-PrigAssembly.ps1
https://github.com/urasandesu/Prig/blob/778e16f3de50f52abb1df478a60fa48aa6ae7fea/Urasandesu.Prig.VSPackage/tools/Add-PrigAssembly.ps1#L163
This is the processing that generates the command that contains the path of Developer Command Prompt of the latest installed Visual Studio to build Prig Assembly.
DRAFT CORRECTION
Generate the path of Developer Command Prompt of Visual Studio from Common Macros. $(DevEnvDir) is the first candidate(See also Common Macros for Build Commands and Properties).


Noisy log message :O

Expected Behavior
When looking at logs from running Tasks (e.g. using tkn pr logs -f to tail logs from a PipelineRun😉 ) the only output should be:

Informative logs from the containers inside the executing pods
Error logs if actual errors occur
Logs from the user provided pods

Actual Behavior
Whenever I look at logs from a running Task I see something like this over and over:
{"level":"warn","ts":1564628323.275128,"logger":"fallback-logger","caller":"logging/config.go:69","msg":"Fetch GitHub commit ID from kodata failed: open /var/run/ko/HEAD: no such file or directory"}

It's not clear to me where exactly this is coming from or what it means.
Steps to Reproduce the Problem

Run a Pipeline or Task, e.g. apply an example
Get the name of the run (e.g. tkn pipelinerun list
Look at the logs, e.g. tkn pipelinerun logs <pipelinerun name>

It's the logger we use from knative, it tries to enrich log lines with a commit ID from kodata - see https://github.com/knative/pkg/pull/506.
I understand that we use the knative logger for Tekton services, but perhaps we should be using a different logger for Tasks and Pipelines.

Thanks for this @tomgeorge ! I think #2000 solves this issue so I'm going to close it but please re-open if I'm wrong :D


Broken Link

The link to DEVICE_REGISTRY_PROPERTY is broken. The correct link is https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/wdm/ne-wdm-device_registry_property

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

ID: b509b635-f6c0-1649-847d-a2c349fadb22
Version Independent ID: d7e77199-f0e6-49d5-e9c3-f4011669adc1
Content: WdfDeviceQueryProperty function (wdfdevice.h) - Windows drivers
Content Source: wdk-ddi-src/content/wdfdevice/nf-wdfdevice-wdfdevicequeryproperty.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks @vkamath1 and @hsebs. This fix is up.


Distributed Run Code Error

Hi @raymin0223,
I have found that when I run code using multiple GPUs, the following errors occur:
Traceback (most recent call last):
File "main.py", line 563, in
main()
File "main.py", line 531, in main
loss, acc = train(train_loader, model, classifier, projector, criterion, optimizer, epoch, args, scaler)
File "main.py", line 384, in train
mix_images, labels_a, labels_b, lam, index = model(images, y=labels, patch_mix=True, time_domain=args.time_domain)
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 172, in forward
return self.gather(outputs, self.output_device)
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 184, in gather
return gather(outputs, output_device, dim=self.dim)
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 86, in gather
res = gather_map(outputs)
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 81, in gather_map
return type(out)(map(gather_map, zip(*outputs)))
File "/home/ygh/anaconda3/envs/pytorch20/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 81, in gather_map
return type(out)(map(gather_map, zip(*outputs)))
TypeError: 'float' object is not iterable
I think it's caused by the parameter 'lam', so I have rewritten the code as follows:

When mixing patches, I put the lam parameter into the tensor. When calculating the contrast loss, I perform the lam.mean() operation.
Will this affect the calculation of contrast loss? If so, what should I do to solve the problem?

Hi @iffuture799,
As all my lab servers have shutdowned today, I will see as soon as possible.

I also found that there is a bug on lambda, as model = torch.nn.DataParallel(model) makes the outputs from model to be gathered.
Thus, using lam.mean() will be dangerous because we cannot guarantee that lambda from multiple GPUs will be similar (they were different, actually).
Solutions may be using DDP or including loss calculation in model.
I haven't checked DP with PatchMix-CL, so sorry for this bug.


Add support for func-decl kind (Z3_decl_kind)

As part of "next steps" after Lund, we are going to do someting along the lines of how TROpcode deals with the pool.


Add My Projects page

What does this PR do and why?
Adds a new top level page for viewing a person's joined projects, as mentioned on notion and issue #256 .
I setup the navbar link to only display if the user has a wallet connected, and to send them to the Projects page if they disconnect while on My Projects.
On Notion, there's also requests for including additional information for each project. I wasn't sure what designs for that might look like, so I left them off this PR for now.
Screenshots

Acceptance checklist

[x] I have evaluted the Approval Guidelines for this PR.

hey @0xzachvai awesome PR! and thanks for the improvements to the Participant parsing and the in graph query operator. couple thoughts here:

we should be able to simplify the query by using the Participants property of the Project graph entity, so we don't have to make two sequential queries. that said, we've recently discovered some issues with how participants are being indexed. i'm hoping to have this solved once the new subgraph version is launched for the v1.1 contracts (in the next 1-2 weeks), but to avoid any added confusion in the app let's hold off on shipping any new features that rely on that until then

from a layout perspective, i'd prefer to see "My projects" as a tab on the existing projects page, rather than a new page. it'll keep the navbar smaller, and i suspect most people won't need sub categories for active/archived. on a related note, i'd be in favor of including an "archived" tag on projects in the list view

Hey @peripheralist , thank for the feedback!

I was trying do this in a single query, but I couldn't personally figure out a way to do so with the current graph setup. Project_filter doesn't support filtering by participants, only creator, and doing this (is this what you meant?):

query {
  projects {
    participants (where: {
      wallet: "0x01f0530e47f85c1d1f71823c47e0f0ca0415c1f2"
    }){
      ...
    }
    ...
  }
}

still returns all projects, regardless of whether it contains a matching participant entity. Let me know if I'm just missing something, I'm still relatively new to GraphQL.

Completely agree, I'll make these changes.

Looks like you're right @0xzachvai, I didn't realize the graph still doesn't support nested queries. Your solution works great for now. Let's still hold off on shipping this until the Participants indexing is solved—this is a priority to have fixed before v1.1 is shipped, so hopefully we can get this out with the new app version in 1-2 weeks!

Okay, My Projects is now a tab under the Projects page. I set it up to only show when a wallet's connected, but I'm not sure that's better than just showing the tab with an empty grid (or hint text), so open to suggestions. In the process, I extracted the tab function into two components, TabGroup and Tab - figured they could be reused elsewhere, especially if the Project page is redone like recently mentioned on the Notion.
Also added that Archived tag to project cards, with a nifty little tooltip - can take a look in the preview build under Archived tab.
Lmk if you catch anything else I can include or improve - otherwise, I'll just keep this branch up to date with main until the indexing stuff is ready.

Thanks for the patience here @0xzachvai 😄 The new subgraph and app version were shipped yesterday, so we're clear to dive back into this! Looks like we made some merge conflicts for you. Could you try getting this rebased against the latest main so we could play with it and get some design feedback? Let me know if the rebase gets too dense, I'm happy to help out

hey @0xzachvai, are you interested in picking this up again? if not no worries, i'll take this and run with it by the end of the week if i haven't heard back. thanks for the great work so far!

Hey! It's been a really busy week offline, but I should be able to pick this back up today/tomorrow. Might need a bit of help rebasing the projects/index.ts, I'll ping you if anything

all good! let me know if i can help

only closing this because it's been rebuilt in a more current PR. appreciate your help with this @0xzachvai !


B 20685 integration

Agility ticket
Summary
Is there anything you would like reviewers to give additional scrutiny?
this article explains more about the approach used.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:

New files have been created under one of the legacy directories
(src/shared or src/scenes). Please relocate them according to the file structure described here.
View the frontend file org ADR for more information

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 0bff54748d46ffc68950a95550f68c6976b4e333

Please provide screenshot of the ANDI report for at least the page with the Wounded Warrior text. I believe you don't have to worry about the scroll bars because you didn't change the size of the scroll bar itself it shouldn't fall below the size threshold for 508 compliance.

Label no longer wraps and horizontal scroll bar present when window is resized.


 I ran go test ./... and i got ../../resources/Endpoints.json: no such file or directory and the tests failed.

we should chat about what type of testing makes sense and is feasible at this point.


auto add storyLoader

Ty @wolverineks I thought a few times to add it, and never did I'm glad that you did it.


#issue145

Added the screen of successful tracking and about the installation, I am running it from the source code, so I kept a screenshot of the local repo.
Looking forward to doing more. Cheers!

I've merged a PR, so please update your fork first.

Here I am enclosing the screenshort of successfully execution of simple tracking command .

@csharp489  create your own PR about issue #145 to earn your points.


Don't always call preventDefault on DOM events handled by .NET

@page "/"
@using Microsoft.AspNetCore.Blazor

Hello, world!

Welcome to your new app.

@functions {
    private void Dragstart(UIDragEventArgs a)
    {
        Console.WriteLine("Start");
    }

    private void Dragend(UIDragEventArgs a)
    {
        Console.WriteLine("end");
    }
}

If the vents are bound to a javascript function they work properly, but binding to C# dont't.  Having this at homepage show the bug

Probably related to #803

This was implemented in 0.4.0.


3D plotting

3D plotting does not work from installation, and is not documented/controllable.
Team WPI has code that can be modified to work, and this issue will be referenced on that pull request.

Thanks! That would be useful.
Be warned that several changes are forthcoming as @vraman and I add code that was used during the evaluation at ICRA 2016. Feel free to open your pull request (PR) anytime. I can work with you later if rebasing is motivated as part of the PR review process.

https://github.com/fmrchallenge/fmrbenchmark/pull/14