[SPARK-46393][SQL][FOLLOWUP] Classify exceptions in JDBCTableCatalog.loadTable

What changes were proposed in this pull request?

This is a followup of https://github.com/apache/spark/pull/44335 , which missed to handle loadTable
Why are the changes needed?

better error message
Does this PR introduce any user-facing change?

no
How was this patch tested?

existing test
Was this patch authored or co-authored using generative AI tooling?

no

cc @MaxGekk @yaooqinn

Thank you
Merged to master

@cloud-fan @yaooqinn
It seems that some UTs have failed, as follows:
https://github.com/panbingkun/spark/actions/runs/9411155017/job/25926739251

I have submitted a PR attempt to fix it: https://github.com/apache/spark/pull/46912

Oops, My bad, can you cherry-pick 82b4ad2af64845503604da70ff02748c3969c991 into your new PR? I just reverted this to recover the CI

Oops, My bad, can you cherry-pick 82b4ad2 into your new PR? I just reverted this to recover the CI

Okay, let me to do it.


Flow error: params is not iterable (fatal)

Flow version: 0.98.1
Expected behavior
I believe my code should pass the flow types test. Just for sake, I'd like to tell I'm using babel's private properties plugin.
Actual behavior
(I'm using linter-flowtype in Atom IDE). I get an error at line 1 of the file, where I put /* @flow */. It says "Parsing error: params is not iterable (fatal)". I'm not sure what that means - I have nothing in my file named params.

Github repo: https://github.com/cs-numerical-analysis/cs-numerical-analysis.github.io/blob/d2fa48c5d345c23e49ad39e7be2782e640f71d7d/src/graphs/Polynomial.js

This is a bug with babel-eslint and error comes from ESLint.


A extends B relation break after wrapping by extra interface

Reproduce link:  https://www.staging-typescript.org/play?ts=4.0.0-beta

The hint indicates that CCELEextPCEle in line 29  is false.  However when I uncomment line 20 and comment 17~19, the result back to true as expected.

Please don't post screenshots of code without accompanying text. It's absolutely tedious to type in 30 lines of code to validate things.
To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.


relax DDL check in is_type_compiled

The check for type availability uses a match on DDL bodies of argument
DDL and that previously compiled and stored in metadata. That check is
not strictly necessary, but more importantly, it is bound to fail when
DDLs are produced from the same schema but in different #ddl_vX{} versions.
Relaxing this check will allow post-1.2 nodes to see DDLs compiled by
different ddl record versions, and thus do not be a blocker in
upgrade/downgrade scenarios.

It's necessary because riak core metadata updates are not guaranteed to arrive in the original order. It's possible that two different nodes could have two different versions of the DDL if the bucket type is redefined.

Then, we would need to compare the two DDLs after converting each to the version this node is running, I suppose. And, write the upgrade/downgrade functions, of course. Work to do.

I gather the concern of this PR is no longer valid, now that DDLs do happily upgrade and downgrade. One last thing to do here is, then, to GC this PR.


perf(db) delete expired rows without transaction with postgres

Summary
The reason is that transaction may acquire unneccessary locks, and is
also non-resilient to errors, e.g. if one table is missing ttl column
no cleanups will be done to other tables either.
This changes the code so that cleanups are ran separately on each table.
It might have something to do with #4601.

@thibaultcha I made a slight change. Now it tries to execute everything in single roundtrip, if that fails it continues from the failed statement (to get error properly logged) and then continues on those that were after the errored one.


Spike Research for credential free connection support for 3rd party databases

Context
Support for managed identity has become a fundamental principle for Azure, providing a security baseline at the individual service level. We believe aligning on those guidelines will also benefit Spring developers at large.
We do not support credential-free connection for any 3rd party libraries today, and for 1st party libraries where we already have support, documentation and samples are lacking to guide Spring developers to complete their journey. Lack of samples has been one of the top challenges reported in the Nov 2021 Java on Azure E2E report.
Goal
Since we planned the feature of connecting free connection support for 3rd party databases for next semester, we should work out the feature spec before we start working on this feature. The goal of this feature is to output:

A feature spec
A design spec
A roadmap of this feature

Here is the feature spec for credential-free support of Spring Cloud Azure:https://github.com/Azure/azure-sdk-for-java/wiki/Credential-free-connection-support-for-Spring-Cloud-Azure.
Close this issue.


⚠ ClamAV has degraded performance

In 93db521, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 4f58a23.


expose amount of requests in queue

Library Affected:
workbox-background-sync
Browser & Platform:
N/a
Feature Request Description:
I've got a custom onSync function for workbox-background-sync as I need to modify requests as they are sent over the networ... anyway, this means my app has a small synchronisation icon in the app bar which displays the number of requests that are still queued up when the customer has no (or a very slow) network.
I currently do this using (await queue.getAll()).length and decrementing the number every time I shift a request from the queue in my onSync function.
This works, except for the case where the user is causing more requests to be added to the queue while onSync is running, in which case the counter will go into the negative.
I can solve this by doing (await queue.getAll()).length before every single request, but that feels like it would be terribly inefficient.
I noticed that getAll (a few levels deep) is essentially:
  async getAllEntriesByQueueName(
    queueName: string,
  ): Promise<QueueStoreEntry[]> {
    const db = await this.getDb();
    const results = await db.getAllFromIndex(REQUEST_OBJECT_STORE_NAME, QUEUE_NAME_INDEX, IDBKeyRange.only(queueName));
    return results ? results : new Array<QueueStoreEntry>();
  }

Seeing idb is used under the hood, we can create a new function (almost of clone of the one above) that uses countFromIndex instead. With that, something like queue.size() could be exposed (or .getCount, whatever).
Would this be something that could be accepted as a feature? I can implement it myself and provide a pull request :-)

See #2044 for a nice screenshot of a shared use-case :-)


ITransitionFactory in CloneableVertexGraphExplorerAlgorithm and TransitionFactoryImplicitGraph

This may be me misunderstanding something....however....
There doesn't seem to be a way to set TransitionFactories in either of the classes that use them (CloneableVertexGraphExplorerAlgorithm and TransitionFactoryImplicitGraph).  The constructors don't accept these and the property is get only.
Again, it may be that my understanding of the expected functionality is flawed, but I don't see how either of these implementation classes can do anything really useful without being able to set a transition factory.
I can fix a copy for what I need - if you would like me to contribute any changes then let me know.

Hello,
I forked this QuickGraph repository (here) and updated these implementations, in my opinion, in a clearer way (see here).
From what I understood, you should add classes implementing the ITransitionFactory to either the graph or the algorithm. And then they will be browsed on demand.
I also made a lot of unit tests in this fork I advise you to have a look at them to have an idea (here).


Add Lastversion sample application

nice!

@rabbah thanks! will of being neither useful nor complete (yet), but heck it was so far 👍

i was happy to see a go action :D


Log block execution latency metric

Motivation
Implementing metrics in https://docs.google.com/spreadsheets/d/1tHo8iKzrlkuYeN2Yu7a6MfwQd-P2KNA9BlnhcASoEQY/edit?pli=1#gid=0
Proposal
Implementing block execution latency Histogram metric
Test Plan
Will update

Current dependencies on/for this PR:

main

PR #1132

PR #1133

PR #1134   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #1132

PR #1133

PR #1134   👈

This comment was auto-generated by Graphite.

Merge activity

Oct 16, 11:20 PM: @andresilva91 started a stack merge that includes this pull request via Graphite.


DJ 1.9+ support (min 1.7)

Use get_backend_timeout() introduced in Django 1.7. As noted in the code _get_memcache_timeout is now deprecated.

This is incorporated into https://github.com/django-pylibmc/django-pylibmc/pull/40.

Cool. Is that project still being maintained?

