Implement Line scrolling like in Visual Studio 2019

If you use the Line-Scroll function, the selected line is preserved - even if it leaves the visible area by scrolling. This is quite annoying and I don't know any editor that does it that way. The other editors make sure that the selected line always stays visible by moving it accordingly when it reaches the top or bottom of the screen. This results in a much better usability. As an example you can have a look at it in Visual Studio 2019.

Duplicate of #91867 , #59247

You can install the VS Keymap extension or you can use the editorScroll command in your keybindings.json:
e.g.
{
	"key": "ctrl+up",
	"when": "editorTextFocus",
	"command": "editorScroll",
	"args": {
		"to": "up", // 'up', 'down'
		"by": "line", // 'line', 'wrappedLine', 'page', 'halfPage'
		"value": 1, // number of units to move
		"revealCursor": true
	}
}


Fix merge-conflict-caused CI breakage

Both the linter and the tests are broken on main. A PR that tightens up the yamllint config was merged concurrently with several PRs that add YAML that the new config complains about. A PR that adds a field to testdata output files was merged concurrently with PRs that add new testdata output files (and so don't have that new field). These are merge conflicts, but Git wasn't able to recognize them as such; fix those conflicts.

Codecov Report

Merging #366 (501ade5) into main (1b4706b) will decrease coverage by 0.22%.
The diff coverage is 41.02%.

@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   59.09%   58.86%   -0.23%
==========================================
  Files          32       32
  Lines        3215     3253      +38
==========================================
+ Hits         1900     1915      +15
- Misses       1206     1229      +23
  Partials      109      109

Impacted Files
Coverage Δ

internal/ir/zz_generated.deepcopy.go
0.00% <0.00%> (ø)

internal/ir/infra.go
70.37% <62.50%> (-0.87%)
:arrow_down:

internal/gatewayapi/translator.go
84.58% <100.00%> (+0.22%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


SMD microswitch push button type

Hello,
Your BOM doesn't include SMD microswitch push button.
I have some but none of the them match the thumb on the board. Legs are too short
Could you refer the kind of SMD mSwitch that you are using?
Thanks

These are standard 6x6mm SMD microswitches. Here you have an example: https://es.aliexpress.com/item/1005003115135192.html

Thank you,
Definitively mine are not the right SMD for it.


some audiosprite sounds do not play properly on firefox

Version

Phaser Version: 3.15.1
Operating system: Linux / Windows
Browser: Firefox 65

Description
First of all, thank you for the great piece of software.
that being said here is my undestanding of the problem:
I have an audiosprite with some short sounds (30ms) and on firefox they won't play, also another sound (171ms) wont loop properly, it seems like that the entire sound is shifted somehow and it isn't played properly.
the same sounds when loaded individually (without audiosprite) plays properly
Example Test Code
// preload
this.load.audioSprite(spriteName, 'assets/audio/as.json', ['assets/audio/vp.mp3']);

// create
const sound = this.scene.sound.addAudioSprite(spriteName);

// short sound won't play
sound.play(marker);
// looping sound which is a bit longer plays but it is cut at the end
sound.play(maker, { loop: true });

Additional Information
I have added a console.log to check the values in for duration, offset etc in Phaser.Sound.WebAudioSound#createAndStartBufferSource and they are the same in both firefox and chrome
i have found this on baseAudioContext documentation regarding the reduced timer precision privacy setting on firefox, which involves the audiocontext.currentTime used to calculate the parameters for the playback, but disabling it did not solve the issue.

I'have found a solution to this issue, which, by the way is not caused by phaser itself but rather by audiosprite or ffmpeg, i write my solution here just in case someone else finds this an thinks it is a phaser issue.
on my linux machine the files created by audiosprite the mp3 version of the audiosprite has some gap at the beginning as shown here:

apparently chrome and other browsers do not consider this gap but firefox does, so the solution was to add ogg format as first one to load
this.load.audioSprite(spriteName, 'assets/audio/as.json', [
    'assets/audio/vp.ogg',
    'assets/audio/vp.mp3'
]);

and everything works smoothly
i hope this will be useful to someone


ARROW-12305: [JS] Update generate.py to python3

Fixes https://issues.apache.org/jira/browse/ARROW-12305

Can we close this in favor of #9959?


hi，bro，please help ，

When I train my own dataset, I always report this result at the beginning. I don't know exactly what went wrong. I'm going crazy. Please help me think about it
2022-12-16 22:46:39 | INFO     | yolox.core.trainer:203 - ---> start train epoch1
C:\cb\pytorch_1000000000000\work\aten\src\ATen\native\cuda\ScatterGatherKernel.cu:276: block: [0,0,0], thread: [0,0,0] Assertion idx_dim >= 0 && idx_dim < index_size && "index out of bounds" failed.
2022-12-16 22:46:42 | INFO     | yolox.core.trainer:195 - Training of experiment is done and the best AP is 0.00
2022-12-16 22:46:42 | ERROR    | yolox.core.launch:98 - An error has been caught in function 'launch', process 'MainProcess' (12104), thread 'MainThread' (15040):
Traceback (most recent call last):
File "tools\train.py", line 133, in
launch(
(x) H:\YOLOX-0.3.0>python tools/train.py -f exps/example/custom/yolox_s.py -d 1 -b 8

pls provide more detailed callstack


Bump Dependencies #6433274218

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/6433274218

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


book: Add interface builder chapter

This PR adds a new chapter about the interface builder.

Otherwise looks fine to me


Added Thames College

Thames International College, is an academic institution located at Surya Bikram Gyawali Mard, Old Baneswor, Kathmandu, Nepal offering a variety of undergraduate programs that leads to a degree from Tribhuvan University
Website: https://thamescollege.edu.np/

@shrijan101 Pull request merged. Thank you.


Integration test harness

Stand up real components to test various functions, especially those that rely on external components:

mongodb
kafka

Run automated tests to carry out end to end scenarios to exercise all the components involved

I have added the example of Kafka integration test to my fork (branch: draft). https://github.com/janrockdev/firefly-fabconnect/tree/draft/test/integration/kafka

create Kafka (Confluent) standalone server with UI and ecosystem
create topic "ffin"
send test message and compare with expected
remove all (this step can be removed to keep Kafka available for other tests of part of a development environment)

I can adjust it for the fabconnect as you will need. I kept logging there just in case you want to use it in CLI etc. It takes ~80sec to run the test with all images pull.


Add ability to set item image tint color.

• Allows for changing item image tint color (for use with template images) via itemImageTintColor property on WhatsNewViewController instance.
Example usage:
let whatsNew = WhatsNewViewController(items: [ . . . ]) whatsNew.itemImageTintColor = .red whatsNew.presentIfNeeded(on: self)

Thanks @erusso1 for the PR, I think this improvements makes sense for the project. I only have one question inline :)


buyProductWithOfferIOS fails with E_UNKNOWN error

Version of react-native-iap
2.5.5
Version of react-native
0.59.3
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
A purchase with an Apple Subscription Offer discount applied should succeed with some feedback, ie. 'Purchase was successful' modal from StoreKit. The discount transaction should also be reflected in the ios receipt thereafter.
Actual behavior
I'm able to get up to getting the StoreKit modals for seeing the correct discount price from the subscription offer and confirming the subscription, but it fails at the end w/ this Alert modal: Unable to Purchase Contact the developer for more information.
Can also confirm that the transaction is also missing from the iOS receipt.
On the other hand, I can confirm that a regular purchase transaction without any subscription offer succeeds in the current sandbox environment and am able to connect to the iTunes Store.
Here's the error stack:
{
    framesToPop: 1,
    code: 'E_UNKNOWN',
    nativeStackIOS: [
      '0   shine                               0x00000001050b9524 RCTJSErrorFromCodeMessageAndNSError + 156',
      '1   shine                               0x0000000105060dfc __41-[RCTModuleMethod processMethodSignature]_block_invoke_2.129 + 176',
      '2   shine                               0x0000000104fb9e18 -[RNIapIos rejectPromisesForKey:code:message:error:] + 512',
      '3   shine                               0x0000000104fbd124 __45-[RNIapIos paymentQueue:updatedTransactions:]_block_invoke + 312',
      '4   libdispatch.dylib                   0x000000018ce01884 <redacted> + 20',
      '5   libdispatch.dylib                   0x000000018ce0e404 <redacted> + 60',
      '6   shine                               0x0000000104fbced0 -[RNIapIos paymentQueue:updatedTransactions:] + 908',
      '7   libdispatch.dylib                   0x000000018ce00304 <redacted> + 32',
      '8   libdispatch.dylib                   0x000000018ce01884 <redacted> + 20',
      '9   libdispatch.dylib                   0x000000018ce0d73c _dispatch_main_queue_callback_4CF + 1012',
      '10  CoreFoundation                      0x000000018d382024 <redacted> + 16',
      '11  CoreFoundation                      0x000000018d37ccd4 <redacted> + 1888',
      '12  CoreFoundation                      0x000000018d37c254 CFRunLoopRunSpecific + 452',
      '13  GraphicsServices                    0x000000018f5bbd8c GSEventRunModal + 108',
      '14  UIKitCore                           0x00000001ba6c44c0 UIApplicationMain + 216',
      '15  shine                               0x0000000104cda748 main + 124',
      '16  libdyld.dylib                       0x000000018ce38fd8 <redacted> + 4'
    ],
    domain: 'SKErrorDomain',
    userInfo: {
      NSLocalizedDescription: 'Cannot connect to iTunes Store'
    }

Tested environment (Emulator? Real Device?)
On a real device using a Sandbox account
Steps to reproduce the behavior

Set up a subscription offer on App Store Connect
Make a signature request to your server and get a returned offer containing the required signature string, nonce, timestamp, and keyIdentifier
Create a subscription offer complete with signature, and call RNIap.buyProductWithOfferIO with productId, uid, and offer

@cl2205 I've not had test this feature throughly done in #455. @sck-v Could you assist this?

Thanks @hyochan. I'm able to consistently connect to the iTunes store and make a regular purchase transaction (via buySubscription()) using the sandbox account -- just not with buyProductWithOffer()

@cl2205 That's bit strange since the error message is NSLocalizedDescription: 'Cannot connect to iTunes Store'. Could you kindly provide your codes here?

@hyochan here you go:
// Purchase with Subscription Offer

const offer = {
    identifier: 'TESTOFFER', // from iTunes products discounts array
    keyIdentifier: 'MYK3Y1D' // from server,
    nonce: <nonce>, // from server
    signature: <base64-encoded-string>, // from server
    timestamp:<PHONE_NUMBER>029 // from server
    };

  const sku = 'com.myteam.myapp.iap.monthly';
  const uid = ""; // this is an option string value for `applicationUsername` that can be an empty str as noted in the server signature docs: https://developer.apple.com/documentation/storekit/in-app_purchase/generating_a_signature_for_subscription_offers?language=objc

  const products = await loadProducts();   // this is successful
  const response = await RNIap.buyProductWithOfferIOS(sku, uid, offer); // StoreKit shows discount is applied but when confirming the purchase, transaction fails w/ above error

---------
// Regular Purchase

const sku = 'com.myteam.myapp.iap.monthly';
const response = await RNIap.buySubscription(sku);  // this is successful

@cl2205 Have you solved the issue? I've looked through here and seems below will help.

@hyochan thanks for sharing that thread!
I do seem to have resolved the issue of getting that particular alert of Unable to Purchase - Contact the developer for more information. and it does seem to have been related to the applicationUsername length. Thank you much for your help on that.
Unfortunately another issue I'm running into now is that the transaction still fails. Now I'm getting the system success modal of 'You're all set. The purchase was successful' even though the transaction fails with an SKError code of 0 -- basically the issue described in this thread except it's happening even with an expired subscription for me: https://forums.developer.apple.com/thread/118286

@hyochan thanks for sharing that thread!
I do seem to have resolved the issue of getting that particular alert of Unable to Purchase - Contact the developer for more information. and it does seem to have been related to the applicationUsername length. Thank you much for your help on that.
Unfortunately another issue I'm running into now is that the transaction still fails. Now I'm getting the system success modal of 'You're all set. The purchase was successful' even though the transaction fails with an SKError code of 0 -- basically the issue described in this thread except it's happening even with an expired subscription for me: https://forums.developer.apple.com/thread/118286

How did you fix Unable to Purchase - Contact the developer for more information?
Please help me. https://github.com/dooboolab-community/react-native-iap/discussions/2629


Make admin bug

Make admin checkbox must be hidden for current user.

Merged in pull request (#125)


Need suggestion on writing in ramdajs syntax

I just learned ramdajs functional programming library and trying to convert the checkbox example into ramdajs syntax, like so:
import R from 'ramda'
import Cycle from '@cycle/core';
import {makeDOMDriver, div, input, p} from '@cycle/dom';

function main(sources) {
  let  _output = function(toggled) {
    return div([
          input({type: 'checkbox'}), 'Toggle me',
          p(toggled ? 'ON' : 'OFF')
        ])
  }
  let exp =
    R.compose(
      R.map(_output),
      R.curry(startWith(false)),
      R.map(R.prop('target.checked'))
    )

  const sinks = {
    DOM: exp(sources.DOM.select('input').events('click'))
  }
  return sinks;
}

But I was given the following error:
ReferenceError: Can't find variable: startWith

Is there any suggestion on this problem? Or doing so is not encouraged? I was loving ramda's approach to be more 'functional' way ;-)

Because it seems to be not defined in your scope :) If you wish to call method of Observable that way, look at invoker function in Ramda. But I'm not sure if it does make sense. In case of more common operators (like filter, map, reduce) look at transduce method of Observable. It's the thing thanks to which you can use functions from Ramda (and not only) with Rx.

Because this isn't really about Cycle Core, I'll have to close this issue, sorry


Is it possible for IDesignTimeDbContextFactory CreateDbContext to return generic DbContext?

I am using DI library in my project.
Is it possible to create IDesignTimeDbContextFactory<DbContext> implementation with CreateDbContext method returning generic DbContext like this?
public DbContext CreateDbContext(string[] args)
{
// Insert code returning MyDbContext here...
}

My class deriving from DbContext doesn't contain parameterless constructor. I am injecting connection string using Dependency Injection.
After executing CLI method:
dotnet ef migrations add NewMigration --startup-project ..\MyProject --project ../AnotherProject --context MyDbContext  --verbose

I get such messages (I omitted those not needed):
Found IDesignTimeDbContextFactory implementation 'DesignTimeDbContextFactory'.
Found DbContext 'DbContext'.
(...)
Microsoft.EntityFrameworkCore.Design.OperationException: Unable to create an object of type 'MyDbContext'. For the different patterns supported at design time, see https://go.microsoft.com/fwlink/?linkid=851728 ---> System.MissingMethodException: No parameterless constructor defined for this object.`

Is this a EF Core limitation that IDesignTimeDbContextFactory<DbContext> CreateDbContext should create concrete DbContext implementation?

@zygmuntix It's never possible to create an instance of a non-concrete type; that's what concrete means. So I'm not really following your question.

@ajcvickers In my question I meant: is it possible to create an instance of DbContext? In samples I saw only CreateDbContext method returning instances of class deriving from DbContext.
I found out that I can use --context DbContext option for dotnet ef migrations add command to use DbContext, not class deriving from DbContext.


Set status of CIP3

Should be implemented now

Why am i getting every notification on this - and yet it says I'm ignoring and not listening - and unsubscribe link in not valid for my account?

Can someone tell me how to unsubscribe – it’s not working.
From: Nam Chu Hoai<EMAIL_ADDRESS>Sent: Thursday, May 7, 2020 3:05 PM
To: celo-org/CIPs<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [celo-org/CIPs] Set status of CIP3 (#10)
Merged #10https://github.com/celo-org/CIPs/pull/10 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/celo-org/CIPs/pull/10#event-3314825854, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANEDKGSLVV2RVHHFYOHQXATRQMV7LANCNFSM4M3WALTA.


[App]: lots of errors in the app

Current Behavior
Over the past few hours I've gotten several errors. Sometimes individual panels not rendering (after having rendered for a long time, so the config is fine) sometimes the Sentry "Crash report" window, I get that a few times a day.

In the dev tools console, a quartet of GraphQL errors occurred with the latest "It looks like we're having issues." crash report .
GraphQL error: An internal error occurred. Please contact support.
GraphQL error: An internal error occurred. Please contact support.
GraphQL error: Error 1040 (08004): Too many connections
GraphQL error: An internal error occurred. Please contact support.

Sometimes if flicks up a 404 message (replacing the whole page) when I haven't even touched anything.
And today a new one about it not being able to display my custom panel (after having rendered for a long time, and refreshing fixed it, so the error is incorrect in only giving me one option: to lose my custom charts).
There was a problem rendering these panels.
Click this button to reset your workspace to the default layout.
This preserves runs, but custom charts will be lost.

And in the CLI network errors a few times a day (I have a rock solid network, usually coincides with some error in the web app so I assume WandB servers failing somewhere).
E.g. An internal error occurred. Please contact support

Expected Behavior
No response
Steps To Reproduce
No response
Screenshots
No response
Environment
OS: Windows 11
Browsers: Chrome
Version: 0.14
Additional Context
No response

+1, I also faced this error many times.

Now it is fixed. No problem any more. Thanks.

Now there still exists this problem for me. It happened 2 times  today, and each  one lasts about 10 minutes.

It's no longer happening for me, but I'll leave the issue open since @ZedRover appears to be having issues still.

@ZedRover
Could you provide me with the following so I can debug your issue:

a link to your workspace
the browser console logs for when the An internal error occurs pops up

I get this bug extremely frequently since a couple of days. About once every 1-2 minutes. It's very annoying. Any ideas?

Just to chip in, I have also encountered this issue recently in the past few days and it's still persisting beyond the big outage last week.

We are closing this issue due to inactivity. If you feel that it merits additional review, please provide a reproduction and add a comment explaining why. Thank you.


SIGINT while waiting for auth to complete results in local echo still turned off.

Needed to reset terminal to get back to seeing what I've been typing.

I'm not able to reproduce this on my OS X environment.  Tried the following:
$ safe set secret/x password
password [hidden]: ^C
$ ls

(Ctrl-C for a SIGINT at the password prompt)
I also tried via kill -INT $PID from another terminal.
In both cases, my terminal gets reset properly.  Can you provide some more repro steps?

I was also unable to reproduce this against our (slow) vsphere deployment during the auth phase.

Closing due to inability to reproduce.


Allow .spec.name to be optional in validation

As of the latest 1.0 specification, .spec.name is allowed to be optional.  To allow for this, remove the validation hooks that require this field to be set.
Signed-off-by: Andy Sadler<EMAIL_ADDRESS>

You're correct, but the current code is also currently conformant to the spec.
We have a defaulter that sets .spec.name from .metadata.name if not already set
https://github.com/servicebinding/service-binding-controller/blob/7208239ed3da4f854adeb8c201f17e4bbf59692e/apis/v1beta1/clusterworkloadresourcemapping_webhook.go#L37-L42
This defaulter is called before the validation is checked
https://github.com/servicebinding/service-binding-controller/blob/7208239ed3da4f854adeb8c201f17e4bbf59692e/apis/v1beta1/clusterworkloadresourcemapping_webhook.go#L79-L90
The default exists so that we can always assume the .spec.name field is set and never need to do the if/else within the controller or projector. It's correct for the validation test to assert that the field is indeed set, but that assertion is only valid because of the defaulter. That specific test case is testing the validation after the default is applied, which is why is why it looks as though the field is required (because it is, but also isn't).
Note: we don't install a mutation webhook, so the defaulted value is not visible to users, only inside the controller. This means that any time we interact with the resource from the api server, we need to apply the defaults (this is a best practice anyway to handle drift in persisted vs current defaults).

We call Default() on ClusterWorkloadResourceMapping resources, yes, but not in ServiceBinding resources.  Instead of taking out the validation hooks for .spec.name, would you rather I call Default() before running validation instead?  IMO it's little strange to me that we validate the existence of a field that the spec says is optional.

We call Default() on ClusterWorkloadResourceMapping resources, yes, but not in ServiceBinding resources.

Oh, I didn't even notice I was looking at the wrong resource 😊

Instead of taking out the validation hooks for .spec.name, would you rather I call Default() before running validation instead?

Yes, we should be calling default before using the ServiceBinding resource. This insures the latest defaults are applied before our business logic interacts with the resource and assumes the defaulted values are applied. When upgrading the controller, there may be persisted resources that used the defaulter at the time that need to have the latest defaults applied. It's a bit academic since we don't actively use a mutating webhook, but it's a good practice.

IMO it's little strange to me that we validate the existence of a field that the spec says is optional.

That's fair. I lean towards yes since we're validating the resource as we consume it. If at some point in the future a regression is introduced that changes the defaulting behavior, it will be easier to catch with this test. Otherwise we'll be waiting for a bit of code further down the line try to use a value that isn't set (hopefully that fails loudly, but empty strings can be sly in some contexts).

This is an area where having conformance tests that can assert the end-user facing behavior will be very helpful.

I've updated this PR to call Default() instead.

This is an area where having conformance tests that can assert the end-user facing behavior will be very helpful.

Agreed.


IsOptional() not work

export class EditDto {
@ApiProperty({ description: 'Имя услуги', example: 'Имя', required: false })
@IsString()
@IsOptional()
name: string;
@ApiProperty({ description: 'Описание услуги', example: 'Описание', required: false })
@IsString()
@IsOptional()
description: string;
@ApiProperty({
type: 'string',
format: 'binary',
description: 'Картинка услуги',
name: 'image',
required: false
})
@IsFile()
@MaxFileSize(1e7, { message: 'File size should not exceed 10MB' })
@HasMimeType(['image/jpeg', 'image/png'], { message: 'File should be a JPEG or PNG image' })
@IsOptional()
image?: Express.Multer.File;
}
return from server:
{
"statusCode": 400,
"message": "Validation error",
"errors": [
{
"field": "image",
"error": "File should be a JPEG or PNG image, File size should not exceed 10MB, Field "image" does not contain file"
}
]
}
I want to create "UpdateDTO" and don't know how to do this "IsFile" optional

@summivar
Hi, has a separate test on for this and it passes successfully, you need to check if you are doing everything accurately according to the documentation.
Test source
Test source DTO
You should also consider that you are using the wrong file type in your dto, class-validator/class-transformer uses this too.
The uploaded file is never Express.Multer.File
It is an instance of one of the [storage](It is an instance of one of the repositories you use when connecting the module. ) you use when connecting the module.
If you don't find the problem, please create a repository to reproduce the problem and it will be fixed.

@dmitriy-nz i have the same issue with the latest version. My code:

  /**
   * appending files
   */
  @Expose()
  @Transform(({ value }) => {
    return value !== undefined && !Array.isArray(value) ? [value] : value;
  })
  @IsFiles()
  @HasMimeType(['image/jpg', 'image/png', 'application/json', 'text/plain', 'application/xml'])
  @IsOptional()
  appending?: FileHashStorage[];

Event if it's optional the mimetypes are validated as invalid. I think it only happens if you have an array of files.

I have the same issue.
`import {
IsInt,
IsNotEmpty,
IsNumber,
IsOptional,
IsString,
Min,
} from 'class-validator';
import {
FileSystemStoredFile,
HasMimeType,
IsFile,
IsFiles,
MaxFileSize,
} from 'nestjs-form-data';
import { Types } from 'mongoose';
import { Transform, TransformFnParams, Type } from 'class-transformer';
const transformDateFn = (params: TransformFnParams) =>
params.obj[params.key] === '' || params.obj[params.key] === 'null'
? null
: params.value;
export class CreateAnnonceDto {
@Transform(transformDateFn)
@IsNotEmpty()
@IsString()
titre: string;
@Transform(transformDateFn)
@IsString()
description: string;
@IsFile()
@MaxFileSize(3e6)
@HasMimeType(['image/jpeg', 'image/png'])
primary_image: FileSystemStoredFile;
@IsOptional()
@IsFiles()
@MaxFileSize(4e6, { each: true })
@HasMimeType(['image/jpeg', 'image/png'], { each: true })
images: FileSystemStoredFile[];
}
`
IsOptional doesn't work on an array of files. The user is forced to upload at least one file.
Does someone has a solution please ?

@summivar @julianpoemp @TidianeRolland  Hi! Issue has been reproduced in fixed in the v1.9.9 version, please try it out

@dmitriy-nz still not working for me with v1.9.9

  @Expose()
  @IsOptional()
  @IsFiles()
  appending?: FileHashStorage[];

appending should be optional, but I still get an error if no files appended.

@julianpoemp can you share your ValidationPipe options?

@dmitriy-nz
{
        transform: true,
        whitelist: true,
        enableDebugMessages: true,
        forbidNonWhitelisted: true,
        disableErrorMessages: false,
        skipNullProperties: true,
        skipUndefinedProperties: true,
        exceptionFactory: (validationErrors: ValidationError[] = []) => {
          Logger.error(JSON.stringify(validationErrors, null, 2));
          return new BadRequestException(validationErrors);
        },
      }

@julianpoemp if you are like me and testing the apis on postman make sure you uncheck the field in your case  you need to uncheck appending.

Thank everyone for the feedback, your case has been reproduced, the issue was that when a property is decorated with Exposed() we are forcing get an undefined value when no value is provided.
I have deployed a new version 1.9.91, please try it out
@julianpoemp

@dmitriy-nz it's working, thank you! 🎉


Add 0.15.6[a]

@Unisay Good catch.  We forgot to bump the minor version for the release of 0.15.6.
The versions table only lists the most recent patch releases.  Would you like to adjust the PR to drop 0.15.5 and 0.15.5a ?  If you'd rather not mess with it, I'm happy to adjust and merge.

@rossabaker Related question for you... versions.md exists in release-0.15.x but is out-of-date...which might be my fault.  Should we bump versions in release-0.15.x and put on the train?  Or perhaps remove/annotate versions.md in release-0.15.x because it's not published?

@Unisay Getting ready to do some merges, so @ChristopherDavenport cherry-picked into the right place in #1016 .  Thanks again for catching the mistake!


Update the OUSD marketing site to link to the IPFS version

Please point the IPFS link to: https://ousd.eth.limo/

After talking with @jonathansnow, I think we should hold off on this until we better understand the consequences for analytics. We wouldn't want to break the conversion tracking between the marketing site and the dapp since they involve different domains.

@micahalcorn should we do this?

OUSD: https://ousd.eth.limo/
OETH: https://origineth.eth.limo/

Noting for future reference.
Google Analytics - Updates when moving to IPFS for DAPPs:

Add IPFS domain for cross-domain tracking
Update GTM lookup table to map the permanent IPFS hostname to the production ID
Publish GTM changes
Test that gl parameters are added to the url when clicking from the marketing site to the dapp


Could not find resource

when i get version 0.5.0, throw excepiton,
Could not find com.apollographql.apollo:gradle-plugin:0.5.0.

you have wrong string com.apollographql.apollo:apollo-gradle-plugin``` notice that it should be apollo-gradle-plugin` after the colon


run cronjob as non-root user

Towards: giantswarm/giantswarm/issues/6013

etcd hostpath dir is owned by root on masters, so nothing can be changed without changing etcd containers to run as non-root and also adjusting permissions accordingly.


dirPagination: page navigation problem

I am using dirPagination in client side as well as server side(Asynchronous Setup). The issue is client side pagination is not navigate another page.
Steps to re produce the issue is,
1.First open the server side pagination page,
2.Then open the client side pagination page.
3. Now click the next page(Page Number 2), data is not changing(Showing first page data), But pagination controls was changed.
if i opened the client side pagination page first means, it working fine.
Page navigation need to work while i am switch from server side pagination page to client side pagination page.

Hi,
I'm afraid I can't quite figure out what you mean. Please could you provide some code?

Following code only i am using.
For Client Side pagination:
`

0"  dir-paginate="result in resultData|filter:search|orderBy:sortKey:reverse|itemsPerPage:noOfRows">
							{{result.firstName}}
							{{result.lastName}}

							{{result.phoneNumber}}
							{{result.emailAddress}}

			No Records Found

				<dir-pagination-controls max-size="5" direction-links="true" boundary-links="true" >
					</dir-pagination-controls>`

For Server Side Pagination:
`

				{{result.name}}

				{{result.phoneNumber}}
				{{result.age}}

						No Records Found!!!

	<dir-pagination-controls ng-show="resultData.length>0" max-size="5" on-page-change="searchFromPage(newPageNumber)"  direction-links="true" boundary-links="true" >
					</dir-pagination-controls>`

Actually the issue is page navigation is not working in client side pagination, if i open server side pagination page first. The page was highlighted as 2nd but data for second page is not loading, only showing first page data.

While using Server Side Pagination:
The instance object returned like this: (displayed in console)
m {$$childTail: m, $$childHead: m, $$nextSibling: null, $$watchers: Array[11], $$listeners: Object…}
$$nextSibling: null
While using Client Side Pagination:
The instance object returned like this: (displayed in console):
m {$$childTail: m, $$childHead: m, $$nextSibling: m, $$watchers: Array[11], $$listeners: Object…}
$$nextSibling: m
I think here is the issue. With out visiting server side pagination, Client side pagination returning $$nextSibling value is m. Once visited the Server side pagination $$nextSibling value is changed to null. Because of this client side pagination navigation is working but data is not changing. Showing first page data only. Even clicking any page.
Please guide me, if i did wrong.
Thanks

Show your controller

I still don't really understand, sorry. Do you mean you are trying to do both server-side and client-side pagination at the same time?
Please describe the use case in detail and, as mentioned above, show your controller code.

Thanks for your response.
Ya i am using both Server-Side and Client-Side pagination. But Server-Side pagination in different page, Client-Side pagination in different page.
Finally, i solved this issue by using pagination id. I am giving different pagination Id for both pagination controls,

Hi, okay. In future, could you please close the issue and state how you resolved it once it is solved? That is helpful for the maintainer (me) so that I don't need to spend time re-reading the thread in the future.


Added variable verbiage

This statement does not effect local or table variables.

@way0utwest Thanks for the suggested improvement. Do you have some sample T-SQL that shows the point you're making here? It looks like maybe something you've tested recently? That would help me confirm and approve. Thanks! -Jason

This should help
DECLARE @founders VARCHAR(50) = 'way0utwest';
DROP TABLE IF EXISTS MyTable;
CREATE TABLE MyTable
(founder VARCHAR(50));
INSERT dbo.MyTable (founder) VALUES ('way0utwest');
BEGIN TRAN;
SET @founders = 'rothja';
UPDATE dbo.MyTable SET founder = 'rothja';
ROLLBACK;
SELECT @founders AS f;
SELECT * FROM dbo.MyTable AS mt;

The change inside the transaction and rollback do not apply as they would for live data.

@way0utwest Thanks! I'll verify on my side with your sample and approve the PR. Appreciate your time.

#sign-off


unlock names per ip

i want block names like [admin] - [mod] ,Except for some ip, so that only admins can have them

@PsycoYT insert this code into the setNickname function in PacketHandler.js:

var UserRoleEnum = require("./enum/UserRoleEnum");
if (this.socket.playerTracker.userRole !== UserRoleEnum.ADMIN && this.gameServer.checkBadWord(name))  {
    skin = null;
    name = "Hi there!";
}

Then add [admin] to the list of bad words. What the code above does is allow people with the user role of "admin" to bypass the name filter, therefore you are the only one who can use [admin]. If you want the same thing to apply for moderators, change UserRoleEnum.ADMIN to UserRoleEnum.MODER.
Or if you want both admins and moders to bypass the filter you can do this:

var UserRoleEnum = require("./enum/UserRoleEnum");
if (this.socket.playerTracker.userRole !== UserRoleEnum.ADMIN &&
    this.socket.playerTracker.userRole !== UserRoleEnum.MODER &&
    this.gameServer.checkBadWord(name))  {
    skin = null;
    name = "Hi there!";
}

I haven't tested this yet, but theres no reason why it shouldn't work.

that code isnt work


updating the branding guidelines withe new content and uploadable images

@cnunciato updating content for the Brand Guidelines

:+1: to merge this as it is; we can come back 'round to tidy up that markup later.


Append date to the date picker

Hi
Is there any props or method way to append the date to date picker.
Any help appreciated.

Hi @shekharsumanpandey-tudip! What do you mean by appending the date to date picker? I would like to help you if you clarify a bit more your requirement.

when I am writing [(ngModel)]="someModel" in  component tags it crashes, someModel type is {start: Date, end: Date}.
I think that is what he meant


Automatically generate a documentation

Maybe that could be a good thing, to generate a documentation automatically. What do you think about this? Maybe some kind of basic sphinx documentation?
Because I plan to do a documentation for my plugin, but thought maybe as I'm working on it I may also contribute to a more general approach for the cookiecutter template...

that action appears to publish docs to the gh-pages branch in a given repo.  which wouldn't help here?
Just to clarify, are you discussing making it so that "ready to go documentation" is part of the plugin after you create it?  or are you talking about docs for napari-cookie-cutter itself?

yeah, the maintenance burden I'm referring to is that if we a "prebaked" docs folder in this template (i.e. if we run sphinx-quickstart once here for people). then those files that were autogenerated by sphinx will, themselves, go out of date eventually.   And we would essentially have to redo that process (or somehow automate it) periodically to keep it up to date.  The github action you're pointing to would simply build already complete documentation into an html site and upload it to your repo (which isn't what we want here... we don't want to actually create a gh-pages branch here in this repo).
I know it's tempting to make these cookiecutters do "all the things" one could ever want out of a repo :) but in this case, given the complexities of actually doing documentation well, I think both we (the maintainers) and the end-users are better off going directly to the source here (i.e. go to sphinx or mkdocs or jupyter-book and follow their up-to-date tutorials).


make DismissItem call ConsumeMessage properly

Previously we were calling the wrong end of the API. BroadcastMessage is
what is supposed to get called when the server informs us of a
message.
r? @maxtaco @mmaxim
This is not yet working. The client RPC call seems to hang forever, and I can't see anything at all in the gregord logs. Probably something thinks it's not connected yet. I will debug with you guys tomorrow.

Well shoot.  This looks like it would do the trick.  I would turn on RPC debugging.

Yeah this looks like a bug with g.cli. I'm not sure why though, we can probably figure out this quickly tomorrow morning.

@oconnor663 did you try adding debugging into the connection.go library?

We also need a test for this, since clearly the tests weren't good enough to catch this bug.

Yeah I realize this would be the first use of g.cli in gregor.go, all other calls into gregord use the client from the RPC library in OnConnect. This is basically the same code gregord uses for authd, so I'm surprised there is a problem. Agreed about the test though.

Actually how are you testing this @oconnor663, is it getting stuck after syncing from gregord and replaying a message?

The point is if this function gets called from a code path originating in OnConnect, it won't work.

@mmaxim seems like the intention is to call this when the UI closes the window.

See go/engine/track_token

Ok Mike helped me figure this out. I think the problem was that my testing was running in OnConnect, when the g.cli object was in a bad state. I've filed https://keybase.atlassian.net/browse/CORE-3136 to think about that more.
Once I tested it right, I found a double encoding bug. That's fixed now.

:mag_right: No visual changes found as a result of these commits.

:mag_right: No visual changes found as a result of these commits.

One more problem I'm seeing in the logs: we're broadcasting the dismissal locally twice. Everything handles that ok, but I'm not sure why it happens.

@mmaxim and I decided that the double dismiss was a gregord problem where it's accidentally rebroadcasting messages to itself. This PR should be good to go.

👍

landed with 7cfb44a382a4be94d0326d54761dcfaad018510f


Compress .css

The github-styled .css-file should be converted to a .min.css-file on every push, and the converter should use said minimized file instead of the unminimized one.

This is done now and it works fine. The file is now compressed to 87% of its original size.


External Reset signals Causing RTL generation to fail

From open-register-design-tool created by neenuprince: Juniper/open-register-design-tool#78
Hi, Recently we noticed that the external reset registers, ( registers that gets reset using an external reset signal) that has resetsignal attribute causes an Error and hence cannot generate RTL.
We had generated the RTL with these resetsignal attribute earlier, but even older releases seems to fail. the error we get is as below.
*** ERROR ***: Unable to resolve rhs assignment reference test_rst_sig
Below I have mentioned the Reg defns and addr map.
Register Definition
signal test_rst_sig {
   async;
} test_rst_sig;

regfile other_regs_defns_rf {
reg attr_resetsignal_t {
name = "Reg resetsignal";
desc = "A register with resetsignal to verify resetsignal works correctly";
default hw = r;                //Register default hw field type to read/write
default sw = rw;                //Register default sw field type to read/write
field {
name = "rstsignal check";
desc = "Verifying that rstsignal works";
resetsignal = test_rst_sig;   //signal used to reset field's register and hardware logic
fieldwidth = 32;
reset = 32'h0;
} fieldrstextsig;
};
attr_resetsignal_t    attr_rstsignal
};
Top RDL address map
addrmap ordtreg_regs {
test_rst_sig    test_rst_sig;
other_regs_defns_rf         other_rf            @ 0x000100;
other_rf.attr_rstsignal.fieldrstextsig->resetsignal = test_rst_sig;
};

All versions of ordt until today do not support access of signals defined in parent hierarchy levels directly in the child, eg resetsignal = test_rst_sig; within fieldrstextsig throws an unable to resolve error.  Using a dynamic assign in the parent, eg other_rf.attr_rstsignal.fieldrstextsig->resetsignal = test_rst_sig; does work correctly, however.  As of 191222.01, both forms are supported.
See example in rdl_examples/issue_18.rdl

Hi ,
Pulling from this fork gives me a .gitignore merge conflict, since I clone from the master first.
Was that intentional , that these are not updated in to the master?

Also one more clarification, can just one of the fields of a multifield register be reset by external reset? is that supported?
Getting the below error whn I try to do that.
VIO#1> [CHK3/error] Missing definition: One or more field in a register has the resetsignal definition and the other fields do not.

Just another request for enhancement.
I noticed that currently , if a register has multiple fields, each of them has to be explicitly addressed with the resetsignal in the address map. is it possible that we can have
other_rf.attr_packedfields->resetsignal = test_rst_sig;
instead of
other_rf.attr_packedfields.fieldrstextsig->resetsignal = test_rst_sig;
other_rf.attr_packedfields.fieldsticky->resetsignal = test_rst_sig;
other_rf.attr_packedfields.fieldstickybit->resetsignal = test_rst_sig;

So I have ben able to generate the RTL now, I don't see the error any more.

Responses to your questions....

assume that the merge conflict seen was relative to Juniper fork master?  This wasn't intentional, but some files went out of sync relative to sdnellen when I left.  Would just clone from sdnellen/master going forward - this is where enhancements/fixes will be done (nothing critical in Juniper fork is lost by moving)

Yes,  just one of the fields of a multifield register can be reset by external reset.  The resetsignal property is applied at field level, so this is allowed per the standard.  Not sure where the "VIO#1> [CHK3/error] Missing definition..." error is coming from, but this isn't an ordt error message.

It is possible to apply a single reset signal to all fields in a register in a single statement.  There are 2 ways to do this - the first below uses ordt field wildcard form, which is not part of the rdl standard....
 other_rf.attr_packedfields.*->resetsignal = test_rst_sig;

The other, which is part of the standard, is to add a default property assign statement in the enclosing register (or regfile) definition, ie...
    default resetsignal = test_rst_sig;


reference fields in fhir-examples generated resources should include resource type

Describe the bug
Many reference fields in the fhir-examples generated resources use short random string values like IRm0J8cmzs. Such values do not conform to any reference format defined in the FHIR specification.
To Reproduce
Build the examples package and inspect the generated resources:

mvn clean install -f fhir-examples-generator
mvn com.ibm.fhir:fhir-examples-generator:generate-examples -f fhir-examples
view fhir-examples/src/main/resources/json/ibm/complete-mock/Patient-1.json

Expected behavior
Reference field values should be a URL or of the form <resourceType>/<logicalId>

Implementation tip. In ModelSupport:
    /**
     * @param modelClass
     *            a model class which represents a FHIR resource or element
     * @param elementName
     *            the name of the reference element
     * @return a set of Strings which represent the the allowed target types for the reference
     */
    public static Set<String> getReferenceTargetTypes(Class<?> modelClass, String elementName) {
        ElementInfo elementInfo = getElementInfo(modelClass, elementName);
        if (elementInfo != null) {
            return elementInfo.getReferenceTypes();
        }
        return Collections.emptySet();
    }

Would love to help out and give it a go!

Would love to help out and give it a go!

Fantastic.  We're happy to coach, and we are generally available vai Zulip on #ibm  https://chat.fhir.org/#narrow/stream/212434-ibm

Thank you. I think to reproduce it should be in the file fhir-examples-generator folder. right?

mvn clean install -f fhir-examples-generator
mvn com.ibm.fhir:fhir-examples-generator:generate-examples -f fhir-examples-generator/pom.xml
view fhir-examples/src/main/resources/json/ibm/complete-mock/Patient-1.json

you may need to build the whole project first since the fhir-examples-generator is currently configured to depend on the 4.4.1-SNAPSHOT version of our other modules.

mvn clean install -f fhir-examples
mvn clean install -f fhir-parent -DskipTests

Unfortunately, no - the examples generator builds JSON files which are destined for fhir-examples not fhir-examples-generator
And yes, @lmsurpre is correct (dependencies must be built first)

Thanks, @prb112 and @lmsurpre
Just adding a few things I found:
I see this in the ExamplesPlugin (and hence my previous comment):
        // Only runs for the fhir-examples-generator project since that is where the definitions directory exists, short-circuits otherwise.
        if (mavenProject.getArtifactId().contains("fhir-examples-generator")) {

It works for me when I use -f fhir-examples-generator (only), and most references to FHIR resources are in the correct format (Patient/XXXX, Encounter/XXXX, Organization/XXXX etc.) in the generated files, but some like Patient.generalPractitioner are not.
This may be related to how they are defined in the definitions/profiles-resources.json. I noticed that all resources not properly referenced have a '-' in their definitions:
Patient.general-practitioner
Claim.care-team

Thanks @dermatologist
Sure - it may be setup like that, and then there would be an extra step to ensure the contents are end up in the fhir-examples project.
Your observations sounds correct with regards to the generated content.  I think Lee is best to answer the detail of the question.
Paul

Moved to Sprint 20

I noticed that all resources not properly referenced have a '-' in their definitions

I don't think thats it.  For example, see Goal.addresses and Goal.outcomeReference.
With respect to

Only runs for the fhir-examples-generator project

I think that was a gap in my own understanding of how this should work.
Typically I just run the ExamplesGenerator main and then copy paste the updated examples over.
However, the end goal was to make this work similar to our fhir-tools plugin that generates the model classes directly to fhir-model.
To this end, I've just opened a pull request that should clean this up significantly: https://github.com/IBM/FHIR/pull/1656
If you're able to rebase onto that, you can invoke the examples generator via build/gen-examples.sh (or do something similar manually on windows) and it should properly update the examples in the right spot of the fhir-examples project.

@dermatologist this one turned out to be more involved than we had first thought.
After a deeper investigation, I found two different cases where we weren't setting the reference type properly:

when the reference was a choice element
when the reference was a repeating

I went ahead and pushed some changes to a pull request at https://github.com/IBM/FHIR/pull/1667/commits which should address the root of the issue.
If there's anything we could do to make it easier for potential contributors like yourself, we'd love your feedback on that.  Thanks!


Add giveaway command/event

:jit:

Works with discord.js modals now.
Moved interactionCreate from index.js to the EventHandler.
Also created a functions folder

Don't get the reactions lost when a longer giveaway is occuring and the bot restarts.
I would suggest implementing our database.

How would a database help this tho? The reactions are still there, it's just that the "countdown" has to be reactivated after a restart somehow.

How would a database help this tho? The reactions are still there, it's just that the "countdown" has to be reactivated after a restart somehow.

The reactions are there on the discord but not stored so ultimately after a restart the whole giveaway needs to be posted again.
Thats why I would suggest saving the id of the message createdAt updatedAt and a message can also have multiple reactions where I would store:
userId
Now the bot can check through those message ids if one ends pick a winner from saved userId's

Also as @ckhenzy is saying we probably want to store some information in the database, message id, giveaway deadline and emoji should be enough and then we can retrive all users who reacted when needed.

I would store the gamekey aswell


Cli segmentation

Small changes to CLI for segmentation.  Mostly to sync with Master.
I've successfully tested instance segmentation on existing NNProbabilities.npy files.  I believe @smguo has tested instance segmentation on probability files generated from microDL.
The segmentation validation does not appear to parse cell_positions.pkl properly.  But I also wonder how much we care for that flag to work -- it's simply to plot some segmentations.
Some feedback on the above points would be appreciated @smguo @miaecle .  If it's ok, then we can just merge and consider the "segmentation" portion of our pipeline complete.

Sorry for the delay in getting to this @bryantChhun. It looks like this PR conflicts with the master after @miaecle merged his PR. Also commits in  "files changed"  got messed up. Can you merge the master to this branch? I'll take another look after that. Thanks!

@smguo Thanks.  I merged master into this branch.  It only needed changes to the file in instance_clustering.py


Seek method should iterate by bytes rather than node tree count

This seems like it could be resolved as part of #5


These additional dependencies are required for 'vagrant up' to complete.

These additional dependencies are required for 'vagrant up' to complete.

Thank you for submitting this! It's a bit unclear to me what changed between when we last published into this repository and now, but your fix worked after I just retested and saw the failure myself.
Cheers,
Phil

I would surmise that the python packages which had these dependencies did not specify them explicitly, but still worked initially because some other package(s) specified them.  However, subsequent software updates modified those other package(s) to no longer have the dependencies, and that broke the python packages.
One way to guard against this would be to disable automatic software updates in the guest, and to instead periodically update the configuration for the vagrant guest in the repo to point to a recently-updated image.  The image update could be done by a scheduled script, and an automatic unit test could be used to test the image before it is committed to the repo.


Multitouch on Mac OS X not capable values

Using openfl.ui.Multitouch
trace(Multitouch.inputMode); //             NONE
trace(Multitouch.mapTouchToMouse); //     Class<openfl._v2.ui.Multitouch> has no field mapTouchToMouse
trace(Multitouch.maxTouchPoints); //        2
trace(Multitouch.supportedGestures); //     null
trace(Multitouch.supportsGestureEvents);//  false
trace(Multitouch.supportsTouchEvents);//    false

This is not the correct values. Mac OS X supports gestures at least two-finger.

Multitouch.mapTouchToMouse is an AIR-only API, at the moment, we're focused on Flash APIs, instead of AIR APIs, I expect that additional functionality will primarily be exposed through Lime
Currently OpenFL does not support gestures, and only provides mouse or touch events. Even though the OS supports gestures, I'm not aware of any gesture event support in SDL. Someone could implement a compatibility library to automatically detect gestures, but at the moment, the above looks correct to me


can't use wildcards (such as *) when searching in the git source code management panel in vscode

When the "Source Control" view is focused , then press F3 to bring up the list find widget, but I can't use wildcards (such as *) for searching.

@benibenj, are wildcards supported in tree search?

Are they needed at all as there's fuzzy search?

Are they needed at all as there's fuzzy search?

The fuzzy search function is also unsatisfactory.
For example, I only want to focus on the changed .c files, but using fuzzy search will bring up many files that I am not interested in.

There is no plans to support wildcards (glob patterns) as this essentially is what fuzzy search is for.


Fixed the null byte test for Zend_Db_Adapter_Pdo

This PR fixes the test issue with null byte for Zend_Db_Adapter_Pdo.

@froschdesign, @akrabat this PR solves the unit test issue for null bytes. Travis CI is failing only on PHP 5.4 for a segmentation fault, not related with the issue. /cc @weierophinney

I'm re-running the 5.4 tests just to see if the segfault was transient or not. :) Otherwise, the change looks reasonable, and will merge shortly.

Same issue. From some googling, it appears to be an issue with xdebug + code coverage. I'll ignore it for now.


crypto: use purego tag consistently

We've been inconsistent in using the purego tag in the standard library because it was not clear what would be its use aside from letting me benchmark assembly vs native code. I noticed on the Gophers Slack that tagging purego consistently would probably make TinyGo's life easier when trying to compile crypto packages, and presumably any other Go compiler. https://tinygo.org/docs/reference/lang-support/stdlib/#crypto
It's easy enough to do, so why not. Might also add a purego compilation test of crypto/... to longtest.
/cc @dgryski

The hash/crc32 package also needs purego tags.

FYI, the purego proposal is accepted #23172
I also had https://go-review.googlesource.com/c/go/+/468795, which implements a purego version of hash/maphash. There, you can add the test with purego tag in cmd/dist.

To be clear, purego means "use Go instead of assembly"?
I'm asking, because it could also be interpreted as "don't use CGo". In TinyGo we do support CGo (with some limitations) but Go assembly is very difficult due to the special syntax and different calling convention. So basically we'd like to set the purego tag by default to avoid assembly but would still like to use CGo.
Related: https://github.com/tinygo-org/tinygo/pull/4116

To be clear, purego means "use Go instead of assembly"? I'm asking, because it could also be interpreted as "don't use CGo". In TinyGo we do support CGo (with some limitations) but Go assembly is very difficult due to the special syntax and different calling convention. So basically we'd like to set the purego tag by default to avoid assembly but would still like to use CGo.
In #23172 it seems that purego is defined as "no assembly, unsafe, or CGo", which combines a number of unrelated features and would not fit very well with TinyGo.

Indeed, it might be a bit coarse, but I think purego means "no assembly, unsafe, or cgo when there is a pure-Go alternative". There's no point making a build tag for breaking a build. So in practice it mostly means "no assembly" and only occasionally "no unsafe".

Hmm, that's unfortunate. I imagine it could limit features/performance in some cases now that we use purego by default.
For CGo, I'd normally assume the cgo build tag would be used. That's the build tag specifically meant for that purpose as far as I know. While TinyGo doesn't support all CGo features, it's pretty close and missing parts are generally not too difficult to add.
TinyGo does support unsafe just fine though. It has a different memory layout for some types, but portable code would try to avoid relying on a specific memory layout anyway.


Hotkeys for moving windows between virtual desktops

Description of the new feature / enhancement
As a power user I would like to be able to be able to use the keyboard when moving windows from one virtual desktop to another.
Something like the shortcuts Win + Ctrl + Shift + Left | Right and Win + Ctrl + Shift + [Number] would be a very useful workflow when changing virtual desktops a lot. Exactly what the shortcut is isn't as important to me as just having one exist.
The current workflow for doing this is to press Win + Tab, then finding the window, click & drag it with the cursor and drop it over the virtual desktop where you want it placed, or to right-click the window and moving it to the virtual desktop where you want it. Both of these require the mouse to be used.
I know it's possible to get this functionality using AutoHotKey with some custom libraries and scripts, but this is a feature that should definitely be baked into Windows itself, or into PowerToys.
Scenario when this would be used?
As a Power user I would use this when setting up new virtual desktops. When I need a new virtual desktop, it is usually because I have opened too many apps/programs on one virtual desktop, and want to split the workflow into logical pieces (like moving all communications apps like email/Teams/Slack/etc to a separate virtual desktop).
As a power user, this feature would improve the efficiency that I can do the described virtual desktop setup, as well as the efficiency of how quickly I can move a misplaced window from one virtual desktop to another.
Supporting information
The following Superuser Q/A has descriptions for how to set this up with AutoHotKey:
https://superuser.com/questions/1685845/moving-current-window-to-another-desktop-in-windows-11-using-shortcut-keys

I would definitely love to see that in PowerToys.
@TorsteinA have you seen these implemented somewhere or you using AutoHotKey still?

/dup #19

/dup #19

#19 appears to be focused on jumping to a specific virtual desktop, which is different from moving a specific window to a specific virtual desktop. So, not exactly a duplicate.

/dup #19

#19 appears to be focused on jumping to a specific virtual desktop, which is different from moving a specific window to a specific virtual desktop. So, not exactly a duplicate.

Yes, I was thinking the same, and agree this isn't a duplicate.
@crutkas

Sorry, /dup https://github.com/microsoft/PowerToys/issues/16


Add new columns to tagging events

navigation_document_supertype is the a supertype which is used to differentiate guidance and other content
base_path is useful to link out to the tagged page

https://trello.com/c/GRPCKmbU

Will the import script handle these new columns?

Yup, it just uses the CSV column names:
https://github.com/alphagov/content-tagger/blob/master/lib/tasks/import_tagging_events.rake#L3-L5


iOS 10+

There are quite a few issues on here which state that iOS is having trouble.
I've noticed that many of these are being closed abruptly. This isn't particularly helpful for the community. Just because one example you have works doesn't mean that all other cases will - and vice-versa - as we well-know.
Since iOS 10, some restrictions were introduced which limit which content can be exposed to the clipboard api. The content must reside in an <input> or <textarea> OR must be flagged as contenteditable. Perhaps this relates to the iOS10-specific issues?
I was also having issues on my iPhone. Unfortunately, this error stopped all subsequent JS from executing. I found that, rather than taking the script from NPM, running it through Webpack, etc.. if I just took the script used on clipboardjs.com (dist/clipboard.min.js) and manually inserted it in to my bundle (yeah I know..) it started working on iOS. This seems odd as the issue was only affecting iOS and was a reference error (ClipboardJS not found).
Posted most of this as a comment elsewhere but posting as a new issue too in the hope that this helps some people.
Please don't close issues before the OP is satisfied, or at least be a good open-sourcer and give clear pointers rather than "must be your implementation".
Cheers!

I met the same problem

i got the problem on ios12.2

i got the problem on ios12.2

cordova-clipboard is useful for ios12.2

@wsy6566 how to use cordova-clipboard in js , i imported it and the package of 'cordova', but there was an   error that 'fs' package was wrong

ios12.4 also clip fail

ios12.4 also clip fail

ios12.4 also clip fail，and there was no response

Upgrade the operating system version number and the problem is gone.

Closing due to inactivity.


Cannot install package

Greets,
I dont know if its a bug or something, but I try to install from nuget the version 1.2.3 and it says that Xeeny doesnt have a compatible version for .NET Framework 4.5.2 or 4.6.1.
Thank you !

Hi, Are you using packages.config for referencing nuget? if so please upgrade your solution to use packagereference. https://marketplace.visualstudio.com/items?itemName=CloudNimble.NuGetPackageReferenceUpgrader


Multi-relational link prediction in a heterogeneous graph

Hi there!
Amazing work with the SEAL approach! I was wondering how could could apply the scoring to multi-relational link prediction task. Since you mentioned in the paper that SEAL is not restricted to homogeneous graphs, I was wondering how the implementation has to be adapted to calculate the score for different edge types. Could you possibly show how to do that?
Best,
Sophia

Hi Sophia,
Thanks! SEAL has already been applied to recommender systems (where relations are of multilple types corresponding to different ratings), see https://github.com/muhanzhang/IGMC. Basically, you need to change GCN to R-GCN to handle edge types, and change the loss to multi-class classification or regression loss.

Hi @muhanzhang,
Thanks a lot for the response! I will try this out. Would you say that I could rather use the IGMC code instead of changing the SEAL code?  What would be the difference?

@sophiakrix The IGMC code reads recommender system datasets such as Movielens, Flixster, etc (which are bipartite graphs). I would still recommend to adapt this SEAL_OGB repo to your task, because it reads general graphs.

Thanks for the reply!

Thanks for the suggestion!


Consider setting http2 session timeout

I'm not sure what exactly the implication here is, but this is a thing:
https://github.com/digitalbazaar/bedrock-server-load-balancer-config/blob/main/lib/main.js


Wrong Mapping of elements [BUG]  mapping of elements in the footer section (usefullinks) is incorrect.

🙁 Actual behavior
I forked from the main branch and saw this error in my console while make the website responsive.

🙂 Expected behavior
later i tried to find the bug and found this.

also this

Clearly the mapping method is incorrect
in each iteration the same index was being used
Also the json file was not framed properly
🔢 Steps to Reproduce the Problem
I have fixed this issue in my next pull request to the responsive branch😄.
i guess this is the reason why vercel bot denyed the pull request of @Yash-Chauhan09 and others🙁.
Pls fix this issue using my code or anything else its kinda demotivating when u get an error in pull request.

Thanks for raising up this error in the Footer. You can definitely make a footer component responsive in a PR and fix the mapping issue in the other one. ⭐
And for the vercel deployment issue, It's just our team account has expired. You can refer to this comment


how to implement the api in pytorch with gluon

There is two api in pytorch, one is "nn.utils.rnn.pack_padded_sequence", other is "pad_packed_sequence", I want to ask is there any api or implement to achieve the similar function of pytorch in gluon, thank you~~

@szha is there any tutorials about how to use masking to deal with? use a very small value like -np.inf?

@kenjewu got it, Thank you so much for your quick answer ! I also use this method! I just want to find out if there are other better implementations in gluon.

@HuangZhanPeng we are looking to extend the RNN implementation in MXNet so that the variable length support can be added directly. This is tracked in https://github.com/apache/incubator-mxnet/issues/9543
Let me know if you need the issue reopened.

@szha Thank you for your response. There is no need to reopen it.


fix: list all groups a user belong to

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 44.974%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4286

Relevant Lines:
9530

💛  - Coveralls


Switch to Doctrine XML

It seems that the switch from AXMLS to Doctrine XML is very easy.
I just tried it with the core db.xml and almost anything is fine (just a few smallint/timyint differences in the generated schema... I will investigate).
PS: I create a script for the conversion from AXMLS to Doctrine XML.

I like that script, should make it easier and more fun to upgrade our old XML files!

I like that script, should make it easier and more fun to upgrade our old XML files!

Me too :wink: Finally we're quite close to support table relations!

yes, no more inconsistency, great!

PS: I integrated the conversion tool from AXMLS to DoctrineXML into http://concrete5.github.io/doctrine-xml/

...and finally all tests passed!
IMHO is worth that @aembler takes a look at this... This PR seems stable and closes #2025
We should also add https://github.com/concrete5/doctrine-xml to packagist and create the first release of doctrine-xml.

yes, no more inconsistency, great!

Here's an example of what we can do with this PS: https://github.com/mlocati/concrete5-5.7.0/commit/63124c4ccac88b8995945d02bcf9276b1bc2be38

This looks great. Once we get out of 5.7.4 and <IP_ADDRESS> we'll Look at including this

Would you consider making this pull request again against the current develop branch? Getting some db.xml and composer.lock issues that I imagine will be pretty difficult to reconcile otherwise.

Woohoo! Again, really nice work:
https://packagist.org/packages/concrete5/doctrine-xml

Would you consider making this pull request again against the current develop branch?

Sure! #2480


Resolved enrollmentDate missing issue in export data and experiment details state time logs

This resolves #2099

I can confirm the Enrollment Time Log is added when an experiment with the "Enrolling" status is created with POST /api/experiments.

I can confirm the Enrollment Time Log is added when an experiment with the "Enrolling" status is created with POST /api/experiments.

@zackcl This PR need to handle update experiment api. This was pointed out by Vivek. Will push another commit for it.

I can confirm the Enrollment Time Log is added when an experiment with the "Enrolling" status is created with POST /api/experiments.

@zackcl You can review it now.


attempt at fixing #1763

Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : https://github.com/karatelabs/karate/issues/1763
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

Not ready to merge yet. Hoping to leverage the CI/CD run and even if it passes I haven't put much more thought into some other different combinations of scenarios of iframes such as iframe inside iframe and using indexes to navigate between iframes.

@ptrthomas any guesses why ci/cd failed? Seems like it couldn't detect the new html I introduced somehow.
Anyhow this is not merge ready. I'm fairly certain I broke some cases of switching frame but the change I put it fixes the problem. Basically need to change the target as the other frames are different processes due to chromes Site Isolation. Just need to think through a way of managing this because we won't want to change the Target at all times.
Fyi @stevenspears

@joelpramos can you re-do this as a clean PR (now the commit history is a bit too much) without the extra docker containers etc. as a special case - I'm ok to merge your fixes to the DevToolsDriver as long as all the existing tests pass


php: ctrl + space not working in "Brackets"

ctrl + space not work in "Brackets"

I assume you meant "Brackets".
Can you be more specific: which language are you using? Do you use some extensions?
If you try Debug \ Reload Without Extensions does work again?

Yes, I'm use php Language
I try  Debug \ Reload Without Extensions , but does not work.

By default, Brackets doesn't come with PHP Code Hints.
You can install an extension to get these. Try "PHP SmartHints", for example. Let us know if you're still having trouble.

Closing as extension available.


[BUG] Unable to interact with files that have spaces in their names

Describe the bug
Whenever a command that takes a file name as an argument (such as RETR or STOR) is used on a file that has spaces on its name, the program crashes
To Reproduce
Steps to reproduce the behavior:

Run as Client
Connect to a server
Use a command that takes a file name as an argument (such as RETR or STOR)
See error

Expected behavior
The program should interact normally with such files
Desktop (please complete the following information):

OS: All Supported Operating Systems

Additional context
The problem is caused by how the program parses arguments. Basically, the program interprets everything that comes after a space as an argument. That means
RETR example of a file name that contains spaces.txt has 8 arguments instead of 2. Note that not even adding quotes around will work, because the file name is a string, and so the program would see it like this, ""example of a file name that contains spaces.txt"". This was just a placeholder way of parsing arguments, as a new method would be necessary in the future due to this fact.

As of v0.9.1, it's now possible to do so by using quotation marks.
However if only one argument has spaces, all other args require quotes as well (even if they don't have spaces). An example is shown below:
STOR "C:\\path with spaces\\file.txt" "file.txt"


[DOC]fix 'spark.memory.offHeap.enabled' default value to false

modify 'spark.memory.offHeap.enabled' default value to false

@JoshRosen

Can one of the admins verify this patch?

ok to test.

LGTM

@zzcclp Could you file this issue in JIRA and modify the title of this PR like [SPARK-XXXX][DOC]fix... ?

@sarutak , i think this is a little fix, dose it need to file a issue in JIRA?

For this one it is probably OK. We can just merge it (into master & branch-1.6).

I've merged this.

@sarutak , @rxin  , thanks

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48910/
Test FAILed.


rename fetchBranch (and similarly named commands) to downloadBranch.

It's suggested to rename to pull.

pull is a wrong name. It's more correct to call it Fetch & Checkout. But since it's too long, it can just imply that all fetch commands is followed by a checkout.


Compatibility with Nuxt 1.4

This PR bundles the work by @cesasol to port this module to the new hooks system, and  the fixes by @apmontgomery for universal and generate mode.
The module now works in all modes with the latest version of Nuxt (v1.4.0).
Note that for nuxt generate mode, you need to configure your baseURL and browserBasseURL to reach the nuxtent API at build time and in production respectively.
An example config could be something like:
api: {
  baseURL: 'http://localhost:3000',
  browserBaseURL: 'http://my-production-domain.com'
}

Help!

@joostdecock please, remove example's folder for test only module.
Duplicate module name: nuxtent-example

@joostdecock or let's remake examples ;)

@derevandal Removed the examples folder, still breaks for the dev tests, and I'm in way over my head

this is a problem with the test setup.  Should just need a couple of small changes to make all tests pass.

change nuxt dev dependency to latest
in test/e2e/common/nuxt.config.js:
delete the first line importing nuxtent
in line 13 change the modules line to modules: ['@@'],

@joostdecock would you like to make those changes to your PR or would you prefer that I submit a different pull request?

Those changes are required to fix the PR build. So if you want a new PR, you should perhaps bundle the changes in this PR in it. That way, your PR can essentially replace this one.
Alternatively, you could patch this PR. I have no preference, so you do you.

closing in favor of #151


Implement Alternate Sourceless Strategy

A continuation of #2
I think, looking at stream.coffee that there already is support for an alternate source to fall back to.
AC:

[x] replicate a scenario for when a fallback stream is triggered
[x] create a silence mp3 for the fallback stream
[x] see if the fallback stream works when the main source is not reachable by streammachine

This makes it sound like there already is a fallback strategy:
https://github.com/StreamMachine/StreamMachine/issues/73
Test out:

[ ] having a fallback stream that's playing something else Nickleback??
[ ] pointing a silence stream to the fallback stream

I just tested it out and it is currently possible to have a fallback stream with the current StreamMachine code. To make it work, I did this:

start streammachine standalone mode ./streammachine-cmd --config ./config/standalone.json  with standalone.json looking something like this (notice the fallback option)

{
	"port":         8001,
	"source_port":  8002,
  "api_port":     8003,
	"log": {
		"stdout":     true
  },
  "streams": {
    "test": {
      "key":              "test",
      "root_route":       true,
      "format":           "mp3",
      "seconds":          3600,
      "metaTitle":        "Testing",
      "host":             "localhost",
      "source_password":  "testing",
      "fallback": "http://live.scpr.org/kpcclive/"
    }
  }
}

The fallback option above is just pointing to our live stream, but this can also be a stream of a silence mp3.

I started a "real" stream with liquidsoap, streaming my test playlist (pretending that this playlist was the real stream) to the now running streammachine instance: liquidsoap 'output.icecast(%mp3, host="localhost", port=8002, password="testing", mount="test", mksafe(playlist("~/playlist.m3u")))'

I then listened to the stream http://<IP_ADDRESS>:8001/test using VLC.  It started playing the audio from playlist.m3u as expected. I then killed the liquidsoap instance I had running from step 2, effectively killing the stream. After a second or two, VLC started playing audio from the livestream (my fallback option)

In theory, if we had another stream coming from somewhere, we can mount it on the same stream our livestream is on, mark it as a fallback, and streammachine should be able to switch to that when the primary stream is down.  I think that's what the fallback option Eric coded in was created for.


Let UiaEnum can be part of UiaTuple

Because we cannot construct a UiaEnum from an AutomationRemoteAnyObject, UiaEnum currently cannot be included as part of a UiaTuple type. This PR adds this constructor for UiaEnum and makes proper tests for this function.

/azp run

@pengyuanjin, do you know why the PR lists our first merge commit? It's probably fine, but would you be able to clean that up just in case?

@beweedon, that "Merge pull request #1 from microsoft/main" was the PR I made in the forked repo to sync up with the origin.
Is there other ways to sync repos?

Basically, you want the branches in your fork to look identical to the ones in the upstream repo. You shouldn't ever have to create merge commits in the main branch of your fork, for example.
Basically, to sync your fork main with upstream main, you do:
git checkout main
git fetch upstream
git rebase upstream/main

This should create a fast-forward merge, meaning your fork's main branch is brought up to point at the upstream's main branch. You don't need a PR at all.
If you're in a situation now where your fork's main branch has this extra merge commit in it, you can hard reset main to the upstream branch with:
git checkout main
git reset --hard upstream/main

This will erase any uncommited changes in the repo, so make sure you don't have any uncommited changes before you do this.

@beweedon Now it only has one commit 😊

/azp run

@pengyuanjin permission to merge?

@beweedon Yes, please! (for both PRs)


Question: Adding Translation

Hi. I'd like to help add translation to my language, but I know little about C# and .NET. How shall I contribute?

Hello, thanks in advance for the proposition 😀
To make a translation, please copy and paste the CLang_en.cs file to a new one called CLang_{the 2 letter code of your language} .cs, this file contains all the defined strings in game so you can add a translation just by translating the dictionnary elements to your language.
Once done please make a pull request for it, I'll integrate it directly once done (there is still a manual step to do by my side to integrate it in-game, by editing the ILang.cs file)
To test how it works in game you can temporarly replace CLang_en.cs by yours, just be sure to re-add the original file before the pull request

Thank you for the instructions!
I have never worked with C# and .NET before, so I have no idea how to build and test the project. Is it OK that I make a pull request without testing?

You are welcome !
To build and test it, you can use Visual Studio (you will also need .Net Framework 4.8 in this case)
It is ok as long as the only edited file within the pull request is your CLang_{xx}.cs file


fix: virtual traffic light/suppress launch

Description
Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@Mergifyio backport #1290


Reason/benefit of loading html file instead of data/blob/srcdoc/blank+document.write in iframe?

Please explain why the iframe couldn't use some of the possible dynamic sources and had to use a static html file.

After trying out some stuff I guess it is for proper sandboxing while still having a non-null origin and thus access to localStorage and similar.
But now that I understand that part, I think a warning in the documentation should be added for people that self-host the module to serve it from a separate origin (i.e. subdomain or port) or the sandbox can be escaped. While that is not critical for normal usage, i.e. when a user is just running their own sample code, it becomes an issue if the ability to run code from other users is added in their use-case. Those using it directly from flems.io are safe of course.

Would be good to document more - PR welcome ;)


Less travis technical nitty-gritty details

Less travis technical nitty-gritty details, more focus on why and how to do it with your code

It would be good to generalize to GitLab CI (#18) but I think it is good to show Travis CI in the current form. The more motivation part is covered by other issues. Closing.


Cloudwatch metrics/alarms

This PR introduces the use of cloudwatch metrics for alarms.
This is to alert us to internal errors, e.g. if it fails to fetch super mode data but can still serve requests.
For now it's just writing a metric for super mode failures (and alerting when an error happens), but in follow up PRs we can add more.
In particular, we're about to start fetching channel tests data from dynamodb - this can be a new metric/alarm.

LGTM 👍 is this in response to a prod issue? or more a pre-emptive measure?

It's pre-emptive, and I picked super mode to begin with because it's a simple example.
Really we should have had alarms around fetching data all along, but it feels particularly important now we're migrating to dynamodb for the tests config

LGTM 👍 is this in response to a prod issue? or more a pre-emptive measure?

It's pre-emptive, and I picked super mode to begin with because it's a simple example. Really we should have had alarms around fetching data all along, but it feels particularly important now we're migrating to dynamodb for the tests config

Yeah makes sense! Looks great 👍


Show resources only in non-debug mode

It may slow down debugging the tests, so let's not show resources whatsoever when in DEBUG mode

[test]

Good idea. It is unfortunate that we still do not have a straightforward way do enable debug mode in CI :/.


Overpowered MEM chips

So as a complete 180° turn from my previous issue with MEM chips #67, i now think these are way overpowered ! 🤡 espicially in the huge levels generated in Endless mode #96 .
While ACC have their price go up for every new ACC chip bought to compensate, and CLK becoming menacing for your CPU health as heat goes up, MEM chips are always interesting to place down.
They ignore attacker strenght, which instantly makes them a must-go for absurdly powered waves in later levels. And while they do recharge slowly, this can be improved by heavily upgrading their speed and range with recruits, and by lowering attackers rate and speed. And if it's still not enough, throw in a CLK chip and you're pretty much set for any situation.

A quick fix would be to apply the same limitations the ACC chip gets, having the price for MEM go up for every new one bought (but that would only temporarily solve the issue in some early levels with MEM, as effective bit value goes down the more powerful attackers get).
My prefered solution would have MEM chips generate heat for each attacker swallowed. This would probably require a bit of balancing of heat generation on CLK, but what i'd expect this solution to create is less use of MEM in early lane (so as to not excessively overheat), and more use as emergency or extraordinary chip placement so it still serves as a useful catch-all chip but also limits their overall number in a set level.

100th issue 🎉

I was thinking a good way to limit their effectiveness would be to require the player to tap on them once they have captured an attacker, before the chip can asorb a new attacker.
You would still be able to place down chips but would be required to manually intervene on each of them so they can attack again. So, a huge number of chips would require a huge amount of player interaction, limiting its power.
Contextually it works too ! as stored data doesn.t just go away, but need to be taken care of, so player interacting with the just could be interpreted as the user moving data around or away into antoher storage.

good idea. I'll try that.

I've implemented this suggestion. Seems to work quite well. For compensation I have reduced the cool down time (from 128 to 72). The whole mechanism still requires some fine-tuning, I suppose.
This is the new introductory text, I hope it is understandable:
Nous introduisons une nouvelle puce, la MEM (pour memory). Celle-ci prend un nombre et le stocke en toute sécurité dans la mémoire de l'ordinateur.
Le cadre plus large montre qu'un nombre se trouve à l'intérieur. Dans cet état, la puce MEM ne peut pas stocker un autre nombre. Elle doit d'abord être 'lue' (en la touchant).  Ce n'est qu'alors que commence son cycle de refroidissement et qu'elle peut absorber le prochain attaquant.
Les puces MEM sont puissantes, car elles peuvent neutraliser n'importe quel attaquant, mais elles demandent de l'attention et sont assez lentes.


i2c_driver.py empty functions with no actions

This package contains required functions with description of the functionality but does not contain the code to perform the actions. i2c_driver.py  contains a whole set of empty functions. How can I update these functions to make use of this package?

This package is meant to be used with our Qwiic devices (listed here) and the supported hardware. Please, review our documentation here.
That file is for general use... i.e. as a reference/template.


Organise .gitignore

Reorganise
Use *.log to be more generic

LGTM, BTW @simison you should have perms to do reviews and push code now to this repo


Provide rpm repos via openSUSE Build Service.

As of right now, there is no way to update atom on linux besides manually downloading and installing a .deb or a .rpm file. While there is support for adding a .deb repo via launchpad, there has been no support for .rpm distributions. Hosting a repository on the openSUSE build service will provide both Fedora and openSUSE users with an option to update atom via package manager.

https://build.opensuse.org/
https://en.opensuse.org/openSUSE:Build_Service

http://www.makehuman.org/doc/node/packaging_rpms_for_susefedora_using_open_build_service_obs.html

:+1:

I'm subscribed to https://github.com/atom/atom/tags.atom to know when I can update and do it manually. It's boring. Fedora here.

So since nobody wants to touch this I am going to try this by myself. I wanted to avoid this since I am kinda new to Linux and I have never compiled something before. If you want to help pls let me know.

@sudopluto Did you ever get this working on open suse build service?

@amayer5125 I haven't tried it yet. I read from somewhere that atom needs an internet connection to be built. Right now the only build service on Linux that has an internet connection is Copr. There is also the fact I had just switched to linux in April, and I still have no clue how to package.
TL;RD : No.

I found a repo link in another issue thread. They just host the prebuilt ones from atom/atom/releases which if fine by me.

Do apps hosted on openSUSE (or any other software repository for that matter) need to be built from source? Or is it fine for them to just use the prebuilt binaries?

@mnquintana I guess it depends on the hosting.
If you are looking for a temporary repo you can use this one. I hashed the most recent version hosted in the rpm vs the one on atom/atom/releases and the hases match up so I guess his server is just grabbing the rpm/debs and hosting them in a repo, which works perfect. I havent used it for an update yet but I installed from it using dnf (yum replacemnt) on fedora 22.

You can build RPM from binaries using nosrc RPMs, but being an open source project that's not very pleasing, and you could automate many things.
RPM normal packages can have several sources and patches included, so there should be no need to have Internet connection to build. If it is, then IMHO it's a bug in Atom.

It seems someone has a spec file for Fedora here: https://raw.githubusercontent.com/LukeCarrier/rpm-specs/master/SPECS/atom.spec
And here you can find the Requires/BuildRequires for openSUSE: https://github.com/atom/atom/blob/master/docs/build-instructions/linux.md#opensuse
If I would know npm better I would package it myself, but I don't know enough about it.

@sudopluto this is still true, that no means exists to automatically update Atom on RPM platforms. I have created a spec file for Atom 1.5.2 that works without issue on platforms with a valid internet connection that is:
Name:           atom
Version:        1.5.2
Release:        0
Summary:        A hackable text editor for the 21st century
License:        MIT
Group:          Productivity/Publishing/Other
Url:            https://atom.io/
Source0:        v%{version}.tar.gz
BuildRequires:  git-core
BuildRequires:  hicolor-icon-theme
BuildRequires:  npm
BuildRequires:  nodejs-packaging
BuildRequires:  libgnome-keyring-devel
BuildRequires:  python-setuptools
BuildRequires:  update-desktop-files
# MANUAL BEGIN
Requires:       nodejs
Requires:       python-http-parser
# MANUAL END
BuildRoot:      %{_tmppath}/%{name}-%{version}-build

%description
Atom is a text editor that's modern, approachable, yet hackable to the core
- a tool you can customize to do anything but also use productively without
ever touching a config file.

%prep
%setup -q
sed -i -e "s/<%= appName %>/Atom/g" \
       -e "s/<%= description %>/Atom is a text editor that's modern, approachable, yet hackable to the core - a tool you can customize to do anything but also use productively without ever touching a config file./g" \
       -e "s|<%= installDir %>/share/<%= appFileName %>/atom %U|/usr/bin/atom %U|g" \
       -e "s/<%= iconPath %>/atom/g" \
       resources/linux/atom.desktop.in
%build
# Hardened package
export CFLAGS="%{optflags} -fPIC -pie"
export CXXFLAGS="%{optflags} -fPIC -pie"
until ./script/build 2>&1; do :; done

%install
script/grunt install --install-dir "%{buildroot}%{_prefix}"
# copy over icons in sizes that most desktop environments like
for i in 1024 512 256 128 64 48 32 24 16; do
    install -Dm 0644 /tmp/atom-build/icons/${i}.png \
      %{buildroot}%{_datadir}/icons/hicolor/${i}x${i}/apps/%{name}.png
done
%suse_update_desktop_file %{name}

%post
%desktop_database_post
%icon_theme_cache_post

%postun
%desktop_database_postun
%icon_theme_cache_postun

%files
%defattr(-,root,root,-)
%doc CONTRIBUTING.md README.md docs/
%{license} LICENSE.md
%{_bindir}/atom
%{_bindir}/apm
%dir %{_datadir}/atom
%{_datadir}/atom/*
%{_datadir}/atom
%{_datadir}/applications/atom.desktop
%{_datadir}/icons/hicolor/*/apps/%{name}.png
%exclude %{_datadir}/%{name}/libgcrypt.so.*
%exclude %{_datadir}/%{name}/libnotify.so.*

%changelog

if it helps. I asked a similar question to this one at forums.opensuse.org too https://forums.opensuse.org/showthread.php/513540-How-to-create-a-spec-file-for-Atom-that-is-suitable-for-the-OBS?p=2753770.

I have find this simple, I guess, solution.
The Yarn project has the following instructions for install in Fedora, for example:

On CentOS, Fedora and RHEL, you can install Yarn via our RPM package repository.
sudo wget https://dl.yarnpkg.com/rpm/yarn.repo -O /etc/yum.repos.d/yarn.repo
Then you can simply:
sudo yum install yarn

I got the yarn.repo and it is like this:
[yarn]
name=Yarn Repository
baseurl=https://dl.yarnpkg.com/rpm/
enabled=1
gpgcheck=1
gpgkey=https://dl.yarnpkg.com/rpm/pubkey.gpg

Let me know if I helped ...

@TJProgrammer how is that relevant to this question? We're not asking how to install Yarn on Fedora, we're asking the Atom development team to provide an OBS repository for Atom.

I am sorry for the mistake.
I take the yarn to an example to atom.
In other words, atom team can do the update system like yarn team do.
But, again, sorry for the last comment.


Responsiveness For Profile and Blob Page

@Abhishek-kumar09
fixed Responsiveness of Profile and blob home page
Bugs Fixed ✅

Responsiveness
profile Section Description
Avatar Profile section
and other minor bugs

@Abhishek-kumar09 Should Close this pr and again pull a draft pr

@Abhishek-kumar09  Done


⚠ GitLab Server has degraded performance

In c4c0c6c, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6405 ms

Resolved: GitLab Server performance has improved in cdd3fde.


Suggested helmfile command line removes first manifest from template results

I initially raised this issue with argocd but its actually a problem with helmfile template command line.
One of our manifests is LimitRange. It is the first to be rendered with helmfile template. And sed -e '1,/---/d' line from helmfile --namespace $ARGOCD_APP_NAMESPACE template | sed -e '1,/---/d' | sed -e 's|apiregistration.k8s.io/v1beta1|apiregistration.k8s.io/v1|g' removes it entirely.
This is the manifest itself and what is being removed exactly:
---
# Source: controlplane/templates/limitrange.yaml
apiVersion: v1
kind: LimitRange
metadata:
  name: limit-range
  labels:
    app.kubernetes.io/managed-by: Helm

  annotations:
    meta.helm.sh/release-name: controlplane
    meta.helm.sh/release-namespace: sidh-test

spec:
  limits:
    - type: Container
      defaultRequest:
        cpu: "10m"
        memory: "50Mi"
      default:
        cpu: "50m"
        memory: "100Mi"
---

We removed sed -e '1,/---/d' line from our helmfile command for now.

@sidh
Thanks for your comment!
Please wait while we check.

@sidh Hey! I don't quite understand what you're suggesting.
sed -e '1,/---/d' seems to be doing exactly what it is intended to do, remove everything from the first occurrence of --- to the second occurrence of ---.
And I see nothing in this repository mentions the use of sed -e '1,/---/d' with helmfile, right? 🤔

sed -e '1,/---/d' seems to be doing exactly what it is intended to do- remove everything from the first occurrence of --- to the second occurrence of ---.
And I see nothing in this repository mentions the use of sed -e '1,/---/d' with helmfile, right? thinking

There it is - https://github.com/chatwork/dockerfiles/tree/master/argocd-helmfile#usage
Yes it does exactly it is supposed to. Which results in removing first manifest rendered by helmfile whatever that manifest is. For us it happened to be limitrange manifest.


updated UI of Copy rights

Related Issue
None
Fixes:  #4034
Description
The Copyright Policy page has been improved with animated boxes that smoothly slide up when the page loads, enhancing user engagement. Each section now features hover effects, where the boxes elevate slightly and change color, providing a more interactive experience. This modern design ensures better readability and a polished look, making it easier for users to navigate and understand the content. Overall, these enhancements create a more inviting and user-friendly environment for visitors.
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
BEFORE:

AFTER:
https://github.com/user-attachments/assets/403fa1f9-d8ee-4986-9cfe-085e9759a857
Checklist:

[X] I have made this change from my own.
[X] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

Pls add level 3 as I work on hover ,animation and dark mode effect to that page  @abhi03ruchi

the grey color does not look good . instead of grey add the brown backgrd color .