HADOOP-17665

Handle properly missing SSL key/trust store configurations when monitoring file changes in HttpServer2.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 45s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 30s

trunk passed

+1 :green_heart:
compile
20m 50s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
18m  8s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
1m  5s

trunk passed

+1 :green_heart:
mvnsite
1m 33s

trunk passed

+1 :green_heart:
javadoc
1m  3s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 39s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
2m 20s

trunk passed

+1 :green_heart:
shadedclient
15m 32s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
15m 56s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 53s

the patch passed

+1 :green_heart:
compile
20m 11s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
20m 11s

the patch passed

+1 :green_heart:
compile
18m 12s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
18m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
1m  5s
/results-checkstyle-hadoop-common-project_hadoop-common.txt
hadoop-common-project/hadoop-common: The patch generated 11 new + 67 unchanged - 4 fixed = 78 total (was 71)

+1 :green_heart:
mvnsite
1m 32s

the patch passed

+1 :green_heart:
javadoc
1m  2s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 39s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
2m 31s

the patch passed

+1 :green_heart:
shadedclient
15m 47s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
17m 21s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
0m 56s

The patch does not generate ASF License warnings.

179m 21s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2978/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2978

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 6a2e1ba14607 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 8487cb5c4f21e882da5fd234b7a3edf865089749

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2978/1/testReport/

Max. process+thread count
1389 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2978/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Support GHC 8.8

Pardon an automatically-posted pull request; this is a part of Operation Vanguard. Note that this does not guarantee that tests and benchmarks are buildable.

This change is

that was a wrong branch

Closing in favor of #10.


README link broken

A few I tried which is not working. Did not go through all of them

Most links were broken. Check and update thoroughly

update java install link to here. https://adoptopenjdk.net/
ask me why if you are not clear

#90
fixed and updated broken links, and reformatted projects showcase into folder-based.

fixed in https://github.com/CertifaiAI/TrainingLabs/pull/90


Websocket Upgrade not possible with Firefox

Firefox sends the following Request on a Websocket upgrade [1]. The Rfc6455Endpoint implementation checks  if the Connection header to be exactly 'Upgrade'.
Firefox instead send Connection: keep-alive, Upgrade. This will lead to an HTTP error response 426:  Bad Request: "Connection: Upgrade" header required'
The HTTP Connection header spec allows more then one token for the header.
[1]
GET /ws HTTP/1.1
Host: localhost:1337
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Sec-WebSocket-Version: 13
Origin: http://localhost:1337
Sec-WebSocket-Extensions: permessage-deflate
Sec-WebSocket-Key: somerandom==
Connection: keep-alive, Upgrade
Upgrade: websocket

Thanks for the report!


Mercs now start with weapons in their inventory

The planets.json config file has the mercenary item listings, and neither one was starting with weapons in their inventory. I added weapons, and everything else worked ok. I also added a placeholder block for code for mercenaries equipping items dropped on them. I don't think we should have them do that, though- not having to worry about equipping mercenaries is easier for the player.

Hooray Jenkins reported success with all tests good!

Ah yes- I should probably add that this resolves issue #109, and also that I really should have payed attention when reading #101, where @Cervator clearly stated that

NPC ships were no longer equipping stuff

(emphasis mine).
Anyway, the next time a experienced member of the project mentions something, I'll try to pay attention XD. At least I only wasted four hours... T_T.

Hehe sorry about the hassle! But thank you so much for figuring this one out :-)

Do you want to have me work on making mercenaries equip items dropped on them, or do you just want to quickly remove the placeholder and pull?

Please leave the placeholder in place, it'll be good to have it handy. I'll merge this soon'ish today, catching up after having mostly missed a day or two :-)
Equipping items dropped on the mercenaries can be done as a separate issue any time later. Maybe we should add a few more options too, I never actually noticed there were only two of them enabled.

Tested on both PC and Android, looking good! Thanks again :-)

Oh, a Git protip as well: Whenever upstream (MovingBlocks) changes favor updating in these two ways:

If you have no changes locally use git pull to simply add the new commits on top of your local workspace - no merge commit
If you do have changes locally use git rebase to sort of sneaky merge in the upstream changes to actually appear to have happened before your changes (it moves your commits till the "head" of the branch) - no merge commit

Both of those approaches keep the history graph cleaner as it avoids merges in favor of just moving commits around cleanly :-)
So right now with this merged assuming you have nothing outstanding locally you should be able to simply pull latest from upstream (git pull upstream develop would be one way that would work via command line, if upstream is the name you gave a git remote link to MovingBlocks)

Thanks for the tip. I'm currently even with MovingBlocks, but next time I get behind I'll try this, I guess.


CI: test with JRuby <IP_ADDRESS>

use v<IP_ADDRESS> for the JRuby tests

Cool! Nice to see the additional cleanup around Install OS packages. Could you add a change log entry?

Sure thing. CHANGELOG updated with 54130a6
Yeah, for the cleanup we are no longer insisting that the Ubuntu 22 container use Java 8 (out of the 3 Javas it has installed on the machine) and allow it to use whatever the default is. It's nice to be able to so for sure.


Data Factory resource provider is needed for "Event Trigger" in Synapse Analytics

When creating a storage event trigger in Azure Synapse Analytics, the creation fails unless you have Data Factory registered as a resource provider. This information is not written here only that you need EventGrid registered.
If you do not have Data Factory registered you simply get a bad error which says that the creation of an "Event Subscription" failed.
My request is simply that you describe the need for registering Data Factory as a provider, when using Synapse Analytics, so that people in the future don't waste their time like I did.

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

ID: b7ee5166-ff40-f4ed-ce07-5312f935e388
Version Independent ID: 68f01b43-e3b2-003b-24cb-446a08aa0c1b
Content: Create event-based triggers - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/how-to-create-event-trigger.md
Service: data-factory
Sub-service: orchestration
GitHub Login: @chez-charlie
Microsoft Alias: chez

@Nyggi Thanks for reaching out. We will investigate and update the thread accordingly.

@chez-charlie Requesting to validate customer feedback and update the document accordingly as appropriate. Thankyou

Thanks a lot for the feedback @Nyggi . Adding now

#please-close


11-tutorial

I have no idea why the old and new outputs of cell 11 are mirror images 🤔 . If you find nothing fishy, than please merge this.

I found out this is because we are using sel on the edge of a cell and we have changed the exclusive/inclusive nature of the cells.


Support noEncap mode on Windows

Describe the problem/challenge you have
Running antrea-agent with option TrafficEncapMode: NoEncap will fail with the error:
F1208 05:08:06.203391   11180 main.go:55] Failed to validate: unsupported features on Windows: {TrafficEncapMode: NoEncap}

Describe the solution you'd like
NoEncap mode also works on windows
Anything else you would like to add?
This gives the feature parity to Linux nodes. It will be useful for customers that don't like the performance penalty of an additional tunnel, as well as the MTU implication.

Thanks Yongkun for your report. NoEncap mode support on Windows is indeed high priority.
We're working on that but is blocked by an OVS issue: https://github.com/openvswitch/ovs-issues/issues/197
Will address the issue soon with OVS team.

Thanks @ruicao93  Do you have a rough estimation when it can be done?


Failed run log request.

@cisaacstern When you have a moment can you post the failed recipe run logs from https://pangeo-forge.org/dashboard/recipe-run/1437?feedstock_id=94

Sure thing! This is an unexpected one...
severity: "ERROR"
textPayload: "Workflow failed. Causes: S13:Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_000/ReshufflePerKey/GroupByKey/Read+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_000/ReshufflePerKey/GroupByKey/GroupByWindow+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_000/ReshufflePerKey/FlatMap(restore_timestamps)+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_000/RemoveRandomKeys+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/finalize+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_001/AddRandomKeys+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_001/ReshufflePerKey/Map(reify_timestamps)+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_001/ReshufflePerKey/GroupByKey/Reify+Start|scan_file|Reshuffle_000|finalize|Reshuffle_001/Reshuffle_001/ReshufflePerKey/GroupByKey/Write failed., The job failed because a work item has failed 4 times. Look in previous log entries for the cause of each one of the 4 failures. If the logs only contain generic timeout errors related to accessing external resources, such as MongoDB, verify that the worker service account has permission to access the resource's subnetwork. For more information, see https://cloud.google.com/dataflow/docs/guides/common-errors. The work item was attempted on these workers:

  a6170692e70616e67656f2d66-12161716-e74o-harness-x9x4
      Root cause: The worker lost contact with the service.,

  a6170692e70616e67656f2d66-12161716-e74o-harness-x9x4
      Root cause: The worker lost contact with the service.,

  a6170692e70616e67656f2d66-12161716-e74o-harness-kd6j
      Root cause: The worker lost contact with the service.,

  a6170692e70616e67656f2d66-12161716-e74o-harness-kd6j
      Root cause: The worker lost contact with the service."

I wonder if it was just some one-off Dataflow internal issue and re-triggering the run would resolve it? These type of obscure start-up errors can come up when we change our sdk_container_image but we haven't done that recently. 🤔

xref https://github.com/pangeo-forge/pangeo-forge-orchestrator/issues/203 (in which the same recipe is being discussed)

@sharkinsspatial I suggest a blank i.e. git commit --allow-empty PR to this repo, so we can get a fresh production run triggered, and see if this was indeed just a random Dataflow error.


Three 0.95.0 support

:wave: Hello @zadvorsky, Thanks for your work on this library!  I'm just getting my feet wet with it and noticed it didn't run under THREE 0.95.0 (the latest as of this PR).  It appears that a shader was renamed in https://github.com/mrdoob/three.js/pull/14406.  This PR makes updates to the materials to support the rename.   I've verified that all the demos continue to work, though I'm unaware of any unintended side effects of this change.

The required change is https://github.com/zadvorsky/three.bas/commit/2a9917b20cb1bec0c7e29b9b0e80695c4087a20f if you want to simply cherry pick that commit.
The other two commits recompile the distributions since they are check in and adds the live-server package so npm run serve runs successfully.

Thanks!

Merged, thanks!


Non-European characters slug duplication

Integrity Error
duplicate key value violates unique constraint “cases_tagulous_case_tags_slug_f02759e7_uniq” DETAIL: Key (slug)=(___10) already exists.

django-tagulous Documentation

A unique SlugField, generated automatically from the name when first saved.

If unidecode is installed, Tagulous will use it to make a best-guess conversion of non-ascii characters to something similar in ascii which will be safe for a slug field. If unidecode is not installed, Tagulous will strip diacritical marks from extended ascii characters, but then just convert any remaining non-ascii characters to underscores. In most cases either option will be adequate for European languages, but if your tags frequently contain non-European characters you'll have less slug clashes if you use unidecode.

Update existing tag manually:
for tag in Case .objects.first().tags.tag_model.objects.all():
    tag.slug = None
    tag.save()


small decimal (14.0) is read as timestamp with an varsignedint larger than 32 bits

/home/travis/build/amzn/ion-js/ion-tests/iontestdata/good/testfile21.ion (72ms)
Error: Values larger than 32 bits need to be marshalled using LongInt

good/testfile21.ion is no longer on the skiplist; closing


Support of stereo signals during separation (throughout the process)

Since I came across this model rather accidentally and have now roughly flown over it, I noticed that this can apparently only separate mono signals so far. From a technical point of view this is of course the much bigger challenge than a stereo signal, because all the room information is lost.
So I asked myself if it is possible to use this additional information. Would it be very difficult to extend the existing model to do so?
I find the approaches very exciting, because I have already got very good results from other models like Demucs, which also works in the waveform domain.
But this model seems to be much lighter than Demucs, for example, and the results seem to be very comparable.
The model would be much better if it could handle stereo signals. Then it would hardly be inferior to Demucs.

Hi, thank you for your interest! The network could be extended to directly handle a stereo signal by changing the number of input/output channels from 1 to 2 in the encoder and decoder, respectively.

Hi, like FSharpCSharp I have been testing Demucs and the ConvTasnet model implemented by Demucs group for a while and I´m very impressed by the results of meta-tasnet and how fast the model compute the separated stems.
How could I change the input/output channels in the code to execute stereo separation in the google colab notebook?. Thank you.

Hi, thanks! Please take a look at evaluate.py where we separate both channels independently. Let me know if that solves your problem :relaxed:

Thank You!!. It took me "a little" to understand the code but now I can separate stereo audio in the colab notebook.

Thank You!!. It took me "a little" to understand the code but now I can separate stereo audio in the colab notebook.

Thank You!!. It took me "a little" to understand the code but now I can separate stereo audio in the colab notebook.

Im curious, how did you make it work. I am trying to make it work with 44.1kHz stereo input and correct output, so 4 stems of 44.1kHz and all stereo.

Hi, the code work with 1.5 minutes stereo songs and the output are four stereo stems with rate 32000 HZ.
I have appended this lines to the resample definition from the evaluate.py:
**mix_left = [s[0:1, :, :] for s in mix]
mix_right = [s[1:2, :, :] for s in mix]**
del mix

And then I duplicate the code for left and right channel:
network.eval()
with torch.no_grad():
    separationL = network.inference(mix_left, n_chunks=2)[-1]  # call the network to obtain the separated audio with shape [1, 4, 1, T']
    separationR = network.inference(mix_right, n_chunks=2)[-1]  # call the network to obtain the separated audio with shape [1, 4, 1, T']

# normalize the amplitudes by computing the least squares
# -> we try to scale the separated stems so that their sum is equal to the input mix
aL = separationL[0,:,0,:].cpu().numpy().T  # separated stems
aR = separationR[0,:,0,:].cpu().numpy().T  # separated stems

bL = mix_left[-1][0,0,:].cpu().numpy()  # input mix
bR = mix_right[-1][0,0,:].cpu().numpy()  # input mix

solL = np.linalg.lstsq(aL, bL, rcond=None)[0]  # scaling coefficients that minimize the MSE
solR = np.linalg.lstsq(aR, bR, rcond=None)[0]  # scaling coefficients that minimize the MSE

separationL = aL * solL  # scale the separated stems
separationR = aR * solR  # scale the separated stems

Finally, concate left_stems and right_Stems:
separation = np.concatenate((separationL, separationR), axis=1)
print(separation.shape)

estimates = {
    'drums': separation[:,[0,4]],
    'bass': separation[:,[1,5]],
    'other': separation[:, [2,6]],
    'vocals': separation[:,[3,7]],
}

Hey, I'm happy that it finally works for you :) Here's my gist for separating a stereo signal and resampling it back to the original sampling rate: https://gist.github.com/davda54/aa555c011866392c32c4906f8a709682

I have two questions after separated a bunch of tracks in mono and also in stereo:

The output stems are too loud. How can i deactivate the normalization of the audio stems?. This normalization causes very hard clipping in the signal. It occurs in stereo and mono separation and in all the tracks that I have tried.

As  FSharpCSharp  says in other issue I have also noticed the signal cut off about 12 dB above 10KHz.

Are these problems caused by any parameter in the estimated code?.

This may be caused by the IPython.display player which normalizes every audio for some reason. Have you tried downloading the separated signal and playing in a proper audio player?
You're right, we're unsure what is the cause of this phenomenon. It seems to be an internal property of the neural network. Please let me know if you find out more about this.

I actually delete  IPython and the yoyutube lines from the code because I don´t use it. This is the code that call separate and write the output file:
audio, rate = soundfile.read(filename)
print("separating... ", end='')
estimates = separate_sample(audio, rate)
print("done")
print("downloading audio files to the client side...")

for instrument in ['vocals', 'drums', 'bass', 'other']:
separation = estimates[instrument]
print(separation.shape)
soundfile.write('WM1_5' + instrument + '.wav', separation, 32000)
After that I dowloaded the file and I opened it with Reaper. I put the original wav file (White Man Worlds from Jason Isbell) with the Demucs vocal track and the MultiTasnet Vocal track.

I see, that doesn't look good – could share the exact code and .wav file that you use? Feel free to send it to my email address<EMAIL_ADDRESS>

I run the notebook again with your separation stereo code instead of my "frankestein" version and it works smoothly. This is your code:
    network.eval()
    with torch.no_grad():
        separation_left = network.inference(mix_left, n_chunks=8)[-1].cpu().squeeze_(2)  # shape: (5, T)
        separation_right = network.inference(mix_right, n_chunks=8)[-1].cpu().squeeze_(2)  # shape: (5, T)

        separation = torch.cat([separation_left, separation_right], 0).numpy()

    estimates = {
        'drums': librosa.core.resample(separation[:, 0, :], 32000, rate, res_type='kaiser_best', fix=True)[:, :audio.shape[1]].T,
        'bass': librosa.core.resample(separation[:, 1, :], 32000, rate, res_type='kaiser_best', fix=True)[:, :audio.shape[1]].T,
        'other': librosa.core.resample(separation[:, 2, :], 32000, rate, res_type='kaiser_best', fix=True)[:, :audio.shape[1]].T,
        'vocals': librosa.core.resample(separation[:, 3, :], 32000, rate, res_type='kaiser_best', fix=True)[:, :audio.shape[1]].T,
    }

    a_l = np.array([estimates['drums'][:, 0], estimates['bass'][:, 0], estimates['other'][:, 0], estimates['vocals'][:, 0]]).T
    a_r = np.array([estimates['drums'][:, 1], estimates['bass'][:, 1], estimates['other'][:, 1], estimates['vocals'][:, 1]]).T

    b_l = audio[0, :]
    b_r = audio[1, :]

and I was using the code:
network.eval()
with torch.no_grad():
    separationL = network.inference(mix_left, n_chunks=8)[-1]
    # call the network to obtain the separated audio with shape [1, 4, 1, T']
    separationR = network.inference(mix_right, n_chunks=8)[-1]
    #Ojo que el parámetro de chunks en el códido original es chunks=2
# normalize the amplitudes by computing the least squares
# -> we try to scale the separated stems so that their sum is equal to the input mix
aL = separationL[0,:,0,:].cpu().numpy().T  # separated stems
aR = separationR[0,:,0,:].cpu().numpy().T  # separated stems
bL = mix_left[-1][0,0,:].cpu().numpy()  # input mix
bR = mix_right[-1][0,0,:].cpu().numpy()  # input mix

Thank you for answer so quick!

Haha, I'm glad it helped :)


Duplicate barcode

Hello,
i apologize for my English... i am using your plugin with Data Wedge profile  (set according to your blog post).
Everything works as it is, scans the codes, and I come across a case where sometimes (possibly between 2 different codes) when I do it so quickly (<2s) the received intent (barcode) is the same as the previous one.
I do not know what I am doing wrong? is it possible that there would be some cache, for example, in setting a date wedge profile or some timeout?
thank you for any help :)
Version: com-darryncampbell-cordova-plugin-intent 1.1.5
Ionic 3.7.1
HW: Zebra TC-25 (Android 6.0)
Code (typescript):
private registerBroadcast() {
		console.log('register broadcast intent');

		window['plugins'].intentShim.registerBroadcastReceiver({
				filterActions: ['cz.myapp.scanner', 'com.symbol.datawedge.api.RESULT_ACTION'],
				filterCategories: ['com.android.intent.category.DEFAULT']
			},
			(intent) => {
				console.log('received intent: ', intent.extras);

				if (intent.extras['com.symbol.datawedge.data_string']) {
					this.events.publish('reader:barcode', intent.extras['com.symbol.datawedge.data_string']);
				}
			}
		);
	}

	private unregisterBroadcast() {
		console.log('unregister broadcast intent');

		window['plugins'].intentShim.unregisterBroadcastReceiver();
	}

Hi, is this reproducible with the Zebra Ionic Demo at https://github.com/Zebra/ZebraIonicDemo ?

Code looks the same and dependecy (your plugin) too... I can try to build this project and test it

Please do and if you can reproduce with the demo app then I'll take a look.  I definitely have not seen this before but it could be related to versions / device / Operating system or any number of variables.

I've been able to reproduce the bug even on the application ZebraIonicDemo...
just get 20 different barcode (code128) and load them fast ... 2x to read the new code read the previous (see screenshots)
DataWedge App version 6.6.49
Android 7.1.2
Zebra TC25

I don't see the same with a TC25 here, it is running DataWedge 6.7 but that should not make a difference.  Is it possible you are scanning the same barcode twice inadvertently?  Perhaps enable Picklist mode in your DataWedge Profile (Barcode Input --> Reader Params --> Picklist --> Software), then only the barcode below the reticle will be decoded.

I really did not scan the same code twice... I try the settings in the datawedge profile.

even after setting up a Picklist - Software, same behavior ...
I've also tried this plugin (https://github.com/BlueFletch/motorola-datawedge-cordova-plugin) and the behavior is the same.
It must be somewhere in setting up that DataWedge profile. DWDemo (DataWedge demo app) uses the Intent Output -> Intent delivery -> Send via startActivity and duplicates are not used.
I use Broadcast intent just like ZebraIonicDemo, otherwise the setting looks exactly the same.

You should be able to use the getIntent() method of this plugin to retrieve the Intent sent by StartActivity, if that is an option for you

and how to use it? Every time I run an app, getIntent () performs only once ... nothing happens after every code is scaned.

Or onIntent().  There are examples of use in the read me: https://github.com/darryncampbell/darryncampbell-cordova-plugin-intent

Thanks, maybe I'm good with some solution.

Another tip might be to always use NgZone.run() in the receiving event. Like
constructior(public zone: NgZone) {
}

private registerBroadcast() {
    console.log('register broadcast intent');
    window['plugins'].intentShim.registerBroadcastReceiver({
        filterActions: ['cz.myapp.scanner', 'com.symbol.datawedge.api.RESULT_ACTION'],
	filterCategories: ['com.android.intent.category.DEFAULT']
    },
    (intent) => {
        this.zone.run(() => {
	    console.log('received intent: ', intent.extras);
            if (intent.extras['com.symbol.datawedge.data_string']) {
                this.events.publish('reader:barcode', intent.extras['com.symbol.datawedge.data_string']);
            }
        });
    }
);

Can this be closed?


Add exception handling guidelines

Which exceptions should be handled, which exceptions should be propagated to the user? When should we use custom exceptions, when should we use built-in exceptions? etc...
The Google Python style-guide has some reasonable suggestions about exceptions.

See related #967

Don't you think this issues needs a label decision record ?
When you create guidelines for something used in multiple places like exceptions maybe it's good idea to record it.

I'm not sure if it's deserving of a separate label, but we should make it clear that the result of every discussion should be documented. Whether a decision record, contributor guideline or some other form.

Good point @MVrachev. But how would the decision record about exception guidelines differ from the exception guideline document itself? I think the reason why I didn't use the label here is because I envisioned the guideline document itself a good central place that justifies the way exceptions are handled in multiple places, and probably this issue page as easily identifiable place to look up any additional discussion, that are not part of the guideline document.
That said, I do see how it's not always clear if something classifies for a decision record or not.  A general rule of thumb to classify something could be:
"The decision is worth being tracked and there is no easily identifiable place (e.g. PR or issue page, commit message, code comment, other documentation or guideline, etc...) for it to be tracked at."
What do you think? At any rate, we should make this clear as part of #1141.


Upgrading/ Reverting vpc-cni addon [v1.12.6-eksbuild.1 <-> v1.12.5-eksbuild.2] on EKS v1.24 breaks underlying network with Cilium as CNI

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
While upgrading vpc-cni addon [v1.12.6-eksbuild.1 <-> v1.12.5-eksbuild.2] on EKS cluster v1.24 breaks networking for all pods in terminating state and new pods that come up in pending state.
This is the error message we received after upgrading the vpc-cni plugin on any workload that was trying to be scheduled :
Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "6e7f82eed3c3c40752c08e8fa672029640c64eb5b966f22597154d9cd1ac6bcb": plugin type="egress-v4-cni" name="egress-v4-cni" failed (add): netplugin failed: "panic: runtime error: invalid memory address or nil pointer dereference\n[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x5598b4660a2c]\n\ngoroutine 1 [running, locked to thread]:\nmain.cmdAdd(0xc000180850)\n\t/go/src/github.com/aws/amazon-vpc-cni-k8s/cmd/egress-v4-cni-plugin/cni.go:256 +0xbec\ngithub.com/containernetworking/cni/pkg/skel.(*dispatcher).checkVersionAndCall(0xc0000d5ed8, 0xc000180850, {0x5598b4798d18, 0xc0000b8e40}, 0x5598b4796be8)\n\t/go/pkg/mod/github.com/containernetworking/cni@v0.8.1/pkg/skel/skel.go:166 +0x20a\ngithub.com/containernetworking/cni/pkg/skel.(*dispatcher).pluginMain(0xc0000d5ed8, 0x5598b48e5000?, 0xc0000d5ec0?, 0x5598b44be809?, {0x5598b4798d18, 0xc0000b8e40}, {0xc000018108, 0x15})\n\t/go/pkg/mod/github.com/containernetworking/cni@v0.8.1/pkg/skel/skel.go:218 +0x245\ngithub.com/containernetworking/cni/pkg/skel.PluginMainWithError(...)\n\t/go/pkg/mod/github.com/containernetworking/cni@v0.8.1/pkg/skel/skel.go:275\ngithub.com/containernetworking/cni/pkg/skel.PluginMain(0x5598b4667db5?, 0x17?, 0xc0000d5f60?, {0x5598b4798d18?, 0xc0000b8e40?}, {0xc000018108?, 0x0?})\n\t/go/pkg/mod/github.com/containernetworking/cni@v0.8.1/pkg/skel/skel.go:290 +0xd1\nmain.main()\n\t/go/src/github.com/aws/amazon-vpc-cni-k8s/cmd/egress-v4-cni-plugin/cni.go:250 +0x8e\n"

This is the error message we received after upgrading the vpc-cni plugin on any pods that were in terminating phase :
error killing pod: failed to "KillPodSandbox" for "a10c54e7-3cad-41d0-aaa4-440db2df3ceb" with KillPodSandboxError: "rpc error: code = Unknown desc = failed to destroy network for sandbox \"3e96ad4dcb4183d542d0bbaecc5bef1d31cca718b95d3f796f6de3805bad5fd5\": plugin type=\"egress-v4-cni\" name=\"egress-v4-cni\" failed (delete): failed to parse config: json: cannot unmarshal string into Go struct field NetConf.mtu of type int"

Note:  Know workaround, deleting cilium-daemonset pod on the node solves this issue
Cilium Version
v1.13.2
Kernel Version
Bottlerocket OS 1.13.3 (aws-k8s-1.24)
5.15.102
Kubernetes Version
v1.24.10-eks-48e63af
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

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

This appears to be the line that is failing: https://github.com/aws/amazon-vpc-cni-k8s/blob/release-1.12/cmd/egress-v4-cni-plugin/cni.go#L256
It means the AWS plugin is failing to parse the cni configuration. Can you please include the file(s) in /etc/cni/net.d/?

It is unlikely that we caused this, since libcni also requires the CNI configuration to be valid json, but anything is possible.

I've raised a similar issue here on the VPC CNI plugin; since the stack trace seems to be coming from there: https://github.com/aws/amazon-vpc-cni-k8s/issues/2364

Looks like there was a breaking change in the cni configuration for the MTU Parameters. Looks like a  Cilium fix would be required.
See: https://github.com/aws/amazon-vpc-cni-k8s/issues/2364#issuecomment-1529926817

Gotcha. 1.14 already fixes this properly, I’ll look in to a fix for 1.13 and earlier.

@squeed FYI
https://github.com/aws/amazon-vpc-cni-k8s/issues/2364#issuecomment-1530117595

Note that https://github.com/aws/amazon-vpc-cni-k8s/pull/2353 refactored the conflist as well, and the name change (egress-v4-plugin -> egress-plugin) could cause a problem for Cilium in the next release, v1.13.x

Cilium doesn’t hard-code the AWS portion of the config; it injects itself in to whatever is already there.
However, it doesn’t re-trigger generation if the “source” config changes. (1.14 already fixed this, but it’s not so easy to backport). I’ll see how we can handle this gracefully in 1.13 and before.

The problem is that in v1.13 and prior, we only generate the CNI configuration file on agent startup. If the "source" configuration file (i.e. 10-aws-cni.conflist) changes after startup, we miss that.
The "easiest" fix is to exit the agent when the file changes, but maybe there's a better way. Hmm.

@squeed just to be sure I get your last comment the proper way: If one faces the issue reported here triggering a rollout of the cilium daemonset shall allow it to regenerate the conf properly and fix the issue ?

@rrey that's correct. I'm still thinking about a proper fix.

For people that may land here, I confirm that the cilium daemonset rollout fixes the issue while the real fix is coming.

The problem is that in v1.13 and prior, we only generate the CNI configuration file on agent startup. If the "source" configuration file (i.e. 10-aws-cni.conflist) changes after startup, we miss that.
The "easiest" fix is to exit the agent when the file changes, but maybe there's a better way. Hmm.

I just hit this in my clusters. Only doubt I have is what could trigger a "source" configuration file change?

If you've read the comments thus far and still haven't been able to fix the issue, in my case, removing the /etc/cni/net.d/05-cilium.conflist from the worker nodes of AWS EKS manually fixed the issue. The cilium uninstall didn't do it automatically!


feat: support for lambda event destinations (#1321)

Issue #, if available:
Cherry picking event destinations
Description of changes:
Description of how you validated changes:
Checklist:

[ ] Write/update tests
[ ] make pr passes
[ ] Update documentation
[ ] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1345 into release/v1.20.0 will decrease coverage by <.01%.
The diff coverage is 92.46%.

@@                 Coverage Diff                 @@
##           release/v1.20.0    #1345      +/-   ##
===================================================
- Coverage            94.41%   94.41%   -0.01%
===================================================
  Files                   78       78
  Lines                 4386     4527     +141
  Branches               870      902      +32
===================================================
+ Hits                  4141     4274     +133
- Misses                 116      119       +3
- Partials               129      134       +5

Impacted Files
Coverage Δ

samtranslator/plugins/globals/globals.py
99.05% <ø> (ø)
:arrow_up:

samtranslator/model/intrinsics.py
95.83% <100%> (+0.52%)
:arrow_up:

samtranslator/model/iam.py
94.28% <100%> (+5.39%)
:arrow_up:

samtranslator/translator/translator.py
99.03% <100%> (ø)
:arrow_up:

samtranslator/model/lambda_.py
93.1% <100%> (+4.64%)
:arrow_up:

samtranslator/model/sns.py
100% <100%> (ø)
:arrow_up:

samtranslator/model/sam_resources.py
93.95% <90.83%> (-1.15%)
:arrow_down:

samtranslator/model/events.py
100% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


model = models.Sequential(); NameError: name 'models' is not defined

base_model = InceptionResNetV2(
include_top=False,
weights='imagenet',
input_shape=INPUT_SHAPE)
Freeze all the layers
for layer in base_model.layers[:]:
layer.trainable = False
Check the trainable status of the individual layers
#for layer in base_model.layers:
print(layer, layer.trainable)
Create the model
model = models.Sequential()
Add the InceptionResNetV2 convolutional base model
model.add(base_model)

It was working perfectly fine yesterday, after committing the code on Kaggle I tried running it, then it started giving me this error.

from keras import models

It was working perfectly fine yesterday, after committing the code on Kaggle I tried running it, then it started giving me this error.
Please let me know if you want me to submit the file

Post gist of your code.

It worked fine after import models and import layers
Thanks


Implement the MustVerifyEmail

Allow using the /verify-email route automatically if the user is not verified.
If the MustVerifyEmail is implemented in the Laravel Project, then, setting the EMAIL_VERIFY=true in env will check the for the verified middleware and redirect to the /verify-email route
Also, if a user tried to manually route to the /verify-email route when the EMAIL_VERIFY=false, it will redirect to the provided redirectIfAuthenticated route.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell
Thank you for your constructive response.
This starter kit implements all the breeze-next authentication, except the must-verify email condition, if this condidtion set on the Laravel App, this starter kit throws Error 403 (after login or register), instead of redirecting the user to the email verification page.
I believe this is a bug and that is what I have fixed.
Thank you.

@taylorotwell
Thank you for your constructive response.
This starter kit implements all the breeze-next authentication, except the must-verify email condition, if this condidtion set on the Laravel App, this starter kit throws Error 403 (after login or register), instead of redirecting the user to the email verification page.
I believe this is a bug and that is what I have fixed.
Thank you.


Add 'FocusedObject' property to GestureManager

I suggest adding the following property to GestureManager.cs:
public GameObject FocusedObject { get { return focusedObject; } }

This allows you to get the currently focused object. In fact the Origami academy tutorial even added this.

Closing as this is no added with associated PR


Slack invitation web site down

Per the wiki and the Apache Incubator email, access to the ONI/Apache Spot (incubating) Slack channel is available by requesting via http://open-network-insight.org:3000/ but unfortunately this web site is down.

back up and running,
Thank you.


Add Info on Opening Console ; Address Messy "Shortcuts" Table

This is the first result for most searches on how to launch the console in Edge and yet it does not actually tell you how to do that. That should be added.
Also the "Shortcuts" section is a bit of a mess. A markdown mistake?

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

ID: 462f093f-9486-8086-da81-c689b7ef7d9e
Version Independent ID: 0424a5af-15a6-1ccd-5732-12edb757838a
Content: Microsoft Edge DevTools - Console - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide/console.md
Product: microsoft-edge
GitHub Login: @erikadoyle
Microsoft Alias: edoyle

Thanks @sirredbeard! Corrected these issues. :)
https://github.com/MicrosoftDocs/edge-developer/pull/140


Outdated gulp-less version produces build error

In the touchstonejs-tasks module the "gulp-less" dependency uses an outdated version "^2.0.2" which leads to a known error with nodejs version 0.12.x: https://github.com/plus3network/gulp-less/issues/140
Fix: Update the gulp-less dependencie to the latest "3.0.1" version and the less task will run successfully.

Fixed by #46


Begin replacing model build methods with updaters

https://trello.com/c/pAjSCymq/746-its-hard-to-follow-how-we-build-a-revision-and-to-see-whats-changed
To some extent I think I've tried to avoid making the ImagesController
even longer, but this may have made it less readable and some more
question methods somewhere might be beneficial. I've outlined an
idea for doing this below, but I worry that it's just disguising a bunch of
single-use code through a convenient interface...
Previously we used a tree of 'build_revision_update' methods on each
model, which has lead to some duplication between models and quite a lot
of extra code in the model classes. This makes a start at extracting
this logic into a set of 'updater' classes with some helper methods.
The updaters take a hash of 'fields', which can include columns,
collections and sub-fields for sub-revisions. I've used inheritance to
DRY-up some of this logic, as well as create a clear interface for the
updater classes that derive from it.
Some of the existing code doesn't fit very well into the updater
classes, as it's specific to a single use-case and doesn't make sense to
share centrally. This does mean the ImagesController now has a bit more
code in it than before. A nice compromise that would avoid bloating the
existing controller and the new RevisionUpdater, would be to use an
'include' pattern to add more image-specific methods to RevisionUpdater.

@kevindew I've now finished adding tests for this. I've also replaced the NotImplementedError with a helpful message, which seems preferable to some anonymous/custom exception; but I'm open to an alternative approach if you have one :-).

@kevindew I've addressed your comments :-).

@kevindew this is ready to merge again if you want to take another look.


how to use rendering cluster

I have 10 servers, my project is about short videos. Today the rendering time is 50 seconds. But the idea is to render under 10s. So, I'm trying to find the best way to do it.

Hello @fyshequ
If projects are really small, and you have a lot of videos, I'd suggest just pushing all of those videos to the nexrender-server and they would be divided by nexrender-worker and rendered simultaneously.
So in case 10 videos you need to render at once, rendering all videos would take roughly 50 seconds, instead of ~10 seconds each accounting up to ~100 seconds total.
And also I suggest you read my comment in the previous issue: #191
Cheers!


Drop SPDY support

Motivation:
SPDY has been superseded by HTTP/2. Chrome has dropped support in 2016 and GFE no longer negociate it.
Modifications:

drop codec
drop examples
drop constants from ApplicationProtocolNames

Result:
SPDY support dropped from Netty 5

@slandelle thanks a lot


Desktop - Logo Position in Large Browser

Logo is resting up against browser bar when in a large browser window.

Updated...


Driving a course in Metadrive test

For #30694 (WIP). This is a draft for the bounty. Currently has 3 failure cases: 2 collisions and drive out of lane. I have a few questions regarding the bounty:

Do you want a new driving course from the old one I'm not sure what you mean by driving a defined course?
For logging as artifacts, this would make this bounty the same as #30693 right? since you would have to be able to run on CI anyway, so do you want that part to be on the other bounty or this one still?

For logging as artifacts, this would make this bounty the same as #30693 right?

This just means we have to get loggerd and encoderd running on PC and it creating logs, adding it as an artifact would be easy to add once we have it creating the logs

Do you want a new driving course from the old one I'm not sure what you mean by driving a defined course?

The circle course is fine, just have it complete that.

For logging as artifacts, this would make this bounty the same as #30693 right? since you would have to be able to run on CI anyway, so do you want that part to be on the other bounty or this one still?

Good point, I moved that to the other bounty.

@bongbui321 I'm unlocking the bounty since it's been a couple weeks. feel free to open the PR back up if you start working on it again


Mobs agro behavior

This is more of a suggestion/question rather than an issue. But, is there any possibility of making the mobs passive at first then agro you when you start attacking them? I've 0 java and minecraft modding experience so idk how feasible that is.

Probably, but that's not the point of this mod, and would require approaching it in a different way


Tabs (Artists, Albums, Songs,..) in "My Library" missing

Since a couple of weeks I'm not able to the tabs: Playlists, Stations, Artists, Songs and Genres anymore.
The whole menu bar is simply missing:

This is what it looks like in the browser:

Here's some information on the system I'm running

I couldn't find any similar issue, if you need further information please let me know.

fixed in v1.4.1.
Please close


Refactor fallibility of conversions from IR to Rust types

The old ToRustTy and ItemToRustTy conversion traits were problematic in that they assumed infallibity. That assumption is problematic because we are often dealing with C++ constructs for which Rust has no equivalent and we don't have a usable layout from which to generate an opaque blob in its stead. But, a usable layout is often "up the stack" if only we had a way to get up there.
For example, Rust does not currently have an equivalent of const value template parameters, and libclang will not give us a layout for template definitions with const value template parameters. However, libclang will give us the layout for an instantiation of such a template definition.
First, this commit separates the concepts of generating an equivalent Rust type from generating an opaque blob of the same size and alignment as an IR thing. This consolidates and DRYs up a ton of code involved in falling back to an opaque blob (and doing our best to get a Layout for the blob) when we can't generate an equivalent Rust type for some IR thing.
Second, it separates fallible and infallible conversions, and provides a nice little framework for when to choose which. This gives us one single place where we do this whole dance:
if could not generate equivalent Rust type:
    if we have a layout:
        return opaque blob based on layout
    else:
        return opaque blob based on a wish and a prayer

The ToRustTy trait is replaced by the TryToOpaque, ToOpaque, TryToRustTyOrOpaque, and ToRustTyOrOpaque traits. The ItemToRustTy helper was just a way to avoid ToRustTy's Self::Extra parameter when it was not needed, and is simply removed without a replacement.  We could introduce ItemTryToOpaque, ItemToOpaque, etc... traits, but the cost of the added boiler plate would outweigh the benefits of not passing &() at call sites, IMHO.
In addition to being a nice code cleanup, this also fixes #573.
r? @emilio

 Warning

These commits modify unsafe code. Please review it carefully!

Also, I should add the original test case from https://github.com/servo/rust-bindgen/issues/573#issue-213401183 to our test suite.

@bors-servo r=emilio

:pushpin: Commit b312c9c has been approved by emilio

:lock: Merge conflict

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

@bors-servo r=emilio

:pushpin: Commit 92c1a25 has been approved by emilio

:hourglass: Testing commit 92c1a25 with merge 4bc1336...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing 4bc133665dc3fb7ca6535d9a8056e6b49a30caac to master...


git: Properly redact oauth from git credential

This ensures that the token is not printed to stdout
even when using verbose, either with the stdout of the
git operation, or thereafter.

Reviews in this chain:
└https://github.com/Skydio/revup/pull/175 credential: Remove leading slash before path
　└https://github.com/Skydio/revup/pull/176 git: Properly redact oauth from git credential

#
head
base
diff
date
summary

0
6dd9492d
41a3467c
diff
Apr 3 12:11 PM
4 files changed, 19 insertions(+), 6 deletions(-)


feat(#488): Handle All Bytecode Instructions

In this PR I added handlers for all bytecode instructions. Now jeo-maven-plugin understands all possible bytecode instructions in methods.
It is the first attempt to enable spring-fat integration test. However we still have some problems related to spring-fat integration test and still can't enable it.
Related to: #488.

History:

feat(#488): handle all opcodes up to 53 instruction
feat(#488): handle all opcodes up to 158 (ifle)
feat(#488): handle all opcodes up to 178 (getstatic)
feat(#488): implement all required instructions
feat(#488): show that we don't have problem with inheritance disassembling/assembling
feat(#488): show that we don't have problem with inheritance assembling
feat(#488): remove redundant tests
feat(#488): disable 'spring-fat' test again
feat(#488): fix some qulice suggestions
feat(#488): fix compilation errors

PR-Codex overview
This PR introduces enhancements related to bytecode instructions, exception classes, and plugin version update.
Detailed summary

Added sipush method to Bar.java
Added returnsShortValue test in BytecodeClassTest.java
Created OriginalException and SomeException classes
Updated plugin version to 0.35.9
Modified bytecode instructions in BytecodeInstructionEntry.java

The following files were skipped due to too many changes: src/main/java/org/eolang/jeo/representation/bytecode/BytecodeInstructionEntry.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

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

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 13min)


TC.1.3 Display the four discard piles on the game page

@Saro7890 add commit code and close

d5b551744ab518fe8a0c7243f0707c31d525bac9


Predicate Variadic Generics Adoption

This adopts the variadic generics feature for all predicate APIs to support multiple inputs for predicates

@swift-ci please test

@jmschonfeld is the compiler crashing expected?

@jmschonfeld is the compiler crashing expected?

Sadly not expected, but @slavapestov is working on getting a fix into the compiler to correct this crash. I'll hold off on merging this until we have a snapshot available with the fix

@swift-ci please test

Moving this back to draft state as I tackle a few outstanding issues here

Closing, will revisit with a new PR rebased on main


Gitea 1.13.4 - very high RAM usage during mirroring process

Gitea version (or commit ref): 1.13.4
Git version:
Operating system: Ubuntu 20.04.2

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Description

When I try to mirror https://github.com/vim/vim than gitea RAM usage is growing to 13GB!!! Reproducible.
Screenshots

With only 8 GB RAM the server is swapping. And at the end Gite uses the allocated RAM.

should be a go-git bug ... fixed in gitea >= v1.14.0

Since this change is so big, we will not back port the change go-git -> git command line and keep v1.13.4 as it is.
What we can do is to release v1.14 ASAP.

Since this change is so big, we will not back port the change of go-git -> git command line and keep v1.13.4 as it is.

It's ok.

What we can do is to release v1.14 ASAP.

Whas is ASAP? There are ~4 opened tickets for the 1.14 release.

Since this change is so big, we will not back port the change of go-git -> git command line and keep v1.13.4 as it is.

It's ok.

What we can do is to release v1.14 ASAP.

Whas is ASAP? There are ~4 opened tickets for the 1.14 release.

In fact, it's only one issue and one pr to this issue and the changelog pr.

Works very good! Thx!


[SQLite] Integers > Number.MAX_SAFE_INTEGER are rounded

Bug description
SQLite uses dynamic typings which enables storing pretty much any value inside a column, regardless of the column type. This lead to issues wrt. to integers.
Consider a column of type Integer, which traditionally, represents a 32-bit signed integer for most databases.
Now, manually store a 64-bit signed integer in that column. eg: i64::MAX == 9_223_372_036_854_775_807.
The Query Engine will read that value as an i64 (because that's how all Integers are deserialized in the Query Engine) and pass it to our json serializer (serde_json), which will serialize it as a number.
serde_json::to_string_pretty("9223372036854775807")?; // "9223372036854775807"

The problem is that Javascript only supports number up to 2^53 - 1. So when deserializing that JSON,  that number will get weirdly rounded. Try the following script below in your browser console 👇
parseInt("9223372036854775807") //<PHONE_NUMBER>854776000

This does not happen for other SQL databases because they're all strongly typed and prevent Int64 values to be inserted in Int32 columns in the first place.
Pretty much the same issue happens in MongoDB though, if you manually insert a Bson::Int64 into your database and try to read it as a Prisma Integer field.
Solutions (all breaking)
1. Return a BigInt if the integer is too large
We could dynamically serialize integers as either Number (if value is in i32 range) or BigInt. However, this would terribly downgrade the developer experience and potentially lead to bad issues.
First, we'd have to change the TS typings for all integers on SQLite to be: number | BigInt. This makes it very cumbersome to deal with client results since you'd always have to check whether a value is one or another.
Second, if you somehow ignore the typings, you might end up with a perfectly working code in dev that fails in production because of your dataset. If your dataset in prod contains a number > i32::MAX, we'd coerce it to BigInt and so any arithmetic operations done on this value with a value that's not a BigInt would lead to a runtime error.
1n + 1 // Uncaught TypeError: Cannot mix BigInt and other types, use explicit conversions

2. Always return BigInts
To prevent the potential inconsistencies in the results, we could always coerce SQLite integers to BigInt, just to ensure consistency and avoid weird JS rounding when values are > Number.MAX_SAFE_INTEGER. However, since BigInts cannot be transparently used with "regular" JS numbers, it makes the experience pretty cumbersome, especially if you know none of your values are greater than an int32.
3. Error out if a value > i32::MAX is read from a column of type Integer
This option would ensure the "strictness" of the data that we return to the Client. The only way for users to read BigInt values from their db would be to migrate their column to BigInt, in which case we'd always coerce the values to BigInt.
How to reproduce

Setup a SQLite db with the Prisma Schema below
Insert the following value in the int column:<PHONE_NUMBER>854775807
Execute the script below

import { PrismaClient } from "@prisma/client";

async function main() {
  const prisma = new PrismaClient();

  console.log(await prisma.testModel.findMany());
}

main();

Prisma information

model TestModel {
  id Int @id
  int Int
}

Is letting the user specify the behavior an option? For example, implement (3) and give the user a way when making the query to ask for (2). Do we have any examples in other areas of that sort of configurability?

Yeah just realized this as well when i realized unix time im getting from the db was wrong.
I was expecting Int to be 64bit since there is no Long or etc and JS already supports bigger numbers
And I thought BigInt was for real numbers.

Prisma generally tries to avoid configuration and choose the correct behavior for all users. Hence we have almost no ways to configure how Prisma Client behaves regarding queries.
Here as well I would say that the Prisma schema does not match the data that is returned from the database, so (3) sounds like a valid choice. (Optionally but optimally with an error message that explains to the user how to circumvent this [replace Int with BigInt in the Prisma schema and optimally migrate the database]).

Note: We agree on option 3. It's the most correct and has a clear actionable way forward for the user.

Pretty much the same issue happens in MongoDB though, if you manually insert a Bson::Int64 into your database and try to read it as a Prisma Integer field.

Just curious. Does this code change also happen to fix the MongoDB issue?


[EXE] Learning KNN supervised classification

Learning Goals
Learn kNN algorithm for supervised classifications. Preferably use the kNN package from scikit-learn.
Prerequisites
Some basic of kNN will be assumed. If scikit-learn is used, some basics of how to install scikit-learn library is assumed.
Data source/summary:
I'm agnostic about which dataset to use, so anything suggested from a textbook exercise/blog is good.

Hey! I would like to work on this. I was thinking about beginner friendly iris plant classification using KNN as the project, using iris dataset. Let me know about your thoughts!

Hi @Sayoni26, please go ahead and implement the codes. Thanks for contributing ! Please read the contributing guidelines and other previous projects in this repo to understand the format and organization. Looking forward to your PR.

hey @gimseng, can I try this code, I m new to machine learning but i can definitely do decision trees.

Since comments and replies take a lot of time, I m making a PR, please check it and approve it ... I am a first timer here

Hello! I would love to make a contribution.  Since I'm also still learning,  I'd love to help fellow learners to understand KNN using simple explanations.
Thanks!

Hello @gimseng,
I'm enthusiastic about contributing to this task and assisting learners in comprehending the strategies for handling imbalanced datasets effectively. I am interested in creating an informative guide that covers various techniques to address class imbalance in datasets, spanning from simple approaches like resampling to more advanced methods like ensemble techniques and using specialized algorithms.
My plan is to develop a comprehensive tutorial that encompasses the following key aspects:
Introduction to Handling Imbalanced Datasets: Providing an overview of why dealing with class imbalance is crucial in machine learning and the potential challenges it poses.
Resampling Techniques: Explaining the concept of resampling, including both oversampling (e.g., SMOTE) and undersampling and oversampling approaches and when and how to use them. I'll provide practical code examples to demonstrate how to implement these techniques using popular libraries.
Cost-Sensitive Learning: Discussing the concept of cost-sensitive learning and how it can be used to assign different misclassification costs to different classes. I'll include code examples to illustrate its implementation.
Ensemble Techniques: Introducing ensemble methods as a way to improve classification performance on imbalanced datasets. I'll explain how techniques like Balanced Random Forest and EasyEnsemble work and provide code examples.
Using Specialized Algorithms: Highlighting algorithms specifically designed to handle imbalanced data, such as the Adaptive Synthetic Sampling (ADASYN) algorithm. I'll walk through how to use these algorithms and showcase their impact.
Comparative Analysis: Comparing the effects of different techniques on an imbalanced dataset, including their impact on model performance, precision, recall, and F1-score. Visualizations will be included to help learners understand these differences.
Discussion: Engaging in a discussion about the scenarios in which each technique is most suitable, considering the nature of the dataset, the algorithm, and the problem at hand.
For datasets, I'm considering:
Credit Card Fraud Detection Dataset: A widely-used imbalanced dataset, suitable for illustrating the application of various techniques.
Diabetes Classification Dataset: To showcase the handling of class imbalance in a medical context.
Online Retail Dataset: For demonstrating the impact of imbalanced datasets on a real-world e-commerce scenario.
I'm open to feedback and suggestions regarding this plan. My aim is to create a user-friendly and informative resource that equips learners with the knowledge and tools to tackle class imbalance effectively in their machine learning projects.
Best regards,
Anurav Modak


Option to disable === warnings for typeof comparisons with string literals

Any typeof comparison with a string literal (typeof X == 'object') is guaranteed to be equivalent to the triple-equals version, as the types of both arguments are guaranteed to be strings. It'd be useful to be able to explicitly allow this, without having to disable triple-equals checking (eqeqeq) across the board as now. I'd actually argue for this to be the default behaviour for eqeqeq, but I'd be happy with a relaxing option at least.
Notably the article the JSHint references about equality comparison in the eqeqeq docs (http://javascriptweblog.wordpress.com/2011/02/07/truth-equality-and-javascript/) actually explicitly describes triple-equals here as an example of 'equality overkill'.
I'd be happy to have a go at implementing this, as long as there's general agreement that it's something JSHint would accept. Thoughts?

"There is irrefutably no benefit to abstract equality comparisons and therefore no reason to ever use them."

The sheer overstatement here is astounding. It is eclipsed only by the irony of the reality: to disable eqeqeq, setting it to false is actually inadequate and fails to update the setting. Instead, you must say
"eqeqeq": true  // enables setting
"eqeqeq": false  // enables setting
"eqeqeq": null  // disables setting

If only someone had used an abstract comparator here.


August Sampling To Do

[ ] Contact Licor and explain our upcoming irrigation test; we assume we want dessicant on-hand for the 7810 but
Will the smart chamber + 7810 be ok in inundated water (with raised collars)?
Explain spot sampling with 7810 and automated measurements with 8100 + 8150
any other maintenance for automated systems prior to event
Video link to change 7810 desiccant: click here!
[ ] Guest housing available at SERC?
[ ] Are we doing full sampling for seawater test?
[ ] System monitoring, what does this exactly mean?
[ ] Download all Licors, wipe CF

Doghouses
Pallets


Swift 2.0 Issues Not working at all.

 Please fix it. Appreciate that.

Please update the pod spec so that the new version gets published for supporting swift 2.0. thank you very much

In your pod , write this :
pod 'SCLAlertView', :git => 'https://github.com/vikmeup/SCLAlertView-Swift.git'

Changes in swift 2 weren't that many, I have made and got project running, not sure how to upload changes though as haven't use GIT much

September 23rd, 10:48: Push for `SCLAlertView 0.4.2' initiated.
September 23rd, 10:48: Push for `SCLAlertView 0.4.2' has been pushed (2.812916978 s).

Thanks @vikmeup You rock!


foundry hud weblate

hi, is it possible to add your mods to foundry hud weblate for us to translate or update the translation faster? https://weblate.foundryvtt-hub.com/

Sure thing. Done.

VAE has been submitted to weblate, but it's simply taking a while. Hopefully not much longer.

Just confirming that VAE is now on weblate.


S147925 - Updating custom chart to include count in pie chart labels

Updated the pie chart labels and tooltip from "value: percentage" to "value: count (percentage)". Also changed label and tooltip formatting to be consistent with each other

nice giphy.  also 🐑 it.


Created group from MS Teams - rule doesn´t work

Hi.
I have set rules and it works when I create group from OWA. But from MS Tems doesn´t work - domain for group is default in tenant. Have you any idea why?

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

ID: 96002890-7e35-a0ab-a849-d94fef254fa8
Version Independent ID: 2c4105db-b713-4829-393e-de355c5c60d4
Content: Choose the domain to use when creating Office 365 Groups
Content Source: Office365-Admin/create-groups/choose-domain-to-create-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@ondraandy - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
If you don't mind please keep us posted here on the resolution so we can get the applicable information you are discovering back into the docs.


Revert "fix(apps/prod/tekton/configs/pipelines): use checkout result as commit sha to downstream tasks"

Reverts PingCAP-QE/ee-ops#1052

/approve


#22: Added test for ApplicationError.Unknown

@rultor merge

@rultor merge

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

@rultor merge

@gilraenburland Done! FYI, the full log is here (took me 4min)


Support ObjectMetadata parameters for media uploads.

In case the application wants to upload an object with metadata. The member functions to change are:

[x] Client::InsertObject
[x] Client::WriteObject
[x] Client::UploadFile.

All upload operations (Client::InsertObjectMedia, Client::WriteObject, and Client::UploadFile) now support this via the WithObjectMetadata option.


documentation: Empty/missing doc on fx-hint

page https://jinntec.github.io/fore-docs/elements/ui/hint/ is just empty :(

good point - actually we haven't used the element ourselves up-to-now and i rather think of removing it as it doesn't add much value over a HTML span with a class. It's a reminicense of the XForms hint element as was bulk-created back in the days. Opinions or ideas why fx-hint may still be worth it?

reviewed again and IMO doesn't make sense any more - i marked fx-hint as deprecated.

Could fx-hint be used to specify tooltip text to be displayed when a user hovers over a control?

I do not think tooltips are a great pattern for most cases: they do not work nicely with screenreaders. From a Fore standpoint they are a mostly visual component and if you need them, (most) CSS frameworks have some way to display them.
I found https://inclusive-components.design/tooltips-toggletips/ a good read.
fx-hint does not feel like a good Fore component


Dynamic Origin For Cors

Node version: v12.16.1
Sails version (sails): 1.2.3
ORM hook version (sails-hook-orm): 2.1.1
Sockets hook version (sails-hook-sockets): 2.0.0
Organics hook version (sails-hook-organics): 1.0.0
Grunt hook version (sails-hook-grunt): 4.0.1
Uploads hook version (sails-hook-uploads): N/A
DB adapter & version (e.g. sails-mysql@5.55.5): N/A
Skipper adapter & version (e.g. skipper-s3@5.55.5): N/A

We need to set origins dynamically for subdomains in our CORS options. I have tried adding the domains on the fly with http middleware but it gets overwritten by the sails CORS implementation.
How should I go about fixing this? The express CORS package does support this behaviour so I would expect that it's possible in sails too.

@Peter-Barrett Thanks for bringing this to our attention.  Do you have an example of the use case for dynamic origins in CORS?  In the meantime, I'll check with the team for further info on best practices for dynamic origin configuration in CORS.

Hi @johnabrams7, we're setting up a system whereby each pull request results in a new subdomain with our web app for testing, we need to be able to check the the domain for a certain string to either allow or deny access via CORS.

@johnabrams7 any further thoughts on this?

Hi, @Peter-Barrett!
I think you're right: the fact that this behavior is supported by the Express CORS package seems to imply that it could be supported in Sails as well.
If you have the time and/or inspiration and want to try your hand at tinkering with the security hook where CORS functionality is being implemented, we'd be happy to review a PR and take a look at any questions you have along the way. If you do submit a PR, feel free to link it in this thread!

Thanks for taking a look, unfortunately I don't have the time to dive into it atm, I had a quick look and it looks like it will take awhile with how CORS is currently set up. We've since moved on from it and solved it from the other direction instead so it's not a requirement for me now.


Using a single theme for PDF and HTML generation

I have a document which currently builds a PDF using a yaml theme. We would also like to host this document on a website and build it to an HTML page, however reading the documentation it looks like I would need to build a second HTML theme using CSS.
Would it be possible to use a single YAML file as a theme for both PDF and HTML generation?

What you are asking about is not possible. These are different theming systems.


The horizontal scrollbar of the log window disappears

Hello Ilia,
I have installed Authentic 19.93.1 version on Webmin 1.994 (minimal package) and I'm using Firefox 100.2.
I use Webmin mainly to read logs; despite the last update (issue: Scrollbar accessibility #1584) the scrollbars of the log window are barely visible.
The horizontal scrollbar, in particular, disappears most of the times after a few seconds, while other times it's visible until you start to scroll the log vertically. In both cases it doesn't reappear until you hit the "refresh" button.
Even when it's visible it's really difficult to grab, being almost transparent and very thin.
Kind regards,
Andrea

Hello,
Thank you for the heads up. You are right, the scrollbars needs a fix. This is done now with the latest commit.

Hello Ilia,
I am here again for a quick update.
I have installed Webmin 1.996 with Authentic 19.95.2, and I see that the problem I mentioned earlier - the horizontal scrollbar of the log window that hides itself - still happens with Firefox (I'm now using the 102.0.1 version) while it does not happen with Edge, for example.
This had led me to investigate a little deeper, until I've found this line of code:
<div class="CodeMirror-hscrollbar" tabindex="-1" cm-not-content="true" style="right: 0px; left: 0px; height: 18px; display: block; pointer-events: none;">
Changing the property of the selector pointer-events from none to auto the horizontal scrollbar stays visible as expected.
Kind regards,
Andrea

Howdy, Andrea!
I am testing a new theme pre-release now, and in 20-30 minutes I will add CodeMirror update that fixes few recent bugs that people reported.
Although, perhaps this is Firefox an adoption of some Windows features?
https://connect.mozilla.org/t5/discussions/scrollbars-disappear-in-firefox-100/td-p/5433

Although, perhaps this is Firefox an adoption of some Windows features?

It look like it, Ilia! I have Windows 11 and indeed the "Always show scrollbars" setting under the Accessibility menu is disabed by default, as you pointed out.
Enabling it fixes the problem. I stumbled into one of the dark corners of Windows evidently... thank you for the suggestion!
Andrea

Glad to hear it works for you!


[SUP1-164] Make webhooks durable

Right now they are best-effort
From SyncLinear.com | SUP1-164

What I heard was performance and reliability issues dealing with high-volume CDC data events where dropping an event resulted in data loss.
Whereas our events don't contain data, but metadata. We are also batch and low-volume. We also have other parts of the system that are not reliable, e.g. the tickets that Albert has logged, so it seems like to address reliability holistically we have to actually take on more tickets/work in addition to this one
Planning for the week; can chat more in-office

Given the comments during the Warmly call that indicated their concerns about the reliability here, I think we should do something.

Given sync frequency I think this is OK for now

Migrated to internal project management tool.


Strange item_type found in p2p.log

test1 tag.
"item_type":3624466160
"item_type":3624527728
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] beginning an iteration of advertise inventory
        node.cpp:1175
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] peer->peer_needs_sync_items_from_us: true
        node.cpp:1188
2015-08-22T22:20:33 p2p:advertise_inventory_loop  clear_old_inventory ] Expiring old inventory for peer <IP_ADDRESS>:1776: removing 0 i
tems advertised to peer (0 left), and 0 advertised to us (0 left)                 peer_connection.cpp:479
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] peer->peer_needs_sync_items_from_us: false
        node.cpp:1188
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] inventory_to_advertise: [{"item_type":1001,"item_hash":"e53c5595bc7f37891ed00c277ddcef06e8eb37bd"}]                     node.cpp:1196
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] *peer->inventory_advertised_to_peer.find(item_to_advertise): {"item":{"item_type":3624466160,"item_hash":"ff7f00002b00000000000000ffffffffff7f0000"},"timestamp":"2023-11-27T10:29:02"}
                node.cpp:1200
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] *peer->inventory_peer_advertised_to_us.find(item_to_advertise): {"item":{"item_type":3624527728,"item_hash":"ff7f00000000000000000000b86309d8ff7f0000"},"timestamp":"1948-09-30T21:23:04"}
                node.cpp:1202
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] advertising item e53c5595bc7f37891ed00c277ddcef06e8eb37bd to peer <IP_ADDRESS>:40479                 node.cpp:1212
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] advertising 1 new item(s) of 1 type(s) to peer <IP_ADDRESS>:40479                    node.cpp:1218
2015-08-22T22:20:33 p2p:advertise_inventory_loop  clear_old_inventory ] Expiring old inventory for peer <IP_ADDRESS>:40479: removing 0 items advertised to peer (105 left), and 0 advertised to us (0 left)                  peer_connection.cpp:479
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] peer->peer_needs_sync_items_from_us: true
        node.cpp:1188
2015-08-22T22:20:33 p2p:advertise_inventory_loop  clear_old_inventory ] Expiring old inventory for peer <IP_ADDRESS>:62015: removing 0 items advertised to peer (0 left), and 0 advertised to us (27 left)                 peer_connection.cpp:479
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] peer->peer_needs_sync_items_from_us: false
        node.cpp:1188
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] inventory_to_advertise: [{"item_type":1001,"item_hash":"e53c5595bc7f37891ed00c277ddcef06e8eb37bd"}]                     node.cpp:1196
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] inventory_to_advertise: [{"item_type":1001,"item_hash":"e53c5595bc7f37891ed00c277ddcef06e8eb37bd"}]                     node.cpp:1196
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] *peer->inventory_advertised_to_peer.find(item_to_advertise): {"item":{"item_type":0,"item_hash":"000000002b00000000000000ffffffffff7f0000"},"timestamp":"2023-11-27T10:29:02"}                  node.cpp:1200
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] advertising 0 new item(s) of 0 type(s) to peer <IP_ADDRESS>:53168                      node.cpp:1218
2015-08-22T22:20:33 p2p:advertise_inventory_loop  clear_old_inventory ] Expiring old inventory for peer <IP_ADDRESS>:53168: removing 0 items advertised to peer (12 left), and 0 advertised to us (93 left)                    peer_connection.cpp:479
2015-08-22T22:20:33 p2p:advertise_inventory_loop advertise_inventory_ ] peer->peer_needs_sync_items_from_us: true
        node.cpp:1188