@marcgibbons barely :(.  Thanks for the fix, I merged #40.

@jwhitlock that's great! thanks for all your work 😄


smtweb should open a browser tab

One issue is that the browser call must take place after starting the server, but the server call does not return. Use multiprocessing or threads?
Imported from Bitbucket issue 23,  created by apdavison on 06-26-2010 at 21:27, last modified: 07-20-2010 at 10:33

The thread module seems to do the job. See [175:c3d583de0bf1]
Trac comment by apdavison on 07-20-2010 at 10:33


When to retain RACDelegateProxy?

In the section "Revisiting FunctionalReactivePixels", there's a NOTE that delegate objects must be retained. However, "collectionViewDelegate" is retained while "RACDelegateProxy *viewControllerDelegate = [[RACDelegateProxy alloc] initWithProtocol:@protocol(FRPFullSizePhotoViewControllerDelegate)];" is not retained.

Interesting, you're right. Look like a bug – I'll fix in an upcoming release. Thanks!


Continuously Recieving error:  Fatal error:  Cannot redeclare mb_ord() in /var/www/html/Codiad/lib/diff_match_patch.php on line 2079

Please advise,
Just installed CODIAD and I continuously receive this output

Fatal error:  Cannot redeclare mb_ord() in /var/www/html/Codiad/lib/diff_match_patch.php on line 2079
regards

I am using termux and I am getting a similar error.
I can not do anything really.  I try to create a new file and I get the error.  This was my first spin of codiad, so I don't know enough to diagnose anything.

Fatal error: Cannot redeclare mb_ord() in /data/data/com.termux/files/home/codiad/lib/diff_match_patch.php on line 2079

See my comment in issue #1082.


Disable Geolocation by commenting it out in the config.js for Event configs

add the comment
The geolocation button has been hidden from the user interface by commenting out the widgetpath for geolocation below. You can enable geolocation by uncommenting the widgetpath for geolocation. The geolocation button, however, will only work on browsers supporting geolocation over HTTP connections.  If a browser requires a HTTPS connection to support geolocation and the button is enabled, the button will not work, but the application will work correctly otherwise.
Do this for Event Calendar and Event Locator only.
Do not do this for other configurations.

@lindsayking @LindsayThomas need to test before packaging


Piror data set problem

Does the prior dataset store the state-action pair in order?
How many independent trajectories in the prior dataset (e.g., pickplace_prior.npy)
Thank you!
Best

For future reference:

Yes, they are in order.
"For this simplified experimental setting, the prior data consists of 10K grasping attempts from a randomized scripted policy (that has a success rate of about 40%, details of this policy are in Appendix A.1)" in Section 6.1 of the paper (https://arxiv.org/pdf/2010.14500.pdf)

Thanks,
Avi


No support for pagination

Is your feature request related to a problem? Please describe.
When running sumocli roles list and it returns more than 100 results there is no way for the cli to obtain the extra results.
Describe the solution you'd like
Pagination support needs to be added to any API calls that support it.
Describe alternatives you've considered
There is the assumption that people may not work with over 100 results but I can see a use case if their reports being run etc.
Additional context
It currently only affects the roles list command.

Support for pagination has been added as commands have been refactored to use the sumologic-go-sdk.


initWithUrlToZipFile crash!

hello, IOS-Pods-DFU-Library works fine before，but today i got error with method  initWithUrlToZipFile,i am sure the firmwarefilePath is right and the firmware is valid(I test it at nrf-toolbox,work fine), can you help me how to resolve this problem?
crash info :         Zip.swift           line-57
    /**
     Set of vaild file extensions
     */
    internal static var customFileExtensions: Set<String> = []
    error:thread1 signal-SIGABART

my code :
- (void)startDFUWithDfuTarg:(CBPeripheral *)dfuTarg {
    NSMutableString * filePath = [[NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES) objectAtIndex:0] mutableCopy];
    [filePath appendString:[NSString stringWithFormat:@"/package/%@", [ConfigService shareService].packageName]];
    DFUFirmware * selectedFirmware = [[DFUFirmware alloc] initWithUrlToZipFile:[NSURL fileURLWithPath:filePath]];
    DFUServiceInitiator *initiator = [[DFUServiceInitiator alloc] initWithCentralManager:[PHBluetooth sharePHBluetooth].DFUCentralManager target:dfuTarg];
    [initiator withFirmware:selectedFirmware];
    initiator.forceDfu = YES;
    initiator.packetReceiptNotificationParameter = 12;
    initiator.delegate = self;
    initiator.progressDelegate = self;
    self.controller = [initiator start];
}

my cartfile.resolved
github "NordicSemiconductor/IOS-Pods-DFU-Library" "3.0.6"
github "ReactiveCocoa/ReactiveObjC" "2.1.2"
github "SnapKit/Masonry" "v1.0.2"
github "realm/realm-cocoa" "v2.4.3"

Can you first try to use DFU library version 3.2.1 and see if the issue is still there in the latest version ?

Also what is the full name of your zip file ?

thanks a lot! DFU library work fine now after updated to version 3.2.1

Perfect ! that's great news, keep up the good work 👍


chore: move list of non-context commands to constant

Small update to allow us to change this easier in the future.

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


CAS v5 PM, AD Server needs encoded password sent

When using CAS v5 RC3 the password management feature does not seem to be working for AD servers. I checked some of the code and it seems that in order for it to work with AD it has to be encoded using UTF-16LE. I did not see any references to it currently unless I missed it somewhere. Trying to change a password results in LdapErr: DSID-0C090C7D, comment: Unknown extended request OID, data 0
I have included the following in my report:

CAS server version : v5 RC3
Container/environment type/version : Tomcat/Java 8 , Windows
Steps to duplicate the problem : Try to change password where LDAP server is AD

Thanks for the feedback. Looks like CAS just does a PasswordModifyOperation with Ldaptive. Have you tried posting your question  to ldaptive forums and see if there is something else the code should do? The type of thing you're describing should be handled by ldaptive, it seems.
https://groups.google.com/forum/#!forum/ldaptive
Ping @dfish3r

I didn't see anything in ldaptive for it but I did see examples for unboundid sdk using UTF-16LE
I also looked at the way Unicon implemented it for their addon they made years ago, and they also had to use the unicodePwd attribute and encode as UTF-16LE if the server type was active directory
[https://github.com/Unicon/cas-password-manager/blob/master/src/main/java/net/unicon/cas/passwordmanager/ldap/ActiveDirectoryLdapServer.java]

Cool, I am happy to accommodate the change (or if you want to post a PR, that's perfectly better since you can confirm it works) but it might be a change better suited for ldaptive native. Please post to their forums and short of nothing else, we can always manually encode it.

I did find it in ldapative's code already, not sure how to translate all that though
https://github.com/vt-middleware/ldaptive/blob/7fe131b85cd4e399abed92203af5e10c68163070/core/src/main/java/org/ldaptive/ad/io/UnicodePwdValueTranscoder.java

Cool. We probably need Daniel's help here. Not sure if that needs to be plugged in somewhere, or whether it's something that CAS should invoke.

Ok, I will see what I can come up with but I am not a java developers so I will get lost pretty quickly probably but we can see what Daniel says,
Thanks!

No worries. We are here to help you out. Post back when you find more plz.

To change the unicodePwd attribute do this:
ModifyOperation modify = new ModifyOperation(conn);
modify.execute(
  new ModifyRequest(
    "dn-of-entry-to-modify",
    new AttributeModification(
      AttributeModificationType.REPLACE,
      new UnicodePwdAttribute("password"))));

Be aware of the caveats surrounding AD password changes:
https://msdn.microsoft.com/en-us/library/cc223247.aspx

Sweet! Thanks!!! It is working now, I just tried the latest snapshot at 5.0.0.RC3-20160928.104306 and it worked, I was able to successfully change the password.

Thanks @dfish3r


why does blink need local network access on startup?

to reproduce, make sure to uninstall app first

it's only when doing local development... so not an issue


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2306

Questions ?
Please refer to the Backport tool documentation


chore(ui): update typegen schema

Summary
schema.ts seems to be outdated due to changes between e44458609f82dae9cf28560ad460decc9bfd3b54 (5.4.3rc1) and current HEAD.
Note: Should double check to ensure the update was generated correctly.
Checklist

[x] The PR has a short but descriptive title, suitable for a changelog
[ ] Tests added / updated (if applicable)
[ ] Documentation added / updated (if applicable)
[ ] Updated What's New copy (if doing a release after this PR)

We should add a CI action that fails when the frontend types don't match the server.

@psychedelicious I can potentially have a look at this. Is there a specific way you want this to be implemented?
I was thinking something like:

create a new github action typegen-checks.yml
run make frontend-typegen (needs both python and js env)
check that schema.ts has no diff

(It's already out of sync again due to https://github.com/invoke-ai/InvokeAI/pull/7393)

@rikublock
Yep, that sounds good to me!


[Bug]: Google LTS release for Beam 2.45.0 to use LTS version of libraries-bom

What happened?
Google would like to use Apache Beam 2.45.0 for an LTS release and requires the BOM to be on an LTS version of the libraries-bom (e.g. 26.1.5) but the libraries-bom LTS is subject to https://github.com/googleapis/gax-java/issues/1858 bug which impacts BQ IO due to #21368 which required updating to the current BOM.
This is to finalize the discussion of whether Google releases an LTS which points to 2.45.0 or Beam migrates to a libraries-bom that has LTS support (e.g. 26.1.6) which contains the gax-java fix.
Issue Priority
Priority: 1 (data loss / total loss of function)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Thank you for creating this issue and trying to find a solution together for this complex problem.

Could the LTS release contains Beam 2.44.0 instead?


How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: https://freefilesync.org/download.php
Publisher: Zenju
Package Name: zenju.freefilesync
Description: Open source folder comparison and synchronization software
Package Version: 13.5
Installer URL: https://freefilesync.org/download/FreeFileSync_13.5_Windows_Setup.exe

Duplicate of #57900


[ATOM] Claim rewards (Stake) pushes don’t work

We need to look at GetBlockByNumber function, try to fetch list of blocks and find the reason why there are no any stake transactions there or maybe it is notifier issue

Cannot see any issues with on atlas side


Support biblatex-software

Roberto di Cosmo has suggested to me that we could support the biblatex-software style extension (docs) as a target style for BibTeX.
I think this is a great idea, because it (a) supports the biblatex-style effort, and (b) further integraytes CFF with ongoing efforts.
To start off, I (or someone else) would need to look into the style to figure out whether we should replace the current BibTeX output, or implement this as an alternative. I'm currently a bit short on time, but we could briefly speak about this during our CFF catch-up call, @jspaaks?

+1

If I can say it would be better to keep the BibText support ( --format bibtex) and add a new one named as BibLatex ( --format biblatex).


BUG:  dlopen("/usr/local/cuda/targets/x86_64-linux/lib/libcublas.so", RTLD_NOW | RTLD_LOCAL) failed, symbol free_gemm_select version libcublasLt.so.11 not defined in file libcublasLt.so.11 with link time reference

When execute tensorflow minist train task, occur the problem, 'Check failed: cublas_handle'.
It caused by dlopen, the complete command is dlopen("/usr/local/cuda/targets/x86_64-linux/lib/libcublas.so", RTLD_NOW | RTLD_LOCAL). And error throwed by dlopen is 'Failed to open /usr/local/cuda/targets/x86_64-linux/lib/libcublas.so: /usr/local/cuda/targets/x86_64-linux/lib/libcublas.so: symbol free_gemm_select version libcublasLt.so.11 not defined in file libcublasLt.so.11 with link time reference'.
According to ldd and nm, libcublas.so depend on libcublasLt.so.11, which linked to '/home/chenqian/Code/cuda_hook/output/lib64/libcublasLt.so.11'. And, there is no symbol free_gemm_select in both '/home/chenqian/Code/cuda_hook/output/lib64/libcublasLt.so.11' and '/usr/local/cuda/targets/x86_64-linux/lib/libcublasLt.so.11'.
Moreover, if without cuda hook, the train task can complete.

You need to hijack free_gemm_select.


Core and heatmap

This is the very basic thing to leaflet.js
I made this because updating existing lib from 0.7 -> 0.11 and generalizing it has almost same complexety.
Also ported Leaflet/Leaflet.heat with funny Ref emitting code.
Depends on garyb/purescript-dom-classy#6
@natefaubion Please take a look.

I've abstracted Ref, so, js doesn't know how refs work. Although creation and writing to it are located in js code.
@natefaubion Is this ok? I can create Ref in PS and leave only modifying in JS or just remove all this ref stuff at all until we actually need it

The main thing is just if there is a need for it to be in JS. We shouldn't put anything in JS that can be easily done in PS. At a glance, it looks like there is no reason that addRemove_ can't just take regular callbacks, and we can do all the Ref coordination in PS. Is that the case, or does the FFI rely on the Ref business?

As far as I understand leaflet code, onAdd should (contract or whatever) return this, so modification of of ref should be done in js. Although instead of Nothing and newRef I can put there ref with Nothing :D

Ah! Yep, init must be called in JS handler because leaflet/layer state could change

I think this will work


Prevent users from overriding our required arguments

We pass several arguments to Meson calls, some of them are required, some of them are simply meant as defaults, which the user can override (see GH-329). We should pass the our defaults first, then the user arguments, and finally, the arguments we require to not be changed.
Supersedes GH-330

With #383 merged, there aren't anymore required arguments passed to Meson.


Don't load zim if zsh < 5.2

and show an error message, as discussed in #184.
Code in modules/environment/init.zsh is also simplified to only support zsh >= 5.2

@Eriner, updated the commit.

LGTM


Entire GraphQL queries within JS files are illuminated

Describe the bug
When the cursor is on anything within a GraphQL query inside a JavaScript file, the entire string is illuminated.
To Reproduce

Copy the minimal config and example JS provided below into files.
Run neovim with the minimal config and let all the plugins and parsers install.
Open the example JS file using the minimal config.
Place your cursor inside the GraphQL query.
See the entire string highlighted as shown below.

Minimal config:
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  { "olimorris/onedarkpro.nvim", config = true },

  { "nvim-treesitter/nvim-treesitter", build = ":TSUpdate" },

  {
    'VonHeikemen/lsp-zero.nvim',
    branch = 'v2.x',
    dependencies = {
      -- LSP Support
      {'neovim/nvim-lspconfig'},             -- Required
      {                                      -- Optional
	'williamboman/mason.nvim',
	build = function()
	  pcall(vim.cmd, 'MasonUpdate')
	end,
      },
      {'williamboman/mason-lspconfig.nvim'}, -- Optional

      -- Autocompletion
      {'hrsh7th/nvim-cmp'},     -- Required
      {'hrsh7th/cmp-nvim-lsp'}, -- Required
      {'L3MON4D3/LuaSnip'},     -- Required
    },
    config = function()
      local lsp = require('lsp-zero').preset({})

      lsp.on_attach(function(client, bufnr)
	lsp.default_keymaps({buffer = bufnr})
      end)

      lsp.ensure_installed({ 'lua_ls', 'tsserver' })

      -- (Optional) Configure lua language server for neovim
      require('lspconfig').lua_ls.setup(lsp.nvim_lua_ls())

      lsp.setup()
    end
  },

  { "RRethy/vim-illuminate" }
}

require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

-- setup treesitter
local ok, treesitter = pcall(require, "nvim-treesitter.configs")
if ok then
  treesitter.setup({
    ensure_installed = { "lua", "javascript", "graphql" },
    highlight = { enable = true },
  })
end

vim.cmd("colorscheme onedark")

Test file:
const query = graphql(`
  query something($varname: Int!) {
    typename(varname: $varname) {
	    field1
	    field2
    }
  }
`);

Output from :IlluminateDebug
buf_should_illuminate 1 true
config {
  case_insensitive_regex = false,
  delay = 100,
  filetype_overrides = {},
  filetypes_allowlist = {},
  filetypes_denylist = { "dirbuf", "dirvish", "fugitive" },
  min_count_to_highlight = 1,
  modes_allowlist = {},
  modes_denylist = {},
  providers = { "lsp", "treesitter", "regex" },
  providers_regex_syntax_allowlist = {},
  providers_regex_syntax_denylist = {},
  under_cursor = true
}
started true
provider table: 0x7f4a1fcedf48 lsp
`termguicolors` true

Expected behavior
Don't illuminate the entire query. Ideally, illuminate individual items within the query when they are under the cursor. If that's not possible, don't illuminate inside the query at all.
Screenshots

Additional context
Seems to match #124.
Thank you for vim-illuminate!

That is matching a string. This cannot be controlled by vim-illuminate since the region is calculated by the LSP.

Gotcha. Thanks for the response! I can experiment with min_count_to_highlight = 2 or under_cursor = false as workarounds...
Any other suggestions? E.g., maybe something with should_enable?


Link error

Describe the bug
Two of different link point to the same page
To Reproduce
Go to 'https://docs.microsoft.com/en-us/SharePoint/getting-started?redirectedfrom=MSDN#where-do-i-start'
Links [2016 Hardware and software requirements] and [2013 Hardware and software requirements] lead to the same page, 'https://docs.microsoft.com/en-us/SharePoint/install/hardware-and-software-requirements'
Link [2013 Hardware and software requirements] have its own address, 'https://docs.microsoft.com/en-us/SharePoint/install/hardware-and-software-requirements-0'

Attempting to fix in https://github.com/MicrosoftDocs/OfficeDocs-SharePoint-pr/pull/2730/

@aflant
This fix has been published. you should see it on the site shortly. Thanks for helping us improve Microsoft documentation! ~Jason

#please-close


Export encode_png to tf.io

Fix https://github.com/tensorflow/tensorflow/issues/37168

This changes API, so it needs API owners review.

Thanks @vijayphoenix.
The context from the bug, makes it clear that this change gets encode_png more inline with all the other image encoding operations.  So I support it.
@mihaimaruseac, what is the API review process for PRs? Are they part of the weekly api-review meeting?

@MarkDaoust yes, they are reviewed during those meetings.


Added compatibility for custom loss functions

Following #1025, #1119, #911 and #868 custom objectives functions still can't be loaded when loading a saved model with model_from_json or model_from_yaml. I made a simple change to check if the names of the outputs were not in the custom_objects passed so we can feed it to the .compile method.
globals() seems to be local at the module level and the custom function is not visible inside the objectives.py module.
As always, I'm not sure if it's the best way to do it but I tried to minimise the changes in other modules and submodules.
For the tests, should I add a custom objective, compile it and try to serialize it using json and yaml?

This PR should close #1119.

Looks good to me. Thanks!


Add background-clip text supports?

This used to be available via MDN but for some reason isn't anymore...I noticed there was an entry in the caniuse DB that needed reviewing & updated so I've done so and the feature is now available at https://caniuse.com/background-clip-text

This used to be available via MDN but for some reason isn't anymore...I noticed there was an entry in the caniuse DB that needed reviewing & updated so I've done so and the feature is now available at https://caniuse.com/background-clip-text


Make doesn't do anything, bombs out immediately

Make doesn't do anything, bombs out immediately
Perhaps its a missing configuration routine is needed to insure whatever this project needs to compile is actually present, dunno because the Make command doesn't tell me anything but No rule to make target.
$ make
make: *** No rule to make target 'third-party/minini/dev/minIni.c', needed by 'all'.  Stop.

Run
git submodule init
git submodule update

Toxirc uses minini to read the configuration file and its included as a submodule.

I added a rule to the Makefile that will do that: make deps


Revert introduction of new feature that can automatically restart PC

The automatic restarts of the PC were introduced in order to get rid of
false positive errors that are shown in the editor. After testing this
feature for some time in usage it turned out that this feature is
basically worthless.
The main problem is that most of the time an editor shown real errors or
warnings and in all of these cases the PC is restarted, even though it
shouldn't. There is no way to test if an error is a false positive or
not, therefore the PC is restarted many times, which not only slows down
the editor but also has weird effects. Semantic highlighting often is
cleared after the PC is restarted and therefore users have to live only
with syntax highlighting for a second or two. The automatic restarts
of the PC basically are an unsafe feature and should only be used
manually by the user. Syncing all of the possible semantic features of
the IDE with the restart times of the PC may be possible but seem to
require heavy efforts, which should not be invested in a feature that
after all is a hack to work around bugs in the PC.
Because of this, this new feature is removed.
Fix #1002741
Re #1002725

LGTM


how to save the best model weights?

Hi,
How to export into saved weights to frozen model?

https://github.com/Visual-Behavior/detr-tensorflow/issues/34#issuecomment-1278721290 use this approach

the above way will save all checkpoints for best among them need to check


fix-url-path-footer

Description

Change url for events for the path https://www.clueless.tech/events
Issue Ticket Number
Fixes #(Issue 313)

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x]  have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] have checked to ensure there aren't other open Pull Requests for the same update/change?
[x]  I have performed a self-review of my own code.
[x] 	I have commented my code, particularly in hard-to-understand areas
[x] 	 I have made corresponding changes needed to the documentation

Picture

Congrats @LucianeFernandesRoque 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


I Have No Idea What I Am Doing to Cause This:

python3 train.py -opt train_sr.yml
Traceback (most recent call last):
File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 500, in <module>
main()
File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 466, in main
opt = parse_options()
File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 25, in parse_options
opt = options.parse(args.opt, is_train=is_train)
File "/home/nickdbts2022/Desktop/traiNNer/codes/options/options.py", line 552, in parse
raise ValueError("Configuration file {} not found.".format(opt_path))
ValueError: Configuration file options/train/train_sr.yml not found.

Uuuuuuuuuhhhhhhh, pardon me?
You lost me.

python3 train.py -opt train_sr.yml
export CUDA_VISIBLE_DEVICES=0
23-02-03 00:44:55.135 - INFO:   name: 001_sr_template
  use_tb_logger: True
  model: sr
  scale: 4
  gpu_ids: [0]
  use_amp: True
  use_swa: False
  use_cem: False
  use_atg: False
  datasets:[
    train:[
      name: DIV2K
      mode: aligned
      dataroot_HR: ['../datasets/train/hr1', '../datasets/train/hr2', '../datasets/train/hr3']
      dataroot_LR: ['../datasets/train/lr1', '../datasets/train/lr2']
      subset_file: None
      use_shuffle: True
      znorm: False
      n_workers: 4
      batch_size: 8
      virtual_batch_size: 8
      preprocess: crop
      crop_size: 128
      image_channels: 3
      use_flip: True
      use_rot: True
      use_hrrot: False
      phase: train
      scale: 4
      data_type: img
      aug_configs:[
        lr_downscale_types:[
          resize:[
            resize_prob:[
              down: 1.0
            ]
            resize_range_up: [1, 1.5]
            resize_range_down: [0.15, 1]
            down_up_min: 0.5
          ]
        ]
      ]
      shuffle_degradations: False
      lr_downscale: True
      lr_downscale_types: [773, 777]
      resize_strat: pre
    ]
    val:[
      name: val_set14_part
      mode: aligned
      dataroot_B: ../datasets/val/hr
      dataroot_A: ../datasets/val/lr
      znorm: False
      lr_downscale: False
      lr_downscale_types: [773, 777]
      phase: val
      scale: 4
      data_type: img
      resize_strat: pre
    ]
  ]
  path:[
    root: ..
    pretrain_model_G: ../experiments/pretrained_models/RRDB_PSNR_x4.pth
    experiments_root: ../experiments/001_sr_template
    models: ../experiments/001_sr_template/models
    training_state: ../experiments/001_sr_template/training_state
    log: ../experiments/001_sr_template
    val_images: ../experiments/001_sr_template/val_images
  ]
  train:[
    optim_G: adam
    optim_D: adam
    lr_scheme: MultiStepLR
    lr_gamma: 0.5
    swa_lr: 0.0001
    swa_anneal_epochs: 10
    swa_anneal_strategy: cos
    pixel_criterion: l1
    pixel_weight: 0.01
    feature_criterion: l1
    feature_weight: 1
    gan_type: vanilla
    gan_weight: 0.005
    manual_seed: 0
    niter: 500000.0
    val_freq: 5000.0
    metrics: psnr,ssim,lpips
    grad_clip: norm
    grad_clip_value: 0.1
    overwrite_val_imgs: None
    val_comparison: None
    lr_steps: [50000, 100000, 200000, 300000]
    swa_start_iter: 375000
    atg_start_iter: 415000
  ]
  logger:[
    print_freq: 200
    save_checkpoint_freq: 5000.0
    overwrite_chkp: False
  ]
  is_train: True
  network_G:[
    strict: False
    type: rrdb_net
    norm_type: None
    mode: CNA
    nf: 64
    nb: 23
    nr: 3
    in_nc: 3
    out_nc: 3
    gc: 32
    convtype: Conv2D
    act_type: leakyrelu
    gaussian_noise: True
    plus: False
    finalact: None
    upscale: 4
    upsample_mode: upconv
  ]
  network_D:[
    strict: True
    type: discriminator_vgg
    in_nc: 3
    base_nf: 64
    norm_type: batch
    mode: CNA
    act_type: leakyrelu
    convtype: Conv2D
    arch: ESRGAN
    size: 128
  ]

23-02-03 00:44:55.421 - INFO: Random seed: 0
Traceback (most recent call last):
  File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 500, in <module>
    main()
  File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 487, in main
    dataloaders, data_params = get_dataloaders(opt)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/train.py", line 134, in get_dataloaders
    dataset = create_dataset(dataset_opt)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/data/__init__.py", line 79, in create_dataset
    dataset = D(dataset_opt)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/data/aligned_dataset.py", line 41, in __init__
    self.A_paths, self.B_paths = get_dataroots_paths(self.opt, strict=False, keys_ds=self.keys_ds)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/data/base_dataset.py", line 235, in get_dataroots_paths
    paths_A, paths_B = read_dataroots(opt, keys_ds=keys_ds)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/data/base_dataset.py", line 171, in read_dataroots
    paths_A = process_img_paths(A_images_paths, opt['data_type'])
  File "/home/nickdbts2022/Desktop/traiNNer/codes/data/base_dataset.py", line 61, in process_img_paths
    paths = get_image_paths(data_type, path, max_dataset_size)
  File "/home/nickdbts2022/Desktop/traiNNer/codes/dataops/common.py", line 82, in get_image_paths
    paths = sorted(_get_paths_from_images(dataroot, max_dataset_size=max_dataset_size))
  File "/home/nickdbts2022/Desktop/traiNNer/codes/dataops/common.py", line 36, in _get_paths_from_images
    assert os.path.isdir(path), '{:s} is not a valid directory'.format(path)
AssertionError: ../datasets/train/lr1 is not a valid directory

I am lost, would someone help please? This is put politely as its the first time I am working with modelling code.

Hello!
These lines:
      dataroot_HR: ['../datasets/train/hr1', '../datasets/train/hr2', '../datasets/train/hr3']
      dataroot_LR: ['../datasets/train/lr1', '../datasets/train/lr2']

Are only an example, you should put here the paths to your image datasets for training (https://github.com/victorca25/traiNNer#training, https://github.com/victorca25/traiNNer/blob/master/docs/howtotrain.md#normal-single-image-super-resolution-esrgan-srgan-pan-etc-models).
Something similar will happen with:
      dataroot_B: ../datasets/val/hr
      dataroot_A: ../datasets/val/lr

and:
    pretrain_model_G: ../experiments/pretrained_models/RRDB_PSNR_x4.pth

The required files will have to exist in the path for them to be loaded correctly.

Oh, well, ok then, I'll try this with my own version of the code.
Thank you.


Alerts Management Content Validation Issue for learn.microsoft website

Section link1, link2, link3, link4, link5, link6, link7, link8, link9

Reason:
Typing annotation Issue: Please unify the format, update to *args:Any, **kwargs:Any or *args, **kwargs.

@rohit-ganguly, @lmazuel for notification.

Close this issue as we'll put it into a summarzied one.


Avoid unclosed sockets while opening connections

When connections could not be properly established (e.g. wrong password, database in maintenance, etc.) they were not being closed. This might be dangerous when making lots of connections, as the number of sockets (and hence the number of open files) pile up.

@oserde I believe the Go SQL code handles this. Do you have test code showing otherwise?

Hi @MichaelS11 . Thank you for your quick response.
I have been using this code for debugging:
package main

import (
	"database/sql"
	"fmt"
	"github.com/mattn/go-oci8"
	"log"
	"os"
	"time"
)

func main() {

	oci8.OCI8Driver.Logger = log.New(os.Stderr, "oci8 ", log.Ldate|log.Ltime|log.LUTC|log.Llongfile)

	db, err := sql.Open("oci8", "myuser/mypass@mydb:1527/xxx")
	if err != nil {
		fmt.Printf("Open error is not nil: %v", err)
		return
	}
	if db == nil {
		fmt.Println("db is nil")
		return
	}

	err = db.Ping()
	if err != nil {
		fmt.Printf("Error: %v", err)
		time.Sleep(time.Minute * 10)
		return
	}
}

Using a wrong password makes the Ping() call to throw an error. I put the Sleep() call to be able to check open sockets by the process. I have also checked tcp connections. The connection is not closed. In fact, it remains as ESTABLISHED. Eventually the server sends an RST packet due to inactivity. But the socket remains there.
I tried to use db.Close(), but it is useless in this case, because when it gets to this line , variable db.freeConn is empty, and no sockets are closed.
So this PR intends to fix two problems:

Sockets might not be closed after a failed connection.
Even though we wanted to fix this behaviour externally and call db.close() when Ping() throws an error, this call to db.Close() is unable to close the sockets, because the connection is not inside db.freeConn.

Do you think this behaviour should be implemented by go database/sql library?
Thank you.

I suggest making the following changes to your test. Move the test code into a function but leave the sleep in the main. Before the Ping add defer db.Close(). In oci8 connection.go Close function add a print/log line at the start of the function, really check Close gets called. I am thinking it will but it is very important to know for sure to track down what is going on.

See the issue, working on it.

Should be fixed in https://github.com/mattn/go-oci8/pull/339
Please test

@oserde Please close this PR if https://github.com/mattn/go-oci8/pull/339 fix the issue.

Hi @MichaelS11 . PR #339 definitely fixes the issue. Thank you!
Closing in favor of #399


rm node_modules in Docker

Fix for https://github.com/maddox/harmony-api/issues/34
Thanks @maddox !

Cool, I guess npm upgrade would work too, but this is cleaner and guaranteed. And since its the container being built asynchronously, time doesn't matter much. This is probably how it should be done anyways.


Backslashes on Windows (again)

Laravel Mix Version: 1.7.2
Node Version (node -v): 8.9.1
NPM Version (npm -v): 5.5.1
OS: Windows 10 x64

Description:
Trying to use html-webpack-plugin as well as trying to use Webpack lazy loading both have an issue on Windows because the paths to the JS and CSS files have backslashes in them instead of forward slashes. While Chrome transparently corrects them to forward slashes, other browsers (Firefox, for example), do not. (The reason it is an issue for lazy loading is that the manifest.js file contains the backslashed paths too.)
The fix for this is simple and was already done once in #323 (for JS files only), but I wanted to find out if there was a reason that it was removed (or whether it was just accidental) before I submit a new PR to put it back.
The fix for JS files is just adding .replace(/\\/g, '/') at the end of the line here, here, and here.
The fix for CSS files is adding the same thing at the end of the line here.
Steps To Reproduce:
(Note: This is a non-Laravel project example, but in a Laravel project the same issue occurs.)
Folder structure:
- src
\- html
  \- index.html
\- js
  \- main.js
  \- ... // doesn't matter
\- styles
  \- app.scss

webpack.mix.js
const { mix } = require('laravel-mix');
const HtmlWebpackPlugin = require('html-webpack-plugin');
const InlineChunkWebpackPlugin = require('html-webpack-inline-chunk-plugin');
const path = require('path');

mix
	.setPublicPath('build')
	.webpackConfig({
		plugins: [
			new HtmlWebpackPlugin({
				inject: true,
				template: 'src/html/index.html',
			}),
			new InlineChunkWebpackPlugin({
				// Adding `path.normalize` here should not be necessary,
				// but it is because of this backslash issue.
				inlineChunks: [path.normalize('/js/manifest')]
			})
		],
	})
	.js('src/js/main.js', 'js')
	.extract([
		'axios',
		'bootstrap-sass',
		'jquery',
		'vue',
		'vue-router',
		'vuex',
	], 'js/vendor')
	.sass('src/styles/app.scss', 'css')
	.version();

The actual contents of the JS and CSS files don't actually matter.

Use Double Backslashes in Strings
In PHP strings, a single backslash is used as an escape character. To use a literal backslash, you need to escape it by using double backslashes (\).

Example:
$path = "C:\\path\\to\\directory";
2. Use Forward Slashes
PHP can handle forward slashes (/) in file paths on Windows. Using forward slashes can help avoid the issue of escaping backslashes.
Example:
$path = "C:/path/to/directory";


Conflict with new num::Float::epsilon

Just a heads up -- we recently added Float::epsilon in rust-num/num#231, and then a Crater run showed that glm broke due to call ambiguity.  See the logs here:
https://tools.taskcluster.net/task-inspector/#ytvixlBAQzCMhtWXQCM3fA/0
The ambiguous call is in your ApproxEq::is_approx_eq, where Self::BaseType::epsilon could now either mean Float::epsilon or your BaseFloat::epsilon.  You could change it to a more explicit call to one of these choices.  Or since you seem to just be extending Float, you might remove your epsilon.  (Float also now has to_degrees and to_radians, if you like.)

Fixed by removing fn epsilon() from BaseFloat.
Thanks for reporting.


Add Scientific Calculation Options

As the current version of the app only performs the basic functions, it will be great if we include scientific operations.

Please assign it to me , I am willing to do it .

Okay sure @ansh711 Happy Coding

Thank you sir

Don't forget to star the repo 😄

sir I have to extend the whole calculator for adding new buttons . that will hamper the designing

what to do sir ?

Complete the scientific operation task and we will preview how it looks, after that we can look at the design how we can make it better.

okay sir

on it sir

Thanks @ansh711 for your time. If you have any suggestions we can do that as well.

sir i have done the possible code ... please check that

Great @ansh711 Have you created a pull request?

changes to create the functions of a scientific calculator #23
please check this

I just previewed the application and you were right Including more buttons Messed up the Design.

yes sir

the buttons alignment and design will be messed up

So can you fix this problem after adding the buttons ?

but the task of having the scientific functions are doing great i think

Did you previewed the application?

yep

Okay Great should I assign You another task to align the buttons accordingly 😄

yes that will be great help for me , because i am late in hactoberfest , can you provide me 2 pull/merge for this two task ?

I will assign that task to you. So be responsible and do it on time.  Thanks for your time

ok sure sir

please accept the pull request kindly

Okay let me review the code Don't worry I will accept it

okay sir, thank you so much

Can you fix the Design today ?

With all the scientific operations ?

i will try my best to do that tonight

but i need to change the css file,tags as i need ...... is that okay ?

Yes, You can change but both your pull requests will be accepted after you do it right 😄 ? Do you dare to take this chance ?

okay I am on it sir

Okay I have created new issue check that as well I will assign it to you.

I will start coding from 7 o'clock , I need to rest now  ..... please excuse me sir

Okay great Same here, I'm sitting in front of my Pc From morning. We need rest 👍🏼 Grab a tea or coffee then comeback to work

sir please check and accept my pull request please

There were some issue with the implementation..... From the deepsource ...... I have solved those ...... Please check that

I have merged your pull request, kindly check and lemme know. Thanks @ansh711 and will see you in future projects.


PUT a tarball to push deploy

adds push deploy, with tests, and readme docs.
problems with this:

had to workaround journey, which did not support streaming.
stream to /tmp and then start with a type="local" repository. which copies again.
also added an /info route which responds with the minimum necessary information about the apps running on the haibu cluster.

Cherry-picked


[Hub Generated] Review request for Microsoft.Network to add version stable/2020-11-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Please go through the contribution checklist added as part of the review and mark them completed. Also Please go through the ARM API Review Checklist. These steps are required before review can be continued.

/azp run

/azp run unifiedPipeline

/azp run unifiedPipeline

NRP usually uses monthly release branches for swagger updates.  Is there a reason this isn't following the normal NRP process?

Removed the breaking changes review label as breaking change rules passed and I don't see any breaking changes issues. @akning-ms please confirm. The PR is still good from ARM side.

Removed the breaking changes review label as breaking change rules passed and I don't see any breaking changes issues. @akning-ms please confirm. The PR is still good from ARM side.

@msyyc  for confirm Pythons SDK breaking change

@msyyc  for confirm Pythons SDK breaking change

Hi @disreedh, Maybe we discussed, but can not recall, Can you comment what is conclusion for below lint failure?:

❌ R3023 - OperationsAPIImplementation
Operations API must be implemented for '/providers/Microsoft.Network/operations'.New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L37

Hi @disreedh, Maybe we discussed, but can not recall, Can you comment what is conclusion for below lint failure?:

❌ R3023 - OperationsAPIImplementation
Operations API must be implemented for '/providers/Microsoft.Network/operations'.New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L37

@msyyc for confirm Pythons SDK breaking change

Breaking change is not caused by this PR.

@msyyc for confirm Pythons SDK breaking change

Breaking change is not caused by this PR.

@akning-ms Sorry I think I did not loop you in offline chats I had with ARM review DRI. For Frontdoor WAF, the operations API is defined under a different path (it's part of the Microsoft.Network APIs) azure-rest-api-specs/operation.json at master · Azure/azure-rest-api-specs (github.com).
The DRI asked me to suppress the issue for approval. For a permanent suppression I have raised a ticket following https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/85/Swagger-Suppression-Process for ARM Violation suppression.

@akning-ms Sorry I think I did not loop you in offline chats I had with ARM review DRI. For Frontdoor WAF, the operations API is defined under a different path (it's part of the Microsoft.Network APIs) azure-rest-api-specs/operation.json at master · Azure/azure-rest-api-specs (github.com).
The DRI asked me to suppress the issue for approval. For a permanent suppression I have raised a ticket following https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/85/Swagger-Suppression-Process for ARM Violation suppression.


More cleanup for QDA

@karlnapf My try in cleaning up QDA. Related to #3218
I would appreciate it if you can guide me through this.

This patch is a nop, closing it.
Please only send patches that compile, and please dont send nop patches, they just eat our time


Mod huber loss

Added Modified Huber Loss objective "binary:huber" for binary classification that is similar to distribution = "huberized" from gbm.
http://en.wikipedia.org/wiki/Huber_loss#Variant_for_classification
http://cran.r-project.org/web/packages/gbm/gbm.pdf#page=7
https://github.com/harrysouthworth/gbm/blob/master/src/huberized.cpp
I saw comparable performance to "binary:logistic" in datasets that I've tried,

Thanks for the PR. One comment:
The second order gradient presented at x > 1 and x <-1 was correct. But I am not completely sure this could be a safe choice (returning eps in second order gradient) for all the cases
Let us consider a safer alternative, for position x, let us find h(x) such that
g(z)  = f(x) + f ' (x) (z-x) + 0.5 h(x) (z-x)^2  >= f(x)

where f(x) is our loss function. A safe choice will be always set h(x) = 2. Some derivation may be needed to get a smaller h(x) such that it is tight.

I've just looked through your slides again, and I kind of see why having a lot of opportunities for very small 2nd order gradient could be dangerous. Maybe it's late, but I'm not quite catching why would you want the f ' (x) (z-x) + 0.5 h(x) (z-x)^2 >= 0 constraint? Or why there would be such freedom possible in 2nd order Tailor expansion?
I'm sorry for the slowness - am not having enough time in days recently...

The reason was not explained in the slide, but here is why:
In general, we do not necessarily need a approximation of the original loss function. It will suffice to optimize an upper bound of the original loss function that has the same value at the current point(delta=0).
The idea is kind like EM or majorization maximization algorithm. Optimizing the upper bound every time will leads to optimization of original loss function. For well behaved function, usually the 2nd order Taylor was a nice tight approximate upper bound. But in cases like huber, you can find that the Taylor(which was a line) will go below the original loss when we do not constrain the movement, this is why I think we need a more conservative upper bound(or constrain the delta of the move)
Note that Taylor expansion was usually accurate when delta is small

Closing this now since the core code will be moving to new codebase https://github.com/dmlc/xgboost/tree/brick
There is also a plugin system that can plug quick experimental features into https://github.com/dmlc/xgboost/tree/brick/plugin

I might want to revisit this at some point... I've tried some MM-inspired ideas, but the convergence with flat gradient just did not look great. That seems to relate to #691


Expiration Notifications

Send notifications when shares (or space memberships) expire

:boom: Acceptance test Core-API-Tests-ocis-storage-8 failed. Further test are cancelled...


Cleanup drawer element when gPlacesAutocomplete $destroy occurs

Inside the gPlacesAutocomplete init code, the drawer element is appended to body: https://github.com/kuhnza/angular-google-places-autocomplete/blob/master/src/autocomplete.js#L88
However, upon $destroy of the gPlacesAutocomplete element, there should be cleanup logic to remove that element from <body> via element.remove().
The code should be updated as such:
function initAutocompleteDrawer() {
	// existing code...

	scope.$on('$destroy', function() {
		$drawer.remove();
	});
}

I'm finding this works but only part of the time. After restarting the server and doing a hard refresh in the browser the drawer is removed, otherwise it persists.

scope needs to be $scope then +1 confirm this fixes #42


Is it a bug when using the wrong PSK with cipher suite TLS_ECDHE_PSK_WITH_CHACHA20_POLY1305_SHA256 without getting any error?

I am using DTLS with cipher suite TLS_ECDHE_PSK_WITH_CHACHA20_POLY1305_SHA256. But when I use the wrong PSK on the client-side, the SSL_connect will wait until timeout. Can anyone help to answer is it a bug or not?

I am using openssl 1.1.1i

If you do the equivalent thing with TLS, using an incorrect PSK is only noticed when we the server attempts to decrypt a record and gets a bad result. This is reported back to the client via a "bad record mac" alert:
Server:
$ openssl s_server -nocert -psk 123456 -tls1_2

Client:
$ openssl s_client -psk 123457
CONNECTED(00000003)
407704A3B17F0000:error:0A0003FC:SSL routines:ssl3_read_bytes:sslv3 alert bad record mac:ssl/record/rec_layer_s3.c:1584:SSL alert number 20

For DTLS the RFC specifies different behaviour when a bad record mac error occurs:
From RFC 6347 section <IP_ADDRESS>:
In general, DTLS implementations SHOULD silently discard records with
   bad MACs or that are otherwise invalid.

So the OpenSSL server is operating according to the spec in simply discarding the record which fails to decrypt properly and not reporting an error back to the client. In this case the failing record is a handshake message, so the client will periodically try to retransmit it until the app decides to timeout.
While slightly unexpected this does seem to be in accordance with the RFC.


Boolean Column Default and Constraint

@sebastienros For info, nothing urgent for me.
Using SQL Server >= 2019.

When doing .AddColumn<bool>("Foo", c => c.WithDefault(@default: false) on an existing index table, the column is added but all values are equal to NULL. Then if we map to a bool we can't use this field in a query as it is neither true nor false. Yes the solution is to update all related content items or manually update all the values through an SQL update query under SQL Studio. Another solution is to map to a bool?, but then the query predicate becomes more complex e.g. index => index.Foo != null && index.Foo == true.

When we use c.WithDefault() as above it adds a constraint named DF_something, the result is that if we then want to drop the column in a migration step it fails. We need to remove this constraint before which can't be done by code, only manually under SQL Studio.
Note: I noticed the same with a non clustered index IDX_something, it needs to be removed first if it includes the column we want to drop, but here that's okay as it can be done by code. Anyway we have to remove a non clustered index and then rebuilt it if we want to change the columns it includes.

To fix the 2nd issue, for a bool column we don't use anymore .WithDefault() as anyway it is not so useful because of the 1st issue. Would be good to be able to add a bool column with a default value used to populate this new column, and then to be able to drop this column without having to first drop manually the related constraint (or make it possible by code).

Okay, just found that to make .WithDefault working, at least for a bool, we have to also use NotNull.
.AddColumn<bool>("Bool1", column => column.NotNull().WithDefault(@default: false))

So that's okay for one of the 2 above issues.
But then, if we want to remove the column in a future migration step, we still have to remove manually the DF_xxx constraint that has been added because of the usage of .WithDefaul().
Note: Not a so big issue as normally when we add a column in a migration step this is not to remove it later on. The other issue, about populating a new column with a default value, was more annoying.


Create flow to grant and remove special roles/access to users

See info here: https://github.com/CDLUC3/dmsp_frontend_prototype/issues/21

Includes the same flow for removing the roles as well
We need an API endpoint to the user table for both these actions so that the user on the frontend that changes the role makes the change happen


[20.09] radare2: add patch for CVE-2021-32613

Motivation for this change
https://nvd.nist.gov/vuln/detail/CVE-2021-32613
Notionally a backport of #125066
In the end the adjustments needed were mostly removal of irrelevant hunks in the patch. Included 5e16e2d1c9fe245e4c17005d779fde91ec0b9c05's fix while I was at it.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125233 at 1c2b2893 run on x86_64-linux 1

5 packages built successfully:

python27Packages.r2pipe
python37Packages.r2pipe
python38Packages.r2pipe
radare2 (r2-for-cutter)
radare2-cutter


ReadTemp() function fails when using analogRead()

The temperature measurements from the ADC goes totally wrong when using the analogRead() function.

the issue is related to voltage reference configuration. ReadTemp requires internal voltage reference (1.1V) and thus register ADMUX is set acordingly in ReadTempinC. analogRead uses voltage reference set by the analogReference call and defaults to power supply voltage if analogReference is not called. This may lead to constant voltage reference switching and causes inacurate readings. From the datasheet (section 28.5.2 ADC Voltage Reference):
"The first ADC conversion result after switching reference voltage source may be inaccurate, and the user
is advised to discard this result."
So to solve the issue:

either call analogReference(INTERNAL)
or call ReadTemp/analogRead twice with some delay (10ms seems to be sufficient) and discard first result


Add a validate_schemas hook to clean up downstream validation code

rerun tests

rerun tests

@karlhigley , please link this to an initiative


Issue with arrow R package but only when running inside ryp

Hi
I'm trying to run ryp in an existing conda environment.
 However if I do it using ryp I get an error:
ryp.r("library(arrow)")

Traceback (most recent call last):
File "", line 1, in
File "/home/user/.conda/envs/global/lib/python3.12/site-packages/ryp.py", line 1314, in r
raise RuntimeError(error_message)
RuntimeError: package or namespace load failed for ‘arrow’ in dyn.load(file, DLLpath = DLLpath, ...):
unable to load shared object '/home/user/R/x86_64-pc-linux-gnu-library/4.3/arrow/libs/arrow.so':
/lib/x86_64-linux-gnu/libp11-kit.so.0: undefined symbol: ffi_type_pointer, version LIBFFI_BASE_7.0

I'm using

Python 3.12.5 | packaged by Anaconda, Inc. | (main, Sep 12 2024, 18:27:27) [GCC 11.2.0] on linux

Have you seen this before? I don't understand why it only happens when I load it inside ryp.

$ R
R version 4.3.3 (2024-02-29) -- "Angel Food Cake"
Copyright (C) 2024 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)
R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.
Natural language support but running in an English locale
R is a collaborative project with many contributors
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.
Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.
[1] "Loading .Rprofile"

library(arrow)

Attaching package: ‘arrow’
The following object is masked from ‘package:utils’:
timestamp

Thanks for your help raising these issues! This works fine on my end on Linux, Windows and Mac, so it seems configuration-specific.
It looks like the root cause is a misconfiguration in conda, which can be fixed by downgrading your libffi with conda install libffi==3.3.

thanks I compare a working conda enviroment with this one and the solution was to do:
conda install libffi==3.4.2 cffi==1.17.1 -c conda-forge


Problem when promisify run

Currently the callback argument for run only accept one parameter, namely err, and lastID and changes can be get from this inside the callback. When promisify using tools like bluebird, things like lastID are lost. It would be nice to have those passed to the callback as the second argument, and this is also a nodejs convention I believe.

@rnewman Thanks.  For others coming in late the link in the above comment is dead and the capability is now available as promise-sqlite.

We'd happily review a PR that implements lastID in the results object, but for now, you can use a library like pify that accepts a multiArgs option to successfully pass multiple arguments into a Promise object.


Feature/support 5.6

Updated to the package to work with Laravel 5.6. This will break compatibility with 5.5.

Tests are failing because L 5.6 needs 7.1+; this has to be modified inside the .circleci/config.yml file.

Relates and closes #383 , #384

Is it also possible to run the tests on L5.5? I have no experience with maintaining several laravel versions..

There are several ways to maintain several versions.

Do not support older versions, this is something @spatie is very good at 😁 I don't want to do this because creating an implementation (saas) can take quite a long time, it would feel horrible to have finished your app only to find this package is no longer patching issues.
Maintain and work on both versions inside the same branch. This requires conditionals and other "hacks" which I seriously have a distaste for.
Maintain and work on both versions using different versions, this is what I did before branches 3.x, 4.x and 5.x for laravel 5.3, 5.4 and 5.5. Getting new stuff in is a nightmare in this scenario.
Work on the latest version and patch the older version. When a serious issue is found we can check out a tag, patch and tag. All new functionality and overhauls are done on the master (or primary branch). This will by default make all existing implementation out of date, I'm not too happy about this either.

Multi version support in all scenarios require a lot more work and thus time. With the current patron donations I still cannot dedicate enough time to work on tenancy at a steady pace. I wish this was differently though.
My current approach has been to support only the LTS version (hence dropping branches 3.x and 4.x from the scope completely, I even deleted all tagged versions).

@Thijmen is going to take care of adding 5.6 support with backward compatibility. This will be part of the upcoming 5.1.0 release.

@Maveriks, Josh thank you for doing the necessary work to get this in. Good job, much appreciated!


Update bot_follow.py - Stop trying to follow users if out of follow

I'm trying to do a quick stop in the follow users cycle when out of follow expecially when the bot is checking a huge list of users.
It works for me and I hope can be usefull for all ;)

Codecov Report

Merging #680 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #680      +/-   ##
==========================================
- Coverage   37.71%   37.66%   -0.06%
==========================================
  Files          26       26
  Lines        2238     2241       +3
==========================================
  Hits          844      844
- Misses       1394     1397       +3

Impacted Files
Coverage Δ

instabot/bot/bot_follow.py
7.05% <0%> (-0.26%)
:arrow_down:

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 45bb011...3d0f3d7. Read the comment docs.


Use vm.runInThisContext instead of eval

Fixes #17
While this gets rid of eval in node and in "mixed" environments like node-webkit or Atom Shell apps, it still resolves to eval when using browserify.
For a solution that is compatible with CSP policies we'd need actual parsing of the literals (wish coffee-script would do it).
See:

https://github.com/substack/vm-browserify/blob/bfd7c5f59edec856dc7efe0b77a4f6b2fa20f226/index.js#L105

:shipit:

v1.0.4

:green_heart:

One way of achieving that without writing a lot of code is to convert the ' string case to the " case using the known-safe JSON.parse. It's a fairly easy replace of \' to \", taking care to not trip up backslashes that are part of a \\ token.
Unless I'm misthinking something.

@johan I wouldn't want to bet that all escaping rules would be preserved with a solution like that. Falling back to JSON.parse is already a hack for double-quoted strings (since it breaks "\," which is now a parse error but resolved to , before). But I'd be happy to accept a PR that goes down that road.

I also missed that " of course needs to become \", but I might try a PR for this. Maybe even escaping \, to resolve the right result after JSON.parse. :-)

It's a bit too late in the evening for a PR, but I think this sketch might be good, both for ' and " strings:
quotes = /\\['"\\,]|"/g
quoted = {
  "'":  "'"
  '"':  '"'
  '\\': '\\'
  ',':  '\\,'
}

parseStringLiteral = (literal) ->
  json = literal.replace quotes, (ch) ->
    '\\' + quoted[ch.charAt ch.length-1]
  JSON.parse '"' + json + '"'

@johan Well, JSON.parse treats unnecessary backslashes in strings as syntax errors. So you're solution will give:
parseStringLiteral("won't") // SyntaxError: Unexpected token '
parseStringLiteral("say \"what\"") = "say \"what\""
parseStringLiteral("lines end with \\n") = "lines end with \n"
parseStringLiteral("say \"what\"") = "say \"what\""
parseStringLiteral("escape using \\") // SyntaxError: Unexpected end of input
parseStringLiteral("col\u000bcol") // SyntaxError: Unexpected token

Code used:
strings = [
  "won't"
  "say \"what\""
  "lines end with \\n"
  "say \u0022what\u0022"
  "escape using \\"
  "col\u000Bcol"
]

strings.forEach (str) ->
  try
    parsed = parseStringLiteral str
    console.log 'parseStringLiteral(%j) = %j', str, parsed
  catch err
    console.log 'parseStringLiteral(%j) // %s: %s', str, err.name, err.message

Your array doesn't actually have legal string tokens in it after compilation; ITYM something more like
strings = [
  '"won\'t"'
  '"say \\"what\\""'
  '"lines end with \\n"'
  '"say \\u0022what\\u0022"'
  '"escape using \\\\"'
  '"col\\u000Bcol"'
]

...but the examples are still good proof my quick hack needs additional work before it can do business. :-)

Actually, you add the wrapping quotes in your parsing code.* Anyhow, those are the basic "happy path" test cases I could think of. :)
[*] Quote: JSON.parse '"' + json + '"'

Ah, my hack was meant to read literal.slice(1, -1).replace to get rid of the wrap apostrophes/quotes, but thanks for the test data; these types of hacks are best done TDD style. And then perhaps subjected to a fuzzer that benchmarks it a zillion times against what eval would have done. :-)


Google for surveys instead of courseplus?

Kind of want to get rid of courseplus entirely haha. Student found multiple sites/links confusing

lol I know what you mean - want to do turning in homework somewhere else? or do you think that is still best on courseplus?

This is done!

I love when we just have to close one we forgot we already did :P


Add marketing_campaign_* fields to com.sendgrid schemas (closes #904)

@BenFradet Do you mean multiple commits within the same PR or separate PRs?
I'm also assuming that each commit will be closing a separate issue rather than have one for all?
Regarding the version, I think it should be at least 1-1-0 but not a simple addition. We're adding fields but we have additionalProperties: true in 1-0-0, which means that some of these properties might have been part of historical events but with a different format. Eg, something that is a string now might have been an integer. We cannot guarantee that all old events will pass validation against these new schemas.
I went for 2-0-0 rather than 1-1-0 because the Redshift DDLs have meanwhile also changed. 1-0-0 DDLs still have the LZO encoding while the new DDLs have ZSTD.

yes I meant mutlitple commits in the same PR.
@chuwy can help us regarding your other qs

Hey @dilyand, I agree with 2-0-0 approach. The main disadvantage is a new table, but it is a lot safer from my POV. Redshift's compression type though is irrelevant here as it doesn't break anything and we can even patch such DDLs.

Awesome that you added the descriptions of the fields @dilyand !

superseded by #942


Undefined function 'loadHeatMap'

Hi, when I run ILSVRC_evaluate_bbox.m to evaluate the bbox, I get the error as follow:
"Undefined function 'loadHeatMap' for input arguments of type 'char'."
The function "loadHeatMap"  used in ILSVRC_evaluate_bbox.m (line 73) is undefined. Would you like to update this script? Thank you!

It is just a wraper like the follows. ILSVRC_evaluate_bbox only provides the skeleton, you need to fill your own subprocess.
function [heatMapSet, value_category, IDX_category] = loadHeatMap( curMatFile)
%LOADHEATMAP Summary of this function goes here
%   Detailed explanation goes here
load(curMatFile);
heatMapSet = 0;
end


pip3 install

mac 10.15.7
lishuang@newuser derrick % sudo pip3 install python-derrick
Requirement already satisfied: python-derrick in /Library/Python/3.7/site-packages (0.1.2)
Requirement already satisfied: whaaaaat==0.5.2 in /Library/Python/3.7/site-packages (from python-derrick) (0.5.2)
Requirement already satisfied: pychalk==1.1.0 in /Library/Python/3.7/site-packages (from python-derrick) (1.1.0)
Requirement already satisfied: docopt==0.6.2 in /Library/Python/3.7/site-packages (from python-derrick) (0.6.2)
Requirement already satisfied: simplejson==3.13.2 in /Library/Python/3.7/site-packages (from python-derrick) (3.13.2)
Requirement already satisfied: prompt-toolkit==1.0.15 in /Library/Python/3.7/site-packages (from python-derrick) (1.0.15)
Requirement already satisfied: Jinja2==2.10 in /Library/Python/3.7/site-packages (from python-derrick) (2.10)
Requirement already satisfied: chardet==3.0.4 in /Library/Python/3.7/site-packages (from python-derrick) (3.0.4)
Requirement already satisfied: MarkupSafe>=0.23 in /Library/Python/3.7/site-packages (from Jinja2==2.10->python-derrick) (1.1.1)
Requirement already satisfied: six>=1.9.0 in /Library/Python/3.7/site-packages (from prompt-toolkit==1.0.15->python-derrick) (1.15.0)
Requirement already satisfied: wcwidth in /Library/Python/3.7/site-packages (from prompt-toolkit==1.0.15->python-derrick) (0.2.5)
Requirement already satisfied: Pygments>=2.2.0 in /Library/Python/3.7/site-packages (from whaaaaat==0.5.2->python-derrick) (2.8.1)
Requirement already satisfied: regex>=2016.11.21 in /Library/Python/3.7/site-packages (from whaaaaat==0.5.2->python-derrick) (2021.3.17)
没有derrick命令
lishuang@newuser derrick % de
debinhex.pl debinhex5.28.pl  decryptdir       delv             dev_mkdb
debinhex5.18.pl  declare          defaults         desdp
咋用的

您好，最新版 derrick 没有任何 pip 安装路径。


SNK #117 - changed public ip for kube cluster #4

Coverage remained the same at 47.45% when pulling 534357985fdeb9b9b00f752d3a1e96bc813cc51a on amarcine:master into 49e80d71d8ee6a3eb87e963b158df05822f442c7 on contiv:master.


Safari loading two images

I know that the picture element isn't supported on Safari, but I noticed a wierd thing on my setup of lazysizes and picturefill.
I first discovered an console error on safari with this setup I have:

    <picture>

    <source data-srcset="image-280x187.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 480px)">
    <source data-srcset=
    "image-660x440.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 1024px)">
    <source data-srcset=
    "image-930x620.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 1400px)">
    <source data-srcset=
    "image-930x620.jpg 1x<EMAIL_ADDRESS>2x">

    <img alt="image with artdirection"
    class="lazyload" data-src=
    "image-930x620.jpg 1x<EMAIL_ADDRESS>2x"
    src=
    "data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw=="></picture>

This gives me a 404 in the console. It seems that it reads the whole data-src attribute as an address:
Failed to load resource<EMAIL_ADDRESS>the server responded with a status of 404 (Not Found)

I don't actually even know, if its possible to use the 1x and 2x in an image tag in this context (?!).
Anyhow, i removed the markup like that and replaced the data-src with just the address to the non-retina image in the img tag. That sort of solved the problem, but then I noticed that the browser loads both the fallback image and the image specific for the screen size. So I would see both images appear in the resources if viewing Safari at ipad size:

image-660x440.jpg
image-930x620.jpg

Same goes for mobile size.
Is this a case with the new Safari 9 or what am I doing wrong?

I don't actually even know, if its possible to use the 1x and 2x in an image tag in this context (?!).

You can't define descriptors inside of a src attribute, you must use the srcset/data-srcset. And you already did by defining these at the last source element.
At the end your new markup should be correct:

    <picture>

    <source data-srcset="image-280x187.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 480px)">
    <source data-srcset=
    "image-660x440.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 1024px)">
    <source data-srcset=
    "image-930x620.jpg 1x<EMAIL_ADDRESS>2x"
    media="(max-width: 1400px)">
    <source data-srcset=
    "image-930x620.jpg 1x<EMAIL_ADDRESS>2x">

    </picture>

I assume, that you have included lazysizes before picturefill. It might happen that at the time where lazysizes transforms the image picturefill isn't loaded yet and lazysizes goes into fallback mode (loads the data-src in non supporting browsers. Then later picturefill kicks in and loads the normal responsive image.
You have two options to fix this:

Move picturefill higher so that it can be used at once with lazysizes
Define a lazysizes polyfill

window.lazySizesConfig = window.lazySizesConfig || {};

window.lazySizesConfig.pf = function(options){
    if(window.picturefill){
        window.picturefill(options);
    }
};

A small tip: In my eyes the 1x/2x pattern is dead. And can be always substituted with the compressive image pattern. Which means, that you only serve the 2x images but with a much higher compression. At the end the image size for the 2x will be lower than of the 1x image. You markup would then look like this:

    <picture>

    <source<EMAIL_ADDRESS>2x"
    media="(max-width: 480px)">
    <source data-srcset=
<EMAIL_ADDRESS>2x"
    media="(max-width: 1024px)">
    <source data-srcset=
<EMAIL_ADDRESS>2x"
    media="(max-width: 1400px)">
    <source data-srcset=
<EMAIL_ADDRESS>2x">

    </picture>

Thanks for a quick answer. Its true, picturefill was loaded after lazysizes, so I placed it before lazysizes, but Safari still loads 2 images after this.
By compressive image pattern you mean that the 2x image should have extra compression by reducing the quality?

..., but Safari still loads 2 images after this.

Then I need a testcase or link to your page/prototype, because I can't reproduce this. If you are using the async attribute on picturefill, it might be still loaded after lazysizes.

By compressive image pattern you mean that the 2x image should have extra compression by reducing the quality?

Yes, you the perceived quality is the same, but you can compress it really high. Here are some informations:

https://www.filamentgroup.com/lab/compressive-images.html
http://imageoptimization.info/Compressive

Here's my page: http://urlgone.com/0b968f/

I haven't looked deeper into this. Because you have an old version of lazysizes. As I can recall you could fix this by changing your markup to this:
If you update to version 1.3.1 everything should be fine.

Brilliant! Seems to fix everything.


compiler: arcmwdt: Fix ccac version checking

Regexp suggested in #50173 PR is too strictly and
needs to be corrected. Previous regexp declines
engineering MWDT versions
(pattern ENG-2022.12-D1039-C39098348").
Signed-off-by: Nikolay Agishev<EMAIL_ADDRESS>

This one also need to be backported to the LTS I guess, what do you think @evgeniy-paltsev?

This one also need to be backported to the LTS I guess, what do you think @evgeniy-paltsev?

We don't have this check in LTS - it has been added in recent release :)


Resolve Mouse Event Deprecations

... by updating things
Additionally, because I need font awesome 5, I've merged in the font awesome 5 branch: https://github.com/ynnoj/ember-cli-notifications/pull/239
Thanks, @basz!

Hey @NullVoxPopuli 👋 I've taken back over this addon so  I'm trying to get through the backlog of issues and PRs
This PR is a bit big and it's hard to see where the actual change is 🙈 I'm working on the new version of the addon here https://github.com/mansona/ember-cli-notifications/pull/251#issuecomment-552737340 so maybe you could help get rid of these deprecations once it's released?

Awesome!
And I think I just upgraded stuff here ? :shrug:

so... it looks like it's more than just an upgrade 🤔 you seem to be using the {{on}} modifier and an action to get around the deprecations to do with mouse events 🤔
Maybe we can worry about adding that change when the other PR is merged 😂 Also do we know if there is a polyfill for the on modifier that works back to Ember 2.12?

Sounds good!
2.18 is as far back as this goes: https://github.com/buschtoens/ember-on-modifier/blob/master/README.md

Awesome 🎉that's a great find. I don't suppose you could create a PR for me sometime next week when the other PR is merged?

Yeah, We'll see if I remember!

Hey @NullVoxPopuli any chance you can update this PR? things have changed a lot now that https://github.com/mansona/ember-cli-notifications/pull/251 is merged 😂

I totally forgot about this. lol.
I'll take a look next time I have open source time!

@NullVoxPopuli sounds good! It might be a lot easier now that the addon has been properly updated 😂 and you could even just do a new PR 🤔
Should we close this one for now?

This was fixed in #263 🎉


Preactivation ResNets

Hello all
Hopefully this is the appropriate place to post this.
I've been working on this off and on since the Preactivation ResNet paper was published (https://arxiv.org/abs/1603.05027) and I finally think I flushed out the bugs in my code. Would this be something worth adding to the Lasagne Recipes? It would be easy to copy and paste my models into the existing ResNet example so that everything is kept consistent.
My code:
https://github.com/FlorianMuellerklein/Identity-Mapping-ResNet-Lasagne

Absolutely, this would be an excellent addition :) Perhaps it's best to keep it separate, since it's about a different paper? Of course we can make sure that they link to each other.

Oh great, I'm glad that I could contribute! What is the best way to proceed? I do my training loops a little differently from Lasagne docs and the other examples. Should things like that be made consistent?

I do my training loops a little differently from Lasagne docs and the other examples. Should things like that be made consistent?

No, that's totally fine!
Looking at your train_nn.py, though, the model to use is hard-coded. Can you change this to allow specifying the model on the command line? Something simple like:
import sys
variant = sys.args[1] if len(sys.args) > 1 else 'normal'
depth = int(sys.argv[2]) if len(sys.args) > 2 else 18
if variant == 'normal':
    from models import ResNet_FullPreActivation as ResNet
elif variant == 'bottleneck':
    from models import ResNet_BottleNeck_FullPreActivation as ResNet
elif variant == 'wide':
    from models import ResNet_FullPre_Wide as ResNet
else
    print ('Unsupported model %s' % variant)
output_layer = ResNet(X, depth)

Also there should be instructions about where to download and put the CIFAR-10 dataset (or an automatic download from within the script as we do for the MNIST example in Lasagne).

No problem, I'll get right on those changes.
Should we remove the wide ResNet though? It's from a different paper and in that paper they different preprocessing and L2. It wouldn't be a reproduction of the results, but probably a more fair comparison.

Different L2 should be easy to fix, different preprocessing maybe not -- but it should be enough to add a disclaimer (i.e., add a comment that says this is not reproducing the wide resnets paper, but just provided for comparison).

I updated my repo with a more informative README, system args added to train_nn.py and a comment in utils.py about where to put cifar-10.

Cool, looks nice and clean now! We can either just add this as a link to our wiki: https://github.com/Lasagne/Lasagne/wiki/3rd-party-extensions-and-code, or you can submit it as a subdirectory to Lasagne/Recipes/papers. In this case, I'd probably exclude the weight files, though, and ask @ebenolson to upload them instead so you can add the links to the README file. Also make sure to include your name in the README file and code if you submit it to Lasagne/Recipes.

I'm happy to do either one, if you all have a preference then I'll do that. I think that it would be cool to see it in Lasagne/Recipes/Papers. I also just updated everything with wide-ResNet results, my wide-ResNet did slightly better than the paper. Perhaps due to the different training and preprocessing.

I think that it would be cool to see it in Lasagne/Recipes/Papers.

Sure, if you prefer that, just do it! It will probably be more visible there than in the wiki, but people googling for "wide resnet lasagne" can already find your repository. Just decide for yourself if it's worth the extra effort.

Was merged two weeks later in #74. Closing.


can you use semver v1.0.0 ? ;)

Thx! :)

