Lambda Consumer with KPL Producer needs more explanation

I followed the guide here - https://github.com/awsdocs/aws-lambda-developer-guide/blob/master/doc_source/with-kinesis-create-package.md - for making a function to process messages from a Kinesis stream.
This worked fine for messages that were produced individually, but when several messages were produced in a short time-frame, the latter messages were prefixed by extra characters, including the partitionKey and some pipe characters.
After some research, I realised that I needed to either disable the Kinesis Aggregation using the producer library (which I chose) or I needed to use https://github.com/awslabs/kinesis-aggregation
It would be enormously helpful if there were some warnings in the documentation about creating a Kinesis function on the subject of aggregation.

Hi,
Thanks for the detailed report! I'll add a note about aggregation for now, and look into providing sample code that uses the aggregation library.
We've also recently added a Kinesis-based sample application that may be of interest-
https://github.com/awsdocs/aws-lambda-developer-guide/tree/master/sample-apps/list-manager
Thanks,
-Michael at AWS

Added a note here-
https://docs.aws.amazon.com/lambda/latest/dg/with-kinesis-create-package.html


arrow key is replaced by number wrongly

VSCode Version:1.15.1
OS Version:
Ubuntu 16.04.1 LTS

Steps to Reproduce:
1.using shortcut key to delete a line(ctr+shift+k)
2.typing the arrow key will be replaced by number(eg: 'up' changing to '8','down' changing to '2', 'left' changing to '4', 'right' changing to '6')

Reproduces without extensions: Yes/No

@liuchanchen I am not 100% sure I understand what you mean. Do you mean you are pressing the arrow keys and you get numbers inserted? This almost looks like one would be pressing the arrow keys on a numpad with NumLock on... or something along those lines.
Are you perhaps using a remote desktoping software?
Here is a quick troubleshooting guide for linux keybindings: https://github.com/Microsoft/vscode/wiki/Keybindings#troubleshoot-linux-keybindings
Does any of those steps apply to you ?

Actually,  I used hyper-v to run ubuntu and win 10 as host

@liuchanchen Would then the very first bullet point in the troubleshooting guide help?


/emoji-data/img-apple-64/1f604.png gives 404

Hi
I keep getting 404 on some images located in /emoji-data/img-apple-64/IMAGENAME.png
Can u please upload emoji.js also, it is hard to debug only in emoji.min.js
But Thanks alot for your awesome work.
Best regards Hac

Did you set your sheets path's?
https://github.com/needim/wdt-emoji-bundle/blob/master/demo/index.html#L210
I think you have some configuration issues.
Here is unminified version of emoji.js;
https://github.com/iamcal/js-emoji/blob/master/lib/emoji.js

Thanks for your reply.. Yes I did set the sheets path's
I looked at the emoji.js and found out that it was line:
225
var img = emoji.data[idx][7] || emoji.img_sets[emoji.img_set].path+idx+'.png';
or / and
240
img = emoji.img_sets[emoji.img_set].path+variation_idx+'.png';
giving me the trouble
I did not need them so I changed to
225
var img = emoji.data[idx][7];
AND
img = '';
By that not saying they are wrong, but in my case I dont need them.
Thank you very much. You have been very help full


Configure VectorDB at a YAML file.

When you configure VectorDB you can use all vectorstores from LlamaIndex.
But, we need to configure it, at least collection name, vector DB URL.
And maybe, it is really easy to understand when we do not configure all vectorDB one by one inside the nodes & modules. It is easy to configure outside node_lines key.

close at #874


Memory leak?

Describe the bug
A memory leak occurs in the garbage collector?
Expected Behavior
Everything to work properly.
Current Behavior
I get a message box saying:

Fatal error in GC: Too many heap sections.

If I check in task manager I get a memory usage of 4,741.4MB for v-analyzer.exe which is absolutely massive.
Reproduction Steps

Install VSCode extension.
Use specified version from releases.
Bug happens.

Possible Solution
No response
Additional Information/Context
No response
Environment details (v doctor output)
V full version: V 0.4.0 45e6e7d
OS: windows, Microsoft Windows 11 Pro v22621 64-bit
Processor: 12 cpus, 64bit, little endian,
(everything else is valid, and thus removed from this message as it was unnecessary)
Editor name
VSCode
v-analyzer Version
v-analyzer version 0.0.1-beta.1
VS Code Extension Version
vscode-v-analyzer-0.0.2

Forgot to mention this happens when the extension says that it is indexing the V standard library.

Have you checked what has been described in #12 ?

I did not, but having taken a look now at the config I did not have a custom vroot. Setting one to where the previous root was with the specified fixes seems to work tho.


doc: fix backslashes in application dev doc

Single backslashes outside of a code-block are treated as escape
characters, so need to double up to get a single backslash rendered in
the generated HTML.
Signed-off-by: David B. Kinder<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6737   +/-   ##
=======================================
  Coverage   54.31%   54.31%
=======================================
  Files         444      444
  Lines       42179    42179
  Branches     7995     7995
=======================================
  Hits        22911    22911
  Misses      16060    16060
  Partials     3208     3208

Continue to review full report at Codecov.

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


Remove long label exclusion filters

I think these can be removed now that text wrap is enabled?

LGTM


use utils in knative/pkg

Signed-off-by: runzexia<EMAIL_ADDRESS>knative/eventing#2884

/hold for https://github.com/knative/eventing-contrib/pull/1619#pullrequestreview-502219256

@n3wscott @zroubalik
Should we pin kafkasource to https://github.com/knative-sandbox/eventing-kafka?
If it's need, I can help

@n3wscott @zroubalik
Should we pin kafkasource to https://github.com/knative-sandbox/eventing-kafka?
If it's need, I can help

@runzexia is that repo the proper source of KafkaSource now? I haven't followed the Kafka related changes recently.

@n3wscott @zroubalik
Should we pin kafkasource to https://github.com/knative-sandbox/eventing-kafka?
If it's need, I can help

@runzexia is that repo the proper source of KafkaSource now? I haven't followed the Kafka related changes recently.

https://github.com/knative/eventing-contrib/pull/1619#issuecomment-707561521
yes, as mentioned

@runzexia is that repo the proper source of KafkaSource now? I haven't followed the Kafka related changes recently.

knative/eventing-contrib#1619 (comment)
yes, as mentioned

@runzexia thanks, we should change that as well then 👍  Would be great if you can look at that.

@runzexia thanks, we should change that as well then 👍 Would be great if you can look at that.

ok will try it ☺

/unhold

/lgtm
/approve


Correction for raw affiliation IRSTEA, UMR G-EAU, University of Montpellier, Avenue Jean-François Breton, 34000 Montpellier, France; LBE, University of Montpellier, INRA, 02, Avenue des Etangs, 00 Narbonne, France

Correction needed for raw affiliation IRSTEA, UMR G-EAU, University of Montpellier, Avenue Jean-François Breton, 34000 Montpellier, France; LBE, University of Montpellier, INRA, 02, Avenue des Etangs, 00 Narbonne, France
raw_affiliation_name: IRSTEA, UMR G-EAU, University of Montpellier, Avenue Jean-François Breton, 34000 Montpellier, France; LBE, University of Montpellier, INRA, 02, Avenue des Etangs, 00 Narbonne, France
new_rors: 022m8c549;051escj72;020g0rw41
previous_rors: 022m8c549;051escj72
works_examples: W3033107982
contact: a8626c3c61b31897c0cfc6bbe4fe385d:889d9b5c1e9efc8c4e61d0938399ffeeefd3 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Add Plane::copy_from_raw_u8().

This uses iterators and zip to avoid bounds checks.

I couldn't measure an improvement timing the whole encoder.


No error / unrelated error when destructuring and assigning a default value on arguments.

Take this code
const foo = ({a = 1}) => true

This is an error in javascript: https://repl.it/@aykev/LastingSimpleDebugging-1
TypeError: Right side of assignment cannot be destructured

However the error flow presents doesn't guide the user towards the real problem:
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoALgTwA4FMwDqAFgIbq4BuuATmALxgDO61AlgHYDmYAPmOwFcAtgCMaAblQYc+AArU42RvTABvHqgCQjOENzoiHTgC5CpclWoAaVDwC+k1AGM47ZmChw4KgBSqdegZGKgDkAIwATADMIXam8oqMAJT0AHxgLAK4ji5u6GAiJLQMPgH6hlymxGSUNKGRMSl06ZnZqEA
Spotted on: https://github.com/facebook/flow/issues/6135

Flow does point you to the correct problem:
const foo = ({a = 1}) => true;
foo();

2: foo();
   ^ Cannot call `foo` because property `a` is missing in undefined [1].
References:
2: foo();
   ^ [1]

Am I missing something?

Oh, try the flow.org/try example, not the repl.it example. That's the specific case I'm talking about

Duplicate of #2093


Need AUTOPLAY for carousel.

Description
Please add AUTOPLAY feature to the carousel component.
Additional context
No response

You can try this out on the Edge channel: https://ui.nuxt.com/dev/components/carousel#autoplay


Fix @smithy/protocol-http import in HttpApiKeyAuth spec

Issue #, if available:
Description of changes:
Without this fix, building the generated SSDK fails:
$ concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types'
$ tsc -p tsconfig.es.json
$ tsc -p tsconfig.types.json
$ tsc -p tsconfig.cjs.json
[build:types] src/middleware/HttpApiKeyAuth/index.spec.ts(4,29): error TS2307: Cannot find module 'smithy/protocol-http' or its corresponding type declarations.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[build:types] yarn run build:types exited with code 2
[build:es] src/middleware/HttpApiKeyAuth/index.spec.ts(4,29): error TS2307: Cannot find module 'smithy/protocol-http' or its corresponding type declarations.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[build:cjs] src/middleware/HttpApiKeyAuth/index.spec.ts(4,29): error TS2307: Cannot find module 'smithy/protocol-http' or its corresponding type declarations.
[build:es] yarn run build:es exited with code 2
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[build:cjs] yarn run build:cjs exited with code 2
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed.
Exit code: 1
Command: /Users/jhecking/.nvm/versions/node/v16.19.0/bin/node
Arguments: /opt/homebrew/Cellar/yarn/1.22.19/libexec/lib/cli.js run build
Directory: /Users/jhecking/path/to/project
Output:

Smithy version used:
smithyVersion=1.33.0
smithyTypeScriptVersion=0.17.0
smithyGradleVersion=0.7.0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the fix!


when multiprocessing.Manager is used, kivy will repeat create new window

Versions

Python:  Python 3.5.2
OS: Windows 10
Kivy: v1.10.0
Kivy installation method: follow the specification on https://kivy.org/docs/installation/installation-windows.html

