DDG 180 Degree Text Flip in Search Result

In all instances of DDG I use (DDG for iOS, DDG macOS) and all browsers using DDG Search (Tor, Safari, Firefox on iOS or macOS), when I type the search terms below, the result preview / confirmation appears flipped 180 degrees.
rotate text box in windows preview with mouse 20degrees
Also happens for:
rotate text box in mac preview with mouse 20degrees
https://github.com/duckduckgo/iOS/assets/113533724/cc1f7172-4989-4e19-b668-c7443d471ef1

Hello, thanks for your feedback.  This is deliberate and not specific to our apps. 😄
This is what's known as a "goodie" and we've been doing this for many years. We have many of them that will trigger depending on the words you use in your search.
Try some of the following:
user agent
qrcode hello world
roll d20
We also have Instant Answers which tend to be more extensive, e.g.
weather in London
coffee shops in Paris
So that's what is happening and shouldn't affect the search result you get.
Hope that helps.
Thanks!


OTP is coming as web login and not detected by the SDK

Step 1: Are you in the right place?
Yes
Step 2: Describe your environment

Android device: Samsung M21
Android OS version: 10
Google Play Services version: 20.47.14
Firebase/Play Services SDK version: _____
FirebaseUI version: 7.1.1 (Latest)

Step 3: Describe the problem:
Problems are regarding sign-in and while upgrading from 6.x to 7.x :
1. SMS OTP is not detected automattically.
See attached video, OTP is not detected in the latest version.
https://user-images.githubusercontent.com/7250906/104368913-ba7eb280-5542-11eb-9bbc-b735bbf38ac5.mp4
2. Showing internal Google project ID/Default firebase hosting site in a custom tab while signing in.
As seen in the above video, while signing in, a blank custom tab is opened with my Default hosting site i.e. < google-project-id >.firebaseapp.in which has internal Google project ID (Instead of my desired site name). Why is this page coming?
3. OTP is showing as web login to Default firebase hosting site
See thess OTP messages, now the OTP is coming as if signing in to web (That too from Default hosting site with internal Google project ID (Instead of my desired site name). Compare with old OTP.

Expected Results:

OTPs to be detected automatically as earlier.
No extra custom tabs (if it's required, then show my selected website from the Firebase hosting sites)
OTP messages should show my app name, instead of website. (if it's required, then show my selected website from the Firebase hosting sites)

@hkchakladar thank you for the super clear bug with video!  I actually think this is a Firebase Android SDK core issue and not an issue specific to FirebaseUI so I am going to transfer it there.

@hkchakladar thank you for the super clear bug with video!  I actually think this is a Firebase Android SDK core issue and not an issue specific to FirebaseUI so I am going to transfer it there.


Repeat roles, not people in metadata editor interface

What feature would like to see being added to the IPT?
Since IPT 2.2, one can add more than one resource contact, resource creator[1], metadata provider and project personnel. In associated parties, this functionality has always existed. With all these (additional) places to add people involved in a dataset, even more contact information has to be repeated in the metadata editor interface. The interface currently provides only one shortcut for repeating information, which is to copy the information from the resource contact.
I think a better setup would be to have one page where people/associated parties can be added ONLY ONCE, but that different roles can be selected for them (e.g. resource creator, principal investigator, etc.). Some of these roles can be made mandatory (e.g. please indicate someone as the resource creator). Advantages:

Better connection between the roles on the basic metadata and the same roles in associated parties
No discrepancies in contact information for the same people
Easier to update contact information
More chance that the different roles of people are indicated (as it is much easier to select an additional role, than to fill out all contact info again).
Project personnel can also be added that way, either as additional roles "project:principal investigator" or by assuming that a role of a person in the dataset is the same for the project.
Underlying EML schema can remain the same.

[1]: As the resource creators are also includes as authors in the automatic resource citation (great!), we are currently moving people from Associated Parties to resource creators for our datasets, so they get the appropriate credit, hence my annoyance in having to copy everything. :-)

<EMAIL_ADDRESS>I assume the editor is populated from the eml, so the IPT could detect identical contact information and collapse to one contact with different roles in the interface. If the contact information differs, those become two contacts with a single role.
That makes it also easier to migrate from the current to a new version of IPT with this feature.

Thanks for reporting this enhancement, I agree it would be a much better way of entering contact information.

At https://github.com/gbif/ipt/issues/1360#issuecomment-1333994478 I proposed how one list of people (contributors) with different roles could be translated to EML were people are grouped by role:

The IPT could still offer to indicate roles for those contributors (e.g. contact). That can be expressed in EML by listing those people under a specific property for that role (cf. current implementation), but in addition to them being listed as creator. It also provides a way to migrate info in the IPT: make metadata editors, creators and contacts all contributors and remove duplicates.

@peterdesmet Thank you for the suggestions

@peterdesmet I'm sorry, but I'm not sure I understood your suggestions.
Currently metadata sections reflect eml sections exactly (the same for all metadata fields). This means we have for example creator so it's stored to eml.creator[index] and so on. With the joined list we can't do that, so we need another approach.

The idea would be to have a translation between the interface and EML:

When the interface is loaded from EML: list all contributors in creator, metadataProvider and contact (and potentially associatedParty) as a single list and indicate the role they have. If a contributor is exactly the same across roles, join the information into one contributor and indicate the multiple roles they have.
When the interface writes back to EML: do the reverse and split contributors with multiple roles again.
The basic metadata section would throw an error if the required roles aren't represented.

Note that the main goal is to reduce the manual work involved in adding contributors. I (and likely many other IPT maintainers) have to enter the same contributor information a lot - across roles and across resources - and the IPT doesn't facilitate this work (except for copy resource contact). It is worsened by the fact that the moment you want to insert a contributor at a specific place (e.g. in an author list), you have to create a new contributor at the end, and then copy/paste all information one down to make room at the desired spot (see #1325).
Repeating roles, not contributors, as suggested above is just one approach. It doesn't solve the issue with facilitating repetition across resources however. Maybe a central contributor registry in the IPT would be better? Suggestions welcome! 😄

I agree with @peterdesmet
In my mind it would be great to have a list of all contributors with associated contact info and IDs and the opportunity for each person to select one or more roles via radio boxes. Thus I could add Peter and his info once and then select him as Resource Creator, Metadata Provider, Point of Contact, and a Programmer. In the current configuration of the IPT, his name and info would be populated twice in the Basic Metadata section and once in the Associated Parties section. This example is not perfect, but I think it begins to paint a picture.
I've also wondered if we couldn't expand the use of the existing Administration>User Accounts registry to host an individual's contact info/IDs and as the location where an admin can provide access to specific resources and also the roles within each resource. Just thoughts.


Add column for permission (read, write, delete, all) in action

It would be great if the tables for roles had another column to isolate just the actual type of permission (read, write, delete, all) for a given action.  It would be even better if there was a way to sort the tables, but I understand that is a limitation overall.  At least this would make it clearer when looking at a list what type of action and make it easier on the eye to catch where there are modify rights.

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

ID: 99581afb-7950-6daa-7227-055d04446b7e
Version Independent ID: f1a5bf89-9e54-aede-3a9a-70788a95927b
Content: Azure built-in roles - Azure RBAC
Content Source: articles/role-based-access-control/built-in-roles.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@quix303
Thanks for your feedback! We will investigate and update as appropriate.

Hi @quix303
This is great feedback. We passed it along to the team.
The team has also considered breaking up the article into multiple articles. Do you prefer one long article or multiple smaller articles?
thanks

#please-close


Slow when padding

Since the implementation of FFT only optimized on the input size that can be factored to 2, 3, 4, 5, 7, 11. When padding to (n + n_kernel - 1) x (m + m_kernel - 1), the size will be factored to some large primes, which slow down the calculation much.

Possible solution: padding to some computed-fast case

Done along with this commit a3c76ca
Code


[Fundamentals] Data products overview

Adding first pieces for the data products overview. Will add "understanding cloudflare datasets" in a later PR.

@tlozoot, mind taking another look at this? Implemented the edits from what we talked about earlier in the week!


fix mutation of recordDefaults

fixing a derp move.  use blank object for Object.assign target to prevent unintended mutation

May also want to tag this for package manager "patch" release.

Another good catch (I missed it during review 😰 ).
I will take care of version bumping, tagging and releasing after this is merged.
(I also have a plan - #16 - to automate all of this through CI as I have done in most of my newer node libs)


Fix compilation with ghc 8.2.2, Support foreign-libs and sub-libs

I couldn't test it though, see #35

Thanks for working on this!
I got the following when building so do bump the version bounds:
Setup: Encountered missing dependencies:
Cabal ==1.24.*, base >=4.5 && <4.10

After fixing that the package builds and runs successfully.

Ha, I was building with stack, and allow-newer = true :) I think it should be ok now

It works, thanks. Hopefully @hvr can merge and publish this soon.

The CI fails for GHC 8.0.2, it may be because it is using a too-recent version of Cabal :
compiled using version <IP_ADDRESS> of the Cabal library

Build-failures like
packunused.hs:15:48: error:
    Module ‘Distribution.Package’ does not export ‘unUnitId'

means that the selected lib:Cabal version doesn't provide the expected API. I see build-depends still allows Cabal-1.24.*; should this rather be build-depends: Cabal == 2.0.* ?

@hvr yes, I should have changed the lower bound, I'll do that!

I also removed the use of LambdaCase to have it compile with older GHCs.
CI is green now!


model.fit(): batch_size vs. steps_per_epoch

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

In the method model.fit(), if "steps_per_epoch" is specified, "batch_size" cannot be specified and it defaults to "None".  This has the effect of setting batch_size to the number of samples.  For example, if you have 25,000 samples and you specify "steps_per_epoch=1000", each epoch will consist of 1000 steps, where each step is a batch of 25,000 samples.  Ouch.
Below is a test case that shows the problem.  When "use_batch_size" is set to True, it works correctly with a batch_size of 32 used.  When "use_batch_size" is set to False, it tries to process the entire 25,000 samples in one batch and runs out of memory.
# batchSizeVsSteps.py - test out model.fit using batch_size vs. steps_per_epoch
from keras.preprocessing import sequence
from keras.models import Sequential
from keras.layers import Dense, Embedding, LSTM
from keras.datasets import imdb

max_features = 20000
maxlen = 80

(x_train, y_train), (x_test, y_test) = imdb.load_data(num_words=max_features)

x_train = sequence.pad_sequences(x_train, maxlen=maxlen)
x_test = sequence.pad_sequences(x_test, maxlen=maxlen)

model = Sequential()
model.add(Embedding(max_features, 128))
model.add(LSTM(128, dropout=0.2, recurrent_dropout=0.2))
model.add(Dense(1, activation='sigmoid'))

model.compile(loss='binary_crossentropy', optimizer='adam', metrics=['accuracy'])

sample_count = len(x_train)
batch_size = 32
steps_per_epoch = sample_count // batch_size
use_batch_size = False

if use_batch_size:
    model.fit(x_train, y_train, batch_size=batch_size, epochs=1)
else:
    model.fit(x_train, y_train, steps_per_epoch=steps_per_epoch, epochs=1)

Any resolution to this?


Smooth sensitivity median -- Implementation

Placeholder to reserve a branch name for Monika.

I took a crack at a smooth sensitivity median lifted directly out of http://www.cse.psu.edu/~sxr48/pubs/smooth-sensitivity-stoc.pdf
Located here:
https://github.com/opendp/opendp/tree/171-smooth-sensitivity-median


Align .text .rodata and thread stacks to use PMP NAPOT addressing

The text and read-only data sections can be given separate permissions
with a single PMP register each using NAPOT addressing instead of TOR.
Thread stacks are also aligned by size, which saves a PMP register. This
reduces the user thread system call latency from ~18000 cycles to ~5280
cycles.
Signed-off-by: Samuel Lindemer<EMAIL_ADDRESS>

@lindemer Thanks.
I test it but don't fin any improvement on latency.
I've detected a regression because I wished to test it on userspace test but don't build anymore with your patch.
I think you modify too much useless stuff, I mean, I don't catch the reason why move "mode status" in read only area. If a product need to HW lock this memory to RO how kernel can modify this global variable if you put it between 2 RO space ?
I think you can to put Program and RO data in NAPOT just by modifying few lines in core_pmp file.

@amergnat You'll have to be more specific about what you mean by "useless stuff". You cannot simply put the .text and .rodata regions into PMP NAPOT registers without aligning them in the linker file. The is_user_mode flag is always modifiable by the kernel, because PMP settings do not have any effect in machine mode.
To replicate my results, just checkout my branch and add CONFIG_CORE_E31=y to the prj.conf in your syscall_perf sample. There must be something different between our build environments if you were able to compile the latest commit on your branch. I had to fix k_sleep and k_thread_create in the sample code to get your code to compile.

The user flag is modifiable on THIS board because you can write on this memory area but is not the case of final product which use ROM.

The user flag is modifiable on THIS board because you can write on this memory area but is not the case of final product which use ROM.

Can you please refer to a specific line in the code so I can understand you better? I have not modified how this flag is set in core_pmp.c. Is this what you are referring to?
thread->arch.u_pmpaddr[index] = TO_PMP_ADDR((ulong_t) &is_user_mode);
uchar_pmpcfg[index++] = PMP_NA4 | PMP_R;

According to you stack explanation:
*  +=========+ <--  0x0
*  +---------+ <--  pmpaddr0 [NAPOT, RX]
*  |  .text  |
*  +---------+
*  +---------+
*  |   mode  | <--  pmpaddr1 [NA4, RO]
*  +---------+
*  +---------+ <--  pmpaddr2 [NAPOT, RO]
*  | .rodata |
*  +---------+
*  +---------+ <--  pmpaddr3 [NAPOT, RW]
*  |  stack  |
*  +---------+
*  +=========+
I believe tou modify "mode" position

Question:
You do 2 NAPOT, one for program, one for ro_data, instead of making a NA + TOR (2 slot then) for program + ro_data. This is not a different way to have the same result ?

Aha I understand now. The code is actually implemented correctly, but my drawing shows the regions in the wrong order. The user mode flag is linked into RAM as it should be:
(gdb) p &is_user_mode
$1 = (ulong_t *) 0x8000001c <is_user_mode>

With regards to using NAPOT vs TOR, the way you describe would allow execution from read-only memory. NAPOT uses one register per region, so you can give them separate permissions. I'll re-order the PMP registers and fix that code comment so it's less confusing.

Ok, still an issue with the linker:
[119/134] Linking C executable zephyr/app_smem_unaligned_prebuilt.elf FAILED: zephyr/app_smem_unaligned_prebuilt.elf  ... /opt/zephyr-sdk/riscv64-zephyr-elf/bin/../lib/gcc/riscv64-zephyr-elf/9.2.0/../../../../riscv64-zephyr-elf/bin/ld: zephyr/app_smem_unaligned_prebuilt.elf section noinit' will not fit in region RAM' /opt/zephyr-sdk/riscv64-zephyr-elf/bin/../lib/gcc/riscv64-zephyr-elf/9.2.0/../../../../riscv64-zephyr-elf/bin/ld: region RAM' overflowed by 6782 bytes
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1
`

Are you running west build -b hifive1_revb && west flash or are you using some other command? This command should work on the latest SDK.

You're NAPOT method allow more granularity ok. Then 2 questions

Do we need to setup ro_data and program different ?
If another read only part is needed (vendor data, ...), that mean we will use more PMP slot (compare to the current NA + TOR method). Should we need to be worried about that ?

This is not a command issue, please try to build:
"west build -b qemu_riscv32 tests/kernel/mem_protect/userspace"

This is not a command issue, please try to build:
"west build -b qemu_riscv32 tests/kernel/mem_protect/userspace"

I see, I thought you were referring to your syscall_perf sample. I'll look more into this and figure out why enabling aligned stacks in include/arch/riscv/arch.h is causing RAM to overflow.

Were you able to reproduce the results in syscall_perf or do you get a linker error?
Do you have a physical HiFive Rev B to test the code on so we're testing on the same board?

I think my modifications to core_pmp.c may not work with stack guards. I'll mark the pull request as ready for review when I resolve that.

Ok, still an issue with the linker:
[119/134] Linking C executable zephyr/app_smem_unaligned_prebuilt.elf FAILED: zephyr/app_smem_unaligned_prebuilt.elf ... /opt/zephyr-sdk/riscv64-zephyr-elf/bin/../lib/gcc/riscv64-zephyr-elf/9.2.0/../../../../riscv64-zephyr-elf/bin/ld: zephyr/app_smem_unaligned_prebuilt.elf section noinit' will not fit in region RAM' /opt/zephyr-sdk/riscv64-zephyr-elf/bin/../lib/gcc/riscv64-zephyr-elf/9.2.0/../../../../riscv64-zephyr-elf/bin/ld: region RAM' overflowed by 6782 bytes
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1
`

I just tried compiling tests/kernel/mem_protect/userspace for both qemu_riscv32 and hifive1_revb from your latest commit and they both overflow RAM. This doesn't appear to be a problem that I introduced.

Not reproducible on syscall_perf
HiFive Rev B doesn't support Zephyr's userspace test cause of lack of PMP slot

I stopped to work on HiFive Rev B because maintainers can't test our patch. It must work on QEMU.

I just tried compiling tests/kernel/mem_protect/userspace for both qemu_riscv32 and hifive1_revb from your latest commit and they both overflow RAM. This doesn't appear to be a problem that I introduced.

"west build -b qemu_riscv32 tests/kernel/mem_protect/userspace" works perfectly without your patch, make a try and don't forget to follow my setup described in my cover letter ;)

@amergnat Could you clarify why init_user_accesses supports a stack guard when userspace is enabled? This is redundant if I understand correctly. In the docs it says "This feature is optional and is not required to catch stack overflows in user mode; disabling this may free 1-2 MPU regions depending on the MPU design."

An user thread have 2 stack, one kernel and one user. If stack guard is enabled, I apply it to the kernel stack (which is used during syscall)

An user thread have 2 stack, one kernel and one user. If stack guard is enabled, I apply it to the kernel stack (which is used during syscall)

It appears that a stack guard is also applied to user-mode threads in init_user_accesses. Since user-mode thread stacks are already in a PMP region, adding a stack guard shouldn't have any effect. Am I misunderstanding something here?

You understand well, I do that to avoid kernel during a syscall when it write user ESF to go over user stack (and then damaged the next stack)

Closing this now; will submit as a PR to upstream Zephyr at a later date!


Sparse

What changes were proposed in this pull request?
This is a rebase of your branch on the most recent master
How was this patch tested?
Does not compile fully, but for good reason:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.5.1:compile (default-compile) on project nd4j-api: Compilation failure: Compilation failure:
[ERROR] /home/huitseeker/DL4J/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/blas/impl/SparseBaseLapack.java:[9,8] org.nd4j.linalg.api.blas.impl.SparseBaseLapack is not abstract and does not override abstract method gesvd(org.nd4j.linalg.api.ndarray.INDArray,org.nd4j.linalg.api.ndarray.INDArray,org.nd4j.linalg.api.ndarray.INDArray,org.nd4j.linalg.api.ndarray.INDArray) in org.nd4j.linalg.api.blas.Lapack
[ERROR] /home/huitseeker/DL4J/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/blas/impl/SparseBaseLapack.java:[15,5] method does not override or implement a method from a supertype
[ERROR] -> [Help 1]

Full log: https://gist.github.com/5ecbff85fbcf1c6e018a19bb9c872faa

Made obsolete after discussion with @audreyloeffel


Clear text in search box when the close button is clicked

Fix for bug: Model Tree Search - Clear doesn't remove search
description + repro steps:
Clicking the 'X' on the search does not remove the search criteria.
Steps:
Open Model Tree
Click search box
enter search criteria - tree filters as expected
Click the 'X' in the search box to clear the search
Result - the search box closes but the filter is still applied to the tree
User most close and reopen the model tree to see the full tree.
The 'X' should clear the search filter.

@calebmshafer still no write access 😔 can you merge this for me?


[zh] sync /controllers/cron-jobs.md

Sync with en text:
content/zh-cn/docs/concepts/workloads/controllers/cron-jobs.md

See preview please.

/approve


Update issue templates

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #79   +/-   ##
=====================================
  Coverage       0%     0%
=====================================
  Files          37     37
  Lines        1537   1537
=====================================
  Misses       1537   1537

Continue to review full report at Codecov.

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


Making run in Android device more convenient

**Godot version:3.x

OS/device including version:

Issue description:

If only one Android  device is connected, just click the android icon to run on that device directly, without one more click to selecting the device.
2.since it take some time  to install and run on device, it would be nice to have an option to make this process running in the background.
Steps to reproduce:

Minimal reproduction project:

If only one Android device is connected, just click the android icon to run on that device directly, without one more click to selecting the device.

This seems to be already a thing in 3.2

Note that 1. was implemented already.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


This commit adds the pkg_provider 'chocolatey' to the module

When using 'chocolatey' as pkg_provider it is not needed to
include :staging, this is excluded in the init.pp and forwarder.pp

We're using Chocolatey as package-provider, for that we need to 'skip' the include of ::staging, this PR fixes that.

Hi @ralfbosz, sorry for the late feedback. Could you please rebase against latest master and take a look at the inline comments?

When pkg_provider is "chocolatey" the source needs to be 'empty', so added selector, but being strict it's a selector in a selector with the 'pick' which is also there...


Image Upload in FlashPlayer

Hi,
I am new to GitHub. So I don't think this should go here. But I found this works for uploading an image using flashplayer as opposed to AIR (i.e. no 'fileReference.uploadUnencoded()'). Useful if you are doing a web based app and don't wanna use java etc. Might wanna add this to the Upload example code section.
var appID:String = "YOUR-APP-ID";

public function uploadFile(): void {
			var fileRef: FileReference = new FileReference();
			var fileFilter: FileFilter = new FileFilter("Images", "*.jpg");
			fileRef.addEventListener(Event.SELECT, fileSelected)
			fileRef.browse([fileFilter]);
			function fileSelected(event: Event): void {
				fileRef.addEventListener(Event.COMPLETE, onFileLoaded);
				fileRef.addEventListener(IOErrorEvent.IO_ERROR, ioError);
				fileRef.addEventListener(ProgressEvent.PROGRESS, onProgress);
				fileRef.load();
			}
			function onFileLoaded(event: Event): void {
				var fr: FileReference = event.target as FileReference;
				var data: ByteArray = fr["data"];
				fileRef.removeEventListener(Event.COMPLETE, onFileLoaded);
				fileRef.removeEventListener(IOErrorEvent.IO_ERROR, ioError);
				fileRef.removeEventListener(ProgressEvent.PROGRESS, onProgress);

				var request: URLRequest = new URLRequest("https://firebasestorage.googleapis.com/v0/b/"+appID+".appspot.com/o/ArtWorks%2F" + fr.name);
				request.method = URLRequestMethod.POST;
				request.data = data;

				var loader: URLLoader = new URLLoader();
				loader.addEventListener(flash.events.Event.COMPLETE, uploadComplete);
				loader.addEventListener(flash.events.IOErrorEvent.IO_ERROR, ioError);
				loader.load(request);
			}

			function uploadComplete(event:Event):void{
				trace("upload complete")
			}

			function ioError(event: IOErrorEvent): void {
				trace("ioError")
			}
			function onProgress(event: ProgressEvent): void {
				trace(event.bytesLoaded + "/" + event.bytesTotal);
			}
		}

Thank you for sharing this!
I actually tried to use Firebase database in a Flash Player project but for some reason I wasn't able to connect to it. That's why I ditched the idea of using Firebase on Flash Player altogether.
Since Firebase Storage is actually Google Cloud Storage it might be the reason that one works well on the Flash Player.

@agentphantom I haven't tried using it through the web. But I was able to get an upooad happening through flash player on desktop. Might require some crossdomain exceptions

More progress on the Flash Player based code. You will need to use a phpProxy such as the one attached and affix the upload URL on your website to your Firebase url request. I've tested this with getting data from realtime database but I am still yet to be able to send.
 To access a node called 'World' just use rawData.World
}

phpProxy.php.zip

Very interesting. I remember doing something similar like this in the Twitter 1.0 API days.
I have been working on something similar. A port of this guide but for Python which can be used as a proxy.
https://github.com/agentphantom/firebase-python

@agentphantom
I am finding that the php works for images other than the ones used on firebase storage. Perhaps it is to do with the download tokens.

@agentphantom So I bit of an update. The php proxy which I've attached earlier can read data from the web. But when it goes to 'PATCH' data it returns what is already there rather than updating it. I am looking into this however if you've got any suggestions seeing as you've worked through a php proxy before let me know.
Here is the php proxy at the moment:
<?php

$post_data = $HTTP_RAW_POST_DATA;

$header[] = "Content-type: text/xml";
$header[] = "Content-length: ".strlen($post_data);

$ch = curl_init( $_GET['url'] );
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_TIMEOUT, 10);
curl_setopt($ch, CURLOPT_HTTPHEADER, $header);

