NodeDef mentions attr 'dilations' not in Op -- Keras is not supported on Tensorflow serving?

Ubuntu 16.04
Python 3.6
tensorflow (1.8.0)
tensorflow-serving-api (1.9.0)
I used tf.keras.layers to build pnet/rnet/onet in MTCNN, instead of writing my own layers like what the authors did. It works fine locally. I am also able to export the models and start the server on Tensorflow serving. However, when I try to use the service, it pops up error msg:
grpc.framework.interfaces.face.face.AbortionError: AbortionError(code=StatusCode.INVALID_ARGUMENT, details="NodeDef mentions attr 'dilations' not in Op<name=Conv2D; signature=input:T, filter:T -> output:T; attr=T:type,allowed=[DT_HALF, DT_FLOAT]; attr=strides:list(int); attr=use_cudnn_on_gpu:bool,default=true; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pnet/p_net/conv1/Conv2D = Conv2D[T=DT_FLOAT, _output_shapes=[[?,?,?,10]], data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 1, 1, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_pnet/input_0_0, pnet/p_net/conv1/Conv2D/ReadVariableOp). (Check whether your GraphDef-interpreting binary is up to date with your GraphDef-generating binary.).
[[Node: pnet/p_net/conv1/Conv2D = Conv2D[T=DT_FLOAT, _output_shapes=[[?,?,?,10]], data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 1, 1, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_pnet/input_0_0, pnet/p_net/conv1/Conv2D/ReadVa...TRUNCATED")
does it mean the Keras layers are not well supported on the latest tensorflow serving?

fixed by updating tensorflow_model_server to latest version


TrueType: changing glyphNameMap and GlyphUnicode lookup order

Fixes #1096

@pdfjsbot test

TEST-UNEXPECTED-FAIL | eq usmanm-bad | in chrome | rendering of page 3 != reference rendering
TEST-UNEXPECTED-FAIL | eq bpl13210 | in chrome | rendering of page 4 != reference rendering

see #1043 (for ref images)

Fixes notes on Page 9 for both Mac and Windows 7 here.
Interestingly, on my Mac the images on the first few pages are messed up, whereas on Windows they're fine. Looks like it's an image mask/transparency/blend issue.
+1 for fixing the font though.

@pdfjsbot makeref


[Feature Request] Balance privacy

User story
If solved, I cannot see a user's Celo balance (tokens) because I know their phone number.

Closing this ticket in favour of the discussion topic here: Feature Request: Balance privacy for accounts with a phone number mapping #9293


Automatically generate single chrom tiled subsets for multi chrom schemes

Provides a valuable backwards compatibility layer. Provisionally agreed scheme subdir name: assets

this will be added to primaschema build


Clarify GTDiff options argument documentation

The diff initializers refer to their options arguments taking "any of the above options", but both GTDiffOptions and GTDiffFindOptions are defined above them and the diff initializers really only accept GTDiffOptions.
I got this mixed up in practice, resulting in this bugfix.

Thanks a lot for the contribution! This makes perfect sense.


Vim-surround mapping is hijacked

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
v0.8.0-dev
Minimal reproducible full config
local packer_repo = 'https://github.com/wbthomason/packer.nvim'
local packer_installation_path = vim.fn.stdpath('data') .. '/site/pack/packer/start/packer.nvim'

vim.fn.system({ 'git', 'clone', '--depth=1', packer_repo, packer_installation_path })
vim.cmd [[packloadall]]

require('packer').startup(function(use)
        use {
                'tpope/vim-surround',
                'wbthomason/packer.nvim',
                {
                        'neovim/nvim-lspconfig',
                        requires = { 'williamboman/nvim-lsp-installer' }
                },
                {
                        'hrsh7th/nvim-cmp',
                        requires = {
                                { 'L3MON4D3/LuaSnip' },
                                { 'hrsh7th/cmp-buffer' },
                                { 'hrsh7th/cmp-cmdline' },
                                { 'hrsh7th/cmp-nvim-lsp' },
                                { 'hrsh7th/cmp-path' },
                                { 'hrsh7th/nvim-cmp' },
                                { 'saadparwaiz1/cmp_luasnip' },
                        },
                },
                { 'nvim-treesitter/nvim-treesitter' },
        }

        require('packer').sync()
        require('packer').compile()
end)

vim.api.nvim_create_autocmd('User', {
        pattern = 'PackerCompileDone',
        callback = function()
                local completion = require('cmp')
                local snippet = require('luasnip')

                local function has_words_before()
                        local line, col = unpack(vim.api.nvim_win_get_cursor(0))
                        return col ~= 0 and vim.api.nvim_buf_get_lines(0, line - 1, line, true)[1]:sub(col, col):match("%s") == nil
                end

                local function on_tab(fallback)
                        if completion.visible() then
                                completion.select_next_item()
                        elseif snippet.expand_or_jumpable() then
                                snippet.expand_or_jump()
                        elseif has_words_before() then
                                completion.complete()
                        else
                                fallback()
                        end
                end

                completion.setup({
                        snippet = {
                                expand = function(args) snippet.lsp_expand(args.body) end
                        },

                        mapping = completion.mapping.preset.insert({
                                ['<C-b>'] = completion.mapping.scroll_docs(-4),
                                ['<C-f>'] = completion.mapping.scroll_docs(4),
                                ['<C-Space>'] = completion.mapping.complete(),
                                ['<C-e>'] = completion.mapping.abort(),
                                ['<CR>'] = completion.mapping.confirm({ select = true }),

                                ["<Tab>"] = completion.mapping(on_tab, { "i", "s" }),
                                ["<S-Tab>"] = completion.mapping(function(fallback)
                                        if completion.visible() then
                                                completion.select_prev_item()
                                        elseif snippet.jumpable(-1) then
                                                snippet.jump(-1)
                                        else
                                                fallback()
                                        end
                                end, { "i", "s" }),
                        }),

                        sources = completion.config.sources(
                        {
                                { name = 'nvim_lsp' },
                                { name = 'luasnip' },
                        },
                        { { name = 'buffer' } }
                        )
                })

                local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())

                require'lspconfig'.cssls.setup {
                        capabilities = capabilities,
                }
        end
})

Description
After #956 , when installing nvim-cmp along with tpope/vim-surround, on opening Neovim, the normal-mode mapping cs is hijacked for exactly two invocations. After that, the mapping behaves as expected. Additionally #963 is also resolved after the initial faulty invocation of cs, so this might be related to #963.
Steps to reproduce

Open Neovim, on first load, plugins are installed.
Re-open Neovim so that plugins are properly loaded.
Invoke mapping cs in normal mode twice, and observe the output in the buffer
Invoke mapping cs after this and it's behaving correctly.

Expected behavior
Normal-mode mapping cs behaves as configured.
Actual behavior
Normal-mode mapping cs only behaves as configured after the first two faulty invocations. The first two times, this text is output to the open buffer:
ll v:lua.cmp.utils.feedkeys.call.run(some_random_number)

Additional context
I don't know if it's related but I found this in the lua/cmp/utils/feedkeys.lua:
local keymap = require('cmp.utils.keymap')
local misc = require('cmp.utils.misc')

local feedkeys = {}

feedkeys.call = setmetatable({
  callbacks = {},
}, {
    -- ...
    if callback then
      local id = misc.id('cmp.utils.feedkeys.call')
      self.callbacks[id] = callback

      table.insert(queue, { keymap.t('<Cmd>call v:lua.cmp.utils.feedkeys.call.run(%s)<CR>'):format(id), 'n', true })
    end
   -- ...
return feedkeys

The string passed to keymap.t() is the original string that was output to the buffer minus the "<Cmd>ca" substring.
Edit: typo

Same issue here, when I open a popup for leaderf, my input is filled with this werid stuff call v:lua.cmp.utils.feedkeys.call.run(5).

Hm... OK. I'll revert the recent commit. Sorry for annoying.


Commented out version of SBT is detected

From a support ticket:
"Was getting a deployment error when deploying a Play Framework 2.0.4 app as follows:
-----> Heroku receiving push
-----> Play 2.0 - Scala app detected
! Error, you have defined an unsupported sbt.version in project/build.properties
! You must use a release verison of sbt, sbt.version=0.11.0 or greater
! Heroku push rejected, failed to compile Play 2.0 - scala app
my project/build.properties file was as follows:
sbt.version=0.12.0
sbt.version=0.11.3
when I removed the 'commented' line, it got past the error. Something is probably parsing commented lines to determine sbt.version, becoming confused, and failing the deploy because of a commented line."

This has been fixed.


Add new Inbound Endpoint support in Azure Batch

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Reviewers: Please note that this is for a data plane REST API version. Look at the second commit (not the first which is just copying the old API version).
@smith1511

Hi There,
I am the AutoRest Linter Azure bot. I am here to help. My task to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:
File: batch/2017-06-01.5.1/swagger/BatchService.json
Number of linter errors/warnings before the PR: 0
Number of linter errors/warnings after the PR: 0
Thanks for your co-operation.

Hi There,
I am the AutoRest Linter Azure bot. I am here to help. My task to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:
File: batch/2017-06-01.5.1/swagger/BatchService.json
Number of linter errors/warnings before the PR: 0
Number of linter errors/warnings after the PR: 0
Thanks for your co-operation.

Hey Veronica: I got confused by the AutoRest linter -- it said there were 0 warnings/errors so I didn't look at the build jobs. I will look now.
For examples I will definitely update the API version, and will look into seeing what it'd take to add this in one of the examples too.

Okay I looked through the linter errors. They are almost all not applicable. It'd be great to reduce the noise here as it's super hard to scan through all of these errors. Is there a way to put a linter configuration file next to our Swagger specification that allows us to turn off some of these errors (either the specific ones or the whole category)?
I have some feedback for the errors... I can put this into a series of issues or you can point me at the ones that already exist if there are any:
These errors are only applicable to ARM:
"code": "APIVersionPattern",
"code": "OperationsAPIImplementation",
"code": "LROStatusCodesReturnTypeSchema", -- and I'm not sure why it's flagging this on our spec anyway...
"code": "PatchInOperationName", - we offer a PATCH and and update, one has replace semantics and one has
"code": "PutInOperationName", - this is the replace semantics as opposed to the patch semantics. Also the code is wrong? (My operation name has no put in it, it has UPDATE in it
These errors/warnings are wrong/suggesting we do something silly:
"code": "ListInOperationName", - The API this is flagged on is a bulk add API which returns a list of results (where each result corresponds to one entry in the bulk add). Just because it returns a collection of things doesn't necessarily make it a list call, so I guess I disagree with the premise of this error.
"code": "XmsClientNameParameter", -- it's erroring because of casing differences which are allowed and actually have an impact... I think I filed a bug for this already here but it doesn't seem to have actually been fixed?
"code": "OperationIdNounConflictingModelNames", - This is trying to protect us, which I appreciate, but the problem is that it is trying to protect us from is more complex than "just append an s and be done" because in some languages (C# for example) the classes that are generated are called "FooOperations" so you end up with "FoosOperations" - which is wrong too. Really (IMO) all the languages should be appending "Operations" to the names of these classes, since the operation groups are just that, operation groups, and that gives the highest probability of success to Swagger authors who are naturally going to craft a resource Foo and an operation Foo_Create.
I am confused by:
{
  "type": "Warning",
  "code": "EnumInsteadOfBoolean",
  "message": "Booleans are not descriptive and make them hard to use. Instead use string enums with allowed set of values defined. Enum Property: parameters",
  "id": "R3018",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///home/travis/build/Azure/azure-rest-api-specs/batch/2017-06-01.5.1/swagger/BatchService.json:4006:10 ($.paths[\"/jobs/{jobId}\"].delete.parameters[3])"
  ]
}

This is hard to read -- I have no idea what parameters[3] is. You should use the name of the actual parameter.
I am also confused by:
  "code": "DefinitionsPropertiesNamesCamelCase",
  "message": "Property named: 'odata.nextLink', for definition: 'AccountListNodeAgentSkusResult' must follow camelCase style. Example: 'odata.nextLink'.",

The recommendation is the same as the value, so I have no idea why there is an error here.

@matthchr Having bot reporting 0 errors/warning is on me :) It's a bug that I am fixing. Thanks for letting us know. Appreciate it.

I created a new issue here discussing linter configuration files which I think would be pretty nice.

I have updated the examples REST API version.
@smith1511 - Can you take a look at adding a JSON snippet to PoolAdd_VirtualMachineConfiguration in the examples directory which demonstrates the use of the new stuff? If it's too esoteric and doesn't make sense to show in the create example then that's okay we can just omit it (I'll let you decide what is best since you know the feature best)

The build job seems to have failed with a failure unrelated to this change, can somebody take a look?

@matthchr As @veronicagg is OOF today, I am looking over this PR. The changes you have done in the swagger looks good. Here are the answers for your questions:

As this is a data plane swagger, we are in the process of categorizing this into ARM v/s non-ARM linter rules. The PR https://github.com/Azure/autorest/pull/2334 is performing that changes. It's be great if you can help us out by just listing the names that you think should not affect data plane and then our team can leverage your knowledge in correct categorization.
"code": "ListInOperationName" regarding this do you think, we agree with you and on permises of potential false positives we kept this as warning category. Feel free to suggest if you think better logic can be implanted here to improve this rule by opening issue :)
"code": "XmsClientNameParameter" I see that you have already commented on the issues. I am just reopening the issue to make sure we get this looked at.
"code": "OperationIdNounConflictingModelNames" Could you please comment your thoughts the issue https://github.com/Azure/autorest/issues/1950 so that we can think about this one.
parameters[3] I see your point is valid from readability perspective but we are using that sourceMaps to leverage intelliSense in the VS Code Extension.
DefinitionsPropertiesNamesCamelCase This is strange. I'll comment on the issue https://github.com/Azure/autorest/issues/2064 to make sure if we missed something.

Regarding bot delivering misleading information, I am fixing this.
Lastly, regarding build failures @lmazuel has done recent changes into swagger-to-sdk that seems to be affecting this. @Imazuel could you please take a look.

Hi @vishrutshah - I listed the ones I think are ARM specific above (see "These errors are only applicable to ARM")

"code": "ListInOperationName" regarding this do you think, we agree with you and on permises of potential false positives we kept this as warning category. Feel free to suggest if you think better logic can be implanted here to improve this rule by opening issue :)

I filed a bug for this here - I'm okay-ish with warnings being more of an "FYI" but the way the output is ordered they get intermixed with the errors and it's hard to tell them apart.
The others I responded to the linked issues.

Closing and reopening to pick up new CI of swagger-to-sdk

Hi There,
I am the AutoRest Linter Azure bot. I am here to help. My task to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:
File: batch/2017-06-01.5.1/swagger/BatchService.json
Number of linter errors/warnings before the PR: 0
Number of linter errors/warnings after the PR: 172
Thanks for your co-operation.

Hi There,
I am the AutoRest Linter Azure bot. I am here to help. My task to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:
File: batch/2017-06-01.5.1/swagger/BatchService.json
Number of linter errors/warnings before the PR: 0
Number of linter errors/warnings after the PR: 172
Thanks for your co-operation.

@smith1511 would include the example changes of reflecting new models into PoolAdd_VirtualMachineConfiguration file.
cc:
@matthchr

Dismissing @veronicagg's review as she is OOF :)

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-node


I might be doing this wrong...beginners' luck, I suppose

From what your page says, I downloaded python and pip, and tried to edit the console.txt, emulator.txt, and the config.txt files, but I must be doing it wrong, because I get the following:
Traceback (most recent call last):
File "", line 7, in
File "c:\program files (x86)\python27\lib\site-packages\PyInstaller-3.1-py2.7.egg\PyInstaller\loader\pyimod03_importers.py", line 389, in load_module
File "ice\runners\command_line_runner.py", line 12, in
File "c:\program files (x86)\python27\lib\site-packages\PyInstaller-3.1-py2.7.egg\PyInstaller\loader\pyimod03_importers.py", line 389, in load_module
File "ice\runners\ice_engine.py", line 14, in
File "c:\program files (x86)\python27\lib\site-packages\PyInstaller-3.1-py2.7.egg\PyInstaller\loader\pyimod03_importers.py", line 389, in load_module
File "ice\backups.py", line 10, in
File "c:\program files (x86)\python27\lib\site-packages\PyInstaller-3.1-py2.7.egg\PyInstaller\loader\pyimod03_importers.py", line 389, in load_module
File "ice\logs.py", line 66, in
File "ice\logs.py", line 53, in create_logger
File "logging\handlers.py", line 117, in init
File "logging\handlers.py", line 64, in init
File "logging_init_.py", line 903, in init
File "logging_init_.py", line 926, in _open
IOError: [Errno 2] No such file or directory: u'C:\Users\FlraD\AppData\Local\Scott Rice\Ice\ice.log'
Any help would be greatly appreciated.

This is the exact same error I'm getting


buf_read method is broken

The buf_read method definition is incomplete/broken.

That is true. Could you fix it?

I thought PR #4 fixed this one.
But I get (this https://github.com/drom/node-iio/blob/master/examples/BBB_query_ADC.js):
node query_adc.js
...
/root/nodejs/query_adc.js:32
iio.buf_read(buffer, function(chnl_id, sample) {
    ^

TypeError: Wrong arguments
    at Object.<anonymous> (/root/nodejs/query_adc.js:32:5)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1157:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at node:internal/main/run_main_module:17:47

Am I doing something wrong?


Stop hard-coding version # in page title

https://github.com/aspnet/Docs/issues/625

/cc @danroth27

I'll setup a private build for this and see how it looks on Read the Docs.

Looks good! :shipit:


⚠ Bofrost.lu has degraded performance

In dd0e540, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1043 ms

Resolved: Bofrost.lu performance has improved in e515277 after 18 minutes.


Is there a bucket size setting for runsettings and Run Functional Tests?

I'm on TFS 2017 update 1 (update 2 is getting installed on 8/26/17), so maybe this is already fixed.
Scenario:
I have about 30 VMs running Selenium UI tests. I am moving from the testsettings and Run #C tests to Run Functional Tests. I have 4 sets of the Deploy Test Agent and the Run Functional Tests with a runsettings file specified to run in different configurations and browser sizes. Each step has 50 to 95 tests to run.
I'm using runsettings and then overriding parameters in the build steps, which is slick
Desires:

I want shorter test runs.
I want the tests to be distributed to all the VMs
I want less time for each Deploy Test Agent steps (right now it is 2 to 3 minutes each time)
Bonus: I want to understand and avoid this error from the deploy test agent log. I get this error on most (maybe all agents), but the test agent does start running.

2017-08-10T19:33:39.1353698Z ##[warning]Task 'CopyFilesToTestAgent' for machine VM-UX-AGENT-43:5985's Log : System.AggregateException: Failed to execute the powershell script. Consult the logs below for details of the error.
2017-08-10T19:33:39.1353698Z Failed to connect to the path \\VM-UX-AGENT-43 with the user .\vpbuild for copying.System error 53 has occurred.
2017-08-10T19:33:39.1353698Z  The network path was not found.
2017-08-10T19:33:39.1353698Z
2017-08-10T19:33:39.1353698Z  ---> System.Management.Automation.RuntimeException: Failed to connect to the path \\VM-UX-AGENT-43 with the user .\buildUser for copying.System error 53 has occurred.
2017-08-10T19:33:39.1353698Z  The network path was not found.
2017-08-10T19:33:39.1353698Z
2017-08-10T19:33:39.1353698Z  ---> System.Management.Automation.RuntimeException: Failed to connect to the path \\VM-UX-AGENT-43 with the user .\buildUser for copying.System error 53 has occurred.
2017-08-10T19:33:39.1353698Z  The network path was not found.

Question: Is there a bucket size setting, like there was in testsettings, that might be defaulting to 10 or something to cause a few agents to queue up all the tests and cause others not to have any work to do?

@aligneddev : Currently it is not available. There are two options. VSTest task in update2 will support distribution in release pipeline. Link to blogpost. The agentcount of the phase setting will determine the number of batches being created. You can also specify the 4 machines in a single DTA/RFT task to get distribution
The batchsize feature is being added to the VSTest task and would be part of the next TFS release.


3807-Add-job-to-check-dependency-vulnerabilities

Context
Add job that posts output of bundle audit to register_tech channel.
Guidance to review
Insight into how to test this new functionality would be very handy

 Run bundle-audit as part of pipeline.


In starknet-wrapper, call prepare methods from final parent methods, not from implemented

e.g. when implementing a wrapper, deploy must be implemented. It has to call prepareDeployOptions . This should be done implicitly in a parent method, not force the developer of every child method to explicitly call it (as we currently do).

Probably obsolete because of https://github.com/0xSpaceShard/starknet-hardhat-plugin/issues/396
I cannot even find prepareDeployOptions anymore


Gdb latest mi interface is version 3. Why termdebug uses version 2?

I have noticed that the latest mi interface is mi3 whereas termdebug uses mi2.
Is there any reason for that?
I tried to update termdebug to mi3 and all the tests in the github workflow have passed.
Shall we update to mi3?

What's the difference between those two versions? Perhaps it wasn't available when the termplugin was developed?

What's the difference between those two versions?

Good question.  I tried to make a search but I find few highlights. I also asked on SO, maybe we will get some answers.

Perhaps it wasn't available when the termplugin was developed?

It could be. According to the official guide mi3 has been introduced in gdb 9.1.

According to the official guide mi3 has been introduced in gdb 9.1.

gdb 9.1 is February 2020. I wonder about some tiny/old single board computer over a serial line that might not have an up to date gdb. Of course, an older version of vim could be used.

I hesitate to mention that if it would turn out there is some good stuff in mi3, like escaping blanks, both version could be supported. But supporting both is the stuff of nightmares.

I hesitate to mention that if it would turn out there is some good stuff in mi3, like escaping blanks,

If you are referring to https://github.com/vim/vim/issues/12357#issuecomment-2222592786 then I tested it and mi3 won't solve the problem. I could dare to guess that other than adding few commands, it may be perhaps easier to parse the messages sent over the pty. And some possible bugfix. But it is just a guess.

both version could be supported. But supporting both is the stuff of nightmares.

I though exactly the same :D

FYI: https://stackoverflow.com/questions/78744538/gdb-what-are-the-differences-between-mi2-and-mi3-interface/78745041#78745041


Dependency on old version of "@pnp/sp"

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
not applicable
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
@pnp/spfx-controls-react version 3.8.1
Issue description
I've noticed that the latest version of the "@pnp/spfx-controls-react" library (version 3.8.1 at time of writing) has a dependency on version 2.5.0 of the "@pnp/sp" library. Are there any plans to upgrade the dependency a V3 version of the "@pnp/sp" library?
Many thanks,
Hugo

As @bcameron1231 mentioned this issue list is specific to the first-party issues, not community libraries.
@hesperanca - feel free to open the issue in PnP React Controls repo.

Thanks for coming back to me. I will raise it there.


SDK 1.12 breaks Dartium running Dart extensions

My version of Dartium might be behind, but when I clear out all my symlink packages directories I can no longer run my extension in Dart.  Switching back to 1.11 / running without clear symlinks works.

Thanks for the report.
How did you clear out all the symlinks? Do you mean, using the --no-package-symlinks ?
Also, what do you mean "my extension" ?

Yep, using --no-package-symlinks.  I have a Chrome extension written in Dart that I load "unpacked" locally from the my_extension/web/ directory.


Stop using regex module

It's overkill, and this is the only case it's used.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 98.883%

Totals

Change from base Build<PHONE_NUMBER>3:
0.006%

Covered Lines:
1416

Relevant Lines:
1432

💛  - Coveralls


ClueWeb09 WARC files faile to parse

The ClueWeb09 dataset WARC files (see sample files) use a single line feed \n as separator between WARC headers. The WarcParser expects \r\n (which would conform to the standard) and fails:
Exception in thread "main" org.netpreserve.jwarc.ParsingException: invalid WARC record at position 9: WARC/0.18<-- HERE -->\nWARC-Type: warcinfo\nWARC-Date: 2009-03-...

See also #25 for a similar issue regarding HttpParser.

I've added a lenient parsing mode which accepts \n line endings and the WARC/0.18 version string.
The ClueWeb09 sample files are quite broken though. They also have incorrect Content-Length headers. My guess is the files originally had \r\n line endings but then had them stripped to \n. I don't see a straightforward way to recover from that during parsing.

It would be nice to have some examples distributed adjacent to the WARC standard of "these are some known broken WARC files out in the wild, you might want to think about either parsing them or at least giving good error messages when you see them".


Any Example ?

is there any example of using this library in an existing project?

Not at the moment. But that's the goal of this library, which is to write something we ourselves can reuse. And that I can shove it into nRF Connect as it's "Scanner engine" so that we ourselves can be sure it's working. But we're only human and have limited time. But yes, you have the right idea.

Currently, the library is under active development.
Examples can be found in nRF-Toolbox (dev branch)

Any example of using it to advertise characteristics using peripheral mode.

Peripheral mode is not supported yet.
Unfortunately, it won't be implemented in the near future.


Binary encoding should be stateless

We have seen multiple instances where the flow between SNORT and the SecurityHub consumer would have trouble synchronizing (typically a process restart).
One key lever we can pull to make the process more robust is to ensure that we keep stateful entries small, giving the parser the option to re synchronize.
What I’m suggesting to do:

A little buffering is OK within the logger, but every write to the pipe should start a new BILL message (even if the message contains more than one record)
The dictionary encoding is elided.

Yes, that’s the case for the scanner (I think the prefix is unique enough that we can use it).
For the logger, that mean that we need to put enough BILLs to allow for the synchronisation.
PS: still chuckling at saying BILL loud all the time.

From our discussion so far, keeping notes on next steps:

[ ] New configuration API (invert serializer / writer dependency)
[ ] Use blocking open without signals in the pipe writer
[ ] Migrate to bill V2


Debugging OSS build only works on one repository, others run but without debug connection

Development Environment:
OS: Windows 7 Professional SP1 (VM under OSX)
Python: ActiveState 2.7.5
Visual Studio 15 Community \ C++ Tools \ SDK 8.1
VS Code: 1.10.2 (Code OSS Build environment)
VS Code Insiders 1.11.0 3-13 build: (Test environment)
Chrome Debugger Extension 2.6.0
Node: 6.9.1
NPM:  3.10.8
Debug Issue:
I have multiple local repositories for VSC code source trees.
I have done this to keep things separate while experimenting and learning.
the problem I am having is that only one local repository of VSC
can be debuged. I have three repositories all that can Build
and successfully execute. all have changes and some debug output
the launch files are all the same an original from the VSC clone.
if I launch the debugger with breakpoints in my main repository
the code launches and debugging will successfully work including breakpoints.
if I launch any other code from another repository, the OSS Dev
version launches but the debugger shows no output in the debug output
window and breakpoints are never hit. No debug errors our ever displayed
the debugger buttons appear as normal.
I'm thinking that there is something non- volatile saved by the one
working repository that interferes with the others. I have tried
full cleans changing names in product.jason without success.
I do think the one working repository was the first I did a win32-package
operation. One of the last tests was to do a clean of the Win32 package.
That still did not work.
I don't know how to get anymore log info as the debug output
is totally blank. And the builder works just fine.
Update:

did exact folder copy of working repository - this works and dbg works correctly

did fresh clone of master vsc in fresh repository/folder - build and execute okay
cannot debug - no errors F6 to pause process takes a long time but finally stops in
net.js - no source debugging/or error log output

found I had another repository where debugging works , was cloned from 2-24-17

repository clone from 3-1 works, each repository 3-10 and my later forks
all will not work with dbg but build and execute

Thanks
Christopher

just for clarification , problem occurs with both release and insiders
VS Code: 1.10.2 (Code OSS Build environment)
VS Code Insiders 1.11.0 3-13 build: (Test environment)
Chrome Debugger Extension 2.6.0
issue started with master branch sometime between 3-1 and 3-10

@cleidigh to be clear, things work fine as soon as you stop debugging the one workspace and start from the other? this only happens when you debug multiple at the same time?

@cleidigh what exactly are you debugging? There are multiple launch configuration in VS Code's launch.json. Which one are you using?

@bpasero
@weinand
@joaomoreno
sorry if my initial post was a bit of a red herring
also my updates may not have been clear I was working
until six a.m. on this...
I think I have a better understanding and narrowing down
of the issue/introduction point.
First I do not think this has anything to do with multiple
repositories (though I do up a question of overlapped registries)
Current Summary:
After having issues debugging my later repositories
I did a binary search/build/debug test on commits
and tags from 3-2 up to and including the insiders
commitment from last night.
Each test was the following:
Clean repository, check out new commit, build, test, debug
git clean -xfd
(check out next test commit eg)
git checkout -b 3-14-Insiders 6a002910356c6e087dd93e819e4ad54c68b9857a
.\scripts\npm install
.\scripts\code install
After successful compilation:

launch Code OSS Dev normally with .\scripts\code
verify normal execution
close development Code
set breakpoint in toggleSidebarVisibility.ts
(easy visible location/operation)

(line 33)
public run(): TPromise<any> {
	const hideSidebar = this.partService.isVisible(Parts.SIDEBAR_PART);

  return this.partService.setSideBarHidden(hideSidebar);

}

Using the current unmodified branch/commitment
start the debugger using Launch VS Code (unmodified from launch.json)

In all scenarios the development VSC gets launched
and is usable. if I add a single console output line
before the breakpoint I can see this in the debug console
of the test code.

in successful commits the breakpoint is hit when In
toggleSidebarVisibility also the debug console output
in the development/debugging VSC shows output

the failure mode still launches the correct executable,
the debug progress bar is normal, and no errors are displayed
but the debug console output shows absolutely nothing
and breakpoints are never hit

The list of branches/commits I tested following
1.10.1        653f8733d Bump version
1.10.2        8076a19fd debug: save all untitled non dirty files before debug start
3-14-Insiders 6a0029103 bump node-debug2
3-2           5790f031d Move LayoutProvider to /common/
3-5           3b16023c4 Clean up webview click handler
3-6           0106ab2e3 Merged in translations
3-7           0529b5c6b Merge remote-tracking branch 'origin/master' into tyriar/21934

3-7-3338bf    3338bf2fc :zap: spawn shared process after first workbench window
3-7-3ae62a    3ae62a8d8 debug: introduce map for sources, remove reference from source uri in workbench
3-7-a39a0f    a39a0f2e1 :zap: shell should wait for shared process to be up and running
3-7-c2cb72    c2cb72d86 debug: fix type when executing add configurations command
master        6a0029103 [remotes/origin/master: behind 26] bump node-debug2
origin/master 49129d126 Fix Webview scroll position preserve broken (#21531)

As per the testing above the problem appears at commit
3-7-3338bf fails
3-7-a39a0f success
The same problem persists in the current insiders build from last night
I have not looked at the files yet, but I'm sure it's beyond my pay grade
Hopefully this is detailed enough (or perhaps too detailed)
but I wanted to do my best especially as I'm new to git.
Christopher

okay, I think I found the problem
the spawn of a shared process addressing issue #22091 somehow causes the debugger
to not communicate or show errors.
if I commented out this spawn request , #263 main.ts then the debugger works
	// Install Menu
	instantiationService2.createInstance(VSCodeMenu);

	// Start shared process here
	sharedProcess.spawn();
});

}
I assume the non- shared process prior to this edit allowed the debugger process
normal communication (I don't pretend to know how...)
but this single line will create/remove the problem...
Christopher

@roblourens isn't this the same issue or related to https://github.com/Microsoft/vscode-chrome-debug/issues/382 ?

Yeah, it's the same as that. Sorry you had to go through that long analysis @cleidigh.
Workaround:
Run the "Launch VS Code" debug config
Then, run the "Attach to VS Code" debug config which will attach to the renderer process.
Working on a better solution in the chrome-debug issue.

@cleidigh thanks a lot for spending so much time on this.
It would have helped immensely to clearly state what exactly you are debugging and what debugger is involved (this is the 'type' attribute in the launch config).
The 'type' determines how we investigate and what developers are dragged into the investigation.

@weinand
Not a problem, as always chasing bugs makes you learn the most - favorite pastime of mine ;-)
I'm hoping to be able to contribute to both accessibility(need of mine)
and communication code (my main software and firmware focus)...
I realize the first post did not specify the launch config - sorry about that. I did think
I was clear about debugging a VSC OSS build , hopefully my follow-up was clear.
Anyway:

read the @isidorn  issue thread #382 for chrome debug - never searched that project

does appear related/same - discussion on shared process clearly of issue
as I can solve my problem as described before without the shared process spawn @joaomoreno
I don't mean to imply that is a solution without potential other problems
but it does enable debugging and breakpoints.

Doing @roblourens  suggested workaround still does not work:
Launch VSC then Attach VSC
(have to stop the debugger after launch to be able to attach)
Tried .\scripts\code then Attach
still does not work, again silent failure

Tried @bpasero  ToggleShareProcess comment , still could not hit breakpoints
but maybe I don't know the sequence for this
I did Launch VSC, toggle share ToggleShareProcess in OSS
stopped debugger,  then Attach VSC
still no breakpoints hit

am I missing something? how can I help?

Christopher

@cleidigh please fix the link to the 382 chrome issue above, because it ends up in the wrong repo which is confusing.

(have to stop the debugger after launch to be able to attach)

Don't stop the debugger. Otherwise you'll attach to the shared process again. You can run two debug configs at once.

@weinand
sorry, could not figure how to reference other project issue - copy of URL seems to work ;-)

@roblourens
Thanks - tried that - had figure out to use the icon F5 does not seem to work when a session is already
started...
Attach VSC after Launch throws the following error:
[C:/Users/Christopher Leidigh/.vscode-insiders/extensions/eamodio.gitlens-2.12.0]: Sorry, but currently only the 'navigation' group of the 'editor/title' menu supports alt-commands
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2601  for  bookmarks.toggle
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2602  for  bookmarks.jumpToNext
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2600  for  bookmarks.jumpToPrevious
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2597  for  typescriptHero.showCmdGui
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2599  for  typescriptHero.resolve.addImport
Ctrl+Alt+ keybindings should not be used by default under Windows. Offender:  2605  for  typescriptHero.resolve.organizeImports
[Extension Host] [C:/Users/Christopher Leidigh/.vscode-insiders/extensions/eamodio.gitlens-2.12.0]: Sorry, but currently only the 'navigation' group of the 'editor/title' menu supports alt-commands
[Extension Host] vscode-icons is active!
[debugger-for-chrome] Error processing "attach": TypeError: Parameter "url" must be a string, not undefined
at Url.parse (url.js:87:11)
at Object.urlParse [as parse] (url.js:81:5)
at LoggingSocket.initAsClient (C:\Users\Christopher Leidigh.vscode-insiders\extensions\msjsdiag.debugger-for-chrome-2.6.0\node_modules\ws\lib\WebSocket.js:597:23)
at LoggingSocket.WebSocket (C:\Users\Christopher Leidigh.vscode-insiders\extensions\msjsdiag.debugger-for-chrome-2.6.0\node_modules\ws\lib\WebSocket.js:79:18)
at LoggingSocket (C:\Users\Christopher Leidigh.vscode-insiders\extensions\msjsdiag.debugger-for-chrome-2.6.0\node_modules\vscode-chrome-debug-core\out\src\chrome\chromeConnection.js:16:9)
at utils.retryAsync.catch.then.wsUrl (C:\Users\Christopher Leidigh.vscode-insiders\extensions\msjsdiag.debugger-for-chrome-2.6.0\node_modules\vscode-chrome-debug-core\out\src\chrome\chromeConnection.js:69:28)
at process._tickCallback (internal/process/next_tick.js:103:7)

Yeah, there is an issue in the chrome debugger when you try to launch without setting the url field. I pushed a fix for it yesterday, so it should work if you sync again.

Perfect! works like a charm thanks !!
As a voice programmer only the double launch changes add up.
can I do a prelaunch task for the first part?
or maybe make a task with the two dbg commands in task.json ?

@cleidigh try multi target debugging: https://code.visualstudio.com/updates/v1_8#_multitarget-debugging

compounds...awesome!
added
"compounds": [
{
"name": "L/A VSC",
"configurations": ["Launch VS Code", "Attach to VS Code"]
}
almost works, I think there's a timing issue. Debug Launch VSC works
but the attach never occurs like when I do the two operations separately.
However , if I do a manual attach after the compound L/A VSC the attached completes

@weinand
@bpasero
@roblourens
very cool, got it to work:

compound debug launch task as above
added prelaunch task to Attach to VS Code (tasks.json task for a 2.5s delay batch file)
seems to sequence okay this way
tried timeout, did not work but was not getting timeout errors anyway

How about adding a prelaunchDelay option?

Cool, glad that works.
Ideally, I will come up with a good fix for this so we don't need a hack like prelaunchDelay :)

would that really be a hack?
for the compound launch actions one could imagine a certain task needing a lot of startup time
and thus a delay capability anywhere in the chain might be useful, no?

@cleidigh we already have some feature requests for the delay, though if I am mistaken feel free to open a new one.

@isidorn
@roblourens
Once I finish my current keyboard window expansion feature, I could look into this if all agree...
In the meantime question - any reason the following compound launch should not work?
no errors, but attach does not complete
"compounds": [
{
"name": "L/A VSC",
"configurations": ["Launch VS Code", "Attach to VS Code"]
}
The slight hack I did with the prelaunch task implementing a delay did work but...
I found out you cannot have a prelaunch task when using the watcher task
for incremental building and debugging.
Actually it's clear you cannot have two tasks at the same time.  That means
no debug set up with the prelaunch task will work with any set up
using something like the watch task (or any other task) I tried an example
of just the echo hello task while using the incremental build task and debugging
fails in that case. standard large tasks, standard code watch task.
Limitation/intended or worthy of an issue post?

@cleidigh yes this is a current limitation of the task framework and basically a duplicate of https://github.com/Microsoft/vscode/issues/981 or some other similar task feature request that would like to be able to run more than 1 task at a time


setup releasing to PyPI

Fixes #63
TODO:

[x] remove __version__ from apidefs/backend

jenkins/superCI comment

Status: success, return code = 0
Result url: https://da.gd/tpas
Comment: See details for logs

@bkabrda could you please review? You are more experienced in this topic.

updated

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/cBWht
Comment: See details for logs

FWIW this PR LGTM ;) 👍

rebased and hoping for good test results

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/mN1X
Comment: See details for logs

jenkins/superCI comment

Status: success, return code = 0
Result url: https://da.gd/msoi
Comment: See details for logs


Transforming parameters causes incorrect type to be inferred

It's me again!
I've got a small issue which I'm hoping isn't a pain to resolve.
Essentially when performing a transform on a schema for a parameter, it seemingly infers the incorrect type. I've added an example below:
const refreshClient = new Zodios(url, [
  {
    method: "post",
    path: "/oauth_token.do",
    alias: "getRefreshTokenWithCode",
    parameters: [
      {
        name: "body",
        type: "Body",
        schema: z
          .object({
            code: z.string(),
            client_id: z.string(),
            client_secret: z.string(),
            redirect_uri: z.string().url(),
          })
          .transform(
            (value) =>
          ),
      },
    ],
    response: z.object({
      access_token: z.string(),
      refresh_token: z.string(),
    }),
  },
] as const);

const refreshResponse = await refreshClient.getRefreshTokenWithCode(
      {
      code,
      client_id: process.env.SERVICENOW__CLIENT_ID,
      client_secret: process.env.SERVICENOW__CLIENT_SECRET,
      redirect_uri: process.env.SERVICENOW__REDIRECT_URI,
    }
  );

What I'd like to be able to do is to effectively simplify the inputs via a transform?

Hello alex,
I recognize oauth with application/x-www-form-urlencoded data. Take a look at the docs, zodios since the plugin rework now supports application/x-www-form-urlencoded natively, you don't need to use zod transform.
So you can just write it like that :
const refreshClient = new Zodios(url, [
  {
    method: "post",
    path: "/oauth_token.do",
    alias: "getRefreshTokenWithCode",
   requestFormat: 'form-url',
    parameters: [
      {
        name: "body",
        type: "Body",
        schema: z
          .object({
            code: z.string(),
            client_id: z.string(),
            client_secret: z.string(),
            redirect_uri: z.string().url(),
          }),
      },
    ],
    response: z.object({
      access_token: z.string(),
      refresh_token: z.string(),
    }),
  },
] as const);

you can take a look at the docs here
Tell me if this handles your use case.

Ah I did see that option but wasn't 100% certain that it did what I expected it to (I probably should've tested it). This fits my use case here perfectly but I do have another use case where I'd like to do a somewhat similar transformation. For example I want to have two inputs which I'll transform into one via a template literal. Happy to raise a separate request however 😊

But i'll try to add support for transformations.
Since zodios advertize zod support, it should handle this use case.
It may not be available a quickly as the other times.
So i hope that my previous answer fullfill you current need.

@alex-way fix added, new version v7.1.4


React dev server not reloading automatically for any change in code

Hi,
In dev mode npm run dev
Any code changes should reload the server.  Simply it's not detecting the change at all .

node version: v8.10.0
npm version: 6.7.0
OS: ubuntu 18.04


Clustering for Polygons (and polylines, etc) Not Supported

Looks like clustering is only supported for billboards, points and labels in 1.27.

new Cesium.DataSource() DataSources/DataSource.js 20
... snip
clustering : EntityCluster DataSources/DataSource.js 96
Gets or sets the clustering options for this data source. This object can be shared between multiple data > sources.
http://cesiumjs.org/Cesium/Build/Documentation/DataSource.html#clustering

new Cesium.EntityCluster(options) DataSources/EntityCluster.js 72
Defines how screen space objects (billboards, points, labels) are clustered.
http://cesiumjs.org/Cesium/Build/Documentation/EntityCluster.html

Attempting to cluster polygons doesn't seem to be doing anything:
http://cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases&gist=5a8bc388758fb201c63b2403b58b81bc

Requested again in https://github.com/CesiumGS/cesium/issues/9532.

Is clustering for other entity types in the pipeline? Would like to cluster by cylinder, but as this is currently not support, my for-now workaround is to render a point entity for each cylinder (same latitude/longitude, but not visible to the user) , and then cluster by the points. It does not feel like a good workaround to double the amount of entities just to have clustering, however. Are there any better ways of clustering cylinders at the moment?


[Enum Request] T780 Certificate of Supply - Document Type

Can you please change the document type into an extensible enum
    field(1; "Document Type"; Option)
    {
        Caption = 'Document Type';
        OptionCaption = 'Sales Shipment,Service Shipment,Return Shipment';
        OptionMembers = "Sales Shipment","Service Shipment","Return Shipment";
    }

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46015.


video doesn't load with https url in wkwebview

html5 video loads fine using http web server in wkwebview on iOS10.
But same doesn't load when I make server secure.
img loads up fine in html page inside wkwebview using http or https webserver.
below snapshot shows 2 wkwebviews. upper one loads html page with https url and lower one loads html page with http url

Modified code in iPhoneHTTPServer to host 2 webservers on different ports in the app.
One secure and other unsecure.
Any suggestion to fix the issue ?

Hi Anand, did you find any fix? I'm facing same issue with audio, video files after I made local host server secure/https.

No. Ended up taking exception for local network.
Its known bug in apple's code. Opened up bug on Apple but it was resolved as dupe to another bug.
Sent from my iPhone
On Jan 26, 2017, at 3:01 PM, iosappdeveloper <notifications<EMAIL_ADDRESS>wrote:
Hi Anand, did you find any fix? I'm facing same issue with audio, video files after I made local host server secure/https.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/robbiehanson/CocoaHTTPServer/issues/168#issuecomment-275541384, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGwkGWMiPbPskG98fA3GyOK57OEUWgLMks5rWSWTgaJpZM4KPEBM.

Thanks for your quick response. If you get a chance, please put a link to the apple bug.

@anandjp I am sure you checked, but double checking with you the URL of you video is https too in the 1st scenario, as loading insecure content in secure HTML will break the ability to start the video

@iosappdeveloper, my bug was duped to Apple bug 28218873.
@charlymr, yes! url of the video was https too.


Add daala forward transform assembly/intrinsics.

This began in #1920. There have been many iterative enhancements since then.

I think this is done for the most part. Open more specific issues for remaining fixes.


memo not working?

using https://cdn.steemjs.com/lib/latest/steem.min.js
i read some stuff online to use steem.memo.decode however memo isn't included in that JS.
am i doin it wrong?

I confirm there is no memo in steem.min.js, however it exists in library, if you import it as npm package.


Update: Move dependecies from blueprints to direct npm dependencies

Moving dependencies from blueprints to direct npm dependencies
Fixes #179

Closing. Splitting into independent PRs


Telescope harpoon marks is empty

After latest update the :Telescope harpoon marks is just empty.
:lua require("harpoon.ui").toggle_quick_menu()
still shows the marks that have been set so I think the telescope integration is broken.
I'm currently using git commit sha 21d0d1bfa3000e4384740bfaefa0ebc51c773786

and for some reason it works now but without updating


Request: allow SpirvToolsEliminateDeadInputComponents() to strip non-trailing members from blocks

Sending the the following fragment shader through SpirvToolsEliminateDeadInputComponents() and SpirvToolsAnalyzeDeadOutputStores() results in t getting eliminated, but not pos. Can't pos also be eliminated? The analyze pass is correctly detecting that only c is being used, and only returning it as part of the livelocs.
And then similarly, SpirvToolsEliminateDeadOutputStores() would need to also to remove pos.
Vertex
out Vertex
{
	vec3 pos;
	vec4 c;
	vec2 t;
} oVert;
void main()
{
	oVert.pos = vec3(0.5);
	oVert.c = vec4(1.0);
	oVert.t = vec2(0.4);
	gl_Position = vec4(0.0);
}

Fragment
in Vertex
{
	vec3 pos;
	vec4 c;
	vec2 t;
} iVert;

out vec4 fragColor;
void main()
{
	vec4 color = vec4(iVert.c);
	fragColor = color;
}

Compiled using 'auto map locations'.
Thanks

As you are likely aware, moving non-trailing members of input blocks in vertex shaders is impossible and would cause incompatibility at the vertex input interface. Likewise for output blocks in frag shaders and the frag output interface.
Technically we could remove non-trailing members from other shader interfaces but we would have to add logic to remap any OpAccessChain instructions that index into the block because the positions of the remaining members would change. This is very do-able. It would not cause any changes to the larger workflow. It would just be some coding effort.

Yep, this is what I would expect. I'm only looking for this optimization in shader->shader interfaces, since those resources are quite limited.


Motion Detection using OpenCV

Aim

The aim of the script is to detect the motion in a video.

Details

It takes a video as input and draws rectangle around the moving object.

GO AHEAD @karthikbhandary2


Fix error from mixing string/numerical hyperparameters

Fix error that happens when mixing a numerical and string hyperparameter types.
Error before:

Fix after:

maybe add a test that reflects exactly what you show in the screenshots?

@micahjsmith Good suggestion. I added in the test case.


Put labels and inputs on root; remove explicit standalone styles for form items

In for example
https://github.com/StarcounterSamples/Products/blob/master/src/Products/wwwroot/Products/viewmodels/EditProductPage.html#L20

    ID

    <input type="text" value="{{model.ID$::change}}" class="form-control" />

    Name

    <input type="text" class="form-control" value="{{model.Name$::change}}" />

    Barcode

    <input type="text" class="form-control" value="{{model.Barcode$::input}}" />

I see no point to wrap everything within divs, I suspect it's just legacy code from the times when juicy-tile-* was not wrapping tiles in shadow DOM.
For sure it makes applying predefined layouts harder (Starcounter/RebelsLounge#14), and even impossible without <juicy-composer>.

@warpech, @miyconst Do you find any reason to keep those divs?

This is to apply default bootstrap styles.

So that is the thing @aloscha is working at?

Yes, he is applying this pattern to KitchenSink project as far as I know.

This pattern in Products app was used to apply a 2-column layout to EditProductPage.html even when Launcher/juicy-tile-list is not running. As we just discussed on Hangouts with @tomalec, this divitis should be removed from Products. We will show this pattern at work in future, when layout composer understands more HTML patterns.
In KitchenSink (TextPage.html, etc), @aloscha has also applied a flat structure as well:

Listening to changes

Bind to ::change listener to send the new value to the server when the field is unfocused. Read more about the target change events in the Polymer docs.

        {{model.NameReaction}}

        <input type="text" value="{{model.Name$::change}}" placeholder="Name" class="form-control">

I have moved items to the root, but standalone mode looks exactly as before. @warpech, do we mind keeping it?

Sure, if standalone mode still works, then event better for us! Closing


Add a clarfying note that Tensorflow was written in C++

Hi @MarkDaoust Can you please review this PR ? Thank you!


Make reporter test less fragile

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@27fbdd1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage          ?   57.12%
  Complexity        ?      429
=========================================
  Files             ?       62
  Lines             ?     2041
  Branches          ?      133
=========================================
  Hits              ?     1166
  Misses            ?      811
  Partials          ?       64

Continue to review full report at Codecov.

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


Add OpenID Connect/SAML Support

Investigate:

https://github.com/pac4j/jax-rs-pac4j
https://github.com/googleapis/google-oauth-java-client

+1 for SAML or OpenID support. What are next steps? Did you ever get around to investigating either of those auth plugins @stevespringett? How did they look?

I have not had a chance to investigate, but I think this should be a capability of the next major release.

I would love to get support for OIDC. Is this still on the roadmap? And can you estimate when the next major release will be?

Yes, still on roadmap. Lots of folks want it. Ticket is labeled 'help wanted', so if someone submits a quality PR with test cases, it will make it into a release sooner than relying on myself to do it.

OIDC is supported. Changing title to SAML


Fix pull request crash when there is a comment

Fix pull request crash when there is a comment.

Thanks.


Add Selenium Tests

Please add a requirements.txt with your python dependencies

Could you remove the following from the repo:

[ ]  tests/geckodriver.log
[ ] anything in pycache


Added libffmpegsumo.so detection for Google Chrome Beta/Unstable

As requested in #100, we should support Google Chrome Beta and Unstable. This PR adds more if checks for their audio libraries. In this PR:

Added if check for Google Chrome Beta
Added if check for Google Chrome Unstable

/cc @wlaurance

:+1:

This has been released in 2.11.0


Use Eigen Ref for quaternions

According to http://eigen.tuxfamily.org/dox/classEigen_1_1Map_3_01Quaternion_3_01__Scalar_01_4_00_01__Options_01_4.html, it is possible to map directly quaternions from vectors or segments.
It must be worth to instantiate such mapping in joint Free Flyer and joint Spherical.

Solved by #151.


[Bug]: Live reload not working with Architect server

Which Remix packages are impacted?

[ ] remix (Remix core)
[ ] create-remix
[X] @remix-run/architect
[ ] @remix-run/cloudflare-workers
[ ] @remix-run/dev
[ ] @remix-run/express
[ ] @remix-run/netlify
[ ] @remix-run/node
[ ] @remix-run/react
[ ] @remix-run/serve
[ ] @remix-run/server-runtime
[ ] @remix-run/vercel

What version of Remix are you using?
1.1.3
What version of Node are you using? Minimum supported version is 14.
16
Steps to Reproduce
Run npx create-remix@latest and create a new project.
Run arc sandbox and nom run dev in two separate processes.
Browse to site, in my case http://localhost:3333
Expected Behavior
Live reload functionality should work, the browser should update when changing code in a route.
Actual Behavior
Live reload does not update the browser, and there is an error shown in the dev tools console:
Warning: Expected server HTML to contain a matching <script> in <body>.
script
LiveReload@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:3284:7
body
html
App
RemixRoute@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:3021:5
Routes@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:3009:27
Router2@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:484:18
RemixCatchBoundary@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:1564:18
RemixErrorBoundary@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:1489:10
RemixEntry@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:2906:24
RemixBrowser@http://localhost:3333/_static/build/_shared/chunk-LAE5MDMM.js:3313:33

As a temporary workaround, you can copy the LiveReload code from https://github.com/remix-run/remix/blob/f361f11c87abf37368528fdafc563bb8c9dbb201/packages/remix-react/components.tsx#L1276 into your project and use that instead. At least that did the trick for me.
I don't know too much about remix internals (yet). Probably someone could point me into the right direction? I would love helping to fix the issue!

Can you try setting NODE_ENV to development in your start script for architect?
{
   ...
   "scripts": {
     ...
     "start": "NODE_ENV=development arc sandbox"
   }
}

It seems like the architect sandbox sets NODE_ENV to testing for the server, hence LiveReload does not render the script on the server.
Might be cool to add that to the docs once you can confirm it works for you too.

Hey @confix , thanks for the tip! Whilst adding NODE_ENV=development to the script call itself didn't work, what DID work was adding that line to my preferences.arc file, which now looks like this:
# The @env pragma is synced (and overwritten) by running arc env
@env
testing
  NODE_ENV development
  REMIX_ENV development

staging
  REMIX_ENV production

production
  REMIX_ENV production

I'll see if I can raise a PR for this.

Thanks for the submission. This has already been merged on the dev branch: https://github.com/remix-run/remix/pull/1749/files#diff-6c273df8f051fa5f013769c5e8c2545d0df0145f9abccf1b02763ec97fc3ff00

Probably running on Windows (I am a Linux user)? Crossenv should work then. Your suggestion did not work for me, tho... after all the issue seems to be the env variable not being set correctly.

Probably running on Windows (I am a Linux user)? Crossenv should work then. Your suggestion did not work for me, tho... after all the issue seems to be the env variable not being set correctly.

Mac, actually! At least the bug is fixed in the next release!

Despite being merged, preferences.arc was later added to .gitignore and as far as I can tell this issue is still present. Is there some better way to resolve this?

@tnightingale would be awesome to have a PR to mention this in the README if you're up for it

@denodell does the last release fix the bug for you?


Reference validation failed

I am sure this is something I am doing wrong, but any assistance is appreciated.
I have the following in my initializer:
  config.attributes = {
    displayName: {
      getter: -> (principal) { principal.full_name },
    },
    givenName: {
      getter: :first_name,
    },
    sn: {
      getter: :last_name,
    },
    title: {
      getter: :title,
    },
    mail: {
      getter: :email,
    },
    telephoneNumber: {
      getter: :phone,
    },
  }

... and I am getting "Reference validation failed" in simplesamlphp. if I comment out the displayName section -or- set it to something simple like ":first_name" (although that is useless that way), it works fine.
I also put it through the validator at www.samltool.com and got the same error, so I don't think its simplesamlphp. Something I noticed when I captured the SAML Response was that the attributes got "corrupted"...
    <AttributeStatement>
      <Attribute Name="displayName" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="displayName">
        <AttributeValue>Sandy Saunders Saunders</AttributeValue>
      </Attribute>
      <Attribute Name="givenName" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="givenName">
        <AttributeValue>Sandy Saunders Saunders</AttributeValue>
      </Attribute>
      <Attribute Name="sn" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="sn">
        <AttributeValue>Saunders</AttributeValue>
      </Attribute>
      <Attribute Name="title" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="title">
        <AttributeValue>Library User</AttributeValue>
      </Attribute>
      <Attribute Name="mail" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="mail">
<EMAIL_ADDRESS>      </Attribute>
      <Attribute Name="telephoneNumber" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri" FriendlyName="telephoneNumber">
        <AttributeValue>3035551212</AttributeValue>
      </Attribute>
    </AttributeStatement>

See how the displayName and givenName both have like FirstName LastName LastName? It does not look like that in the database. I am betting that would be the reason for the "reference validation" issue. I am sure it has something to do with the "getter" for displayName. Note that I also tried setting the getter to :full_name which is just a simple method that combines the :first_name and :last_name values in a string.
~tommy

Eh, disregard.... the more I looked at the User.full_name method, I realized it was the source of the problem.  Sorry for the trouble.


[DNM] CI: Include stdlib build for Wasm in the Linux buildbot

Check the current CI build time bottleneck

@swift-ci smoke test Linux

@swift-ci smoke test Linux


Add test for URI with multiple slashes

Q
A

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets

Documentation

License
MIT

I've seen implementation violating the spec RFC 3986, e.g. https://github.com/zendframework/zend-diactoros/commit/68fc7424a66735926589dbaf74c0659c09e75764
So let's add a proper test for this.

ping @Nyholm


Unresolved dependency tree in go

While checking the dependencies of github.com/ProtonMail/go-crypto, I get an unresolved list.
E.g the dependencies of golang.org/x/crypto (direct dependency) are not shown.

This is actually the correct behavior, although we could definitely do a better job of explaining why.
It's because of Module Graph Pruning, where only the modules required to build or test the packages in the main module (ProtonMail/go-crypto) are considered part of the final module graph. So even though there are other modules dependencies of golang.org/x/crypto, the go-crypto module doesn't use them and therefore doesn't depend on them.
If you clone the go-crypto repository and use the go list command to show the transitive dependencies of all the packages in the module, you can see that they don't transitively import packages from any modules other than golang.org/x/crypto and github.com/cloudflare/circl.
~/src/go-crypto $ go list -f '{{range .Deps}}{{printf "%s\n" .}}{{end}}' ./...  | sort -u | grep '\.'
github.com/ProtonMail/go-crypto/bitcurves
github.com/ProtonMail/go-crypto/brainpool
github.com/ProtonMail/go-crypto/eax
github.com/ProtonMail/go-crypto/internal/byteutil
github.com/ProtonMail/go-crypto/ocb
github.com/ProtonMail/go-crypto/openpgp
github.com/ProtonMail/go-crypto/openpgp/aes/keywrap
github.com/ProtonMail/go-crypto/openpgp/armor
github.com/ProtonMail/go-crypto/openpgp/ecdh
github.com/ProtonMail/go-crypto/openpgp/ecdsa
github.com/ProtonMail/go-crypto/openpgp/ed25519
github.com/ProtonMail/go-crypto/openpgp/ed448
github.com/ProtonMail/go-crypto/openpgp/eddsa
github.com/ProtonMail/go-crypto/openpgp/elgamal
github.com/ProtonMail/go-crypto/openpgp/errors
github.com/ProtonMail/go-crypto/openpgp/internal/algorithm
github.com/ProtonMail/go-crypto/openpgp/internal/ecc
github.com/ProtonMail/go-crypto/openpgp/internal/encoding
github.com/ProtonMail/go-crypto/openpgp/packet
github.com/ProtonMail/go-crypto/openpgp/s2k
github.com/ProtonMail/go-crypto/openpgp/x25519
github.com/ProtonMail/go-crypto/openpgp/x448
github.com/cloudflare/circl/dh/x25519
github.com/cloudflare/circl/dh/x448
github.com/cloudflare/circl/ecc/goldilocks
github.com/cloudflare/circl/internal/conv
github.com/cloudflare/circl/internal/sha3
github.com/cloudflare/circl/math
github.com/cloudflare/circl/math/fp25519
github.com/cloudflare/circl/math/fp448
github.com/cloudflare/circl/math/mlsbset
github.com/cloudflare/circl/sign
github.com/cloudflare/circl/sign/ed25519
github.com/cloudflare/circl/sign/ed448
golang.org/x/crypto/argon2
golang.org/x/crypto/blake2b
golang.org/x/crypto/cast5
golang.org/x/crypto/hkdf
golang.org/x/crypto/sha3
vendor/golang.org/x/crypto/cryptobyte
vendor/golang.org/x/crypto/cryptobyte/asn1
vendor/golang.org/x/net/dns/dnsmessage
vendor/golang.org/x/net/route

I'll need to investigate why golang.org/x/sys appears in our dependency set for go-crypto - I suspect it's related to test dependencies, or maybe it's a bug somewhere (either in deps.dev code or the go tool itself).
Let me know if there is anything about this that doesn't make sense to you. Thanks for the report.


Split off static builds

It seems that in
https://github.com/conda-forge/fftw-feedstock/pull/50
A discussion started on whether or not we should include the static files.
I believe conda-forge has since pushed away from static libraries.
Would it be acceptable to remove the static libraries from this package and deal with the issues when they arise downstream?
Can we split the package instead into two?
fftw-static   # not recommended but for downstream projects to migrate
fftw  # only dynamic modules

Thoughts?

Sounds good to me.

Maybe @beckermr has some thoughts, seeing as he was involved in that earlier discussion as well.


.NET project file not updated

While packages.config file was updated the csproj file was not updated in this PR: https://github.com/bbtsoftware/TfsUrlParser/pull/25/files

Re: the noise... for those who aren't aware there are ways to disable security updates or version updates.

really miss this feature 😢


Update to new Table 1.2

Can you please correct the spelling error in Biospecimen in the title and also add the following to the footnote: "Note: Any collections indicated under ’Home’ setting reflect collections in which home mouthwash was the only sample collected. Home setting is assumed to be 0 (0%) when not listed."  (it does not need to be italicized in the report, just did that to make the new text stand out here.)
Also, please confirm, this is for any collection (research could be blood, urine and/or mouthwash) and for clincial collections (blood and/or urine), correct?

Title and footnote updates. The logic for any collection is blood or urine or mouthwash collected, regardless of collection setting. Then, it splits the counts between research, clinical, or home. Though participants can't be in multiple settings. The code reviews participants in clinical setting participants, then research, then home.

Updated in the 12/30 report


feat(service overview): add deactivate

Description
Added deactivate functionality to service overview for active services
Why
To enhance active services
Issue
Ref: CPLP-2821
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally

redundant fetchImage code

@lavanya-bmw is this done - if so - please assign (via the top right menu) the pr to martin (via the assignees tag)


Googletrans: solves a long-standing issue

I'm pretty sure this fixes the long-standing problems such as #5, #6.

Now py-googletrans is renamed to googletrans and bumped to version 2.0.0.
Includes a token generator to access translate.google.com ajax api.
Tests that cover all codes.
Better performance using requests.Session
HTTP/2 support

See https://pypi.python.org/pypi/googletrans/2.0.0

Changes Unknown when pulling eb5cb4ab1140da6d53960284bff5e4426ffe2eea on clean-workflow into ** on master**.


Update Kleo Network About Page as mentioned in the figma.

https://www.figma.com/design/JN3XzNNAqf1HeH1B0x4sl6/KLEO-Design-system?node-id=29-7828&node-type=canvas&t=zKTlQdSK85cIBcfm-0
It should be mobile responsive and clean code.
Please update the Memo Page, to the one mentioned in figma.
Bounty: 135 USDC.
Assigned to @princedalsaniya

@vaibhavgeek I have raised the PR with all the designs with responsive implementations.
Please go through it and Update the TODO items. We need to update the links and some numbers in cards.


CASMINST-4589: Install document linting, corrections, and improvements

Summary and Scope
During the mug install I noticed some small errors in the docs, and some areas for improvement. This PR contains those, plus the necessary linting to make the style and spell checkers happy.
One thing I removed was a step in Deploy Final NCNs which had the installer change the NCN root password if they didn't customize it in the NCN image. Now that we require it to be customized in the image, that step is no longer needed.
csm-1.2 backport: https://github.com/Cray-HPE/docs-csm/pull/1562
csm-1.0 backport: https://github.com/Cray-HPE/docs-csm/pull/1563

/backport release/1.2

Since this guy is still short 1 review and has been out for a week, I've posted in the #docs-csm channel to beg for another reviewer.


made the nav responsive

I have made the button responsive. If the window size is less than 768px then a burger menu will be visible at the top right and on click of that button the navbar will be visible.

Thank you for your contribution to this project. I appreciate your effort and interest in contributing the code. However, I’m sorry to inform you that this pull request is for an issue that was closed a while ago and now this pr has a conflict with the main branch. I hope you understand and don’t feel discouraged. Please feel free to check out other open issues and submit new pull requests in the future. Thanks again for your support and cooperation.👍


MRG, FIX: Remove force render

Over at MNE we have been getting failures that seem to be triggered by the GUI stuff we do when __del__ is called. This hopefully avoids the problem.
WIP until we can do some runs in MNE to see if it makes the problem go away.

Added Azure as a CI while I was at it

Same failures occur in MNE, but I still think this is worth having since the _force_render does not seem to be required here

@mwaskom feel free to review/merge if you're happy

Thanks!


POL-35 delete provider link

I just added a test for the controller (which mostly just ensures that it gets the user Id without an error), and moved some of the duplicate private helpers into a TestUtils class. Let me know if anyone has any feedback on that!


tf.data.Dataset allow change of structure

System information

TensorFlow version (you are using): v2
Are you willing to contribute it (Yes/No): only python code

Describe the feature and the current behavior/state.
My current use case is as follows:
I get a dataset from tensorflow_datasets.load() where the structure is (image, label), e.g. MNIST. I would like to only use the image information of that dataset. Therefore I tried to find a way to "unpack" that dataset from a dict into a single element (so no key required to work on it).
# Currently

dataset = tfds.load('MNIST', split='train')
for d in dataset:
  # Now d has the "shape" {'image': tensor, 'label': tensor}

# What I would like to do
dataset = tfds.load('MNIST', split='train')
dataset = tf.data.Dataset.unpack(dataset, 'image')
for d in dataset:
  # Now d is a tensor

When the dataset is unpacked I can also do further transformations on it, e.g. filter, batch, ...
Will this change the current api? How? No
Who will benefit with this feature? Users of tensorflow_dataset I assume.
Any Other info.

It could also be seen as a kind of subset, where when the subset size is 1 it unpacks the dict to a tensor.
This would allow selecting multiple keys as a subset from a tf.data.Dataset.

Hi @sleighsoft you can use the map transformation to do what you want:
dataset = tfds.load('MNIST', split='train')
dataset = dataset.map(lambda x: x['image'])
# now dataset only contains sequence of images


using Dictionary for caching causes multi-thread issues.

In RAdapter and FontsHandler, Dictionaries are used to cached brushes, pens and fonts.
Single instance pattern is used by PdfSharpAdapter, and this instance is used by all HtmlContainers generated during system running time.
It causes problem when using this single instance in a multi-thread environment. Refer Here
Please replace Dictionary with ConcurrentDictionary.

I can assure this bug. This happened for us and had to spent hours to figure out the issue.


DRILL-4607: Add a split function that allows to separate string by a …

…delimiter
This patch allows to apply a split function by providing a string and a delimiter.
Addressed the review comments from Sudheesh.

+1 LGTM.

I missed the updated PR somehow. Sorry.
@adityakishore are you going to merge this?

Nvm, you just did :)


feat(gi): moved all functions dealing with optval and optpair to giopt and gi2opt

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

Project coverage is 100.00%. Comparing base (cb9c841) to head (c89d087).

Additional details and impacted files
@@            Coverage Diff             @@
##              main       #41    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           58         3    -55
  Lines          984       248   -736
==========================================
- Hits           984       248   -736

Flag
Coverage Δ

gi
?

optional
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

coverage: 100.0%. remained the same
when pulling 2a2359a63d5479f599a67d3f441d92978daef3b4 on feature/gi-2
into cb9c841a188584347361ee716f6c0253ed1c6514 on main.


Cannot run "gradlew run"

I'm following the instructions at http://ea.marklogic.com/features/data-integration/entity-services/ , and for "gradlew run", Gradle errs out because it can't find the data-movement dependency:
$ ../gradlew run
:entity-services-examples:compileJava
FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':entity-services-examples:compile'.

Could not find com.marklogic:data-movement:1.0-SNAPSHOT.
Searched in the following locations:
https://repo1.maven.org/maven2/com/marklogic/data-movement/1.0-SNAPSHOT/maven-metadata.xml
https://repo1.maven.org/maven2/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.pom
https://repo1.maven.org/maven2/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.jar
file:/C:/Users/rrudin/.m2/repository/com/marklogic/data-movement/1.0-SNAPSHOT/maven-metadata.xml
file:/C:/Users/rrudin/.m2/repository/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.pom
file:/C:/Users/rrudin/.m2/repository/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.jar
https://jcenter.bintray.com/com/marklogic/data-movement/1.0-SNAPSHOT/maven-metadata.xml
https://jcenter.bintray.com/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.pom
https://jcenter.bintray.com/com/marklogic/data-movement/1.0-SNAPSHOT/data-movement-1.0-SNAPSHOT.jar
Required by:
entity-services-parent:entity-services-examples:unspecified

It seems that the release went out with a snapshot dependency.

Hi Kiran, we'll need to take another look at the release this week.  I'd like to just take care of these early version-related issues that @rjrudin has raised, and push things out again.
https://github.com/marklogic/entity-services/issues/87

Actually, I noted that the dependency is updated properly for the release tag, but I botched the master branch.  What branch are you using @rjrudin ?  We had considered leaving develop depending on SNAPSHOT DMSDK, but if that's inconvenient we could reconsider.

Ah, I'm on "develop", but that's apparently the default branch - i.e. following the instructions will clone the repo and you'll start on the develop branch. So at least per the instructions, a user will run into this issue, unless they change over to master.

EA users should be working off of master, no? A public release should be tagged on master. develop should be stable, but not necessarily aligned with a release tag.

Sounds right, but when you run "git clone https://github.com/marklogic/entity-services.git", you start off on the develop branch. Same is true for when you go to https://github.com/marklogic/entity-services - the default branch is develop.

Ah yes, we should change the default then.
@grechaw do I have the workflow correct here? If so, can you please change the default branch?

It seems that the release went out with a snapshot dependency.

@kkanthet, I thought all non-GA releases would have SNAPSHOT versions, for the sake of Maven. Is that not the case?

For EA release we mentioned will have EA versions and develop is the only place we will have SNAPSHOT.
I think what's missed here is during the release is

merge EA release to master branch
Change default branch to Master

merge EA release to master branch
Change default branch to Master

Yes, that sounds perfect.

We merged last week.  Will make the change to default branch.

OK I've changed the default branch to master now.  I'll close this issue as that's the remediation, but keep it coming!  You may after all want to try develop after the gradle version is updated :)


SDK-2334:added support for enabling expanded document fields

PR looks good. Should remove idv.exe executable from the '_examples/idv' folder.


refactor with some newer apis

@L3MON4D3 are those test failures from my changes?

@L3MON4D3 are those test failures from my changes?

I think so.. they can (should be able to, at least) also run locally, via make test. Could you try if that works?

I can't run the tests because I get an error when compiling the dependencies (I have no clue what this is xD)

Okay, I can take a closer look later.
What's the error, though? Maybe it's a quick fix😅

Taking a full look at the code, I think it would be easier to read if we abstract the vim.api.nvim_create_autocmd(event, {callback=..., group = augroup}) to a function ls_autocmd(event, callback) (locally inside _setup)

done

anything else that should be done @L3MON4D3

I'm taking a look at the tests right now, they're acting up and I can't quite figure out why :/
But I don't think there's anything left, I did a few small changes.. I'll push them, then you can take a look at those, but then, apart from fixing the tests, nothing (at least code-wise, we'll need to add a breaking-change notice in the commit-messages, but let's wait with that until I've got my changes in :D)

I'd give the function a shorter name

Oh, right, that's an artifact from when I though remap/noremap was causing issues.
I'll change that

Okay soo "User None" as the default-event obviously no longer works, seems like a race-condition caused the tests to fail (autocommand and lua-call modify snippet simultaneously => bad) (not completely sure what the autocommands are registered for if "User None" is passed as the event, but nothing good, apparently xD)
I'm on the fence on how to handle this problem: if someone had "User <something>" in their setup, this now acts differently from before, BUT I don't really think anyone has that in their config. Leaning towards only putting a warning into the breaking change notice.
Oh, and I missed that the doc should be updated now that *_events also accepts a table :/
Would you do that, and make proper commits (including the breaking change notice)? Your first commits are nice, but my last commits have to be split up a bit :|
If you don't want to deal with that, I can do that later, too, but I'll take a break now :D

I probably won't have time to do it in the next two days (perhaps also more)
so if you want to get this merged sooner you should do it

Okay :+1:

@leiserfg wanna take a look?

On it

Okay perfect, thank you :+1:

Thank you for initiating this, and making the majority of the changes so quickly @max397574 :)


