undefined method `new_file'

New Issue Checklist

[x] Updated struct to the latest version
[x] I have read the documentation to make sure my issue isn't covered already

Struct version
2.2.1
Spec version
2.2.1
Which type of spec are you using?

[x] project.yml
[ ] project.json
[ ] Specfile

Context
Running struct generate through Jenkins is giving me this error:
struct generate
Processing project 'project'...
undefined method `new_file' for #<Xcodeproj::Project::Object::PBXFileReference:0x007ffe01baf958>

I'm also using Fastlane to invoke struct generate, and running the same fastlane lane after copying the Jenkins workspace contents to a different folder works fine.  This error only happens when running through Jenkins.
Expected result
Expected project file to be created
Actual result
undefined method new_file' for #Xcodeproj::Project::Object::PBXFileReference:0x007ffe01baf958` error

Hi @steveholt55, thanks for filing this issue. Would you be able to upload your project.yml so I can take a look? Either that or a simple test case would be very helpful. Thanks!

@lyptt - This is an example of my project.yml.  I had to change a few things related to my project but this is the gist of what I'm doing. project.txt.  I'm also using this for Jenkins (https://github.com/stisti/jenkins-app)

Thanks for this! I think this may just be purely an indentation issue. Unfortunately the error struct is giving here isn't that helpful. Line 36 should be two spaces to the left in order to ensure it contains the 'sources' statement, rather than sitting alongside it.
Would you mind giving that a try?

I'm closing this issue for now, @steveholt55 please let me know if you're still having issues!

@lyptt I'm still having issues with this, updated to struct 3.1.0 and I'm still getting the error.  It isn't the spacing issue because I'm able to run struct perfectly if I just cd to my Jenkins directory. The issue only happens when running the struct command inside of Jenkins.  Also the spacing issue was just because I was changing some of the real values before I attached it here and didn't look over it close enough.

This is the full stack trace I get.  You're seeing lots of fastlane because I'm running struct from fastlane.
Processing project 'project'...
undefined method `new_file' for #<Xcodeproj::Project::Object::PBXFileReference:0x007fe3da879e28>

	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane_core/lib/fastlane_core/ui/ui.rb:14:in `method_missing'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/helper/sh_helper.rb:56:in `sh_control_output'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/helper/sh_helper.rb:14:in `sh_no_action'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/fast_file.rb:180:in `block in sh'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/actions/actions_helper.rb:50:in `execute_action'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/fast_file.rb:179:in `sh'
	from Fastfile:61:in `block (2 levels) in parsing_binding'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/lane.rb:33:in `call'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:204:in `try_switch_to_lane'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:156:in `trigger_action_by_name'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/fast_file.rb:146:in `method_missing'
	from Fastfile:34:in `block (2 levels) in parsing_binding'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/lane.rb:33:in `call'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:204:in `try_switch_to_lane'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:156:in `trigger_action_by_name'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/fast_file.rb:146:in `method_missing'
	from Fastfile:108:in `block (3 levels) in parsing_binding'
	from Fastfile:104:in `each'
	from Fastfile:104:in `block (2 levels) in parsing_binding'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/lane.rb:33:in `call'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:49:in `block in execute'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:45:in `chdir'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/runner.rb:45:in `execute'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/lane_manager.rb:54:in `cruise_lane'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/command_line_handler.rb:30:in `handle'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/commands_generator.rb:104:in `block (2 levels) in run'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/gems/commander-fastlane-4.4.5/lib/commander/command.rb:178:in `call'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/gems/commander-fastlane-4.4.5/lib/commander/command.rb:153:in `run'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/gems/commander-fastlane-4.4.5/lib/commander/runner.rb:476:in `run_active_command'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:66:in `run!'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/gems/commander-fastlane-4.4.5/lib/commander/delegates.rb:15:in `run!'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/commands_generator.rb:303:in `run'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/commands_generator.rb:42:in `start'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/fastlane/lib/fastlane/cli_tools_distributor.rb:66:in `take_off'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1@global/gems/fastlane-2.62.0/bin/fastlane:20:in `<top (required)>'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/bin/fastlane:23:in `load'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/bin/fastlane:23:in `<main>'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `eval'
	from /Users/brandonjenniges/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `<main>'

@lyptt I added a test project demonstrating the issue folder.zip.  Note that after extracting the .zip you'll see a folder called non_hidden. There is also a folder that is hidden called .example Both folders should have identical contents, the only difference is one is hidden and the other is not.
Running cd .example/project;struct generate; from the root extracted folder yields this error for me:
undefined method `new_file' for <PBXFileReference path=`folder1` UUID=`D6789A7925358B92B4143AE2`>:Xcodeproj::Project::Object::PBXFileReference

But running cd non_hidden/project;struct generate; from the root works perfectly fine.

Hi @steveholt55, thanks for the sample project! I'll check this out and see what's going on.

I believe I've identified a fix for this issue, thanks for the additional help in reproducing this!

Verified this fixes my issue. Thanks!

Great! I'll release a new version with this fix in shortly.


#nowarn "211" in loaded script files is ignored

When you have a script with #I that points at a non-existing folder, but you also have #nowarn "211" to hide the warning, you still get a warning.
To reproduce, create a file, say C:\temp\nada.fsx containing
#nowarn "211"
#I "totally-non-existing-folder"

And then run the following in F# Interactive:
Microsoft (R) F# Interactive version 14.0.23020.0
Copyright (c) Microsoft Corporation. All Rights Reserved.
For help type #help;;

> #load @"C:\temp\nada.fsx";;

[Loading C:\temp\nada.fsx]
C:\temp\nada.fsx(2,1): warning FS0211: The search directory
    'C:\temp\totally-non-existing-folder' could not be found
namespace FSI_0002

Is this supposed to be this way, or is it a bug? /cc @dsyme

this is fixed in #1239, right? can we close it?

Yes, thanks


Fix simple clippy warnings

There are a few more clippy warnings, but those aren't as clear-cut as the ones I fixed in this PR.

Thanks!
Any thoughts on how to approach the one in uniform? I don't think uniform distribution is very well defined over an infinite (or semi-infinite) interval, but I'm not a formal mathematician.
If you agree, I'll ensure boundedness in new and drop the additional checks.
After all, calls to our sample would fail in UniformFloat::new in the rand crate and there's not a way to define inverse_cdf.

Any thoughts on how to approach the one in uniform?

I think the logic itself is fine, clippy just doesn't like that there are two else if branches because that could be done in one:
} else if x >= self.max || x.is_infinite() && self.max.is_infinite() {

That's less readable though, so I would just keep it as-is and add an #[allow] attribute.

Regarding the logic, is it not redundant?
If the first case is false, then self.min > NEG_INF and x > NEG_INF
If the third is true, then x = INF and self.max is either NEG_INF (implies x>) or INF (implies x=)
Both options would have satisfied the second, no?

Can you share what you think that use case might be? I'll take a chance to expand my brain.

Regarding the logic, is it not redundant?
If the first case is false, then self.min > NEG_INF and x > NEG_INF If the third is true, then x = INF and self.max is either NEG_INF (implies x>) or INF (implies x=)
Both options would have satisfied the second, no?

You're right. The other cases already neatly handle bounds being +/- infinity. I only skimmed the code because clippy is complaining about something else, but the entire block can be removed AFAICT.
Makes me wonder if the else if self.max.is_infinite() below that is also redundant, but I can't be bothered to do the logic in my head right now :P

Can you share what you think that use case might be? I'll take a chance to expand my brain.

Well, my original argument was going to be about API flexibility and that even if it might not be mathematically defined, it could still be useful to "just handle it" transparently instead of erroring. That said, it does look like the uniform distribution requires finite bounds from what I can find online.

Okay, I'll fix that separately since it's a behavior change instead of linting.

Added some more trivial clippy fixes, some from nightly.

Think this is all of them that we can handle as lints then, right? Merging, then I'll put in the change for Uniform.
Not sure how I want to handle that one in Empirical. It does seem that ordered_float's NotNan<T> has settled since Empirical was introduced; perhaps we can let clippy allow it for merging CI.


Too many admin roles.

16 Admin Roles ! Sixteen! Well done Micro$oft for making life 16 times more complicated than it needs to be.
[Enter feedback here]

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

ID: f18d8345-78f3-0b16-f585-1e53f49d138f
Version Independent ID: 015e98be-2db5-11a3-9abc-6f89b44c3f96
Content: About admin roles in the Microsoft 365 admin center - Microsoft 365 admin
Content Source: microsoft-365/admin/add-users/about-admin-roles.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@programify Thank you for your feedback. These admin roles are unique in their own way. Thank you for providing feedback to help improve a Microsoft product! To make sure that your feedback gets routed to the appropriate team, please post it on the Microsoft 365 Q&A portal.
Hope this helps!
Thanks
Sri


Update documentation for fixing lib issues

//CC: @zhengbli

Can you add a README.md in src/lib that simply has the added content? Casual contributors might not even see this change otherwise.

Can you add a README.md in src/lib

I could, but most PRs we got were changing the file in the LKG folder (lib) instead. not sure if ppl even thought about looking under src

I definitely looked in lib at first, not src/lib. CONTRIBUTING is the right place.

I could, but most PRs we got were changing the file in the LKG folder (lib) instead. not sure if ppl even thought about looking under src

Sounds like you should also put a README in the lib directory saying "DO NOT TOUCH THIS." We shouldn't be averse to pointing users in the right direction once they're going in the wrong direction.


Update Gradle to 7.5

I had troubles with publishing to mavenLocal and using Gradle so I updated it

I will look into why Bitrise is falling


Update forms.md to fix some nits

I think we should cover replaced elements elsewhere or in a Go Further. Filed an issue to track in #16.


Unable To Remove Gridmap

Godot version:
eb9fa02
OS/device including version:
Manjaro Linux
Issue description:
Unable to remove gridmap with right click. Cursor instead moves to center.
Steps to reproduce:
Open Platformer 3d demo.
Select gridmap from Inspector.
Add gridmap with left click (works).
Remove Gridmap with right click (failes).

Have the docs been updated?

Have the docs been updated?

I don't know, can you point to the docs you were referring to?


unit-MixtureModel is flaky

fails every now and then....

although it always fails on zesty and yakkety builds: http://buildbot.shogun-toolbox.org:8080/#/console


HSEARCH-2213 Cryptic error

https://hibernate.atlassian.net/browse/HSEARCH-2213
Built on top of https://github.com/hibernate/hibernate-search/pull/1069
Not sure how to test it now that we have relaxed the status to yellow.
But the error message now explains that you should check the status of the cluster.
It also shows the status at the time of the request

Sending another one with a different message


Setup.py Clobbering config.py

We currently use setup.py to drop a config.py into /opt/hacksports/; however, this doesn't protect an already existing configuration.
We should remove this functionality and instead move it to picoCTF-platform where we copy the config file over.
I have ran into the issue where upgrading the shell manager ruins my deployment because the config has changed. This is not good.

This actually turns out to be a regression from #25 where the install.sh would protect the configuration file from setup.py.
I think the better solution would be to remove setup.py's responsibility of placing the configuration file and instead just makes the directory. Then when shell_manager is called it checks to see if a configuration file is present. If one does not exist it drops a default configuration file and exists alerting the user. If there is a configuration file all goes as expected.


fix: defaultCellMinWidth better cross-browser support

It turns out the approach taken in https://github.com/ProseMirror/prosemirror-tables/pull/253 was not supported well on safari. It doesn't support the min-width property on col elements.
This PR takes a new approach by setting the min-width on <td> and <th> elements directly. The selector makes sure this is only added when the cells don't have an explicit width (colwidth) set. We also needed to make sure the min-width is not set when we're actively resizing a column; because in this case, the cells will not have a colwidth yet as the resize action is "pending". To resolve this, we added a decoration to the cell nodes that are being resized.

It turns out the approach taken in https://github.com/ProseMirror/prosemirror-tables/pull/253 was not supported well on safari

Which Safari version did you test? Safari v18.0.1 seems to work well as shown in the screenshot below, where the empty cell has a default 100px min width correctly.

I tested with safari v15 (currently replying from mobile, so can check exact version later). I had someone else report the same issue with chrome on windows.
For testing; good way to test is to remove the width: 100% from the table in css, and then add a column all the way on the right via the toolbar. But issue might be fixed in safari 18 I suppose (i'm one mac upgrade behind)

Thanks @YousefED


let cpu submission node to run as long as possible

currently, we have
https://github.com/marrlab/DomainLab/blob/24e4c5f6933d7863b96f9615fb88a495947ebf2f/sbatch4submit_slurm_cpu_node.sh#L20
change this to the maximum according to the cluster resources so that snakemake could run 4 days or even longer instead of 2 days.

https://github.com/marrlab/DomainLab/pull/599,
we might also change cpu_normal to cpu_long conditioning on user input

commit f899cd0b6a26743b44be01a5c8858c4070020de4,
we have different script for different length submission
use
bash submit_slurm_via_cpu_10days.sh xx.yaml 0
or
bash submit_slurm_via_cpu_3days.sh xx.yaml 0


How to make segment of index.html as template

Hi I am trying to create a form wizard with different views. I have different segments of form in a html file returned from the server with PHP which I want them to be display as different views on the front end. Very similar to http://plnkr.co/edit/M03tYgtfqNH09U4x5pHC?p=preview
How can I make a segment of form like template: '#segment' as a template instead of templateUrl: 'template.html'

http://bit.ly/UIR-SOF


[doc] Fix typo in network policy document

Fix #431 which is the typo in network policy doc.
Test Plan

[x] acceptance tests

[ ] …

References

/ok-to-test sha=62c92ab

/ok-to-test sha=62c92ab


Load service descriptors from proto files

It seems nodejs supports loading service descriptors from proto files.  http://www.grpc.io/docs/tutorials/basic/node.html#loading-service-descriptors-from-proto-files
Does any of the other language supports this feature?

No, only the Node library has that feature.


Expose kernel  API

For #
TODO:

[ ] Allow anyone to use this in debug mode
[ ] Add telemetry for usage

Codecov Report

Merging #8545 (8d9e118) into main (ea6a20d) will decrease coverage by 0%.
The diff coverage is 10%.

@@          Coverage Diff           @@
##            main   #8545    +/-   ##
======================================
- Coverage     71%     71%    -1%
======================================
  Files        378     381     +3
  Lines      23985   24347   +362
  Branches    3692    3738    +46
======================================
+ Hits       17266   17451   +185
- Misses      5227    5372   +145
- Partials    1492    1524    +32

Impacted Files
Coverage Δ

src/client/api/types.ts
100% <ø> (ø)

src/client/common/utils/async.ts
40% <0%> (-8%)
:arrow_down:

src/client/datascience/jupyter/jupyterSession.ts
71% <0%> (-2%)
:arrow_down:

src/client/datascience/types.ts
100% <ø> (ø)

src/client/extension.ts
61% <0%> (+2%)
:arrow_up:

...science/jupyter/kernels/kernelConnectionWrapper.ts
3% <3%> (ø)

src/client/api/kernelApi.ts
13% <13%> (ø)

src/client/api/apiAccessService.ts
22% <22%> (ø)

src/client/api.ts
46% <33%> (-6%)
:arrow_down:

src/client/api/serviceRegistry.ts
100% <100%> (ø)

... and 55 more

Looks good, just had some questions about where the KernelConnectionMetadata types are defined (seems like they're in more than one spot)


Webhook follow up

Summary
This PR implements a couple of suggestions from https://github.com/mattermost/mattermost-plugin-apps/pull/114. The most eminent is the usage of crypto/subtle to compare the secret.
Ticket Link
Follow up on https://github.com/mattermost/mattermost-plugin-apps/pull/114

@levb I thought about dropping the JSON unmarshaling as mentioned in https://github.com/mattermost/mattermost-plugin-apps/pull/114#discussion_r607633964. What do you think about this?


docker error!

when i run the command:
docker build -t ledger-ref .
get the error like this:
E: The repository 'http://security.ubuntu.com/ubuntu groovy-security Release' does not have a Release file. E: The repository 'http://archive.ubuntu.com/ubuntu groovy Release' does not have a Release file. E: The repository 'http://archive.ubuntu.com/ubuntu groovy-updates Release' does not have a Release file. E: The repository 'http://archive.ubuntu.com/ubuntu groovy-backports Release' does not have a Release file.
how can i slove this problem?

i try to change the ubuntu version, the problem is sloved. thanks


Mr

NY SORY

Looks like this was created by accident, closing. If needed, feel free to reopen with more details, thank you!


Multiple Event Sources Only Using First

I've got two eventSources: a JSON feed and a function that returns events. The problem is that only the first eventSource is getting called. Am I missing something?
    data: function () {
        var self = this;
        return {
            eventSources: [{
                events: function (start, end, timezone, callback) {
                    var events = getEvents(start, end, timezone);
                    callback(events)
                },
                events: function (start, end, timezone, callback) {
                    self.$http.get('//myjsonfeed', {
                        params: {
                            start: start.format(),
                            end: end.format()
                        }
                    }).then(function (response) {
                        callback(response.data);
                    });
                }
            }],`

Hi @noynek,
Your eventSources array isn't formatted properly, your event functions are part of the same object and thus overridden. They should be in separate objects, like so...
        eventSources: [
            {
                events: function (start, end, timezone, callback) {
                    var events = getEvents(start, end, timezone);
                    callback(events)
                },
            },
            {
                events: function (start, end, timezone, callback) {
                    self.$http.get('//myjsonfeed', {
                        params: {
                            start: start.format(),
                            end: end.format()
                        }
                    }).then(function (response) {
                        callback(response.data);
                    });
                }
            }
       ],`

Hope that helps.
Brock

Got it! Thanks for clarifying, Brock.


camera artefacts

I am currently trying out a setup similar to the turtlebot demo (http://svl.stanford.edu/igibson/docs/simulators.html).
However, I keep getting garbage rgb images when running the simulator on a remote docker instance. I am basing myself of the igibson containers provided. And have tried running it on various different hosts and different gpu's.
Does anybody know what is causing this issue?

The rendered images are by default 4-channel (r,g,b,a), and looks like you are reshaping them into 3 channels, which causes the artifacts.
If you only take the first 3 channels it should look fine.


[feature request] cf-cli allows client authentication grant type

In some cases (such as PWS), clients have greater authentication scopes than users. It'd be extremely nice if the cf-cli supported this by allowing a user to specify a client_id/client_secret.
Note: some actions - such as creating security groups - require highly elevated scopes such as cloud_controller.admin.

Hi @jadekler
FYI, I'm approaching the UAA PM to check their roadmap and ensure we're aligned.
Regards,
Dies Koper
CF CLI PM

We have scheduled this as part of #881. I'll update that issue when the feature is available in a release.


JCB 052118 Debug Doc Updates

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
This PR adds some additional information for debugging Extensions in a hosted workbench for the Debug SharePoint Framework solutions in Visual Studio Code article.  The original content lacks content for stepping through Extension code.  Most of the info was sourced from the individual Demo articles with some additional content from me.  This is in response to issue #1592 opened by me.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: john88benson sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/debug-Extension-Breakpoints.png
:white_check_mark:Succeeded
View

docs/images/debug-Extension-Loaded.png
:white_check_mark:Succeeded
View

docs/spfx/debug-in-vscode.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx John, appreciated.


example: add exmaple for parsing kubernetes logs

Signed-off-by: Dominik Rosiek<EMAIL_ADDRESS>Description:

Link to tracking Issue:
Testing:
Documentation:

in commit title exmaple -> example
and please resolve conflicts.

@pmalek-sumo please check again


Bring back the move_in_saga config

Is your feature request related to a problem? Please describe.
The config to remap the hjkl keys for moving in saga was removed. Since my keyboard layout is different, this is a must for me
Describe the solution you'd like
A config that does the same as the old "move_in_saga={prev={}, next={}}" did

I think you can do it in your local

vim.api.nvim_create_autocmd("Filetype", {
 pattern = 'lspasgafinder',
 callback = function()
    vim.keymap.set('n', your key , '<Down>', { buffer = true, nowai =true})
 end


Api client lib

:warning: Too big to review, go to my fork and navigate the api/client package :warning:
I extracted a standalone client library from the cli that can be used outside of docker. In the process of doing that I end up removing a bunch of code from the cli.
Why do you do this to us??!!
We should own the main go library to interact with docker. This makes the docker cli be the first consumer of this library.
How did you do this??!
I extracted all the calls the client needs into an interface, you can see it here:
https://github.com/calavera/docker/blob/api_client_lib/api/client/client.go#L19
Functions are very well defined but there are a couple things to consider.
I removed cli.stream and cli.hijack entirely.
After reading every single line in the client, I believe nobody knows how those two functions work and they made our code very complicated. You can compare the start command, no need for a diff:
Master: https://github.com/docker/docker/blob/master/api/client/start.go
My branch: https://github.com/calavera/docker/blob/api_client_lib/api/client/start.go
Every response in the new library that can be streamed returns an io.ReadCloser. It's up to the caller to implement streaming. This makes the intention of the caller much more clear. Compare the image create functions:
Master: https://github.com/docker/docker/blob/master/api/client/create.go#L53-L70
My branch: https://github.com/calavera/docker/blob/api_client_lib/api/client/create.go#L51-L63
did you know that the response of that function was always a json message that we stream in the client?
Breaking changes
Of course there are always consequences. I introduced what can be considered as a breaking change.  In my opinion, the new behavior is more correct :smile_cat:
Until now, docker inspect continued evaluating containers if you typed a wrong id, see this example:
$ docker run -d --name test alpine true
$ docker inspect this_container_does_not_exist test

The message about this_container_does_not_exist not existing would be followed by the test container output, a json payload of about 100 lines. This makes almost impossible to see the error.
With the new behavior, the cli stops execution as soon as it finds an error. This is a more predictable behavior. Users know sooner and more clear that there is a problem with the command they just executed.
The code in inspect has also been highly simplified. Compare both files:
Master: https://github.com/docker/docker/blob/master/api/client/inspect.go
My branch: https://github.com/calavera/docker/blob/api_client_lib/api/client/inspect.go
https://github.com/calavera/docker/blob/api_client_lib/api/client/inspect/inspector.go
This is impossible to review
I'm afraid so too. I tried very hard to keep changes very well defined. But the best way to review this change might be to browse my branch directly, and to play with the cli.
The new client library is in https://github.com/calavera/docker/blob/api_client_lib/api/client/lib

I think probably inspect should fail if any of containers doesn't exist :)

Oh my :heart_eyes: That looks pretty impressive!

Everything failed :fireworks:

Haha, seems like tests fixing won't be easy

Its kind of disturbing that the change in semantics you mentioned didn't result in a change to a testcase.  Aside from squashing this down :-)  can we get a new testcase for that?
btw - I really like the idea of a cli lib!

:heart_eyes_cat: Will have to take a look but \o/.

docker cp seems to be broken. I get http: read on closed response body when copying from container and Error response from daemon: Untar re-exec error: exit status 1: output: unexpected EOF when copying to container. May may need to add more tests if this is the case(maybe they only check API atm).

@tonistiigi yep, there are still a few tests failing that I'm currently fixing. docker cp included.

Definitely high-level +1! :heart:

nice love the description definitely helps :)

@duglin I totally agree with you, I was very suspicious about not seen inspect test failing. I've added some in my last commit. All inspects commands are implemented exactly in the same way, so all of them behave alike :raised_hands:
https://github.com/docker/docker/commit/1db5748a35eb984a2e52ddb04a13723af0e7cbf9

@calavera don't really want to start a name bikeshedding, but I don't think it's a good idea to have a package be named lib. dockerclient would be more appropriate, plus it's also what was used for samalba/dockerclient.

@tiborvass that name is totally provisional, until we move all that code to a dedicated repo.
I am not going to call it dockerclient because it doesn't follow that other library's conventions and it can be confusing and misleading.
PS: The repo is already created, but it's private for now :smile_cat:

@tiborvass

I wonder if we shouldn't bring runconfig to the API

We should. I'd rather leave the configuration structs where they are for now, this is already big as it is. But we're working on consolidating those structs that are shared by client and server.

I don't like that JSONConfig is a string

Fair enough, I'll make the change.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "api_client_lib" git@github.com:calavera/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~53
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

I looked over most of the changes and played with this a bit. I think the API we provide should be in a bit higher level and easier to use for the third party apps, but I think there is no need to start going over these changes now. As a ported state of what our CLI needs this is a good starting point. But we should add a comment to the api package about this API being highly unstable atm.
I think we should try to get this in ASAP, to avoid heavy rebases and have plenty of time to test this before release. Everything doesn't need to be perfect in the first PR.
LGTM

Thanks for the fix, LGTM

userns had a flaky test I reported. Merging

My thoughts go to all the people who now have to rebase

And to their families.

And to those having to ping "needs a rebase", lol

:cat:

@calavera Can you take a look at this change - it's broken Windows building on master....

@jhowardmsft checking

@jhowardmsft it looks like windows is still building with Go 1.4.2 while the rest is building with 1.5. Is that intentional?
The problem is that this function was added to 1.5:
https://golang.org/pkg/text/template/#Template.Option
We can go back to the workaround we had with 1.4.2. Although I'd rather not, the fix for 1.4.2 is to replace the current rawElementInspect implementation with this function:
https://github.com/docker/docker/blob/release/v1.9/api/client/inspect.go#L175-L203

well, if we did go back to 1.4.2 that would fix the issues we're seeing with building on Power with gccgo where 1.5 isn't official available yet
ping @brahmaroutu

https://github.com/docker/docker/blob/master/project/PACKAGERS.md#build-dependencies
We do still officially support compiling with Go 1.4, so we'll need to
figure out something (or decide whether that requirement can be bumped, but
it's sounding like that's not really plausible right at the moment).

okay, I'll remove that call and go back to the ugly workaround :pensive:
Thanks for checking!

@calavera You can use version build tags!

@LK4D4 indeed. Doing that.

@calavera Hang fire - I'll update the CI nodes, shouldn't take too long. I have already updated node 1 as I'm playing with Windows CI on it. @mikedougherty I assume you're OK me updating 2..9?

gccgo master builds failed because of this. I think the best would be to add gccgo PR builds to catch go1.5 related breakage earlier?
@tianon go1.5 support is on main branch, can we build gcc:6.0 image from master and use it for x86, Power and Z?

It's OK, he's adding a shim for Go 1.4 support (which is still worthwhile
to keep for now IMO).

We have to move to 1.5.2 on those machines anyway. I just accelerated the update. All nodes are updated with the exception of Jenkins-prs-3. @mikedougherty I sent you an email about that node - I was unable to logon to it, even after a restart through the portal.

Wow, this is really amazing!! :+1:

@jhowardmsft just got host 3 back up and running. sent you an email with details. thanks for the ping


chore(CI/CD): bump version 0.0.1 -> 0.1.0

Added

Plotting basic functionality with CLI

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Support to Windows Explorer extensions such as Tortoise Git

Windows Explorer context menu items are supported and should show in the context menu in Files. Is this what you mean by extensions?

We need a little more information from you in order to investigate this issue, please reopen this issue once you have filled out the issue template.

This is probably related to #8633 as the TortoiseGIT/SVN items aren't shown in the context menu on empty folder areas.


ng-submit doesn't work when called ng-change funtion

Hi ,
I want to submit form when ng-change function called on select. but form doesn't submit.
$scope.resultCountFilter = function(e){
var size_val = $( "#Resultcount" ).find( "option:selected" ).text();
$('#display_size').val(size_val);
angular.element('#searchFrm').triggerHandler('submit'); }
Please provide me solution to submit form from other function without reloading page.for ng-submit not working when i use above code.

Please, don't keep re-submitting issues that have already been closed (see #13008).
What @pkozlowski-opensource meant (https://github.com/angular/angular.js/issues/13008#issuecomment-145456981) is that this is not the appropriate place to post your question.
It is a general support question and as such should be directed to one of the appropriate support channels.
GitHub issues are reserved for bug reports and feature requests.
Thx !


Supervision does not get triggered if child actor throws in Started message

If a child actor throws when receiving the Started message, the parent will not get notified.
I assume that there is something missing in that sequence, e.g. that the parent is not yet set or something.

Is it because the exception is swallowed? https://github.com/AsynkronIT/protoactor-dotnet/blob/dev/src/Proto.Actor/LocalContext.cs#L236

@rogeralsing Is there any reason why we have that catch? Wouldn't the parent always want to get notified on failed system messages, so they can be retried?

Well spotted, agree we can get rid of the try catch there, ofc failing system messages should also be escalated to parent.

I modified the existing supervisor tests to throw an exception when the started message is received (after cahnging the code above to rethrow the exception) and 2/4 of them failed... i think it was restart and possibly escalate strategies. It was pretty difficult to debug but i think the flow was correctly sending a restart message, but then inside the processing of that it was throwing another exception, as another started message must have been processed and this then resulted in a failed task for processing the restart message... but then my kids, partner and mother came in and that was the end of that!


Add SVD file for D1-H of Allwinner

This is a copy of an incomplete SVD file for D1-H of Allwinner maintained by me in the aw-pac

Thank you @duskmoon314


WebSocket issue

Hi I try Get_Server and all work fine except for the websocket.
When I try to test websocket (for example with a Chrome extension) my server go down and raise an unhandled exception:
Unhandled exception:
WebSocketException: Invalid WebSocket upgrade request

Now if this is a protocol problem the should not terminate.

Problem solved using another WebSocket client extension to run tests.


Add more sub-generators

Project structure and modularization needs to be defined first: #31
improvements:

named functions for sub-generators
multi-module project structure/settings/generators (ask for module?)

more generators:

template
directive
filter
constant
find more: https://github.com/yeoman/generator-angular

out of scope:

create routes

gulp

adjust paths in generator gulpfile
adjust generated gulpfiles (necessary?)

readme

update!

Can I ask why theres now create route (state) sub gen?
Or if theres plans to make one?

Hi @LunarDevelopment, the main reason we decided not to create one is because it would require to parse and manipulate the 'module'.js files and this can be quite cumbersome and error-prone especially when there's a lot going on in this file. For every other sub-generator so far it suffices to create and inject new files. So reason # 1: simplicity, testability.
Another reason is that we don't want to build an app-build or something of that fashion. Working with the generator should not keep you from learning the technologies that you use (in this case the ui-router), it much rather should gently guide you towards learning and comprehending on your own. Thus enabling to keep your mind focused on the project's unique requirements while taking all the repetitive and time-consuming setup and configuration tasks off your shoulders.
Setting up a state depends on many factors: are you using different ui-views, is it an abstract one, what kind of templates are you using (inline, or template files), etc... etc... In our opinion there's not much boilerplate much rather a lot of logic that depends on your project and thus, the code needed to reproduce that behavior is already the most efficient and comprehensive way to achieve that.

All very good reasons, thanks for taking the time!

 We know our reasoning isn't bullet-proof so we take any chance to validate it, so thank you for bringing it up.


fix: IMIDIMapping wasn't used correctly in VST3 wrapper

the init order was wrong, now the wrapper specifics get determined before setting up audio and MIDI busses and parameters.
Since it is a hot fix, the version is being bumped to 0.7.1 and PR goes directly to main.

@u-he-frank please have a look if you agree with this.

Looks good.


Unmigrated database leads to minimal error message for missing table

When e.g. running Prisma Client in a Netlify function, this is what the error output looks when you did not migrate your database yet (and all tables are missing):
◈ Error during invocation:  {
  errorMessage: 'Error occurred during query execution:\n' +
    'ConnectorError(ConnectorError { user_facing_error: None, kind: TableDoesNotExist { table: "images2.RequestLog" } })',
  errorType: 'Error',

The error message here could be much more helpful, e.g. suggesting the option that the database might be empty and not migrated yet or similar.

This is easily locally reproducible on postgres:

We should handle this error. In order to reproduce this, take any prisma project and connect it to blank database and try executing the client.


Downgrade autocomplete plugin again to fix it.

This fixes https://github.com/prometheus/promdash/issues/453

@stuartnelson3

@stuartnelson3 is away for some days, so merging for him to take a look later.


port settings

I am following the steps to set up the application. I am hosting the docker on Microsoft Azure. And after I set up the Endpoint. I connect to the mattermost, but it shows the 502 bad gateway from "nginx/1.4.6 (Ubuntu)".
I also check some document about change the Nginx configuration in sites-available. is it necessary? if so, could you please guide the process?
The default web port is 80, and the default app port is also 80. Is it correct? I saw some document set the app port should be 8065?
I try to use docker-compose up without -d to see the error when I access the web container.
Error as below:
2017/02/22 09:28:58 [error] 11#0: *1 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:80/", host: "<IP_ADDRESS>:58060"

I don't think you need to change the sites-available. Yes, default port is 80 for nginx, which is a reverse proxy for mattermost app. MM app container doesn't expose any port to the host. the web and app containers are connected through docker network.
Could you post the full log of docker-compose up? Ideally, if you could remove the previous generated files with rm -rf volumes (you shouldn't have any data there since it failed to start. But if there is any, make sure you backup your data).

I change the yml file to map 8060:80 for tcp. and then I map the endpoint on Azure 58060 to 8060.
the log like below :
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
mattermostdocker_db_1 exited with code 1
web_1  | <IP_ADDRESS> - - [23/Feb/2017:07:47:48 +0000] "GET / HTTP/1.1" 502 583 "-" "Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36"
web_1  | 2017/02/23 07:47:48 [error] 11#0: *5 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:80/", host: "jenkinsdps.chinacloudapp.cn:58060"
web_1  | <IP_ADDRESS> - - [23/Feb/2017:07:47:50 +0000] "GET /favicon.ico HTTP/1.1" 502 583 "http://jenkinsdps.chinacloudapp.cn:58060/" "Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36"
web_1  | 2017/02/23 07:47:50 [error] 11#0: *5 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET /favicon.ico HTTP/1.1", upstream: "http://<IP_ADDRESS>:80/favicon.ico", host: "jenkinsdps.chinacloudapp.cn:58060", referrer: "http://jenkinsdps.chinacloudapp.cn:58060/"

It looks like there are some permission issues with database directory.  Could you run ls -lha volumes/db/var/lib/postgresql/data to check the ownership of the db directory? If the user is different from what you are currently login, you may have to change the file ownership or remove the directory completely.

[devops@jenkinsdps mattermost-docker]$ ls -lha volumes/db/var/lib/postgresql/data
total 8.0K
drwxr-xr-x. 2 root root 4.0K Feb 23 07:44 .
drwxr-xr-x. 3 root root 4.0K Feb 23 07:44 ..
[devops@jenkinsdps mattermost-docker]$
Does it relate to the command I run by sudo? could you show me how to change the file ownership?

Yes. Looks like your current user is devops and the owner of the directory is root. So looks like you have run it before with root (possibly by sudo). If you want to run as devops user (which may not have enough permission to use port 80 on the host), you can run the following command chown -R devops:devops volumes. This will change the ownership to devops for all files and directories under volumes. Then you can try to run docker-compose.
If you still get some permission issues, you may want to run docker-compose with root.

I would like to take a try. But I did run every command with Sudo, otherwise the command would say that it can't connect to docker Deamon. I will have a try by Chown, then update back.
Thanks
On Fri, Feb 24, 2017 at 8:23 AM +0800, "Pan Luo"<EMAIL_ADDRESS>wrote:
Yes. Looks like your current user is devops and the owner of the directory is root. So looks like you have run it before with root (possibly by sudo). If you want to run as devops user (which may not have enough permission to use port 80 on the host), you can run the following command chown -R devops:devops volumes. This will change the ownership to devops for all files and directories under volumes. Then you can try to run docker-compose.
If you still get some permission issues, you may want to run docker-compose with root.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I run docker-compose build, up-d all with "sudo" at the very beginning, so do you thing the privilege is still a problem?

if you run everything with sudo, it should be fine.

but all these issue/error message are the result running with sudo...

Could you try to remove the volumes directory completely and run with root user? e.g. su - and then docker-compose up?

I am confused,  I run all the command with sudo, it should be ok for the volumes privilege with root.

It should. But privilege to run postgres is dropped from root to postgres inside db container. So postgres won't be able to create the files. My suggestion is just shot in the dark. Not sure why the postgres data is created by root but if you remove it, the container may create the directory with appropriate ownership and permission. Another idea is to change the ownership manually. e.g. chown -R 999:999 volumes/db/var/lib/postgresql and run docker-compose up again.

after run the command , the result is like below. But the error still exists
[devops@jenkinsdps mattermost-docker]$ ls -lha volumes/db/var/lib/postgresql/data
total 8.0K
drwxr-xr-x. 2 systemd-bus-proxy ssh_keys 4.0K Feb 24 03:23 .
drwxr-xr-x. 3 systemd-bus-proxy ssh_keys 4.0K Feb 24 03:23 ..
[devops@jenkinsdps mattermost-docker]$

So you still see error like db_1 | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied. Weird. Could you post the full log again? In case there are some clue.

Please check the full logs
web_1  | Starting Nginx
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
web_1  | ln: failed to create symbolic link '/etc/nginx/sites-enabled/mattermost': File exists
app_1  | Configure database connection...cp: cannot create regular file '/mattermost/config/config.json': Permission denied
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
app_1  | sed: can't read /mattermost/config/config.json: No such file or directory
app_1  | sed: can't read /mattermost/config/config.json: No such file or directory
app_1  | sed: can't read /mattermost/config/config.json: No such file or directory
app_1  | sed: can't read /mattermost/config/config.json: No such file or directory
app_1  | OK
app_1  | Wait until database db:5432 is ready...
app_1  | sed: can't read /mattermost/config/config.json: No such file or directory
mattermostdocker_db_1 exited with code 1
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
..........
db_1   | AWS_ACCESS_KEY_ID is required for Wal-E but not set. Skipping Wal-E setup.
db_1   | chown: cannot read directory ‘/var/lib/postgresql/data’: Permission denied
.........

any insight?

What's your docker-compose version?

docker-compose version 1.11.1, build 7c5d5e4

any update?

It seems like there still permission issue.  Not sure if there is anything special on Azure. Never played with it.

I encountered the same problem today. And couldn't get it running, Now it's working (yay!)
Using commit a034f8f7ab9b52d65ff20cd4b5da10893b385d70 on master.
I'm running on a VPS instance, all root install.
Here's the info on the system:
Virtualization: microsoft
Operating System: CentOS Linux 7 (Core)
CPE OS Name: cpe:/o:centos:centos:7
Kernel: Linux 3.10.0-514.16.1.el7.x86_64
Architecture: x86-64

Client:
 Version:         1.12.6
 API version:     1.24
 Package version: docker-common-1.12.6-11.el7.centos.x86_64
 Go version:      go1.7.4
 Git commit:      96d83a5/1.12.6
 Built:           Tue Mar  7 09:23:34 2017
 OS/Arch:         linux/amd64

Server:
 Version:         1.12.6
 API version:     1.24
 Package version: docker-common-1.12.6-11.el7.centos.x86_64
 Go version:      go1.7.4
 Git commit:      96d83a5/1.12.6
 Built:           Tue Mar  7 09:23:34 2017
 OS/Arch:         linux/amd64

docker-compose version 1.12.0, build b31ff33
docker-py version: 2.2.1
CPython version: 2.7.5
OpenSSL version: OpenSSL 1.0.1e-fips 11 Feb 2013

What I've changed is:
     build:
       context: app
       # comment out for team edition
-      dockerfile: Dockerfile-enterprise
+      #dockerfile: Dockerfile-enterprise
     restart: unless-stopped
     volumes:
-      - ./volumes/app/mattermost/config:/mattermost/config:rw
-      - ./volumes/app/mattermost/data:/mattermost/data:rw
-      - ./volumes/app/mattermost/logs:/mattermost/logs:rw
+      - ./volumes/app/mattermost/config:/mattermost/config:Z
+      - ./volumes/app/mattermost/data:/mattermost/data:Z
+      - ./volumes/app/mattermost/logs:/mattermost/logs:Z
       - /etc/localtime:/etc/localtime:ro

     build: db
     restart: unless-stopped
     volumes:
-      - ./volumes/db/var/lib/postgresql/data:/var/lib/postgresql/data
+      - ./volumes/db/var/lib/postgresql/data:/var/lib/postgresql/data:z
       - /etc/localtime:/etc/localtime:ro

apocalypsebg's solution fix my issue. It seems a difference between CentOS and ubantu

Thanks for the update.


Read from remote file system

Since #374, typhon partially supports the fsspec protocol.  However, we cannot read files remotely, even when a backend may support this in principle.  For example:
import datetime
import s3fs
from typhon.files.fileset import FileSet

fs = FileSet(
        path=(
            "noaa-goes16/GLM-L2-LCFA/{year}/{doy}/{hour}/"
            "OR_GLM-L2-LCFA_G16_s{year}{doy}{hour}{minute}{second}*_"
            "e{end_year}{end_doy}{end_hour}{end_minute}{end_second}*_c*.nc"),
        fs=s3fs.S3FileSystem(anon=True))

finf = fs.find_closest(datetime.datetime(2021, 11, 10, 10))
fs.read(finf)

fails with
Traceback (most recent call last):
  File "/data/gholl/checkouts/protocode/glm-fileset-remote.py", line 16, in <module>
    fs.read(finf)
  File "/data/gholl/checkouts/typhon/typhon/files/handlers/common.py", line 114, in wrapper
    return method(*args, **kwargs)
  File "/data/gholl/checkouts/typhon/typhon/files/fileset.py", line 2707, in read
    data = self.handler.read(decompressed_file, **read_args)
  File "/data/gholl/checkouts/typhon/typhon/files/handlers/common.py", line 114, in wrapper
    return method(*args, **kwargs)
  File "/data/gholl/checkouts/typhon/typhon/files/handlers/common.py", line 670, in read
    self._ensure_local_filesystem(file_info)
  File "/data/gholl/checkouts/typhon/typhon/files/handlers/common.py", line 260, in _ensure_local_filesystem
    raise NotImplementedError(
NotImplementedError: File handler NetCDF4 can only read from local file system, not from S3FileSystem

However, reading directly from s3 is possible in principle:
sfs = s3fs.S3FileSystem(anon=True)
fp = sfs.open(finf.path)
ds = xarray.open_dataset(fp)

Satpy also supports reading files directly from s3fs, for example, for Advanced Baseline Imager (ABI) files.
It would be nice if it would be possible to use the power of routines such as FileSet.icollect(...)  while reading directly from a remote filesystem.

Currently the NetCDF4 FileHandler in typhon opens the file directly with netCDF4.Dataset.  Although xarray can read from a remote filesystem, it can only do so with the h5necdf backend.  For typhon to support reading NetCDF files from a remote filesystem, it too would have to use h5netcdf.File.  Changing that may have some unintended side-effects.


Populating associations

I'm requesting an addition for populating nested associations.

@gyzerok what do you mean under this?
user's friends connection isn't a good example for nesting?

@hekike I mean JOINS between collections. If you store nested collections not like embed document but as separate collections.

@hekike Sorry, I'm kinda stupid. Never mind :)


feat: URL Parameters

(FROM README)
URL Parameter Configuration

Option
default
type
description

record_key
-
String
A key from a JSON record that will be used to insert a value to the parameter

url_key
-
String
Parameter's key name

prefix
-
String
String to be added before the value

suffix
-
String
String to be added after the value

Example: Inserting an ID from a JSON record into a URL Parameter
Let's assume a scenario that the following endpoint requires a SQL condition to update a piece of information: https://someurl.com/
It accepts updatecondition=<SQL CONDITION HERE> as a parameter to set the condition for the exact row that we want to update.
We have the following JSON record that we want to get updated:
  {
    "id": 2901,
    "name": "Luiz Barros Rocha",
    "age": 26
  }

We could write a url_parameter containing the following information:
  {
    url_parameters:
    - url_key: updatecondition
      record_key: id
      prefix: "user_id = "
  }

This would be the ending result:
https://someurl.com?updatecondition=user_id+%3D+2901

Good suggestion.   However,  content transformation use can be infinite.  Instead, intent is to have content to be processed using transformation approach like here: https://infinyon.com/blog/2023/09/github-stars-to-slack.
So approach is create SmartModule to process url and send out appropriate transformation

@Day-OS if you cherry pick the commit on https://github.com/infinyon/http-sink-connector/tree/ac/params-reuse-request I refactored so the client and most of the request is not reinitialized each loop.
(I think there was also a little buglet fixed in this refactor where the loop was sending the current record to the previous records URL with query parameters)

@Day-OS if you cherry pick the commit on https://github.com/infinyon/http-sink-connector/tree/ac/params-reuse-request I refactored so the client and most of the request is not reinitialized each loop.
(I think there was also a little buglet fixed in this refactor where the loop was sending the current record to the previous records URL with query parameters)

It looks good, I've cherry-picked it


.Net [4877] SK format the weaviate

Motivation and Context

Issue should be at: https://github.com/microsoft/semantic-kernel/issues/4877
Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@dmytrostruk, could you review this to make sure this one line change is ok?


chore: update workflow to move prs to projeect

If we add the eng label this workflow won't run

@rudouglas i might be blind but which part determines that this doesn't run if we have an eng label?

Oh whoops sorry @LizBaker that was the initial pass at this, JV wanted it to only route to the Project Board if all changed files were in the quickstarts folder, so only content updates.


trying on 5 classes data

Hello, I want to use your model on 5 classes data.
I read closed issues saying change the 'low rank' to 1 when applying your model on 2 classes data.
How can I change the 'low rank' on 5 classes data?
Thank you

You could try different low_rank, e.g., from 1 to 3, since we did not evaluate our method on 5 classes dataset.


KerasTensor和tf.tensor之间的转换

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf2.4
Custom code
Yes
OS platform and distribution
Window10
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
KerasTensor没有numpy（）这种，应该如何转换
Standalone code to reproduce the issue
File "C:/Users/KM Group/Desktop/lmx/SemanticCompression-Speech/DeepSC-S-main/random_mask_training.py", line 73, in <module>
    sem_dec = sem_dec_model(frame_length, stride_length, args)
  File "C:\Users\KM Group\Desktop\lmx\SemanticCompression-Speech\DeepSC-S-main\model_tfnn.py", line 191, in sem_dec_model
    _output = sem_dec(_intput, batch_mean, batch_var)
  File "C:\Users\KM Group\Desktop\lmx\SemanticCompression-Speech\DeepSC-S-main\model_tfnn.py", line 142, in __call__
    _input = tf.convert_to_tensor(keras.backend.get_value(_input))
  File "C:\Users\KM Group\Anaconda3\envs\speech-SC\lib\site-packages\tensorflow\python\keras\backend.py", line 3615, in get_value
    return x.numpy()
AttributeError: 'KerasTensor' object has no attribute 'numpy'

Relevant log output
No response

@lmx666-gif It happens for the older version of TF . Could you try to upgrade it using the following;
pip install tensorflow --upgrade

Otherwise you can also run the following and restart the kernel
import tensorflow as tf
tf.enable_eager_execution().
Please let us know if it helps?
Thank you!

Hello, neither of two methods can turn a kerasTensor into a TF.tensor.  I found "tf.compat.v1.disable_eager_execution()" could let KerasTensor changed into TF.Tensor, however, this problem:RuntimeError: tf.data.Dataset only supports Python-style iteration in eager mode or within tf.function.

@lmx666-gif Thank you for the response!
Could you please share the standalone code to replicate the issue here.
Thank you!

Thank you for your response!
The first is the map function:
def map_function(example):
    feature_map = {"wav_raw": tf.io.FixedLenFeature([], tf.string)}
    parsed_example = tf.io.parse_single_example(example, features=feature_map)

    wav_slice = tf.io.decode_raw(parsed_example["wav_raw"], out_type=tf.float64)
    wav_slice = tf.cast(wav_slice, tf.float32) / 2 ** 15

    return wav_slice

The second one is the training process:
 for epoch in range(args.num_epochs):

    trainset = tf.data.TFRecordDataset(args.trainset_tfrecords_path)
    trainset = trainset.map(map_func=map_function,
                            num_parallel_calls=num_cpus)  # num_parallel_calls should be number of cpu cores

    #trainset = trainset.shuffle(buffer_size=args.batch_size * 200, reshuffle_each_iteration=True)
    trainset = trainset.batch(batch_size=args.batch_size)
    trainset = trainset.prefetch(buffer_size=args.batch_size)

    # train_loss for each epoch
    train_loss_epoch = []
    train_loss = 0.0

    # record the train time for each epoch
    start = time.time()
    # MASK参数

    #EMA_MODEL来选择mask的index

    binary_mask = RandomMaskingGenerator(input_size,frame_length,mask_ratio)
    # bmr:0为掩码，1为未掩码；
    # bm_T:1为掩码,0为未掩码；
    # bm,bm_T = binary_mask()

    for step, _input in enumerate(trainset):
        bm, bm_T, _ = binary_mask.random_mask(_input.shape[0],alpha_e_max)#.totally_random_mask(_input.shape[0])
        print("_input",_input)
        # print("bm_shape", bm.shape)
        # print("bm_T_shape", bm_T.shape)
        loss_value = train_step(_input,_input*bm,bm_T)
        loss_float = float(loss_value)

        train_loss_epoch.append(loss_float)

        # Calculate the accumulated train loss value
        train_loss += loss_float

    # average train loss for each epoch
    train_loss /= (step + 1)
    train_loss_all.append(train_loss)

    # print log
    log = "train epoch {}/{}, train_loss = {:.06f}, time = {:.06f}"

The third one is the train_step function:
@tf.function
def train_step(_input,_input_mask,bm_T):
    with tf.GradientTape() as tape:
        enc_output, batch_mean, batch_var = sem_enc(_input_mask)
        #输入进去semantic decoder
        print("main:",enc_output)
        _output = sem_dec([enc_output, batch_mean, batch_var])
        loss_value = mse_loss(tf.multiply(_input, bm_T), _output)
        tf.print(loss_value)
        loss_whole = loss_value

    grads = tape.gradient(loss_whole, weights_all)  # compute gradients
    optimizer.apply_gradients(zip(grads, weights_all))  # update parameters

    return loss_whole

The _ouput generated by sem_dec() is the value I wanted change it from kerasTensor to TF.tensor; I could send you the whole codes, if you need, this is my email<EMAIL_ADDRESS>Thank you for your reply!

In addition, before the new question, I found that the tensor in my model is kerasTensor; however, the tensor returned to my main function is chaned into tf.tensor; Rediculously!

@lmx666-gif Could you share the complete code and other dependencies to replicate this issue. I have faced different issues here. Thank you!


perf(errors): add retry error

retry when catch botocore.exceptions.ResponseStreamingError

Codecov Report

Merging #158 (cd3b2f4) into main (184eb81) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #158   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           33        33
  Lines         3842      3844    +2
=========================================
+ Hits          3842      3844    +2

Impacted Files
Coverage Δ

megfile/errors.py
100.00% <100.00%> (ø)

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

:+1:


Copy stake dapp

Closes: #1944
WIP...

Screenshoots

Can we align it?

One thing that might be a bit confusing is when an account doesn't have any "old" delegations, the banner would still be visible. Maybe we should hide it or during the first step of upgrade specifically say that there are not delegations to be upgraded.

Upon dApp loading

Scenario: From "accounts[0]" (grant manager) 0xAAA go through the "Upgrade Stake - Recommended" path. In "Delegations" you will see a new row with green "Stake copied". Then from 0xAAA create a grant for 0xBBB. Switch account to 0xBBB
Problem: In 0xBBB you will see the same "Stake copied" delegation as you see for 0xAAA account. Screenshot below is a grantee account 0xBBB, but we see "Stake copied", even though 0xBBB didn't have any "old" delegations

Scenario: open "Upgrade Stake" flow and chose 2nd option "Undelegate and wait..". Upon finishing the flow close the flow and open it up again.
Problem: The delegation that you chose ^ is still visible.
Note: When you chose the first recommended option and finish the flow, you won't see that delegation if you open up a flow again. I expected the same behaviour for the 2nd option.

Do we want to display enabled "Undelegate Tokens" button in Operations for the "old" operators after going through the Copy & Stake flow?
As an owner of that old delegation I cannot do anything (just can see the green label) but an operator can undelegate it.

Scenario: open "Upgrade Stake" flow and chose 2nd option "Undelegate and wait..". Upon finishing the flow close the flow and open it up again.
Problem: The delegation that you chose ^ is still visible.
Note: When you chose the first recommended option and finish the flow, you won't see that delegation if you open up a flow again. I expected the same behaviour for the 2nd option.

pls see 8886a9298ced7bcc79706e29fbb55e34714b0f38

Do we want to display enabled "Undelegate Tokens" button in Operations for the "old" operators after going through the Copy & Stake flow?
As an owner of that old delegation I cannot do anything (just can see the green label) but an operator can undelegate it. It will appear under "Undelegations" but an owner won't be able to Recover it.

Based on Flowdock convo its ok. In the future, we are going to add "actions" for the copied delegations.

Can we align it?

pls see 2321056


Add C._zero_for_deref to list of builtins

This PR builds on top of #278, which added specific, more idiomatic extraction for reads on arrays.
The previous PR added a special value, C._zero_for_deref, which serves as a marker for more idiomatic extraction.
This PR marks this value as a builtin, to avoid adding an undefined reference to C._zero_for_deref when C is not in scope.
cc @tahina-pro

Aymeric and I chatted about this, and there is a fix (add C if not already in there, leave it and don't add the extra definition otherwise).
But overall, it's a pretty big violation of invariants for F* to generate a reference to the C module when it's not already in the dependency graph. Isn't there a cleaner solution? Some options Aymeric and I chatted about:

when extracting Steel.X.read, generate a reference to Steel.X._zero_for_deref so that there's no extra module added in the graph
add a wrapper in Steel.X, such as inline_for_extraction let read = read' C._zero_for_deref so that it appears directly in F*

Let us know?

when extracting Steel.X.read, generate a reference to Steel.X._zero_for_deref so that there's no extra module added in the graph

I think this is the cleanest option from the point of view of the user, but it is less tractable from the point of view of F* ulib developers, since they will need to add Steel.X._zero_for_deref into krml src/CStarToC11.ml, and potentially for every reference library.

add a wrapper in Steel.X, such as inline_for_extraction let read = read' C._zero_for_deref so that it appears directly in F*

I don't think this is possible: krmllib module C cannot be referenced from the F* ulib sources.


Review/update AR Reminder quick guide

Review this guide to ensure the process is correct, make changes as needed

@LANHALL2, @sarahelz19  - fyi


Create source file with configuration params

Hello! Is it possible to generate source file with some parameters from configuration? For example I want to create Fasfile with current app identifier and xcodeproj name.

As in like a project template ?

@jcampbell05 Yeah, exactly

So this tool is focused on generating projects, but I am working on one for project files.


Fix edit command when application has interview

The bug was that the code tried to check for the presence of the interview on the newly created editedApplication to decide whether to add an interview to it.
This fixes #170

Codecov Report

Merging #185 (3877f19) into master (9c68392) will increase coverage by 0.38%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #185      +/-   ##
============================================
+ Coverage     73.53%   73.91%   +0.38%
- Complexity      717      720       +3
============================================
  Files           106      106
  Lines          2078     2074       -4
  Branches        245      245
============================================
+ Hits           1528     1533       +5
+ Misses          475      469       -6
+ Partials         75       72       -3

Impacted Files
Coverage Δ

.../seedu/application/logic/commands/EditCommand.java
97.61% <100.00%> (+8.98%)
:arrow_up:

...ation/model/application/UniqueApplicationList.java
79.31% <0.00%> (+1.14%)
:arrow_up:

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


Links

Created .yaml file and put the links
Add links to Footer Component

Make a verification for footerData and change the links verification

Change the max-width to have fixed values

Format the links on .yaml file to an array of objects
And format the Footer Component


server should default to index.html

Instead of 404, the server should send "index.html" and let the router of the web application handle the path.

Fixed by merging the hyper branch.


Using "client doctor" or "client gen" sometimes crashes server unhandled System.ObjectDisposedException

Hi,
I'm using the new Entitas 46.3 from the asset store and the jenny.exe server crashes after using "client doctor" or "client gen" 1-3 times.
I'm working on windows, used a new unity project with a fresh import from the asset store.

Cheers

Hi, thanks for posting! I will try to reproduce after the holidays.

this may be a bug of Unity 2017.2.
2017.3 seems fixed.

Ok cool. I will probably add a safety check to avoid this error. I really wanted to avoid any safety checks because I'm still convinced that this cannot happen :) but as it turns out, I'm wrong and it seems to behave differently on Windows, so I will surrender, give up, and add a safety check :)

i just installed unity 2017.3.0f3 and tried it, and it still happens.
it happens even without having unity open (after a few tries).
the error message suggests it has something to do with the server <-> client socket connection so unity version shouldn't affect that.


Recoverer in AfterRollbackProcessor

Hi,
I would like to be able to recover messages in a BatchListener using Transaction and AfterRollBackProcessor.
Use case :
Having a process which is publishing to 2 differents topics (A, B).
If an error occures between A and B, we have to rollback to prevent A to be published.
Thus prevents us from catching and handling the error in the listener directly.
I would like to fallback into a single record listener to identify the message which is failing, rollback again and send it to a DLT.
To be able to do that, maybe the AfterRollbackProcessor from the BatchListener could recover the messages to another topic C which will be read by a single record listener.

I believe the "new" recovering batch error handler will satisfy your needs: https://docs.spring.io/spring-kafka/docs/2.5.4.RELEASE/reference/html/#recovering-batch-eh


fix debian and add suse support (#1741)

Proposed Changes
fix debian and add suse support to resolve ticket 'Improve documentation for installing open-iscsi' (#1741)

Add code logic to install open-iscsi in SUSE (zypper) platform.
Use bash instead of sh to enable platform check using [[ ]].
Update Alpine from 3.7 to 3.12.

Types of Changes
Improvement.
Verification

Deploy RKE/K3S on Centos/Ubuntu/SLES15SP2 and Verify if containers run successfully on each node have package installed.

Linked Issues
Refs: #1741
Further Comments
None
Signed-off-by: cclhsu<EMAIL_ADDRESS>

@cclhsu the patch LGTM. Can you rebase your PR to v1.1.1 branch? master branch for longhorn/longhorn is considered stable. The v1.1.1 branch is the base for the future v1.1.1 release.

@cclhsu the patch LGTM. Can you rebase your PR to v1.1.1 branch? master branch for longhorn/longhorn is considered stable. The v1.1.1 branch is the base for the future v1.1.1 release.

Ok,  I have rebased the PR to v1.1.1.

Ok,  I have rebased the PR to v1.1.1.

Sorry I missed one thing during the first review. This yaml file is synced from https://github.com/longhorn/longhorn-manager/blob/master/deploy/iscsi/longhorn-iscsi-installation.yaml , which is, in fact, the source of truth. I've create a wiki page to explain where should the PR go. https://github.com/longhorn/longhorn/wiki/Repositories-for-contribution-guideline
Can you put a PR to longhorn-manager instead? Thanks.

Sorry I missed one thing during the first review. This yaml file is synced from https://github.com/longhorn/longhorn-manager/blob/master/deploy/iscsi/longhorn-iscsi-installation.yaml , which is, in fact, the source of truth. I've create a wiki page to explain where should the PR go. https://github.com/longhorn/longhorn/wiki/Repositories-for-contribution-guideline
Can you put a PR to longhorn-manager instead? Thanks.

I have created PR https://github.com/longhorn/longhorn-manager/pull/819 in longhorn-manager and close this one.

I have created PR https://github.com/longhorn/longhorn-manager/pull/819 in longhorn-manager and close this one.


Pull: Overwrite all local objects

Hi,
when pulling there are always a lot of objects identified that were adjusted locally. Even without touching the objects after deserialization and generation there are differences when the objects will be serialized. At least for me those differences don´t make any difference and I would just ignore them. So in the end depending of the number of objects it´s a bit annoying to confirm a lot of confirm popups.
That´s why I would like to ast if some 'overwrite all' dialog option would be some improvement for the project.
As an additional configurable button won´t work for the FM popup_to_confirm and a custom built dialog would mean a couple of new global objects I would propose another approach:
When going through the objects and display the dialog for each one, count up to five when the user always presses 'yes'. On the 6th element some other popup_to_confirm dialog is displayed asking the user if he want´s to overwrite all. If so, then this will happen; If not the per-object-dialog query will be continued as it is now - no further counting for the current pull transaction
What do you think?

yeah, it is annoying, both the popup and the fact that the check does not work as intended
also see https://github.com/larshp/abapGit/issues/704

@larshp Something like this could be implemented quite easily using a salv table with some checkboxes in the first column. I did this before, it´s not much code. If you want I can adopt this for here.

feel free to submit a pull request 😄
however there is a risk this might be refactored in the future, I would like to have a reuseable selection list using HTML like the rest of the abapGit gui, plus we need this part refactored to work in a RESTful manner ie. via API calls.
These are all ideas, and will take some time to implement, and users will benefit having an ALV in the meanwhile

@larshp see PR #934

PR merged, can the issue be closed?

yeah, sure


deprecated function call

https://github.com/ratchetphp/Ratchet/blob/5012dc954541b40c5599d286fd40653f5716a38f/src/Ratchet/Server/IoServer.php#L58
https://github.com/ratchetphp/Ratchet/blob/5012dc954541b40c5599d286fd40653f5716a38f/src/Ratchet/App.php#L72
LoopFactory::create(); is deprecated. I think it should be replaced with :
\React\EventLoop::get();

Thanks for reporting! Looks like an easy fix, PRs welcome! :+1:

@clue Done: https://github.com/ratchetphp/Ratchet/pull/957.


Strange comparison of the range

https://github.com/d-mozulyov/UniConv/blob/f9a7142610eb5afb0d7584bf636b46a1e2157018/sources/UniConv.pas#L12368

Yep, it looks like "if (S1 in [$21..$28, $30..$74]) then" should be used. But it need to be tested. The original code was taken from libiconv library, cp932_mbtowc function


HIVE-27032: Introduce liquibase for HMS schema evolution

Before reviewing please read the attached documents
What changes were proposed in this pull request?
Introduce Liquibase to handle the schema evolution under the hood, while leave Schematool API interface unchanged.
Why are the changes needed?
To replace the custom schema evolution logic with a well established robust and industry standard one.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manuall and through unit tests

Schematool with liquibase developers' guide.pdf
Schematool with Liquibase design document.pdf

retest


Update dbcc-shrinkfile-transact-sql.md

Hi. Shouldn't we mention about 0 as well.  Actually DBCC SHRINKFILE (Data_File, 0) is the same as DBCC SHRINKFILE (Data_File) but there is no remark about it in the documentation.

@RaufAsadov : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@pmasl - Hi, would you review this proposed update to your article?
Thanks.

Any news?

@pmasl @stevestein Could one of you review this update to the documentation? Thanks.

@pmasl @MikeRayMSFT @rothja - Would one of you be able to review this PR from August? Thanks!

I'll try to find some time to research/test unless @pmasl knows the answer off-hand. Thanks! Sorry for the delay. #assign:rothja

#reassign:@bluefooted

Thank you for your attention. I really appreciate your time and effort.

@bluefooted Pam, could you take a look and see if you agree with this change? Also, note the clarification on the tests that @RaufAsadov ran in the comments in the middle of this conversation thread. Thank you!

@bluefooted Pam, could you take a look and see if you agree with this change? Also, note the clarification on the tests that @RaufAsadov ran in the comments in the middle of this conversation thread. Thank you!

Thanks, all.

Thanks, all.


Cherry pick token urls

Adds token-urls feature to the work here: https://github.com/torchbox-forks/wagtail-sharing/pull/2
Please see support ticket for notes: https://torchbox.monday.com/boards/1124794299/pulses/1124795922/posts/105937758

@zerolab I just wanted to make you aware of this work to add back the token URL's work we use for GOSH. I'll get one of us to code review it as usual but any input you have will be appreciated. Thanks

Hi @nickmoreton , most of my reviews would be on the 5.1 upgrades.
https://github.com/torchbox-forks/wagtail-sharing/pull/2
The token urls parts LGTM.


Add com.statusgator/status_change/jsonschema/1-0-0

Thanks @ronnyml ! Nice work. Scheduling for the next-but-one release...


feat(cdk/scrolling): ScrollToIndexIfNeeded function

Feature Description
We have scrollToIndex function on CdkVirtualScrollViewport. But we cannot determine whenever an index is or is not withing visible viewport.
const viewPort: CdkVirtualScrollViewport;

scrollToIndex(index: number): void {
	const range = viewPort.getRenderedRange();

	if (range.start > index || range.end < index) {
		// Scroll is 100% needed.
		viewPort.scrollToIndex(index, 'auto');
	} else {
		// The index is within range, but not necessarily within visible viewport!
		// From my testing I have +-8 items above and bellow at the time
	}
}

Use Case
It would enable scrolling to an item if only needed.
Note
If there was CdkVirtualScrollViewport.getVisibleRange, we could even scroll our desired element to the middle of the viewport!

@crisbeto Hi, do you think this could be added? I tried looking into it but I could not orient myself in that code well enough to do changes comfortably.


AppleXRConcept.unity Cannot Use

I want see the AppleXRConcept.unity scene, but the VisionOSDemo.apk Only have VisionOSDemo,unity, and i canot  build with free Trial.
hope can switch scene in apk

Because the AppleXRConcept.unity scene in its current form is mostly "on rails", meaning only a subset of the UI is implemented and/or state dependent, as mention in the README, we don't currently have plans to include it in the VisionOSDemo.apk.
However, even with the free trial version of Nova, you can play around with the AppleXRConcept scene in VR over Oculus link!

Just wanted to follow up here and let you know the latest version of the Nova - Free Trial (v2.0.0) does now include build support!


Fuzz tests

Motivation: to help harden ocean.py

what does this mean? What should be done here? There's not even 70% coverage in the existing unit tests right now so what exactly is the goal of those so far undescribed tests?

We already have several github issues to fix tests for code coverage.
Fuzz tests aren't going to be that beneficial.
Therefore this issue isn't going to happen.


chore(main): release 2.0.23

Auto generated release
2.0.23 (2024-09-19)
Build System

deps-dev: bump typescript from 5.5.4 to 5.6.2 (#234) (f9196e7)
deps: bump @modular-forms/solid from 0.22.1 to 0.23.0 (#237) (bff1dfc)
deps: bump @solid-primitives/storage from 4.2.0 to 4.2.1 (#235) (5e665b6)
deps: bump valibot from 0.41.0 to 0.42.0 (#236) (5c42a96)
deps: bump vinxi from 0.4.2 to 0.4.3 (#233) (bd1e33e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.23
:sunflower:


Remove TreeSitterConfig.name from documentation

TreeSitterConfig doesn't seem to have the name attribute, anymore.
I discovered this discrepancy while integrating Markdown language parser
in PR #380.

Merged in cbfc70c0b, thanks :)


Removed buffer acces from handle cheatcode

Stack:

#1220 ⬅
#1219
#1218

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

Is this one relevant?

I broke something while rebasing :)))


Sortable attributes in http-ui

Add a reset_sortable_attribute method
Add the sortable_attributes to http-ui
Fix some broken test in http-ui

The tests I fixed were broken since multiple PRs.
I think we should either add them to the CI or delete them entirely (to me, they seem useless?)

Ah yes, sorry I totally forgot about this PR (even though I reviewed it :see_no_evil: )
And what do you thinks of the tests?

Those tests are good, thank you for fixing them! :)


Chaining multiple handlers

Needs some testing where values in the state change.
Don't have time to do it now...
Handlers are to be added this way:
mta := mta.NewDefault(c, []mta.Handler{
    mta.HandlerFunc(mail),
    // more handlers
    mta.HandlerFunc(handleMailDir),
})

The mta itself should not know about multiple handlers. It would be better to have seperate struct that implements the mta.Handler interface which calls the rest of the handlers.
    type ChainHandler struct {
        handlers []mta.Handler
    }

    func (h *ChainHandler) HandleMail(s *mta.State) {
        for _, handler := range h.handlers {
            handler(s)
        }
    }

Then pass a chain handler to the mta.
Or do it this way: https://github.com/codegangsta/negroni

Fixed in #22.


Wrong reference to genome Danio rerio

Hi Brad,
When trying to install the reference genome for Danio rerio through the upgrade option "--genomes Zv9", the python script tries to download ftp://ftp.ensembl.org/pub/release-78/fasta/danio_rerio/dna/Danio_rerio.Zv9.78.dna.toplevel.fa.gz however, the reference at the Ensembl ftp server is named Danio_rerio.Zv9.dna.toplevel.fa.gz.
just a heads up
Greetings
Matthias

Matthias -- thanks much for the heads up on the problem. I updated the Ensembl based genomes to use the latest releases and also conform to the new file naming, so they should hopefully work cleanly for you now. Let us know if you run into any other issues at all and thanks again.


Update CSD prefixes and stage codes to new ones

CSD documents are now labeled as below.
=== Project code

. The project code shall be of the form `[n]`, where `[n]` is any
selected positive integer of less than 5 digits.

. The numbering system applies across all project types.
Different project types shall not share an identical project code.

. The project code shall be assigned on proposal acceptance
of the project.

. Co-published documents shall attempt to use an identical project
code in both CalConnect and at the collaborative publisher to prevent
potential ambiguity, if possible.

=== Document identifier

==== Representation

The document identifier takes the form of `[type][stage] [code][:][y]`, where:

* `[type]` is the document identifier prefix;
* `[stage]` is the document stage;
* `[code]` is the project code of the project that created the document; and
* `[y]` is the 4-digit edition year.

==== Edition year

The edition year is the calendar year in which the document is published.
On updates, a document will retain the same project code but may be
assigned a different edition year in which the update is published.

==== Co-publishing

Co-published documents shall incorporate codes from all collaborative
publishing parties in the document identifier, joined together via a
"`solidus`" sign ("`/`").

[example]
CalConnect deliverables co-published with ISO are referenced as as "`CC/ISO`",
with IETF as "`CC/IETF`".

[[document-type]]
==== Document types

Document types shall be represented in the "`type`" as follows:

. Standard, "`CC`"

. Directive, "`CC/DIR`"

. Guide, "`CC/Guide`"

. Specification, "`CC/S`"

. Report, "`CC/R`"

. Amendment, "`CC/Amd`"

. Technical corrigendum, "`CC/Cor`"

[example]
"`CC/Guide/DS 38002:2018`" is a CalConnect Guide at DS stage with
project number 38002, published in 2018.

==== Document stages

Document stages shall be represented in the "`stage`" as follows:

. Published, "` `" (blank)

. Final draft standard, "`/FDS`"

. Draft standard, "`/DS`"

. Technical committee draft, "`/CD`"

. Project group draft / Working draft, "`/WD`"

[example]
"`CC/FDS 10668:2018`" is a CalConnect standard document at FDS stage
with project number 10668, published in 2018.

[example]
"`CC/Guide/DS 20080:2018`" is a CalConnect guide at DS stage with
project number 20080, published in 2018.

This is currently partially done in #51

Done

@opoudjis could you also update the README to reflect this? Thanks.

Done

Thanks!


Add "ES384" as option of "codeChallengeMethod"

I was trying to use the library to connect to my provider (Logto) and they use undisputedly the "ES384" algorithm, option that is not currently available in the "oauth2" project but inside the "crypto" project.

As far as I'm aware, this isn't part of any standard or a de facto standard. Oslo only intends to support standardized features


Fix foreach variable replacement

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@geirst please review
@havardpe FYI

Travis build failure is unrelated: MasterElectionTest.testZooKeeperUnavailable:283->waitForCompleteCycles:94->waitForCompleteCycle:89 Â» IllegalState


Update Protobuf, fix publishing

This changeset performs two operations:

Updates the Protobuf library version since the one previously used has some recently discovered vulnerabilities. Also, it migrates the project to the latest base library, which also has its Protobuf dependency updated.

Excludes all transitive dependencies onto io.spine:spine-validate artifact, which are somehow brought via the project dependencies. It caused this PR to fail due to the version conflicts with this artifact.

Fixes (and changes) the publishing of Validation modules. There is now two fat JAR artifacts published:

io.spine.validation:spine-validation-java-extensions — ProtoData extensions allowing to generate the Validation code in Java;
io.spine.validation:spine-validation-java-runtime-extensions which is a runtime library required for the generated Java code to work.

Please note that both of the published fat JARs have NO classifier. It has been removed for simplicity.
The library version has been set to 2.0.0-SNAPSHOT.29.

@alexander-yevsyukov PTAL.
Hopefully, once this is merged mc-java will become buildable again. I will let you know once I am done with mc-java's branch.

@armiol, can we have the 2nd artifact name shorter, like spine-validation-java-runtime?


Test bug report for duplicate

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):

What are your environment parameters?

DeFi App Version: v2.3.0
Operating System (OS): linux

Anything else we need to know?:

/duplicate


Conflicting URL information on MS blog vs AVD Safe URL List page

The blog post (published 31st Jan 2022) https://techcommunity.microsoft.com/t5/azure-virtual-desktop/important-new-changes-in-required-urls/m-p/3094897#M8529 says the URL should be *.prod.warm.ingest.monitor.core.windows.net and should be allowed as soon as possible.
However the page this issue is raised for does not have this URL as a requirement. Is the URL *.prod.warm.ingest.monitor.core.windows.net needed or not?

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

ID: 406f8b6d-6471-c983-c102-9d60a57964cd
Version Independent ID: 98fa8587-36e1-690c-12c5-b865955ada0b
Content: Azure Virtual Desktop required URL list - Azure
Content Source: articles/virtual-desktop/safe-url-list.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@bubbletroubles Thank you for your question!  We are reviewing this and will get back to you shortly.

@kobulloc-MSFT thanks for clarifying. Will this info be updated in the docs? Some organisations require official documentation from the vendor for firewall rules.
E.g. our organisation wouldn't be able to use the new url yet as it's not documented.

@bubbletroubles Thank you for sharing the requirements some organizations have on official documentation before adopting firewall rules.  I'm looking at the best way to approach this as we don't want to cause confusion and I'll get a PR out to address this.

There is already confusion because the new URLs are not on the official page that documents the URLs :)

@bubbletroubles Fair point :)  I just want to make sure that we don't confuse those who can't start using the new URL immediately, giving time for a transition.

@bubbletroubles A PR has been created and @Heidilohr has been added for review.

@bubbletroubles A PR has been created and will be merged as soon as it gets a sign off from @Heidilohr.  We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Update notification table Firebase Messaging README.md

I think in the table showing when a notification will be displayed the titles should be shifted over to the right like this.

@prestopasto you need to sign the cla

@triniwiz  I thought I did yesterday after the cla-bot mentioned it, but just in case I just filled it out again.

@cla-bot check

@prestopasto you need to sign the cla

Oops, filled it out yesterday and forgot to '@cla-bot check' to force an update.  Thanks for the reminder.


Error running mbtiles-extracts

mbtiles-extracts zoomstack.mbtiles portsmouth.json admin

/usr/local/lib/node_modules/mbtiles-extracts/index.js:162
    return name.toLowerCase().replace(/ /g, '_') + '.mbtiles';
                ^

TypeError: Cannot read property 'toLowerCase' of undefined
    at toFileName (/usr/local/lib/node_modules/mbtiles-extracts/index.js:162:17)
    at result.forEach (/usr/local/lib/node_modules/mbtiles-extracts/index.js:64:37)
    at Array.forEach (<anonymous>)
    at Stream.onData (/usr/local/lib/node_modules/mbtiles-extracts/index.js:63:24)
    at Stream.emit (events.js:182:13)
    at drain (/usr/local/lib/node_modules/mbtiles-extracts/node_modules/through/index.js:36:16)
    at Stream.stream.queue.stream.push (/usr/local/lib/node_modules/mbtiles-extracts/node_modules/through/index.js:45:5)
    at emit (/usr/local/lib/node_modules/mbtiles-extracts/node_modules/split/index.js:37:14)
    at next (/usr/local/lib/node_modules/mbtiles-extracts/node_modules/split/index.js:49:7)
    at Stream.<anonymous> (/usr/local/lib/node_modules/mbtiles-extracts/node_modules/split/index.js:54:5)

hi 👋
seeing the same result as above - I tried running a few previous versions and got the same error, guessing it might be related to a dependency?

Try to use the propertyname 'name' e.g.:
mbtiles-extracts [path_to_mbtiles] [path_to_geojson] name
I was also facing this problem and debugged the javascript to find out that in the geojson file should look like this:
{
"type": "FeatureCollection",
"features": [
{
"type": "Feature",
"properties": {
"name": "NameOfTheArea"
},
"geometry": {
"type": "Polygon",
"coordinates": []
}
}
]
}
and that the argument 'property name' to give to mbtiles-extracts is in this case NOT "NameOfTheArea", but the identifier : name.
Hope this helps.


No Visual Studio project file

The Visual studio project file was removed in v7.3.0. The readme has not been updated. Is there any plan to add the VS project file? What is the recommended way to build on Windows?

You can use Qt Create to build on Windows (the .pro files are qmake files).

Thanks for the help Julien. Having an issue with the Qt build. It is reporting -lcast cannot be found. -lcast is used in LIBPATH in the .pro file

Make sure to download the library binaries from https://github.com/clariusdev/cast/releases and place them where specified in the project file:
work/src/cast % tree .
.
├── LICENSE
├── README.md
├── blob
└── src
    ├── example
    │   ├── caster
    │   └── caster_qt
    ├── include
    │   └── cast
    ├── lib
    │   ├── libcast.dylib
    │   ├── libcast_plugin.a
    │   └── pycast.so
    └── python

Also make sure the library version corresponds to your app version (v9.0.0 at the time of writing).

The build completes but the application crashes on run. The files you indicated are not the same as the windows 9.0.0 release. Error libgcc_s_seg-1.dll was not found. Is the structure below correct?
work/src/cast % tree .
.
├── LICENSE
├── README.md
├── blob
└── src
    ├── example
    │   ├── caster
    │   └── caster_qt
    ├── include
    │   └── cast
    ├── lib
    │   ├── cast.dll
    │   ├── cast.lib
    │   └── pycast.pyd
    └── python
    ```

I was able to get caster_qt running without issue, but caster crashes

your firewall needs to be open, you can give special access to your executable or read the port back from the api (it is dynamic however), and enable that.


Add the ability for exclusions

Whether through a config value array, sitemap_exclude, and/or metadata on the page, :nositemap:, to be excluded.

Hey @jdillard, awesome work on the extension, any chance that the sitemap_exclude feature is still somewhere on the roadmap?

I can try and make an attempt at it "soon", but no promises. Were you thinking something like this would work for you?
sitemap_excludes = [
    "/path/to/page1.rst",
    "/path/to/page2.rst",
]

Thanks for the quick reply! In theory yes, but for my use-case it's more about the search.html and genindex.html which are basically empty and recognized as soft 404s by search engines, so I would target system pages instead of rendered .rsts

Hi @msbt for sphinx basic theme and furo, the meta robot have no index value.

basic theme
furo

@kryskool thanks, but I'd rather not have it in the sitemap in the first place, this piles up the "discovered but not indexed" pages in various analytics tools.

@msbt How does this PR look? https://github.com/jdillard/sphinx-sitemap/pull/91

@jdillard oh wow, thanks so much for the addition and quick implementation, appreciate it a lot! PR looks great, exactly how I imagined it would work!