provide Python module

I'm using git-archive-all from within a Python script (in addition GitPython), and right now what I'm doing is issuing a command line call for git-archive-all OUTPUT_FILE. Given that git-archive-all is written in Python, a cleaner approach would be to allow for
import git_archive_all

which is then used in the script as well as the executable you provide.

I guess we can rename python file as long as make script installs it under git-archive-all executable.

@nschloe Please take a look.

This is wrong:
$ python setup.py install
[...]
Installing git_archive_all.py script to /usr/local/bin
[...]

Python module and executable should be two different scripts, one that can be imported and one that can be executed (using import git_archive_all). The file setup.py should then probably feature
setup(
  # [...]
  packages=['git_archive_all'],
  scripts=['git-archive-all']
  # [...]

I'd reopen this if I could.

@nschloe What about now?

After
sudo python setup.py install

I still see
>>>  import git_archive_all
---------------------------------------------------------------------------
ResolutionError                           Traceback (most recent call last)
<ipython-input-1-48edcc1c8be4> in <module>()
----> 1 import git_archive_all

/usr/local/bin/git_archive_all.py in <module>()
      3 __requires__ = 'git-archive-all==1.12'
      4 import pkg_resources
----> 5 pkg_resources.run_script('git-archive-all==1.12', 'git_archive_all.py')

/usr/lib/python2.7/dist-packages/pkg_resources.pyc in run_script(self, requires, script_name)
    526         ns.clear()
    527         ns['__name__'] = name
--> 528         self.require(requires)[0].run_script(script_name, ns)
    529
    530     def __iter__(self):

/usr/lib/python2.7/dist-packages/pkg_resources.pyc in run_script(self, script_name, namespace)
   1386         script = 'scripts/'+script_name
   1387         if not self.has_metadata(script):
-> 1388             raise ResolutionError("No script named %r" % script_name)
   1389         script_text = self.get_metadata(script).replace('\r\n','\n')
   1390         script_text = script_text.replace('\r','\n')

ResolutionError: No script named 'git_archive_all.py'

@nschloe I think this is the result of previous install... Try in a fresh virtualenv.

Got it, thanks!


repeated use of cell macros for lookup wastes loads - optimisation suggestion

This sort of pattern is common in the touch handling code in ls_touchInfo.ino, ls_handleTouches.ino and possibly other places:
 cell(col, row).touched = untouchedCell;
  cell(col, row).lastTouch = 0;
  cell(col, row).clearSensorData();
  cell(col, row).vcount = 0;
  cell(col, row).velocity = 0;

cell is a macro, looking up a value in a doubly nested array:
#define sensorCell()               touchInfo[sensorCol][sensorRow]
#define cell(col, row)             touchInfo[col][row]
Each of these involves 3 loads - get the value of touchInfo, the contents of the columns, and then the contents of the row.  Looking at the spec sheet for the Arduino Due (AT91SAM3X8E?) I find no mention of caches beyond a note that they aren't usually used.
Each of these loads is going to cost anywhere from 5 (guess, with L1 cache which may not be present) to 100s of cycles.  And the answer is always going to be the same (if I understand correctly - you don't create new cells and replace existing cells) : the inner pointer to the cell structure.
The repeated accesses can't be optimised away because the compiler can't prove that a write to (say) thing.touched isn't a write into the array structure or the touchinfo pointer itself.
However, you can give the compiler the information it needs:  An easy optimisation here would replace this sequence with
  currentCell = cell(col,row)
  currentCell.touched = untouchedCell;
  currentCell.lastTouch = 0;
  currentCell.clearSensorData();
  currentCell.vcount = 0;
  currentCell.velocity = 0;

etc.
Hope this is useful.

It can't be optimised away because it can't prove that a write to (say) touched isn't a write into the array structure or the touchinfo pointer itself.

Thanks for the suggestion, it's indeed a nice optimization opportunity that slipped through the cracks :-)
The repeated locations are infrequently used, so I doubt the impact would be noticeable though, but I'll see if it does.

It makes me think of one optimization I now remember I wanted to perform, but forgot when working on other things. Ie. sensorCell() can be pre-evaluated at each loop iteration as currentSensorCell, and not require any lookups anywhere during that loop afterwards.

Now I remember why the optimization wasn't done, doing this: currentCell = cell(col,row) will copy the TouchInfo structure values, which is hugely expensive. So the only option is taking the address of the array element and storing it into a pointer. That by itself is quite an expensive operation and only makes sense if there's really a lot of use for that particular instance, for instance in the case of sensorCell(). For instance, merely taking the instance address and storing it into a pointer for each loop iteration of the sensor scan, adds 182 microseconds to the full surface scan. Thankfully sensorCell() is very widely used, which makes this pre-lookup gain 319 microseconds overall, yielding an overall improvement of 137 microseconds. Not much, but sufficient to put this in place. I don't think it makes sense to do this elsewhere though.

good point on the struct thing; not clear on the semantics of this language - is it c or merely c-like? taking the address of the array element and storing it into a register  is what the compiler is doing in that operation, and should be cheap apart from the fact that a register or stack element is used.  pointer writes should be cheap enough - I clearly have some learning to do.
Can you add some context to those numbers? is it 319 microseconds per second of operation? or per benchmark which takes 5 minutes?
Thanks

319 microseconds per full surface scan, i.e. about 200 operations. So it's a bit more than 1 microsecond per operation.

It's C with C++ extensions

what's the total cost currently? you mentioned 4ms before, so 137 is 3%.  I'd pay good money for 3% in my project.

Going from 4280 to 4143.

Ok now i'm curious : if one store and the associated loads elsewhere costs 182ms, which is 4%, that suggests that elimination of other stores might be worthwhile.  If it's the additional loads, then passing the cell around as a register argument may get rid of that cost.

or is it possible that its accidentally doing a struct copy that you pointed out was so problematic?

did you measure the store by adding the store alone and leaving the read macros in place?

how many CPU cycles is 182ms?  I don't know the clock speed (80Mhz?) or the IPC of this processor.

I explicitly took the address, so it's not copying the struct. Yes, I measured by just doing the store and leaving the read macros in place. Second timing, removing the read macros and replacing them with the pointer deference to the previous lookup.

84MHz

it's not ms, it's µs (microseconds)

what's the % level of noise on these performance measurements?
xperhaps grab the gain we got, but 15000 cycles for one store.  seems excessive to me.

No noise, it's a real-time system, all the measurements are deterministic and 100% reproducible.

and the control timing was done just before the store test one, cold boot and everything?  Could you kick off another control run please?

I know how to measure this :-)

I have no doubt, it's just I can't measure mine; it's under the xmas tree :)

These timings are correct, and yes it with cold reboot, multiple samples, repeated scenarios, etc etc

oh, when I get the build environment running, do you have a performance test harness  (or technique) you can share?

Not really, there's no test harness short of some of the built-in test and debug methods that are in the code. There are no automated tests for features or functionalties.

Could you share your timing procedure if you don't mind?  It sounds like a fun thing to do.
the number 15000 is interesting though - whether we consider it the cycle count for the gain or for the cost of the store, it's the right sort of scale for a full scan.  15000 / 25 / 8 is 75.

Compile the code with DEBUG_ENABLED (ls_debug.h).
Set the LinnStrument to serial mode (Update OS Global option).
Connect the Arduino serial monitor and start it.
Activate the secret switch in column 17 row 5 that prints out the timing of each full surface scan over serial.
Watch it run.

I gotta go now though

sure, thanks for taking the time to hand hold me.


DataBase

Hi Friends, i'm interessed to triage issues in project. But i don't find the sql file to create database.
Anybody help-me?

Everything you need should be [here] (https://github.com/M-Zuber/MyHome/tree/Testing/Sql Files).
Please work on the Testing branch for now.
For now you need mmsql installed and then to run the script manually.

Thanks!


Bot is Down

Hiya there!
For a Community named "Gay Witches" (findable via Disboard" the bot is refusing to run basic commands we used previously.
The bot is still online, yet it just doesn't do anything. It's unwilling to run our moderation commands.

The Bot is working for someone else, it was just having issues for me ig.


regression prediction in python text get_dump() is off by 0.5

For regression (y is centered Gaussian like, from -4.0 to +4.0), when I try to parse the string trees, it seems the values in string trees are 0.5 less than the prediction returned from model.predict().
Is this true ? If so, is there a reason for this ? Seems a bug from binary classification.
parameters are as the following:
param = {}
param['nthread'] = 10
param['max_depth'] = 6
param['eta'] = 0.01
param['objective'] = 'reg:linear'
param['tree_method'] = 'exact'

It's b/c default base_score value is 0.5.
Setting it to average value of your y is usually a good idea.

@khotilov thanks very much, I am still exploring xgboost.

This should probably be closed ?

yes, doing it now. thanks.


this is not an issue

if we can replace influxDB with Postgres?

Hi @cah-jyoti-prakash, is it possible, but I do not see the benefits here. Influx DB or Elastic Search can be used as New SQL or No SQL BD to save data while performance tests. Postgres has no such capabilities as DBs mentioned above. Maybe for the functional testing, Postgres can be used..
BR,
Mike

How about this- https://docs.timescale.com/latest/using-timescaledb

@cah-jyoti-prakash In my opinion - What are business needs? The place to store the output from JMeter while high load and data from the hardware of the System Under Test (CPU, Memory, Network, App logs) - ELK stack if we do not need a high load we can simply use the Influx DB, since scaling need to be paid, can be used without scaling.
Really do not understand why you need Postgres? What are business needs?
BR,
Mike


Add Tor project

https://www.torproject.org/
https://www.torproject.org/static/images/tor-logo/

Added in c122ccfcfdb15d9958a85696ff2460ac3b01f8ca
Thanks


Azure Documentation not loading my subscription

My alias was changed in Microsoft and after that I don't have access with my old alias and with my new alias the account is not shown.
I logged in documentation with the same alias I use to log in Azure portal

@gabrielnespoli In Azure Portal, is your subscription active?  Where's the doc?


fix: improve url detection to avoid false positives

I've noticed that the url detection is a bit too simple as it creates false hyperlinks for regular words starting with http.
e.g. httpstatus would create a hyperlink even if it's unlikely expected to be there.
Note: I'm not sure if it's only for me but the test script isn't working. It also seems like there's no github action to actually execute the tests.
 FAIL  test/spec.ts
  ● Test suite failed to run

    TypeError: Cannot read properties of undefined (reading 'createNodeArray')

      at visitor (node_modules/ts-jest/dist/transformers/hoist-jest.js:84:61)

Thank you for your PR and pointing out to the infra issues. I fixed the tests in CI. Please rebase and I'm happy to merge

Thanks. I've rebased onto master.


Track casing of r-string prefixes in the tokenizer and AST

Summary
Following #10298, I wanted to look for places in ruff where we could simplify and/or speedup our code by using the information newly available to us in the AST nodes. One obvious place was the formatter, but when I looked at the formatter's code, I realised it was important to the formatter whether or not an uppercase or lowercase r prefix was used for a raw string:
https://github.com/astral-sh/ruff/blob/0c84fbb6db646550e00eed5560a692f2ea1b6224/crates/ruff_python_formatter/src/string/mod.rs#L165-L171
https://github.com/astral-sh/ruff/blob/0c84fbb6db646550e00eed5560a692f2ea1b6224/crates/ruff_python_formatter/src/string/mod.rs#L107-L118
That information isn't currently captured in the tokens or AST, so we wouldn't, right now, be able to simplify any of that formatter code. This PR makes it so the tokens and AST do capture that information.
The PR should be easiest to review commit-by-commit.
Test Plan
cargo test

Hmm I'm not sure if we should track this. It goes a bit beyond what an AST tends to capture. But it's also not clear to me what the downsides would be.
What does the formatter use the lower prefix for? I thought we always normalize to lower case

What does the formatter use the lower prefix for? I thought we always normalize to lower case

The formatter normalises all prefixes to lowercase, except for raw strings, where it retains the casing in line with what black does for raw strings: https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#r-strings-and-r-strings

Black normalizes string quotes as well as string prefixes, making them lowercase. One exception to this rule is r-strings. It turns out that the very popular MagicPython syntax highlighter, used by default by (among others) GitHub and Visual Studio Code, differentiates between r-strings and R-strings. The former are syntax highlighted as regular expressions while the latter are treated as true raw strings with no special semantics.

I see and I guess it's fine adding it now that we already track prefixes
The reason I'm hesitant to add such fine grained data to the ast are

This kind of Informationen could probably not be represented if we ever migrate to a CST (or would limit us in the design options for the CST)
Tracking the information increases the struct sizes which has a cost in memory consumption, data to write and to read (both slow operations). It can often times be faster (when considering the entire compilation pipeline) to reparse information that is needed rarely than paying the cost everywhere

Also, I think the arguments for preserving the casing of r-strings in the formatter apply equally well to the expression generator, and being able to losslessly round-trip these kinds of details in the expression generator was the original motivation for this series of changes (https://github.com/astral-sh/ruff/issues/7799, #9663, etc)

Thanks @dhruvmanila! I've implemented many of your suggestions, in particular most of the ones relating to naming and documentation

The reason I'm hesitant to add such fine grained data to the ast are

This kind of Informationen could probably not be represented if we ever migrate to a CST (or would limit us in the design options for the CST)
Tracking the information increases the struct sizes which has a cost in memory consumption, data to write and to read (both slow operations). It can often times be faster (when considering the entire compilation pipeline) to reparse information that is needed rarely than paying the cost everywhere

This was my initial reaction too, but...

Can you explain more about why this would be a problem for a CST? I don't understand that part.
I don't think this has a cost, since we're using a bitflag of the same size either way, though I may be mistaken.

Can you explain more about why this would be a problem for a CST? I don't understand that part.

It depends on the CST design. For example, Roslyn, rust-analyzer, Biome, and libswift all use a uniform representation for all nodes that mainly consists of the node's kind, its children and possibly some flags (single flags structure shared between all nodes). That means that node specific attributes or flags aren't possible. You can see this in Biome's playground when opening the syntax tab and looking at the CST content. For each level, it only shows the kind followed by its children.
A uniform representation has the advantage of traversing the tree without casting to the specific AST nodes. This makes it easy to implement traversal methods like descendants or storing a reference to a node.
The way we would need to model this with such a CST design is to look at the tokens directly (similar to what we used to do with Locator)

I don't think this has a cost, since we're using a bitflag of the same size either way, though I may be mistaken.

That's correct and what I mentioned in

I see and I guess it's fine adding it now that we already track prefixes


Can't get correct output when running in NVIDIA TensorRT

Hi Chuanqi305,
I am Deep Learning Software Engineer in NVIDIA Shanghai.
Thanks for your nice job here, and I tried to run your model in NVIDIA TensorRT optimizer.
But unfortunately can't get correct ouptput, can I add your WeChat to discuss more ?
Thanks.
Best Regards
Nic Ma

Yes, your WeChat account?

Any progress?
I am also working on this.
If tensorRT+mobilenet works, please post your instruction here.

any update?

我最近也在研究用TensorRT来优化你的模型，能否加你的qq或者微信进行交流请假呀。谢谢 @chuanqi305 @toxic2m

@toxic2m Hi, what layers did you have to implement using the custom layer API? Are you able to provide the source for this?

@toxic2m @chuanqi305  can you guys elaborate on the status? share the implementation on TensorRT

Hi @chuanqi305 and all,
Sorry for my delay response, I don't login github frequently.
Could you please share your email?
You can touch me at<EMAIL_ADDRESS>BTW, I have already implemented your code in TensorRT 3.0. But unfortunately I can't share the source code.
You need to implement Depthwise layer with TensorRT plugin.
Thanks.


Issue 44781: Update application.properties and LabKeyServer to use "encryptionKey"

Rationale
This PR replaces the usage of deprecated property master encryption key with encryption key in embedded tomcat.
Related Pull Requests

https://github.com/LabKey/Dockerfile/pull/9

Created https://github.com/LabKey/server/pull/214
I think we can let people know that they'll need to do pickDB task to refresh the application.properties.


Expanding the truncated commit messages... = Error: Bad credentials

Azure DevOps Extension you are using
XplatGenerateReleaseNotes
Generate Release Notes (Crossplatform)
Where are you running it?

Azure DevOps Service (VSTS)
GitHub

Version of Extension/Task
Version: 2.20.1 (Current Public Release / Azure Market)
Expected behaviour and actual behaviour
Expected behaviour:
Pull commit info from GitHub
Generate release notes to Release.md
Actual behaviour:
Getting the current build details
Expanding the truncated commit messages...
##[error]Error: Bad credentials
##[error]Error: Bad credentials
Finishing: XplatGenerateReleaseNotes
Steps to reproduce the problem

task: XplatGenerateReleaseNotes@2
inputs:
outputfile: 'Release.md'
templateLocation: 'File'
templatefile: 'Documentation\ReleaseNoteTemplate.md'
delimiter: ':'
fieldEquality: '='

Unknown, task worked until this morning then recent commits which involved added .xml files to repo the task failed with the bad credentials issue. Nothing particularly strange about the commits although there was a message body on one of them instead of just title information.
Other Azure DevOps projects have the same setup / credentials and are still working.
Following the debug error messsage it appears the issue is occurring inside 'export async function expandTruncatedCommitMessages' and probably the line 'await restClient.get(change.location)'
The rest of the build process is successfully pulling Github data and building, other projects are still working on creating release notes (and then additionally submitting to the wiki using the uploader).
Any thoughts on why this credentials error is occurring?

There has been updates to these extension for a few weeks, and I have just re-run my integration test suite and they pass.
So it would appear to be some effect based the commits, that is strange.
Can you re-run with a variable system.debug=true and let me see log file (anonymised as yo wish)

I can confirm that it is definitely commit based, I eventually disabled the release notes tasks to get a working build out and have since re-enabled it, it now works without those commits in its queue/backlog?
I have run a build of the commit that first failed the release notes task with the debug variable in place and (unfortunately?) it succeeded but Azure noted that there were 0 commits in the changes when the error builds have 2 commits or more (I'd had an earlier broken build before release notes task due to forgetting to push some files).
I'll attempt to get a replication tomorrow but if that doesn't work then I'll close the ticket.

I finally got a replication, I believe it's to do with the commit messages or perhaps having multiple queued commits.
 This worked which was annoying.
I re-branched a second time to do them one at a time which also worked. So it seemed like the commit contents wasn't causing the error.
As a final test I decided to test that the commit title / messages was the issue by just modifying my readme with an extra space / letter for two commits then ran the pipeline on those two outstanding commits with their original commit title / message. This replicated the issue.
Below is the log:
##[debug]agent.TempDirectory=d:\a_temp
##[debug]loading inputs and endpoints
##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
##[debug]loading INPUT_DELIMITER
##[debug]loading INPUT_EMPTYSETTEXT
##[debug]loading INPUT_FIELDEQUALITY
##[debug]loading INPUT_OUTPUTFILE
##[debug]loading INPUT_SHOWONLYPRIMARY
##[debug]loading INPUT_SORTWI
##[debug]loading INPUT_STOPONREDEPLOY
##[debug]loading INPUT_TEMPLATEFILE
##[debug]loading INPUT_TEMPLATELOCATION
##[debug]loading SECRET_AZUREDEVOPSPAT
##[debug]loading SECRET_SYSTEM_ACCESSTOKEN
##[debug]loaded 14
##[debug]Agent.ProxyUrl=undefined
##[debug]Agent.CAInfo=undefined
##[debug]Agent.ClientCert=undefined
##[debug]Agent.SkipCertValidation=undefined
##[debug]Starting Tag XplatGenerateReleaseNotes task
##[debug]System.TeamFoundationCollectionUri=https://redacted.visualstudio.com/
##[debug]System.TeamProject=project
##[debug]templateLocation=File
##[debug]templatefile=d:\a\1\s\Documentation\ReleaseNoteTemplate.md
##[debug]inlinetemplate=null
##[debug]outputfile=Release.md
##[debug]outputVariableName=null
##[debug]emptySetText=None
##[debug]delimiter=:
##[debug]showOnlyPrimary=False
##[debug]fieldEquality==
##[debug]stopOnRedeploy=False
##[debug]SortWi=False
##[debug]System.AccessToken=***
##[debug]Creating the credential handler
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
(node:2656) Warning: Use Cipheriv for counter mode of aes-256-ctr
##[debug]Release.ReleaseId=undefined
Getting the current build details
##[debug]Build.BuildId=1169
Expanding the truncated commit messages...
##[debug]Expanding commit [558083fbc71bd653079c8f447f1c7038aa81fe60]
##[error]Error: Bad credentials
##[debug]Processed: ##vso[task.issue type=error;]Error: Bad credentials
##[debug]SYSTEM.HOSTTYPE=build
##[debug]BUILD.DEFINITIONNAME=redacted.project
##[debug]BUILD.BUILDID=1169
##[debug]SYSTEM.TEAMFOUNDATIONSERVERURI=https://redacted.visualstudio.com/
##[debug]SYSTEM.TEAMPROJECT=project
##[debug]BUILD.BUILDID=1169
##[debug]TASK.DISPLAYNAME=XplatGenerateReleaseNotes
##[debug]SYSTEM.JOBID=12f1170f-54f2-53f3-20dd-22fc7dff55f9
##[debug]AGENT.VERSION=2.164.7
##[debug]Processed: ##vso[telemetry.publish area=XplatGenerateReleaseNotes;feature=reliability]{"hostType":"build","definitionName":"redacted.project","processId":"1169","processUrl":"https://redacted.visualstudio.com/project/_build?buildId=1169","taskDisplayName":"XplatGenerateReleaseNotes","jobid":"12f1170f-54f2-53f3-20dd-22fc7dff55f9","agentVersion":"2.164.7","version":{"Major":"2","Minor":"20","Patch":"1"},"issueType":"error","errorMessage":"{"stack":"Error: Bad credentials\n    at RestClient. (d:\\a\\_tasks\\XplatGenerateReleaseNotes_7b0a5887-75c4-4ade-a915-f92a93c934cb\\2.20.1\\node_modules\\typed-rest-client\\RestClient.js:173:28)\n    at next (native)\n    at fulfilled (d:\\a\\_tasks\\XplatGenerateReleaseNotes_7b0a5887-75c4-4ade-a915-f92a93c934cb\\2.20.1\\node_modules\\typed-rest-client\\RestClient.js:6:58)\n    at process._tickCallback (internal/process/next_tick.js:109:7)","message":"Bad credentials"}"}
##[debug]task result: Failed
##[error]Error: Bad credentials
##[debug]Processed: ##vso[task.issue type=error;]Error: Bad credentials
##[debug]Processed: ##vso[task.complete result=Failed;]Error: Bad credentials
Finishing: XplatGenerateReleaseNotes
The commit [558083fbc71bd653079c8f447f1c7038aa81fe60] it chocked on was just adding a space to the readme file so I don't believe it's content related therefore it is likely the title / message:
Added missing publish.xml files for DriveAuto
Resolving OPS-1975 incorrect DB name in DACPAC variables
I suspect it either doesn't like the dot in the title or the hyphen in the message

That is strange indeed. It looks to be failing in ReleaseNotesFunctions.ts around line 107. This when an extra call is made to unpack the full commit message as opposed to just the start if the string (80 char. as I remember)
Can see an obvious reason why this might be failing.
Sorry I am a little unclear over your repo, is this right so I can try to debug?

Repo with a text file in it
Edit the text file, add a character - then commit with a message 'Added missing publish.xml files for DriveAuto'
Edit the text file, add another a character - then commit with a message Resolving OPS-1975 incorrect DB name in DACPAC variables'
push to Azure DevOps
Run a build that includes these two commits
Run the release with the release notes generator - get the error

That does not seem enough, but I will try to repo it

No, sorry I probably wasn't clear.

Repo with text file in it
Edit the text file, add a character - then commit with a title 'Added missing publish.xml files for DriveAuto' and message Resolving OPS-1975 incorrect DB name in DACPAC variables'
Commit to repo
Azure DevOps - Run a build that includes these two commits - includes the build notes generator - get the error (No release in Azure just build)

My releasetemplate.md just uses buildDetails because its only a build not a release:
(I just build in Azure DevOps and then pass the artifacts to Octopus Cloud for it to do releases)
Release notes
Build Number  : ${buildDetails.buildNumber}
Build started : ${buildDetails.startTime}
Source Branch : ${buildDetails.sourceBranch}
Associated work items
@@WILOOP@@

** ${widetail.fields['System.WorkItemType']} ${widetail.id} ** Assigned by: ${widetail.fields['System.AssignedTo']}  ${widetail.fields['System.Title']}
@@WILOOP@@

Associated commits
@@CSLOOP@@

** ID ${csdetail.id} ** Commited by:  ${csdetail.author.displayName} (${csdetail.author.uniqueName}) ${csdetail.message}
@@CSLOOP@@

Sorry still struggling I'm being slow today

Edit the text file, add a character - then commit with a title 'Added missing publish.xml files for DriveAuto' and message Resolving OPS-1975 incorrect DB name in DACPAC variables'

What do you mean by title and message ?

A commit message the title on the first lines, a newlines and then the message - as done in VS Code of Visual Studio
Some git command line format

git commit -m "the subject" -m "the first line"$'\n'"the second line"$'\n'"the third line"
Also

Commit to repo

Can this 2nd commit be anything or do you mean push to Azure DevOps

Sorry, I tend to use tools, in this case GitHub desktop

So:
Summary = 'Added missing publish.xml files for DriveAuto'
Description = 'Resolving OPS-1975 incorrect DB name in DACPAC variables'
I'm not entirely sure what command that's doing in the background.
Commit to repo = push the above changes to Github and thus trigger the Azure DevOps CI (or be available for a manual build)

OK that make sense. i will work out that that client does. I bet it just adds the newline between the summary and description

The issue seems to be with the creds passed into the function that expands truncated commits.
I have spent a couple of hours trying to get a repo, and can't. I can't seem to create a commit that is truncated. I used your text and GitHub Desktop.
Until I manage a repo not sure there is much I can do.

I'll see if I cannot make a public repo that has the issue although if it's authentication issue then it probably won't replicate what my private repo's are doing.
I'll try to get a better replication over the weekend and see if I can pin down the issue more specifically.

https://github.com/ExpertSolutionsUK/DevOpsTest
Is a public repo that reproduces this issue.
It has:
YAML file for DevOps
ReleaseTemplate.md in documentation folder
Readme.md to modify to create new commits
You need an Azure DevOps pipeline based off the YAML file (just release notes task), for the last test builds I literately just used the default DevOps project (private), disabled all but pipelines and linked to the repo / YAML file.

I have repo.

If I fork your repo in Github, add a commit (with the failing title and description) and run a build it fails as you have seen
However if I import the repo into Azure DevOps and run the build it seems to work.

So can I confirm you are building directly from A GitHub hosted repo?
At this point it is worth saying is extension was only ever designed and tested to run against Azure DevOps hosted repos. There is an enhancement request to add full support for GitHub issues, but has not been looked into.
I guess this is going to be the first step in enhancing GitHub support.
I will try to get to the bottom of it.

I can confirm that I am building directly from a GitHub hosted repository.
Using the Third-Party (Microsoft) "Azure Pipelines (OAuth)"  service / plugin?
Just a standard GitHub / Azure DevOps setup

I have found the cause problem and and released potentially a fix as 2.21.1.
The issue was that in some cases the commit message is truncated (to the title). If this is detected an extra call is made to get the rest of the message. The problem is that this extra call is made using the RestClient used for the other Azure Devops API calls i.e. with the Azure DevOps creds. The problem was in your case that as the repo was on GitHub so these were the wrong credentials.
The fix I have done will work for public GitHub repos (as no creds are passed). However, I fear in your case this won't work if the repo is not public.
I see two options

Get the creds from the connector (not sure how to do this or tie the call to the right GitHub connector if there are many - this needs thought)
Put in the option to not expand truncated commit messages, at least fail in a nicer manner

Any thoughts?

Just checked it on a private repo, and as suspected it fails to expand the commit message, but does fail gracefully - you at least get the title

Well in my use case (and hence why I didn't spot this immediately) most commits have just a title and further information is just a clarification or note. Typically a title has a issue or ticket number in it which is used for storing most of the info. Therefore it failing gracefully is acceptable and you can note in the documentation that GitHub will have truncated commit messages for future users.
This ticket was fixing credentials error which appears to have been done and fully adding GitHub commit / credentials seems like a future todo feature. I'd stick it on the todo list until you have a chance to look at the Azure/GitHub Connector.

Also I confirm that my test repo / build project is no longer erroring.

@Stericycle-MMorris  glad I got yo unblocked
I am not going to close this, as I think the thread is useful. Instead I am going to add the enhancement label. I have also, as you suggested updated the WIKI

We are also experiencing the issue in Azure DevOps, but we are using Azure Repositories.
2020-02-28T15:30:10.0241448Z Task         : Generate Release Notes (Crossplatform)
2020-02-28T15:30:10.0241879Z Description  : Generates a release notes file in a format of your choice from the build or release history
2020-02-28T15:30:10.0242261Z Version      : 2.22.1
2020-02-28T15:30:10.0242515Z Author       : Black Marble
2020-02-28T15:30:10.0243588Z Help         : Version: 2.22.1. [More Information](https://github.com/rfennell/vNextBuild/wiki/GenerateReleaseNotes---Node-based-Cross-Platform-Task/)
.
.
.
2020-02-28T15:30:12.4014765Z Detected 175 commits/changesets and 17 workitems between the builds.
2020-02-28T15:30:12.4017292Z
2020-02-28T15:30:12.4045815Z Expanding the truncated commit messages...
2020-02-28T15:30:13.3999857Z ##[error]Error: Failed request: (401)
2020-02-28T15:30:13.4107302Z ##[error]Error: Failed request: (401)
2020-02-28T15:30:13.4111111Z ##[section]Finishing: Generate Release Notes

We recently migrated from Bitbucket to DevOps, maybe it has something to do with that?
Is there a way to disable expanding?

@coonmoo sorry I missed you comment, are you still suffering the same issue?
I can't see why migrating from Bitbucket should cause an issue, all the information is now in Azure DevOps. But 401 does suggest an access issue.
In answer to your question currently there is no means to switch off expanding message, but it should fails safe and continue if gets an error

Expanding commit messages (showing not just the first line) has been provided since 2.21.1. However there was a limitation. The expand worked for Azure DevOps hosted repo both public and private, and also for public GitHub hosted repos but fails for private GitHub repos.
I have done an extensive investigation to try to get around this problem. But there are two issues

In a build there is a GitHub Service Connection that allows the cloning of the repo. The problem is this does not expose a access token
In a release there is no even this Service Connection.

The issues therefore is that the token is just not available. The only option is therefore to allow a GitHub PAT to be provided to this task as parameter gitHubPat
This new features as been released as 2.43.2


Fix remove duplicate expr in grouping set

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


deletes Admin folder along with all its contents, they are no longer …

…being used in the app
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[X] Folder / File deletions, cleanup bloat

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have removed unnecessary comments/console logs from my code
[ ] I have made corresponding changes to the documentation if necessary (optional)
[X] My changes generate no new warnings
[X] I have checked my code and corrected any misspellings
[X] No duplicate code left within changed files
[X] Size of pull request kept to a minimum
[X] Pull request description clearly describes changes made & motivations for said changes

The cleaning up of the unused files is a beautiful site to see, very well done!


TextureMerge prefab sensitive to parent UMAGenerator transform

As described in the forum thread here, rotating the UMAGenerator breaks atlas generation. We should probably ditch that prefab and have it build the camera in code, or at least put some sort of identity transform onto it before it's updated.
http://forum.unity3d.com/threads/uma-unity-multipurpose-avatar-on-the-asset-store.219175/page-65#post-2262366

This is apparently fixed in an earlier commit.


error while using 0_userdata as praefix

const praefix = "javascript.0.Timer."; //Grundpfad is working fine.
when using 0_userdata as praefix the DPs TimerCount, TimerCountText and TimerCountValues get nulled an also the DPs SwitchTarget and SwitchTargetName for the first two timers become also nulled.

I think, its not an Script Issue. The support for 0_userdata is very new, and im not sure if its fully native supported by the JS Adapter, an when, at which version. Please use the javascript.0.xxx path or create an Issue at the js adapter.


Failed to clean-up vcredist_x64.exe installer

During the dcos-net setup step, the initialization script sometimes fails when trying to do cleanup after the install of vcredist_x64.exe is complete.
This is the failing line: https://github.com/dcos/dcos-windows/blob/master/scripts/dcos-net-agent-setup.ps1#L33
Snippet from the DCOSWindowsAgentSetup.log log:
Install VCredist: https://download.microsoft.com/download/2/E/6/2E61CFA4-993B-4DD4-91DA-3737CD5CD6E3/vcredist_x64.exe
Finished to install VCredist: https://download.microsoft.com/download/2/E/6/2E61CFA4-993B-4DD4-91DA-3737CD5CD6E3/vcredist_x64.exe
Install VCredist: https://download.visualstudio.microsoft.com/download/pr/11687625/2cd2dba5748dc95950a5c42c2d2d78e4/VC_redist.x64.exe
Finished to install VCredist: https://download.visualstudio.microsoft.com/download/pr/11687625/2cd2dba5748dc95950a5c42c2d2d78e4/VC_redist.x64.exe
Cannot remove item C:\Windows\TEMP\vcredist_x64.exe: Access to the path 'C:\Windows\TEMP\vcredist_x64.exe' is denied.
Failed to setup the dcos-net Windows agent
The upstream DCOS init script failed
at <ScriptBlock>, C:\AzureData\DCOSWindowsAgentSetup.ps1: line 126
at <ScriptBlock>, <No file>: line 1
at <ScriptBlock>, C:\AzureData\dcosWindowsProvision.ps1: line 200
at <ScriptBlock>, <No file>: line 1
at <ScriptBlock>, <No file>: line 1
Failed to initialize the DCOS node for CI

It looks like some process it's still using the vcredist_x64.exe installer, even after the installation is complete and gives the error:
Cannot remove item C:\Windows\TEMP\vcredist_x64.exe: Access to the path 'C:\Windows\TEMP\vcredist_x64.exe' is denied.

It is recommended to check if any process is using any of the files that we want to cleanup. A pull request will follow to achieve this.

Fixed by https://github.com/dcos/dcos-windows/pull/62

Reproduced yesterday for:

http://dcos-win.westus.cloudapp.azure.com/dcos-testing/809/windows_agents/<IP_ADDRESS>/AzureData/DCOSWindowsAgentSetup.log

I sent out two PRs:
https://github.com/dcos/dcos-windows/pull/99
https://github.com/dcos/dcos-windows/pull/100
please take a look when you find time :)


Update maven to 3.8.7

Codecov Report

Merging #627 (b629941) into main (56915cb) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #627      +/-   ##
============================================
+ Coverage     92.79%   92.83%   +0.03%
- Complexity     2614     2615       +1
============================================
  Files           280      280
  Lines          5371     5371
  Branches        587      587
============================================
+ Hits           4984     4986       +2
+ Misses          238      237       -1
+ Partials        149      148       -1

Impacted Files
Coverage Δ

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
96.29% <0.00%> (+3.70%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Correct way to configure scicat to run with RabbitMQ

Configure Scicat to run with Rabbit MQ
Summary
I have followed the documentation to change the configuration files for scicat so it will run with a local service of RabbitMQ running in a docker container. I have tested my local service using the pika python client so I know that I can connect to it independently of Scicat. For testing purposes I also have scicat running locally, I am trying to get this instance to talk to my RabbitMQ server. I have tried to follow the template of the component-config.local.js from this repos CI/PSI/component-config.local.js to set up my RabbitMQ server. I have also edited the config.local.js file to give the correct ports for my local version of RabbitMQ.
I can start the instance of Scicat with these two component files, however I cannot see any evidence that Scicat is sending messages on rabbitMQ. I have tried checking with rabbitmq cli to list the bindings and queues, but cannot see those that I define in my component config. I have also tried posting a Job through loopback explorer and seeing if a message appears in the rabbitmq logs but do not see anything. Should I set up the queue and the exchange before connecting with Scicat? Or does Scicat create these through the config?
The part of my config.local.js file relating to RabbitMQ is:
rabbitmq: {
    enabled: true,
    host: "<IP_ADDRESS>",
    port: 5672,
    queue: "client.jobs.write",
  }

The component-config.local.js file:
{
  "loopback-component-explorer": {
    "mountPath": "/explorer"
  },
  "../node_modules/loopback-component-mq/lib": {
    "options": {
      "restPort": 5672,
      "acls": [
        {
          "accessType": "*",
          "principalType": "ROLE",
          "principalId": "$unauthenticated",
          "permission": "DENY"
        }
      ]
    },
    "topology": {
      "connection": {
        "uri": "amqp://guest:guest@<IP_ADDRESS>:5672/"
      },
      "exchanges": [
        {
          "name": "jobs.write",
          "type": "topic",
          "persistent": true
        }
      ],
      "queues": [
        {
          "name": "client.jobs.write",
          "subscribe": true
        }
      ],
      "bindings": [
        {
          "exchange": "jobs.write",
          "target": "client.jobs.write",
          "keys": [
            "jobqueue"
          ]
        }
      ]
    }
  }

I use docker compose to start up my instance of Scicat, here is the docker compose part for catamel. I am also starting rabbitmq as a service:
services:
 rabbitmq:
    image: library/rabbitmq:latest
    ports:
      - "5692:5692"
  catamel:
    image: dacat/catamel:latest
    depends_on:
      - mongodb
      - openldap
      - rabbitmq
    volumes:
      - ./config/catamel/config.local.js:/usr/src/app/server/config.local.js
      - ./config/catamel/config.local.js:/usr/src/app/server/component-config.local.js
      - ./config/catamel/providers.json:/usr/src/app/server/providers.json

Any help with how to connect and anything I have missed would be appreciated! Many thanks.

Are there any error messages from the catamel logs ? As far as I remember (too long ago) I set up the queues manually in Rabbitmq before connection.

So far nothing in the catamel logs. When I set up the queues manually before in python I was getting a heart beat error in RabbitMQ on opening scicat. On googling, it seems that this might have been caused by my connection in python. I will have a go again but this time set things up through a rabbitMQ config file or through the CLI.
Another question, is there anyway one can test the connection to rabbitMQ through the loopback explorer (or with another method)? I have just been posting jobs because I saw in the architecture diagrams that a job entry in mongo triggered the messaging system so that is where I started, but I wonder if there is a better way.
Thanks again for your help.

I also only know this way, creating a job...no direct option via loopback explorer

What branch did you use ? Could you give me the log output when catamel starts up ?

In fact what we use here at PSI is not the Rabbitmq guest user, but a user which is assigned administrator rights. This user I created via the RabbitMQ GUI, using port 15672. The line
"uri": "amqp://guest:guest@<IP_ADDRESS>:5672/"
is adjusted then with user:password
Why do you use port 5672 in component-config.local.js, but 5692 in your service definition ?

Thanks for this, good spot on the ports. I have put this back to the correct ports but still has not solved the issues. There was also a second problem in the docker-compose.yaml where I copy the component-config.local.js to the wrong location. I have fixed both these issues but it is still not working. My next attempt was to look into the ACLs and users. I had set up the queues and exchange manually but still was not sending messages. I will follow your direction on setting up the user and let you know.
Many thanks for looking into this.

I have managed to set up a rabbitmq server with the management console and have added a user as you suggest. I connect as this user through the python client on the correct ports and receive messages on the correct queue. However when I start Scicat I have no evidence of connection. The catamel logs look like this for start up and receiving a post request for a job:
[+] Logger set to: Console

Datasource host mongodb, database dacat

Adding ACLS for UserIdentity

INFO - Adding relations to User

{"relation":"UserSetting"}

INFO - Adding ACLS for User related models

{"relation":"UserSetting"}

Account already exists: proposalIngestor

Index on field datasetlifecycle.archivable created successfully

Index on field datasetlifecycle.retrievable created successfully

Index on field datasetlifecycle.publishable created successfully

Index on field datasetlifecycle.archiveStatusMessage created successfully

Index on field datasetlifecycle.retrieveStatusMessage created successfully

Text Index on Dataset created successfully

Text Index on Sample created successfully

Text Index on Proposal created successfully

Text Index on OrigDatablock created successfully

Role already exists: proposalingestor

Rolemapping already exists: proposalIngestor proposalingestor

Database Autoupdate result: undefined

Role already exists: globalaccess

Rolemapping already exists: proposalIngestor globalaccess

Account already exists: archiveManager

Role already exists: archivemanager

Rolemapping already exists: archiveManager archivemanager

Role already exists: globalaccess

Rolemapping already exists: archiveManager globalaccess

Account already exists: ingestor

Role already exists: ingestor

Rolemapping already exists: ingestor ingestor

Role already exists: globalaccess

Rolemapping already exists: ingestor globalaccess

Account already exists: admin

Role already exists: admin

Rolemapping already exists: admin admin

Role already exists: globalaccess

Rolemapping already exists: admin globalaccess

Web server listening at: http://localhost:3000

Browse your REST API at http://localhost:3000/explorer

[+] Logger set to: Console

Datasource host mongodb, database dacat

Adding ACLS for UserIdentity

INFO - Adding relations to User

{"relation":"UserSetting"}

INFO - Adding ACLS for User related models

{"relation":"UserSetting"}

Account already exists: proposalIngestor

Index on field datasetlifecycle.archivable created successfully

Index on field datasetlifecycle.retrievable created successfully

Index on field datasetlifecycle.publishable created successfully

Index on field datasetlifecycle.archiveStatusMessage created successfully

Index on field datasetlifecycle.retrieveStatusMessage created successfully

Text Index on Dataset created successfully

Text Index on Sample created successfully

Text Index on OrigDatablock created successfully

Text Index on Proposal created successfully

Role already exists: proposalingestor

Rolemapping already exists: proposalIngestor proposalingestor

Database Autoupdate result: undefined

Role already exists: globalaccess

Rolemapping already exists: proposalIngestor globalaccess

Account already exists: archiveManager

Role already exists: archivemanager

Rolemapping already exists: archiveManager archivemanager

Role already exists: globalaccess

Rolemapping already exists: archiveManager globalaccess

Account already exists: ingestor

Role already exists: ingestor

Rolemapping already exists: ingestor ingestor

Role already exists: globalaccess

Rolemapping already exists: ingestor globalaccess

Account already exists: admin

Role already exists: admin

Rolemapping already exists: admin admin

Role already exists: globalaccess

Rolemapping already exists: admin globalaccess

Web server listening at: http://localhost:3000

Browse your REST API at http://localhost:3000/explorer

INFO - Fetching metadataKeys

{"fields":{"mode":{},"isPublished":false,"metadataKey":""},"limits":{},"options":{"prohibitHiddenPropertiesInQuery":true,"maxDepthOfQuery":12,"maxDepthOfData":32,"accessToken":{"id":"E4InXUXXwbClch5Zxg2TBHoGXYht3lCFwwCk2yb95SAd4pGYdFLgi02oYFGFUOOf","ttl":1209600,"created":"2020-11-27T13:33:12.924Z","userId":"5fa51991c48be430dbc57623"},"authorizedRoles":{"$authenticated":true,"$everyone":true},"currentUser":"ingestor","currentUserEmail":"scicatingestor@your.site","currentGroups":["globalaccess","ingestor","ingestor"]},"blacklist":["/.*_date/","/runNumber/","/Entrych*./","/entryCh*./","/FMC-PICO*./","/BW_measurement*./","/Linearity_measurement*./","/Pulse_measurement*./"]}

INFO - Found datasets

{"count":10}

INFO - Raw metadata array

{"count":10}

INFO - Curated metadataKeys

{"count":0}

When I send a job through the loopback explorer I get a 200 reponse but no messages generated or connections seen.

I appreciate  that this error can not be easily replicated - is there some message that should appear in the catamel logs on connection that I can look out for?
Many thanks

In response to my last question, I have found this in the server.js script:
if ("queue" in configLocal) {
    var msg = "Queue configured to be ";
    switch (configLocal.queue) {
        case "rabbitmq":
            console.log(msg + "RabbitMQ");
            break;
        case "kafka":
            console.log(msg + "Apache Kafka");
            break;
        default:
            console.log("Queuing system not configured.");
            break;
    }
}

I noticed that I was missing the queue: 'rabbitmq' line in the config.local.js and have added it. However I still don't see any of the above messages. Should I be seeing something like this?

AT startup you should see the "Queue configured to be RabbitMQ" message. DO you see it ?

Sorry I am using a docker-compose file from the rfi which pulls in the latest image of dacat/catamel and catamel , and adds our local configs. I have cloned these images 12 days ago and I see there has been updates in the last week.
Thanks for letting me know about the message. I do not see the message on set up but I will pull the latest docker image again.

I do not know which version the docker images have. It is usually safer to create your own image...
Anyway: depending on the version you may need a different syntax inside job.json:
index 3c92c91..49f9304 100644
--- a/common/models/job.json
+++ b/common/models/job.json
@@ -8,10 +8,12 @@
         "validateUpsert": true
     },
     "replaceOnPUT": false,
-    "MessageQueue": {
-        "producers": {
-            "publishJob": {
-                "exchange": "jobs.write"
+    "mixins": {
+        "MessageQueue": {
+            "producers": {
+                "publishJob": {
+                    "exchange": "jobs.write"
+                }
             }
         }
     },```

Thanks for the tip, I will try making the image myself, from the latest release branch.

This has now worked. Catamel tries to contact rabbitMQ but fails due to security issues on the rabbitMQ side
catamel_1        | [+] Logger set to: Console
catamel_1        | Queue configured to be RabbitMQ
catamel_1        | Email settings detected
catamel_1        | Datasource host mongodb_data, database dacat
catamel_1        | Adding ACLS for UserIdentity
catamel_1        | INFO - Adding relations to User
catamel_1        |  {"relation":"UserSetting"}
catamel_1        | INFO - Adding ACLS for User related models
catamel_1        |  {"relation":"UserSetting"}
catamel_1        | Unable to configure Rabbit topology No endpoints could be reached
I will work on my rabbitMQ server, but will close this issue for now.


Fields inside block is editable even if you set access.create and access.update as false

Link to reproduction
https://github.com/jhnsml/payload
To Reproduce

Open Pages collection
Add QuoteBlock
QuoteText has update as false. The field should not be editable

https://github.com/payloadcms/payload/assets/30227295/0d58f28b-203e-42f9-9bea-6058016c101d
Describe the Bug

Fields inside blocks doesn't respect access control create and update setting. It's always editable
This doesn't happen if you add the same field inside tab or the root level of the documen

Payload Version
1.8.3

Hey @jhnsml thanks for reporting this! Took a little while for us to carve out some time to get this implemented. It will be available in the next release.


need to include reference to unlink near discussion of copy, move, and rename of file

The problem
How to remove a file is missing from the general discussion of file operations

Closed with e88d5ac.


Cron update

Context

adds a Cronjob to check for updates
Cronjob can be set like the backup cronjob from @Dashboy1998
Addressing #38

Choices

using the steamapi to get the current build and compare it to the buildId in the file palworld\steamapps\appmanifest_2394010.acf. I am not sure it that is the best way to check. Maybe you have input on this.
to "update" the server, the script will broadcast a message to the players and restart the server in x minutes. This will only work if the env UPDATE_ON_BOOT is set.
I have set the default cron expression to once per hour and the warning minutes to 30 minutes

Test instructions

Set  UPDATE_CRON_EXPRESSION to * * * *
Set WARN_MINUTES to 1
check in the log that the cronjob is running correctly
(Optional) login to the server and create a character
change the buildId on line 11 in the file `palworld\steamapps\appmanifest_2394010.acf`` to any other value (for example 13289561)
Verify that the server restarts in one Minute after the cronjob has started

A Real world test is still needed
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I've added documentation about this change to the README.
[x] I've not introduced breaking changes.

Works for me.

thank you for the review and fixing by bad code. Great Input. You're a legend 💯

Works for me.

thank you for the review and fixing my bad code. Great Input, you're a legend 💯

You're code was pretty good already. I just pointed out a few typos and mentioned a few edge cases.

@Luatan thanks for your work!

@Luatan Tested it locally ran into no issues,  thank you for your amazing work, merging :)


Upgrade our issue templates to use GitHub issue forms ✍

Introduction
GitHub has recently rolled out a public beta for their issue forms feature. This would allow you to create interactive issue templates and validate them 🤯.
Appwrite currently uses the older issue template format. Your task is to create GitHub issue forms for this repository. Please use Appwrite's issue templates as a reference for this PR.
Tasks summary:

[ ] Fork & clone this repository
[ ] Prepare bug report issue form in .github/ISSUE_TEMPLATE/bug.yaml
[ ] Prepare documentation issue form in .github/ISSUE_TEMPLATE/documentation.yaml
[ ] Prepare feature request issue form in .github/ISSUE_TEMPLATE/feature.yaml
[ ] Push changes to master and test issue forms on your fork
[ ] Submit pull request

If you need any help, reach out to us on our Discord server.
Are you ready to work on this issue? 🤔 Let us know, and we will assign it to you 😊
Happy Appwriting!

Hey, can I work on this?

@drph4nt0m Assigned it to you! All the best 🚀

@Meldiron  can I work on this issue? can you assign me?.... Thank you

@Goheljay  We would love you to check out some of our other open issues and join us on discord if you need help finding some
https://appwrite.io/discord


Drupal 8 nginx configuration doesn't serve static pages

Describe the bug
We are using patternlab in Drupal 8 that serve up static content
When going to  the url below, it should be reading the index.html file that lives inside that directory.
https://example.com/themes/custom/custom_theme/patternlab/?p=viewall-global-colors
There is an index.html page that lives in that folder there that it is supposed to hit but does not happened, instead loads the root /index.php file of Drupal.
To Reproduce
Steps to reproduce the behavior:

https://example.com/themes/custom/custom_theme/patternlab/?p=viewall-global-colors
Loads the Drupal 404 page.

Expected behavior
Going to https://example.com/themes/custom/custom_theme/patternlab/?p=viewall-global-colors, should load the pattern lab page.
Additional context
I've changed https://github.com/drud/ddev/blob/master/containers/ddev-webserver/files/etc/nginx/nginx_drupal8.conf#L13 to
try_files $uri $uri/ /index.php?$query_string

That seem to work.

I'm pretty sure that correct drupal config never allows accessing anything directly inside a theme does it? And especially indexing it?
You absolutely can replace the entire config and check it into your project, https://ddev.readthedocs.io/en/stable/users/extend/customization-extendibility/#providing-custom-nginx-configuration

I've ended up extending the nginx config using the following
    location ~ \/patternlab {
        absolute_redirect off;
        try_files $uri $uri/ /index.php?$query_string;
    }

As far as the theme folder, you can access certain files directly within the theme folder but not indexing it.

I'm glad you got this solved, and put the solution here. In general, most nginx and apache configurations for Drupal don't allow direct access to html files, and certainly not "indexing" of directories (which is what index.html is about)
Closing for now, happy to continue the conversation.


Cleaning the whole repo! +remove singletons!

… one commit. bang! Happy? Don't be sad :(

I have modifiy the Position a bit to avoid type collision with np.ndarray. If you need the array form of the position, use pos.array.

I have remove the no inpection in the main files. only the «shitty» file contains them to avoid warning when inspectiing code. less noisy, more fun.


Fancy Zones off by 1 pixel

Microsoft PowerToys version
0.59.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
Open up the Layout editor and set the zone size.
It will show to be set correctly, after confirming it will jump by 1 pixel.
Screen resolution 5120x1440

✔ Expected Behavior
Set the zones to:
1280x1400
2560x1400
1280x1400
❌ Actual Behavior
Zones are set to:"
1281x1400
2559x1400
1280x1400
Other Software
No response

/bugreport


Search users by username first

I ran into an issue where users with short usernames sometimes don't get returned from the search-users API in the first few rows. This is a problem, as the @ dialog in the comment box uses search-users with a limit of 6, so there is sometimes no way to mention a user, even with an exact username match. This change adds a query to search for an exact match username and present that as the first result, if present.
Note: I've not gotten manifold running locally, so I haven't been able to test this.

Hey thanks for submitting this! Could you share a username this works for that our current search doesn't work for?

I was trying to tag Vo username @d on this market.

It's a bit more complicated because if I type zvi it should match ZviMowshowitz and not 'zvi', so not sure how to handle that yet

I support the exact match use case, even with the downside that it might not surface the most popular person.
One solution would be to put the exact match after the first other result!

Yeah, the problem I was trying to fix was mostly that it's currently impossible to tag @d, even with an exact name match, and there's not really anything to do to get more specific. I do think there could be more work put into the scoring of results, but as someone who has worked on search scoring, you can spend a decade on it and still not feel done.
For the ZviMowshowitz example, even if someone registered Z, Zv, Zvi, ZviM, ZviMo, ZviMow, ... you could still tag the full username if you typed the whole thing out. It's an interesting problem, though.
Some other example sites, on X if you start typing @e, the first result is @elonmusk, even though @e exists. Same for the rest of the prefixes (@e through @elonmus).

However, I would also note that if I just left it as-is, the tweet would mention @e, which is different from Manifold where you have to select from the dropdown to trigger a mention. Perhaps it would be a better solution to tag the exact match if it exists on typing @..., even without selecting from the suggestions.

I like the suggestion of making it auto-mention the exact match if no other is selected!


Add this plugin to rockerBOO nvim plugins list

I really like your plugin, it's pretty much what I was looking for in a buffer manager/switcher.
Would be cool to add it to rockerBOO's plugin list, as for me at least it does something that I wasn't quite able to replicate with other plugins.
https://github.com/rockerBOO/awesome-neovim
Anyway, thanks for the cool plugin.

Thanks for your suggestion! It just got merged!


ADC correction

Initialization ADC peripheral may not be corrected. In next iteration of correction errors need have some time to working in this sector

 Список багов и их устранение

Ivalid is complete. Rewrited the function ADC_Init and delete deprecated macro


Taiko generated code often omits a crucial 'await' command, leading to errors when running script

Expected behavior
Successful steps in REPL mode should translate to successful run of associated code generated by Taiko.
Actual behavior
What seems to be happening is that the resultant code doesn't always have an 'await' before goto("url"); and when this happens, running that script from Taiko causes errors (see attached image and file).
Workaround: if user manually edits the generated code to:- await goto("url"); this resolves the issue.
Steps to reproduce

Steps shown in attached Powershell screenshot image.
Notice that often, the Taiko generated Javascript code doesn't run unless user manually edits it to include any missing 'await' command.

Versions
Taiko Version: 1.0.3 (Chromium: 81.0.3994.0)
Node Version: 12.14.1

This happens when you put a space between a command and its opening parenthesis goto (foo) vs goto(foo).  Then the subsequent.split() in isTaikoFunc() results in looking for goto  instead of goto.
Workaround is to not use a space.

@ndbhamvani  This issue has been fixed and the same is available on the master branch.


Fix Error in contributors.js

There is an error in first-contribution/data/contributors.js file due to missing '{'. Can I rectify it?

Assign it to me!

No need to fix that I will fix that. Closing this issue. @devilkiller-ag


FWPM_LAYER_ALE_CONNECT_REDIRECT_V4: Can't redirect to local proxy

Hi all, I am trying to develop a WFP driver which can be used to redirect outgoing TCP connections to a local proxy server. To better understand the different components involved, I tried running WFPSampler project on a new Windows 11 installation:
WFPSampler -clean all
WFPSampler.exe -s PROXY -l FWPM_LAYER_ALE_CONNECT_REDIRECT_V4 -iprp 443 -ipp TCP -pra <IP_ADDRESS> -prp 8866 -plspid 13220 –v

After running this command, it seems that the WFPSampler service is successfully invoked through RPC:
INFO:  ProxyScenarioExecute : RpcInvokeScenarioProxy() [status: 0]

and I can see that the following filter is registered at the ALE_CONNECT_REDIRECT_V4 layer:

Unfortunately, no traffic is ever reaching the locally running proxy server. If I make an HTTP connection to a remote server, it simply succeeds as if there are no WFP rules.
I am new to kernel development and not sure how can I further debug this and whether the callout function is ever invoked. I never managed to get TraceView working.
Any pointers would be greatly appreciated.

Sorry, this Connect/Redirect does not work between loopback addresses :(
I'm trying to get this fixed.


Backwards Compatibility with older Tableau Versions

Receive the following error when accessing from 9.2:
"The version of Tableau that you are using cannot use the web data connector that you are trying to access. The connector requires at least version '1.1.1' of the web data connector API."
Is there a way to maintain backwards compatibility with the older API?

@samdavey can you please share a bit more about how you're hitting this issue? also which connector is this for (one in particular or all of them)?

@samdavey This should be fixed now.  I have reverted the hosted samples on the gh-pages so they will continue to work as expected in Desktop.  Please confirm and I will close this.

Hi team, the issue seems to be resolved. Thanks!
Note that the above error messages were still returned the next time that I attempted to use the connector (Google Sheets Connector), but after ignoring the error and using the Sign In link, I was able to successfully connect. Repeating it a second time did not return an error and again successfully connected.
Thanks and regards,
Sam


Push v0.3.0 to PyPI

@terrytangyuan: The release has been tagged, just needs to be uploaded.

Done! https://pypi.python.org/pypi/metric-learn/0.3.0


Fix bar plot bug when transpose

There is a bug when opening a new bar plot after the last one opened was transposed

Fixed


feat(updates.jenkins.io): dedicated File Share for the httpd service

As the httpd service doesn't need to have a common File Share with the mirrorbits service, this PR disables it from the mirrorbits-parent release, and adds a dedicated httpd release from its own chart so it can use its own updates-jenkins-io-httpd File Share created in https://github.com/jenkins-infra/azure/pull/676.
We're keeping the mirrorbits and the rsync services under mirrorbits-parent umbrella as they still need to share the same updates-jenkins-io File Share.
Corresponding secret added in https://github.com/jenkins-infra/charts-secrets/commit/688566ee492e9026f09289d79ece7f8f4f06cdf3 (private repo).
Follow-up of:

https://github.com/jenkins-infra/kubernetes-management/pull/5175

Ref:

https://github.com/jenkins-infra/helpdesk/issues/2649#issuecomment-2076659536

I'll split this PR in two separate ones to avoid the current error:

Error: Failed to render chart: exit status 1: Error: Unable to continue with install: ConfigMap "updates-jenkins-io-httpd" in namespace "updates-jenkins-io" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "updates-jenkins-io-httpd": current value is "updates-jenkins-io"

Opened https://github.com/jenkins-infra/kubernetes-management/pull/5186 to be merged before this PR.

Bummer...

PersistentVolume "httpd-binary" is invalid: [spec.capacity: Required value, spec.capacity: Unsupported value: core.ResourceList(nil): supported values: "storage", spec.azureFile.secretName: Required value]

Reverted, will rework it later.


Add next_day Spark function

Add next_day Spark function. #4741

@mbasmanova  Could you help review this fuction?

@mbasmanova  Could you help review this fuction?

@PHILO-HE ok.


Feat(global): Implement Preview Database for Isolated Branch Testing

To Do:
Requirements:

[x] Making changes to data used in a branch does not affect the production app.

Added a development database.


Add an FAQ

A first pass at an FAQ.

That should address your comments.

And again :). PTAL

:+1: and awesome, after addressing the last comments :)

I think we had a race condition, how's that?
Also updated community.html.

Cool, just needs an LGTM by @discordianfish about the Docker stuff.

LGTM!

Merging this for now, as the contributors file is a separate issue (but we should look into having a broken-link checker for the docs to detect if/when we e.g. rename it).


fix: fix the informer panic issue

What type of PR is this?
/kind bug
What does this PR do?

Which issue(s) does this PR fix?

Fixes #378
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/lgtm

/approve

/lgtm
/approve


No Belongs-To-Many relation to current model in BREAD

Laravel Version: 5.5.20
Voyager Version: 1.0
PHP Version: 7.1.8
Database Driver & Version: MySQL 5.6.35

Description:
I would like to create a "belongs-to-many" relation from article to article which is called "related articles".
When I try to do this I can choose a pivot table but the system automatically chooses the keys for the relation. In this case it tries to use twice article_id as keys which is obviously not possible.
The fix would be to allow users to customize the key names which should be used inside of the pivot.
Steps To Reproduce:

Create a database table "test"
Create a database table "related_tests" with test_id and related_id as columns
Create a bread for this table
Add relationship
Choose belongs to many with test as target
Choose related_tests as pivot

Hello,
found out a solution to this problem:
create a new model that extends your model, and specify the model table
namespace App;
use Illuminate\Database\Eloquent\Model;

class RelatedTest extends Test
{
 protected $table = 'tests';
}

so now you have a copy of the model that uses the same table but has a different name that can be matched in pivot table.
in this case you can create a pivot table named "test_test" with "test_id" and "related_test_id" columns


Dynamic styles are broken:  Internal server error: Qrl($) scope is not a function

Qwik Version
0.11.0
Operating System (or Browser)
macOS
Node Version (if applicable)
16.x
Which component is affected?
Qwik Runtime
Expected Behaviour
It should parse CSS with variables in the template string.
Actual Behaviour
When you try to use useStyles$ or useStylesScoped$ it fails if you put any variable in the string.
For example this breaks:
const className = "test"
const css = `.${className} { background: red; }`;
useStyles$(css);

with the following error:
Qrl($) scope is not a function, but it's capturing local identifiers: className
9:41:56 PM [vite] Internal server error: Qrl($) scope is not a function, but it's capturing local identifiers: className
  Plugin: vite-plugin-qwik

Same error happens if I try to make any or all of the CSS string dynamic.
But this runs fine without any issues.
const css = `.test { background: red; }`;
useStyles$(css);

The string in the end should be the same.
Additional Information
Unrelated, but please consider supporting a CSS in JS solution rather than string templates 🙏 They are just all around easier to deal with.

Duplicate of #1563 imho

I am closing this issue but keeping this open: https://github.com/BuilderIO/qwik/issues/1728


Nicer syntax for (de)serialization

Add operator>>(std::string&) and operator<<(const char*) to serialize and parse json.
This would provide symetric operators for symetric operations and allow the following simple initialization:
Json a << R"({"foo":"bar"})";

Overloading << or >> operators would give the false impression that Json behaves as a stream. Instead, serialize and parse methods will be overloaded with an extra version accepting std::ostream and std::istream respectively.

Finished implementing istream/ostream interface in commit 54a61d58330757a601d80db23e7dd3e24a0f50c2


Add UNICEF as a supported source

Update sources.json, docs, tests, and What's new.
UNICEF returns ReportingBegin and ReportingEnd header fields; add these to message.Header
Use Python's built-in datetime objects for Header fields, read/write strings in XML.

Two uncovered lines in the codecov/patch check are in the source tests that will be run nightly, so not an obstacle to merging.


Create Models In SQLAlchemy

As per the DB Schema (present in dropbox paper), we should make SQLAlchemy models.
The following guidelines to be followed:

The current schema will be barebone and we will need to update them with constraints.
Each model should be placed appropriately in the right file and module.

cc/ @mukul-mehta @grapheo12 Please coordinate on this :small_airplane:

Please take your time, learn about SQLAlchemy and use best practices while making the models.

@mukul-mehta Since we're using getters and setters for validation checks. Lets follow the synonym method described in this link.
Actually, while using setters I found that setter name and actual attribute (preferably private) must be different otherwise program runs in infinite loop.
Please comment on what you think.

This looks cool.
On Fri, Jul 12, 2019 at 9:57 PM Shubham Mishra<EMAIL_ADDRESS>wrote:

@mukul-mehta https://github.com/mukul-mehta Since we're using getters
and setters for validation checks. Lets follow the synonym method
described in this link https://gist.github.com/luhn/4170996.
Actually, while using setters I found that setter name and actual
attribute (preferably private) must be different otherwise program runs in
infinite loop.
Please comment on what you think.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/iit-technology-ambit/common-backend/issues/17?email_source=notifications&email_token=AH2K2IG64HMF4A7RFC6VSV3P7CWFPA5CNFSM4H5JDEGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ2HOMQ#issuecomment-510949170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH2K2IDWX3T6BM272IFNMC3P7CWFPANCNFSM4H5JDEGA
.

Closing this since we have implemented.


creation of Issue type page

…content/en/internal/helpdesk/features/issue-type-values/_index.md
moving this page https://wiki.egi.eu/wiki/GGUS-IssueType-Values
new PR created since the one made by Renato had some issues. So I copied the file created by him.

Summary

Related issue :

@paolini78 I would have expected we would have discussed and fixed everything in the PR  #404 created by @renatoEGI instead of preparing another one.
Also you shouldn't have deleted the page content/en/internal/helpdesk/features/_index.md but you should have added proper content to it, please add it back and include something, even if it's just a sentence.
I will look at addressing the errors reported by the linters and reject the PR #404 prepared by @renatoEGI.

I've added back the missing index page, seems content was mixed in the other file.
Is all this information up to date?


Vault specific journal files

Please Select if your Request is Either something new or an Enhancement

[ X ] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ X ] UI
[ ] Workbench
[ ] Publishing
[ ] Pods
[ ] Markdown
[ ] Hierarchy
[ ] Graph
[ ] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
When creating a Daily Journal with more than one vault in a vscode workspace the markdown file can't be created in a specific vault.
Describe the solution you'd like
If there are multiple vaults in a vscode workspace ask which vault to create the markdown file in.
Describe alternatives you've considered
I can't think of any, but right now I am moving the journal file into the appropriate vault after creation.
Additional context
I can provide some if needed.

@jeffabailey - sorry for the late response but this config should be what you're looking for: https://wiki.dendron.so/notes/f83c1d87-eac0-48f3-a5cf-8a69989d8ec1.html#lookupconfirmvaultoncreate
Just drop this at the end of dendron.yml and you should be golden
lookupConfirmVaultOnCreate: true

Closing since this should already exist. Please reopen if you're still finding issues here @jeffabailey


Display Tooltip outside of Chart

Is it possible to display the tooltip outside of the '.line-chart  .metadata-group' classes?
I'm trying to do this here, and running into errors: http://jsbin.com/quqipiy/edit?html,js,console,output
It almost works, but doesn't output the actual value, which is really all I want (and the date).
This would ideally allow me to show tooltip-like functionality in any situation where space was a significant constraint.
Expected Behavior
Date and Value should be shown
Current Behavior
Only date is shown
Steps to Reproduce (for bugs)
http://jsbin.com/quqipiy/edit?html,js,console,output
Context
In limited spaces, it would be useful to output chart values to another location/div to allow flexibility in revealing chart details.

Hi @hisnameisjimmy, thanks for the question!
Yes, you can do anything with the information you get from customMouseOver/Out/Move.
In general, those handlers will give you:
function(e, d, [x, y]){}
where e is the event, d is the datapoint you are hovering and x and y the position of the mouse in that moment. Probably the datapoint will have the data you need, along with the date and any extra info you are passing to the chart.
You can use that information to update any other element. You don't need to use a tooltip at all.

Thanks for getting back @Golodhros, not to be greedy, but is there anyway you could show me an example of this?

Sure, no problem!
It will be similar to what we do in the brush chart demo: https://github.com/eventbrite/britecharts/blob/master/demos/demo-brush.js but instead of using brushStart/End you will use the mouseOver/mouseMove events.


Display HTTP URL in /apps list

Just as the title says, instead of http, display https:/x.y.z for installed HTTP apps

Any security issues that may arise from this? Should end-users be able to know where the http server resides?

Codecov Report

Merging #325 (70f54aa) into master (9e0ed4b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 70f54aa differs from pull request most recent head 66cdab9. Consider uploading reports for the commit 66cdab9 to get more accurate results

@@           Coverage Diff           @@
##           master     #325   +/-   ##
=======================================
  Coverage   28.17%   28.17%
=======================================
  Files          76       76
  Lines        4479     4479
=======================================
  Hits         1262     1262
  Misses       3088     3088
  Partials      129      129

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 9e0ed4b...66cdab9. Read the comment docs.

Sorry just pushed another update, please review again

Any security issues that may arise from this? Should end-users be able to know where the http server resides?

@mickmister no - it's a sysadmin-only command.


minimatch deprecated - please upgrade

from Travis CI, npm install command:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

please do

upgraded in #24


New Ergo Compiler + Concerto 1.0

Changes
Build

Upgrade from OCaml 4.07 to  OCaml 4.11
Upgrade from Coq 8.8 to Coq 8.12
Switch from ocamlbuild to dune build system

Compiler

Switch backend from Q*cert 1.4 to Q*cert 2.1 (See release notes here: https://github.com/querycert/qcert/releases)
Redesign of compiler architecture
Proof of correctness for Ergo calculus to Q*cert initial IL
Complete code refactor

Runtime

Migrate to Concerto 1.0
Migrate from moment to dayjs

API

Additional utcOffset parameter can be passed to Ergo engine

Tech notes:

BREAKING The Ergo compiler no longer generates ES5, but ES6 (notably the generated code uses let)
New compiler includes several additional ILs on the backend side, notably generates code using the Imp imperative intermediate language

Wow! Good work, Jerome!
I've reviewed as far as I can, however my understanding is limited. It's difficult to review such a large PR cold.

Perhaps a paired review would help? Or are there particular areas that you're looking for feedback on?

Not really need too much feedback on this one. A review of the new compiler is a pretty significant effort. Happy to do that but this should be planned. Generally speaking the new compiler is a massive improvement on the old one (should be much more robust and provide real solution to very tricky issues that could happen with the previous one e.g., #485). There are other major improvements in code organization, notably for the runtime part which should make the code much more maintainable. I'm sure I forget other things.

I was interested to see QcertToScala.v and QcertToSpark.v, what are the implications of that?

No real implication. The Q*cert backend emits for multiple languages (there is a still remaining bit for Spark which isn't really maintained but is around). But you need to provide a few hooks for those to instantiate the compiler. For Ergo this isn't used.

Do you have an updated version of the compiler architecture diagram, please?

Best I have at the moment is this: https://vimeo.com/showcase/6378459/video/385496040


feat(ci): Dependabot config

feat(ci): Dependabot config
Overview
Introduces jonhoo's dependabot config, which notifies on outdated github
actions as well as outdated crate dependencies.
chore: Bump dependencies
Overview
Bumps dependencies in the workspace, before dependabot gets to them.

#236  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @clabby and the rest of your teammates on  Graphite


[1.10-rc2] Error initializing network controller: could not delete the default bridge network: network bridge has active endpoints

I just got this error and now my 1.10-rc2 daemon won't start:
FATA[0001] Error starting daemon: Error initializing network controller: could not delete the default bridge network: network bridge has active endpoints

This was encountered when doing a hard kill on the docker daemon with about 100 containers running on the default bridge network.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Here's a bit more info from the logs
DEBU[0000] Registering ipam driver: "default"
ERRO[0001] getNetworkFromStore for nid d5eece51105d49cce8874265e1ba30f2c3a9c9b42443f6edcd9a700c09889b2b failed while trying to build sandbox for cleanup: network d5eece51105d49cce8874265e1ba30f2c3a9c9b42443f6edcd9a700c09889b2b not found
ERRO[0001] getEndpointFromStore for eid 381b95e2725b5812607f4eb1e7d9a5a3c9a790b5e968fe9f6507d581238f3755 failed while trying to build sandbox for cleanup: could not find endpoint 381b95e2725b5812607f4eb1e7d9a5a3c9a790b5e968fe9f6507d581238f3755: []
WARN[0001] Failed getting network for ep c13dd53a0b2391818c85e42f26983a3bb94197613fbea226912ffe37b38bbfbd during sandbox de5b822a465563c1b512c822633e98750776efa5579325392f51db1e54dcc1c9 delete: network d5eece51105d49cce8874265e1ba30f2c3a9c9b42443f6edcd9a700c09889b2b not found
WARN[0001] Failed deleting endpoint 381b95e2725b5812607f4eb1e7d9a5a3c9a790b5e968fe9f6507d581238f3755: failed to get endpoint from store during Delete: could not find endpoint 381b95e2725b5812607f4eb1e7d9a5a3c9a790b5e968fe9f6507d581238f3755: []

ERRO[0001] failed to delete sandbox de5b822a465563c1b512c822633e98750776efa5579325392f51db1e54dcc1c9 while trying to cleanup: could not cleanup all the endpoints in container d4ad4ae2fcb3d374f39c181106e1734ce84f8a65aae7107fde1c7af9acc46b0b / sandbox de5b822a465563c1b512c822633e98750776efa5579325392f51db1e54dcc1c9
DEBU[0001] Cleaning up old shm/mqueue mounts: start.
FATA[0001] Error starting daemon: Error initializing network controller: could not delete the default bridge network: network bridge has active endpoints

there are Network not found and could not find endpoint errors during the cleanup process (due to unclean shutdown) that happens in bootup. But, the container d4ad4ae2fcb3d374f39c181106e1734ce84f8a65aae7107fde1c7af9acc46b0b seem to be refering to this network and endpoint. Since you confirmed that all the containers belong to the default network, I would assume that the above missing network is that of default network (docker0). Do you know why the network and store be missing from the store ?

Adding this to the 1.10.1 milestone; we'll investigate further after the release

Ping @ibuildthecloud

Ping @ibuildthecloud

Closing. Might be already fixed. Feel free to comment when you have new info.

I get the same when ipv6 is enabled:
root@main:~# docker version
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   20f81dd
 Built:        Thu Mar 10 15:54:52 2016
 OS/Arch:      linux/amd64
An error occurred trying to connect: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.22/version: read unix @->/var/run/docker.sock: read: connection reset by peer

Yes, got the same guys. Any clue?

@matheusgontijo "got the same" as the original issue, or problem connecting to /var/run/docker.sock ? What version of docker?  If you are on Docker 1.13, please check if it's https://github.com/docker/docker/issues/30295; check if your name server has an %<iface> in it's address;  see https://github.com/docker/docker/issues/30295#issuecomment-273983364 (and https://github.com/docker/docker/issues/30295#issuecomment-275315635). If it's the same; this issue has been fixed and will be in the docker 1.13.1 release.
Otherwise, please open a new issue with details - commenting issues that were closed over a year a go is not the best way to report bugs, they may go unnoticed and can be unrelated to the issue you're commenting on

Thanks @thaJeztah for the reply. Yes, I already fixed my issue. Everything I had to do was to install a previous version "1.12.0" of Docker and everything worked fine.


Consider switching to flutter_timezone

flutter_native_timezone has been abandoned for more than 2 years.
I've created a fork, flutter_timezone that you might consider switching to instead as some users are starting to go get build issues with the old dependency.

@tjarvstrand , I checked the latest code, the deps has been deleted.
I think we need it published to pub.dev with a patch fix like 4.3.2?

I have no opinion of matter.  I took on the dependency and it broke my build.  Just trying to help with a patch for updates - as they can build with minor tweaks.  If the forked dep should be used cool, removing it is fine too just wonder what we lost in taking it out.

Any update?

Sadly all current and past maintainers have next to no time left. Tbh most of us became maintainers to fix a bug.
To get this issue fixed as soon as possible, it would be best if someone provides a pull request that is ready to be merged.

No changes are needed, only a release to get the changes out that are already in main

@tjarvstrand I don't see the fork flutter_timezone being used on master. To be clear, don't we need a PR replacing flutter_native_timezone with flutter_timezone?

The example project is still relying on flutter_native_timezone and it's included in the dependencies as a result.

The example project is still relying on flutter_native_timezone and it's included in the dependencies as a result.

No, that shouldn't affect the dependencies of the released package.
The last release was 4.3.1, made 5 months ago, apparently from master. There seems to have been some cherry-picking going on from develop to master but notably, this commit does not seem to have been included, which is why the 4.3.1 still has the flutter_native_timezone dependency.

As a workaround you can use the following snippet in your pubspec.yaml

That's great. I think there's a risk however, of other people hitting the same issue and the probability that they will stumble upon this thread isn't super high. It could be worth making a bump to 4.3.2 (or even 4.3.1+1 with what's actually in the tagged commit) to avoid that scenario.
Just my 2 cents though, as I'm not really affected by this :)


bundle install fails on WSL when using a git repository with a path

I'm currently using Docker for Windows (1.12.6) and WSL on Windows 10 (Linux DESKTOP-DLRRFKC 3.4.0+ #1 PREEMPT Thu Aug 1 17:06:05 CST 2013 x86_64 x86_64 x86_64 GNU/Linux)
I'm having issues when running bundle install --path vendor/bundle, note that this ONLY happens in WSL when writing to NTFS, both in Linux and OSX this works fine
`git clone --no-checkout --quiet
"/myapp/vendor/bundle/ruby/2.3.0/cache/bundler/git/bootstrap-sass-b9ed78534df7520565c4f55770e38b8d270f127f"
"/myapp/vendor/bundle/ruby/2.3.0/bundler/gems/bootstrap-sass-cf9a9b181fdf"` due
to error (4/4): Bundler::Source::Git::GitCommandError Git error: command `git
clone --no-checkout --quiet
"/myapp/vendor/bundle/ruby/2.3.0/cache/bundler/git/bootstrap-sass-b9ed78534df7520565c4f55770e38b8d270f127f"
"/myapp/vendor/bundle/ruby/2.3.0/bundler/gems/bootstrap-sass-cf9a9b181fdf"` in
directory /myapp has failed. If this error persists you could try removing the
cache directory
'/myapp/vendor/bundle/ruby/2.3.0/cache/bundler/git/bootstrap-sass-b9ed78534df7520565c4f55770e38b8d270f127f'error:
internal error: refs/remotes/origin/2.0-stable is not a valid packed reference!

I've put up a repository with an example of what I mean at https://github.com/omarqureshi/not_valid_packed_reference
I'm pretty sure that this is a Git issue and will also be looking to report this to the Git mailing list, however, they may equally say that this is a Docker and/or NTFS issue
The volume mount is the cause of this https://github.com/omarqureshi/not_valid_packed_reference/blob/master/docker-compose.yml#L8
I can get around this by NOT using --path, however, it would be preferable if this just worked on WSL too.
If you'd like any other information, please let me know, otherwise, the Dockerfile should have enough information there to let you know what my setup is like.

I'm honestly not seeing how this could be a bundler issue if the same git command works everywhere else -- it looks like an issue either with git or WSL to me.

I personally think it's more of an issue with Git interacting with NTFS, however, given that the use of bundler is the only place that I have experienced this (and I have successfully been using Docker, Git and WSL on other Rails apps before this), it's probably not a bad place to put this issue.
I'll probably try testing this by installing Ruby on WSL and then bundler too.

Can you run the git command successfully outside of bundler?

I've tried a git clone --no-checkout bootstrap-sass/ other-bootstrap/ and that seems to work, however, that's not quite the same here

What exactly does a bundle install do when given a github repository and a tag?

Also, with the example, it makes no difference whether I specify a tag or not.

There was a similar issue recently on Git for Windows with fix https://github.com/dscho/git/commit/2174823bd176818aa072d2137070d81de737831a and a Git test https://github.com/git/git/commit/16615f0fbc445ef958f2453c926526d5e2280700 with fix https://github.com/git/git/commit/7814fbe3f1e7cea1a675408df7b5dd1f5be731cc
Check if it is relevant

I'll have a look later on this evening, thanks!

I'm closing this for now. If you're still experiencing your original issue don't be afraid to re-open this ticket.