if ( strlen($post_data)>0 ){
    curl_setopt($ch, CURLOPT_POST, 1);
    curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
}

$response = curl_exec($ch);
$response_headers = curl_getinfo($ch);

if (curl_errno($ch)) {
    print curl_error($ch);
} else {
    curl_close($ch);
    header( 'Content-type: ' . $response_headers['content-type']);
    print $response;
}

?>

I am wondering what would be needed so that the Header will 'PATCH' data rather than just returning it.

@agentphantom Got it working! So this has gotten a bit derailed from the original upload in flashplayer topic. But here is where I found luck at patching to Firebase
So all you need is a couple of tweaks to the php and the URLRequest. Here is all you need to change.
First the PHP
<?php

$post_data = $_POST['data'];

$ch = curl_init( $_GET['url'] );
$header = array( "X-HTTP-Method-Override: PATCH");
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_TIMEOUT, 10);
curl_setopt($ch, CURLOPT_HTTPHEADER, $header);

if ( strlen($post_data)>0 ){
    curl_setopt($ch, CURLOPT_POST, 1);
    curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
}

$response = curl_exec($ch);
$response_headers = curl_getinfo($ch);

if (curl_errno($ch)) {
    print curl_error($ch);
} else {
    curl_close($ch);
    header( 'Content-type: ' . $response_headers['content-type']);
    print $response;
}