and if you want a use-case, check the current demo here:
https://metanov.github.io/firebase-element-extended/
cheers,
Pascal

Does this help

just to be consistent with the current semver scheme accross component, up to you! :)

I've done it anyway.  its the same state as the one I marked as v1.0.

thx again!


Can't figure out how to use ajv within Angular 6 app (typescript complains)

can you provide details on how to use in ng 6 app ?
Thanks

That's a general question of using any library with angular, out of scope here. Please post the question to StackOverflow.


Please update the document with new feature NTLM Password sync

Hi Team,
Please update the document with new feature NTLM Password sync as the screenshot is the old one. Also please mention when this feature was implemented.
Regards,
Vinay

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

ID: 1ca07bf8-d285-e7ae-aaf1-a9292bbdf75c
Version Independent ID: 7694cb10-8eb6-397d-2cbb-4e249556b88d
Content: Secure Azure AD Domain Services
Content Source: articles/active-directory-domain-services/secure-your-domain.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @Justinha
Microsoft Alias: justinha

@vinay230890 , Thanks for reaching out. We will investigate and update the thread accordingly.

@vinay230890 thanks for comment! The topic has been updated.

#please-close


FAQ - Update for India

The partner link should be in.getusppe.org/give

@gauravguptak Sorry, I thought this could be achieved through a link and text change but the devs need to create a partial large_donation_sites.handlebars file for India's FAQ to fix this