Description
try to use multiprocessing.Manager to communicate between servers, It's OK to run in PyCharm, but after I follow the Guide(https://kivy.org/docs/guide/packaging-windows.html#packaging-a-simple-app) to build with PyInstaller on windows, the output exe will just repeat create new windows again and again ...
Code and Logs
🔢 the following code is simplified from my project, but it's enough to repeat the case
from kivy.app import App
from kivy.uix.button import Button
import multiprocessing

class T007_App(App):

    def build(self):
        return Button(text='test', on_release=self._change)

    def _change(self, button):
        if 'test' == button.text:
            button.text = 'joke'
        elif 'joke' == button.text:
            button.text = 'test'

if '__main__' == __name__:
    manager = multiprocessing.Manager()
    T007_App().run()

🔢 the log is OK, but there will be tens of them, until python hangs(or blocks)
[INFO   ] Logger: Record log in C:\Users\Administrator.kivy\logs\kivy_18-06-14_0.txt
[INFO   ] Kivy: v1.10.0
[INFO   ] Python: v3.5.2 (v3.5.2:4def2a2901a5, Jun 25 2016, 22:18:55) [MSC v.1900 64 bit (AMD64)]
[INFO   ] Factory: 194 symbols loaded
[INFO   ] Image: Providers: img_tex, img_dds, img_sdl2, img_gif (img_pil, img_ffpyplayer ignored)
[INFO   ] Text: Provider: sdl2
[WARNING] stderr: Traceback (most recent call last):
[WARNING] stderr:   File "G:\Projects\LANRemoteController\LANRemoteController\Test\T007_kivy_start.py", line 17, in
[WARNING] stderr:     manager = multiprocessing.Manager()
[WARNING] stderr:   File "F:\Develop\Python35\lib\multiprocessing\context.py", line 55, in Manager
[WARNING] stderr:     m.start()
[WARNING] stderr:   File "F:\Develop\Python35\lib\multiprocessing\managers.py", line 483, in start
[WARNING] stderr:     self._address = reader.recv()
[WARNING] stderr:   File "F:\Develop\Python35\lib\multiprocessing\connection.py", line 250, in recv
[WARNING] stderr:     buf = self._recv_bytes()
[WARNING] stderr:   File "F:\Develop\Python35\lib\multiprocessing\connection.py", line 306, in _recv_bytes
[WARNING] stderr:     [ov.event], False, INFINITE)
[WARNING] stderr: KeyboardInterrupt

Is there some restricts to combine kivy and multiprocessing or multithreading ?

I think you're missing freeze_support.

@pythonic64 Thanks , that's it, add freeze_support() solve this case

Nice 👍


Priority 9 - search box in center and group page is not working

What do you mean by it? I checked the search box, I think its working fine , without any issues.


Create a script that ingests transactions from the first ledger

Since we have all ledger entries in Horizon ingestion pipeline (after #1819 is merged) we can run the pipeline starting at ledger 1. This will allow us to find corner cases and bugs that appeared due to processing many transactions from the last couple years. We can then use the script to check pipeline if it changes in the future.

Realized it's actually requires: #1470.


Trust relationship error when trying trying to download boxstarter

I get an error when running the install.ps1 as admin on a fresh Windows 7 x86 VM. The full error string: Exception calling DownloadString with 1 argument(s): The underlying connection was closed: Could not establish trust relationship for the SSL/TL S secure channel.
I fixed by adding the following from SO (https://stackoverflow.com/q/11696944/902766) to the start of the script:
add-type @"
    using System.Net;
    using System.Security.Cryptography.X509Certificates;
    public class TrustAllCertsPolicy : ICertificatePolicy {
        public bool CheckValidationResult(
            ServicePoint srvPoint, X509Certificate certificate,
            WebRequest request, int certificateProblem) {
            return true;
        }
    }
"@
[System.Net.ServicePointManager]::CertificatePolicy = New-Object TrustAllCertsPolicy

Fixed in 237f87ecb516e06a47a21db00743c369119caf1f


Change/preupdate

Fixes #286

Pull Request Test Coverage Report for Build 1063

15 of 15 (100.0%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.002%) to 88.602%

Files with Coverage Reduction
New Missed Lines
%

broker/lib/fabrik/DBManager.js
1
98.85%

Totals

Change from base Build 1052:
0.002%

Covered Lines:
6571

Relevant Lines:
7185

💛  - Coveralls

@ashishjain14 You are right, we should use agent credentials from currently deployed manifest only even for other operations like backup and restore. But that will increase load on bosh as we will be making one call to bosh every time we make request to bosh.
One solution that @jagadish-kb proposed is to force-update whenever services team make changes in agent credentials.

@amitmalav since we already maintain the deployments cache, why can't we manage agent credentials as well? Secondly is force update available as a feature?


How this package relates to tar package?

That could be in the description field of the cabal file.
Also it's unfortunate that both package use Codec.Archive.Tar module name.

It doesn't have the deeply-ingrained file system dependencies, and is thus suitable for use in the HaLVM and other, similar contexts.


BULK UPLOAD - TX - Lone Start Pediatrics

Bulk Upload Ready for Lone Star Pediatrics -  Requestor email<EMAIL_ADDRESS>


ui: Use NEXT_PUBLIC_API_BASE_URL env var for registry URL

Replace hard-coded registry URL with environment variable.

Closed in favor of https://github.com/tembo-io/trunk/pull/356


Add loading feedback while Batch Upload tool is working

Fixes #2569

coverage: 94.245% (-0.003%) from 94.248% when pulling 011b2d8384acee059c9e08c15fa48347905d7497 on Paul-Taiwo:paul-batch-upload-feedback into 2ba7b4f27fb55b153c6726c5ffeb5a3fb2bc1f45 on medplum:main.

✨


Adding bootstrap styles to rows dynamically

How is the most effective way to format a row based on a cells contents?
If I had a row with a date before today it should be styled with "className='danger'"
A row with a date within a week is should be styled with "className='warning'"
A row with a date more than a week but less than a month should be styled with "className='info'"
I only see "selecting a row" but its not really what I am trying to do, nor do I understand its implementation or how I could use it with multiple styles.
Thanks in advance!!  Love the table!!

@jsaddiction You can use trClassName prop. You can use it as follows
class MyTable extends React.Component {
  render() {
    return (
      <BootstrapTable data={ products } trClassName={this.rowClassNameFormat}>
          <TableHeaderColumn dataField='id' isKey={ true }>Product ID</TableHeaderColumn>
          <TableHeaderColumn dataField='name'>Product Name</TableHeaderColumn>
          <TableHeaderColumn dataField='price'>Product Price</TableHeaderColumn>
      </BootstrapTable>
    );
  }

  rowClassNameFormat(row, rowIdx) {
    if (row.date < todayDate) {
      return 'danger'
    } else if () {
    } else {
    }
    // adjust as needed here
  }
}

Apparently, the documentation has a mismatch here so I couldn't point you to the link.
@AllenFang Section Use trClassName as Function on <BootstrapTable> of this page has a gist mismatch. I've updated the gist here to better match the section example. However I'm not sure if more parameters are returned for function rowClassNameFormat .

@jsaddiction trClassName will be your answer.
@prajapati-parth thanks for quick response, and the gist already updated 👍

I am trying something similar on the code below, and rowClassNameFormat is not getting called as the console.log is not printing.
`import React, { Component } from 'react'
import { BootstrapTable, TableHeaderColumn }
from 'react-bootstrap-table'
import 'Table.css'
import '../node_modules/react-bootstrap-table/dist/react-bootstrap-table-all.min.css'
import 'react-bootstrap-table-next/dist/react-bootstrap-table2.min.css';
class Table3 extends Component {
render() {
return (

ID

Name

Value

)
}
}
function rowClassNameFormat(row, rowIdx) {
// row is whole row object
// rowIdx is index of row
console.log("Row is " + row)
return row['name'] === 'George Michael' ?
'GeorgeMichael-Row' : 'Other-Row';
}
export default Table3`


[Backport 2.0] Fix minor bugs and pass in user context when retrieving user's notification channels

Issue #, if available:
Description of changes:

Fix minor bugs and pass in user context when retrieving user's notification channels
Backport #447
CheckList:
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #448 (1e8fb57) into 2.0 (9b43845) will increase coverage by 0.17%.
The diff coverage is 90.90%.

@@             Coverage Diff              @@
##                2.0     #448      +/-   ##
============================================
+ Coverage     76.67%   76.84%   +0.17%
  Complexity      176      176
============================================
  Files           166      166
  Lines          8360     8371      +11
  Branches       1232     1231       -1
============================================
+ Hits           6410     6433      +23
+ Misses         1344     1332      -12
  Partials        606      606

Impacted Files
Coverage Δ

.../opensearch/alerting/DocumentLevelMonitorRunner.kt
81.85% <50.00%> (-0.24%)
:arrow_down:

...rg/opensearch/alerting/BucketLevelMonitorRunner.kt
76.92% <100.00%> (ø)

...in/kotlin/org/opensearch/alerting/MonitorRunner.kt
62.06% <100.00%> (+5.46%)
:arrow_up:

...org/opensearch/alerting/QueryLevelMonitorRunner.kt
79.48% <100.00%> (ø)

...main/kotlin/org/opensearch/alerting/model/Alert.kt
82.97% <100.00%> (+0.14%)
:arrow_up:

.../opensearch/alerting/model/DocumentLevelTrigger.kt
81.08% <100.00%> (+0.52%)
:arrow_up:

...ing/script/DocumentLevelTriggerExecutionContext.kt
100.00% <100.00%> (ø)

...rg/opensearch/alerting/core/model/DocLevelQuery.kt
76.66% <100.00%> (ø)

...destinationmigration/DestinationConversionUtils.kt
68.88% <0.00%> (-2.23%)
:arrow_down:

...ing/model/destination/DestinationContextFactory.kt
75.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


bugfix for error while scrolling to the selected year on the datepicker

Fixes #99

@jefersonestevo Thanks for the PR, we have fixed this issue throw another way.


function name changed to mainPatches

File "/CNNArt/correction/main_correction.py", line 9, in
import cnn_main
ImportError: No module named cnn_main

corrected


"help wanted" The target machine actively refused it

Showing error while using code iwr -useb https://christitus.com/win | iex in Windows 11 Terminal with Administrator privilege.
Error: Invoke-WebRequest: No connection could be made because the target machine actively refused it.

You might have better luck hitting the file directly. My guess is your IP is blocked somewhere in the chain.
https://raw.githubusercontent.com/ChrisTitusTech/winutil/main/winutil.ps1


facet: false to prevent a mark from being faceted?

Currently we rely on the equality operator to determine whether to facet marks:
https://github.com/observablehq/plot/blob/650529ab865e2e403c1147c77c5371ec6a71e843/src/facet.js#L41
I wonder if in addition to this equality check, we could provide a generic facet option, which if set to false would disable faceting for the mark. (I don’t think you’d be able to set it to true if the data is different that the facet, but maybe that would be okay if it has the same cardinality?)
In other words, instead of
Plot.dot(penguins.slice(), {
  x: "culmen_depth_mm",
  y: "culmen_length_mm",
  r: 2,
  fill: "#ddd"
})

you’d say
Plot.dot(penguins, {
  facet: false,
  x: "culmen_depth_mm",
  y: "culmen_length_mm",
  r: 2,
  fill: "#ddd"
})

(Though, the former would still work…)

Fixed in #450.


Enhancement: Add link to Contributors-list.md to README.md

Simple edit for anyone looking to practice creating pull requests.

Fork the repo
Download all files
Open README.md
Add the following (or something similar) to README.md

Contributors List

Commit change
Open a pull request

Questions can be asked here

I'd like to give this a try.


Async question

Hello, zielmicha. Please help.
What is a way in reactor to send async requests and get asynchronous responses from multiple urls?
I mean how to request few urls at once and get it's responses asynchronous.
Lib API is not so clear to understand.
Thanks

You can do something like that:
# you can also return something by declaring return type as Future[TYPE]
proc myrequest(): Future[void] {.async.} =
  let resp = await request(newHttpRequest("GET", "http://google.com/").get)
  # do something with response...

proc main() {.async.} =
  # note we don't use await here
  let resp1: Future[void] = myrequest("http://google.com")
  let resp2: Future[void] = myrequest("http://amazon.com")
  # now two requests are running, await them
  await resp1
  await resp2

Hopefully I'll add more documentation and some examples "soon".


[2.x] fix: concurrency control around Scala 3 compiler

Fixes https://github.com/sbt/sbt/issues/7873
Problem
Early initialization thread throws ConcurrentModificationException.
Solution
This puts in synchronized to lock for the access.

https://eed3si9n.com/december-adventure-2024/ has commentary on the whereabout of the thread.


VMs created for DevTest-labs does not appear in VM list

It seems that vms created using the DevTest-labs ARM api are not added to the vm list for the Lab they were created under.
The seem to respect any rules set for the lab, such as schedules, so it's only a visual thing, it does however prevent you from editing Labs specific configurations for the machine using the Azure Portal.
I created the machines with the NewVirtualMachineClient using the CreateOrUpdateResource method on the client instance.

@zlepper can you please provide more info? VirtualNetworkOperationsClient does not sound right. Have the calls succeeded? Can you see anything on Azure Portal or Azure CLI? Any more inputs are appreciated. I don't think we clearly understand the problem on your side. The developers of Go SDK team are not really familiar with DevTest Labs product, but if you could provide more info, we can route your issue accordingly.

Sorry, was supposed to be NewVirtualMachineClient instead of VirtualNetworkOperationsClient.
The calls succeed and the machine is created as intended, with all templates and schedules and such attached. The machine appears in the Virtual Machine list in the Azure portal.
However when I select the lab it is supposed to be in in the Azure Portal, then it's not in the labs list.

@zlepper Are you using a service principal credential to create the VM? If yes, then you can find the VM under Settings --> All VMs

Yes i do, and they are there, thank you.
But may I know why they don't appear on the first vm list?

You will see the VMs created using your credentials only in the lab dashboard by default. You can see all the lab VMs created by all other lab users by going to All VMs. This is to ensure that the default dashboard is not cluttered with many VMs and also to give you a better performance.


Error: unable to verify the first certificate

I am unable to serve up directories. I used create-react-app, ran a build, and attempted to serve it up.
Ran into this error...
(node:22821) UnhandledPromiseRejectionWarning: Error: unable to verify the first certificate
    at TLSSocket.<anonymous> (_tls_wrap.js:1105:38)
    at emitNone (events.js:106:13)
    at TLSSocket.emit (events.js:208:7)
    at TLSSocket._finishInit (_tls_wrap.js:639:8)
    at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:469:38)
(node:22821) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:22821) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Here are my versions...
{ asdf: '0.1.0',
  npm: '5.6.0',
  ares: '1.10.1-DEV',
  cldr: '32.0',
  http_parser: '2.8.0',
  icu: '60.1',
  modules: '57',
  nghttp2: '1.25.0',
  node: '8.11.1',
  openssl: '1.0.2o',
  tz: '2017c',
  unicode: '10.0',
  uv: '1.19.1',
  v8: '6.2.414.50',
  zlib: '1.2.11' }

