I/E specification in SysSettings

Additional records (representing i/e options) are generated for TIMES-NZ in #256 for the attributes that have i/e options specified in VT files. The discrepancy appears to be due to Veda behaviour described here (i.e. redefinition (via any tag in SysSettings) of interpolation options defined in Base/SubRES is ignored in Veda2.0).
Should the tool do the same or should we introduce a switch to control this behaviour?

Comment:  I don't think it is due to that VEDA2 "things that is different", which is only related to using TFM_INS in SysSettings. The SysSettings facility for IE options has always been meant to provide default values, which should be overridden by any IE options explicitly defined by the user  elsewhere. VEDA has thus always prevented the SysSettings IE defaults from overriding what has been defined in both VT or SubRes files. And in my view that is the only sensible convention: Values explicitly defined for specific attribute instances in VT or Subres files should not be overridden by those generic bulk definitions made in SysSettings. For regular scenarios this usually works automatically (because SysSettings is best placed before regular scenarios in the scenario order).

Well, I am only quoting the official source. 🤷 I guess, it must be true what is written there... At least at some point.
Imho, using the transform tags, meant for data manipulation, to specify defaults is not a good design choice, as it overcomplicates things. Also, why should the placement of syssettings matter?
In any case, we could implement the same behaviour and use it by default, but allow turning it off. In my view, the main benefit of having the switch is that it will make it more explicit that there is an exception to the general data manipulation routine.

Being able to define default IE options like we can in SysSettings is in my view extremely valuable.  If you would take that away, it would be a fundamental regression.
The NZ model does explicitly state the default nature by the sheet name: "Interpol_Extrapol_Defaults".  So, it should be clear to anyone that they are meant to be defaults.
Of course the placement of SySsetting matters, like the placement of all regular scenarios exactly in the scenario order defined by the user is critical for reproducing any model run case. As you should know, GAMS overwrites parameter values defined earlier when the same parameter instances are defined in data statements defined later. And so, if SysSettings would be included as the last scenario, the defaults would not work at any for regular scenarios.  On the other hand, putting  SysSettings as the first scenario of all model data would be ideal (then no special handling would be needed in VEDA for the default IEs), but it is not possible with VEDA, because VEDA does not have a separate file for defining all the domains first.

Imho, using the transform tags, meant for data manipulation, to specify defaults is not a good design choice

Looks like you may have adopted a meaning for TFM ("transform") tags somewhat different from VEDA.  Under VEDA, the TFM_ tags are, in general, not doing any data manipulation which would change any existing data. They basically only generate new scenario-specific data into the scenario where the tags are, and do not change the data in other scenarios at all.  As far as I can recall, with the exception of the cases where TS_Filter is being used, the existing data is left untouched in all other scenarios, just like they should be when generating the default IE options. However, if in xl2times the TFM tables are, in fact, causing some additional data manipulation, I understand it might cause consistency issues with VEDA models.

@Antti-L it is probably just the language issue! Neither you nor I is a native english speaker which now and then seems to cause misunderstandings. 🤷


Ontroller LEDs refresh timer

Hi there,
I added a timer to refresh the LEDs every 2 seconds, without this the ontroller would reset it's LEDs after roughly 3s after the last WinUSB message.
By the way, i don't know if you saw this, but Dniel97 added MU3 LEDs support today to the hook, handling both io4 and 15093 boards! So Ontroller side LEDs is at arms reach!
https://gitea.tendokyu.moe/Dniel97/segatools/releases/tag/2024-05-12
Regards.

Here is a video of the issue before the proposed fix :
https://github.com/SirusDoma/Mu3IO.NET/assets/9607300/119328b7-7c09-4ad0-a756-07ca0b9e9238

Hi! Thanks for raising the PR!
I'm not really familiar with how the game deals with the LED and I don't have an ontroller myself so this is really helpful.
I have one concern about this change however, rather than using a Timer, I prefer to put this in Poll or other update method. I know Poll is not an ideal place, but I think I would rather avoid using a Timer (or any other independent timing mechanism), because they're simply aren't synchronized with game timing at all.
Let me hear what your thoughts

Oh, and if you're to follow my suggestion, I would also like to hear the performance impact of writing the leds every time Poll gets called

Hi, it really seems overkill to do it every Poll, but it will work for sure and the impact will be non significant I think. The original Ontroller code, does it in a while loop like crazy, so ...
But maybe I can add a "lastLedRefreshTime" enforcing a 1 or 2 seconds wait in the poll method tho?
I can try that later this evening if you want?
Also, did you saw Dniel97 pre-release I was mentioning, maybe we could conform to the specs he describes in the mu3 segatools.ini :
`; -----------------------------------------------------------------------------
; LED settings
; -----------------------------------------------------------------------------
[led15093]
; Enable emulation of the 15093-06 controlled lights, which handle the air tower
; RGBs and the rear LED panel (billboard) on the cabinet.
enable=1
[led]
; Output billboard LED strip data to a named pipe called "\.\pipe\ongeki_led"
cabLedOutputPipe=1
; Output billboard LED strip data to serial
cabLedOutputSerial=0
; Output slider LED data to the named pipe
controllerLedOutputPipe=1
; Output slider LED data to the serial port
controllerLedOutputSerial=0
; Serial port to send data to if using serial output. Default is COM5.
;serialPort=COM5
; Baud rate for serial data
;serialBaud=921600
; Data output a sequence of bytes, with JVS-like framing.
; Each "packet" starts with 0xE0 as a sync. To avoid E0 appearing elsewhere,
; 0xD0 is used as an escape character -- if you receive D0 in the output, ignore
; it and use the next sent byte plus one instead.
;
; After the sync is one byte for the board number that was updated, followed by
; the red, green and blue values for each LED.
;
; Board 0 has 61 LEDs:
;   [0]-[1]: left side button
;   [2]-[8]: left pillar lower LEDs
;   [9]-[17]: left pillar center LEDs
;   [18]-[24]: left pillar upper LEDs
;   [25]-[35]: billboard LEDs
;   [36]-[42]: right pillar upper LEDs
;   [43]-[51]: right pillar center LEDs
;   [52]-[58]: right pillar lower LEDs
;   [59]-[60]: right side button
;
; Board 1 has 6 LEDs:
;   [0]-[5]: 3 left and 3 right controller buttons
;`
I'm not sure how the pipe thing works along with the new mu3_io_led_init and mu3_io_led_set_colors functions.

Hi, it really seems overkill to do it every Poll, but it will work for sure and the impact will be non significant I think.

No, it's not overkilled, in my opinion; the payload is small, and with USB, it is way too trivial. I believe the hardware itself may expect the data to be written continuously (don't quote me). The performance impact concern is pointed towards the comparison between calling it on Poll vs setting leds on demand (via SetLeds). And my main concern was more about the hook flow / structure, but I'm generally fine putting this in Poll.

Also, did you saw Dniel97 pre-release I was mentioning, maybe we could conform to the specs he describes in the mu3 segatools.ini

I haven't, and probably I can't check this anytime soon as I don't have an ontroller and I have other things to do. I think the problem is that we need to figure out how to write the led data of side led of the ontroller (that is, if the left led is controllable at all)

I believe the hardware itself may expect the data to be written continuously (don't quote me).

It is, my testing on the real hardware confirm this, but not that often. As shown in the video I posted.

The performance impact concern is pointed towards the comparison between calling it on Poll vs setting leds on demand (via SetLeds). And my main concern was more about the hook flow / structure, but I'm generally fine putting this in Poll.

Let's put it in poll and see how it goes then. I'll open a new PR, I opened this one with a github account I was going to delete by mistake ...

Also, did you saw Dniel97 pre-release I was mentioning, maybe we could conform to the specs he describes in the mu3 segatools.ini

I think the problem is that we need to figure out how to write the led data of side led of the ontroller (that is, if the left led is controllable at all)

Yes it is controllable, I analysed it, this is how the ontroller hook is doing it :

In short, we have to get the side LED data from the new Dniel97 hook, format it the right way and append it to the USB payload. We may encounter the same issues as the ontroller hook + Bepinex mod tho, that is that setting all the LEDs at once can cause issues to the middle LEDs. We will need to mitigate those, this is what I was trying to do at first by patching their hook but this was too complicated and improving your project seemed the clever and open-source way of doing it.
Here is a video of the issues I am talking about :
https://github.com/SirusDoma/Mu3IO.NET/assets/9607300/1f147df3-a37b-4b44-b36d-17a153bcdc04

Here you go, it works great and the CPU usage is barely increased :)

Hi, sorry for the late response.
I'm kind of busy this week but I'll get back to you once I'm free.


docs(ComponentExample): remove useless components and cleanups

This PR makes some cleanups in components in our docs.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


include-pattern ruleset configuration is not working

PHPMD version: all
PHP Version: all
Installation type: all
Operating System / Distribution & Version: all

Current Behavior
The xsd schema specify an allowed node named "include-pattern" . But there is no occurence in the code where this node is supported
https://github.com/phpmd/phpmd/blob/caf04a5c1c91bdecefbe900c2a7d6235aa5c6106/src/site/resources/web/xml/ruleset_xml_schema_1.0.0.xsd#L13
Expected Behavior
When "include-pattern" is set in the rule set it must be take into account and PHPMD must parse the files included byuu this pattern
Steps To Reproduce:

Create a project with :

index.php
src
   test.php

phpmd.xml:

<ruleset name="PHPMD RuleSet"
    xmlns="http://pmd.sf.net/ruleset/1.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://pmd.sf.net/ruleset/1.0.0 http://pmd.sf.net/ruleset_xml_schema.xsd" xsi:noNamespaceSchemaLocation="http://pmd.sf.net/ruleset_xml_schema.xsd">
    <description>PHPMD RuleSet</description>
    <include-pattern>src/*</include-pattern>
</rulset>

run phpmd . ansi phpmd.xml

PHPMD will parse index.php while it is not in src/
Checks before submitting

[x] Be sure that there isn't already an issue about this. See: Issues list
[x] Be sure that there isn't already a pull request about this. See: Pull requests
[x] I have added every step to reproduce the bug.
[x] If possible I added relevant code examples.
[x] This issue is about 1 bug and nothing more.
[x] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

I don't see that option in the documentation?? Does that exist?
It would be really handy, that is for sure.

I don't see that option in the documentation?? Does that exist?
It would be really handy, that is for sure.

At least it referneced in xml schema: https://github.com/phpmd/phpmd/blob/47bb51c63d9340fa27db67595f65e254a5bd3db3/src/site/resources/web/xml/ruleset_xml_schema_1.0.0.xsd#L13

Isn't this because you are expecting it to overwrite the argument but it's in addition?

I don't think so, there is no occurrence in the code where "include-pattern" can be used


Adding item to Group

Hi Marco, this line of code looks like it needs to be changed
in assembly>Group class
def __iadd__(self, obj):
    """
    Add an object to the group
    """
    if not vedo.utils.is_sequence(obj):
        obj = [obj]
    for a in obj:
        if a:
            self.AddPart(a)   ### here
    return self

needs to be
def __iadd__(self, obj):
    """
    Add an object to the group
    """
    if not vedo.utils.is_sequence(obj):
        obj = [obj]
    for a in obj:
        if a:
            try:
                self.AddPart(a.actor)  ### here
            except:
                self.AddPart(a)
    return self

like in Distance_Tool
If I come across more, I will let you know
Best Regards

Sure would you mind making a PR?

Of course I dont mind, but I havent done anything like that before : )
Please let me try and I will let you know...


API not idiomatic

Get* functions are not idiomatic in Go.

What about something like https://golang.org/pkg/net/http/#Header?

See https://golang.org/doc/effective_go.html#Getters
(For http.Header the Get function sets a generic field so it's okay)

Ok, so then that would imply a change to the read adapters' signatures. Something along the lines of:
type identityReadAdapter interface {
    ID() (string, error)
    Type() (string, error)
}

type attributesReadAdapter interface {
    Attributes() (map[string]interface{}, error)
}

...etc

It doesn't seem like implementing any of those methods would conflict with other packages. The real only issue I have here is that they aren't true getters since they return a value/error pair, so is it even appropriate to consider them under this idiom?
Also I'd think to leave the jsonapi.Links and jsonapi.Relationships signatures unchanged since they work in a similar way as http.Header.

Why do these functions return an error in the first place?

Because errors can occur while converting data to/from the resource. For a concrete example, consider that the specification states that resource id members must be of type string. However, it is not uncommon to use an integer to identify an object on the backend. In order to avoid ignoring errors in cases like this (for instance, calling something like strconv.ParseInt), all read adapter implementations should return a value/error pair and all write adapter implementations should return an error.


A problem occurred configuring project ':flutter_inappwebview'.

    classpath 'com.android.tools.build:gradle:8.1.4'

distributionUrl=https://services.gradle.org/distributions/gradle-8.0-all.zip
Environment

Technology
Version

Flutter version
3.16.5

Plugin version
5.8.0

What went wrong:
A problem occurred configuring project ':flutter_inappwebview'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

 If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

 classpath 'com.android.tools.build:gradle:8.1.4'
distributionUrl=https://services.gradle.org/distributions/gradle-8.0-all.zip

## Environment
Technology	Version
Flutter version	3.16.5
Plugin version	5.8.0

You need to use new version 6.


Improve language support for mollie

@wvanbergen
:tophat:'d :locale change

Is there any way we can get this through i18n so it shows up in the right language for everybody? @Shopify/i18n

@haeky wanna take a look?
FYI @davidcornu

I will look into that


Windows build fixes

A few fixes for the Windows build with the new trk patch

Does this fix #382?

Seems fixed to me on second build from @squalus.
https://github.com/squalus/ungoogled-chromium-winbuild/releases

@waqas Alright, thanks. I'll release a new version of 65.


rsnapshot module: Pidfile in /run/rsnapshot/

My approach on solving #6289

Did you actually test this? I'm sure it fails because preStart runs as the rsnapshot user.

No, I did not. I don't know how! :smile:

To test you can build your local generation based on this config..

@matthiasbeyer the rsnapshot module does not define a systemd service, its a system cron job. The preStart script will never run. It looks to me like rsnapshot will be executed as root via cron and that patching is unnecessary. I wonder, did the original error occur when you invoked rsnapshot directly?

@matthiasbeyer: for info about building your system against your local git repo to test your work:
http://nixos.org/nixos/manual/sec-getting-sources.html
From the bottom of that page:
nixos-rebuild switch -I nixpkgs=/my/sources/nixpkgs

I'm not at home ATM, but I will test this as soon as I'm at home again.

I'm not sure we should continue this one.
I use rsnapshot without problems for about 3 months now. I'll close this now (after rebasing to latest master, if someone wants to use this patches), reopen if necessary!


Add code coverage reports

To test, pass -DCODE_COVERAGE=ON and build the target ccov-MLIREmitCTests.


no-class does not catch local classes

The no-class rule does not warn for this case:
export const Foo = (bar: string) => class FooBar {};

Fixed in #70.

Released in v4.5.2.


Automatically update new users in active competition

Now that competitions automatically take a snapshot for all participants when it starts it should also have the ability to automatically snapshot a user that was added to a competition after it started if they don't have any existing data points within the competition timeframe.

Sounds good, similar/duplicate of #777


Error: return code 1

ranscoding 8 Audio files
'/home/student/student-MP3s/gettysburg10-16_19_32.mp3' - return code 1
'/home/student/student-MP3s/OSR_us_000_0014_8k-9_32_43.mp3' - return code 1
'/home/student/student-MP3s/OSR_uk_000_0020_8k-16_20_3.mp3' - return code 1
'/home/student/student-MP3s/84-121123-0001-10_4_53.mp3' - return code 1
'/home/student/student-MP3s/gettysburg10-9_21_40.mp3' - return code 1
'/home/student/student-MP3s/84-121123-0002-10_13_44.mp3' - return code 1
'/home/student/student-MP3s/OSR_uk_000_0020_8k-9_32_37.mp3' - return code 1
'/home/student/student-MP3s/84-121123-0000-10_4_32.mp3' - return code 1
Traceback (most recent call last):
File "/bin/flac_to_mp3", line 98, in
subprocess.run(["notify-send", "--urgency=low", MESSAGE])
File "/usr/lib/python3.6/subprocess.py", line 423, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.6/subprocess.py", line 729, in init
restore_signals, start_new_session)
File "/usr/lib/python3.6/subprocess.py", line 1364, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'notify-send': 'notify-send'
Error in sys.excepthook:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/apport_python_hook.py", line 145, in apport_excepthook
os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0o640), 'wb') as f:
FileNotFoundError: [Errno 2] No such file or directory: '/var/crash/_bin_flac_to_mp3.1000.crash'
Original exception was:
Traceback (most recent call last):
File "/bin/flac_to_mp3", line 98, in
subprocess.run(["notify-send", "--urgency=low", MESSAGE])
File "/usr/lib/python3.6/subprocess.py", line 423, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.6/subprocess.py", line 729, in init
restore_signals, start_new_session)
File "/usr/lib/python3.6/subprocess.py", line 1364, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'notify-send': 'notify-send'

ERROR
Traceback (most recent call last):
  File "/Users/bigoper/Downloads/temp/flac-to-mp3/flac_to_mp3", line 98, in <module>
    subprocess.run(["notify-send", "--urgency=low", MESSAGE])
  File "/Users/bigoper/anaconda3/lib/python3.6/subprocess.py", line 423, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/Users/bigoper/anaconda3/lib/python3.6/subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "/Users/bigoper/anaconda3/lib/python3.6/subprocess.py", line 1364, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'notify-send': 'notify-send'

INFO
OS: Mac OS

Fixed. Should work fine without errors on Mac and Windows now.


Change main.js to app.js

The HTML page is coded to look for app.js, but I wasn't sure if you needed the file to be called main.js.  Can you confirm, and change either the HTML reference or the name of the file?

I think @seksenov changed the HTML reference here: 4f3863b67e70fefeaa98b38fd9401b955dae2a88
So if you uninstall and reinstall the generator, it should be in the right place now.

I noticed this changed the reference to main.js in the template earlier when I noticed the error. Not sure which was intended.


[STRMCMP-1161] Adding way to capture application env to kinesis and s…

…3 source python sdk

Also, can you please extend s3.py to take in environment as an argument.

Also, can you please extend s3.py to take in environment as an argument.

Woops, missed that. Done and rebased.

/approve (👍)


Directory 'D:\a\1\a' is empty. Nothing will be added to build artifact 'drop'.

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:Hosted VS2017

Issue Description
I import my project [https://github.com/RiseSoho/RiseUnity] to vsts and CI it .

Error logs
2018-05-18T09:38:47.8669682Z ##[section]Starting: Publish Artifact
2018-05-18T09:38:47.8672893Z ==============================================================================
2018-05-18T09:38:47.8673095Z Task         : Publish Build Artifacts
2018-05-18T09:38:47.8673247Z Description  : Publish build artifacts to Visual Studio Team Services/TFS or a file share
2018-05-18T09:38:47.8673365Z Version      : 1.133.0
2018-05-18T09:38:47.8673466Z Author       : Microsoft Corporation
2018-05-18T09:38:47.8673817Z Help         : More Information
2018-05-18T09:38:47.8673977Z ==============================================================================
2018-05-18T09:38:48.1337734Z ##[warning]Directory 'D:\a\1\a' is empty. Nothing will be added to build artifact 'drop'.
2018-05-18T09:38:48.1355716Z ##[section]Finishing: Publish Artifact

You don't copy anything to $(build.artifactstagingdirectory) in your definition. You are missing a 'Copy Files'-Task before the 'publish artifact'

@JohannesDrexler is correct.  You can also avoid the copy and pass a directory to publish other than the artifacts directory.  Note that copy takes all the mini match patterns and publish just take a directory.

I am having same issue. I tried all the above options but stuck on this one.
Fresher Joe :- Was this resolved for you?

can we get information on how you resolved this issue as I've hit the same wall.
thanks!
@sshrishi and @Fisher-Joe

Do you still have issue?

@SuperDuperOne For more information on the Copy Files Task, take a look at: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/copy-files?view=azure-devops&tabs=yaml. Specifically for my setup running Ubuntu 18.04, the following block was implemented before Publish in azure_pipelines.yaml:
- task: CopyFiles@2   inputs:     sourceFolder: $(Build.SourcesDirectory)     targetFolder: $(Build.ArtifactStagingDirectory)

I tried a .net core app, encountered the issue as well. But it doesn't need a copy file task, instead we can specify an output location for the "dotnet build" command, to let it output to $(Build.ArtifactStagingDirectory). Samples as below.
`steps:

script: dotnet build --configuration $(buildConfiguration) --output $(Build.ArtifactStagingDirectory)
displayName: 'dotnet build $(buildConfiguration)'

task: PublishBuildArtifacts@1
inputs:
PathtoPublish: '$(Build.ArtifactStagingDirectory)'
ArtifactName: 'drop'
publishLocation: 'Container'`

@Fisher-Joe Can you help me, how you resolved this issue as I am facing same thing

I was able to resolve this issue by building to the Artifact Staging Directory:
pool:
vmImage: 'vs2017-win2016'
variables:
solution: '**/*.sln'
buildPlatform: 'Any CPU'
buildConfiguration: 'Release'
steps:

task: NuGetToolInstaller@1

task: NuGetCommand@2
inputs:
restoreSolution: '$(solution)'

task: VSBuild@1
inputs:
solution: '$(solution)'
msbuildArgs: '/p:OutputPath=$(Build.ArtifactStagingDirectory)'
platform: '$(buildPlatform)'
configuration: '$(buildConfiguration)'

task: PublishBuildArtifacts@1
inputs:
pathtoPublish: '$(Build.ArtifactStagingDirectory)'
artifactName: 'MyServiceName'

I was able to resolve this by building to the Artifact Staging Directory:

pool:
  vmImage: 'vs2017-win2016'

variables:
  solution: '**/*.sln'
  buildPlatform: 'Any CPU'
  buildConfiguration: 'Release'

steps:
- task: NuGetToolInstaller@1

- task: NuGetCommand@2
  inputs:
    restoreSolution: '$(solution)'

- task: VSBuild@1
  inputs:
    solution: '$(solution)'
    msbuildArgs: '/p:OutputPath=$(Build.ArtifactStagingDirectory)'
    platform: '$(buildPlatform)'
    configuration: '$(buildConfiguration)'

- task: PublishBuildArtifacts@1
  inputs:
    pathtoPublish: '$(Build.ArtifactStagingDirectory)'
    artifactName: 'MyServiceName'

Hello
I get this error when I want to add Azure pipeline how can I fix it

##[warning]Directory '/home/vsts/work/1/a' is empty. Nothing will be added to build artifact 'drop'.
can someone help to get reslove the issue
Maven
Build your Java project and run tests with Apache Maven.
Add steps that analyze code, save build artifacts, deploy, and more:
https://docs.microsoft.com/azure/devops/pipelines/languages/java
trigger:

master

pool:
vmImage: ubuntu-latest
steps:

task: Maven@3
inputs:
mavenPomFile: 'pom.xml'
mavenOptions: '-Xmx3072m'
javaHomeOption: 'JDKVersion'
jdkVersionOption: '1.8'
jdkArchitectureOption: 'x64'
publishJUnitResults: true
testResultsFiles: '**/surefire-reports/TEST-*.xml'
goals: 'package'
task: CopyFiles@2
inputs:
SourceFolder: '$(agent.builddirectory)'
Contents: '**/.war'
TargetFolder: '$(build.artifactstagingdirectory)'
task: PublishBuildArtifacts@1
inputs:
PathtoPublish: '$(Build.ArtifactStagingDirectory)'
ArtifactName: 'drop'
publishLocation: 'Container'

pipeline link: https://dev.azure.com/edisonvaz009/EdisonVaz/_build?definitionId=1


Move K8s service account creation for GKE out of Terraform.

Due to Terraform limitations, the Kubernetes provider may give errors when making changes to the underlying cluster resource. The provider documentation indicates that Kubernetes resources should not be configured in the same root module or apply step as the cluster resources.

This change is


vendor/typicms/history/src/repositories

EloquentHistory
CacheDecorator
HistoryInterface
has colliding function name empty()
this resulting in:
syntax error, unexpected 'empty' (T_EMPTY), expecting identifier (T_STRING)
on PHP 5.6.14
Changing name to f.e. clear() seems to be good workaround and quick fix, but need another steps to bring back history clear function online.

You are right, but If you run composer update, it is already corrected. Am I wrong?


Config schema enforcement

This add (pretty light) JSON schema enforcement to the hub's config JSON.
Any currently working configuration should continue to work -- though if the config file contained invalid or unused properties, that file would no longer work.
We don't use required parameters for the most part in this file, because any of these settings could instead be set via environment variables.
I added test cases to cover sample configs and test enforcment of a bad JSON file.

This addresses #202

For the documentation release, I took a stab at the config page.  I need to expand it with defs of each section
https://docs.blockstack.org/storage/config-schema.html
There is a how to section also
https://docs.blockstack.org/storage/gaia-admin.html

Close this in favor of https://github.com/blockstack/gaia/pull/229 ?


Merge upstream 2.0.12 to circuitpython; fix to use new I2C driver

Merge upstream version 2.0.12.
Make changes necessary for new ESP-IDF I2C driver, which will be used in CircuitPython 9.2.0. This includes fixing the bugs described in https://github.com/espressif/esp32-camera/issues/689.

We don't pass around i2c_master_dev_handle_t instances when using the new i2C driver. Instead we just create and destroy them on each I2C read/write, and os sccb-ng.c had to be reworked to act like ports/espressif/busio/I2C.c. In the long run, perhaps this library should be more integrated into CircuitPython.
Also the upstream updates assumed v5.4, which is not out yet.
Tested on Memento with the standard PyCamera application. I have a few fixes for the library as well. It takes pictures!

@tannewt or @jepler do you want to review this or should I just merge it? It's a precursor for a PR to update the I2C driver in circuitpython.


Create another class of QGField that computes only QGPV and LWA

It happens that the stability of $U_{ref}$ can be affected by boundary conditions and resolution. If the user only wants the computation of wave activity, this new class will do the job.
This tasks would possibly involve refactoring/redesigning of existing classes. Think of a solution that would not affect existing users, i.e. not changing interface of any existing class.

Given #118 is done, this is no longer needed so I'll close this ticket.


SocialFish problem

one error in python sh: ./Server/ngrok: not executable: 32-bit ELF file
[] Ngrok URL:
[] Waiting for credentials...
[Fri Apr  6 10:52:48 2018] Failed to listen on <IP_ADDRESS>:80 (reason: Address already in use)
How to solve

Its because that port is in Use (80)
You can fix it by Changing the port number of SocialFish.py File..
Open it with an Editor and go through to the bottom and you'll see a line like this

system('./Server/ngrok http 3000 > /dev/null &')

Change the port number into something like 3000..
Eg :-  system('./Server/ngrok http 3000 > /dev/null &')
Also change this Line...to the same Port number used above

system("cd Server/www/ && sudo php -S <IP_ADDRESS>:80")

Eg :- system("cd Server/www/ && sudo php -S <IP_ADDRESS>:3000")
That's It !! ... Save the edited file and Start :+1:

try this one.... https://github.com/An0nUD4Y/SocialFish

it has many features....

Thanks


python save with specific fileformat

Hello,
I'm having problems with saving the data from the MFIA via python. I want to save it in a specific directory and under a specific name and fileformat, here .csv as an example. There is a bunch of other code so I'm trying to show you only the relevant parts.
sweeper = self.daq.sweep()
self.sweeper.set('save/directory', 'directory')
sweeper.set('save/save', 1)
sweeper.set('save/filename', 'name')
sweeper.set('fileformat', 1)
result = sweeper.read()
I get the following error: Error during sweep execution: Path fileformat not found.
I would really appriciate your help! Thank you!