?>

Now for all you need to change in the URLRequest from the post earlier
var proxyURL:String = [URL TO PROXY ABOVE]
var directory:String = [TARGET-DIRECTORY]
var request: URLRequest = new URLRequest(proxyURL+directory + "/.json");
			var urlVars:URLVariables = new URLVariables();
			urlVars.data = JSON.stringify(myObject); // create this
			//request.data = JSON.stringify(myObject); // Instead of sending this
			request.method = URLRequestMethod.POST;
			request.requestHeaders.push(header);
			request.data = urlVars // so you can send this


Added mapping of media volume

Proposed Changes
The change adds mapping of media volume for AppDaemon so that it's able to manipulate content stored there.
I'm reading some images from external systems with an AppDaemon app, process them and then store them in media volume for usage through Home Assistant (for example you can cast the result to a display).

Is this change something that requires discussion (I can imagine that someone can have problems with AppDaemon having access to more and more things) or is it a no-brainer?

Anyone? Seems to be a small change, only concern could be security...
In the meantime I'm running the modified version for some time without any problems, feeding some images back to Home Assistant through media folder after processing.


ClusterExternalSecret + matchExpressions not detecting new namespaces

I want to create a secret containing the container registry credentials in every namespace.
I used this namespaceSelector config:
  namespaceSelector:
    matchExpressions:
      - key: kubernetes.io/metadata.name
        operator: Exists

It works as expected, now all my namespaces have the docker registry credentials.
The problem is that if I create a new namespace it is not detected automatically.
The only way is to delete the ClusterExternalSecret and create it again.
Can we make the ClusterExternalSecret detect new namespaces?

I found the problem, I had the refreshTime set to 1h, I changed it to 1 minute and it works as expected.
Working example:
apiVersion: external-secrets.io/v1beta1
kind: ClusterExternalSecret
metadata:
  name: test
spec:
  externalSecretName: test
  refreshTime: "1m"
  namespaceSelector:
    matchExpressions:
      - key: kubernetes.io/metadata.name
        operator: Exists
  externalSecretSpec:
    secretStoreRef:
      name: vault-backend
      kind: ClusterSecretStore
    refreshInterval: "1h"
    target: {}
    dataFrom:
      - extract:
          key: my/vault/path


Attempt to fix multiline prompt resize issue

Tried to fix #809 in this PR.
The problem lies in the handle_resize function https://github.com/nushell/reedline/blob/4634f71705785a772c894aa45e30db88f3c7a8d5/src/painting/painter.rs#L475-L492 where the new position is set according to cursor, while the cursor's row is larger than the prompt start row in multiline cases.

Thanks for taking this on. We've had prompt issues for a long time and a while back (incorrectly) disabled some of it. What I've learned since then is some terminals, when resized, leave it to the shell to redraw things. Ghostty is one of those shells. May be related or unrelated, just thought I'd share.
I'm trying to figure how where/how to test this. Do we have an example with a multi-line prompt?

Thanks for taking this on. We've had prompt issues for a long time and a while back (incorrectly) disabled some of it. What I've learned since then is some terminals, when resized, leave it to the shell to redraw things. Ghostty is one of those shells. May be related or unrelated, just thought I'd share.
I'm trying to figure how where/how to test this. Do we have an example with a multi-line prompt?