I do have a cafile set in my .npmrc file. Another program, httpster, works fine.
I removed the cafile setting to no avail.

I have the same issue. I just try to do this: node_modules/.bin/serve --ignore node_modules. I use  nexus3 as registry:
my ~/.npmrc is
proxy=http://myproxy:8080/
https-proxy=http://myproxy:8080/
registry=http://nexusrepo/nexus3/repository/npmjs-group/
@myscope:registry=http://nexusrepo/nexus3/repository/npmjs-my-hosted/

@Vajidbaba
; cli configs
metrics-registry = "https://artifacts.d.company.com/api/npm/npm-virt"
scope = ""
user-agent = "npm/5.6.0 node/v8.11.1 darwin x64"

; userconfig /Users/karbica/.npmrc
@company:registry = "https://artifacts.d.company.com/api/npm/npm-virt"
always-auth = true
cafile = "/Users/karbica/Documents/companycert.pem"
email =<EMAIL_ADDRESS>registry = "https://artifacts.d.company.com/api/npm/npm-virt"
strict-ssl = true

; node bin location = /Users/karbica/.nvm/versions/node/v8.11.1/bin/node
; cwd = /Users/karbica
; HOME = /Users/karbica

And here is this...
$ npm ls -g --depth=0
/Users/karbica/.nvm/versions/node/v8.11.1/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Closing this now, as this issue is not related to serve, but rather to the way how you're installing it from npm.
Thank you for the effort you've put into this!
Please also read the docs for the new release.
🙏

Thanks, but i don't clear


JsonSerializerSettings on input and output JSON formatters

Hi
I noticed that the input and output JSON formatters independently create serializer settings.
On the input formatter you are explicitly creating a JsonContractResolver - https://github.com/aspnet/Mvc/blob/5bb69e495e9a7f62ac42bf77e1e8bcbb04af73cf/src/Microsoft.AspNet.Mvc.Core/Formatters/JsonInputFormatter.cs#L40 - while the output formatter is using the default contract resolver.
The problem with doing this is each contract resolver will independently resolve each type, which involves a lot of reflection, possibly some IL gen and dynamic compilation, and will increased memory used.
My question is this necessary? If not, you should avoid it for performance reasons.

@JamesNK we currently have this behavior because the input and output formatters are designed to have separate behaviors. For example, in MVC 6 by default for inputs there is extra behavior added to properties with the [Required] attribute to alter its behavior, but for outputs there is no such behavior.
Is there any way to have the contract resolvers share their caches? If it's just internal caching then it seems like maybe they could share a cache because there should never be any conflicts - cached reflection data reflects truths and is thus immutable.
Related to this, is it possible to compose contract resolvers? Currently it seems like the behavior is that you can have only one and the way to compose it is to "wrap" the current one with a custom one. Is that the recommendation for customers? Or is there a way to have a "pipeline" of contract resolvers so that they can compose independently?

Is there any way to have the contract resolvers share their caches?

Not right now. When I created the resolver class I imagined the contract resolver would be the cache. I could easily make it so that all reflection and IL generation for getting/setting/creating values is cached in a global static cache. I'm worried that there would then be no way to reclaim that memory. XmlSerializer was fairly infamous for creating unrecoverable memory leaks.

Related to this, is it possible to compose contract resolvers?

I can't think of a way. This wasn't something I thought about when designing the class. If you have a resolver modifying the contracts returned from another resolver, because those contracts are cached then you will be modifying them for all users of that resolver, regardless of whether it is wrapped. I assume you want to support allowing the user to define their own resolver and have it live along side yours?

I assume you want to support allowing the user to define their own resolver and have it live along side yours?

Yes right, an example scenario: A user would like to support camel casing of properties(which requires using camel case contract resolver) + our own resolver.

Hmmm, maybe a way to compose resolvers would be adding some logic to clone the contracts. The wrapping resolver could then modify what it needs, cache it, and then return it as needed.
Its would look completely different from DefaultContractResolver however. All it would have is the ResolveContract method and you would have to search the hierarchy of contracts that are returned, and modify them. And I believe other contracts can be resolved when the initial contract, sometimes being cached to internal fields for performance.
That probably won't work.

Re: Sharing caches
Yeah I agree that sharing caches has memory reclamation problems... don't know a great answer to this. Of course, instead of sharing the cache "dictionary" they can potentially share the cache entries, so of resolver 1 used A, B, C, D, E, and resolver 2 used A, B, X, Y, Z the entries (and cost) for A and B could be shared between both resolvers, and the non-overlapping items could be cleared out. Just a thought.

The simple answer to avoid duplicating the reflection cost for types is to use the same custom contract resolver for both the input and output formatter by default.

We should consider:

Do we really need a contract resolver at all? Can we make a change to use the RequiredAttribute without any wire semantic?
If we decide we do want the contract resolver should we use the same resolver by default for input and output?

We could really just make the settings part of MVC options if we want to. This is one of the more frequently touched knobs and that's about as front and centre as we can put it.

Where are the MVC options? I haven't touched MVC6 yet.

Do we really need a contract resolver at all? Can we make a change to use the RequiredAttribute without any wire semantic?

Can you point me towards your custom contract resolver? I didn't see it.

If we decide we do want the contract resolver should we use the same resolver by default for input and output?

I don't see any reason why you wouldn't want to.

Where are the MVC options? I haven't touched MVC6 yet

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/MvcOptions.cs
Default set of options for MVC ( Gets added when Mvc is configured for an application).
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc/MvcOptionsSetup.cs

Can you point me towards your custom contract resolver? I didn't see it.

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/Formatters/JsonContractResolver.cs

Thanks.
I think I see a bug in your JsonContractResolver: the MemberInfo passed into CreateProperty could be a PropertyInfo or a FieldInfo. Have you got a unit test where [Required] is on a field?

You could put a JsonSerializerSettings on the options. It would make it a lot more accessible. Right now I think you would need to do something like:
var jsonOutput = options.OutputFormatters.OfType<JsonOutputFormatter>().Single();
jsonOutput.SerializerSettings.Formatting = Formatting.Indented;
var jsonInput = options.InputFormatters.OfType<JsonInputFormatter>().Single();
jsonInput.SerializerSettings.Formatting = Formatting.Indented;