Applying reduction GLPK_MI appears to be in an infinite loop

Describe the bug
I am using cvxpy (latest version) to solve a very large mixed integer problem for some of the subsets of data I get stuck on the "Applying reduction GLPK_MI" step.  By get stuck I mean the code seems to be running indefinitely and clicking the interrupt button in my notebook won't kill it.  I have to restart my cluster to get out of this.  Most times things work perfectly, but for a few of my subsets I get this problem.
To Reproduce
It would be difficult to put that here.  I have tried to find a minimal example which creates this issue, but so far the smallest one I can find is 1393 variables, 883 constraints.
Expected behavior
I expect that if my constraints are infeasible, that the problem will tell me that so that I can remove some constraints and try again.  But I'm in the worst possible case, no errors, so I can't use a try:, except: block.  It just gets stuck.
Output
(CVXPY) Sep 26 07:56:41 PM: Your problem has 1393 variables, 883 constraints, and 0 parameters.
(CVXPY) Sep 26 07:56:41 PM: It is compliant with the following grammars: DCP, DQCP
(CVXPY) Sep 26 07:56:41 PM: (If you need to solve this problem multiple times, but with different data, consider using parameters.)
(CVXPY) Sep 26 07:56:41 PM: CVXPY will first compile your problem; then, it will invoke a numerical solver to obtain a solution.
                              Compilation