2015-08-22T22:20:33 p2p:advertise_inventory_loop  clear_old_inventory ] Expiring old inventory for peer <IP_ADDRESS>:34762: removing 0 items advertised to peer (0 left), and 0 advertised to us (60 left)                     peer_connection.cpp:479
2015-08-22T22:20:33 p2p:advertise_inventory_loop         send_message ] peer_connection::send_message() enqueueing message of type 5001 for peer <IP_ADDRESS>:40479                  peer_connection.cpp:365
2015-08-22T22:20:33 p2p:advertise_inventory_loop send_queueable_messa ] peer_connection::send_message() is firing up send_queued_message_task                   peer_connection.cpp:354

Good catch.  It's actually reading something undefined at the end of the vector.  The code reads like this:
          if (peer->inventory_advertised_to_peer.find(item_to_advertise) == peer->inventory_advertised_to_peer.end() )
             wdump((*peer->inventory_advertised_to_peer.find(item_to_advertise)));
          if (peer->inventory_peer_advertised_to_us.find(item_to_advertise) == peer->inventory_peer_advertised_to_us.end() )
             wdump((*peer->inventory_peer_advertised_to_us.find(item_to_advertise)));

These statements need fixed, they are currently dumping an undefined element is minimal value.  I'm not sure what they are intending to dump.  I am tempted to suggest removing these four lines entirely since we have enough p2p logspam.
I will assign this ticket to our p2p guru @emfrias but if I don't hear back in a day or two, I'm going to delete the code and close the ticket.