Compared to:
options.JsonSerializerSettings.Formatting = Formatting.Idented;

The input and output formatters (and any other places that use the serializer) could then use these settings unless different settings are explicitly set against them. That provides a mix of easy of use and flexibility.

:+1: on the options idea!

For your contract resolver issue, one option is I could add support for RequiredAttribute to json.net's DefaultContractResolver. It would have to be disabled by default and enabled by a flag for backwards compatibility, but that would allow someone to set there own contract resolver and then set that flag to true. I did a similar thing before WebAPI was released for ISerializable.
https://github.com/JamesNK/Newtonsoft.Json/blob/629c32062ea039cf2ba82b1ba0ec48180110e409/Src/Newtonsoft.Json/Serialization/DefaultContractResolver.cs#L178

@JamesNK you know we've been wondering if our behavior is even correct when it comes to the Required attribute meaning "must be in the JSON." That's somewhat of an abuse of what that attribute means, so we're actually thinking we remove that behavior entirely... in which case it might be easier for us to just use the defaults from Json.NET.

Ok. I would agree that sticking with the default serialization rules does make things simpler for people.
Quite often people would wonder why Web API serialized some objects differently than others, and the cause was its custom handling of serializable: http://stackoverflow.com/questions/12334382/net-webapi-serialization-k-backingfield-nastiness

And people can achieve the same behavior with Json.NET attributes:
public class Videogame
{
    [JsonProperty(Required = Required.Always)]
    public string Name { get; set; }

    [JsonProperty(Required = Required.AllowNull)]
    public DateTime? ReleaseDate { get; set; }
}

Although it is a little more long winded than [Required]. Maybe I should add a [JsonRequired] to mirror [JsonIgnore]?

Now that would be interesting...