No test case found for the resize handling.
Hope these recordings are convincing.
nu 0.98
https://github.com/user-attachments/assets/24d722d6-3ded-449f-b155-cbdf16ee97a1
nu 0.98 + fixed reedline
https://github.com/user-attachments/assets/f8412d90-5f86-4770-b71c-579108534766

BTW, I tested it on both wezterm and alacrity. This PR probably won't fix all multiline prompt drawing issues at once, but the code I quoted above is obviously buggy, prompt_start_row will just increase over each resize op. So I think we'd better start from fixing this function.

When I tested earlier, it seems ok on the left prompt, but the right prompt was still wrapping but not unwrapping. I'm fine with leaving that to another PR.

When I tested earlier, it seems ok on the left prompt, but the right prompt was still wrapping but not unwrapping. I'm fine with leaving that to another PR.

You are right, the right prompt wrapping is not tackled at all. I guess another call of repaint inside handle_resize can fix it (more changes, not sure we want it). In my experience, the left prompt is much more annoying because tmux new pane will trigger it.

Maybe in another PR because I'm guessing there will be edge cases.

Let's run with this to see how it goes this week.


Record & Replay - Fix issue of holding left click

Contains
Fixes the issue in which holding left click during a recording was causing replays to not reproduce block destruction correctly.
How to test
Record a game in which you destroy blocks by holding left click and then watch its replay to check that everything was destroyed correctly.
Outstanding before merging
If anything. You can use neat checkboxes! Feel free to delete if not needed

[ ] This PR fixes the "left click" problem, but the same issue exists when holding "right click".

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Tested it out. Pretty subtle, but I could provoke a difference before vs after :-)
Merged, thanks!


ALL ACS Policy Definitions should be in repo

ASC Azure policies listed on this page should have a corresponding GitHub repo for Azure Policy definitions json files for change tracking.

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

ID: 9c78cb0a-edfb-a403-1096-1aeaaa5344fc
Version Independent ID: 77d5f52a-08ad-edc0-14e5-f8f9fb05c739
Content: Azure policy definitions monitored in Azure Security Center
Content Source: articles/security-center/security-center-policy-definitions.md
Service: security-center
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

@nconder
Thanks for the feedback! We are looking into this and will get back to you shortly.

I have assigned this issue to the content author to look into your suggestion. You can also leave this feedback in User Voice. https://feedback.azure.com/forums/34192--general-feedback

Feedback request link
https://feedback.azure.com/forums/347535-azure-security-center/suggestions/37504357-publish-azure-security-center-default-azure-police


[webview_flutter] Send history events

Description
Send URL change events when the user navigates in the Browser. This also works for JavaScript Navigation that does not actually make any request and therefore would not trigger page load or finish events.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

hi @wwwdata could you please rebase (& start the next version number)

hi @wwwdata could you please rebase (& start the next version number)

Just rebased, increased version number and pushed. Thanks for the fast feedback

@ened Thanks a lot for the good review feedback. Implemented the requested changes.

What's the update on this?

Hi @wwwdata, is this pull request still active ?
I also need this implementation, and see that's it is an old PR.
I see that it requires @amirh approval, any update on this feature ?
Thanks.

So far I didn't hear anything official... I would be happy to rebase and fix all conflicts here again if somebody then also does a review...

I also need this as well

This would be a great feature to have :)

This would be a great feature to have :)

I am going to close this as stale. Feel free to re-open if you are going to rebase and apply requested changes.

I am planning to look at adapting this to the current state of the plugins once the iOS conversion to Dart+Pigeon is complete.

The Dart conversion is complete, but in the meantime we found https://github.com/flutter/flutter/issues/94051 which blocks this, since we currently implement WebViewPlatformCallbacksHandler in our implementations, making changes to it breaking changes. We're actively working on that issue, and hope to have it resolved soon.

Status update from triage: this is still blocked, but we hope to have it unblocked soon.

This is unblocked now that v4 has landed. @bparrishMines This is one we should look at updating.

Closing as this was obsoleted by https://github.com/flutter/plugins/pull/7113; the implementation had changed enough that it was easiest to start a new PR rather than update this one. Thanks!


support netstandard 1.6

Please have a look. If there is no problem and update the nuget.

Can you please commit only changes you made in order to support netstandard? You committed a lot of formatting changes which doesn't have anything to do with supporting netstandard.

Ok, I will repull them.


Please add tests

🙏🏾
https://github.com/JuliaData/Parsers.jl/issues/150#issuecomment-1315116507

Sure, but I only have limited time. Contributions welcome!


Chaining check causes duplicated errors

request.check('email', 'Email is not valid').notEmpty().notNull().isEmail();
With no "email" field set this will return 3 repeatitions of the same message.
Shouldn't the error message be unified on chained call like this?
Thank you

Please merge! :-)

So how was it? merged?

Any updates?