Fixed by 8ebc1cfc43e97060f19dae78fbaa3f361fd30aa8


More Alert Types

Currently Alert component has only one default type i.e Warning. It would be better to have more Alert types such as Danger.

Coming soon:

with Espresso Design #52


Should operate on network address

Experiencing this error on master:
2016/03/06 21:53:26 Token created 5c09a708-f9ed-4b85-8ca5-1156453a6254
2016/03/06 21:53:26 Agent: Entering k8sPodUpHandler()
2016/03/06 21:53:26 Agent: Got request for network configuration: &{{veth0-11957 de:ad:be:ef:00:00 <IP_ADDRESS>} map[namespace_isolation:off]}
2016/03/06 21:53:26 Agent: Entering k8sPodUpHandle()
2016/03/06 21:53:26 Isolation is false
2016/03/06 21:53:26 Helper: Waiting for interface veth0-11957, 0 attempt
2016/03/06 21:53:26 Agent: ensuring interhost routes exist
2016/03/06 21:53:26 Acquiring mutex ensureInterhostRoutes
2016/03/06 21:53:26 Acquired mutex ensureInterhostRoutes
2016/03/06 21:53:26 Helper.Executor: executing command: /sbin/ip ro show <IP_ADDRESS>/16
2016/03/06 21:53:26 Helper: creating route
2016/03/06 21:53:26 Helper.Executor: executing command: /sbin/ip ro add <IP_ADDRESS>/16 via <IP_ADDRESS>
2016/03/06 21:53:26 Releasing mutex ensureInterhostRoutes
2016/03/06 21:53:26 Agent: Unspecified error (Agent: Can't create IP route (target <IP_ADDRESS>/16 -> <IP_ADDRESS>: cause External command unsuccessful (/sbin/ip [ro add <IP_ADDRESS>/16 via <IP_ADDRESS>]: exit status 2)))

Bit odd, since this was fixed and merged from https://github.com/romana/core/pull/42

ok


Change Tanner traits

Tanner has to vote someone every moring from the second day, or else, he will die
After 7 days, the rule in 1 is not applied anymore and Tanner will become a normal villager

Please also have a look at these parts:
https://github.com/dnh-bot/dnh-werewolf-bot/blob/main/json/role_info.json#L134
https://github.com/dnh-bot/dnh-werewolf-bot/blob/main/game/text_template.py#L209

I updated the attributes in Tanner class to remove some variables need to track in main flow.
I also changed Tanner party to Volatile Allegiance ref here, I think when we use this party, it may help us in the future for some roles that can change party in the game.


fix issue #2570

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.43% :warning:

Comparison is base (e0e0043) 79.22% compared to head (b21fe99) 78.79%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2819      +/-   ##
==========================================
- Coverage   79.22%   78.79%   -0.43%
==========================================
  Files         633      632       -1
  Lines       52242    52131     -111
==========================================
- Hits        41391    41079     -312
- Misses       8808     9012     +204
+ Partials     2043     2040       -3

Flag
Coverage Δ

go-1.15-386
?

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
?

go-1.17-386
?

go-1.17-amd64
?

go-1.18-386
?

go-1.18-amd64
78.79% <100.00%> (-0.03%)
:arrow_down:

go-1.19-386
?

go-1.19-amd64
?

go-1.20-386
?

go-1.20-amd64
?

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

Files Changed
Coverage Δ

os/gmlock/gmlock_locker.go
100.00% <100.00%> (ø)

... and 9 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Ability to access remote resources while developing locally🚀 Feature Request:

Describe the solution
Add an option to have wrangler execute locally, yet be able to fetch / interact with remote resources like KV and etc.?
In my use case, I have some resources that are sensitive that I don't want to store on a dev box, even if they are just "dev keys".  So, I want my worker to be able to retrieve them via KV.  Yet, I want to be able to perform integration tests against locally hosted stuff - i.e.; i'd like to call fetch("http://<IP_ADDRESS>:80/getThingy").

@kyle-pena-nlp have you tried wrangler dev --remote? Here's some more info on how to use it: https://developers.cloudflare.com/workers/observability/local-development-and-testing/#develop-locally-using-remote-resources-and-bindings

@kyle-pena-nlp have you tried wrangler dev --remote? Here's some more info on how to use it: https://developers.cloudflare.com/workers/observability/local-development-and-testing/#develop-locally-using-remote-resources-and-bindings

Certainly!  But isn't it the case that when developing with wranger dev --remote, that any requests issued through the fetch API will originate from cloudflare's servers rather than the local machine?  In which case, an integration tests involving a local server would be impossible without setting up a tunnel or something.


Agricultural Journal...

Would it be possible that when Breeding or Cross-Breeding with the Agricraft Farming Station, that the information from the Seed Analyzer Module be recorded in the Agricultural Journal?

The journal has no API for me to access it, but if @InfinityRaider would add one, I'd be happy to add it to the farming station.

Ill get on it when I have some time, really busy wiht master thesis lately.

I've added it to APIv2, When I release the next version (1.4.6)  I'll also release APIv2.

Great. Thx!


fixed a bug where sequence number check is checking against itself while editing

fixed a bug where sequence number check is checking against itself while editing

Done with the testing, no issues found. MR ready to merge.


Allow to manage TableWidget pageSize state externally

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1589

Relevant Lines:
2052

💛  - Coveralls


.pf-c-backdrop backdrop-filter causing visual defects in Chrome 78

We're seeing this in OpenShift console, but it also reproduces on https://www.patternfly.org/v4/documentation/react/components/aboutmodal.  Workaround fix is to override backdrop-filter (set it to none), which is what we're doing for OpenShift 4.3 since we can't bump PatternFly at this point in the dev cycle.

Override fix for OpenShift https://github.com/openshift/console/pull/3449

Hey @rhamilto, thanks for reporting this. We have a forum post with this recommended workaround, too - https://forum.patternfly.org/t/bug-with-the-backdrop-component-in-chrome/277
The issue has since been resolved in Canary, and I expect a fix in Chrome soon.

@rhamilto this appears to be fixed in the latest versions of chrome. If you can confirm, are you OK with us closing this issue?


Support using ccache

https://ccache.dev/
This issue is only about local usage. For using ccache in CI see #27.

Fixed with #22


[eas-cli][ENG-6229] add -e as a shortcut for --profile flag

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Adds -e as a shortcut for the --profile flag.
How
Add the -e shortcut to all places where we use --profile flag.
Test Plan
Run commands locally and see if the new flag is available in help and if it works.

/changelog-entry new-feature Add -e as a shortcut for --profile flag


duplicate ui-view tags in 0.2.10

Still having the same issue described in https://github.com/angular-ui/ui-router/issues/958 with 0.2.10 and AngularJS 1.2.21.

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


Add onRowDoubleClick option to BootstrapTable

This would resolve #250

@jlinn, thanks for contribution. I'll check this out. :)

Released on v2.5.6. Thanks :0


How to read a signal's data built via a store?

let todos = store({ x: 42 }, /* some setters */)

I guess I have to do: todos.value.x ? I was expecting todos.x to work, or at least to have value documented somewhere? Or should I also add getters along the setters? Or am I missing something?
Also, the codepen is outdated and use the old version API.

Afaict todos.x does work? Both with store() from v12.x and signal() from v13.0.0.

Ah, I must do something wrong:
let {store} = reef;

let todos = store({ x: 42 }, {})

console.log(todos.value.x); // 42
console.log(todos.x); // undefined

Also, the codepen is outdated and use the old version API.

Looks like it's using signal() to me, unless it was just changed. ;-)

From https://reefjs.com/advanced/#stores:
Isn't that outdated: Try store action functions on CodePen → ?

Ah, I must do something wrong:

Maybe you're using a really ancient version? Double check if your wires are plugged in. ;-)

Isn't that outdated

Yes, I was talking about https://codepen.io/cferdinandi/pen/wvmoPMg

Good catch! This is an oversight in the docs.
For data store() objects, the data you get back is immutable and has to be accessed on the value property. This ensures that the action methods don't collide with the data and provides a way to ensure the data can't be overwritten directly.
I'll update the docs to properly document this.

Fixed the docs!


[Windows] Add signature validator for MongoDB

Description
Adding digital signature validator based on "Get-AuthenticodeSignature" Powershell method. Includes example based on MongoDB installer.
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,windows2022


Get query button working in explorev2

Done:

Create new endpoint for updating explore viz
Send over new form_data when query button is pressed
Pass in slice_id and slice_name from explore endpoint
Move slice_id and slice_name from form_data to viz object

needs-review @ascott @mistercrunch

LGTM :ship:


[$1000] Half of LHN content disappears briefly after closing the Search page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on Search icon then close the page.
Notice that half of LHN content disappears for a brief moment.

Expected Result:
LHN content shouldn't disappear after returning from Search page.
Actual Result:
Half of LHN content disappears for a brief moment
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] iOS / Safari