Was just fixing this @linderr


travis: fix gocc installation

It looks like Travis have set GO111MODULE=on, breaking our approach to versioning gocc for the code generation checks for graph/format/dot. This gets rid of the use of go get for this pull.
Please take a look.

This is a dogs breakfast! I knew modules would be a pain for us.


identifier is not an image, when push the iamge  in storage

https://github.com/containers/storage/issues/1202

https://github.com/labring/sealos/discussions/932

I think this issue should have more details

buildah pull nginx
run demo call storage, container, rootfs.


Master to Live

Docs Build status updates of commit 5c9cdb8:
:clock10: Full build: the average full build time is 13 min(s) 52 sec(s), this is based on the last 36 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 5c9cdb8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-eop/connect-to-exchange-online-protection-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mail-flow/Set-RemoteDomain.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-audit/New-MailboxAuditLogSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-audit/Search-MailboxAuditLog.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-content-search/New-MailboxSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-content-search/Set-MailboxSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Set-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

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


App crash when pick video is not screen recording

Bug report

Summary
When I select a video from my gallery, except for the screen shot video, my application always crashes

Environment info
Android

react-native info output:
# paste it here

Library version: x.x.x
Steps to reproduce

Select a video from my gallery
After short time my App crashed

Describe what you expected to happen:

I could select my video and do my stuffs

Reproducible sample code

i dont know how but right after i created this issue and it works LOL

@hanzoow how did u resolved this?

Facing same issue


Rename deployment command to stack

remove "stack describe"
rename "stack new" to "new"
rename "deployment apply" to "stack up"
rename "deployment delete" to "stack down"
rename "deployment list" to "stack list"
remove user aliases
remove unimplemented command "provider list"

fixes #137

Codecov Report

Merging #140 (50b9a66) into develop (3e9326f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #140   +/-   ##
========================================
  Coverage    32.74%   32.74%
========================================
  Files           43       43
  Lines         3329     3329
========================================
  Hits          1090     1090
  Misses        2136     2136
  Partials       103      103

Flag
Coverage Δ

unittests
32.74% <ø> (ø)

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

Continue to review full report at Codecov.

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


added lru_cache

python standard library already has this sort of functionality so i cleaned removed this class and replaced it with functools lru_cache

i think the default value of 128 is enough to start out with, we can update it later if needed.

@snafees usually the PR author is the one who merges into master, just an FYI.


[Bug]: fail to export result set for sql with * and nvl function

ODC version
4.2.4
OB version
4.2.x oboracle
What happened?
When export result set for sql with * and nvl function like select *, nvl(1,0) from whatever_tbl, it would fail.
This is because ODC would rewrite table to add select * from (xxx) fetch first x rows only statement on the sql.
It may cause  invalid SQL statement error.