I will look into this. The only issue is that it would require a major version update since it would break all unit tests (not to mention people's existing code in the wild). I won't be able to touch it this weekend, but I'll try to get to it this week.

Would there be an issue by adding it as an option in the middleware options?

That could work.
Thanks,
Rusty Bailey

On Oct 18, 2015, at 4:37 PM, Joshua Filby<EMAIL_ADDRESS>wrote:
Would there be an issue by adding it as an option in the middleware options?
—
Reply to this email directly or view it on GitHub.

This one should be fixed ASAP guys!

+1 Please merge. To make it backwards compatible and not break existing installations, turn this into an option with the default being current behavior.

Hi, did you merge this fix? It's very important...

Has there been made any progress on this 3 year old issue?
I still get duplicate error messages when chaining, and I can't find anything in the docs about preventing it by passing in anything in the middleware options.

I don't really see this as a breaking change for existing code out there, as we could just ad an additional option when attaching express validator to the app to tell if we want to have duplicate errors or not.
What do you think?

Hi @TheHominid. I'm glad that you found something that works for you at the moment.
Anyway, with the introduction of #269, it's easy to add opt-in support to new methods that fix issues like this.
That's my plan, so this issue can finally be closed.

Hi everyone! The fix for this has finally landed in v3.0.0!
req.getValidationResult().then( result => {
  var errors = result.useFirstErrorOnly().array(); // enjoy an array with no duplicated errors for any given parameter!
})

@Hassaan-Ahmed please open a new issue with detailed information.


Add try from str

@badboy rebased away the version change commit, should be good to go now!

@badboy okay to merge and release?


[Backport 23.05] zellij: 0.36.0 -> 0.39.0

Description of changes
This PR backports updates to the zellij package.
Main motivation is https://github.com/zellij-org/zellij/issues/1625 which was fixed with the 0.38.0 release of zellij.
All commits where cherry-pick -xed.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 267401 run on x86_64-linux 1

1 package built:

zellij

Shouldn't we be waiting for #267174 ?

If that PR is merged before this one, I'll gladly cherry-pick the commit to this PR as well!


Package import specifier "#imports" is not defined in package

Environment

Operating System: Windows_NT
Node Version:     v16.15.0
Nuxt Version:     3.0.1-rc.0-27898092.1e8da22
Nitro Version:    2.0.0-27897925.b91db6f
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      extends, modules
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/florian-lefebvre/nuxt-typo-readme/tree/windows-import-error
Describe the bug
When running the app using pnpm dev, I get the following error:
Package import specifier "#imports" is not defined in package

Additional context
It should be fixed by nuxt/nuxt.js#14084 and nuxt/framework#5461
Logs
[nuxt] [request error] [unhandled] [500] Package import specifier "#imports" is not defined in package D:\DEV\PROJECTS\nuxt-typo-readme\node_modules\.pnpm\@nuxt+content-edge<EMAIL_ADDRESS>imported from D:\DEV\PROJECTS\nuxt-typo-readme\node_modules\.pnpm\@nuxt+content-edge<EMAIL_ADDRESS>  at new NodeError (node:internal/errors:372:5)
  at throwImportNotDefined (node:internal/modules/esm/resolve:461:9)
  at packageImportsResolve (node:internal/modules/esm/resolve:841:3)
  at moduleResolve (node:internal/modules/esm/resolve:997:21)
  at defaultResolve (node:internal/modules/esm/resolve:1218:11)
  at ESMLoader.resolve (node:internal/modules/esm/loader:580:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:294:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:80:40)
  at link (node:internal/modules/esm/module_job:78:36)

Are you still experiencing this?

I don't know tbh; I'll try again and let you know

Looks like its revolved with "nuxt": "npm:nuxt3@latest" thanks!


[ic-date-input] - spacing of input field and helper text does not match designs

Summary of the bug
When a date input is displayed with helper text, the input field is too close to the helper text
📸 Screenshots or code
figma (on the left) has more spacing:

Additional info
visual regression baseline images will need updating

hey @ad9242 , i would like to take up the issue.
Could you please give some more details on the bug!!

Do this in the v3 branch with it being for Text Field


Landing - Search

Could we also close issues #5990 and #5991 ?

I search "Decoded" from landing

btw new stmn is gone and now ahk, hopefully end users won't be confused


Keep retry registering node controller with RegistryServer

Instead of exiting with error, driver has to wait and retry 'node controller'
registration till the registry server up and registration get succeed.
FIXES: #182
Signed-off-by: Amarnath Valluri<EMAIL_ADDRESS>

good improvement!
retrytimeout was just not used and we set again to 10 sec it seems.. good fix as well.

Confirmed, works fine in my "make test" cluster

@pohly  is it good enough to merge?


Rename Greek translation folder

Closes #109
The official ISO 2-letter language code for Greek is el.

Hello @kobakei, This fix is absolutely needed, no application can be published in .aab format without renaming the language to 'el'.

@kobakei please merge this ASAP. Using this as a temporary workaround
bundle {
    density {
        // Different APKs are generated for devices with different screen densities; true by default.
        enableSplit true
    }
    abi {
        // Different APKs are generated for devices with different CPU architectures; true by default.
        enableSplit true
    }
    language {
        // This is disabled so that the App Bundle does NOT split the APK for each language.
        // We're gonna use the same APK for all languages.
        enableSplit false
    }
}


cbmc: 5.65.1 -> 5.67.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cbmc/versions.
meta.description for cbmc is: "CBMC is a Bounded Model Checker for C and C++ programs"
meta.homepage for cbmc is: "http://www.cprover.org/cbmc/"
meta.changelog for cbmc is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.66.0 with grep in /nix/store/ps1pdn5nywlxl1mp4s7xkgkrs76kirg5-cbmc-5.66.0
found 5.66.0 in filename of file in /nix/store/ps1pdn5nywlxl1mp4s7xkgkrs76kirg5-cbmc-5.66.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cbmc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ps1pdn5nywlxl1mp4s7xkgkrs76kirg5-cbmc-5.66.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cbmc https://github.com/r-ryantm/nixpkgs/archive/5bbaedbd24a4aeeece0f7f1a688047b565a449f6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ps1pdn5nywlxl1mp4s7xkgkrs76kirg5-cbmc-5.66.0
ls -la /nix/store/ps1pdn5nywlxl1mp4s7xkgkrs76kirg5-cbmc-5.66.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cbmc

Maintainer pings
cc @jiegec for testing.

Result of nixpkgs-review pr 191613 run on x86_64-linux 1

1 package built:

cbmc

Result of nixpkgs-review pr 191613 run on x86_64-linux 1

1 package built:

cbmc


Issue #130: KeyError in Alter Field Migration

Subject: Resolves #130
Feature or Bugfix

Bugfix

Purpose

The template used in the specific part of the code clearly has a variable in it that is not present in the supplied dictionary. Using an empty string for that value is how that is handled in the base django package.

django-redshift-backend 4.0.0 has been released.
I think I've solved this issue, so please check. If it doesn't seem to be resolved, please open an issue.
https://pypi.org/project/django-redshift-backend/4.0.0


Function calling integrated with the browser

Hello @n4ze3m , first of all, congratulations for the idea and implementation of Page Assist. What an amazing feat!
After using it for a while, I was wondering what you think about giving it superpowers by leveraging function calling support that exists in a lot of models. I explain:

imagine if I create a prompt to extract structured information (named entities, for example) from web pages AND copy the results as JSON (or pairs label-value) to the clipboard;
every time I visit one page of interest, I just open your sidebar and ask it to run my prompt, which makes Page Assist to process the current page, extract the data and copy the results to clipboard;
if I have another extension that pastes structured data from clipboard into form fields, you just created an automation tool for data entry.

Thanks for your attention!

Hey, thank you for the suggestion. This will be a great feature, and I will try to add it in upcoming updates

I thank you!!


Add snippet for Android sound classification

/gcbrun

/gcbrun

If you run npm run lint theres a couple of simple lint errors in the markdown file.
Here's what currently triggering error:
Step #1: markdown/audio/tflite/Android.md: 1: MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "## Test your model with our sample app"]
Step #1: markdown/audio/tflite/Android.md: 3: MD013/line-length Line length [Expected: 100; Actual: 136]
Step #1: markdown/audio/tflite/Android.md: 3: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. Download the [sample app](h..."]
Step #1: markdown/audio/tflite/Android.md: 5: MD013/line-length Line length [Expected: 100; Actual: 159]
Step #1: markdown/audio/tflite/Android.md: 7: MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "## Integrate your model into your own app"]
Step #1: markdown/audio/tflite/Android.md: 9: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. Put the `soundclassifier.tf..."]
Step #1: markdown/audio/tflite/Android.md: 10: MD013/line-length Line length [Expected: 100; Actual: 301]
Step #1: markdown/audio/tflite/Android.md: 11: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "3. Initialize a `SoundClassifi..."]
Step #1: markdown/audio/tflite/Android.md: 12: MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```kotlin"]
Step #1: markdown/audio/tflite/Android.md: 18: MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]
Step #1: markdown/audio/tflite/Android.md: 19: MD029/ol-prefix Ordered list item prefix [Expected: 1; Actual: 4; Style: 1/1/1]
Step #1: markdown/audio/tflite/Android.md: 19: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "4. Start capturing live audio ..."]
Step #1: markdown/audio/tflite/Android.md: 20: MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]
Step #1: markdown/audio/tflite/Android.md: 20: MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
Step #1: markdown/audio/tflite/Android.md: 22: MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]
Step #1: markdown/audio/tflite/Android.md: 23: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Step #1: markdown/audio/tflite/Android.md: 23: MD013/line-length Line length [Expected: 100; Actual: 161]
Step #1: markdown/audio/tflite/Android.md: 23: MD029/ol-prefix Ordered list item prefix [Expected: 1; Actual: 5; Style: 1/1/1]
Step #1: markdown/audio/tflite/Android.md: 23: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "5. Receive classification resu..."]
Step #1: markdown/audio/tflite/Android.md: 24: MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```kotlin"]

I've fixed the lint errors.

LGTM! thanks for adding the notes on the emulator!

/gcbrun

/gcbrun


Create a Contribution Guide

Write a comprehensive contribution guide for this repository, explaining how contributors can suggest new repositories, update existing ones, and help maintain the list.

I'll work on this assign me this.

@reallywasi
Assigned the issue to you


Update all code recipes from code to 3.1

I am working on this and obtaining the inputs from reviewers


Support Qwen2-VL's multimodal RoPE implementation

Summary

Support Qwen2-VL's multimodal RoPE kernel. See original implementation here: https://github.com/huggingface/transformers/blob/a3d69a8994d673899608a7c17fbf4f953f50474e/src/transformers/models/qwen2_vl/modeling_qwen2_vl.py#L203-L245
Finished the TODO left in #175. Complete feature request #165.

Testing Done

Hardware Type: A800-SXM4-80GB
[x] run make test to ensure correctness
[x] run make checkstyle to ensure code style
[x] run make test-convergence to ensure convergence

Very nice!
Can you update the convergence tests for qwen2_vl to include your RoPE implementation as well? Right now there is a line excluding rope for qwen2_vl models specifically.
Also are you available to add a benchmark for your kernel? https://github.com/linkedin/Liger-Kernel/blob/main/docs/CONTRIBUTING.md#adding-a-new-kernel

Thank you for making the non-trivial contribution!

Can we rebase https://github.com/linkedin/Liger-Kernel/pull/388 to properly test convergence?
I notice there is another apply_rotary_pos_emb_vision, should we implement and patch that too?
Are you on wechat? Very impressed by your contribution and want to discuss more with you. My id is wxid_nn8pbmlh9ae712

Thank you for making the non-trivial contribution!

Can we rebase Qwen2-VL Bug / Incompatibility Fixes #388 to properly test convergence?
I notice there is another apply_rotary_pos_emb_vision, should we implement and patch that too?
Are you on wechat? Very impressed by your contribution and want to discuss more with you. My id is wxid_nn8pbmlh9ae712

@ByronHsu Hi,

I've already rebased onto the latest master including this commit #388 and enabled RoPE for Qwen2VL.
The current apply_rotary_pos_emb_vision implementation is inefficient since it recomputes cos & sin for q & k. It's better to optimize it in modeling_qwen2_vl.py in upstream transformers to apply RoPE on both q & k at the same time. Then we can reuse the RoPE triton kernel of llama / mistral.
Thanks for the invitation. I'm on wechat but I couldn't find your account based on wxid. Maybe you could email me the QR code?

Thank you @li-plus !!


Add support for wildcard artifacts

Using an artifact like *.tar.gz does not work as it does for the standard GoCD Artifact publishing (where all files matching the regex will get published).
Using a specific filename like templates.tar.gz does work.

I am working on many improvements to this plugin with a goal to make it a substitute for the GoCD artifact store. I have already implemented pattern matching and directory upload and fetch #2.
WIP is in my fork - https://github.com/tomzo/gocd-s3-artifact-plugin/tree/dirfetch

@tomzo feel free to open a PR here if you want me to add your changes to the Plugin and make a new version of it :)

Close this as it was fixed by PR #11


Allow Ledger to sign Komodo reward claim transactions

Resolves https://github.com/LedgerHQ/ledger-app-btc/issues/60
In Komodo you can claim rewards (essentially interest) based on the age of your UTXOs. This mints the new coins in the claim transaction which means the output amount is expected to be larger than the input amount.
Asking the Ledger to sign a Komodo reward claim transaction will result in an error because it won't allow a transaction to have a larger output amount than an input amount.
This PR ignores that check for Komodo. This simple change is wrapped in G_coin_config->kind == COIN_KIND_KOMODO so it will not affect any other currencies. It also will not break the way Komodo currently works in Ledger Live or other existing Komodo Ledger apps. Normal transactions will continue to work as normal, but now reward claim transactions can be signed too.
Let me know if this is ok or if you require any changes :)

It is a pity not to be able to make from "Ledger wallet" the claim of the reward in Komodo.
As a user of Ledger and Komodo I ask Ledger to implement the changes proposed by Lukechild to solve the problem.

For everyone interested in this, it's been approved internally and should be merged soon:

Merged for 1.3.0


Environmental variables at compile time not available

Image: Elixir
Code that fails the build:
config :my_app,
  fcm_batch_size: "FCM_BATCH_SIZE" |> System.get_env() |> String.to_integer()