There's an interesting parallel developing here with:
[JsonIgnore] <-> [BindNever]
[JsonRequired] <-> [BindRequired]
JsonPropertyAttribute et al definitely do more than our BindingBehaviorAttributes, but I'm wondering if we can avoid having to double up on attributes if you use a model sometimes for JSON and sometimes for a form post.
Maybe the default contract resolver is good enough for most and we ship a sample or in the box contract resolver that can respect our attributes (or an MMP that can respect Json.net's attributes).

I noticed [JsonProperty(Required = Required.Always)] is validated when deserializing and serializing. I'm guessing that is why MVC has different contract resolvers for input and output. You don't care about [Required] when serializing.
If I added [JsonRequired] it would be validated in both directions.

What are the dates around your beta 5? If you want ModelMetadata to use [JsonRequired] then I would need to update Json.NET first.

On second thought, in ModelMetadata you could use the contract resolver to get a contract for a type and then access JsonProperty and get information like whether a property is ignore or required there. That is available today.

Yeah I don't think we'd need to depend on anything new in Json.NET. The new attribute, should you add it, would be strictly a "convenience attribute" for app developers. MVC would never reference it (except maybe in tests :smile: ).

I have updated the Newtonsoft.Json package to 7.0.1-beta3. This beta includes a new [JsonRequired] attribute.

de63075 and fcf7b15
@harshgMSFT sorry I missed fact this issue was assigned to you.
@JamesNK thanks again.


Activation of Scope should fire @Initialized event

Both org.jboss.weld.junit.WeldCDIExtension#activateContexts  and
org.jboss.weld.junit.WeldCDIExtension#deactivateContexts
currently don't not fire any Events.
It would be nice to have them fire @Initialized and @Destroyed events as they would in an application server environment.

That's a good point. And the payload will be java.lang.Object.


proposal: cmd/go: support local experiments with interdependent modules; then retire GOPATH

Go "tinker mode" proposal.
 It's time to retire it. [...] we're happy to listen. But GOPATH needs to go. (@rsc)

@bcmills wrote: Many module issues and questions seem to center on editing, testing, and deploying multiple (possibly mutually-interdependent, possibly cyclic) modules (@bcmills)

The main workaround at the moment is to add replace directives among the modules to be edited, but maintaining those directives is tedious and error-prone. @rogpeppe's gohack tool automates away some of the tedium, but doesn't seem to remove the risk of accidentally checking in a go.mod with what were intended to be local, temporary replacements.

Related: #26640, #37755, #26377, #25053.
If an environment variable named GOTINKER is defined and set to an absolute path to the existing location on the local filesystem, and the import path of an include can be found under $GOTINKER/ directory, then build commands treat the $GOTINKER/src/import/path as a final authoritative source of the import; foregoing both vendor/ and any go.mod directive perpeting to this import/path.
Ie. build commands like 'go build' and 'go test' will compile modules present in the $GOTINKER directory instead of accessing the network, local module cache, or vendor directory.
Under tinker mode GOBIN, GOCACHE, GOMODCACHE, and GOENV are bound to locations relative to the GOTINKER: $GOTINKER/bin, $GOTINKER/cache, $GOTINKER/pkg/mod, and  $GOTINKER/goenv - respectively.
$GOTINKER tree should be populated by the user. For yet some time to come the last version of Go to support GOPATH can be used to ease this task, ie. GOROOT=/where/go1.16 GO111MODULE=off GOPATH="$GOTINKER" go1.16 get
GOTINKER path last element may start with an underscore character so experiments can be kept inside any project tree.
Security considerations
Both object code and the executable built under the tinker mode should not accidentally leak to the production environment. Ie. while objects are built and cached under $GOTINKER, then built executable MUST be amended (by the compiler) to refuse to run in an environment where GOTINKER is not set, or it is set but does not match the last element of the GOTINKER path that was compiled in.
Ie. "tinkered with" executable preserves the last part of the GOTINKER path then matches it to the last part of GOTINKER string where it runs. If these do not match, exectutable exits immediately with "Experimental but GOTINKER is not set or did not match" error message.

edits:

2021/02/17 added "match compiled in GOTINKER with run one" in Security
2021/02/18 be explicit about possibility to keep GOTINKER tree inside a module tree, eg. in a _tinker/ subdirectory.

CC @bcmills @jayconrod

cc @matloob @stamblerre @ianthehat

I like this proposal more than other alternatives to the problem, hope it get more attention.
I used to have GOPATH=/globally-cached/go-deps:/personal-tinkering/go-devs:/team-tinkering/go-pkgs, works perfectly well with organization-wide shared filesystems.
And I hesitate to pickup further development of my rusted Go projects at this time, just because go.mod is the right way to go, but it still lacks sufficient support rival to my historical workflows with GOPATH like that.

I don't understand the significant difference between this proposal and just continuing to use GOPATH (and, I guess, giving it priority over modules). That is, GOTINKER seems to be almost identical in its interpretation to GOPATH. The most significant deviation seems to be the "refuse to run if GOTINKER is not set" part, correct? I don't see the benefit of that - to me, if the binary is able to run with some environment variable is set, even if it doesn't look at that, this seems to serve as conclusive proof that setting this environment variable doesn't provide any benefit.

That is, GOTINKER seems to be almost identical in its interpretation to GOPATH.

Yes it is. Yes — almost: the main and important difference is that it operates in module "units".
There are modules under the GOTINKER/src, not packages.
I can do cd _tinker/src/import/path, clone/pull the source, mod-tidy it, then make changes at will. Repeat for other dependent modules pulled to _tinker/. When bug is fixed i can tag and push all the fixes to the modules repo simultanously.
Same for bootstraping and early experimenting with code structure.
When I am done with experiments, simple rm -rf _tinker && unset GOTINKER cleans up my workspace.

The most significant deviation seems to be the "refuse to run if GOTINKER is not set" part, correct?

The most significant deviation is described above. "Production fuse" is the second, along with builddirs moved inside GOTINKER. Both prevent experimental code leaks to the CI/CD pipelines. It shouldn't happen but it happens — and when it does, it hurts a lot.

if the binary is able to run with some environment variable is set, even if it doesn't look at that

It looks at, and if it sees wrong environment it gently refuses to do any work except saying why it won't run "here".

setting this environment variable doesn't provide any benefit.

To the developer, probably not much - she already has it set. But deployment people will be less nervous when asked to fire up a field test.

and just continuing to use GOPATH

GOPATH destiny, according to previous discussions, is to retire. GOTINKER needs the least code I could think of to get GOPATH benefits back. It boils down to:

check GOTINKER first; if set do:
in-memory: set GOENV (BIN CACHE)
scan the tree under GOTINKER
for every import/path found there and seen in go.mod too:

in-memory replace go.mod sourced entry relating to import/path with

require import/path v0.0.0-unpublished
replace import/path v0.0.0-unpublished => {{tinkerpath}}/src/import/path

...(skip code in vendor)...

There are modules under the GOTINKER/src, not packages.
I can do cd _tinker/src/import/path, clone/pull the source, mod-tidy it, then make changes at will. Repeat for other dependent modules pulled to _tinker/. When bug is fixed i can tag and push all the fixes to the modules repo simultanously.

Apparently I'm still not getting it. This doesn't sound very different to me. For example, I currently have GOPATH=$HOME and under ~/src/github.com/Merovius/nbd I have a checkout of the module module github.com/Merovius/nbd - i.e. the module is in the import path, relative to GOPATH. If I set GO111MODULE=off, I can cd ~/src/github.com/Merovius/nbd, work in GOPATH mode (and thus use the rest of my ~/src, where the rest of the modules I work with is also checked out under the correct impor tpaths), commit and push. All of that seems exactly the layout of what you describe when you say GO111MODULE=off go get … can be used for the setup of GOTINER.
So, I really don't see the difference, TBQH. To me, that seems to be exactly the setup and workflow you are describing.

GOPATH destiny, according to previous discussions, is to retire.

Yes. That's why I'm confused. To me, what you are describing is almost exactly the (for now) still existing GOPATH based workflow - except for a renamed environment variable. If we think we need that workflow, it seems more straight-forward not to retire GOPATH . And if we do want to retire GOPATH, surely we wouldn't just re-introduce it under a different name.

If I set GO111MODULE=off, I can [...]

Minus that with 1.17 the GO111MODULE will always be on.

what you are describing is almost exactly the (for now) still existing GOPATH based workflow

It is supossed to be almost identical - except for all tools being now in modules mode, so I can work only with, say, three repositories out of of thirty or eighty. Modules that I do not plan to tinker with will come from cache (copied or hardlinked to the $GOTINKER/mod/cache). Also all GOPATH related code, esp. VCS one, may retire.
GOTINKER code will need do a little bit of magic to internal representation of go.mods read — then anything else works in module terms as if you'd edited-in all those require/replace directives into proper places by hand.

If we think we need that workflow
Yes "we" think we need. See Rationale at top.

FTR, by "we" I mean "the Go project", not a specific subset of people. It certainly includes you and me and many others, many of whom likely agree with you and many of whom don't. It's possible for "the Go project" to come to a conclusion, even though some or many of its members disagree with it. The proposal process is how these conclusions are reached and are thus what decides what the "we" I was referring to "thinks".
So, let me rephrase my questions. If the outcome of this discussion is, that the workflow presented here is important enough to implement and given its extreme similarity (by design) to the existing GOPATH based workflow, wouldn't it be preferable to simply not retire GOPATH? And if, on the other hand, there are good, convincing reasons to retire GOPATH, why would those reasons not apply to this proposal? Surely, whether the environment variable is spelled GOPATH or GOTINKER does not affect those reasons?
Alternatively, this proposal is not as similar to GOPATH as it seems to me, which means I misunderstood something. That would certainly make these questions obsolete and provide justification for this proposal. Which is why I tried to understand these difference. So, is it actually the case that the workflow you describe is currently (we are not talking about go 1.17+) captured by setting GO111MODULE=off? If not, what are the differences? Your last answer "GOTINKER operates on modules, not packages" is confusing to me, as GOPATH seemingly contains exactly the same directory layout, containing exactly the same repositories with exactly the same files. But maybe I misunderstood something?

@Merovius As I understand it, one very important difference is that while GOPATH is an exclusive choice against go.mod, GOTINKER can work together with go.mod.
I for myself especially want the benefits of GOPROXY with go.mod on (to workaround GFW as one reason), I can't get those by choosing GOPATH, while GOTINKER is hopeful to provide equally ideal workflows like with GOPATH, and still have all modern & good things from go.mod.

@complyue

As I understand it, one very important difference is that while GOPATH is an exclusive choice against go.mod, GOTINKER can work together with go.mod.

What does that mean? Concretely? Like, presumably the intent is to actually use local modifications, ignoring versions specified in go.mod. Doesn't that just mean "ignore go.mod"? If there are stanzas of go.mod we don't want to ignore, couldn't we just teach the go tool to use them in "GOPATH-mode"?
FWIW, as far as I can tell so far, the only reason to call it GOTINKER is to avoid association with GOPATH. There is no technical difference, but rather "GOPATH is going away, but maybe, if we call it GOTINKER, it can stay". That's an uncharitable interpretation, though, which is why I'm trying to coax out the actual differences that are proposed here.

I for myself especially want the benefits of GOPROXY with go.mod on (to workaround GFW as one reason), I can't get those by choosing GOPATH

I don't understand why this would require a new environment variable. To me, this seems to simply mean downloading the zip and unpacking it in the right directory (same as gohack already does). The go tool, with GO111MODULE=off" does not care whether the code in $GOPATHis from a git repository, or a zip-file - AFAIK it even completely ignores the VCS aftergo get`.

FTR, by "we" I mean "the Go project", not a specific subset of people.

I meant specific subset, hence quotation marks.

So, let me rephrase my questions. If [...] the workflow presented here is important enough to implement [...] given its extreme similarity to the existing GOPATH based workflow, wouldn't it be preferable to simply not retire GOPATH?

Probably keeping GOPATH mode would be a least friction solution to the interdependent edits problem in the short term. But, after a period of being in denial, I now second Rob Pike's "GOPATH must go" statement — because the technical debt from keeping GOPATH workflow intact will IMO hurt the ecosystem in the long run. Modules with their clearly, automatically accounted and double-checked versioning mechanics are superior to GOPATH for any software that evolves fast, for long, or both.
That said, modules workflow has a blindspot where interdependent edits of two or more modules come with a huge footgun loaded: you need to be very careful editing your replace directives by hand, in many places, then you need to be very careful again when you need to clean your laboratory table after. Citing @bcmills again:

Many module issues and questions seem to center on editing, testing, and deploying multiple (possibly mutually-interdependent, possibly cyclic) modules.

(Just now /04.2021/ I see three discussions about "how to do it with modules" active on the golang-nuts list. Such questions emerge there almost every month since go1.11)

And if, on the other hand, there are good, convincing reasons to retire GOPATH.

GOPATH is all-or-nothing. When it is on, all used modules repos must go there, and proper (re being investigated code) versions must be checked out by hand. Then vendoring status of each must be consulted too. GOPATH mode is currently, ie. in modules filled world, good only for bootstraping new code, it is unusable for the bughunt purposes.

why would those reasons not apply to this proposal? [...]
Alternatively, this proposal is not as similar to GOPATH as it seems to me

This proposal keeps only good things from GOPATH mode — all sandboxed and all implemented by already existing mechanics of the modules.

as GOPATH seemingly contains exactly the same directory layout, containing exactly the same repositories with exactly the same files.

No, not the same files. And almost certainly not the same as being investigated production code uses.

But maybe I misunderstood something?
[...] what are the differences?

In GOPATH mode tools operate on repo's HEAD. It is up to you to do proper checkouts of everything your, possibly big, app or service uses.
In proposed GOTINKER mode tools operate on modules as usual, so anything else but repos you pulled under GOTINKER path is kept at version specified in respective go.mod. All bookkeeping is done for you, nothing will drift apart or leak accidentally.

@Merovius

Doesn't that just mean "ignore go.mod"?

Not entirely, I mean, I only want a few modules I put in local filesystem, to override what go tools would find according to go.mod, for vast of the rest dependencies, I would like them managed by the modern go.mod way.

If there are stanzas of go.mod we don't want to ignore, couldn't we just teach the go tool to use them in "GOPATH-mode"?

I'm not sure I understand you correctly, but the whole idea is to avoid things local-tinkering-only to go into go.mod, ideally it should always be what the general public is expected to see it.

this seems to simply mean downloading the zip and unpacking it in the right directory

I would like all the way go.mod is meant to work for hundreds of dependencies I base my projects on, my local WIP modules are relative very small in number. And GOPATH's way of go get is superseded by go.mod, isn't it?

@ohir My 1 cent for yet another proposal as I said in #44649 , I suggest a go.project or go.home file to manifest a local directory as farm of wip modules atop how go.mod works as is. I'm not sure you like the idea, and I don't have budget to propose it up, shame on me.

@complyue

I suggest a go.project or go.home file to manifest a local directory as farm of wip modules atop how go.mod works

"Project specific settings" already are addressed in the proposal by the means of local GOENV file inside the GOTINKER path. There is no need to add an another "special" file as a knob. Such file (like the go.mod changes) would be another box to check on the setup/cleanup lists.
You can use direnv to set GOTINKER for you. (Even now you can use direnv to set GOPRIVATE/GOPROXY in the WiP tree. Works well if you do not use much external modules).

Intended bootstrap workflow using GOTINKER is just to set it to the "project" dir. You then work in intended src/import/paths below with all the bells and whistles of module mode. Ie. all external dependencies are resolved as usual, your code import paths are replaced relative to each other internally by build tools.

@ohir

using GOTINKER is just to set it to the "project" dir.

To make myself clearer, I consider it would be another proposal without the GOTINKER env var, there go.project or go.home serves as the alternative marker of the "project" dir, instead of an env var.
Plain filesystem structure is simpler than that plus env vars, and will be much easier to up-scale without env vars involved. Somewhat like NodeJS uses node_modules to mark localized dependencies, if go.project file is used to mark a module farm, then we'll be able to support nested farms to reuse overriding modules in parent farm, while to place more specific overriding modules in the nested farm with higher precedence.
I mean I think that might be even better than GOTINKER proposed here, but I don't have the necessary funding (of time & energy) to pursuit that idea. So I support your proposal here as far as the situation goes at time of speaking.

I suggest GOTINKER may add support for colon (or semicolon) separated multiple directories as the good old GOPATH does.
Besides GOPATH, similar things have worked well with PYTHONPATH, JAVA_PATH etc., and the very original PATH for all these years, so why not with GOTINKER.

I suggest GOTINKER may add support for colon (or semicolon) separated multiple directories as the good old GOPATH does.

It was considered, then refuted.  Rationale: GOTINKER is not meant to keep yet another way to build final version of a Go software as explicit tag-based versioning supported by the modules ecosystem is clearly superior to the multi-tree one.
(With GOPATH, the only way to keep semi-stable versioning of being used code was to keep multiple code trees — and staging versions there or keeping tinkering under vendor/ of a stub package [since v1.6]. This was as much error-prone as using go.mod replacing directives is now).

I believe that the overall use cases being addressed in this proposal are also handled by multi-module workspaces, #45713.
I suggest we close this as a duplicate of #45713 and work on making that proposal handle any use cases it is missing.

While I think that #45713 is much overengineered I accept that the work on its implementation already begun, so this much lighter proposal can't stand. Closing.


Test other agent

Do we need to update the count in wait-for-traces files?

We detect is using env var so don't really need it in the test


Update _settings.scss

Hi!
Thank you very much for this project, it's awesome!
I found a typo mistake so I fixed it.
regards,

Thank you!


after selecting dropdown it shows error

Steps to Reproduce

...
...
...

Target Platform:
Target OS version/browser:
Devices:
Logs

Hi @damanjeetkaurchadha
please fill all or some of the information required in the template
when opening an issue.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Make coveralls api integration an optional feature

Today there is no way to disable the coveralls integration when compiling tarpaulin.
This features is the only reason that openssl needs to be brought in. Openssl is one of the main pain points when attempting to compile cargo tarpaulin.
For some reason even with the vendored-openssl feature enabled, tarpaulin still relies on a dynamically linked openssl1.1 when building in a docker container. Being able to disable the coverall API integration which we do not need would make this problem go away.

I've just pushed out a potential fix for this, if you have some time to test it to make sure it works for your use case https://github.com/xd009642/tarpaulin/pull/1583
And I think the vendored openssl feature thing might be because of libgit? I know that's one of the users of openssl in the dependency tree

Ah never mind I did my cargo check without excluding the feature :facepalm: still some stuff to change

okay that should work now for you :pray:

Release pipeline has been initiated so provided it all goes smoothly there should be a new version in under an hour with the ability to remove coveralls from the install


Enable Geoip Org DB

Hi All,
I'm just getting to grips with the nginx geoip module and i've come across the $geoip_org parameter which requires an organisation DB to be downloaded. I wanted to to create this issue as a feature request and hopefully start the discussion around it, and if it's straightforward enough I might try tackling it myself.
Cheers!

cool thanks for the update :+1:


Add support for CloudFront functions

what
This follows the naming of Terraform's AWS provider. Lambda@Edge
functions are "lambda functions", whereas CloudFront functions are
simply "functions".
references

https://github.com/hashicorp/terraform-provider-aws/releases/tag/v3.41.0
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cloudfront_distribution#function-association
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cloudfront_function

I've tested this by deploying my own distribution with an attached function 👍

By adding this, shouldn't we also update the aws provider pin to > 3.41.0 ?
It would be nice to link the docs for the function_association resource argument.

Thanks for the feedback, done both!

thank you!


set_params method

🚀 Feature Request
Add method to change parameters of etna objects
Proposal

BaseMixin.set_params: Callable[Dict] -> BaseMixin
It works like
assert Pipeline(model=CatboostMultiSegmentModel(n_iterations=100)).set_params({'model.n_iterations': 1000}) == Pipeline(model=CatboostMultiSegmentModel(n_iterations=1000))
We suppose to make conversion via to_dict methods (dict -> patching dict -> creation new object from dict)

Test cases

check that set_params change params
check if input dict have unknown parameters we do nothing

Additional context
No response

I'd like to take this issue. I'll start coding and report in one-three days.

I'd like to take this issue. I'll start coding, and I'll report in one-three days.

Ok. You're welcome

Should there be checks that the value assigned is of appropriate type? Or we'd rather let users be fully responsible for parameter-setting they do?
As a concrete example, should
Pipeline(model=CatboostMultiSegmentModel(n_iterations=100)).set_params({'model': 1000})
perform model=1000, or should it throw a TypeError instead?

Should there be checks that the value assigned is of an appropriate type? Or we'd rather let users be fully responsible for parameter-setting they do?

No, it's all on a user's responsibility.


Migrations support

Hi!
This package support migrations implementation? I did a test here but the new columns who I've added aren't pulled then sync.

Hi!
Yes. I think it really was missing in this package. Take a look on my implementation: https://github.com/rafwell/laravel-watermelon/commit/01dd6bf38be0ada2524e116cdd0912434f4e2f0f
I'm basing on this https://watermelondb.dev/docs/Sync/Backend#implementing-pull-endpoint who said 6. If migration is not null, you MUST include records needed to get a consistent view after a local database migration
Now i'm getting the new columns when columns has change by migration. I think the same must be did for new tables, if you like I can do it via PR.

Ohhh yes I see. Yes if you opened a PR that would be fantastic. Would you also be able to add a feature test where it demonstrates pulling records when there is a column that has been migrated?


What is the best practice to present a modal UIViewController?

I want to present a modal UIViewController. What is the best practice to show and dismiss it?

Use the .present transition to show the viewController and the .dismiss transition to dismiss it.
I'm not sure, whether that fully answers your questions... Please be more specific and make sure to read the README, if you have further questions.

@pauljohanneskraft Thanks for explanation!


Add function to create did-configuration

The identity foundation has published a specification to link DNS domains with DIDs: https://identity.foundation/specs/did-configuration/
This requires to create a JWT for the DID of the blockstack name.
There should be a method that produce the JSON of an entry described in the spec
The command line arguments should be:
blockstack-cli get_did_configuration BLOCKSTACK_ID OWNER_KEY

An example of how to use it on your website/app:
https://identity.foundation/.well-known/resources/did-configuration/demo/build/index.html

I left comments on #30.  Thanks guys!


Rename RecExpr to Expr

Renamed everywhere except the changelog.
recexpr -> expr.
RecExpr -> Expr.
Increased the crate version to 0.10.0 in Cargo.toml.
To close https://github.com/egraphs-good/egg/issues/37

Thanks for the contribution, but I think we'll keep RecExpr for now to avoid breakage.


Middle mouse button click on Tree-Navigation Pages

To consider: this functionality (middle mouse click) should close a page as this is expected behavior on many browsers.

Ah, I did not know that. I don't have a middle mouse button on my laptop I develop with. So is this middle-click on other browsers is on the tab itself? Can you confirm for me a couple of behaviors?

What happens when you middle-click a tab title in FF, FF with tree-style tabs, and/or Chrome? What happens in Doogie?
What happens when you middle-click a link in FF, FF with tree-style tabs, and/or Chrome? What happens in Doogie?

The goal is usually to replicate most of these features so if it's common for middle-click of a tab title to close the tab, we can likely support that (though it's not the fastest way to close).

Firefox: a tab is closed as if clicking on a "close tab" button. Firefox with tree-style tabs: dunno. Chrome: Same as in Firefox. Doogie: a tab is selected as if by left clicking on it.
Firefox: The same as right clicking a link and selecting "Open in a new tab". Firefox with tree-style tabs: dunno. Chrome: Same as in Firefox. Doogie: The same as right clicking a link and selecting "Open in a Background Child Page"

@milleniumbug - Thanks. Since both Chrome and FF support middle clicking to close a page, Doogie should also (and will just bake it in for now since Doogie doesn't have a well defined config system).


Implement incomes and expenses charts

I have to implement a way to show all incomes and expenses in a chart.
What is best chart? @sebastiaohns

Line chart
Bar chart (I think it is  better!)
Pizza chart

bar chart
why are you speaking in english?
2018-04-16 13:06 GMT-03:00 Eduardo Vieira<EMAIL_ADDRESS>
I have to implement a way to show all incomes and expenses in a chart.
What is best chart? @sebastiaohns https://github.com/sebastiaohns

Line chart
Bar chart (I think it is better!)
Pizza chart

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/euodeionomedeusuario/mamom-project/issues/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AR3QSt6cjv8R6vvqDh39ZU-QkloPHQwxks5tpMGIgaJpZM4TW0ku
.

É o recomendado! :)


Bullet Chart: fixed for sandbox 2311 Weird bottom scrollbar behavior …

Bullet Chart: fixed for sandbox 2311 Weird bottom scrollbar behavior (service only)

Hi @deniskudelin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Denis Kudelin (Akvelon)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Reviewed, lgtm


Add support for udp6 multicast

So this is my first attempt at supporting udp6 multicast for broadcast-stream.
It is off by default, and uses ff02::114 but you can override that.
I also added a lua dissector for wireshark as a useful utility, but maybe that should be in a subfolder like util/ or contrib/ ?
Would love to get your feedback. Cheers!

FYI this is not ready to merge

please don't put any thing ssb specific in this module, but happy to add links to things. also keep it es5 (so no destructuring please).
let me know when it's ready


Placeholder Selector

You can use the ::placeholder pseudo-element

yes as said by @Felixdiamond you can use pseudo element for that


Allow specifiying an epsilon or precision for matching numbers

Clear and concise description of the problem
Allow specifying a precision or epsilon for matching numbers.
Floating point math is inherently not precise, and standard practice is to compare them based on some epsilon or a specific precision.
Suggested solution
.toMatchObject() may provide an additional param e.g. { epsilon: 1e-10 } which would compare the difference between two numbers against the epsilon, or { precision: 10 } which would call .toPrecision(10) on all numbers before comparison.
Alternative
Epsilon: a custom .toMatchObject() can be implemented by the user, but would need to significantly duplicate the logic in toMatchObject().
Precision: this can be done by the user recursively before passing the object into toMatchObject().
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Can you explain more concrete use cases and what kind of tests you are writing? Seeing a concrete code (before and after with your suggested API) would help understand the issue better.

Depending on the context, you may try equality tester https://vitest.dev/api/expect.html#expect-addequalitytesters to make all number comparison to be imprecise.
https://stackblitz.com/edit/vitest-dev-vitest-yhzoqr?file=test%2Frepro.test.ts
import { test, expect } from 'vitest';

expect.addEqualityTesters([
  (a: unknown, b: unknown): boolean | undefined => {
    if (typeof a === 'number' && typeof b === 'number') {
      return Math.abs(a - b) < 0.1;
    }
    return (typeof a === 'number') !== (typeof b === 'number') || undefined;
  },
]);

test('repro', () => {
  expect(1).toEqual(1);
  expect(1).toEqual(1.05);
  expect(1).not.toEqual(1.2);
  expect(1).not.toEqual(2);
});

I think a custom equality tester is the solution here. I don't see the usefulness of this feature in core.


Modular builds for Front50

Storage providers to include can be specified usingthe includeProviders flag. These providers are added
as runtime dependencies and wired using Spring's@ComponentScan. By default, all providers are included.

Tests passing now. PTAL!

Addresses https://github.com/spinnaker/spinnaker/issues/3337


make starter_dev doesn't work when run with WSL2 on Windows

The Bug:
When I tried to run the make starter_dev command on WSL2, it gave me an error message: "drupal /var/www/drupal/vendor does not exist and could not be created".
How to Reproduce:
Stack:
1) Docker Desktop 4.34.2
2) WSL version: <IP_ADDRESS>
3) Latest ISLE-dc version as of October 2024 (not sure how to find what version I'm on sorry)
Open the isle-dc folder through WSL, then run the make starter_dev command.
Expected Behaviour:
The Islandora Starter Site will be created locally.
Actual Behaviour:
There is an error, "drupal /var/www/drupal/vendor does not exist and could not be created"

I tried to run sudo make starter_dev and also creating the folder manually, but that didn't work. What I had to do was:

Go into the isle-dc folder on Docker, then open drupal-1 container in the terminal
Run command "composer install"
go back into WSL inside the isle-dc directory and run "sudo make starter-finalize ENVIRONMENT=starter_dev"

Tested #320 to see if it would resolve the issue; but the issue still persists:
s6-envdir: fatal: unable to envdir /run/s6/container_environment: No such file or directory
Waiting for /var/www/drupal directory to be available...
make[1]: Leaving directory '/home/sshaw28/isle-dc'
docker compose exec -T -u nginx drupal sh -c 'composer install'
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Package operations: 182 installs, 0 updates, 0 removals

In Filesystem.php line 260:

  /var/www/drupal/vendor does not exist and could not be created:

install [--prefer-source] [--prefer-dist] [--prefer-install PREFER-INSTALL] [--dry-run] [--download-only] [--dev] [--no-suggest] [--no-dev] [--no-autoloader] [--no-progress] [--no-install] [--audit] [--audit-format AUDIT-FORMAT] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--apcu-autoloader-prefix APCU-AUTOLOADER-PREFIX] [--ignore-platform-req IGNORE-PLATFORM-REQ] [--ignore-platform-reqs] [--] [<packages>...]

make: *** [Makefile:181: starter_dev] Error 1

> ls -l
total 128
-rw-r--r-- 1 sshaw28 sshaw28          6925 Feb 29  2024 CONTRIBUTING.md
-rw-r--r-- 1 sshaw28 sshaw28          1077 Feb 29  2024 LICENSE
-rw-r--r-- 1 sshaw28 sshaw28         34152 Oct 16 11:03 Makefile
-rw-r--r-- 1 sshaw28 sshaw28         21399 Jul  3 16:27 README.md
drwxr-xr-x 6 sshaw28 sshaw28          4096 Feb 29  2024 build
drwxr-xr-x 2 sshaw28 sshaw28          4096 Oct 16 11:06 certs
drwxr-xr-x 7 sshaw28 systemd-journal  4096 Oct 16 11:07 codebase
-rw-r--r-- 1 sshaw28 sshaw28          4075 Sep 27 10:33 default_settings.txt
-rw-r--r-- 1 sshaw28 sshaw28         16112 Oct 16 11:06 docker-compose.yml
drwxr-xr-x 2 sshaw28 sshaw28          4096 Feb 29  2024 docs
-rw-r--r-- 1 sshaw28 sshaw28           729 Feb 29  2024 sample.Dockerfile
-rw-r--r-- 1 sshaw28 sshaw28          7021 Jul  3 16:27 sample.env
drwxr-xr-x 4 sshaw28 sshaw28          4096 Jul  3 16:27 scripts
drwxr-xr-x 4 sshaw28 sshaw28          4096 Feb 29  2024 secrets

> ls -l codebase
total 528
-rw-r--r-- 1 sshaw28 systemd-journal   7543 Oct 16 11:07 CONTRIBUTING.md
-rw-r--r-- 1 sshaw28 systemd-journal  18092 Oct 16 11:07 LICENSE
-rw-r--r-- 1 sshaw28 systemd-journal  18775 Oct 16 11:07 README.md
drwxr-xr-x 4 sshaw28 systemd-journal   4096 Oct 16 11:07 assets
-rwxr-xr-x 1 sshaw28 systemd-journal   5309 Oct 16 11:07 composer.json
-rw-r--r-- 1 sshaw28 systemd-journal 468872 Oct 16 11:07 composer.lock
drwxr-xr-x 3 sshaw28 systemd-journal   4096 Oct 16 11:07 config
drwxr-xr-x 3 root    root              4096 Oct 16 11:07 web

Tagging @DonRichards

@Sora201710 I'm not sure but I think running make with sudo will cause issues. The Makefile is calling sudo at times so this will likely cause a conflict.


more about evidence  [sf#107]

Reported by eilbeck on 2008-07-18 16:54 UTC
Copying in an email from Sandra at the biosapiens so it does not get lost.
Please see below we have had for a couple of new terms. Basically,
Matthias would like to annotate the strength of the evidence for a
particular protein sequence, and they would seem at me to fit best into
SO. One possibility would be to change the definition of
"polypeptide_sequencing_information" to a more general "Sequence as
given - evidence and incompatibility in the sequence due to some
experimental problem", with a new child "sequence_evidence" which itself
has 2 children "evidence_at_protein_level" and
"evidence_at_transcript_level" (which fits in nicely with the UniProt PE
designation).
polypeptide_sequencing_information
Sequence as given - evidence and incompatibility in the sequence due to
some experimental problem
sequence_evidence
Experimental evidence for the sequence as given
evidence_at_protein_level
Experimental evidence for the sequence as given derived from direct
protein sequencing
evidence_at_transcript_level
Experimental evidence for the sequence as given derived from mRNA sequencing
best wishes
I go on holiday, as of tomorrow, for just over 2 weeks so you have time
to think this one over.
Best wishes
Sandra

Commented by eilbeck on 2008-07-24 17:18 UTC
Logged In: YES
user_id=742851
Originator: YES
Hi Sandra,
I have a couple of comments about this request.
First off, SO provides a set of evidence relationships to be used to link a feature with the EST or other feature that supported the boundaries. They are:
evidence_for_feature
complete_evidence_for_feature
partial_evidence_for_feature
I am a bit worried by this term:sequence_evidence
Is this intended to be a feature or to be some kind of value or technique?
If it is a feature then you would be better to name the actual kind of feature it is. The the children of this term would also not be necessary probably.
I am not sure that the changes here help too much, although the original definition does need some work
polypeptide_sequencing_information
Sequence as given - evidence and incompatibility in the sequence due to
some experimental problem
original = Incompatibility in the sequence due to some experimental problem.
We could change it to be something like "A remark pertaining to a region of sequence with incomplete or conflicting information"
--karen

Commented by orchard on 2008-08-07 08:41 UTC
Logged In: YES
user_id=545117
Originator: NO
Hi Karen
Yes, I think these terms will probably fulfil Matthias's requirements - we will have to import them into the Biosapiens ontology for him. I mail him back and check this.
I think we were misinterpreting how you use polypeptide_sequencing_information.
Snadra

Updated by eilbeck on 2009-01-22 21:25 UTC

status: open --> closed


Prevent Dragon Body from Despawning + Make Dragon Unrevivable

About the PR

Dragon is now unrevivable and the Dragons dead body will no longer despawn after a period
Why / Balance

#29836, make Dragon unrevivable because there is nobody that should be reviving a dragon...
Technical details

RiftAccumulator now only counts up when RiftAccumulatorActive is true, which only becomes false when the dragon dies
Media

https://github.com/space-wizards/space-station-14/assets/83733158/057d433a-80d9-42d6-a1ec-336b35b49acb

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Dead Space Dragons no long despawn

That feels like it's just going to end badly for everyone still. If someone somehow got a dragon body(which would be full of crew bodies) and revived it, the dragon would just end up killing said person or dying quickly due to low health. The main purpose of the PR is to prevent dragons already killed from despawning with all the bodies they ate. If someone defibs the dragon into a crit state and it vanishes while they are healing it, it's not going to be fun for the antag that spent their time trying to revive them or all the people that got eaten by the dragon.

being revived should reset the timer or remove the rift part, so the revived dragon is a safe pet

make Dragon unrevivable because there is nobody that should be reviving a dragon...
Carp suit

Carp suit

Carp suit just gives you an AI team, player-driven carp and dragon are still going to eat you alive.

being revived should reset the timer or remove the rift part, so the revived dragon is a safe pet

They are still full of bodies… the main purpose of the pr is to make it so you don't lose those players due to a bug. Maybe in another pr though, there would need to be a system to get the people out without destroying the dragon body.

Okay hold on, a though just occured to me. Is it possible to harvest ichor from a dead dragon? If yes, is there a limit to the amount you can get? Or did we just create Medical's newest resource asset

Okay hold on, a though just occured to me. Is it possible to harvest ichor from a dead dragon? If yes, is there a limit to the amount you can get? Or did we just create Medical's newest resource asset

No, so how Ichor is created is that it's just added when a dragon eats someone. The issue with harvesting it is that a dragon metabolizes at an insanely high rate, do it only ends up in their blood for about 5 seconds.

Carp suit just gives you an AI team, player-driven carp and dragon are still going to eat you alive.

roleplaying game when people roleplay:

You could also just alter the code for pacifist to player carps and dragons but only make it apply to carp suit wearers (I don't know if this is possible tbh)

You could also just alter the code for pacifist to player carps and dragons but only make it apply to carp suit wearers (I don't know if this is possible tbh)

I'm going to do that anyways so the dragon and carps don't FF so much, but not in this pr 🙃

Does unrevivable prevent it becoming a zombie? Because zombie dragon is pretty peak.

Does unrevivable prevent it becoming a zombie? Because zombie dragon is pretty peak.

I don't think so, humans don't become zombies when unrevivable. I'm not in a place to test it right now.

I can just add the comp and vv and check. One moment.

I'm going to do that anyways so the dragon and carps don't FF so much, but not in this pr 🙃

zombies have code for preventing ff for zombies, could be change to work with factions then have zombies use zombie faction and this use dragon faction

Dragon code stinky but this is fine I suppose.


Critical error sourceNodes.sourceNodes build step.

I have a running WP development instance hosted on pantheon and I'm trying to use the DB for localhost.
This is my gatsby-config.js
{       resolve: gatsby-source-wordpress-experimental,       options: {         url: https://my-site-hosted-on-pantheon/graphql`,
},
}`
and this is my error msg.
`ERROR
gatsby-source-wordpress
    Encountered a critical error when running the sourceNodes.sourceNodes build step.
    See above for more information.

not finished source and transform nodes - 4.391s
not finished  gatsby-source-wordpress  fetching nodes - 4.336s - 59 total
not finished  gatsby-source-wordpress  Category - 4.344s
not finished  gatsby-source-wordpress  Comment - 4.347s
not finished  gatsby-source-wordpress  Page - 4.330s
not finished  gatsby-source-wordpress  Post - 4.326s
not finished  gatsby-source-wordpress  Tag - 4.322s

Hey @oliverfeher !
Do you also see error Cannot destructure property 'id' of 'undefined' as it is undefined. in your log output? If so this might be a duplicate of #72

Hi @woodwoerk !
I haven't seen that error. Every time I'm trying to run gatsby develop it will throw this error...
This is my most recent gatsby-config.js
{       resolve: gatsby-source-wordpress-experimental,       options: {         url: "https://my-site.pantheonsite.io/graphql",         verbose: true,         schema: {           queryDepth: 5,           typePrefix: "Wp",           timeout: 30000,         }       },     }
and I get this error..
  Encountered a critical error when running the sourceNodes.sourceNodes build step.     See above for more information.

Hi @oliverfeher , it's hard to tell what it could be from the description. Can you share a reproduction repo or a the GraphQL endpoint url so I can use it to debug? If it's private you can send it to me at<EMAIL_ADDRESS>Thanks!

Oliver DM'ed me on twitter, it appears the problem here is a WPGraphQL bug where JSON in post content seems to cause an error (my best guess) and that wasn't being caught by this plugin. I DM'ed @jasonbahl the details and we'll work out an upstream fix for this. I'll also update the error handling here to catch this error more gracefully in the meantime 👍

@TylerBarnes , thank you so much for your help!

Anytime, always happy to help :)

@oliverfeher I tried the endpoint that @TylerBarnes shared with me and it appears it's not a valid GraphQL endpoint. Did you de-activate WPGraphQL on that endpoint?

@oliverfeher I tried the endpoint that @TylerBarnes shared with me and it appears it's not a valid GraphQL endpoint. Did you de-activate WPGraphQL on that site?

@jasonbahl , is there a possibility that we could DM? https://twitter.com/Oliver92F

Via DM we were able to narrow the issue down to a single post_content field which contained JSON data. We weren't able to reproduce by copy/pasting that JSON data into our test WP instances, even after attempting to apply a post_content filter to JSON parse the data. Our best guess is that there is some custom logic in this WP instance that is filtering the content or something else to cause GraphQL to return an empty string instead of a JSON response for that post (when the content field is queried).
Next steps are for @oliverfeher's backend engineer to try to prevent this logic from running on that post during a graphql request.
@oliverfeher if it's possible for us to get a copy of this WP instance we're happy to continue debugging and helping out with this, but I'm closing this issue as it's not a bug in the source plugin or in WPGraphQL.
Thanks again for all your patience @oliverfeher !


Reason-React 2!

Adapted everything to the new API (inclusive RNTester) and got rid of the old API. So this would be a breaking change, but I think that's okay at this early stage...
Before we merge this, we should wait for the official roll out.
TODO:

[ ] Fix package.jsons to the release versions

some updates

Bravo 👏


feat: support suite.each, test.each, close #120

Check supported usages: https://github.com/vitest-dev/vitest/blob/eed0a2d467531cebfeb62690e130a1227bcfc3e0/test/core/test/each.test.ts
Note that the template literal usage from Jest is NOT supported intentionally. Mainly due to its lack of type safety and introduces too much complexity.

lack of type safety

I think tagged template literal is typed nicely? Would like to hear more on this.


Use tomllib/tomli for reading .toml configs

Use the built-in tomllib module in Python 3.11 and the modern tomli package in older Python versions to read .toml configs instead of the unmaintained and broken toml package.
Fixes #599
Fixes #600

Thanks for submitting a PR!
Please make sure to check for the following items:

[x] Add unit tests and integration tests where applicable.  (n/a)
If you've added an error code or changed an error code behavior,
you should probably add or change a test case file under tests/test_cases/ and add
it to the list under tests/test_definitions.py.
If you've added or changed a command line option,
you should probably add or change a test in tests/test_integration.py.
[x] Add a line to the release notes (docs/release_notes.rst) under "Current Development Version".
Make sure to include the PR number after you open and get one.

Please don't get discouraged as it may take a while to get a review.

Rebased on top of poetry.

Thanks for this PR!

Thanks!

This was a breaking change for our team, btw -- we're on Python 3.10 and have had toml installed, but updating to the latest version meant that our configuration file was ignored and so we got a bunch of unexpected errors:
WARNING: The pyproject.toml configuration file was ignored, because the `tomli` package is not installed.

Now that I understand what changed, it's a simple fix, but wanted to let y'all know that this wasn't 100% backward-compatible. Either way, thanks for the improvement -- I'm increasingly excited to join the Python 3.11 gang.

@christianbundy how were you installing pydocstyle with toml support? Were you using pydocstyle[toml] ? If so, it should not have been a breaking change.

We're using pre-commit, here's the config we were using:
  - repo: https://github.com/pycqa/pydocstyle
    rev: 6.2.3 # previously using 6.1.1, upgrading caused errors
    hooks:
      - id: pydocstyle
        args:
          - --config=pyproject.toml
        additional_dependencies:
          - toml~=0.10.2

Should we have been doing this differently? I think maybe we could add pydocstyle[toml] as an additional dependency, but I think that would install pydocstyle twice? Not sure how that would work.

@christianbundy you want the following -
https://github.com/pre-commit/pre-commit/issues/1526#issuecomment-649938146

Oh, nice! Thank you.

@christianbundy you want the following -
pre-commit/pre-commit#1526 (comment)

Could someone ask upstream to include that in the official documentation? It's not the most predictable way of sorting this particular problem. Unfortunately, I can't do that since upstream likes banning people for no apparent reason.

Just to spell it out for any others, it took me a few tries to connect the comment above to what we need here. My .pre-commit.yaml now has
  - repo: https://github.com/PyCQA/pydocstyle
    rev: "6.3.0"
    hooks:
      - id: pydocstyle
        additional_dependencies: ['.[toml]']
        exclude: test

and it made the tomli error go away


Update Readme and Mange the Directories

Anyone who is great at documentation can contribute.

Bring all profile of Contributors to readme
Include some animations and Images
You can display your creativity. Get started now

Assign me this issue.

Assigned @naikmubashir go ahead Happy coding  😃

It will be great if you can add all the contributors Profile in the GitHub readme file.

Don't forget to star the repo before your pull request.


Abide: Manually Check if Form is Valid

I don't see a method to manually check if a form is valid. There is a trigger .on('valid' but I'm having issues with it. It seems to get called after the .on('submit' trigger no matter what I do.

@imolorhe I've run into this issue again and calling directly like you've done works well for now. Looking forward to when the milestone is complete.

@rdetert :+1:

Hi all, I've made a commit where there's an option for different levels of control when it comes to validation. As this would be a breaking change (i.e. we're holding off until Foundation 6 to make this MAINSTREAM), I've still included the previous live_validate and validate_on_blur options.
You can, however, use this option when you initialize Foundation, like so:
$(document).foundation({
abide: {
validate_on: 'manual'
}
})

etc.
If you go with 'manual,' you can trigger your custom validations in this sort of manner:
$('#myInput').on('keydown', function() {
  $(this).trigger('validate.fndtn.abide');
});

Closing this issue for now with https://github.com/zurb/foundation/commit/3603d82e84cc43827b09d78bbf64c1da672155d5


Not matching on git shas

Recently updated my tmux plugins (been a while since my last update), and for some reason prefix + alt-h after a git log no longer matches the git hashes. If I hit n, I get "No previous regular expression (press RETURN)".  Any ideas?

Experiencing the same issue too

I'm getting the same issue here too.

Just installed copycat, I'm experiencing this.

@chris-fran #143 this should fix it. Could you give it a try?

Just fixed matching sha256. Please reopen the issue if this is still a problem.


[BUG] changing the viewbox size doesn't seem to expand the code to fit.

Tested on 4.3, 5.0-beta, and main.
Describe the bug or unexpected behaviour
setting the svgViewBoxSize to an int, doesn't seem to fill the viewBox with the QrCode.
Without setting the svgViewBoxSize, the output makes it <svg xmlns="http://www.w3.org/2000/svg" class="qr-svg qrcode" viewBox="0 0 41 41" preserveAspectRatio="xMidYMid">.
Setting it, and the only thing that changes is the viewBox="0 0 500 500"
Unfortunately I don't know enough about SVG to know if this is the intended output change, or if there should be more.
Hopefully I'm just overlooking some setting that I should be using.
Steps to reproduce the behavior
Current code, simplified for testing.
    $this->options = new QROptions([
            'svgViewBoxSize' => $this->getConfig('svgViewBoxSize', 500),
            'eccLevel' => $this->getConfig('eccLevel', EccLevel::H),
            'outputType' => $this->getConfig('outputType', QROutputInterface::MARKUP_SVG),
            'svgUseFillAttributes' => $this->getConfig('svgUseFillAttributes', true),
            'drawLightModules' => $this->getConfig('drawLightModules', false),
    ]);
$this->data = "https://localhost";
$optionsLight = clone $this->options;
//$optionsLight->setColor($color);
$qrLight = new ChillerlanQRCode($optionsLight);
$qrLight->render($this->data, $qrImagePathLight);

Expected behavior
That if I set the viewBox size, that the QR code,  (and svg logo inside) would expand to fill the view box.
Screenshots
SVG Gist

Environment (please complete the following information):

PHP version/OS: 8.2
Library version: 4.3, 5.0-beta, dev-main

Additional context

Hi. This is not a bug. The expectation that changing the viewbox attribute also changes the fill size of the content is a misconception. The SVG documentation says:

The viewBox attribute defines the position and dimension, in user space, of an SVG viewport.

It's useful to change this attribute if you want to add additional content, e.g. text under the QR symbol. The setting $svgViewBoxSize is deprecated because of this and the preferred method is to override the method QRMarkupSVG::header().
https://github.com/chillerlan/php-qrcode/blob/8c0da25c72e2b2f2303ce460a7a7193555cc7c52/src/Output/QRMarkupSVG.php#L90-L110
In order to change the size of the SVG to a fixed, which is what i assume you expect, you need to either add width and height attributes to the svg root element or change its size via CSS.

In fact, i should remove this option entirely before releasing v5. I added it a long time ago when i knew nothing about SVG and fell for the same misconception. It is misleading and most likely not widely used anyway.

I was experimenting in the past but there's no "best" way to set the size for the SVG, so i decided to leave this to the user entirely. the $scale option in combination with the width and height attributes isn't the best as it would fix the size which is often not preferred (ImagMagick requires them for properly sized output, though). SVG output is almost exclusively used in a web browser where CSS is available, and if it isn't, the size can be changed via CSS in the <defs> element or the aforementioned attributes.

Sigh, yeah. The whole point of the "S" in SVG. The issue I'm coming across, specific to me. I'm trying to import the QR codes, and the viewport is 0 0 41 41. which gimp makes a 41x41 pixel import. I can scale it up when importing, but it then becomes really fuzzy, like trying to upscale a jpg/png/etc. I was just trying to solve this from an import perspective.

That sounds like the same issue i had when converting via ImageMagick - try the output from that example and see what happens.

Ok, I just pulled/composer update and I'm running on dev-main.
I copy-pasta'd your svg example here:
https://github.com/chillerlan/php-qrcode/blob/main/examples/svg.php
and getting a viewport of 53 and it's trying to import as a 53x53 image:
<svg xmlns="http://www.w3.org/2000/svg" class="qr-svg qrcode" viewBox="0 0 53 53" preserveAspectRatio="xMidYMid">

That is correct. I meant try the output from the imagick conversion example with the changed <svg> element.

I'm going to close this issue here. Feel free to open a discussion for further questions.

I could be wrong, but it'd be nice to have the svg be large by default, for importing into things like gimp/photoshop/cad/non-html, browser. Then make them scale it down with css/html attributes.

Like i said, that is not the main focus or use case for SVG and the reason why i decided not to add these attributes.

Oh, derp.
I got: <svg xmlns="http://www.w3.org/2000/svg" class="qr-svg qrcode" viewBox="0 0 41 41" preserveAspectRatio="xMidYMid" width="205" height="205">

I could be wrong, but it'd be nice to have the svg be large by default, for importing into things like gimp/photoshop/cad/non-html, browser. Then make them scale it down with css/html attributes.

Like i said, that is not the main focus or use case for SVG and the reason why i decided not to add these attributes.

Gotcha, well at least you made it extendable, and you did a great job on the documentation, I personally appreciate that part! :-p

Glad to hear because that's exactly why i'm doing this.

Well I don't mind being your userland guinea pig.
I'm using it over here, in case you're curious: https://qr.fr3nch.com

I chose yours, because it's much more configurable, and it runs locally, verse google's graph api.
I also wasn't going to pay bitly and other QR code hosters $35/month when I can make my own site.

Ah yes the QR code scammers. Still a wild concept to me so generate a QR code and charge for it on a monthly basis - for tracking statistics you can get for free elsewhere. If i had the time and capacity i'd run such a website for free haha. Oh wait, i did (cheaply)