What did you expect to happen?
do not rewrite user's sql
How can we reproduce it (as minimally and precisely as possible)?

Anything else we need to know?
No response

pass


Comments/Ratings

Hello,
It seems that I am not able to submit a review on the recipe page. Upon submission the text clears. However, I am able to submit a review from the 'My account' page. Is there something that I am missing?

Look's like I have a bug that if you don't submit a rating and a comment the form just get's cleared out without any error message.
Can you make sure that when you submit a comment that you include a rating and a comment and see if that works?

I just tried doing a rating and a comment, with the same result.

Do u mind posting the message that you are getting from the API when you try to summit a comment. AKA: opening your dev tools in the browser and posting the results from the API call.

My apologies for my dumb moment, I did not try on the superuser account. This attempt was from a regular user account. I will need to adjust permissions accordingly.

However, from the super user account, I am required to have both a review and comment.

Normal accounts should have access to adding a comment. I'll take a look into why normal users can't create comments

I did try to manually add permissions for the regular user. That was unsuccessful. I will await your investigation.

There was an issue with the way that permissions where setup that prevented non admin users for creating comments.
I have a fix for this on the latest master. Please check it out and let me know if u have any issues. See here for the update quide: https://github.com/open-eats/OpenEats/blob/master/docs/Updating_the_App.md

Works like a charm! Thanks a bunch for your help!