Error:
Compiling application :
  ✓ Starting docker container
  ✓ Preparing environment for compile
  ! Compiling code
    - Cleaning up any previous releases...
    - Copying code...
    - Compiling mix dependencies :
      ** (ArgumentError) argument error
          :erlang.binary_to_integer(nil)
          (stdlib) erl_eval.erl:677: :erl_eval.do_apply/6
          (stdlib) erl_eval.erl:885: :erl_eval.expr_list/6
          (stdlib) erl_eval.erl:240: :erl_eval.expr/5
          (stdlib) erl_eval.erl:232: :erl_eval.expr/5
          (stdlib) erl_eval.erl:233: :erl_eval.expr/5
          (stdlib) erl_eval.erl:885: :erl_eval.expr_list/6
          (stdlib) erl_eval.erl:408: :erl_eval.expr/5

    ! FAILED TO COMPILE APP !

    Exit
     1

    Command
     su - gonano -c "cd /opt/nanobox/engine/bin; export PATH=\"/data/sbin:/data/bin:/opt/gonano/sbin:/opt/gonano/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin\"; export CODE_DIR=\"/app\"; export DATA_DIR=\"/data\"; export APP_DIR=\"/mnt/app\"; export CACHE_DIR=\"/mnt/cache\"; export ETC_DIR=\"/data/etc\"; export ENV_DIR=\"/data/etc/env.d\"; export CONFIG_RUNTIME_TYPE=\"string\"; export CONFIG_RUNTIME_VALUE=\"'elixir-1.7'\"; export CONFIG_ERLANG_RUNTIME_TYPE=\"string\"; export CONFIG_ERLANG_RUNTIME_VALUE=\"'erlang-21'\"; export CONFIG_NODES=\"runtime,erlang_runtime\"; stdbuf -o0 /opt/nanobox/engine/bin/compile '{\"code_dir\":\"/app\",\"data_dir\":\"/data\",\"app_dir\":\"/mnt/app\",\"cache_dir\":\"/mnt/cache/app\",\"etc_dir\":\"/data/etc\",\"env_dir\":\"/data/etc/env.d\",\"config\":{\"erlang_runtime\":\"erlang-21\",\"runtime\":\"elixir-1.7\"}}'"

    Output
     - Compiling mix dependencies :
       ** (ArgumentError) argument error
           :erlang.binary_to_integer(nil)
           (stdlib) erl_eval.erl:677: :erl_eval.do_apply/6
           (stdlib) erl_eval.erl:885: :erl_eval.expr_list/6
           (stdlib) erl_eval.erl:240: :erl_eval.expr/5
           (stdlib) erl_eval.erl:232: :erl_eval.expr/5
           (stdlib) erl_eval.erl:233: :erl_eval.expr/5
           (stdlib) erl_eval.erl:885: :erl_eval.expr_list/6
           (stdlib) erl_eval.erl:408: :erl_eval.expr/5

Error   : failed to execute hook (compile) on a6cc6f1fd118eb708da7cc00e4b1311961b2dbc2178fdc3ae225db435ee13efe: util:Exec:/opt/nanobox/hooks/compile: bad exit code(1):
Context : failed to compile the code -> failed to run the (compile)compile hook

Is there a way to make env vars available at compile time?

No, partly because a compile isn't meant to be used in one exclusive environment, and partly because binaries should never be built with environment values hard coded into the compiled result - that misses the entire point of environment variables.

So how can I use secret key base or external DB URL in my config files?
On Fri, 12 Oct 2018 at 07:00, Dan Hunsaker<EMAIL_ADDRESS>wrote:

No, partly because a compile isn't meant to be used in one exclusive
environment, and partly because binaries should never be built with
environment values hard coded into the compiled result - that misses the
entire point of environment variables.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nanobox-io/nanobox/issues/685#issuecomment-429204685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADDRZdPaWCKUgIk4dbGuA-Yito4fl6gKks5ukCHbgaJpZM4XX9fn
.

Same way. Especially for DB info/credentials, you want to pull those from the environment at runtime – even with only a single deploy environment, that information may change between compile and startup.
As to the exact approach you should use to ensure that happens correctly, that's highly language- and framework-dependent, so you're better asking about that in communities more familiar with those.

OK, that won't work in my case. Too bad, thanks.

I know Elixir supports it. I don't know how. Ask around.


[DOC] cyclecloud import_cluster or import_template

Hello
I am following the readme, and I am wondering in the section "Start an LSF cluster" if you want to run
cyclecloud import_cluster LSF -f lsf.txt -t
or
cyclecloud import_template LSF -f lsf.txt -t
Thanks,
Alex

Thanks AJ - modified the readme to be more clear. Note: import_cluster has a sub command import_template which can be invoked by -t
https://github.com/Azure/cyclecloud-lsf/blame/master/README.md#L195


I can't fix player slot 1.

So in Fortnite I couldn't move in game so I went here to try and fix it. When I go to Device manager the Xbox 360 Peripherals don't show up, even when I click "show hidden devices." How can I fix this?

Reach out on Discord in the #double-movement-support channel for a Live chat.
We can help you out better over there.

You should not have any problems with Xbox 360 Peripherals on the latest version of this app as it has switched over to use DS4 since version v1.2.0.
Download the latest version of the app here, or from the Release page.


Feat/refactoring

📑 NOTE
This pr includes a lot of changes.  Your components have been moved inside src folder, I have removed the next imports from your components, and now are working as used to, but is good to recheck
Summary

❌ removed next from dependencies and also all the linting plugins of nextjs
♻  moved all the files into src directory
🔼 Support for React 18
🛠✅ Starting point now is called popup.js inside the popup directory
🔼 Updated all the dependencies
:white_check_mark: Resolves 👉 #14
✨ Created two scripts inside package.json, now run npm run start to watch building and build project, and npm run build for building the project.

📝 TODO

Right now the feature screenshot (files) , are outside src folder, I have to rewrite those files and include them. But now I have to rewrite it in typescript.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


PostgreSQL: include permissions in schema DDL

Hi,
Would it be possible to add grants in schema DDL for PostgreSQL?
Thanks in advance.

Currently DBeaver doesn't support schema DDL at all. We need to support it first (and it is not clear what should it be - DDL of all schema objects or just schema create statement + permissions (like in pgAdmin).

Please, vote for this feature. Grants like pgAdmin for PostgreSql in DBeaver is only what stop me to migrate on DBeaver. I hope in future all grants Postgres feature will be support in DBeaver.

I've added schema DDL similarly to pgAdmin (create + authorization).
Nested objects DDL (with permissions) isn't included.

Thanks!


Handle area and length calc exceptions (Update)

Update conditional to use explicit check

@hackermd I forgot to add explicit checks for the individual if statements while generating measurements. Can you review + merge?


Issue with Number of Image Classes Counted vs Number Actually Present