(CVXPY) Sep 26 07:56:42 PM: Compiling problem (target solver=GLPK_MI).
(CVXPY) Sep 26 07:56:42 PM: Reduction chain: FlipObjective -> Dcp2Cone -> CvxAttr2Constr -> ConeMatrixStuffing -> GLPK_MI
(CVXPY) Sep 26 07:56:42 PM: Applying reduction FlipObjective
(CVXPY) Sep 26 07:56:42 PM: Applying reduction Dcp2Cone
(CVXPY) Sep 26 07:56:43 PM: Applying reduction CvxAttr2Constr
(CVXPY) Sep 26 07:56:43 PM: Applying reduction ConeMatrixStuffing
(CVXPY) Sep 26 07:56:49 PM: Applying reduction GLPK_MI
(It is now 08:10 PM and my notebook is still "running command", most problems of this size finish in about 5 seconds.)
Version
I'm running in python 3.8 via a databricks notebook, spark runtime version 10.3.  I'm using the latest versions of cvxpy and cvxopt

Here is a fairly small problem which induces this behavior
Maximize:
var2383[1] + var2383[2] + var2383[4] + var2383[5] + var2383[6] + var2383[8] + var2383[9] + var2383[11] + var2383[12] + var2383[13] + var2383[15] + var2383[16] + var2383[18] + var2383[19] + var2383[20] + var2383[22] + var2383[23] + var2383[25] + var2383[26] + var2383[27] + var2383[29] + var2383[32] + var2383[34] + var2383[36] + var2383[37] + var2383[39] + var2383[40] + var2383[41] + var2383[43] + var2383[44] + var2383[46] + var2383[47] + var2383[48] + var2383[50] + var2383[53] + var2383[57] + var2383[58] + var2383[60] + var2383[61] + var2383[62] + var2383[64] + var2383[65] + var2383[67] + var2383[68] + var2383[69] + var2383[71] + var2383[72] + var2383[74] + var2383[75] + var2383[76] + var2383[78] + var2383[79] + var2383[81] + var2383[82] + var2383[83]
Subject to:
0.0 <= var2383
Sum(var2383[0:7], None, False) <= 1.0
Sum(var2383[7:14], None, False) <= 1.0
Sum(var2383[14:21], None, False) <= 1.0
Sum(var2383[21:28], None, False) <= 1.0
Sum(var2383[28:35], None, False) <= 1.0
Sum(var2383[35:42], None, False) <= 1.0
Sum(var2383[42:49], None, False) <= 1.0
Sum(var2383[49:56], None, False) <= 1.0
Sum(var2383[56:63], None, False) <= 1.0
Sum(var2383[63:70], None, False) <= 1.0
Sum(var2383[70:77], None, False) <= 1.0
Sum(var2383[77:84], None, False) <= 1.0
Sum(var2383[1:84:7], None, False) + -Sum(var2383[2:84:7], None, False) <= 0.0
Sum(var2383[2:84:7], None, False) + -Sum(var2383[4:84:7], None, False) <= 0.0
Sum(var2383[4:84:7], None, False) + -Sum(var2383[5:84:7], None, False) <= 0.0
Sum(var2383[5:84:7], None, False) + -Sum(var2383[6:84:7], None, False) <= 0.0
Sum(var2383[6:84:7], None, False) + -Sum(var2383[1:84:7], None, False) <= 0.0

