Messages based on Commands are not fed into stream

Hey Dag,
I am trying to use Elmish Stream in the Safe Confplanner and I am banging my head for the last couple of hours because it is not working as (I) expected.
In the Confplanner I need to work with commands (so no mksimple) and I have a couple of situations where I need to create messages from commands within the update function. For this I am using the elmish Cmd.OfFunc.result MyMsg et al functions. It seems that those message are never fed into the message stream (it is somehow circumvented and the messages go directly into update). Is this expected behaviour? I can create a minimal repro but I wanted check first if this is the way it is supposed to be.

I think the reason is that the new dispatch function is only put into the view (https://github.com/dbrattli/Elmish.Streams/blob/c8341378cd4d72ba04172a5fbecac2382e3d1dc2/Fable.Elmish/src/Program.fs#L84). That means that only messages generated by the view are fed into the stream.
Maybe it would be better to use the withSyncDispatch function to get a dispatcher that works everywhere (https://github.com/elmish/elmish/blob/d3770c3d94aa4542e2debab06f03756ee8d050f5/src/program.fs#L111)
In the current state it is (imho) not possible to add elmish.streams incrementally or partially to an application that already uses commands.

I spend almost two days figuring it all out. I think at the moment this is not possible because syncDispatch is overwritten when Program.map is called. I will file a bug over there

@dbrattli ok now that Eugene has closed the issue I am pretty clueless how to go on from here. Maybe I am missing something obvious and there is a good solution. I am still convinced that it is totally worthwile to be able to react to messages created by commands. What do you think?

Hi @rommsen. I will look into this, but I might not have any real amount time until the weekend.

Hi @dbrattli . I think most of the stuff can in principle be done as you said. My point is that the best way to raise adoption and usage of Elmish.Streams is by allowing it to be introduced gradually. I love the library but people are pretty resistent. Another thing to learn. Pretty much all real elmish apps use commands. I think that it is not very good to tell people that only messages that are generated in a view can actually be fed into the msg stream. Imagine people already doing their http requests etc with commands and now they want to use the messages that are created when such a request returns within the event stream. It is just not possible and imho this is a very severe limitation. People needed to actually change the way they write elmish apps completely and I think this is not going to happen.
Does this make sense?

Yes, I get the point. I'll look into it.

Ohh yeah, it happend in my case. It took me some days to get used to it, but I'm really happy that Cmd's are now gone (e.g. composition of updates got much better, because it's just a simple fold when you only have the model). I am not completely happy with how the stream function works (can't remember exactly, but it had to do with the fact that the function is called every time the model changes), so I rewrote it a bit. But the overall idea is brilliant IMO.

And when you feed messages coming from update back into the stream (where they came from) then you can very very easily get endless loops, which are also hard to understand in the beginning and hard to avoid.

Hello @johannesegger.  I would like to better understand what you meant when you said

...I'm really happy that Cmds are now gone (e.g. composition of updates got much better, because it's just a simple fold when you only have the model).

Are you saying that instead of having update return the new model and a list of commands, you passed into update a function that accepts a command (and in production would put it into some stream), so then update now directly returns just the new model and indirectly returns the commands by invoking the given function on them?

@bender2k14 not directly. With streams I don't use Cmds at all. update has the signature 'msg -> 'model -> 'model instead of 'msg -> 'model -> 'model * Cmd<'msg>. The side effects that are usually performed by Cmds are now performed as part of the stream. Mocking those side effects - e.g. in tests - might become harder but I consider those tests to be too trivial to create anyway (probably an unpopular opinion).
I found that the coupling between a state update and deciding what side-effect to perform sometimes makes things unnecessarily complicated (e.g. throttling/debouncing is not trivial with Cmds but built into streams).

Fable.Reaction (new name) does not depend on Elmish anymore and will use it's own message loop. Thus, components built on Fable.Reaction will not directly be able to handle stuff made from Commands. The way to communicate with Fable.Reaction components is currently to use properties (Props). You can pass your (pre-composed) dispatch function to have a component communicate back.


[GSSOC'23] Does not show error for wrong email ID in login page

Reproduction steps
In the Login page, it does not show error for invalid email ID and also password length is not set. Please assign me this issue under GSSOC'23.

an
Expected behavior
It should show an error for an invalid email address and also password length constraint should be included.
Actual Behavior
Now it does not show an error for an invalid email ID.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Sure go ahead @Mansi209aps


[New game]:  Multiplayer Online Battle Arena

🎮 Game Request
I want to develop a real-time multiplayer game where players control heroes with unique abilities and battle against each other in teams.
Point down the features
The game would include features like:

Real-time multiplayer with WebSockets.
Character abilities and leveling system.
Dynamic maps with various terrains.
AI-controlled minions and bosses.
In-game chat system.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Sambit-Mondal , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4282

Hey @Sambit-Mondal ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Sambit-Mondal !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


CHEBI:90359 please correct formula

Dear ChEBI Team,
Could you please correct the formula for this compound?
While already checked, the formula should anyway be C27H50O15, as I mentionned it at time of the submission (in the definition field). This is a monoacyl known compond where only the sn- position is unknown, so the 2R.
Thank you very much in advance.
Kinds regards,
Anne
Reported by: anneniknejad

Dear Anne,
Sorry for the problems with this one - hopefully it is now fixed. I reckon that the compound should have 14 oxygens rather than 15 (5 from each galactosyl group, 3 from glycerol, and one for the dodecanoyl group, giving a molecular weight of 598.68). Let me know if I've got it wrong.
Kind regards,
Gareth


Error on patrol test " No .xctestrun file was found"

Describe the bug
When I run command:
patrol test --flavor qa I get "No .xctestrun file was found at .../build/ios_integ/Build/Products
My tests were successfully running few weeks ago, with different versions and patrol drive.
To Reproduce
Steps to reproduce the behavior:

Run patrol test --flavor qa
Logs:

No device specified, using the first one (iPhone 14 Pro)
Every test target will be run 1 time(s)
• Building app with entrypoint authentication_test.dart for iOS simulator...
✓ Completed building app with entrypoint authentication_test.dart for iOS simulator (37.7s)
• Running app with entrypoint authentication_test.dart for iOS simulator on simulator iPhone 14 Pro...
Error: No .xctestrun file was found at .../build/ios_integ/Build/Products
See the logs above to learn what happened. Also consider running with --verbose. If the logs still aren't useful, then it's a bug - please report it.
FAIL  authentication_test.dart on iPhone 14 Pro
Expected behavior
The test should run.
Device information

Device: iPhone 14 pro simulator
OS version: iOs 16.2
Does it also both on physical and virtual devices? [Yes, no, didn't check] Didn't check

Additional information
Flutter 3.7.6 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 12cb4eb7a0 (3 weeks ago) • 2023-03-01 10:29:26 -0800
Engine • revision ada363ee93
Tools • Dart 2.19.3 • DevTools 2.20.1
patrol: ^1.0.8
patrol_cli v1.1.0

Hi @AlexisDeslandes, sorry for the problems!
Can you show me the output of ls -lAh build/ios_integ/Build/Products?

It might got broken by #1090

Sorry mistake I forgot that,
the file is here.
ls -lAh build/ios_integ/Build/Products  total 8 drwxr-xr-x@ 82 adeslandes  staff   2.6K Mar 21 16:01 Debug-qa-iphonesimulator -rw-r--r--   1 adeslandes  staff   439B Mar 21 16:40 qa_iphonesimulator16.2.xctestrun

Looks good.
One last ask – could you run patrol test --flavor qa --verbose and show the log output? I'm interested in the part between invocations of $ xcodebuild -showsdks -json and $ xcodebuild test-without-building -xctestrun ... (they are blue, so should be easy to find - but paste the whole log anyway). Thanks!

No Rosetta is not enabled. Yes i'm on apple silicon.
$ xcodebuild -showsdks -json
Assuming SDK version 16.2 for iphonesimulator
Looking for .xctestrun matching qa_iphonesimulator16.2-*.xctestrun at /Users/adeslandes/work/enterprise/build/ios_integ/Build/Products
Error: No .xctestrun file was found at /Users/adeslandes/work/enterprise/build/ios_integ/Build/Products
#0      throwToolExit (package:patrol_cli/src/base/exceptions.dart:7:3)
#1      IOSTestBackend._xcTestRunPath (package:patrol_cli/src/ios/ios_test_backend.dart:240:7)
<asynchronous suspension>
#2      IOSTestBackend.execute.<anonymous closure> (package:patrol_cli/src/ios/ios_test_backend.dart:152:15)
<asynchronous suspension>
#3      DisposeScope.run (package:dispose_scope/src/dispose_scope.dart:44:7)
<asynchronous suspension>
#4      IOSTestBackend.execute (package:patrol_cli/src/ios/ios_test_backend.dart:137:5)
<asynchronous suspension>
#5      TestCommand._executorFor.<anonymous closure> (package:patrol_cli/src/commands/test.dart:273:9)
<asynchronous suspension>
#6      _NativeTestRunner.run.runTestsOnDevice (package:patrol_cli/src/test_runner.dart:243:15)
<asynchronous suspension>
#7      Future.wait.<anonymous closure> (dart:async/future.dart:522:21)
<asynchronous suspension>
#8      _NativeTestRunner.run (package:patrol_cli/src/test_runner.dart:258:5)
<asynchronous suspension>
#9      TestCommand.run (package:patrol_cli/src/commands/test.dart:182:21)
<asynchronous suspension>
#10     CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#11     PatrolCommandRunner.runCommand (package:patrol_cli/src/runner/patrol_command_runner.dart:284:18)
<asynchronous suspension>
#12     PatrolCommandRunner.run (package:patrol_cli/src/runner/patrol_command_runner.dart:223:18)
<asynchronous suspension>
#13     patrolCommandRunner (package:patrol_cli/src/runner/patrol_command_runner.dart:47:20)
<asynchronous suspension>
#14     main (file:///Users/adeslandes/.pub-cache/hosted/pub.dev/patrol_cli-1.1.0/bin/main.dart:6:20)
<asynchronous suspension>

See the logs above to learn what happened. Also consider running with --verbose. If the logs still aren't useful, then it's a bug - please report it.
 FAIL  authentication_test.dart on iPhone 14 Pro

xcodebuild test-without-building -xctestrun ... is never called.

Okay, it's almost certainly a bug.
The fix is to change these lines to allow for no explicit specification of architecture.


Visual Studion code does not keep the old search results in the search pane or provide a drop down list of previous search phrases.

VSCode Version: Code 1.5.2 (66f37fd2a99eb9d628dd374d81d78835b410c39b, 2016-09-12T13:30:03.456Z)
OS Version: Windows_NT ia32 6.1.7601

Steps to Reproduce:

Search for any symbol using the search option in VS code IDE.
Search results will be shown in the search results pane.
Search for another symbol.
Search results for the second search will replace the search results of the first search.
It is very annoying because I, as a developer, at many times am interested in following more than one symbol.
Keeping the previous search results in the same search results pane is very much needed feature.  Newer search results may be listed at the top and previous search results may be listed below the newer results in chronological order.
Also please provide a dropdown menu for the search results listing the previous search strings.  Keeping all the search strings in some editor and copying them again and again for searching is very annoying and time consuming.

Search history is implemented in this - #5508. You can try this by downloading our insiders build
With this feature you can always navigate between previous and current results. IMO, showing search results of current and previous search would in the same view might be confusing.
@thomas-anil Let me know if this suffice your requirement?

Closing this as there is no feedback from last 14 days.

Hi Sandeep,
Thank you very much for the support.  I tried to download the insiders build. But I am not able to visit the page. the system says "
Sorry, but we’re having trouble signing you in.
We received a bad request."
I am able to login to my outlook mail account. Can you please check or help me fix this. Then I will checkout the build for the feature I requested.
or
Can you please send me a direct link to the build or some other way of accessing the build?
Thank you very much
regards
Anil Thomas

From: Sandeep Somavarapu<EMAIL_ADDRESS>Sent: Monday, September 19, 2016 1:33:40 PM
To: Microsoft/vscode
Cc: thomas-anil; Mention
Subject: Re: [Microsoft/vscode] Visual Studion code does not keep the old search results in the search pane or provide a drop down list of previous search phrases. (#12184)
Search history is implemented in this - #5508https://github.com/Microsoft/vscode/issues/5508. You can try this by downloading our insiders buildhttps://vscode-builds.azurewebsites.net/insider
With this feature you can always navigate between previous and current results. IMO, showing search results of current and previous search would in the same view might be confusing.
@thomas-anilhttps://github.com/thomas-anil Let me know if this suffice your requirement?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/12184#issuecomment-247932041, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVOUUWA1vf8c7iIsfzKK3yC3l1-g0HmRks5qrkHcgaJpZM4J_jsi.

@thomas-anil Which link are you accessing for insiders?

IMO, showing search results of current and previous search would in the same view might be confusing.

I would very much like to have search results history in different panes, as I was used to in Netbeans for example. May be as an option for people who don't want that.

Duplicates #16488


Error when user logs in for the first time

When a user logs in for the first time using a non-local auth provider an error is received.
2018/07/13 16:40:30 [ERROR] SAML: User does not have access Operation cannot be fulfilled on users.management.cattle.io "user-2bhg5": the object has been modified; please apply your changes to the latest version and try again

rancher/rancher:master at 25 july
I was able to confirm that no error appeared with non-local auth providers


added blank-string matcher

#153

@rultor merge

@rultor merge

@graur OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@graur Done! FYI, the full log is here (took me 3min)


Doesn't calculate colors properly when font weight is changed

The problem is that when user's combination is accessible, we still show all the 3 preview blocks. We need to show only the original. This will be addressed by #5.


Secret not found error when optional with webhook

Describe the bug:
When an optional secret doesn't exists in secrets, there is an error thrown.
Error creating: Internal error occurred: failed calling webhook
"pods.vault-secrets-webhook.admission.banzaicloud.com":
an error on the server ("{\"response\":{\"uid\":\"***\",\"allowed\":false,\"status\":
{\"metadata\":{},\"status\":\"Failure\",\"message\":
\"secrets \\\"secret-***\\\" not found\"}}}") has prevented the request from succeeding

Expected behaviour:
Because secret can be optional in Kubernetes definition type no error should be thrown
// SecretKeySelector selects a key of a Secret.
type SecretKeySelector struct {
	// The name of the secret in the pod's namespace to select from.
	LocalObjectReference
	// The key of the secret to select from.  Must be a valid secret key.
	Key string
	// Specify whether the Secret or its key must be defined
	// +optional
	Optional *bool
}

I think this line causes the error.
Steps to reproduce the bug:
Create a deployment with an optional secret not created into the environment.
containers:
- name: container-name
  image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
  envFrom:
  - secretRef:
      name: my-optional-secret #the secret not existing
      optional: true

Regards
/kind bug

The fix has been merged to master, could you please try it with the latest master image once it gets published @callaertanthony ?


v59 - Questions and Suggestions

What do you think about this fixed "data:application/json,{}" value ?

/* 0360: disable new tab tile ads & preload & marketing junk ***/
used:

user_pref("browser.newtabpage.directory.source", "data:text/plain,");
new (to prevent browser console spam):

user_pref("browser.newtabpage.directory.source", "data:application/json,{}");
https://github.com/MrAlex94/Waterfox/issues/496#issuecomment-376914186

"datareporting.healthreport.infoURL" - value (URL link) is not empty

Although the main option is deactivated, clearing URL value would be welcome.
I think, the setting can be placed at the corresponded section (0333):
/* 0333: disable health report
 * [SETTING] Privacy & Security -> Firefox Data Collection & Use -> Allow Firefox to send technical... data ***/
user_pref("datareporting.healthreport.uploadEnabled", false);
user_pref("datareporting.healthreport.infoURL", "");

infoURL (default value): https://www.mozilla.org/legal/privacy/firefox.html#health-report

Telemetry. Should be cleared these default values also?

services.sync.telemetry.maxPayloadCount			500
services.sync.telemetry.submissionInterval		43200
toolkit.telemetry.infoURL				https://www.mozilla.org/legal/privacy/firefox.html#telemetry

Later this one has disappeared in some case (probably, after some general option was disabled):
devtools.telemetry.supported_performance_marks
value = contentInteractive,navigationInteractive,navigationLoaded,visuallyLoaded,fullyLoaded,mediaEnumerated,scanEnd

Passwords. This password settings are unclear, enabled and missed in the "ghacks-user.js" list.
Password settings are important, as well. What you can say about this (defaults):

services.sync.engine.passwords					true
services.sync.prefs.sync.browser.safebrowsing.passwords.enabled	true
signon.recipes.path						chrome://passwordmgr/content/recipes.json

If SB & TP are disabled (uBO will be used + stock FF pass management), which values for this settings should be the most safe? What are they mean? What are chrome passwords doing in FFox ?!
May be the next short guide will appear helpul for implementation:
Disable Firefox Sync (data stored on Mozilla owned servers):
https://www.stigviewer.com/stig/mozilla_firefox/2017-03-22/finding/V-57673

pref("devtools.webide.autoConnectRuntime", false)		- NOT EXISTING in the "ghacks-user.js" list (default = true and still enabled)

Firefox silently installs "ADB Helper" and "Valence" Plugins (remote access to PC), when you open and close WebIDE in Developer tools for the first time. Also "adb.exe" background process running in the Task Manager. WebIDE’s main use case today is to connect to devices, and for this to function, the ADB Helper addon is downloaded when it is first opened.
Removing "ADB Helper" and "Valence" Firefox Add-ons (don’t worry, they won’t be installed next time, even when you access WebIDE again):
1. Add-ons Manager -> Extensions
2. Uninstall "ADB Helper" and "Valence" add-ons, one by one.
http://techdows.com/2015/08/shocking-firefox-silently-installs-adb-helper-and-valence-add-ons.html
Сan be disabled by setting the following preferences:
pref("devtools.webide.enabled", false)				- OK (2611)
pref("devtools.webide.autoinstallADBHelper", false)		- OK (2611)
pref("devtools.webide.autoinstallFxdtAdapters", false)		- obsolete (2611, FF57)
pref("devtools.webide.autoConnectRuntime", false)		- NOT OK, NOT EXISTING in the "ghacks-user.js" (default = "true")

Thank you for your Time & Power. The project is great.

Hi,

that pref was removed in FF60 and we will move it to 9999 as soon as FF60 is released. Not worth changing the value anymore.
all the *.infoURL's are the links behind "Learn more" links/buttons. Nothing is ever sent to that URL and unless you click the "Learn more" button(s) those URLs are never used. We want "Learn more" buttons to remain functional and there's really no reason to break them.

maxPayloadCount and submissionInterval are irrelevant when telemetry is disabled.
toolkit.telemetry.infoURL see point 2.
devtools.telemetry.supported_performance_marks = 0 results for FF59 ie not used anymore.

services.sync.engine.passwords allows Sync users to sync their passwords across devices. I know, sounds horrible but some people apparently like it. We don't disable anything to do with Sync because people who signup for it do so willingly and they should understand the potential risks.
services.sync.prefs.sync.browser.safebrowsing.passwords.enabled - this syncs the state of the pref browser.safebrowsing.passwords.enabled. Again, we don't disable Sync-related stuff.
chrome://passwordmgr/content/recipes.json looks to be some selectors which improve password field detection on those few sites. Seems useful for people who use the password manager. You can disable pw-storage with 0901

What are chrome passwords doing in FFox ?!

chrome:// urls are used to access certain files shipped with Firefox. It has nothing to do with Google Chrome.

devtools.webide.autoConnectRuntime should be covered by devtools.webide.enabled=false

@earthlng Thanks a lot for your attention. I'll make a correspondent notes in my own preffered settings. Happy to know, that everything is under control.
As I can see, you trust to the "general" switches of the relative blocks, while I prefer kill them all (in the most dangerous sections) one by one. Why?

in the future, when you will have a quick look at all-done settings, searching "did I miss smth?" or "what kind of new spying options have been preperared for us this time", catching this ones will be easier. This is better "eye-catch" feature (for those, who don't read bugzilla and specs as much as you).

Hope, you understand: Search  -> false, false, false, false, true.... (oops!), false...
(I know about mirror wording)

general option may have been changed/reworked/replaced by other. It will start working again (and secondary active options will support the new one again). If you didn't catch this change in time, you'll get full spying for a long time. Look at Windows... My experience says that killing all dengerous in all different places is much more safe.

("Learn more" links are OK, now I'm understand why, thanks. If you say chrome:// urls  are ok, then I prefer do not disrupt "saving passwords", e.g. useful settings. I'm prefer "smart balance", but not too relaxed)

^^ everything earthlng said except...
we use data:text/plain, in a few places, I've never seen anyone use data:application/json,{} before. This is not my area at all. I believe that what we have is fine - eg some pref overrides are blank/null, some are data:text/plain, . Earthlng is our resident expert on these (and from memory, the code behind some prefs barf at different values, or lack of them).
If there's nothing else to do here, then I think we can close this.
Oooh, and as I was about to post, @iWARR replied. Hi IWARR :) We fully understand the need for future-proofing and in the past the occasional master pref has been bypassed due to bug regressions. We already include a number of extra prefs that technically we do not need to (i.e to cover our asses and give peace of mind). We also pick up on every single change between versions (including system addons) - you can click on the "diffs" label in issues to see them all.

As I can see, you trust to the "general" switches"

Sorry, just to clarify, no we don't. We Earthlng actually digs into the code and checks it all out. And we do cover our asses in a few places where we strictly don't need to, just in case.

Hello again. I have a small question about a name of the editor (didn't find any mentioning in the Wiki).

It is recommended that you use an editor that supports syntax highlighting.

"For example, ........."
Working with a user.js, I like screenshot №2.

What the name of THIS editor?
Are some special color highlightning settings needed to look exactly the same?

Thank you.

It's Notepad++ https://portableapps.com/apps/development/notepadpp_portable

@iWARR Settings>Style Configurator - theme is Obsidian, and font is Hack - https://github.com/source-foundry/Hack

I NEED YOUR HELP!
I don't want to open a new issue, because my trouble is... very-very strange.
I can't run frashy installed FF v59.0.3 normally. (2 installs - the same thing)
Seems like a big troubles with java. For example:

In Violentmonkey Add-on properties i can't open "Settngs" page and "About" page. But the main works!

 But "Import" works!

Now (2-nd installation) I can't add script from https://greasyfork.org
The button of the script-installation doesn't press....

What's going on?! May be it's my custom override rules?
Have you any mention how to troubleshoot, what the setting to switch?
I tried many things, but nothing was helpful.


Worker With private ssl cert

Hi, i am trying out your awesome tool.
Sadly, I have the problem that my  Worker cant connect to my Server.
The Reason is simple, I use a Custom certificate, so the worker won't connect
certificate in certificate chain
At C:\Users\me\Desktop\windows-scripts\RunWorker.ps1:2 char:2
+  .\nexrender-worker.exe  --host=https://nex_server-test.int ...
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (FetchError: req...rtificate chain:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

    at ClientRequest.<anonymous>
(C:\snapshot\nexrender\packages\nexrender-worker\node_modules\@nexrender\api\node_modules\node-fetch\lib\index.js:1461:11)
    at ClientRequest.emit (events.js:311:20)
    at TLSSocket.socketErrorListener (_http_client.js:426:9)
    at TLSSocket.emit (events.js:311:20)
    at emitErrorNT (internal/streams/destroy.js:92:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:60:3)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  message: 'request to https://nex_server-test.apps.int/api/v1/jobs/pickup failed, reason: self signed certificate in
certificate chain',
  type: 'system',
  errno: 'SELF_SIGNED_CERT_IN_CHAIN',
  code: 'SELF_SIGNED_CERT_IN_CHAIN'
}

I tried adding the root certificate in windows but it didn't helped :(
is there a way to give the work a valid certificate or disable this check?

You can use let's encrypt to get a valid cert, however this error is happening in node-fetch library http call and is not about nexrender.

I am in an internal network with its own certs, so this is not an option but thx


Link to sandbox endpoint always opens in new tab

Background
I'm rendering the following Typeform embed
<Widget
  {...}
  enableSandbox={isStaging}
  redirectTarget="_self"
/>

In production, when a user completes the form, the iframe (because of redirectTarget="self") redirects to my ending URL. This URL uses window.parent.postMessage() to send a message to my SPA to update the UI as appropriate.
The problem
In Sandbox mode, the Typeform iframe redirects to a final page that displays a link to the ending URL. If I click on that URL, I expect the same behavior as if the user submitted the form in non-sandbox mode (besides the lack of a form submission). Namely, I expect the redirect to the ending URL to happen inside the iframe due to my redirectTarget setting.
Requested solution
If my embed is running in sandbox mode, the ending URL link displayed after completing the form should be equal to the redirectTarget value.

Hello @jkjustjoshing
how did you setup the redirect in your form? Can you share the form URL, please?

Form ID f9wO6H39 @mathio

Thanks you! We have identified the issue.
I will update you here when we have a fix ready, however this is not a priority at the moment. Sorry.


Update docs and add example app

This commit updates docs to reflect the existence of the new CRA template
for FluentUI that uses react-scripts for dev/build paths. It also adds
an example FluentUI app that the user can examine while following along
the 'Getting Started' section of the docs.

cc @JasonGore

Thanks for the PR! I have some questions I'd like to work out with you

Thanks for the review, and absolutely, please keep the review comments coming!

Comments addressed @JasonGore, ready for the next round of reviews!

Please consider not force pushing every time you update this branch. It makes pulling updates painful. Commits are squashed anyways, so doing force pushes doesn't really add any benefit.


NSFF Quality on Custom Dataset

Hi all. I'm trying to run this method on custom data, with mixed success so far. I was wondering if you had any insight about what might be happening. I'm attaching some videos and images to facilitate discussion.

First of all, we're able to run static NeRF using poses from colmap, and it seems to do fine

https://user-images.githubusercontent.com/2993881/119438855-24784700-bcef-11eb-97c9-ef6f2a811a25.mp4

Likewise, setting the dynamic blending weight to 0 in your model, and using only the color reconstruction loss produces plausible results (novel view synthesis result below, for fixed time)

Using the dynamic model, while setting all the frame indices to 0 should also emulate a static NeRF. It does alright, but includes some strange haze

Finally, running NSFF on our full video sequence with all losses for 130k iterations produces a lot of ghosting (04_nsff_result.mp4).

https://user-images.githubusercontent.com/2993881/119439006-6d300000-bcef-11eb-8df5-b8c9f98165be.mp4
Even though the data driven monocular depth / flow losses are phased out during training, I wonder if monocular depth is perhaps causing these issues? Although again the monocular depth and flow both look reasonable.

Let me know if you have any insights about what might be going on, and how we can improve quality here -- I'm a bit stumped at the moment. I'm also happy to send you the short dataset / video sequence that we're using if you'd like to take a look.
All the best,
~Ben

Hi,
Could you share the input video sequence?
I recently do find that for the dynamic model, if camera ego-motion is small (i.e, camera baseline is small in consecutive video frames), our local representation sometimes has difficulty reconstructing the scene very well due to our neighborhood local temporal warping.

Thanks for the quick response, and for your insight about when the method struggles. I've uploaded the video sequence to google drive here: https://drive.google.com/file/d/1C7XHilFxdpc9pcgfMieo7BuH0Y3Y1qeR/view?usp=sharing

@zhengqili Is there any theoretic reason why this happens, or this is pure observation? In my opinion the camera baseline shouldn't matter if COLMAP still estimates the poses correctly. Or do you mean that small baseline causes larger error for COLMAP (relative to the baseline)?

My thought is that for 3D reconstruction, a small camera baseline could lead to a degenerate solution. In classical depth estimation or MVS, we have to choose input frames that contain enough motion parallax (either using keyframe selection or input subsampled frames from the original video if everything is static) before performing triangulation, otherwise, the triangulation solver can be ill-conditioned.
NeRF is very beautiful for a static scene and it has no such problem because it's a global representation. But for our approach, you can think we are doing local triangulation in a small time window, so if the camera baseline is small or the background can be better modeled as Homography matrix than Essential matrix  (which actually is the case in this video), the reconstruction can be stuck in a local degenerated minimum.

Got it, but in my opinion

the background can be better modeled as Homography matrix than Essential matrix

this happens when there is no prior knowledge of the geometry.
However here we have monodepth estimation which seems reasonably accurate, is that still not enough? In your opinion what can we do to solve this issue, or we cannot do anything? Like the other issue #15 , it seems that we should be very selective in the video so that nsff works.

I also agree that monocular depth should help, but I also found that if we decay those priors during training, optimization sometimes can still get stuck in local minima in the end (I observed that the depth of moving objects will be completely wrong even if the initial estimate is correct for 10% videos I tried).
Thus, I believe this is still an open research challenge for the dynamic scene.

Yes, I also have the impression that the weight is decayed too fast, and at later epochs the depth becomes weird. What I have tried is to decay the weight very slowly, I find this helpful in my scenes, but don't know if it generalizes to other scenes.

Hi @breuckelen, I quickly tried this video. Since I have graduated and I don't have a lot of GPUs currently, I just use 3 frames consistency by disabling chaining scene flow during training (but the results should be very similar for view interpolation). I also subsampled the input frames by 1/2 since all the hyperparameters were validated on ~30 frames sequences.
If I use the default view interpolation camera path in my original github code, it does not look so bad (see the first video below), although it still contains some ghosting. However, if I change to larger viewpoint change, the ghosting will be more severe. To investigate this issue, I tried to render images only from the dynamic model (in function "render_bullet_time", change "rgb_map_ref" to "rgb_map_ref_dy"), the ghosting seems to disappear (see the second video), so my feeling is that the blending weights sometimes do not interpolate very well for larger viewpoint change in this type of video.
https://user-images.githubusercontent.com/7671455/119682415-de1b0900-be10-11eb-94bc-567b74373bf4.mp4
https://user-images.githubusercontent.com/7671455/119682475-ea06cb00-be10-11eb-9215-85ec4f415df3.mp4

Thanks! To confirm, is the second video is a rendering of the dynamic model only? Also, what is your N_rand (ray batch size) set to here? 1024?

Yes. In function "render_bullet_time()", you can change "rgb_map_ref" to "rgb_map_ref_dy" to render images from dynamic model only. My N_rand is still 1024, num_extra_sample is set 128 due to the limited GPU of my own machine :), but it should not be any difference in this case.

This is not relevant to the current implementation, but if you are interested in how to fix ghosting for the full model, I found one simple trick that can help reducing ghosting: adding an entropy loss for blending weight in the losses:
entropy_loss = 1e-3 * torch.mean(- ret['raw_blend_w'] * torch.log(ret['raw_blend_w'] + 1e-8))
This will make blending weight to be either 0 or 1, which helps to reduce the ghosting caused by learned semi-transparent blending weight.  The rendering result shown below is better. I haven't tried this trick for a lot of videos, but it's worth trying if you see the ghosting effects.
https://user-images.githubusercontent.com/7671455/120115033-e85e3f80-c14f-11eb-9535-7623cc921812.mp4

If I use the default view interpolation camera path in my original github code

Hi, what do you mean using "the default view interpolation camera path" and how to modify the viewpoint change? Could you please give some instructions on COLMAP SfM, I read the project you give and followed its prosedure, but still didn't get the same camera parameters as you provide, using the kid-running data.
Thanks a lot!

I'm curious -- is the spiral meant to track the input camera motion? Or is the entire scene moving (and being reproduced in the dynamic network) due to incorrect input poses?

The output camera trajectory is circular camera pose offset I create + interpolated input camera poses based on current fractional time indices, you can check how it works in function render_slowmo_bt:

@breuckelen Hi, can you also share the colmap files?

@zhengqili For this scene I observe that sometimes the dynamic part gets explained by the viewing direction, I'm trying to remove view dependency in training. Have you encountered this problem before? Theoretically, there is no way to distinguish dynamic or view dependency in my opinion, for example the shadow can be explained by both.

Yes. for shadow or other dynamic volumetric effects such as smoke, they can go either way. So if you don't care about modeling view-dependent effects in a dynamic region (in most cases it's indeed hard), it's a good idea to turn of view-dependent conditioning.

Hi I tried this using my latest implementation, and it works as well as zhengqi's modifications.
https://user-images.githubusercontent.com/11364490/123016335-51f70400-d405-11eb-922b-3c15e36736a8.mp4
I did not find

My thought is that a small camera baseline could lead to a degenerate solution for geometry reconstruction. In classical SfM, SLAM or MVS, we have to choose input frames that contain enough motion parallax (either using keyframe selection or input subsampled frames from the original video) before performing triangulation, otherwise, the triangulation solver can be ill-conditioned.
NeRF is very beautiful for a static scene because it has no such problem thanks to its unified 3D global representation. But for our 4D approach, you can think we are doing local triangulation in a small time window, so if the camera baseline is small or the background can be better modeled as Homography matrix than Essential matrix (which actually is the case in this video), the reconstruction can be stuck in a degenerated minimum.

this a problem. I use all 60 frames to reconstruct the poses and train.
To show you a better comparison, I do not use blending weight and let the network learn how to separate static and dynamic objects. This is the background learnt by the network (quite reasonable, the real background and some part of the body that only move a little):
https://user-images.githubusercontent.com/11364490/123016587-f0836500-d405-11eb-8d53-23493541633f.mp4
The advantage is, for static regions we know that it is going to perform as well as a normal NeRF, so no ghosting for these regions, and only the dynamic part might subject to artifact. While in this NSFF aside from the static network, the final rendering also depends on a blending weight that we cannot control. Like @zhengqili said maybe

My feeling is that the blending weights sometimes do not interpolate very well for larger viewpoint change in this video.

Time interpolation also looks good (artifacts around the face and the body, it actually moves but looks static), I'm thinking if I can use some prior to favor the network to learn all the body as dynamic...
https://user-images.githubusercontent.com/11364490/123018406-7654df80-d409-11eb-9afe-60696bb777e8.mp4

@kwea123 Hello, can you share the releated turtorial for using this project for our custom video data.


az ts show Doesn't Report Spec Versions

Describe the bug
When using the az ts show command, the property named versions is "null", even when there are versions (which is a required property on a template spec).
Command Name
az ts show --name <templateSpecName> --resource-group <resourceGroupName>
az ts show --name <templateSpecName> --resource-group <resourceGroupName> --query versions
Errors:
No error is returned, but the value of the versions property of the above command is null.
To Reproduce:
I was testing this using the Azure Cloud Shell, but I get the same result when running Az CLI 2.18.0 on Windows. Using Az CLI 2.18.0 in either cloud shell or on windows should be able to reproduce this behaviour.

az ts show --name {} --resource-group {}

Expected Behavior
I would expect to see all versions of the template spec, similar to the Versions property when running the PowerShell cmdlet Get-AzTemplateSpec.
Id                    : /subscriptions/7ecbc33f-d50e-4339-a26b-eb307001d280/resourceGroups/rg-templatespecs/providers/M
                        icrosoft.Resources/templateSpecs/vnet-two-subnets
Name                  : vnet-two-subnets
ResourceGroupName     : rg-templatespecs
SubscriptionId        : 7ecbc33f-d50e-4339-a26b-eb307001d280
Location              : eastus
Description           : Deploy a VNET with two subnets
Versions              : {1.0, 1.1}
CreationTime(UTC)     : 1/02/2021 6:45:58 PM
LastModifiedTime(UTC) : 1/02/2021 6:49:27 PM

Environment Summary
Linux-4.15.0-1106-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.18.0
azure-cli-ml 1.20.0

Extensions:
ai-examples 0.2.5
azure-cli-ml 1.20.0

Additional Context

ARM

@detienne20 Could you please take a look at this issue~


Gestion des dépendances

Resources qui traitent de l'impact des dépendances sur la qualité logicielle
Exemple de fichier de dépendances pour le projet gdext
[workspace]
resolver = "2"
members = [
    "godot-bindings",
    "godot-codegen",
    "godot-ffi",
    "godot-core",
    "godot-macros",
    "godot-cell",
    "godot",

    # Godot integration
    "itest/rust",
    "itest/repo-tweak",
    "examples/dodge-the-creeps/rust",
    "examples/hot-reload/rust",
]

[FR] gestion des dépendances

[FR] gestionnaire de paquets

[FR] dependabot

renovate

Automating dependency updates in practice: An exploratory study on github dependabot

BUMP: A Benchmark of Reproducible Breaking Dependency Updates

What NPM Should Do Today To Stop A New Colors Attack Tomorrow

Hash-Based Bisect Debugging in Compilers and Runtimes

On the impact of security vulnerabilities in the npm and RubyGems dependency networks

Dependency update strategies and package characteristics

Characterizing Dependency Update Practice of NPM, PyPI and Cargo Packages

Push and pull: when and why to update your dependencies

The Risks and Benefits of Updating Dependencies

Revival Hijack – PyPI hijack technique exploited in the wild, puts 22K packages at risk

Nix: A Safe and Policy-Free System for Software Deployment.
Nix

[FR] GNU Guix est un gestionnaire de paquets pour GNU/Linux

Unikernels are small, lightweight, single address space operating systems with the kernel included as a library within the application.

Taming behavioral backward incompatibilities via cross-project testing and analysis

poetry pour la gestion de dépendances en python
https://python-poetry.org/docs/

Conda provides package, dependency, and environment management for any language

Safety checks Python dependencies for known security vulnerabilities and suggests the proper remediations for vulnerabilities detected

A case study of developer bots: motivations, perceptions, and challenges, 2023


Uncaught TypeError on UNDO - REDO

The Undo - Redo buttons are returning a TypeError

@JeswinSunsi If you undo/redo when there is nothing to undo/redo, this happens, Will try to fix it so that it doesn't cause errors.

@JeswinSunsi If you undo/redo when there is nothing to undo/redo, this happens, Will try to fix it so that it doesn't cause errors.

Hmn might have a bug somewhere in there.
https://user-images.githubusercontent.com/39522664/103672189-6b121280-4f95-11eb-8cee-6a694bc98801.mp4
Everything else feels pretty great

Hmn might have a bug somewhere in there.
https://user-images.githubusercontent.com/39522664/103672189-6b121280-4f95-11eb-8cee-6a694bc98801.mp4
Everything else feels pretty great

@JeswinSunsi That looks like a bug, Thanks for informing.

@JeswinSunsi That looks like a bug, Thanks for informing.

Hi, @theabbie @JeswinSunsi can you assign this to me, I'd like to resolve this bug.


Fails to build with GCC 13

It has been reported that molmodel 3.1.0 fails to build from source with GCC 13. Full report is here. Next Debian release (in 2025) will be based on GCC 13.

Fixed in master

I confirm commit 968a0af8bc1a224337a3d1e1bd2fec3aec9e3fa1 fixes the issue.

great to hear!
On Thu, Jul 20, 2023 at 10:49 AM Andrius Merkys @.***>
wrote:

I confirm commit 968a0af
https://github.com/samuelflores/molmodel/commit/968a0af8bc1a224337a3d1e1bd2fec3aec9e3fa1
fixes the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/samuelflores/molmodel/issues/11#issuecomment-1643529687,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQ44PPZYPN4GBNH5AENXADXRDWIBANCNFSM6AAAAAAZOGTG34
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


This should lower false positives of performance checks

Fixes https://github.com/lemire/simdjson/issues/298
The idea is that we run 7 tests with the reference code, 7 tests with the new code. If all 7 new tests are worse than all 7 reference tests, then we know (???) something is fishy.
If we just have overlapping values, then we can't know for sure that there is a problem.
This will only pick up the obvious performance regressions, but I don't think this can be helped. And it is already useful to catch the really big regressions.

cc @jkeiser


Improve documentation by Anyblock feedback

Related to: trustlines-network/project#812

According to the first feeback at trustlines-network/project#812 we need to check the contracts.json copy command documentation.

I just made an attempt on my own in a fresh environment and followed our guide step by step. It worked without any issues. I need to contact Sascha to ask for details.

Okay, I misunderstood Sascha his feedback. He was not strictly following the documentation since he adopted the setup for his use-case. Therefore he was in need to tinker around and that was cumbersome. So he is asking if we could not provide the contracts.json file as it is within the repo like we do for the addresses.json. This should not be part of this issue.

Since there was no other feedback related to a documentation, but it was exclusively positive, I'll close this issue. We could still re-open it if the need occurs.


Doesn't work for me

Getting this :
Loigning into the server
[!] Login failed

Unfortunately, google stopped gmail login with an unknown app
So I'm still working on it

Unfortunately, google stopped gmail login with an unknown app
So I'm still working on it

Umhh, understand. So nothing could be done ? We should just wait or ? The instrument's look good, I wanted to test it haha : ( ) . Hope you find a solution !

Yea.
Wait till I find solution, and if you can help, you can contribute

It has been updated, delete the older version by using this command
rm -r -f Mailer

It has been updated, delete the older version by using this command
rm -r -f Mailer

Then install again

Thank you for your quick and responsive feedback.
Yes, you have improved some of the things and actually almost everything works. Thank you for that too ! Regarding the social hacking : when trying with an account registered with gmail, the mail is successfully sent to the inbox, but when trying with another email provider, it goes to spam ? Can this filtering be avoided, or is it entirely up to the developers and their protections ? But considering that it passes the gmail protection, maybe it could bypass other's ? Separately could be add some number here like sending could be like 5-10 mails with "fake alert" in total say, it will be really more effective for this kind of attack ?

Separately, the mass send option didn't work for me ? It only sends 1 message at a time ? Is the problem with me or you have not touched this option ?

Thanks for reporting this issues
I'm currently working on another version
I apologize for inconvenience
As for the spam, I'll try as much to fix that

Thanks for reporting this issues
I'm currently working on another version
I apologize for the inconvenience
As for the spam, I'll try as much to fix that

Thank you for your reply and information ! I'll keep an eye on the project and your stuff you create ! I hope you manage to fix it or create a new better one ! Do you have a discord / telegram where I can have a contact with you ? I also wanted to ask something else, in relation to another tool of yours ! Thanks again!

Thanks for reporting this issues
I'm currently working on another version
I apologize for the inconvenience
As for the spam, I'll try as much to fix that

Any new's , friend ?

Hello, bro ! Is there something new or this one could be fixed? @spider863644

@spider863644 Hello, my friend. I see you've made improvements and now it works. Is it possible to increase the emails to a larger number rather than just 1 message ?

Separately, there is a problem with the social media option, whatever site I write, it never appears in the message itself, meaning the message arrives but it is text only, with no link/url inserted ? Is it possible to fix this, as otherwise meaning is lost

@spider863644 Hello, my friend. I see you've made improvements and now it works. Is it possible to increase the emails to a larger number rather than just 1 message ?

Separately, there is a problem with the social media option, whatever site I write, it never appears in the message itself, meaning the message arrives but it is text only, with no link/url inserted ? Is it possible to fix this, as otherwise meaning is lost

Yes
I'm currently working on it

@spider863644 Hello, my friend. I see you've made improvements and now it works. Is it possible to increase the emails to a larger number rather than just 1 message ?

Separately, there is a problem with the social media option, whatever site I write, it never appears in the message itself, meaning the message arrives but it is text only, with no link/url inserted ? Is it possible to fix this, as otherwise meaning is lost

Yes I'm currently working on it

Any news?
-Also I can't run the instrument now, could u guide me ? When I try to run the instrument with : python3 mailer.py ,  I got this :
"Traceback (most recent call last):
File "/home/myname/Mailer/mailer.py", line 1, in
import pyfiglet
ModuleNotFoundError: No module named 'pyfiglet
Okay : I'm typing this : sudo pip install pyfiglet and got  :
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support pip 21.0 will remove support for this functionality.
Requirement already satisfied: pyfiglet in /usr/local/lib/python2.7/dist-packages (0.8.post1)
What is this ? My version on python when check is this : Python 3.10.12 ? Could u tell me , what do to , to run the instrument ?

I've fixed all issues
I'll upload the new version today

I've fixed all issues I'll upload the new version today

Hello, is this version is usable now or not ? When I tried to use something - got this : invalid file path, doesn't matter which option I choose

I've fixed all issues I'll upload the new version today

Hello, is this version is usable now or not ? When I tried to use something - got this : invalid file path, doesn't matter which option I choose

It's usable now, make sure the file path is correct
You will need a list of emails

I've fixed all issues I'll upload the new version today

Hello, is this version is usable now or not ? When I tried to use something - got this : invalid file path, doesn't matter which option I choose

It's usable now, make sure the file path is correct You will need a list of emails

Hey, I tried to use it again, but what this mean ? List of emails ? What should I do to run the instrument ? - "ge is in text and html format...Type "" to enter another paragraph...And you can input your link manually since its in html"

List of email should be in a text file
For example
@.***
@.***
On Mon, Dec 30, 2024, 11:44 AM profesionalleet1337 @.***>
wrote:

I've fixed all issues I'll upload the new version today
Hello, is this version is usable now or not ? When I tried to use
something - got this : invalid file path, doesn't matter which option I
choose
It's usable now, make sure the file path is correct You will need a list
of emails
Hey, I tried to use it again, but what this mean ? List of emails ? What
should I do to run the instrument ? - "ge is in text and html format...Type
"
" to enter another paragraph...And you can input your link manually since
its in html"
—
Reply to this email directly, view it on GitHub
https://github.com/spider863644/Mailer/issues/3#issuecomment-2565310698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS7IXHSLWBXDJA4JMP3JFHL2IEPXTAVCNFSM6AAAAABUL3ICFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRVGMYTANRZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

If you are still getting errors, I'll make a video
On Mon, Dec 30, 2024, 12:58 PM web spider @.***> wrote:

List of email should be in a text file
For example
@.***
@.***
On Mon, Dec 30, 2024, 11:44 AM profesionalleet1337 <
@.***> wrote:

I've fixed all issues I'll upload the new version today
Hello, is this version is usable now or not ? When I tried to use
something - got this : invalid file path, doesn't matter which option I
choose
It's usable now, make sure the file path is correct You will need a list
of emails
Hey, I tried to use it again, but what this mean ? List of emails ? What
should I do to run the instrument ? - "ge is in text and html format...Type
"
" to enter another paragraph...And you can input your link manually since
its in html"
—
Reply to this email directly, view it on GitHub
https://github.com/spider863644/Mailer/issues/3#issuecomment-2565310698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS7IXHSLWBXDJA4JMP3JFHL2IEPXTAVCNFSM6AAAAABUL3ICFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRVGMYTANRZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


feat: Add mssql_ad_join to allow using other domain join methods

Enhancement: Add the mssql_ad_join variable to allow using other domain join methods.
Reason: Sometimes, users use other methods to join managed nodes to AD that would be broken if that role were to run.
Result: Users can set mssql_ad_join to false and join managed nodes to AD themselves prior to running the role. By default, mssql_ad_join = true, so the current behavior is not changed.
Issue Tracker Tickets (Jira or BZ if any):
Resolves #210

@JasonN3 Thank you for submitting #210, please review this, mainly changes to README.md.

[citest]

@spetrosi ad_integration_user and ad_integration_password are really only needed if the user specified in mssql_ad_sql_user_name does not exist. Most admins I've worked with will want to create that ahead of time themselves along with setting the SPN. If you use mssql_ad_sql_user_name and mssql_ad_sql_password as default values for those variables, it will still allow you to check that the user exists with the same commands, which could also just be skipped in that case.
If you were to allow ad_integration_user and ad_integration_password to be omitted by using mssql_ad_sql_username and mssql_ad_sql_passwd as the defaults, you could skip lines 702-742 in tasks/main.yml with the condition that ad_integration_user was omitted. The username and password would already be verified when kinit was run on line 689. The rest should be able to succeed just fine. There might be an error when adding the SPN, but that would just be an error to tell them they skipped a step they wanted to do externally to this role.

Another scenario is to pass only a pre-created keytab file to mssql-server as described in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16 steps 3+. I think this is the one that more experienced users would be interested in. This is not.
And for the scenario in this PR, I would like to consciously provide ad_integration_user and ad_integration_password so that the role ensures that required properties and fails if not. It's not a big burden on the user, and it makes the process less error prone. WDYT?

ad_integration_user is also required to create this user in SQL Server in https://github.com/linux-system-roles/mssql/blob/main/tasks/main.yml#L848
So it's good if the role ensures that is it is present.

Another scenario is to pass only a pre-created keytab file to mssql-server as described in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16 steps 3+. I think this is the one that more experienced users would be interested in. This is not. And for the scenario in this PR, I would like to consciously provide ad_integration_user and ad_integration_password so that the role ensures that required properties and fails if not. It's not a big burden on the user, and it makes the process less error prone. WDYT?

Being able to optionally specify the keytab file would make it more flexible. None of the organizations I've worked with allowed the DBAs to have access to an account that could modify any AD users. If a DBA is running this Ansible role, they would need to be able to fill something in to there and the ad sql user would be enough permissions to verify that the AD admins completed the work correctly. It would only have an error if something wasn't pre-staged correctly and it would give enough information in the error that the DBA could tell the AD admin they didn't pre-stage the account correctly. Least privileged access and separation of duty are core security practices that should be accounted for if you want people to use this role.

So is it fine to require users to provide ad_integration_realm, ad_integration_user, and ad_integration_realm when mssql_ad_join is false?
I think we could make it so that if user provides mssql_ad_join: false and mssql_ad_keytab_file(I will implement it soon) then the only required variable would be ad_integration_user. See below why ad_integration_user is needed.

ad_integration_user is also required to create this user in SQL Server in https://github.com/linux-system-roles/mssql/blob/main/tasks/main.yml#L848 So it's good if the role ensures that is it is present.

Why are you creating the user account for ad_integration_user? If that account has the ability to add computers to the domain and create and modify user accounts, it should not be an account that you're logging in to the SQL database with. Again, least privileged access needs to be a core concept that is encouraged. You can give options to ignore LPA, but it shouldn't be the default or the only option.

Per the documentation in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16#createsqllogins or https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-ad-auth-adutil-tutorial?view=sql-server-ver16#create-active-directory-based-sql-server-logins-in-transact-sql
I specifically verified with Microsoft developer that here the login with the username of ad_integration privileged user must be created, so ad_integration_user.

Another scenario is to pass only a pre-created keytab file to mssql-server as described in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16 steps 3+. I think this is the one that more experienced users would be interested in. This is not. And for the scenario in this PR, I would like to consciously provide ad_integration_user and ad_integration_password so that the role ensures that required properties and fails if not. It's not a big burden on the user, and it makes the process less error prone. WDYT?

Being able to optionally specify the keytab file would make it more flexible. None of the organizations I've worked with allowed the DBAs to have access to an account that could modify any AD users. If a DBA is running this Ansible role, they would need to be able to fill something in to there and the ad sql user would be enough permissions to verify that the AD admins completed the work correctly. It would only have an error if something wasn't pre-staged correctly and it would give enough information in the error that the DBA could tell the AD admin they didn't pre-stage the account correctly. Least privileged access and separation of duty are core security practices that should be accounted for if you want people to use this role.

So is it fine to require users to provide ad_integration_realm, ad_integration_user, and ad_integration_realm when mssql_ad_join is false? I think we could make it so that if user provides mssql_ad_join: false and mssql_ad_keytab_file(I will implement it soon) then the only required variable would be ad_integration_user. See below why ad_integration_user is needed.

When mssql_ad_join is false, ad_integration_realm is going to be needed when verifying all of the pre-staging is done right, so, yes, that should still be required.
ad_integration_user is used to verify mssql_ad_sql_user_name is configured correctly, which could be done by the user itself which is why I'm saying you can use mssql_ad_sql_user_name as the default value, and you're allowing it to log in to the SQL database, which is the below issue.

ad_integration_user is also required to create this user in SQL Server in https://github.com/linux-system-roles/mssql/blob/main/tasks/main.yml#L848 So it's good if the role ensures that is it is present.

Why are you creating the user account for ad_integration_user? If that account has the ability to add computers to the domain and create and modify user accounts, it should not be an account that you're logging in to the SQL database with. Again, least privileged access needs to be a core concept that is encouraged. You can give options to ignore LPA, but it shouldn't be the default or the only option.

Per the documentation in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16#createsqllogins or https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-ad-auth-adutil-tutorial?view=sql-server-ver16#create-active-directory-based-sql-server-logins-in-transact-sql I specifically verified with Microsoft developer that here the login with the username of ad_integration privileged user must be created, so ad_integration_user.

Those commands are to allow a user to log in to the SQL server. It's even labeled "Create Active Directory-based logins in Transact-SQL". My point is that you shouldn't be using the highly privileged ad_integration_user  to log in to your SQL server. Instead you should be using an account that should have that access. That access should be granted in the mssql_post_input_sql_file script. Those commands do need to be run in order to allow an AD user to log in, but they should target a user or group that is supposed to be allowed to log in, which is going to vary based on the company and is going to be much easier to manage if they're grouped together with the GRANT statements.

Why are you creating the user account for ad_integration_user? If that account has the ability to add computers to the domain and create and modify user accounts, it should not be an account that you're logging in to the SQL database with. Again, least privileged access needs to be a core concept that is encouraged. You can give options to ignore LPA, but it shouldn't be the default or the only option.

Per the documentation in https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-active-directory-authentication?view=sql-server-ver16#createsqllogins or https://learn.microsoft.com/en-us/sql/linux/sql-server-linux-ad-auth-adutil-tutorial?view=sql-server-ver16#create-active-directory-based-sql-server-logins-in-transact-sql I specifically verified with Microsoft developer that here the login with the username of ad_integration privileged user must be created, so ad_integration_user.

Those commands are to allow a user to log in to the SQL server. It's even labeled "Create Active Directory-based logins in Transact-SQL". My point is that you shouldn't be using the highly privileged ad_integration_user to log in to your SQL server. Instead you should be using an account that should have that access. That access should be granted in the mssql_post_input_sql_file script. Those commands do need to be run in order to allow an AD user to log in, but they should target a user or group that is supposed to be allowed to log in, which is going to vary based on the company and is going to be much easier to manage if they're grouped together with the GRANT statements.

So there should be another variable with which users must provide a list of logins to create in SQL Server for this purpose, called e.g. mssql_ad_based_logins. And would it make sense to instead use mssql_ad_sql_user_name by default here (probably not, but I want to verify?

My choice would be no to mysql_ad_based_logins. Like I said, allowing someone to login should be grouped with their access (GRANT statements) to make it easier to ensure that everyone that can login has some form of access and everyone who has access can login. If you have a list of users/groups that can login in the role, then the GRANT statements still need to be in the post file and then you need to make sure the two completely separate lists have the same users/groups listed. It's not worth trying to make generating the GRANT statements part of the role because you need the databases created first. You'll end up adding complexity for very little return.

So should I rather remove this Create the privileged account login task to avoid security issues?

So should I rather remove this Create the privileged account login task to avoid security issues?

Yeah. That way there's no accounts created unexpectedly. If they don't create any users in the post file, they'll still have the sa account

@JasonN3 please check the last commit where I removed creating the privileged account login. I would like to chat about it sometime next week. Could you add me on slack or email me on<EMAIL_ADDRESS>Thank you

I put in 2 review comments that show up much earlier in this thread

@JasonN3 I think I applied all your corrections. I decided to add another variable mssql_ad_obtain_kerberos to control whether to obtain kerberos or not. It is more transparent then deciding on whether users provide ad_integration_user or not.
I moved removing creating the privileged account login to #217 for clarity.
Please review.

[citest]


MudChart: Add TimeSeries line and area chart components

Description
Adding a time series version of charts and implementing support for areas and lines to coexist on it. It reused a large amount of the existing line chart code with the changes needed to support time series and area charts. The same approach could be used to make the non time series chart support an area style.
How Has This Been Tested?
Manually tested the created charts and added a docs page.
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

#8973

The two sections have identical text. I'd suggest removing the area usage section and just add a MudSwitch to the basic usage example to switch between line and area.

The two sections have identical text. I'd suggest removing the area usage section and just add a MudSwitch to the basic usage example to switch between line and area.

I have squashed it down into a single chart with multiple series.

Nice, can you post an updated screenshot please?

Nice, can you post an updated screenshot please?

edited the first post to reflect

I think it was merged a bit too early, as it missing explicit bUnit tests which is required by our rules.
It will be harder to migrate to generic math without worrying about that something got broken.

Sorry, I forgot to check for tests :(. Shall we revert and wait for resubmission with tests or will you PR the tests separately @radderz ?

Good to merge. Just for my understanding @radderz, what is the reason for introducing the MudCategoryChartBase?

Basically MudCategoryChartBase is the old MudChartBase and and moved the common parts for the time series to a new MudChartBase, MudCategoryChartBase was all I could think of naming wise for the existing charts that are based on category/value.

Sorry, I forgot to check for tests :(. Shall we revert and wait for resubmission with tests or will you PR the tests separately @radderz ?

I hadn't planned on doing tests for the timeseries, but based on the line series tests if it basically just did a single chart of each and checked the result matches a known value that'd be straight forward I guess.

if it basically just did a single chart of each and checked the result matches a known value that'd be straight forward I guess.

That would be enough.


Trubrics Integration into Knowledge_GPT

Nice first PR @PBozmarov, congrats!

Let's also have a description for the PR. I'm thinking we should structure it like this:
Introduction to what the PR does (it allows the app owner to collect user feedback and analyse it in Trubrics, etc)
Setup
To save feedback to your Trubrics account, follow these steps:

Create a free account with Trubrics here
Add your Trubrics email & password to st.secrets

Preview
Here you can add two screenshots -  1 of the feedback component in KnowledgeGPT, and 1 of the comment page showing a raw feedback saved in Trubrics. Both 1 sentence explaining the image!


fix for ocamlbuild's pack+ocamlfind handling

This PR fixes the issue that causes the build failure discussed in
https://github.com/ocaml/opam-repository/pull/11628#issuecomment-375697444
and
https://github.com/mirleft/ocaml-nocrypto/issues/143
I have another branch v0.5.4-ocamlbuild-pack against the v0.5.4 tag, where the _tags changes are slightly different.

hmm, IIUC, this makes ppx_sexp_conv (which depends on ocaml-migrate-parsetree etc.) a runtime dependency of nocrypto!? my intuition is that these should better stay build-time only dependencies, is there another way?

ppx_sexp_conv (maybe only recent versions?) have a runtime library that I suppose they use for hygiene (to alias things from the global environment that code generators use), so I don't think we can avoid having it as a runtime dependency. There is a more precise ppx_sexp_conv.runtime-lib ocamlfind package, but I don't think using it improves things noticeably, and it breaks on older versions of the library.

Note: the package ppx_sexp_conv.runtime-lib would suffice, but it is
only available as such under recent ppx_sexp_conv versions, so its
explicit use would make the build description (needlessly)
incompatible with older ppx_sexp_conv versions.

Wouldn't ppx_sexp_conv also pull in ppx_sexp_conv.expander, which in turn contains the whole build-time ppx machinery? sexplib v0.10 already included the runtime library, so I'd be in favour of constraining to sexplib>=v0.10 and using ppx_sexp_conv.runtime-lib rather than using ppx_sexp_conv.

You should test this, but I think from the META file that the .expander dependency should only exists under the ppx_driver predicate.

You should test this, but I think from the META file that the .expander dependency should only exists under the ppx_driver predicate.

Ah, indeed it doesn't get linked. (tested by adding module X = Ppx_sexp_conv_expander in the code and making sure it doesn't compile).

@gasche In the v0.5.4-ocamlbuild-pack branch the ocamlfind_and_pack commit is reverted. Is this due to the fix already being in ocamlbuild? I re-applied the commit and it compiles fine. I'm about to submit these commits as patch files to opam-repository, including the reverted commit.


Support non-standard console methods

I would like to use js-logger with console.group. I know it's not standard, but I don't see why js-logger couldn't implement all console methods available in the current environment.

+1
chrome's console is awesome and I don't know if i can not use some features like console.group, color (console.log('%c solething', 'color:blue')) ...

My idea for coloring is that you want to abstract it out so you can have it work on node as well as browsers that support it. E.g.:
import { blue } from 'ucolor'
log.info(blue('something'))

I think it should be in a separate library. Something like colors or chalk that works universally. I once reserved the NPM namespace ucolor for it, but I never got around to implementing it.


transition animation

Whether or not a transition animation has been added to the sorting，thanks

You can directly use Vue's TransitionGroup or Transition component to implement animation. A simple example is as follows
<template>
  <!-- add TransitionGroup -->
  <TransitionGroup name="list" tag="div" style="width: 400px">
    <Card
      v-for="(card, index) in cards"
      :id="card.id"
      :key="card.id"
      :text="card.text"
      :index="index"
      :move-card="moveCard"
    />
  </TransitionGroup>
</template>

<style>
 /* add transition style */
.list-move, /* apply transition to moving elements */
.list-enter-active,
.list-leave-active {
  transition: all 0.2s ease;
}

.list-enter-from,
.list-leave-to {
  opacity: 0;
  transform: translateX(30px);
}

/* ensure leaving items are taken out of layout flow so that moving
   animations can be calculated correctly. */
.list-leave-active {
  position: absolute;
}
</style>

好的,谢谢


Add a vipsthumbnail engine

Add another engine, this time for vipsthumbnail. This is an experimental hack and not production-ready code.
Motivation
vipsthumbbnail can thumbnail large images quickly and using little memory. For example, for a 10,000 x 10,000 pixel RGB image:
$ time convert wtc.png -resize 128x128 x.jpg
real	0m5.604s
user	0m7.012s
sys	0m0.384s
peak RES: 600mb

$ time vipsthumbnail wtc.png --size 128x128 -o x.jpg
real	0m3.530s
user	0m3.508s
sys	0m0.016s
peak RES: 60mb

For jpg, vipsthumbnail uses shrink-on-load, so the speed difference is larger. convert would be much quicker with -define hints, but I'm not sure sorl-thumbnail uses them. Did I miss them in the code?
$ time convert wtc.jpg -resize 128x128 x.jpg
real	0m2.833s
user	0m4.308s
sys	0m0.320s
peak RES: 600mb

$ time vipsthumbnail wtc.jpg --size 128x128 -o x.jpg
real	0m0.297s
user	0m0.284s
sys	0m0.012s
peak RES: 10mb

See also:
http://www.vips.ecs.soton.ac.uk/index.php?title=Speed_and_Memory_Use
What's missing?
This fails 12 tests, mostly due to a missing _crop() method. I'm not sure what to do about this, vipsthumbnail has a --crop option, but it crops to the dimensions given in --size, you can't specify a different crop box.
That's running against git master vips. If you install vips-tools from apt, you'll see more failures than that.
vipsthumbnail has some options for colour management which are not expoosed. You can give an output profile and a fallback input profile. I'm not sure how to handle this.
vipsthumbnail has an auto-rotate option, but it only supports the basic four rotates, it doesn't do the flips. This causes another couple of tests to fail.
vipsthumbnail does not support conversion to greyscale.
libvips has a Python binding and it would be pretty easy to make a sorl engine which used that. This would be a fair bit quicker, and it would be easy to support the missing features. It would also avoid the
write-to-file / read-from-file steps.

Can you check which version of coverage tox has installed?

How would I get the exact version? Sorry, I'm not very experienced with tox. I see:
$ tox --showconfig
tool-versions: tox-1.8.0 virtualenv-1.11.6
config-file: tox.ini
toxinipath: /home/john/GIT/sorl-thumbnail/tox.ini
toxinidir:  /home/john/GIT/sorl-thumbnail
toxworkdir: /home/john/GIT/sorl-thumbnail/.tox
setupdir:   /home/john/GIT/sorl-thumbnail
distshare:  /home/john/.tox/distshare
skipsdist:  True

... snip ...
[testenv:py27-django18-vipsthumbnail]
  basepython=python2.7
  _basepython_info=<executable at /usr/bin/python2.7, version_info (2, 7, 9, 'final', 0)>
  envpython=/home/john/GIT/sorl-thumbnail/.tox/py27-django18-vipsthumbnail/bin/python
  envtmpdir=/home/john/GIT/sorl-thumbnail/.tox/py27-django18-vipsthumbnail/tmp
  envbindir=/home/john/GIT/sorl-thumbnail/.tox/py27-django18-vipsthumbnail/bin
  envlogdir=/home/john/GIT/sorl-thumbnail/.tox/py27-django18-vipsthumbnail/log
  changedir=/home/john/GIT/sorl-thumbnail/tests
  args_are_path=True
  install_command=['pip', 'install', '--pre', '{opts}', '{packages}']
  commands=
    ['py.test', '-rw', '--cov-config', '.coveragerc', '--cov', 'sorl']
  deps=[pytest, pytest-cov, pytest-django, Pillow, Django>=1.8,<1.9]
  envdir=    /home/john/GIT/sorl-thumbnail/.tox/py27-django18-vipsthumbnail
  downloadcache=None
  usedevelop=False

Each subfolder within .tox with a name like one of the test environments is a regular virtualenv so the following command should list everything installed in py34 with django1.8 and vips:
.tox/py34-django18-vipsthumbnail/pip freeze

This branch is up-to-date with the base branch
Merging can be performed automatically.

I see:
john@kiwi:~/GIT/sorl-thumbnail (add-vipsthumbnail)$ .tox/py27-django18-vipsthumbnail/bin/pip list
argparse (1.2.1)
coverage (4.0b1)
Django (1.8.3)
Pillow (2.9.0)
pip (1.5.6)
py (1.4.30)
pytest (2.7.2)
pytest-cov (2.0.0)
pytest-django (2.8.0)
setuptools (12.2)
wsgiref (0.1.2)

Try pinning coverage to 3.7.1 in the tox file.

@relekang yes, that seems to fix it for me. I can now tox with the exclude_lines = lines present.
12 tests still fail, all related to unimplemented _crop


Pre-populate CopilotTextarea contents

Proposal
Allow for specifying prompt for pre-populating copilot textarea contents with a given prompt

Hey @ataibarkai Can we discuss  aboout these issue ? Looking forward to it !


background-image: url('data:image/png;base64... are misinterpreted

Hi,
I am using a bower plugin called sumoselect. If i use the bowerbundle-plugin the base64 encoded urls are changed.
e.g.:
background-image: url('data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAA0AAAANCAYAAABy6+R8AAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3wMdBhAJ/fwnjwAAAGFJREFUKM9jYBh+gBFKuzEwMKQwMDB8xaOWlYGB4T4DA0MrsuapDAwM//HgNwwMDDbYTJuGQ8MHBgYGJ1xOYGNgYJiBpuEpAwODHSF/siDZ+ISBgcGClEDqZ2Bg8B6CkQsAPRga0cpRtDEAAAAASUVORK5CYII=')
changes to:
background-image: url("siDZ+ISBgcGClEDqZ2Bg8B6CkQsAPRga0cpRtDEAAAAASUVORK5CYII=");
So it clips the first part of the url.

Thanks for pointing this out.
v2.1.3 fixes the data URIs being mangled. Try updating and if you still have problems, please shout.


BO: Prevent dual results on multi store

On a multi-store with multiple languages, the Configuration page could display every cms category and the included cms entries twice.
e.g.
CONTENT PAGES - HOME
ID	Name
1	Delivery
1	Delivery
2	Legal Notice
2	Legal Notice
CONTENT PAGES - HOME
ID	Name
1	Delivery
1	Delivery
2	Legal Notice
2	Legal Notice
PRODUCT PAGES
etc...

Even if solves the duplicated content, now just appears the content's link of default store. It's even worst than duplicated content because this can take the user out of the navigation flow, affecting the seo.

It's been a year and I stopped using PS, but:
It is using the passed shop id; which should be right, I presume?
id_shop = {$this->shop->id}

Fixed with #48


[Feature Request]: Change hashedUserId to be different to the Oculus PC counterpart

Is your feature request related to a problem?
When someone tries to join a lobby with the same account twice, the server would block that and the player will get an connection error.
Describe the solution you'd like
Change the userId to be different for Quest and allow adding a custom prefix.
Describe alternatives you've considered
None
Additional context
No response
Device
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Added in #31


Semicolon insertion section is FUD

This is the current example for when semicolon insertion bites back:
return
{
    foo: 1
}

The rest of the section then proceeds to explain how such issues are avoided by just using semicolons everywhere.
 Read the example again. Does just putting semicolons in that code fix the problem? No, but removing the newline after return does. You're just propagating FUD that I wrote a whole blog article about.
The second example you used is a little better, but it's convoluted and overly complicated while explained. There is a much simpler example and shorter explanation in this document on mozilla.org.
My point: you can advise people to always use semicolons, but please pay attention on the kind of examples you provide. Just don't tell people that semicolons should never be omitted because that simply isn't true. Here is a fork of jQuery with most semicolons removed—all tests pass in all browsers. See if, with all your JavaScript expertise, you can find a bug in it.

Though the issue has been marked closed (and I appreciate that work has been done), I'd like to suggest a reopen. As a new reader, I still found the language too "aggressive" and impartial.
Some suggestions:

"The automatic insertion of semicolon is considered by some to be "
-"one of the biggest design flaws" +"a significant flaw in the language's design"
Expand the "conclusion" section, renaming it to "Coping" or "Solutions" or "Workarounds" or etc. Present "Mandatory insertion" as one approach, include perhaps Crockford's talk on the issue and JSLint. Present "prepend semicolons to parens/brackets" as another, include perhaps some of the popular articles and JSHint. Discuss strengths and weakness to both approaches -- mandatory: more clear about intent; does not solve all problems ("return;" example), adds noise. automatic: removes noise, is a part of the specification, keeps syntax quirks on the left where it is more likely to be noticed; could cause unexpected, unintuitive behavior (I'm admittedly less aware for cons here as I prefer to leave off semicolons).

Javascript Garden is a public service, and an incredible one at that. Thank you for making an effort to be informative and impartial in what is a somewhat controversial issue in Javascript.

+1 @nkwiatek


[97] Add schemes and update README.md

https://github.com/nimbl3/ios-templates/issues/97
What happened

[x] Add schemes to template
[x] Configure build configurations for schemes
[x] Update README.md for the post-actions to configure schemes after creating a project with this template

Add blueprint identifiers
Remove the folder Removable Resources

[x] Update README.md for the post-actions of build configurations

Remove duplicated settings
Add note

Insight

Currently, I leave the blueprint identifier with empty values. I provided post-action in the README.md file
Some references:

https://michele.io/the-project-file-part-2/
https://www.rubydoc.info/gems/xcodeproj/Xcodeproj/XCScheme/BuildableReference
https://developer.apple.com/library/archive/documentation/DeveloperTools/Reference/XcodeBuildSettingRef/1-Build_Setting_Reference/build_setting_ref.html#//apple_ref/doc/uid/TP40003931-CH3-SW106

Proof Of Work

@vnntsu Please take a look 👁 👁  Thank you 💯


triagram_lab.py from Session04 and mailroom2.py from Session 05

Finished the triagram lab from last week and added "safe_input" method to mailroom2.py

I made a bunch of comments, but this is very good work. :)


V4.0b1 - icon.png permissions too strict to display in IBM notifer

I think the icon.png file in /Library/Management/super that super creates when pulling the image from an online source has permissions that are too tight for standard user accounts to view when its placed into an IBM Notifier prompt.
I got the icon of IBM Notifier displayed, when I changed the file permission to 777 and triggered super again it then displayed the correct file

Guessing the SIP process is running as root and creates the $DISPLAY_ICON_FILE_CACHE with root permissions only.  Prob just needs to change the permissions to read only for everyone?

Thanks for finding this one!

While I can't replicate the behavior, I'll still add this line just in case...
chmod a+r "${DISPLAY_ICON_FILE_CACHE}"

https://github.com/Macjutsu/super/releases/tag/v4.0.0-beta2

Re-opening, icon file is still showing permissions of 600, unless the reset argument doesn't clear out the cached icon?

Same here!
V4-beta3

If you're referring to --reset-super then yes it does reset the cached icon.
Here is the code change... I mean... literally chmod all can read.... so I'm not sure why the resulting cached icon for you has those permissions.
log_super "Status: Copying requested --display-icon-file from: ${display_icon_file_option}"
sips_response=$(sips --setProperty format png "${display_icon_file_option}" --out "${DISPLAY_ICON_FILE_CACHE}" 2>&1)
chmod a+r "${DISPLAY_ICON_FILE_CACHE}"

And unsurprisingly my result is acceptable... I can't replicate this issue...
❯ ls -la /Library/Management/super/icon.png
-rw-r--r--  1 root  wheel  783521 Sep 29 18:02 /Library/Management/super/icon.png

Ah yes, can see the icon deleting itself but it just re-appears with 600
❯ ls -la /Library/Management/super/icon.png
-rw-------  1 root  wheel  783521 2 Oct 15:30 /Library/Management/super/icon.png

That works fine, wondering does that chmod need to also exist on LN1807?

Doh! and also probably ~LN1827... basically after every sips_response...
Try that and lemme know how it works.

Yep, worked on my test device adding chmod a+r "${DISPLAY_ICON_FILE_CACHE}" to LN1807, didn't add to the additional line you just commented on

https://github.com/Macjutsu/super/releases/tag/v4.0.0-beta4


[IconMenu] Add open, onRequestChange properties, fix #2341

Hi,
This is an update of https://github.com/callemall/material-ui/pull/2343
I rebased the branch to the latest upstream code.

I'm ok with this, but do you think 'esc' should be 'escape' to keep consistent with LeftNav? or should I rename the LeftNav one? Which one makes more sense? @oliviertassinari @quangbuule

I think 'escape' have more sense. Personally, I like the way that just send the event to user to decided.

escape :+1:

I updated to escape reason already. @oliviertassinari @subjectix

:+1: Do a rebase and make this a single commit, then we are good to merge. right? @oliviertassinari

I rebased at begining of the PR @oliviertassinari

@quangbuule I think you are missing a force push:
git push origin icon-menu --force

That will forcefully update your remote branch and clear the history. It will the look exactly like how you see it after you rebase locally

@subjectix @oliviertassinari take a look :smile:

@quangbuule Perfect :+1:

Nice work :+1: :+1: Thanks :grin:


I added Spanish translation

I added Spanish translation

Yes, I saw, I will push an update in this week 🙂


Added sentry and minor changes

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


SkateJS ssr example just not working

Bug report

Skate version: 5.2.4
Affected browsers (and versions): Chrome

Current behaviour
git clone https://github.com/skatejs/skatejs.git
yarn add global bolt
run bolt
go into cd packages/ssr/example
run npm install
run npm run start
get error
{"statusCode":400,"error":"Bad Request","message":"Invalid cookie value"}

Expected behaviour
should run server and serve ssr page.
Thanks
Really would love to see this working.  Will play with it more .. it's prob me

Hey, @matthewharwood, it definitely wasn't you! The example was never updated after the new APIs were released. I've since updated it. LMK if you're still having issues.


fix(api-reference): endpoints description

this pr adds collapse state to endpoints component in order to clamp description when collapsed and fix #2970:
https://github.com/user-attachments/assets/49e167f0-5dde-4c6c-b02f-b03800664a82

LOVE IT, keep those PRs coming


fix: added test case and resorted tests for arguments-optional page

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #50402

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #50402

Fixed a failing test case from previous commit. All tests now passing


[FLINK-17570] Fix recursive call in BatchTableEnvironment#fromValues

What is the purpose of the change
Fix recursive call in BatchTableEnvironment#fromValues
Verifying this change
Added test org.apache.flink.table.runtime.batch.table.JavaTableEnvironmentITCase#testFromValues
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thank you for reporting the issue @WeiZhong94. Could you have a look at this PR ?

@dawidwys Thanks for your PR! LGTM, +1 to merge. 👍

Thank you for the review. Will merge once azure gives green.


Routes with autoAliases not considered for started

If a API gateway has a configuration using autoAliases, the service will be marker as "ready" (started) even if the routes are not registered yet:
name: "xyz",
routes: [{
    path: "/download",
    use: [cookieparser()],
    whitelist: ["xyz.download"],
    autoAliases: true,
    mappingPolicy: "restrict",
    logging: false,
}]
...
actions: {
    xyz: {
        rest: {
            method: "POST",
            path: "/"
        },
       ...
    }
}

If this syntax is used, it works as intended:
name: "xyz",
routes: [{
    path: "/download",
    use: [cookieparser()],
    whitelist: ["xyz.download"],
    aliases: {
        "POST /": "xyz.download"
    }
    mappingPolicy: "restrict",
    logging: false,
}]
...
actions: {
    download: {
        rest: {
            method: "POST",
            path: "/"
        },
       ...
    }
}

this test will randomly fail for the first case:
describe("test", () => {
    beforeAll(async () => {
        broker.createService(xyzService);
        await broker.start();
        await broker.waitForServices(["xyz"]);
    });
    // test to request /download
});

This is the expected functionality. The reason is, that the auto aliases function executes with a little delay to wait for all services should be started. In the case of a monolith, it looks like no reason to wait, but in a multi-node environment, if API gateway started, it doesn't mean all other services are loaded.
For tests, you should add a delay after the broker started, or set the debounceTime to zero in API gateway settings.


gpio: Added split gpio addresses with their names.

This GPIO branch will be needed for testing https://github.com/lowRISC/sonata-system/pull/290

This is now out of date; as there should be 0x20 spacing between GPIO regions, not 0x10. Probably better to do a more in-depth PR later as there is a lot of things to update in the board description file.


rotate().metadata() doesn't return rotated metadata.

[ x] Running npm install sharp completes without error.
[x ] Running node -e "require('sharp')" completes without error.

Are you using the latest version of sharp?

[ x] I am using the latest version of sharp : 0.31.3

What are the steps to reproduce?
var metadata = await sharp(buffer).rotate().metadata()
// old metadata as if it wasn't rotated

var rotated_buffer = await sharp(buffer).rotate().toBuffer()
var rotated_metadata = await sharp(rotated_buffer ).metadata()
// rotated_metadata is now correctly rotated as the sharp instance was created from a rotated buffer.

What is the expected behaviour?
var metadata = await sharp(buffer).rotate().metadata()
// metadata should be rotated

Please see https://sharp.pixelplumbing.com/api-input#metadata

This is taken from the header of the input image. It does not include operations, such as resize, to be applied to the output image.

Please also see the "Based on EXIF rotation metadata..." example.

I've added further clarification to the docs via commit https://github.com/lovell/sharp/commit/fc03fba6021f2b24275d7cc2edc7390f229b7543


chore(deps): update dependency @semantic-release/release-notes-generator to v10

This PR contains the following updates:

Package
Type
Update
Change

@semantic-release/release-notes-generator
devDependencies
major
9.0.3 -> 10.0.2

Release Notes

semantic-release/release-notes-generator
v10.0.2
Compare Source
Bug Fixes

deps: update dependency conventional-changelog-writer to v5 (#212) (6724eca)

v10.0.1
Compare Source
Bug Fixes

deps: update dependency import-from to v4 (#248) (bb00694)

v10.0.0
Compare Source
BREAKING CHANGES

the minimum required version of node is now v14.17.

Thanks
@abel-mak

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will ignore this upgrade and you will not receive PRs for any future 10.x releases. However, if you upgrade to 10.x manually then Renovate will then reenable updates for minor and patch updates automatically.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


An error in my AS：Error:(18, 1) A problem occurred evaluating project ':app'. > Could not find property 'MATERIAL_KEYSTORE_FILE' on SigningConfig_Decorated{name=release, storeFile=null, storePassword=null, keyAlias=null, keyPassword=null, storeType=null}.

I‘m new in Android, so please do me a favor about this kind of problem.. Thank you very much..

I have solve this problem just now. just alter two files. You can try this repo.


Box loader Added

Add a box loader using svg and CSS  animation

Thanks for contributing @hash-tagg
Star the repo  and follow me on github.


Fix base class error of abstract class

Regen:  https://github.com/Azure/azure-sdk-for-net/pull/30740
Description
Add your description here!
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

When fixing this issue, I realized that we might have a potential duplicate conflict issue in the abstract base class implementation, opened this issue to track: https://github.com/Azure/autorest.csharp/issues/2678


org.eclipse.ui.navigator java.lang.IllegalArgumentException: newPosition > limit: (973669377 > 176322) (IDFGH-9718)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
esp-idf-master-v5.1
IDE
Version: 2.8.1
Build id: 20230207-1119
Operating System used.
Windows
How did you build your project?
Command line with Make
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32C6 DIY
Power Supply used.
USB
What is the expected behavior?
Run debigging
What is the actual behavior?

log.log
Steps to reproduce.

Setup esp32c6 toolchans
Run debug
Get the error
If priceed it than debug doesn't catch breakpoints, just flash and terminate

Debug Logs.
in What is the actual behavior?

More Information.
#Sun Mar 26 16:21:55 YEKT 2023
org.eclipse.core.runtime=2
org.eclipse.platform=4.25.0.v20220831-1800

@RJSDevel for IDF eclipse plugin related issues you might be better of reporting it directly at https://github.com/espressif/idf-eclipse-plugin

Could you move it? I can't see any opportunity

Sorry, I dont have access to do it either. Think you'll just have to close this one and open a new one.