Hello
I am new to coding and I ran into some problems when using Image AI to help run an Image Detection Project on Google Colab.
I have 6 different image sets in folders labeled 00, 01, 02, 03, 04 and 05 which have been split into a train and test folders using split_folders: https://pypi.org/project/split-folders/
(The datasets are rather large so I couldn't do it manually)
So when I ran the code:
from imageai.Prediction.Custom import ModelTraining
model_trainer = ModelTraining()
model_trainer.setModelTypeAsResNet()
model_trainer.setDataDirectory("/content/Training/Items")
model_trainer.trainModel(num_objects=6, num_experiments=200, enhance_data=True, batch_size=32, show_network_summary=True)
It received an error:
ValueError: A target array with shape (32, 7) was passed for an output of shape (None, 6) while using as loss categorical_crossentropy. This loss expects targets to have the same shape as the output.
On closer look I found that it detected 7 image classes when I only had 6 image classes uploaded as seen below:
WARNING:tensorflow:period argument is deprecated. Please use save_freq to specify the frequency in number of samples seen.
Using Enhanced Data Generation
Found 11282 images belonging to 7 classes.
Found 4838 images belonging to 6 classes.
JSON Mapping for the model classes saved to  /content/Training/Items/json/model_class.json
Number of experiments (Epochs) :  200
Epoch 1/200
I am not sure where the error may have occurred so any help would be much appreciated!

There is a chance a file was automatically added to the train folder which shouldn't be there E.g system generated files

There is a chance a file was automatically added to the train folder which shouldn't be there E.g system generated files


Request: Optional packages for LTSC 2019

Since there is an option to integrate the Store and the Edge browser package, I would like to suggest that the following packages also be available:

Package
Source
Contents

Deduplication
Server 2019
Microsoft-Windows-Dedup-ChunkLibrary-Package Microsoft-Windows-Dedup-Package Microsoft-Windows-FileServer-ServerCore-Package Microsoft-Windows-VdsInterop-Package

Connect App
Windows 10 1809 (Non-LTSC)
PPIProjection ?

Ironically I was just extracting the Dedup Packages when I read this.  I will implement this once I make sure it does not negatively affect anything.
I will take a look at PPIProjection, too.

For what it's worth, I can confirm that it works perfectly and does not negatively affect anything on my systems. I am running a LTSC 2019 x64 modified using this script and manually integrated using DISM the package CABs for dedup and PPIProjection.
I extracted the packages from Server 2019 for dedup and from Enterprise non-LTSC for PPIProjection using the "Aunty Mel's Cheap And Nasty SxS File Expander" scripts from MDL forums. Everything works fine, signatures are correct and there is no WinSxS corruption or any other issues. There has only been one update on WU since I did it, but it updated flawlessly of course.

Can I ask why you closed this?
I have been running an image with these packages for more than 2 weeks already with absolutely no ill effects. Have you discovered some bad interaction with other components?


Format swagger json output with pretty print?

Is there an option to pretty print swagger output?
If not, I would propose to add support for querystring parameter pretty to turn this on
docs/v1/swagger?pretty=true
Happy to provide a PR for this.

Looks from https://github.com/domaindrivendev/Swashbuckle/issues/541 like this got implemented in PR https://github.com/domaindrivendev/Swashbuckle/pull/577
I'm using it with v5.5.3 like GlobalConfiguration.Configuration.EnableSwagger(c => {.PrettyPrint(); })

Fixed by #577


How do I read/write to a peripheral?

As far as I can tell, there are no examples for reading from and writing to a peripheral, and no documentation whatsoever for the writeValue method. I'm having trouble receiving data from my peripheral but I'm not sure if I'm even using the readValue method correctly.

Hi! There are some examples in the README.md from which you can see how to use the readValue method. The writeValue works very similarly to readValue. You can find more in depth examples in the AsyncBluetooth Cookbok.
Let me know if after reading these you're still running into issues.

Hi thanks for the response. Can you point me to where in the cookbook it covers reading and writing? Thanks for your help


Docmentation and Link on expandjs.com for mat-shell

Seems to be missing for mat-shell ... Are you phasing this element out?

Yep, forgot to update the readme. We are in the works of deprecating this element and we are replacing it with mat-drawer-panel.
mat-shell's original purpose is no longer needed (managing app settings, main layout and what not), as time passed we have split these behaviors in different elements. (like mat-settings and mat-drawer-panel)


CMake long build time for "Hello World"

Brief Issue Summary
I am trying to build a hello world program from here and CMake Tools take a very long time to build and run the test (2-3 minutes). I am new with C++ and CMake so I don't know if there is something very obvious that I'm missing. Creating a blank project with CMake Quick Start seems to work fine
Behavior:

CMake: Configure
CMake: Build

CMake Tools Log
[build] Starting build
[proc] Executing command: C:\cmake-3.15.2-win64-x64\bin\cmake.exe --build c:/Users/MinhDuc/Documents/Programming/exercism/cpp/hello-world/build --config Debug --target all -- -j 6
[build] Scanning dependencies of target hello-world
[build] [ 25%] Building CXX object CMakeFiles/hello-world.dir/hello_world.cpp.obj
[build] [ 50%] Linking CXX executable hello-world.exe
..then it takes a lot of time before moving on and completing the build
Developer Tools Log
Platform and Versions

Operating System: Windows x64
CMake Version: 3.15.2
VSCode Version: 1.36.1
CMake Tools Extension Version: 1.1.3
Compiler/Toolchain: gcc 7.3.0

Other Notes/Information

Hmm...  The project builds for me in close to 30 seconds with GCC.
[build] Starting build
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --build d:/git/cpp/exercises/hello-world/build --config Debug --target all -- -j 10
[build] [1/5  20% :: 1.297] Building CXX object CMakeFiles/hello-world.dir/hello_world.cpp.obj
[build] [2/5  40% :: 1.784] Building CXX object CMakeFiles/hello-world.dir/hello_world_test.cpp.obj
[build] [3/5  60% :: 18.230] Building CXX object CMakeFiles/hello-world.dir/test/tests-main.cpp.obj
[build] [4/5  80% :: 26.303] Linking CXX executable hello-world.exe
[build] [5/5 100% :: 26.730] cmd.exe /C "cd /D D:\git\cpp\exercises\hello-world\build && D:\git\cpp\exercises\hello-world\build\hello-world.exe"

It's much faster building with Visual Studio
[build] Starting build
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --build d:/git/cpp/exercises/hello-world/build --config Debug --target all -- -j 10
[build] [1/5  20% :: 1.094] Building CXX object CMakeFiles\hello-world.dir\hello_world.cpp.obj
[build] [2/5  40% :: 1.266] Building CXX object CMakeFiles\hello-world.dir\hello_world_test.cpp.obj
[build] [3/5  60% :: 4.230] Building CXX object CMakeFiles\hello-world.dir\test\tests-main.cpp.obj
[build] [4/5  80% :: 6.857] Linking CXX executable hello-world.exe
[build] LINK : hello-world.exe not found or not built by the last incremental link; performing full link
[build] [5/5 100% :: 6.999] cmd.exe /C "cd /D D:\git\cpp\exercises\hello-world\build && D:\git\cpp\exercises\hello-world\build\hello-world.exe"

This doesn't look like a bug in the extension to me.  Perhaps it's just a slow compiler?

@bobbrow Thanks for the response. I also try gcc 9.1.0 and have the same issue.
Yes I also realized after posting the issue that it does not seems like it's a specific issue with the extension. I try to build by running cmake from the terminal and it's still as slow
The test framework included in the build is pretty large so maybe that's the issue. If it does take 30 secs for you then I guess it's expected to take that long to build on my laptop (even though I have never had big issue with build time in other languages before). And Visual Studio probably optimizes the framework better?
I guess you can close the issue since it's not because of the extension. I just want some thoughts on this problem.

Thanks for the follow up.


renamed tenant to workspace

tests will be fixed later

(Jove Github Bot) added it to the current sprint.

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).


error in Sentiment Classification Projects.ipynb

Hi, I'm working on Sentiment_Classification_Projects.ipynb.
And I am confused with following code at the end of project 3:
mlp = SentimentNetwork(reviews[:-1000],labels[:-1000], learning_rate=0.1)
It will preprocess the given reviews to fill word2index  which is a dict, but when I execute next line:
mlp.test(reviews[-1000:],labels[-1000:])
It throws KeyError: 'reprogram'.
It seems that there are some words only existing in last 1000 reviews, and I think first line should be:
mlp = SentimentNetwork(reviews, labels, learning_rate=0.1)
I'm not sure about it, did I miss something?

That error happens because a test review contains a word that is not in training reviews so that word is not saved into word2index set. You can ignore those unknown words by not setting input vector whenever these words are encountered.

This should probably be fixed in the programming instructions or else it makes it really confusing to debug

For future reference something like this will work :
for word in review.split(' '):
    try:
        self.layer_0[0,self.word2index[word]] += 1
    except:
        pass

The following worked for me
indices = set()
for word in review.split(' '):
    if word in self.word2index.keys():
        indices.add(self.word2index[word])

We will make appropriate changes in the refresh. Thanks :)


[DO NOT MERGE] Simplify Condition Resolution

This reverts the major content of #5778, the Condition Resolution pass, and replaces it with a far simpler implementation of condition resolution along with canImport.  When combined with the optimizations in #6279 we get the best of both worlds with a performance win and a simpler implementation.
This is one of two ways towards a stable implementation of this feature.  The concern here is that even with optimizations Parse will have to hit the disk at least once per canImport statement which could potentially slow down a very hot path.  On the other hand, that cost cannot simply be deferred to Sema without significantly complicating DeclRef resolution, semantic analysis, and condition resolution.
⚠ Do Not Merge ⚠
@benlangmuir, @jrose-apple thoughts?
Resolves SR-3438, SR-3544, and SR-3546

@swift-ci please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 588a5d5c2a787a6a3854495392bab3457e017262
Test requested by - @CodaFi

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 588a5d5c2a787a6a3854495392bab3457e017262
Test requested by - @CodaFi

@swift-ci please smoke test.

@swift-ci please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 588a5d5c2a787a6a3854495392bab3457e017262
Test requested by - @CodaFi

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 588a5d5c2a787a6a3854495392bab3457e017262
Test requested by - @CodaFi

There.  @jrose-apple given the recent spate of issues, this is the version of this patch I'm most comfortable with.

I guess this is the best we can do. Would still like @benlangmuir's opinion as well—it's annoying that we've added a language feature that means we can't parse without search paths set up. Maybe we can recover this later by taking @DougGregor's idea—the contents of active conditional blocks not having to be splatted into their parent. But the first priority is to restore source compat.

I still strongly disagree with making parsing depend on (a) search paths, and (b) what is on disk in those search paths.  Not only is this a performance issue for syntax-only parsing, but also leaves us unable to reliably reproduce syntax-only SourceKit bugs without knowing the state of the filesystem or command-line argument, both of which are harder to find out and are less likely to be provided in bug reports than the source code.  We are stuck with that for semantic issues, but we shouldn't have to be for parsing.

Then we're stuck with parse-only producing a slightly different AST for now. Or we turn the feature back off for 3.1 and give ourselves a little more time to plan.

Trouble is we're also stuck with it for Parse issues.  Parse depends on being able to have total knowledge of DeclRefs, classes introduced that depend on Foundation, and local type declarations in addition to structure in general.  Condition Resolution demonstrated that at least the last two could be shifted to Sema, but without ASTScope or NameLookup taking the first Parse is going to wire up nonsense.  Modeling conditions dynamically would make querying compound parts of the AST have to go through condition resolution which is still subject to at least one of the linked SRs (the one about deinitializers being especially concerning).

The deinitializer one isn't so bad; the syntax-only parsing doesn't need to include members synthesized on demand. Sema can take care of that. We also don't actually care about resolving decl refs at all in a syntax-only parse.
But we're out of time. If @benlangmuir isn't happy with this approach, we should…well, we should still take this patch, so that we go back to what we had before, but we should disable the support for canImport until after Branch Day, and then think this through from the start again.

we should…well, we should still take this patch, so that we go back to what we had before, but we should disable the support for canImport until after Branch Day, and then think this through from the start again.

SGTM

OK, I'm going to remove support for the canImport feature, but add a requires annotation to the tests that are here for when we can work this out in the future.

Thanks, Robert. I feel really bad that you've put all this work in (multiple implementations, effectively!) and we can't ship it yet. Ben and I talked a little more about what we think the syntax-only parser wants out of this and I'll send you and swift-dev an email on it later.

Don't worry about it, I'm most concerned about keeping the feature and language stable around the change.

@swift-ci please smoke test.

Due to #6631
@swift-ci please smoke test and merge.

I can't find the test anywhere, so
@swift-ci please smoke test Linux platform.

⛵


File upload problem using gridfs-stream 0.5.0

My code for uploading files was working with gridform until updating Mongoose to v4.x.x.  When a file upload is attempted the following error occurs:
/Users/kenny/Dropbox/Dev/vqfolio/node_modules/gridform/node_modules/gridfs-stream/lib/writestream.js:44
this.id = new grid.mongo.BSONPure.ObjectID;
^
TypeError: Cannot read property 'ObjectID' of undefined
at Stream.GridWriteStream (/Users/kenny/Dropbox/Dev/vqfolio/node_modules/gridform/node_modules/gridfs-stream/lib/writestream.js:44:38)
It looks like similar issues have been seen for others using gridfs-stream: https://github.com/aheckmann/gridfs-stream/issues/74
Does the version of gridfs-stream and mongodb that gridform currently use need to be updated to fix this issue?   (current versions used by gridform 0.1.7 are gridfs-stream 0.5.0 and mongodb 1.4.3)