From what I see the problem seems to be that you specify the wrong path specifier for the fileformat
The path is /save/fileformat (see https://docs.zhinst.com/labone_api_user_manual/modules/sweeper/index.html#node_save_fileformat) but you used fileformat
so probably changing sweeper.set('fileformat', 1) to sweeper.set('save/fileformat', 1) should fix the problem

Also, from what I can see, you initiate the save sweeper.set('save/save', 1) before setting the filename and format. Not sure if that is intended or by accident. You should call sweeper.set('save/save', 1) after you have set all the save specific parameters.
Random side note: Instead of using sweeper.set('fileformat', 1) you can also use sweeper.set('fileformat', 'csv') as the node is of an enumerated integer.

Thank you! I changed it accordingly but it still doesn't seem to work. I'm specifing the path with sweeper.set('save/directory', 'path') when setting the filename, could that somehow be problematic? Or maybe it's the placement? I've placed it like this:
sweeper.set('save/directory', 'path')
sweeper.set('save/filename', measurement['saveas_var'])
sweeper.set('save/fileformat', measurement['fileformat_var'])
start_time = time.time()
sweeper.execute()
sweeper.set('endless', 1)
while (time.time() - self.start_time) < (measurement['duration_var'] * measurement['durationunit_var']):
         time.sleep(0.05)
sweeper.set('endless', 0)
sweeper.set('save/save', 1)
result = self.sweeper.read()
sweeper.finish()

And can I access the webserver with using python (impedance.set('path', '/settings/LabOne/WebServer'))?

Thank you! I changed it accordingly but it still doesn't seem to work. I'm specifing the path with sweeper.set('save/directory', 'path') when setting the filename, could that somehow be problematic? Or maybe it's the placement? I've placed it like this:
sweeper.set('save/directory', 'path')
sweeper.set('save/filename', measurement['saveas_var'])
sweeper.set('save/fileformat', measurement['fileformat_var'])
start_time = time.time()
sweeper.execute()
sweeper.set('endless', 1)
while (time.time() - self.start_time) < (measurement['duration_var'] * measurement['durationunit_var']):
         time.sleep(0.05)
sweeper.set('endless', 0)
sweeper.set('save/save', 1)
result = self.sweeper.read()
sweeper.finish()

Now it worked, thank you for your help! The second question still stands though.

Now it worked, thank you for your help!

And can I access the webserver with using python (impedance.set('path', '/settings/LabOne/WebServer'))?

The LabOne APIs including the Python API run on the host computer and cannot have access to the files in MFLI/MFIA storage managed by the Webserver running on the device. So if the compensation files are saved on the MFLI/MFIA, the Python API can't understand the "path" to the device, and you need to download the compensation files and put them in a local folder in your computer and inform the impedance module by assigning the path to the local folder.
If the files are in your computer, then, you can assign the full path to the impedance module, for example if the files in the default Webserver setting directory in your computer:
impedance.set('path', 'C:\Users\[user]\AppData\Roaming\Zurich Instruments\LabOne\WebServer\setting')

One more question, is there a way to save the data as a .txt file, like when using the web-based application with only three columns (frequency, Impedance, Phase)?

One more question, is there a way to save the data as a .txt file, like when using the web-based application with only three columns (frequency, Impedance, Phase)?

The API always records the full data set. This can be in text-based format like .csv or tailored for numeric tools like .mat and .h5. Then, the user can extract the desired data from the full data set and process it similar to what the GUI does using basic post-processing.
In the case of Sweeper module for impedance, there is also .txt format which is tailored for ZView program. This has a smaller data set with focus on relevant impedance data.


fix: add support for new emojis in v5 (closes #1020)

Summary
This PR addresses issue #1020 by adding support for new emojis in Rocket.Chat v5, particularly handling ZWJ (Zero Width Joiner) emoji sequences in the MessageEmoji and MessageEmojiBase components.
Changes

Updated MessageEmojiBase to detect and render ZWJ sequences.
Modified the MessageEmoji component to ensure proper handling of new emojis in v5.

Testing

Tested various ZWJ emoji sequences to confirm they render correctly in Rocket.Chat v5.
Passed all existing tests and added new tests for ZWJ sequences.

Related Issue
Closes #1020

 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.Vansh5632 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.

 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.Vansh5632 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.

i have signed the CLA but it is showing only CLA not signed please look upon it


Should v-model work on components using both v-bind="$attrs" and v-on="$listeners"?

What problem does this feature solve?
Following this discussion, given a CustomInput component with this template:

  <input v-bind="$attrs" v-on="$listeners">

The following currently does not work:
<CustomInput v-model="myValue" />

The problem is that v-model on a component expects a value on input events, but a DOM Event is passed to the listener instead.
What does the proposed API look like?
Option 1: Make v-model smarter
Since I think it's unlikely that anyone will want to use v-model with a DOM Event, perhaps when v-model is used on components, it could check if the argument passed to the listener is instanceof window.Event. Then if it is, use event.target.value instead.
Option 2: Make v-on smarter
Maybe a non-enumerable property could be added to $listeners (e.g. __$listeners__: true, to help v-on detect uses of v-on="$listeners". Then in cases where the $listeners object is passed to v-on and it's used on an element, listeners relevant to v-model could be wrapped:
function (event) {
  listener(event.target.value)
}

But now we're throwing away data. If someone wants to access a keyCode - e.g. with @input instead of v-model - they're out of luck. However, if modifiers were also supported for v-on's object syntax, we could fix this by making .native disable the automatic wrapping behavior.

Thoughts? Are there strategies or implications I'm not considering? I'm definitely open to alternatives.

I would say for the first question, I don't care if it works automatically, but that it at least doesn't require another abstraction to accomplish. It should basically work close to how it would work if there was no component.
Literal Values
This is a huge part of the problem for me. I haven't been able to solve it because of a few cases where I want to use enum constants (I know these don't exist in JS, but they do in TypeScript, and I'm using a similar convention in a JS module) in option values.
$event.target.value is always a string, and thus, this solution would not work where the types of the value props should be literals/expressions:
<select>
	<option :value="someExpression">One</option>
	<option :value="someExpression">Two</option>
	<option :value="someExpression">Three</option>
</select>

My proposal
Maybe it would be possible to make v-model="$attrs" work similar to v-bind="$attrs" and be smart enough to not warn about direct prop mutation?

@RobertBColton in your example $event.target._value would work ;) just a hint.

@nickmessing Thanks so much, that does seem to work, but I can't seem to make the following work without a vue warning about directly mutating the value prop.
<template>
    <select class="combo" v-model="value" @input="update">
      <option v-for="(item, key) in items" :value="item">{{key}}</option>
    </select>
</template>

Ideally I wouldn't want to and do not want to add another value to the data section or similar as it would seem redundant. Is there a way of achieving this simple component without getting too verbose?

@RobertBColton, Github issues are not for asking questions, I am most of the time online in official chat (you can try stack overflow or forum too).

My first impulse when 2.4 got out was to try using v-on="$listeners" and v-bind="$props" with v-model (and got a little sad when it didn't work 😢).
It would be an amazing feature imo 👍
As for the two options, I'm more inclined to agree with the first (seems simpler as far as I understood).

@victor-am You could experiment implementing option 2 just for your component, like in this simple example.

I'm not sure this is desirable.
First off, $attrs and $listeners were added to make it easier to pass on these things from a parent component to a child component, to improve the situation for HOC-like components.
Making this work with v-model was not a goal in this original discussion.
Secondly, If we somehow make this work - that you can use v-model on a component with v-model on an element in that component, it feels to me like we essentially recreated inter-component two-way bindings which we originally dropped in Vue 2 for good reason.
Yes, under the hood it's technically still sending events and updating in the parent, but what happened under the hood never was the problem with .sync. It was that it made data flow harder to follow.
Do we re-create this weekness by allowing for a seamless v-model chain through many components down to some nested element in a form? It feels like we would to me.

@LinusBorg I think you might misunderstand the proposal. There's no state syncing between components, as v-model is only used on the parent component (check out the example in the issue description).
This comment suggested what you'd like to avoid, with state being maintained in both parent and child, but his use case and proposal isn't related to this feature request.

Oh right, sorry. I misunderstood completely.

v-model works differently for components and native elements: https://github.com/vuejs/vue/blob/dev/src/platforms/web/compiler/directives/model.js#L47
Therefore, I guess making v-model "smarter" is not trivial. The input event probably needs to be modified manually from the inner component to the outer component @input="$emit('input', $event.target.checked)" (or $event.target.value, etc.).
In my app, I'm wrapping a third party component and just adding custom style. I thought it should work with v-on="$listeners" v-bind="$attrs", but I noticed that adding :value="value" is also required.
I think it is a bit weird that a component with a declared v-model has input in $listeners but doesn't have value in $attrs. Thinking about it twice, I guess this is because value is automatically added to the accepted props due to v-model, and thus it is not included in $attrs (non recognized props) anymore.
However, this.value does not exist unless props: ['value'] is specifically declared. Perhaps value could be added to $attrs if it is not manually specified as a prop?

I think it is a bit weird that a component with a declared v-model has input in $listeners but doesn't have value in $attrs.

@frandiox I might have misunderstood what you mean, but I'm seeing value in $attrs.

Well, if the child properly declared the prop, I would expect to see it in $props, not $attrs, and that's what's happening.

@LinusBorg I think @frandiox is suggesting he's seen value automatically registered as a prop on a child component due to the use of v-model on a parent. I haven't noticed that behavior personally.

@LinusBorg @chrisvfritz You are passing value attribute manually so it is normal to have it in $attrs. However, I think normally you only pass v-model="foo" and it is internally expanded to :value="foo" @input="foo = $event", so there is no need to manually pass value. If you omit it, value is not included in $attrs and is not included in $props unless props: ['value'] is specified in the component (it is used internally in $vnode.data.props, though).
If it is declared as a prop, we all agree it should be included in $props (and it is). However, if it's not declared, shouldn't it be included in $attrs?

@frandiox Ah, thanks for the example. I agree it's strange that value doesn't appear in $attrs. That should be easily fixable by having v-model not assume that value is registered as a prop, adding to $vnode.data.attrs instead. That works in both cases.

I made some time this afternoon to fix this in #6327. 🎉  Reviews welcome. 🙂

Definitely something I want/need. Looks like the increasing popularity of HOC has made this a necessity.

So, I finally gave this proposal a proper investigation, and it ends up being more complicated than it seems. The problem lies within consistency between different input types. The current implementation in #6327 assumes a text input and ignores non-input events. While this simplifies wrapping plain text input fields, it makes the whole thing inconsistent and feels like a special-case convenience hack.
I managed to make it work with <input type="checkbox">, which requires the component to specify the model option, and then injecting the event type and prop type back into the component v-model handler. It's already messier than I hoped - but even if this works, it becomes prohibitively complex to mimic the full behavior of native v-model (e.g. checkboxes with array binding, radio, select...)
At this point I feel the implementation cost and the inconsistency outweighs the simplification this change would bring.
Taking a step back, the whole idea of component v-model is opening up finer-grained control over the two-way binding of form inputs, not simplifying it. Its job is expanding the sugar consistently into a pair of input and output, and letting you fill in the details. Mixing that up with $attrs and $listeners feels like magic where it doesn't belong.

All we want is to be able to create "fully transparent wrapper" components and add some functionality/customization on top... to have some semblance of inheritance in Vue.
Why all these edge cases and special syntax just for that? Why is it not the default behavior already?


ci(lint): enable errcheck, errorlint, gci, misspell, nonamedreturns and staticcheck linters

Signed-off-by: Matthieu MOREL<EMAIL_ADDRESS>

Your two comments are against revive's rules.
I removed it from this PR. It can be added in another.

Your two comments are against revive's rules.

Yes, I'm aware, that's why I mentioned adding specific exceptions for violations.

I removed it from this PR. It can be added in another if you want to apply it's rules.

Sounds fine. Thanks for contributing this PR!


Add Las Cruces, NM

Letter to Las Cruces Mayor Ken Miyagishima and City Council:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>My name is [NAME]. I am a resident of [CITY/NEIGHBORHOOD], and I am emailing today to demand an overhaul to our Las Cruces Police Department budget in light of the Black Lives Matter protests across the US, but specifically in Las Cruces.
WE ARE JOINING TOGETHER TO DEMAND THAT YOU, MY LOCAL OFFICIALS, WILL:
Vote no on all increases to police budgets;
Vote yes to decrease police spending and budgets;
Vote yes to increase spending on healthcare, education, housing, and community programs that keep us safe.
The militarized tactics that police departments are using against citizens is unacceptable and unwarranted. We have seen mounting evidence that police departments are racist and ineffective institutions that put citizens at risk of injury and death. This is amply demonstrated by the recent killing of Antonio Valenzuela in Las Cruces Police custody using a banned “vascular neck restraint"--the same restraint used to kill George Floyd in Minneapolis.
The approved 2020 FY budget allocated over 61% of our Public Safety fund to the Las Cruces Police Department. To ensure your dedication to our city and citizens, I demand that you defund the Las Cruces Police Department and start providing more support and funding towards our community efforts and organizations that serve our predominantly Black and Latinx communities. By doing so, I believe this will prevent further police brutality and violence in the future.
Black and Latinx communities are living in persistent fear of being killed by state authorities like police, immigration agents or even white vigilantes who are emboldened by state actors. According to the Urban Institute, in 1977, state and local governments spent $60 billion on police and corrections. In 2017, they spent $194 billion--a 220 percent increase. Despite continued police profiling, harassment, terror and killing in Black and Latinx communities, local and federal decision-makers continue to invest in the police, which leaves Black and Latinx people vulnerable and our communities no safer.
Where could that money go? It could go towards building healthy communities, to the health of our elders and children, to neighborhood infrastructure, to education, to childcare, to support a vibrant Black and Latinx future. The possibilities are endless.
We join in solidarity with the freedom fighters in Minneapolis, Louisville, and across the United States. And we call for the end to police terror.
Sincerely, [NAME] [PHONE_NUMBER] [EMAIL] [ADDRESS]
Some content borrowed from Albuquerque template.

Budget info confirmed here, p47: https://las-cruces.org/DocumentCenter/View/3082/FY2020-Budget-Adopted-book-PDF?bidId=
Edits in bold below
Letter to Las Cruces Mayor Ken Miyagishima and City Council:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Dear Mayor Miyagishima and the Las Cruces City Council:
My name is [NAME]. I am a resident of [CITY/NEIGHBORHOOD], and I am emailing today to demand an overhaul to our Las Cruces Police Department budget in light of the Black Lives Matter protests across the US, but specifically in Las Cruces.
I AM DEMANDING THAT YOU, MY LOCAL OFFICIALS, WILL:

Vote no on all increases to police budgets.
Vote yes to decrease police spending and budgets.
Vote yes to increase spending on healthcare, education, housing, and community programs that keep us safe.

The militarized tactics that police departments are using against citizens is unacceptable and unwarranted. We have seen mounting evidence that police departments are racist and ineffective institutions that put citizens at risk of injury and death. This is amply demonstrated by the recent killing of Antonio Valenzuela in Las Cruces Police custody using a banned “vascular neck restraint"—the same restraint used to kill George Floyd in Minneapolis.
The approved 2020 FY budget allocated over 61% of our Public Safety fund to the Las Cruces Police Department. To ensure your dedication to our city and citizens, I demand that you defund the Las Cruces Police Department and start providing more support and funding towards our community efforts and organizations that serve our predominantly Black and Latinx communities. By doing so, I believe this will prevent further police brutality and violence in the future.
Black and Latinx communities are living in persistent fear of being killed by state authorities like police, immigration agents or even white vigilantes who are emboldened by state actors.
According to the Urban Institute, in 1977, state and local governments spent $60 billion on police and corrections. In 2017, they spent $194 billion—a 220 percent increase. Despite continued police profiling, harassment, terror and killing in Black and Latinx communities, local and federal decision-makers continue to invest in the police, which leaves Black and Latinx people vulnerable and our communities no safer.
Where could that money go? It could go towards building healthy communities, to the health of our elders and children, to neighborhood infrastructure, to education, to childcare, to support a vibrant Black and Latinx future. The possibilities are endless.
We join in solidarity with the freedom fighters in Minneapolis, Louisville, and across the United States. And we call for the end to police terror.
Sincerely,
[NAME]
[PHONE_NUMBER]
[EMAIL]
[ADDRESS]

on it


Database cleanup does not work for Fanbox images

Presumably caused by #680.

Other database function may be broken as well. Please verify.

Okay... Will have a look at it though. But could you give more detailed description?

https://github.com/Nandaka/PixivUtil2/blob/3b026ae5d5e1eb8de19cce0a994b7cd7c087a3af/PixivDBManager.py#L753
The fanbox_master_post table you added in your PR is not checked.

Oh yes.. Sorry, I had never used that function before...
Will be fixing right away.
And you mentioned other database functions? Do you mind telling me what they are?

Basically I meant it would be good to go through PixivDBManager.py and see if any other functions there are affected by your changes. I only confirmed cleanup and interactive cleanup no longer work as expected.

Ah. I see. I had just looked at the cleanup function.... And actually I didn't know about this function. And I don't really use it so I didn't write a cleanup function for fanbox table.
And I have not touched other functions for regular tables, but I will double check.
May I ask how do you want the cleanup function to be? Same as the clean up function for regular images? Cuz it kinda was designed differently that it doesn't record download files, just some brief information, so it may be a little different...
If necessary I can use the file formats...
But please do tell me how you want it to be. Thank you.

I am not the only user of this software, so my opinion alone should not define what happens, but for my part, I would expect the cleanup function to detect when something is missing from the file system so that it can be redownloaded if necessary. Beyond that, I would expect all database management functions to work for Fanbox posts as they did before. Breaking changes to UX should not happen.
@Nandaka, please weigh in on this issue too. @bluerthanever, it may be pertinent to wait for @Nandaka before you start any corrections.

Okay... I understand... I will be waiting.
But before this, the program does not keep records of FANBOX download history. It just goes through all files the posts until the last page and skip downloading if the files exist...
It kinda does now, just in a really simple way. I will write it in the end of this post, if you are not interested....
But if it's just that you want to check the existence of each file every single time, I think a switch would do.... But I will wait.
How I implement this:

There's published date and updated date for each post, it's retrieved in json from API
When ever the post information is retrieved, its published date and retrieved date would be recorded in database
If the post is restricted, it will not be processed and the program goes to the next post.
If it's not restricted,  recorded updated date will be retrieved from database. And if it's empty or earlier than the new updated date, which means either the post is not downloaded before or that it's updated, the program would start processing it.
5.After processing it, the updated date would be written into database

And in case some files are like deleted or moved, that's why I added the delete FANBOX download history by post_id and member_id function...

I think it is useful to know with certainty which posts exist locally and which don't. Some artists move posts to higher tiers after a period elapses, which makes them impossible to redownload, so the sooner I can know that a post is missing from my file system, the better. As is, verifying manually seems to be the only option.

So.... the request for cleaning up fanbox database table still stands right?
I will be waiting for my trial.... haha.

Yes, I would like to have it.
It is not a trial. We are discussing how to solve the issue. The requirements should be defined before you start working, and the owner of the repository should have a say in them.

Haha. I am just joking.
It was pretty much based on my own requirement, for I am a user too, and nobody else was requesting, and also Nandaka-san himself doesn't use FANBOX....

Yep, I have no access to Fanbox as I don't really use it (free user here).

Since Nandaka has no strong opinions, I would suggest implementing working database cleanup for Fanbox posts as I said. This means detecting when files are missing on the file system and removing the corresponding entries from the database. It can be done either by checking the existence of each individual file or just the destination directory. I believe the latter would suffice.

Sorry for the late reply.
Hmm... I think the latter might do but will be low efficient, and not accurate, given the circumstance, that the filename formats might be quite different for different users, and some elements might be missing (not recorded in db), and using regular expression to iterate each file and compare would be low efficient,
And the former option, would require a new table to record every file downloaded. This should be higher efficient, to sacrifice space for speed. But if the post contains no images or files, and the user saves the texts by image info and they delete files, this would not be detected at all...
If it's okay with you, I will probably go with the former one, this might take some time though.

If you prefer to record each file, fine. It will not significantly increase the size of the database anyway. Take your time with the development. I do not need this fixed immediately.

Actually, I would not delete the posts I downloaded, or any images at all. so I don't really need to know which files do not exist locally, And I pretty much think posts which the author moves to higher tier will be downloaded when you run all posts through and hit it again, because the updatedDate would be changed.
But as you said, it's not my preference, but to satisfy the many users of this tool, And also I wasn't comprehensive enough.

This should have been fixed in #695. Not sure if you have used it so far and whether it meets your expectation.
Please leave a feedback. Thanks.


Security updates, updated to php 8.3

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (3.x@ab46bd8). Learn more about missing BASE report.

Additional details and impacted files
@@          Coverage Diff           @@
##             3.x      #34   +/-   ##
======================================
  Coverage       ?   72.15%
  Complexity     ?       65
======================================
  Files          ?       12
  Lines          ?      334
  Branches       ?        0
======================================
  Hits           ?      241
  Misses         ?       93
  Partials       ?        0

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


Fixed grammar in README.txt

Hi, I fixed a bit of grammar in the README.txt file.
Kindly check it out and save the changes if you find them useful.
Cheers!

Thanks!

Thanks a lot. My first commit in OSS!!!

Congratulations, @ja5h! :+1:


[BACKPORT][v1.2.1] Not possible to add PostgreSQL component to existing environment

Please backport task #2904 (PR #2902) to v1.2 branch.
DoD checklist

[ ] Changelog updated (if affected version was released)
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Case covered by automated test (if possible)
[ ] Idempotency tested
[ ] Documentation updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Backport tasks created / doesn't need to be backported

1.2.0 is out of support.


Disabling drafts leads to API error

Link to reproduction
No response
Describe the Bug
My admin panel for pages collection was showing only one page out of 12. The API however was returning all of them correctly. While searching for solutions I tried disabling the drafts on this collection to see if it brings any changes and the admin started showing all pages but now the API gives an error:
    at sanitizeQueryValue (/cms/node_modules/@payloadcms/db-mongodb/src/queries/sanitizeQueryValue.ts:43:13)
    at buildSearchParam (/cms/node_modules/@payloadcms/db-mongodb/src/queries/buildSearchParams.ts:95:46)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async parseParams (/cms/node_modules/@payloadcms/db-mongodb/src/queries/parseParams.ts:60:35)
    at async buildAndOrConditions (/cms/node_modules/@payloadcms/db-mongodb/src/queries/buildAndOrConditions.ts:29:22)
    at async parseParams (/cms/node_modules/@payloadcms/db-mongodb/src/queries/parseParams.ts:43:33)
    at async Function.buildQuery (/cms/node_modules/@payloadcms/db-mongodb/src/queries/buildQuery.ts:43:22)
    at async Object.find (/cms/node_modules/@payloadcms/db-mongodb/src/find.ts:39:17)
    at async find (/cms/node_modules/payload/src/collections/operations/find.ts:144:16)
    at async findHandler (/cms/node_modules/payload/src/collections/requestHandlers/find.ts:30:20)

To Reproduce
I really don;t know what caused the admin to only show one page in the first place. This is my config:
import {webpackBundler} from '@payloadcms/bundler-webpack'; // bundler-import
import {payloadCloud} from '@payloadcms/plugin-cloud';
import {mongooseAdapter} from '@payloadcms/db-mongodb';
import nestedDocs from '@payloadcms/plugin-nested-docs';
import redirects from '@payloadcms/plugin-redirects';
import seo from '@payloadcms/plugin-seo';
import type {GenerateTitle} from '@payloadcms/plugin-seo/types';
import path from 'path';
import {buildConfig} from 'payload/config';
import {slateEditor} from "@payloadcms/richtext-slate";

import Categories from '@/collections/Categories';
import {Media} from '@/collections/Media';
import {Pages} from '@/collections/Pages';
import {Posts} from '@/collections/Posts';
import Faqs from '@/collections/Faqs';
import Testimonials from '@/collections/Testimonials';
import ContentBlocks from '@/collections/ContentBlocks';
import Features from '@/collections/Features';
import UseCases from '@/collections/UseCases';
import TextContent from '@/collections/TextContent';
import Cards from '@/collections/Cards';
import Tokens from "@/collections/Tokens";
import Users from '@/collections/Users';
import BeforeDashboard from './components/BeforeDashboard';
import {FooterGlobal} from './globals/Footer';
import {HeaderGlobal} from './globals/Header';

const generateTitle: GenerateTitle = () => {
	return 'Anonymous proxies';
};

const mockModulePath = path.resolve(__dirname, './emptyModuleMock.js');

export default buildConfig({
	editor     : slateEditor({}),
	admin      : {
		user      : Users.slug,
		components: {
			beforeDashboard: [BeforeDashboard],
		},
		webpack   : config => ({
			...config,
			resolve: {
				...config.resolve,
				alias: {
					...config.resolve?.alias,
					express: mockModulePath,
					'@'    : path.resolve(__dirname, './'),
				},
			},
		}),
		bundler   : webpackBundler(),
	},
	serverURL  : process.env.PAYLOAD_PUBLIC_SERVER_URL,
	collections: [
		Users,
		Categories,
		Pages,
		Posts,
		Media,
		Features,
		Faqs,
		Testimonials,
	],
	globals    : [
		HeaderGlobal,
		FooterGlobal,
	],
	rateLimit  : {
		max       : 10000, // limit each IP per windowMs
		trustProxy: true,
		window    : 2 * 60 * 1000, // 2 minutes
	},
	typescript : {
		outputFile: path.resolve(__dirname, 'payload-types.ts'),
	},
	graphQL    : {
		schemaOutputFile: path.resolve(__dirname, 'generated-schema.graphql'),
	},
	cors       : [process.env.PAYLOAD_PUBLIC_SERVER_URL || ''].filter(Boolean),
	csrf       : [process.env.PAYLOAD_PUBLIC_SERVER_URL || ''].filter(Boolean),
	db         : mongooseAdapter({
		url: process.env.MONGODB_URI,
	}),
	plugins    : [
		nestedDocs({
			collections: ['pages', 'posts', 'categories'],
		}),
		redirects({
			collections: ['pages', 'posts'],
		}),
		seo({
			collections      : ['pages', 'posts'],
			generateTitle,
			uploadsCollection: 'media',
		}),
		payloadCloud(),
	],
});

Payload Version
2.1.1
Adapters and Plugins
No response

When versions are enabled Paylod will store documents in a seperate _versions collection. The list view or any find with drafts operations, will use the versions collection instead of the actual collection.
When turning on/off versions you also need to create a migration or run a script to resave all documents.
The only reason I can think for why you stopped seeing all your pages in the admin UI would be a read access control. Have you changed access.read that would cause drafts to not be returned?
It would be helpful if you could share your pages collection config. The network requests that have different results could also help reveal the problem if you can share that as well.
Also, you cut off the error in what you pasted starting with: at sanitizeQueryValue (/cms/node_modules/@payloadcms/db-mongodb/src/queries/sanitizeQueryValue.ts:43:13) I can see the stack, but not what was thrown so that doesn't tell me much.
Thanks!

Updated the initial message with the required info.

Hey @ciokan — did you recently upgrade to Payload 2.0?
When drafts are enabled, in order for pages to show up in the List view or be returned from the API with ?draft=true, the newest drafts for each page need to have latest: true on them.
This is written up in detail within the changelog here:
https://github.com/payloadcms/payload/releases/tag/v2.0.0
I'm pretty sure this is what you're seeing. There is a migration that you can run as part of our 2.0 migration guide that will solve this for you by going through and ensuring that each most recent draft of your Pages documents is marked as latest: true. But this is not an issue with Payload - rather, this is an issue with your data being inconsistent with what Payload expects.
I will close this for now, but if it turns out that this is indeed related to a Payload bug, we will need a reproduction repo in order to diagnose further.
We're happy to continue helping on this though!


How can I set map projection on EPSG:5514

I have data in KROVAK (EPSG:5514) for the Czech Republic and I would like to set the map projection to the same EPSG:5514. How can I achieve this?

Oh, Krovak. Krovak is the projection that has an exemption from criminal affairs? Every time someone mentions Krovak, everything is automagically arranged. Do you know this?

Imagine only if someone had a job where he had to work with Krovak, you could anything with that person, and he would still pardon you. Too bad there is no work for programmers anymore.

Thank you for your comments, but I am still looking for specific technical steps on how to set the map projection to EPSG:5514 for my data in KROVAK. Could you please advise me on how I can do this?

find the crs in espg.io
copy the proj4js defs
def and register this projections in your project
have fun


F5 isn't blocked by task-based problems

Having a launch.config that runs a watch task, doesn't seem to block when there are errors on the first run -- feels like a timing issue.
Steps:

Create an extension using our yo code template:  TypeScript (with or without Webpack)
Create an error in extension.ts, and close the file
At this point there should be no errors in the problems view
Press F5
This kicks off the watch task, but even though problems do show up in the problems view, the Extension Development Host window will still be launched.

Should be fixed now!


Bump CouchDB to 3.2.2 (#3369)

Update CouchDB to 3.2.2.
Update CouchDB doc refererences to 'stable' so that the links
don't point to outdated CouchDB versions.
Signed-off-by: David Enyeart<EMAIL_ADDRESS>

/azp

/azp run

/azp run

Hey @denyeart @ryjones
I just sent a message to the Azure DevOps team to see if they can figure out whats going on here

/azp run

Hmmm, it didn't work for this one. Dave, can you do a git commit --amend --no-edit and then force push to reset the CI, lets see if that works.


Certificate chain verification in circom

The electronic passports trust chain has two levels:

top-level certificates (CSCAs) are used to sign intermediate certificates (DSCs)
intermediate certificates (DSCs) are used to sign passports

Right now, we get the list of DSCs from the ICAO website and check if passports are signed by any of them. However, we strongly suspect this list is incomplete as countries have no obligation to transmit these intermediate certificates, and the top ones are supposed to be the authoritative source.
We should work on verifying the whole certificate chain, which follows the X.509 format. The MynaWallet team has a circuit in which they are parsing one with fixed indexes, but we should build a more modular chain certificate verifier that supports multiple signature algorithms and parsing any data from certificates. This would also be super useful to projects like zk AppAttest and zk DNS trust chain verification that also require X.509 chain verification.
Some work has been done by Richard and Sachin in halo2.
The endgame for us would be having only the root of a merkle tree containing the CSCAs from the masterlist on chain, then verifying the whole chain in the circuit.

From what I understood I have two questions :

by root of merkle tree means its the hash of all the documents from csca , right ?
Can you tell me what is the structure of the merkle tree for CSCA master list ?

update on the issue:
sha256 rsa ✅ -> code
sha1 rsa ✅ -> code
sha256 rsapss 🚧 -> issue
TODO (help wanted)
sha384 ecdsa (secpr1)

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
With my strong background in Rust, Solidity and JavaScript, I can apply my expertise in solving this issue

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have 4 years working in development environments, have encountered several challenges, requests and teams focusing in reusable and efficient code, so that gives me a better understanding of the code and how to solve problems efficiently.
How I plan on tackling this issue
I would focus in a comprehensive solution involving several key components: a certificate parsing module to extract relevant fields, a signature verification module supporting multiple cryptographic algorithms, and a chain building and verification module to validate the entire chain from end-entity certificates to root CSCAs. Additionally, we plan to integrate a Merkle tree for storing CSCAs on-chain, facilitating secure and efficient verification. The approach also includes leveraging existing work in halo2 for circuit implementation, ensuring scalability and performance. By prioritizing security, including certificate revocation checks and algorithm agility, this solution will also support related projects like zk AppAttest and zk DNS, providing a robust and future-proof certificate verification system.


Remove unnecessary error for id_token existing

Having id_token in the jwt is bad form and the provider should stop doing that, but it's likely not enough to throw an error and prevent the system from moving forward.

That error condition is very much intentional, when your provider is returning ID Tokens then you're using OpenID Connect. In that case, use the processAuthorizationCodeOpenIDResponse to process the response as an OIDC one.

That error condition is very much intentional, when your provider is returning ID Tokens then you're using OpenID Connect. In that case, use the processAuthorizationCodeOpenIDResponse to process the response as an OIDC one.

I'm working with a provider that is not offering OpenID connect for this flow, but includes that key. There is no reason to throw a fatal error here because they added an extra field to the token.

You get an ID Token, so use the processAuthorizationCodeOpenIDResponse method, it's an OpenID Connect Authentication Response that needs to be validated as such.

This is not an OpenID Connect provider though. They offer OpenID connect functionality for some users, but not the user type I'm trying to authenticate. They're missing other fields that are required for OIDC for this user type.
I'm sure I could work around this upstream. But then I need to go fork next-auth because this library is really unhappy that a field exists and thinks that it could be possible that I misconfigured things (instead of just having a debug-level warning.)

It actually looks like you even handle this situation (where a provider returns that field unexpectantly, but it's empty.) This library just complains (with a fatal error) if that field actually has anything in it. I'm just asking for the library to handle a full id field the same way it treats an empty one.

I understand your frustration but this is the correct behaviour. An ID Token is an OIDC artefact and is to be validated as per its validation definition when present.
If you were working with just oauth4webapi I'd suggest to clone() the response, update the body, instantiate a new Response, pass that to the oauth callback method.
Since you're working with @auth/*, use the means @auth/* built in to work around non-conform responses to do the above.
cc @balazsorban44 are the non-conform workaround interfaces exposed to userland?


ClientModel: Use BinaryData.Empty instead of bespoke solution

See PR comment: https://github.com/Azure/azure-sdk-for-net/pull/41016#discussion_r1442382912
This will involve:

Upgrading to System.Memory.Data 6.0.0 or higher
Upgrading to System.Text.Json 6.0.0, which System.Memory.Data brings as a dependency
Checking with PowerShell and Functions to ensure we don't upgrade above a version that they're using

This should be unblocked by https://github.com/Azure/azure-sdk-for-net/pull/46134


KON-656 Add KoTypeParameterDeclaration

Merge after https://github.com/LemonAppDev/konsist/pull/1439

Added KoTypeParameterDeclaration and methods for it (typeParameters, hasTypeParameter {})
Added KoUpperBoundsProvider and methods for it (upperBounds, hasUpperBound {})

Merge after https://github.com/LemonAppDev/konsist/pull/1439


Bug: Using a real router with Composition API example is missing flushPromises

Describe the bug
In the docs there is a code example for Using a real router with Composition API. Here is the code example in it's current state:
import { mount, flushPromises } from '@vue/test-utils'
import { createRouter, createWebHistory } from 'vue-router'
import { routes } from "@/router"

let router;

beforeEach(async () => {
  router = createRouter({
    history: createWebHistory(),
    routes: routes,
  })

  router.push('/')
  await router.isReady()
});

test('allows authenticated user to edit a post', async () => {
  const wrapper = mount(Component, {
    props: {
      isAuthenticated: true
    },
    global: {
      plugins: [router],
    }
  })

  const push = jest.spyOn(router, 'push')
  await wrapper.find('button').trigger('click')

  expect(push).toHaveBeenCalledTimes(1)
  expect(push).toHaveBeenCalledWith('/posts/1/edit')
})

flushPromises is imported but is not used anywhere.
How should that be handled? Should the flushPromises be called after the button trigger event? I'd be happy to put up a PR for it!

Hi @evandroguedes
I think flushPromises is not needed here, and can simply be removed.
A PR fixing this would be great, thanks!


ensure saved prerendered assets are stored as file and not as a folder

closes #9560
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Does this really solve the problem or just shift it elsewhere? If you have api/+server.js and api.json/+server.js then there's a conflict with this PR.

Yeah, I just realized that this does not relly fix the issue.. It must probably be a unique file ending (.sveltekit_prerender ?).
And also the code that references the generated file needs to know about this file ending or esle it tries to import a file that does not exists.

Or use the __data.json approach like it already happens for load functions? Shouldn't prerendered load and GET be the same except that GET could return not just json?
So maybe rewriting GET responses always to __data.txt?

closing per https://github.com/sveltejs/kit/issues/9560#issuecomment-1511969203


refactor(recipe): improve code readability

PR implements new argument passing for page navigation and functions to reduce duplicate code in recipe_bloc, removes an unused dependency and standardises ingredient json fields ahead of the import/export feature implementation.

I just had a peek and it looks good, will review the PR later 👍.


Fix fallback filename content type inference

When using a fallback filename (#1226), the path used when setting the response's Content-Type is an empty string, so all responses are returned as application/octet-stream. This changes the content type update to refer to the the fallback filename, if given, or the path.

@jmvrbanac serves me right for working on this too late in the evening. Thanks for catching that; updated!

@kgriffs fine with me! Can I help in some way? Should I close out this PR?

Thanks @bendemaree for your world on this, it is very appreciated!

The new PR reflecting the joint effort is out: https://github.com/falconry/falcon/pull/1348
Thanks to everyone involved for the genuine effort in fixing this bug! I have merely put the pieces together and added some unit tests & documentation notes.

[superseded by  #1348]


Transformable

It may actually be useful now that we have the RawRepresentable extension.
Something like:
public protocol Transformable: Decodable {
    typealias RawDecodableType: Decodable
    static func transform(rawValue: RawDecodableType) throws -> Self?
}

Purpose: Automatically throw an useful error if transform fails. This would replace the error for the RawRepresentable extension.
Note: Tempted to call it Transformable but probably shouldn't

Though perhaps RawRepresentable is the protocol we need


test(sdk): make all potentially-stateful parametrize() args lazy, to avoid reusing them between tests

Description
Here's a sneaky bug: consider this (slightly dumb, but correct) function.
def retry_forever(f, *args, **kwargs):
    while True:
        try:
            return f(*args, **kwargs)
        except Exception:
            pass

Here's a simple test for it:
@pytest.mark.parametrize(
    "fn",
    [
        Mock(),
        Mock(side_effect=[Exception(), None]),
        Mock(side_effect=[Exception()] * 100 + [None]),
    ],
)
def test_retry_eventually_succeeds(fn):
    retry_forever(fn)

I run this, and... it... hangs.
Why?
Hint: elsewhere, I defined...
@pytest.fixture(autouse=True, params=[True, False])
def some_parametrized_fixture(request):
    return request.param

Solution: if there are multiple @parametrizes relevant to a test, then the parameters get reused between runs. Therefore, the parameters must not be stateful. Mocks are stateful (e.g. they have side-effect lists, and remember method calls).

oh 😮


Add commas to number

Adds a comma after every three digits to the left of the decimal point.

Can you resolve the conflicts? I will remove package-lock.json.

Still there is conflicts in this pull request !


startxfce fails on focal (chromebook pixel 2013)

name: focal
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/focal...
crouton: version 1-20200724165050~master:08dac970
release: focal
architecture: amd64
xmethod: xorg
targets: xfce
host: version 10895.78.0 (Official Build) stable-channel link
kernel: Linux localhost 3.8.11 #1 SMP Wed Oct 3 23:30:23 PDT 2018 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/focal...

Please describe your issue:
startxfce blanks the screen, then crashes and returns me to the terminal:
a0293089d98433d4a1c848cfaa815b5f6480fb4438fa821c8613" noinitrd vt.global_cursor_default=0 kern_guid=3c0bb525-6ada-ec48-b484-db69b9c7ae57 add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic iTCO_vendor_support.vendorsupport=3  gpt
Build Date: 24 June 2020  06:00:21AM
xorg-server 2:1.20.8-2ubuntu2.2 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.38.4
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
        (++) from command line, (!!) notice, (II) informational,
        (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Sat Aug 29 22:40:59 2020
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
Error org.freedesktop.DBus.Error.ServiceUnknown: The name org.chromium.LibCrosService was not provided by any .service files
method return time=1598755260.984997 sender=:1.6 -> destination=:1.56 serial=1674 reply_serial=2
   boolean true
xf86EnableIOPorts: failed to set IOPL for I/O (Operation not permitted)
pkexec version 0.105
Error executing command as another user: Not authorized

This incident has been reported.
crouton: version 1-20200724165050~master:08dac970
release: focal
architecture: amd64
xmethod: xorg
targets: xfce
host: version 10895.78.0 (Official Build) stable-channel link
kernel: Linux localhost 3.8.11 #1 SMP Wed Oct 3 23:30:23 PDT 2018 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Wrong __data_start/_end pair
Aborted (core dumped)
/usr/bin/xinit: connection to X server lost

waiting for X server to shut down Hangup
Error org.freedesktop.DBus.Error.ServiceUnknown: The name org.chromium.LibCrosService was not provided by any .service files
method return time=1598755262.432335 sender=:1.6 -> destination=:1.58 serial=1675 reply_serial=2
   boolean true
(II) Server terminated successfully (0). Closing log file.

If known, describe the steps to reproduce the issue:
startxfce

I get the same behavior if I directly run /usr/bin/xinit /usr/local/bin/croutonxinitrc-wrapper /etc/xdg/xfce4/xinitrc -- /etc/X11/xinit/xserverrc :1 . However, if I run xinit under gdb, it works fine and launches me into xfce. (I notice that glxinfo works but glxgears aborts, but that may be a different issue)

I can reproduce it under strace, though: the process aborting seems to be xbindkeys:
15990 execve("/usr/bin/xbindkeys", ["xbindkeys", "-fg", "/etc/crouton/xbindkeysrc.scm"], 0x7f5eb5637a58 /* 20 vars */ <unfinished ...>
15990 --- SIGABRT {si_signo=SIGABRT, si_code=SI_TKILL, si_pid=15990, si_uid=1000} ---


Can ADLOPL3 synthesizer increase the possibility of arithmetic?

ADL OPL3-YMF262 4op can also add 2 more arithmetic units 4op+2op=For example, Yamaha DX7 6op or 4 more arithmetic units 4op+4op=OPX sound chip 8op~ Is it possible?

In the same way that pseudo-4op (2x2op) is possible, so are 6op and 8op arrangements.
The limit is that these arrangements can be layered additively only, not intermodulated.
So you'd fall short of the full abilities of the DX7.
The synth doesn't support this possibility outright, but you would be able to obtain similar effect by layering 2 of them in a DAW.

What month is the next update?


Connect Four Game in JS

Is your feature request related to a problem? Please describe.
The problem is to create a Grid-based Game using HTML, CSS, and JavaScript.
Describe the solution you'd like

The would-be 6 rows and 7 columns, and it's a two-player game.
The first one to connect any four consecutive cells horizontally, vertically or diagonally, will be declared the winner.

@Ayush7614  Can I work on this as a part of HCSSoC'21


Do you support python? If so, please include in your docs

Either you do or don't support python. If you do, can you please consistently document how the API works.

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

ID: 788014ae-dc3b-f509-3dbd-933eb5a423a4
Version Independent ID: 250d17de-f226-d3e3-7071-3ecb14da018e
Content: Grant limited access to data with shared access signatures (SAS) - Azure Storage
Content Source: articles/storage/common/storage-sas-overview.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@lesterlitch - Thanks for your feedback! We are working on extending all of our code examples in task-based content to other programming languages and to PowerShell and Azure CLI. We hope to have this work completed in the upcoming months.
#please-close


Should I use paypal checkout or paypal?

General information

SDK version:
3.14
Environment:
Sandbox
Browser and OS
Not relevant

Issue description

As a developer working on an extension to support this for our stores, I'm confused as to why two seperate API's are offerd for paypal. We're currently using braintree-web.paypal but all official v3 docs reference braintree-web.paypalCheckout. paypal seems to work for what we aim to accomplish here, but I wouldn't know why I would want to use one over the other.
Is there a chance someone could help clear this up for me and in doing so we might be able to update documentation to reflect these differences.

Hi @Senjai,
The PayPal Checkout component is the preferred method for integration. The PayPal component will continue to work, but any new features will only be added to the PayPal Checkout component.
The basic difference between the two is that the PayPal component requires you to manage any UI for preventing your customer from interacting with your page while the PayPal popup is open, while the PayPal Checkout component (along with checkout.js) manages all the UI for you.
Thanks for the feedback. We'll do some work to make the documentation clearer.


Querystring parsing of dates inconsistent with Body parsing

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of Nancy
[x] I have verified if the problem exist in both DEBUG and RELEASE mode
[x] I have searched open and closed issues to ensure it has not already been reported

Description
Using TryParse on the DynamicDictionaryValue object doesn’t work if you send in like so localhost:1234/users/date?from=2016-01-01T00:00:00.0000000+00:00 for example var date = this.Request.Query.from.TryParse<DateTimeOffset>(new DateTime(1979, 01, 01));
Take the time zone suffix off and it's fine, same with using Bind<T> however the dates are of DateTimeKind.Unspecified
Binding dates with a suffix from a json body appear to be correct. If the suffix is applied the DateTimeKind value is DateTimeKind.Local, without and its DateTimeKind.Unspecified
Steps to Reproduce

System Configuration

Nancy version: 1.4.3
Nancy host

[ ] ASP.NET
[ ] OWIN
[x] Self-Hosted
[ ] Other:

Other Nancy packages and versions: Nancy.Hosting.Self 1.4.1
Environment (Operating system, version and so on): OSX, Mono 4.2.3
.NET Framework version:
Additional information:

What makes you say DateTimeKind.Local is correct without the offset? Also, what happens if you pass 2016-01-01T00:00:00.0000000Z?

It's correct because it read the suffix and adjusted the resulting time to a local time however the question for all of this is what should Nancy return from any parsing. Local, UTC or Unspecified.
Using 2016-01-01T00:00:00.0000000Z in the querystring & body results in DateTimeKind.Local

This is really something that the user needs to specify; i.e. what input are you expecting? local? universal? It's impossible for Nancy to know.

How would they input that?
On 26 April 2016 at 12:49, Kristian Hellang<EMAIL_ADDRESS>wrote:

This is really something that the user needs to specify; i.e. what input
are you expecting? local? universal? It's impossible for Nancy to know.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2426#issuecomment-214715245

Actually looking into it more, the body deserializer adjusts to UTC not Local

Fixed by https://github.com/NancyFx/Nancy/pull/2454


Support Pug HTML tags with hyphens

Thank you for this plugin, and special thanks for adding Pug support! Writing Pug is a delight compared to HTML, and I'm glad to see it getting attention in Nova.
I've noticed that while this extension supports Pug syntax, it seems to stop highlighting where tag names contain hyphens:

Could hyphens be added so custom multi-word components get fully highlighted?
Thank you ❤

Hi there! Thank you too for enjoying nova and the extension!
I'll give it a look but that's probably due to the PUG extension, my extension just reference that other syntax.

Hi there! Thank you too for enjoying nova and the extension!
I'll give it a look but that's probably due to the PUG extension, my extension just reference that other syntax.

A took a look at it and I confirm the problem is not on my side.
This is a vue component using pug that shows the same problem you are pointing out.

As you can see placing the same code in a plain pug file the result is the same...

You should reach out the folk behind Nova's Pug extension, which provides the syntax highlighting also for Nova's Vue.
https://github.com/kpmcguire/nova-pug-syntax/issues
I close the issue.

A took a look at it and I confirm the problem is not on my side.
This is a vue component using pug that shows the same problem you are pointing out.

As you can see placing the same code in a plain pug file the result is the same...

You should reach out the folk behind Nova's Pug extension, which provides the syntax highlighting also for Nova's Vue.
https://github.com/kpmcguire/nova-pug-syntax/issues
I close the issue.


Cypress fails on CI (github action)

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Create a new project, deploy it with the github action, it will fails as it seems there's a bug with the latests NPM version.
This is the error
 npm ERR! 'npm ci' can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with 'npm install' before continuing.
FYI: I fix this issue changing the node version of the Cypress state to node-version: 16.10.0 (last v7 npm version) and it works.
Expected Behavior
Not fail in the cypress step on github action.
Actual Behavior
Cypress step of the build fails.

Will probably be fixed by #102

Closed by #102

I don't think this fully fixed it 😢 I just created a new project by running npx create-remix --template remix-run/indie-stack and following the readme. npm ci works locally but fails on CI.

This got me past that npm ci error.
index d01b1b0..a45574c 100644
--- a/.github/workflows/deploy.yml
+++ b/.github/workflows/deploy.yml
@@ -100,7 +100,7 @@ jobs:
       - name: 📥 Download deps
         uses: bahmutov/npm-install@v1
         with:
-          useLockFile: false
+          useLockFile: true

       - name: 🛠 Setup Database
         run: npx prisma migrate reset --force

ah yep, the first run won't have a lock file until the deps are installed due to https://github.com/remix-run/indie-stack/pull/79, personally i think it's fine to have them by default and if someone wants to use a diff package manager, they'll know what to do

@mcansh Could we add some output to the CI about this to avoid confusing users?

something like? I'm not sure this is better than just having one by default as we still use npm in the Dockerfile
- name: Check for lockfile
  run: |
    FILE=package-lock.json
    if [ ! -f "$FILE" ]; then
      echo "$FILE does not exist. Please run `npm install` before continuing"
      exit 1
    fi

I did "npm install" first. I did have a package-lock.json.
The step that's failing is complaining the lock file is out of sync. I was
able to fix that by adjusting an earlier step and telling it to use the
lock file to check out the dependencies.
I think by explicitly disabling the lock file, it would check out different
versions of packages and then that cuses "npm ci" to complain about them
being out of sync.
On Fri, Jun 10, 2022, 8:15 AM Logan McAnsh @.***> wrote:

something like? I'm not sure this is better than just having one by
default as we still use npm in the Dockerfile

name: Check for lockfile
run: |    FILE=package-lock.json    if [ ! -f "$FILE" ]; then      echo "$FILE does not exist. Please run npm install before continuing"      exit 1    fi

—
Reply to this email directly, view it on GitHub
https://github.com/remix-run/indie-stack/issues/103#issuecomment-1152469464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACPLB3CE2KAKZNQRYCKLL3VONLYFANCNFSM5YJ22BFQ
.
You are receiving this because you commented.Message ID:
@.***>

I think by explicitly disabling the lock file, it would check out different versions of packages and then that causes "npm ci" to complain about them being out of sync.

something is definitely up, but i don't think that's it as opting out of the lock file uses npm install and not npm ci

interesting, i just created a new app and the cypress job passed https://github.com/mcansh/grunge-stack-test/runs/6970287593?check_suite_focus=true

Closing this issue as no response was provided.
Feel free to re-open it if it's still a problem.


unable to detect version control system for code.google.com

Looks like someone ran that migration too quickly :)
go version
go version go1.5.2 linux/amd64

go get code.google.com/p/mahonia
package code.google.com/p/mahonia: unable to detect version control system for code.google.com/ path

See https://go-review.googlesource.com/#/c/18952

@dz0ny code.google.com has shut down.  I don't think there is anything we can do to fix this issue.  Please feel free to reopen if you think there is.


Fix scans being registered twice in Command::Run

Hi, relatively minor issue I ran into while testing, but I noticed that scans for each type (pool/ballot/refresh/etc) were being registered each time I ran oracle-core run for the first time (with scanIds.json removed). After testing it with other commands that also create a new OraclePool that were not having this issue, it seems the cause was that the HTTP Get API thread and the main thread in Command::Run were both running in parallel and registering scans twice.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 65.163%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/main.rs
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/src/main.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1964

Relevant Lines:
3014

💛  - Coveralls


Attribute Error: 'module' object has no attribute EPD

I'm getting this error on any if the supplied code that attempts to access  epd7in5_V2.EPD() (eg line 36 in helloworld.py).
I'm unsure why this is occured as I had tested out the project previously before installing on a new Raspberry Pi. All Waveshare libraries and installation instructions were followed along with instructions provided by Tom on this GitHub. Screen in Waveshare 7.5in V2.
Has anyone encountered this before or reason why it could be occurring?
Thanks,
A

I found the solution and writing here in case anyone else has this problem. Don't install setup.py twice. It seems to cause a dependency issue.


Ordering by project using '+project'

In previous releases, there was an option to list tasks per project by using ultralist list +project but that doesn't work anymore. Is there a specific reason to have removed this type of filtering? I know there now is the ultralist list project:project but that's harder to type of course.

this was part of a larger rewrite of how filters work.  I'll look into supporting how it used to work.  You could always pipe into grep or fzf!

I've for now added a simple function:
function ull () {
  ultralist list project:"$1"
}

That keeps the original color output.

gonna close for now.


[NO-TICKET] AWS Conformance pack module

Change description

We would like to deliver aws conformance pack as part of aws config starter script to better align with aws service delivery validation process.

Test Demo:

Type of change

[ ] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

@sgtursun Can you add Terraform docs support to this repo please?
This repo has a great example of terraform-docs, and once it's implemented it's crazy easy to maintain.