Version Number: 1.2.76-7
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/221236455-34fa3e66-5be3-49c5-b547-5188a6fcab66.MP4
https://user-images.githubusercontent.com/43996225/221236897-8c50f97b-4fd4-43ca-ba1b-44f8eb8da5ed.MP4
Expensify/Expensify Issue URL:
Issue reported by: @hungvu193
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1677220623213269
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01919120e74f961e83
Upwork Job ID:<PHONE_NUMBER>388055552
Last Price Increase: 2023-02-28

Triggered auto assignment to @greg-schroeder (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

The platforms in the GH and in the Slack post are different - asking to confirm that info w/ @hungvu193

@greg-schroeder Whoops! This issue is 2 days overdue. Let's get this updated quick!

I reproduced this, checking to see if there's anything similar

Looks like a unique issue I think, couldn't find anything similar based on a few searches. Going to mark External

Job added to Upwork: https://www.upwork.com/jobs/~01919120e74f961e83

Triggered auto assignment to @maddylewis (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

Triggered auto assignment to @danieldoglas (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@greg-schroeder I think this is related to https://github.com/Expensify/App/issues/15543

Ah, I searched for a long time and couldn't find a duplicate issue. Bad search I suppose 🤔

Agreed it is related. Closing for duplicate on behalf of https://github.com/Expensify/App/issues/15543


Use vshard.router.map_callrw for map-reduce queries

https://github.com/tarantool/vshard/commit/a394e3f4008a7436f011c81028dcaadc270eebf6

Blocked by https://github.com/tarantool/vshard/issues/333

Blocked by tarantool/vshard#333

But it's not so. You can use map_callrw only if mode is write and current logic for other cases.
When https://github.com/tarantool/vshard/issues/333 will be solved you can replace "current" approach with map_callro/call_bro/...

Blocked by tarantool/vshard#333

But it's not so. You can use map_callrw only if mode is write and current logic for other cases. When tarantool/vshard#333 will be solved you can replace "current" approach with map_callro/call_bro/...

I'm not sure it's worth to implement half-baked solution


[Feature Request] Alternate Resin timing

The Resin calculator is super handy - I use it all the time. But it would be even more helpful if it could also show (or be set to show) when you'll hit relevant amounts of resin (as opposed to how much resin you'll have in certain amounts of time). Basically, how long until you hit any given multiple of 20 so you know when you can hit your next domain, boss, etc.
This would be even better if you could set your time zone and see what time you'll hit these milestones, and best of all if you could set reminders for them with the reminders bot (ping me whenever I have 40 resin available, for instance)

if you could set reminders for them with the reminders bot (ping me whenever I have 40 resin available, for instance)

This bit is already possible, see #27.

Exactly what I was going to request as well :D

Gladly your code is super well structure so I could quickly patch up #50 🎉

@NissaCam for now you can go to Customize > Manage Data and carefully edit resinEstimateMode from time to value

@NissaCam

It's actually already possible to switch between the estimation modes by clicking on the resin icon left to the calculator, but a setting like this would make it more explicit.
from: @phosphene47


Bump winit to 0.25

bevyengine/bevy#2186 bumped winit to 0.25, causing a dependency mismatch.

Nevermind, I got mixed up and thought that this pull request was meant for this branch.


Use CASE instead of bool to float casting in truth_space_table

Type of PR

[x] BUG
[ ] FEAT
[ ] MAINT
[ ] DOC

Is your Pull Request linked to an existing Issue or Pull Request?
https://github.com/moj-analytical-services/splink/pull/1893
Give a brief description for the solution you have provided
Casting boolean to float doesn't work in Postgres. This PR uses CASE instead to do that casting, when making the truth_space_table
PR Checklist

[ ] Added documentation for changes
[ ] Added feature to example notebooks or tutorial (if appropriate)
[ ] Added tests (if appropriate)
[ ] Updated CHANGELOG.md (if appropriate)
[x] Made changes based off the latest version of Splink
[ ] Run the linter

@ThomasHepworth  and @RobinL  Could you please have a look?


feat: countDistinct with multiple columns

Closes #1776

Sorry for the delay on this, but I'm looking to implement it now and found a weird edge-case. At the moment, you can specify an alias for countDistinct like builder.countDistinct('column as alias') and that get's transformed to select count(distinct column) as alias.
This of course won't work with multiple fields so I suggest we change this to the regular object notation like builder.countDistinct({ alias: 'column' }) or with multiple fields builder.countDistinct({ alias: ['column1', 'column2'] }).
I'll leave the current behaviour and only add the object notation as a new feature but maybe we can even add a deprecation notice for it. What do you think @elhigu?

I'm thinking about what usually { alias: ['foo', 'bar']} syntax should do. I don't know if there is any use cases which should be left untouched. Are you planning to add special case how that syntax works only with .countDistinct or general change how {alias:[]} is interpreted?
@joelmukuthu I'm sorry  I'm not sure if I understood what are you planning to do and what effects it will have to knex backwards compatibility? :)

I could be missing something but as far as I could tell the parts I touched are only dealing with building statements of type aggregate (excluding aggregateRaw) so the new {alias: [column1, column2]} syntax would only be used for those. I think that makes sense since you can't do
select (column1, column2) as alias from table

(or is it possible?) but you can definitely do
select sum(column1, column2) as alias from table

I made sure that the changes don't break the current behaviour, so for instance this test still passes: https://github.com/joelmukuthu/knex/blob/a3a5255ecbb2b4613eab21c237473d407f42591b/test/unit/query/builder.js#L2878-L2905.
FYI I decided to not bother fixing that failing mariasql test since it seems the dialect is being dropped anyway

This is ready for review. Notes:

Added support for multiple calls for all aggregate methods:

// single column:
builder.count('column1', 'column2');
// multiple columns:
builder.countDistinct('column1', 'column2');

Added support for object alias notation for all aggregate methods:

// single column:
builder.count({ alias: 'column' });
// multiple columns:
builder.countDistinct({ alias: ['column1', 'column2'] });

Kept support for string alias notation, though this only works with one column:

builder.count('column as alias');

I skipped validating whether or not the dialect supports calling whatever aggregate function with multiple columns i.e. no error is thrown if a user does something they're not supposed to do like countDistinct with multiple columns if the database doesn't support it.
I also skipped integration tests for aggregates with multiple columns for these dialects.

Docs: https://github.com/knex/documentation/pull/97

I think this  is good, sorry for taking so much time lookin these. I've been even more busy lately than I was before 😬

@elhigu no worries, thanks for taking a look! We appreciate the hard work you put in :)

@jrschumacher No, MySQL and PostgreSQL only, judging by tests. Does SQLite support all necessary features for this?

@kibertoad after some tests:
select count(distinct 'CustomerThing'.'Customer_id', 'CustomerThing'.'thing') from 'CustomerThing' - wrong number of arguments to function count()

select count(distinct('CustomerThing'.'Customer_id', 'CustomerThing'.'thing')) from 'CustomerThing' - row value misused

Works
select count(*) from (
  select distinct "CustomerThing"."Customer_id", "CustomerThing".thing from "CustomerThing"
) "CustomerThing"

@jrschumacher Would you consider raising a PR with a fix and expanded test?

@kibertoad Absolutely, just wanting to get some input. I'm not familiar with the use of distinct. I'm working on bookshelf/bookshelf#1936


On Linux, agda-mode hijack the keybinding

Situation
If I type Ctrl+x, it will pop a message say "already press ctrl+x, waiting for the second command", and this only happened in agda files.
Version
版本: 1.87.2
認可: 863d2581ecda6849923a2118d93a088b0745d9d6
日期: 2024-03-13T17:44:04.525Z
Electron: 27.3.8
ElectronBuildId: undefined
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.8.2-arch2-1

It's not appeared now, probably not an issue of the plugin.