well its sad ... I get the same error ...

I hope this will can be fixed.
For the meantime we can apply the hotfix given in the reference above:
var MongoDB  = require('mongodb');
MongoDB.BSONPure = require('bson').BSONPure; //hotfix for gridform v0.1.7

I tried that fix in the past and found it didn't work for me... instead I took a copy of the files, file.js and gridform.js to my project and then included the following in my project.json file:
"events": "*", "formidable": "*", "gridfs-stream": "*",
then "require" the gridform file into your project.

Today I updated my project and I got the same issue! Bson is now at Version 1.0.4 an there seems to be no BSONPure anymore ...
My working new hotfix is
var MongoDB  = require('mongodb');
MongoDB.BSONPure = require('bson').BSON; //hotfix for gridform v0.1.7 with bson v1.0.4

Cheers!


variable-name doesn't support injecting constructors with angular

The following code makes the variable-name inspection fail, despite the fact it is a valid use case.  Any suggestions?
class MyController {
  constructor(ExternalConstructor: IExternalConstructor) {
    var z: IExternal = new ExternalConstructor();
  }
}

TSLint isn't aware of types, so it doesn't know that ExternalConstructor is being used a constructor. You could write a custom rule that rule that is aware of variables that are used as constructors and allows them to be uppercased.

constructor(@Inject(Http) http:Http)

[typedef] expected parameter: 'Inject' to have a typedef

@Delagen  this looks like an unrelated bug. Moving this to its own issue.

https://github.com/palantir/tslint/issues/479

Is there any way to specify an argument as a constructor? Maybe by using
function (<Function()=>void>UserConstructor)

Or even
function (UserConstructor: typeof User)

Or some similar syntax.  The type definition itself should allow tslint to know it isn't a normal variable

We should be able to check in the constructor function block if a variable is used as a constructor, i.e. invoked with new.
Also I think the latest TS language services provide some access to type information, but I haven't looked into it too much yet.

I think this request is best now addressed by the pascal-case option for the variable-name rule. The above solutions seem overly complicated compared to that option, so I'm closing this out.


Keep info about source root in sources.jar file

Necessary if we want to map sources.jar back to the local filesystem

Does this mean that the source jar artifact will contain information about the local paths on the machine that built it?

@odisseus
In fastpass we generate *-source.jar files for internal dependencies which were not explicitly selected to be imported. The path stored in source-root file contains path to these files, relative to the project root.


chore: add custom Initializable contract

What does this PR introduce?

Adding Initializable contract to Custom folder where this version has the booleans checks initialized and initializing public for making the implementation as generic as possible. One example would be to check these booleans by external contracts for security reasons when performing delegatecall.

Will be imported from ERC725.
Introduced here: https://github.com/ERC725Alliance/ERC725/pull/111


Fixed 'edit page' button URL

There was an old URL in edit page button - just fixed it :)

Thanks! Good catch. 😄


Fixing issue when color picker freezes because of locked clipboard

Summary of the Pull Request
Color picker can freeze when you pick a color because we are trying to access clipboard using Clipboard.SetText - but this method might as some other applications can hold clipboard.
Switching to use Clipboard.SetDataObject I couldn't reproduce this issue anymore
PR Checklist

[ ] Applies to #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
Validation Steps Performed
Unless you use some special tools for clipboard, it is hard to reproduce as it can happen randomly, but to get a stable reproduction, install Clipjump tool with this running this issue is reproducible all the time and it basically simulate the same issue that can happen randomly with some other application. With the fix, this issue does not appear anymore.

Please do not close this yet, I am adding another fixes


請問RAD-110NX1是否可連接

請問RAD-110NX1是否可連接
已更新至最新版
並且連上帳號密碼
但目前顯示不可用

2022-06-26 23:42:30 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration jcihitachi_tw which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-06-26 23:42:38 ERROR (MainThread) [custom_components.jcihitachi_tw] Assertion check error: Some of device_names are not available from the API.
2022-06-26 23:42:38 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration jcihitachi_tw: Integration failed to initialize.
以上訊息
感謝

device name須與App內自訂的名稱相同。


Cannot create Siesta.Error.InvalidJSONObject

I am attempting throw an error on all "text/*" responses.
I copied the 'TextResponseTransformer' func into my own class in attempt to reduce it to...
public func TextResponseTransformer(transformErrors: Bool = true) -> ResponseTransformer
{
    return ResponseContentTransformer(transformErrors: transformErrors)
        {
            (content: NSData, entity: Entity) throws -> String in
            let error = Siesta.Error(userMessage: "Response transformation failed.", cause: Error.Cause.InvalidJSONObject())
    }
}

Xcode7.2  is giving me this error 'Cannot invoke value of type 'Error.Cause.InvalidJSONObject.Type' with argument list '()'

The error you’re getting is because InvalidJSONObject doesn’t have a public initializer. (It’s only meant for client code to detect, not to create.) To do it using the approach you were using, you can make up your own error type:
enum MyAppErrors: ErrorType {
    case ResponsesMustBeJSON
}

…and then:
service.configure {
    $0.config.responseTransformers.add(
        ResponseContentTransformer() {
            (content: Any, entity: Entity) throws -> Any in
            guard content is NSJSONConvertible else {
                throw Siesta.Error(
                    userMessage: "Cannot parse response.",
                    cause: MyAppErrors.ResponsesMustBeJSON)
            }
            return content
        }
    )
}

But wait! Siesta already has that type checking built in. Siesta infers a content transformer’s expected input type from the parameter type of the closure you give, and flags it as an error if the incoming type in the pipeline is wrong. So believe it or not, all you have to do is pass a closure that declares a specific type for the content, but just passes it through unmodified:
    $0.config.responseTransformers.add(
        ResponseContentTransformer() {
            (content: NSJSONConvertible, entity: Entity) in   // Note: NSJSONConvertible instead of Any
            content
        }
    )

And because adding a custom ResponseContentTransformer is so common, Siesta provides a shortcut:
configureTransformer("/**") {
    (content: NSJSONConvertible, entity: Entity) in
    content
}

…or if you’re willing to lean hard on some Swift type inference magic:
configureTransformer("/**") {
    $0.content as NSJSONConvertible
}

That’s all you need!

Note that all the code above lets the default content-type-based transformers do their thing, then errors if something other than a JSON-like array or dict came out the other end.
If instead you want to attempt to parse everything as JSON, regardless of content type:
configure {
    $0.config.responseTransformers.add(JSONResponseTransformer())
}

By default, Siesta applies JSON parsing only to responses with a content type of */json or */*+json. This additional config makes Siesta apply JSON parsing to everything.
You want might want to parse even plain text responses as JSON. To do that, pass useDefaultTransformers: false to your Service initializer. (Why? Any transformers you add get appended to ones that are already there. Siesta includes transformers for text, images, and JSON by default. With those default ones in there, the server could return valid JSON with a content type of text/plan, the default text transformer would catch that and turn that into a string, and then the JSON transformer would say, “Hey! I’m supposed to get NSData” and give you an error.)

Mathew — I’m closing this one, but please feel free to keep following up (or reopen if it’s not resolve for you).

Thanks for the detailed response! That last part was exactly what I was looking for.
I assumed that the Error structs were useable since they are "public" structs. I guess I need to go back and read up on that.

The Error.Cause structs are not publicly constructable because synthesized struct initializers aren't public (I think).
The causes themselves are public (even though their initializers aren't) so that you can do error-specifc recovery. There’s an example of that in the docs.


Securing your CI/CD pipeline from code to deployment

The first steps for securing your team’s CI/CD pipeline include locking down configuration managers, systems that host repositories, and the build servers. The pipeline should be monitored from end to end with watertight access control across the entire toolchain. In this CDF workshop, a guided tour of how to scan-build script for vulnerabilities, and steps needed to be taken into consideration for regularly monitoring source code for vulnerabilities prior to the app deployed to production.
By following along with this workshop you will be familiarized with best practices for securing your CI/CD pipeline including the following:

Remove hard-coded secrets from CI/CD config files.
Have rigorous security parameters, such as one-time passwords, for secrets regarding more sensitive tools and systems.
Distribute secrets among CI/CD config files to reduce the potential attack target of each file.
Use password managers, and rotate passwords after each use.
Know who has access to what. Whether it’s role-based, time-based, or task-based, there should be a clear repository of access management. Another option to consider is segmenting secrets based on levels of access.
Ensure secrets are not inadvertently passed on during builds for pull requests via your CI/CD pipelines.
Deploy the practice of least privilege. Give access only to requisite secrets. Besides employee access, the least privilege also applies to applications, systems, or connected devices that require privileges or permissions to perform tasks. You should regularly audit levels of access to maintain the level of least privilege

Steve's final abstract:
The first step in protecting your software supply chain should include adding security actions to your CI/CD pipeline, from scanning your repos to locking down your builds. The pipeline should be evolved to include available open-source tools that can shift your DevOps pipeline to a DevSecOps pipeline.  In this CDF workshop, Steve Taylor will cover 5 phases of the DevOps process that must be reinforced to improve your supply chain security. In this workshop, you will learn about new open-source security tooling that you can immediately add to your pipeline to implement good security practices, including: Learn what phases of the pipeline need security actions.

Learn how to implement the OpenSSF Security Scorecard.
Learn where signing and SBOM generation should be added.
Understand SLSA and how the Prysia decentralized package network can help you achieve SLSA compliance.
Learn about Ortelius as an evidence store to consolidate security logs and build an organizational-level security profile.
Learn how CDEvents will simplify adding new tooling to your pipeline to maintain a secure software supply chain.

Securing your organization from cyber hacks is not just the job of production teams. It is time for development teams to play their part. Building security into your CD Pipelines is the first step. This workshop will help you get there.

This is scheduled for June 22nd.