This is the CVXR repo, not CVXPY. In any case, we can't proceed unless we have a reasonably sized, complete, reproducible example.

Sorry,
Are you saying my example doesn't qualify as reasonably sized, complete or reproducible?


BUG: Modin on HDK fails during apply call

Modin version checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest released version of Modin.

[X] I have confirmed this bug exists on the main branch of Modin. (In order to do this you can follow this guide.)

Reproducible Example
# %%
import modin.pandas as pd

import modin.config as cfg
cfg.StorageFormat.put('hdk')

# import pandas as pd

import random
import string

def get_random_string(length):
    return ''.join(random.choice(string.ascii_letters) for i in range(length))

# %%
s2id = {get_random_string(20): i for i in range(1000_000)}

# %%
df = pd.DataFrame({'user': list(s2id.keys()) * 10})

# %%
df['new_id'] = df['user'].apply(lambda x: s2id[x]).astype('int64')

# %%
print(df)

Issue Description
Modin on HDK fails, executing the code
Expected Behavior
It's a bug
Error Logs

UserWarning: Distributing <class 'dict'> object. This may take some time.
UserWarning: <function DataFrame.to_numpy> is not currently supported by ExperimentalHdkOnNative, defaulting to pandas implementation.
Please refer to https://modin.readthedocs.io/en/stable/supported_apis/defaulting_to_pandas.html for explanation.
UserWarning: <function DataFrame.applymap> is not currently supported by ExperimentalHdkOnNative, defaulting to pandas implementation.
Traceback (most recent call last):
  File "hdk_test.py", line 25, in <module>
    print(df)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/pandas/base.py", line 3948, in __str__
    return repr(self)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/pandas/dataframe.py", line 246, in __repr__
    result = repr(self._build_repr_df(num_rows, num_cols))
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/pandas/base.py", line 185, in _build_repr_df
    if len(self.index) == 0 or (self._is_dataframe and len(self.columns) == 0):
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/pandas/base.py", line 4023, in __getattribute__
    attr = super().__getattribute__(item)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/pandas/base.py", line 591, in _get_index
    return self._query_compiler.index
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/storage_formats/hdk/query_compiler.py", line 471, in _get_index
    return self._modin_frame.index
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/execution/native/implementations/hdk_on_native/dataframe/dataframe.py", line 2109, in _get_index
    self._execute()
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/execution/native/implementations/hdk_on_native/dataframe/dataframe.py", line 1895, in _execute
    new_partitions = self._partition_mgr_cls.run_exec_plan(
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/execution/native/implementations/hdk_on_native/partitioning/partition_manager.py", line 302, in run_exec_plan
    at = omniSession.executeRA(cmd_prefix + calcite_json)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/execution/native/implementations/hdk_on_native/hdk_worker.py", line 91, in executeRA
    return cls._executeRelAlgJson(ra)
  File "/nfs/site/home/ekrivov/large/miniconda3/envs/hm/lib/python3.8/site-packages/modin/experimental/core/execution/native/implementations/hdk_on_native/hdk_worker.py", line 76, in _executeRelAlgJson
    res = rel_alg_executor.execute()
  File "_sql.pyx", line 103, in pyhdk._sql.RelAlgExecutor.execute
RuntimeError: Hash join failed, reason(s): Could not build a 1-to-1 correspondence for columns involved in equijoin | Cannot join on rowid | Cannot fall back to loop join for non-trivial inner table size

Installed Versions

INSTALLED VERSIONS
commit           : a6862b51cc1d4be328f8f52a5fb69fb0650f7e97
python           : 3.8.15.final.0
python-bits      : 64
OS               : Linux
OS-release       : 5.4.0-136-generic
Version          : #153-Ubuntu SMP Thu Nov 24 15:56:58 UTC 2022
machine          : x86_64
processor        : x86_64
byteorder        : little
LC_ALL           : None
LANG             : en_US
LOCALE           : en_US.ISO8859-1
Modin dependencies
modin            : 0.19.0+6.ga6862b51
ray              : 2.2.0
dask             : 2023.3.0
distributed      : 2023.3.0
hdk              : present
pandas dependencies
pandas           : 1.5.3
numpy            : 1.23.5
pytz             : 2022.7
dateutil         : 2.8.2
setuptools       : 65.5.0
pip              : 22.3.1
Cython           : 0.29.32
pytest           : 7.2.0
hypothesis       : None
sphinx           : None
blosc            : None
feather          : None
xlsxwriter       : 3.0.3
lxml.etree       : None
html5lib         : None
pymysql          : None
psycopg2         : None
jinja2           : 3.1.2
IPython          : 8.7.0
pandas_datareader: None
bs4              : 4.11.2
bottleneck       : 1.3.5
brotli           :
fastparquet      : None
fsspec           : 2022.11.0
gcsfs            : None
matplotlib       : 3.6.2
numba            : None
numexpr          : 2.8.4
odfpy            : None
openpyxl         : None
pandas_gbq       : None
pyarrow          : 8.0.0
pyreadstat       : None
pyxlsb           : None
s3fs             : None
scipy            : 1.9.3
snappy           : None
sqlalchemy       : 1.4.39
tables           : None
tabulate         : None
xarray           : None
xlrd             : None
xlwt             : None
zstandard        : None
tzdata           : None

can't reproduced for
modin-hdk                 0.19.0          py310hff52083_0    conda-forge
pyhdk                     0.4.0           py310h1234567_0_cpu    conda-forge


Copy Changes - Suggestion

https://docs.google.com/document/d/1z-2tpkjj-L_pJViX4U__H1mT9oX05xFtxhnM36NrCww/edit?usp=sharing

Thank you @carmem-s I will incorporate them.


two recent wontfix EoP by GPZ

Issue 2207: Windows: WFP Default Rules AppContainer Capability Bypass EoP
https://bugs.chromium.org/p/project-zero/issues/detail?id=2207
Issue 2194: Windows: CreateProcessWithLogon Write Restricted Service EoP
https://bugs.chromium.org/p/project-zero/issues/detail?id=2194

Thank you for your feedback :-) This list was intended to be a summary of what happend in July 2021 and I decided I'll keep it that way, because I honestly think I don't have the energy to maintain an up-to-date list of ALL won't fixes Microsoft has to offer.
I added a remark to clarify this.


perf(commitizen): Commitizen support

Commitizen helps in writing conventinal commit messages that will help in semantic versioning.

Hi thank you for your PR. Please have a look at the PR guidelines, every PR should solve an issue. I don't see an issue for this request.
What are you trying to solve?
Please open an issue first so we can discuss it.


Exercises-w1

Exercises
Name
Olaf Driessen
Class
Tech 3
GitHub username
https://github.com/OlafDs
Bootcamp exercises
https://github.com/OlafDs/js-bootcamp/tree/master/Week1-Exercises
Bootcamp notes

👋 Thanks, we'll look at the exercises from exercises-w2 for your final grade!


fix: start of counter

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: