useFaketimers not behaving as a spy function after updating to 27.0.0

🐛 Bug Report
I updated my repository to use<EMAIL_ADDRESS>and started to get error because of jest.setTimeout not behaving as a spy function anymore.
To Reproduce
created a simple project to demonstrate the issue
app.ts
import {timeoutFn} from './app'
describe('test func', () => {
	test("test timeoutFn", () => {
		jest.useFakeTimers()
		timeoutFn()
		expect(setTimeout).toBeCalled()
	})
})

app.test.js
export const timeoutFn = () => {
	console.log('the first statement')
	global.setTimeout(() => {
		console.log('the second statement')
	}, 1000)
}

Expected behavior
earlier the test should have passed perfectly without any issue but now I get the error
 ● test func › test timeoutFn

    expect(received).toBeCalled()

    Matcher error: received value must be a mock or spy function

    Received has type:  function
    Received has value: [Function setTimeout]

      18 |                      jest.useFakeTimers()
      19 |                      timeoutFn()
    > 20 |                      expect(setTimeout).toBeCalled()
         |                                         ^
      21 |              })
      22 | })

envinfo

envinfo
  System:
    OS: macOS 10.15.7
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Binaries:
    Node: 14.15.4 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.10 - /usr/local/bin/npm
  npmPackages:
    jest: ^27.0.0 => 27.0.4

Edit:
just checked out release notes to see that with new jest release, it gives option to use jest.useFakeTimers('modern') and jest.useFakeTimers('legacy') even tho legacy works fine now but I face the same error mentioned above while using 'modern'

Thats intended
Check changelog for jest 27
You should spy on global.setTimeout and then assert the calls

It's also worth pointing out that a spy should be set up after jest.useFakeTimers, as otherwise your expectations may not pass as intended.
it('calls setInterval', () => {
  jest.useFakeTimers()
  setIntervalSpy = jest.spyOn(global, 'setInterval')

  foobar()

  // Works as intended
  expect(setIntervalSpy).toHaveBeenCalled()
})

vs
it('calls setInterval', () => {
  setIntervalSpy = jest.spyOn(global, 'setInterval')
  jest.useFakeTimers()

  foobar()

  // Expected number of calls: >= 1
  // Received number of calls:    0
  expect(setIntervalSpy).toHaveBeenCalled()
})

I don’t see where the changelog mentions this change?
I came here from the Jest timers documentation that still has the “old” style, just FYI.

Even with your code @edmundask i don't get all call like i should.
test Code :
export const timerGame = (callback: any) => {
  console.log('Ready....go!');
  setInterval(() => {
    console.log("Time's up -- stop!");
    callback && callback();
  }, 1000);
}

and the test
import { timerGame } from '../timer';
// test.ts
jest.useFakeTimers();

test('waits 1 second before ending the game', () => {
  const setIntervalSpy = jest.spyOn(global, 'setInterval')
  timerGame(() => {});
  expect(setIntervalSpy).toHaveBeenCalledTimes(0);
  jest.advanceTimersByTime(1000)
  expect(setIntervalSpy).toBeCalled();
  expect(setIntervalSpy).toHaveBeenLastCalledWith(expect.any(Function), 1000);
  expect(setIntervalSpy).toHaveBeenCalledTimes(1);
  jest.advanceTimersByTime(1000)
  jest.advanceTimersByTime(1000)
  expect(setIntervalSpy).toHaveBeenCalledTimes(3);
});

it's always call 1 time only

I don’t see where the changelog mentions this change?

[jest-config, jest-runtime] [BREAKING] Use "modern" implementation as default for fake timers (https://github.com/facebook/jest/pull/10874 & https://github.com/facebook/jest/pull/11197)

I came here from the Jest timers documentation that still has the old (now broken) code, just FYI.

#11974


IE11 error

IE11 shows absolutely nothing (https://brainhubeu.github.io/react-carousel/docs/examples/multipleItems) and shows following in the Error Console:
Unhandled promise rejection TypeError: Object doesn't support property or method 'endsWith'
I guess it's missing some polyfill(s)...
Thanks

@dviry
Thanks for reporting.
Maybe we'll fix that but if you want, you're more than welcome to open a PR.
It looks to be strange because:

react-carousel source code doesn't contain any endsWith
Object doesn't have endsWith method
so maybe one of the react-carousel dependencies uses an object in IE11 instead of a string.

Unfortunately, I don't have IE11 but I'm able to reproduce this error with https://ie10.ieonchrome.com/#https://brainhubeu.github.io/react-carousel/docs/examples/multipleItems

I managed to run IE11 in VirtualBox so there I see a blank page with the Object doesn't support property or method 'endsWith' error.

a video: https://recordit.co/OHTfv0dXNF
On the hand, the simulator (https://ie10.ieonchrome.com/#https://brainhubeu.github.io/react-carousel/docs/examples/multipleItems) shows menu and the Multiple items You can show more than one item per page, using slidesPerPage prop. text

React-carousel docs now work on ie browsers 🎉


added a method for HELP command

Tried to summarize the existing commands and show them for user convenience.

@appledora  are you there??

Hellooo, yes I am here, I was just a bit occupied this week. I will get back to you in 2-3 days. Does it work for you??

@mogiiee I think you should reassign this to someone else :| I am having a hard time to sit with it this week :cry:

@appledora  heyy thats okay...  this is actually a useful feature.. if you don't mind could i make some changes to this and merge it?

Sure!! No trouble on my end 😄


mapping OCONUS states and territories alongside CONUS data

I was asked by someone in Vizlab if anyone in my branch has any strategies for mapping OCONUS states and territories alongside CONUS data. Vizlab currently has an approach (shown below, source), but they wanted to ask around to see what others are doing before they proceed with sharing their current approach. Just wanted to see if anyone on HyTEST has any examples of other ways it's been done!

I do not have any experience on doing this outside of the ESRI environment.

I have other information but let's move this discussion to a side-channel, please.


simple-linked-list: unnecessary generic method

I believe that the example solution unnecessarily uses a generic method when entire class is generic. Specifically it seems to me that private <T> T[] newArray(Class<T> clazz, int size) can be replaced by private T[] newArray(Class<T> clazz, int size).

@rstockbridge will  you be submitting a PR to fix this? cc: @stkent

I'm a Java learner - am I correct about this? Feel free to make the PR
yourself if you want, otherwise I can do it.
On May 15, 2017 9:49 PM, "morrme"<EMAIL_ADDRESS>wrote:
@rstockbridge https://github.com/rstockbridge will you be submitting a PR
to fix this? cc: @stkent https://github.com/stkent
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xjava/issues/568#issuecomment-301652406, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIXZzRJfAibNU9DUBRz8924r8denZ37cks5r6QDEgaJpZM4NajMW
.

@stkent  is a team member. It looks correct to me but I don't have a say.  I'd like to encourage you to try to make the change yourself, if you're up for it. I am happy to provide support, and there are several others here who are as well!

Disclaimer: @rstockbridge and I know each other in real life, so we've often pre-discussed issues :)
In this case, I'm not 100% sure without going and re-reading some of the details of generics what the right action would be here. There's no need to think of track maintainers as different from any other contributors when it comes to language semantics - we're all fallible and we're all still learning. If either of you are interested in doing some digging in the language documentation and settling this one either way with some references, it'd be most welcome!

I've looked at this and I agree with @rstockbridge. I'm happy to make a PR for it unless you want to @rstockbridge? :)

Go ahead! :)
On Sat, May 20, 2017 at 10:29 AM, FridaTveit<EMAIL_ADDRESS>wrote:

I've looked at this and I agree with @rstockbridge
https://github.com/rstockbridge. I'm happy to make a PR for it unless
you want to @rstockbridge https://github.com/rstockbridge? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xjava/issues/568#issuecomment-302876617, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIXZzRAoZHAhsO63DspOMuZXYH3ANq9tks5r7vjdgaJpZM4NajMW
.


build: Bump react-markdown

Bump react-markdown
close #293
There is incompatibility between ref types. I suppose that we can just skip passing code ref to SyntaxHighlighter

Awesome. Thank you for your work.


find and replace not working properly like before

Issue Type: Bug
find and replace not working properly like before.
When I try to replace a CSS snippet with empty space, it doesn't work as it should.
VS Code version: Code 1.43.1 (fe22a9645b44368865c0ba92e2fb881ff1afce94, 2020-03-18T07:01:20.184Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4310U CPU @ 2.00GHz (4 x 2594)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.88GB (4.33GB free)

Process Argv
F:\dribbble\You Exec\Real work\Brave

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

prettier-vscode
esb
3.20.0

beautify
Hoo
1.5.0

debugger-for-chrome
msj
4.12.6

remove-comments
pli
1.2.2

LiveServer
rit
5.6.1

vscode-status-bar-format-toggle
tom
2.0.0

Find and replace all also not working.
Used to be able to use command + enter to replace all.
Now it opens and editor with all lines to files.
The issue isn't the page. The issue is that now the key binding is command + option + enter... but only works once and then doesn't work again unless I close the app or do a hard refresh on the app.

dup of https://github.com/microsoft/vscode/issues/92443


Saving in dev mode breaks

Looks like vfsSaveHook needs to be adapted. archive.write expects callback but is called as promise.
@oliver---- just curious, why did we switch back to callbacks instead of the promises there? Guess for easier testing?

@michael the problem with the Promises is that Chrome has removed halt-on-exception support for those, making debugging very hard. Thus, I want to wait with that until this is properly supported at least by Chrome.

I have changed the DAR API to use cb as last argument. So it should be easy to fix this.

Fixed via #724


Error: Cannot find module 'node:fs/promises'

Description:
Cypress won't start up due to the titular error. The only plugin added in the config is cypress-image-snapshot. Tried several NodeJS versions from 12.x.x to 20.x.x, but this seems to be irrelevant.
Versions:
OSX Ventura 13.4.1
Cypress v12.27.3
Cypress Image Snapshot v8.0.2
Full Stack Trace:
Error: Cannot find module 'node:fs/promises'
Require stack:
-<EMAIL_ADDRESS>- /Users/avarga/pro-insight-development/proweb/cypress.config.ts
- /Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js
- /Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/require_async_child.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:889:15)
    at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as _resolveFilename] (/Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/node_modules/@cspotcode/source-map-support/source-map-support.js:811:30)
    at Function.Module._load (internal/modules/cjs/loader.js:745:27)
    at Module.require (internal/modules/cjs/loader.js:961:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at Object. (/Users/avarga/pro-insight-development/proweb/node_modules/@simonsmith/cypress-image-snapshot/plugin.js:7:36)
    at Module._compile (internal/modules/cjs/loader.js:1072:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
    at Module.load (internal/modules/cjs/loader.js:937:32)
    at Function.Module._load (internal/modules/cjs/loader.js:778:12)
    at Module.require (internal/modules/cjs/loader.js:961:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at Object. (/Users/avarga/pro-insight-development/proweb/cypress.config.ts:2:1)
    at Module._compile (internal/modules/cjs/loader.js:1072:14)
    at Module.m._compile (/Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/node_modules/ts-node/dist/index.js:857:29)
    at Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
    at Object.require.extensions. [as .ts] (/Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/node_modules/ts-node/dist/index.js:859:16)
    at Module.load (internal/modules/cjs/loader.js:937:32)
    at Function.Module._load (internal/modules/cjs/loader.js:778:12)
    at Module.require (internal/modules/cjs/loader.js:961:19)
    at require (internal/modules/cjs/helpers.js:92:18)
    at loadFile (/Users/avarga/Library/Caches/Cypress/12.17.3/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js:89:14)

Could you provide a reproducible example? You can clone this repo and run the tests here as one example. I'm pretty sure this is tightly related to the Node version. I'm having no trouble running this on node 18 both in this repository and others.
If I can find a way to create the error then I can assist. What version of node produced the error above?

Thanks for getting back so quick. There is something really funky going on on my machine. Your repo throws Error: Cannot find module './dist/plugin' when I spin Cypress up. Also I got this while installing:
The engine "node" is incompatible with this module. Expected version ">=v14.21.3". Got "14.17.5"

This leads me to think that n did not in fact update the node bin that yarn uses. So this might be the root-cause of the issue.

My node version should by: v18.17.0 currently, by the way.

Yes! I had yarn installed from nix-pkgs and that binary used it's own outdated node version. So the issue was on my end. Thanks for addressing it though!
Cheers!

Excellent, glad you solved!


Move ctx.session to ctx.state.session to correctly type ctx.

When I use app.use(session.initMiddleware());, I'm having this error:

It looks like that SessionContext type of ctx in session.initMiddleware() seems to be the problem.
Plus, I need to add property session to type Context to avoid some ts errors.
I think move ctx.session to ctx.state.session wil solve the problem just fine.

or maybe add ctx.state.session as a refrence to ctx.state to implement the Context type in oak.

Weird, works for me. What version of Oak are you using? Do you have an example I can see?

Nevermind, I see now after converting my server test script from JS to TS. I'll look into it.

It's a TS type error. The code still functions well.

Type SessionContext isn't assignable to Context (oak requires ctx to be Context in their code) due to the extra session property.
I think the existing state property in Context is a good place to store session of a ctx.
It's better to just reference ctx.session in ctx.state.session to avoid a breaking change.

I made a PR to do the small change. #2

Thanks. I think it's a good idea to use the .state property, it works properly with TS now.
However, it doesn't seem this will fix the user-facing code when actually implemented as per #2.
In my local testing, I need export the SessionContext interface, right now it's not available to any other file:
export interface SessionContext extends Context

Import the SessionContext
import { SessionContext } from './src/Session.ts'

Then, the user has to attach the new context type to their router context somehow:
.get("/", session.initMiddleware(), async (context : SessionContext) => {})

So I think I'm just going to cut the cord on backward compatibility and do a new breaking version to 3.0. The library is still young, this is a change I want to make early.
Thanks!

For now, I've included your hotfix, and I updated the readme, so it should allow TS users to be able to use the library without breaking JS user's stuff. However, for the next breaking version, I'm going to remove any references to ctx.session.


feat: krew website updates

updated from develop
updated tailwind

@nratering I've added a space after ':' in the title, so that the 'semantics' check will pass ;-)

In general, it's advised NOT to mix dependencies updates with features/fixes, since it might introduced merge conflicts with (many) other PR's.


[OpenAI] Update README based on feedback

Packages impacted by this PR
open-ai
Issues associated with this PR
#26652
Describe the problem that is addressed by this PR
Update README based on feedback
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Adding initial spec for Reunion Picker API.

This is the high level specification for the File/Folder picker in Reunion. This initial draft includes the API of the proposed File/Folder picker.

Is this intending to be a wrap for the existing picker experiences, or will this be a new File/Folder Picking UI and UX which could be built in Xaml, and used by all lifecycles of WinUI and Reunion apps?

Is this intending to be a wrap for the existing picker experiences, or will this be a new File/Folder Picking UI and UX which could be built in Xaml, and used by all lifecycles of WinUI and Reunion apps?

This API will reuse some code from the existing experience. This will for the most part be a rewrite and hence will benefit from being build in Xaml.

Is this intending to be a wrap for the existing picker experiences, or will this be a new File/Folder Picking UI and UX which could be built in Xaml, and used by all lifecycles of WinUI and Reunion apps?

This API will reuse some code from the existing experience. This will for the most part be a rewrite and hence will benefit from being build in Xaml.

Great to hear!  File Picking dialogs are some of the most commonly seen UI (like the UAC prompt), and having them be modern and reflect the modern Shell and Fluent design will go a long way to making the whole OS feel more consistent.
I would like to see most things that are currently possible with the Win32 dialogs carried over, (with the relevant permissions dialogs).
The removal of outdated or undesirable extensibility and features should be considered, as Reunion and WinUI 3 are both opt ins, and so there is an expectation of code changes for the most part - but keeping existing code working should be a goal as much as possible.

One other thought: I’m torn about the namespace Microsoft.Reunion.Storage.Pickers. I can think of three different responses to this:

Keep the Microsoft.Reunion part and have all other namespaces in this repo start with that string. Clunky, but much more consistent.
Change the namespace to Microsoft.Storage.Pickers, by analogy to Windows.Storage.Pickers.
Change the namespace to Microsoft.FilePickers or Microsoft.StoragePickers.

My personal namespace design principles lean heavily towards option 3. If it was my decision, every Reunion API would have a root namespace something like Microsoft.MultiWordDescriptionWithoutDots, with sub-namespaces under that namespace if required. While I understand the reasoning behind the choice of Windows.Storage.Pickers for the original API, parroting that design exactly (while removing the use of the reserved Windows namespace) I feel does a disservice towards reorganizing these APIs. WinUI 3, with its Microsoft.UI.Xaml namespace, is IMO a very bad choice of namespace, but it is of course too late to change that project. I am raising this early so we don’t make the same mistake. Thanks!

Will this specification include the notion of WinUI/UWP apps acting as file providers, rendering UI within this new File Dialog?
The conversation here https://github.com/microsoft/microsoft-ui-xaml/issues/2854 brought up apps like Camera which can appear within the Win32 file dialog, but which do not work very well today.

Some preliminary feedback.
Also, I just wanted to make sure that I can get a filesystem path out of the StorageItem (at least in a full-trust process) so I can use it with whatever API I need. The UWP storage subsystem is infamously slow and difficult to use (cf. #8). Thanks!

That's not in the scope of the picker, and not always possible for all items. Some things like the libraries (Documents, Pictures, videos, etc...) don't actually have a file system location, they are basically shell virtual folders that can contain only links to other folders. So the path for them is always empty since they don't exist as a path at the file system level. Any item that is backed by a real file system location should have a non-empty string as the Path property. This proposal should not alter that.

I think another feature that can be added in reunion picker api is the choice to add custom options to picker dialog like Win32 apps Notepad and Microsoft Word do:

This can either be done in the same way PrintManager allows custom options or hosting custom xaml control in the picker dialog itself.

We were discussing implementing those custom items with a future File Picker Dialog.

https://github.com/microsoft/microsoft-ui-xaml/issues/2854

The namespace is not set in stone yet. We can revisit this closer to finalizing the spec.

In reply to: 652028635 [](ancestors = 652028635)

According to @Steve Maillet, this may not be possible always depending on the file locations.

In reply to: 440352690 [](ancestors = 440352690)

Currently this is not in scope. We can look to address this as part of future work.

In reply to: 659771383 [](ancestors = 659771383)

Can we consider the case for using the FileSavePicker to allow the user to specify a filename in a folder location without allowing the "overwrite existing file" prompt?  Some sort of boolean to control if selecting an existing files is allowed would be wonderful.

so this is opt-in for apps? :(

This is not going to replace the existing FilePicker api. Apps can choose to use this API instead of the existing FilePicker.

In reply to: 700438667 [](ancestors = 700438667)

Will this API apply to platforms like Xbox where file storage is handled differently?

I am not aware of Reunion effort targeting platforms other than Desktop. Maybe @jonwis can comment on this.

In reply to: 700884503 [](ancestors = 700884503)

This can be enabled via a property similar to Win32 IFileDialog interface.

In reply to: 698086497 [](ancestors = 698086497)

@ZenBird what happened to this PR?

Yea, what is the status of the updated XAML based File Pickers?


Documentation infrastructure: a mix of automatic docstring rendering and manual tutorials.

This version is available at: https://gflownet.readthedocs.io/en/make-docs/
PR changes:

Add __init__ files everywhere (otherwise the autoapi extension fails to document sub-packages and sub-modules
Add docs/ folder that contains the documentation software's instructions. Main components are:

conf.py contains the parameters for Sphinx and its extensions
index.rst is the home page's .rst structure

It references manual files
It will be temporarily changed by autoapi when it automatically discovers the contents & docstrings of the gflownet package

requirements-docs.txt that will be used by ReadTheDocs.io (and by you if you want to contribute to the docs and build locally)
Makefile contains instructions to build the docs, typically $ make html to render the docs locally to docs/_build/

To-do:

[x] Improve help on how to create good documentation
[x] Instructions for how to make the docs locally
[x] Mention how to create documentation for a branch
[x] Explore MyST in order to use extended Markdown instead of ReStructuredText
[ ] Add Github Webhook in the Integrations (@alexhernandezgarcia as owner of this repo you should do that here: https://readthedocs.org/dashboard/gflownet/integrations/)
[x] Integrate elements of https://github.com/executablebooks/sphinx-design
[ ] Update ogp_site_url to point to latest doc right before merge

Discussion

MyST enables Markdown instead of RST to write documentation

Unfortunately this cannot be setup easily to also parse docstrings
So for simplicity and consistency I'd say we stick to .rst
That being said, using the include statements in RST, we can still write Markdown if we really want it (see env/__init__.py for instance )
Thoughs?

There are conflicting docstring styles (Numpy vs Google)

We should probably only recommend one even though Sphinx can parse both.
I recommend Google style since it's generally thought to be more succinct and readable and it's used by torch

Question: All the *.rst files in ./docs/_templates/autoapi and specifically in directory python, are they conventional stuff or is there anything specific to this repo?

Question: All the *.rst files in ./docs/_templates/autoapi and specifically in directory python, are they conventional stuff or is there anything specific to this repo?

Short: They are conventional.
Long: AutoAPI uses those files as templates to create temporary documentation .rst files which are then parsed by Sphinx to render the HTML and deleted after the rendering is complete. The way for one to customize the templates yielding the temporary .rst files is by copying whichever they want from AutoAPI's source code into that repository. I put them all there so that you or anyone else could adjust the AutoAPI templates without having to go through the process of finding the source code, the files, finding where to put them in our repo and referencing them in conf.py. Note also that the index.rst template is the one that is displayed on the first API Reference page, and is already (very slightly) customized with a mention of gflownet.

We could create a github actions workflow for building the sphinx documentation. This way, we could make sure that future PRs don't break the docs. It's pretty easy to do : https://github.com/marketplace/actions/sphinx-build

Todo: fill-in the package and module docstrings, and write tutorials :)


Actions ignored during charging session

Hi
I would like to balance the electrical consumption by regulating the charging power while charging the car. However, I'm facing some issues:
When no car is connected to the charging station, the 'set current' action functions correctly. The charging station adopts the set current, and if I set a delay time, I can see (in my case) the entity "sensor.keba_p30_time_until_planned_current_is_set" counting down. Once the delay time elapses, "sensor.keba_p30_set_current" adopts the preset current, which seems to be working correctly.
However, as soon as a car starts charging, the previously set current reverts back to its maximum value. After this, it is no longer possible to reduce the current. If I attempt to use the set current action (which uses the underlying UDP command currtime), the delay timer counts down for a few seconds, but before the timer reaches 0s the current is reset already back to its maximum value and the timer stops counting. I don't think it's only this action that has an issue but something more in general, because the start and stop commands also seem to be either ignored or overridden during a charging session. If I try to stop the session, charging stops for approximately 2 seconds and then resumes.
This issue occurs with various car brands, so it doesn't seem to be caused by a specific car. And I already tried with 'failsafe mode' on and off, no difference.

Using custom_component version 3.0.3 in Home Assistant version 2024.11.1.
Charging station: Keba P30, firmware version 3.10.53, with SolarEdge branding.

Does anyone have tips on how I can better troubleshoot where the resetting/overwriting during charging comes from, or has anyone experienced similar problems and already found an explanation or solution for this?
Thank you in advance!

It works perfectly for me.

@VLWilliam If it working if you send the udp commands directly during a charging process? I would try to find out if it is caused by the charging station or by the home assistant integration

@VLWilliam Another idea: some charging stations have a swtich as input to authorize the charging process. If you say, even when you stop it, it restarts again, maybe the switch input is activated all the time?

Hello, same problem here. Until a few days ago, everything was working fine, but now during charging, I can't change the amperage or stop the charging process. Actually, the commands are successful, but after a few seconds, they are overridden by something. Here are the log lines:
2024-12-24 14:38:21,671 DEBUG - LogEvt: Event time: [2024-12-24 14:38:21,652] [24828377] timed ENA condition met: 13000
2024-12-24 14:38:25,282 DEBUG - LogEvt: Event time: [2024-12-24 14:38:25,263] [24828377] timed ENA condition met: 25000
I'm going crazy!

@VLWilliam Is the Keba charging station connected to a solar management system? If so, what brand do you have? I'm trying to understand if the issue might be coming from that. I tried sniffing the UDP commands between the charging station and Home Assistant, but I didn’t find anything unusual. I also tried turning off Home Assistant and sending the UDP commands manually, but it behaved the same way.

@VLWilliam The common points are the integration for reading data from the inverter and the brand of the inverter (even though I have a different model, but still SolarEdge). I tried turning off Home Assistant and sending UDP commands directly, and the behavior is the same, so I would rule out an issue with the integrations.
The things that could be done are:
•	Update the charging station. I see that mine is version .14, while the latest release is .18. Are you also on this version?
•	Try disconnecting the OCPP connection with SolarEdge.
Have you already tried either of these two attempts?

Ok, it seems that some external source is changing the state of your charging station. That is nothing I can fix in the integration of Home Assistant. I will close this issue now.


ionic3 webpack lazy loading

Sorry to disturb  @ionic-team。
My company use webpack to package our ionic app.
I know ionic-cli is a good tool, but we need to full control our project.
So, we can not use lazyloading in our ionic3 project.
I can not understand the principle of ionic3 lazyloading.
Can you give us a starter  of ionic3 webpack lazyloading?@ionic-team
Please!

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. As this seems like more of a support question I am going to close this issue.
I will ask around a bit to see if someone can get you started on some stuff to look at.
That said, I urge that you ask this question on our forum or on our slack channel.
Thanks for using Ionic!

@elon-hu,
Josh Moroney has written some good content on the topic:
https://www.joshmorony.com/what-does-lazy-loading-do-in-ionic/
This branch of a sample application also should give you some details:
https://github.com/ionic-team/ionic-conference-app/tree/lazy-load
We also have docs on the subject on the Ionic Framework site. I hope this helps.
Thanks,
Dan


Showing selected range label

When a range is selected ( e.g. "Today" ) , I would like to show "Today" instead of the daterange ( from - to ).
I expected 'apply.daterangepicker' to show wheter a custom range had been selected, but I can only access to the "model".
Is there any way to access to the range "label" information when applying ?
Thanks!

line 145 --> 151 in this file add picker
eventHandlers: {                 'apply.daterangepicker': function selected_date(e, picker){                   console.log(picker)                  } }
for (eventType in opts.eventHandlers) {   results.push(el.on(eventType, function(e,**picker**) {     var eventName;     eventName = e.type + '.' + e.namespace;     return $scope.$evalAsync(opts.eventHandlers[eventName],**`picker`**);   }));  }
You can now catch the selected range


Update ProphetModel to handle external timestamp

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes
Look #186.
Closing issues
Closes #186.

@d-a-bunin What do you think if Prophet would pick timestamp implicitly?
We can choose timestamp column by default. If timestamp column is of type int, we should choose the next timestamp typed column. If there are multiple timestamp typed columns - choose the first one.
And we should specify this implicit logic

Where do you think this could be useful? I can see it useful for working with pre-defined models on different datasets, e.g. in auto-ml there set of configs is pre-defined.

For now, it seems implicit, and I don't really like it. We could probably add such behavior in the future with some special parameter.
It still breaks if we have multiple timestamp columns.
We don't have such logic for DateFlagsTransform, TimeFlagsTransform and HolidayTransform.
I don't know how should we make auto-ml configs work in all those conditions.

We could make smth simple (like it works now) and easy-to-improve and then improve it according to our needs later.

It still breaks if we have multiple timestamp columns.

No, I we choose random one

Where do you think this could be useful?

It's just more simple to support.
You've changed a lot of code here. Do we really want to change so many places just for supporting corner case?

No, if we choose random one

It works unpredictable, I don't think it is a good idea. Moreover, you suggested selecting the first one in the first message.

It's just simpler to support.

I don't think so. We have to make the same code changes + add logic for automatic column detection, which selects timestamp_column automatically instead of manually. I think that if we are going to provide some automatic selection we should also provide ability of a manual selection.
Current solution could be extended into automatic in the future by adding, e.g. parameter timestamp_mode with two possible values: "auto", "manual" with default "manual" to save current behavior.


from google.protobuf import descriptor as _descriptor ImportError: No module named google.protobuf

Hey guys I'm working on a project using tensorflow on the Amazon Web Services microservices platform and I've run into a few problems. I installed tensorflow using pip into a virtualenv. Then I took the contents of /env/lib/python2.7/site-packages/ and zipped it along with my source code into a development package for AWS Lambda (env is the name of my virtualenv). This is the process for using python libraries for an AWS Lambda process and I haven't run into any problems with other libraries.
However when I attempt to use tensorflow it returns this:
Unable to import module 'classify': Traceback (most recent call last): File "/var/task/tensorflow/python/__init__.py", line 52, in  from tensorflow.core.framework.graph_pb2 import * File "/var/task/tensorflow/core/framework/graph_pb2.py", line 6, in  from google.protobuf import descriptor as _descriptor ImportError: No module named google.protobuf   Error importing tensorflow. Unless you are using bazel, you should not try to import tensorflow from its source directory; please exit the tensorflow source tree, and relaunch your python interpreter from there.
classify is the source script AWS Lambda calls when it is invoked and was zipped along with the contents of  /env/lib/python2.7/site-packages/  as I described.
These errors have been addressed a couple of other places but none of the solutions I found on those pages have worked for me, probably because none of them were using lambda. Is it possible that tensorflow just will not work with lambda because of the way lambda imports libraries? If so is there anyway import the tensorflow package from just a source folder that I upload as a zip?
Specific Steps to Reproduce:

virtualenv env
env/bin/pip install https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.9.0-cp27-none-linux_x86_64.whl
zip contents of /env/lib/python2.7/site-packages with .py file classify
upload to aws lambda function
invoke aws lambda function which calls classify and runs into an issue with "import tensorflow as tf"

It's occurred to me that this probably isn't something tensorflow is probably supporting given that most people aren't using tensorflow inside AWS containers but hopefully someone can help or at least give it their best shot even if they don't specifically have experience with AWS.
Thanks!!

Okay I have no idea what this does but it fixed this whole issue::
cd into site-packages
touch google/init.py
^bash

That is fascinating. I have no idea what the problem is, but it may be endemic to AWS lambda. I'll close this issue.

Stumbled over this on my travels. It's not Lambda, it's the protobuf module: https://github.com/google/protobuf/issues/1296

@JamesBelchamber Did you face the same problem after doing touch google/__init__.py  . I am not able to solve it. Trying to make pyrebase work on AWS Lambda. @aerointern16 . Any idea why this is not working in this case ?


Game Crashes while playing

Describe your bug here.
Recently downloaded v0.6.0, and when I ported all my mods and started playing, it froze, the music stopped playing, then I heard like 3 seconds of the song and then it crashed. So... help?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Not yet because I don't know how to lmao

Memory leak that just now got fixed in source

Memory leak that just now got fixed in source

And is the Null Object Reference also fixed?

I haven't seen that error myself yet so I don't know


FAQ to add

What types of resources can be secured with Entra for Customers.

SharePoint Online in provider tenant?
Azure resources in provider tenant

Can a trust between the "customer" tenant (B2C) and the primary provider tenant be established or are they totally separate?

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

ID: e407c207-e1d7-f901-e184-f0daa7bd3d02
Version Independent ID: a503616f-3fb2-4530-589b-9c421fd829c0
Content: Frequently asked questions - Microsoft Entra
Content Source: articles/active-directory/external-identities/customers/faq-customers.md
Service: active-directory
Sub-service: ciam
GitHub Login: @msmimart
Microsoft Alias: mimart

@danmyhre
Thanks for your feedback! We will investigate and update as appropriate.

@danmyhre
Entra for Customers is a decentralized identity solution that helps you safeguard your organization. It allows you to issue and verify credentials. Verifiers can use the service's free REST API to easily request and accept verifiable credentials in apps and services. In both cases, your Azure AD tenant needs to be configured to either issue your own verifiable credentials or verify the presentation of a user's verifiable credentials issued by a third party.
Regarding the types of resources that can be secured with Entra for Customers, it can secure Azure resources in the provider tenant and SharePoint Online in the provider tenant.
As for the trust between the "customer" tenant (B2C) and the primary provider tenant, they are separate tenants and there is no direct trust relationship between them. However, you can use Azure AD B2B collaboration to invite external users to access resources in your tenant, including resources secured with Entra for Customers.


Correct the file name for install_pmutools.sh

Summary
A small summary of the requirements (in one/two sentences).
Implementation Notes :hammer_and_pick:

Briefly outline the overall technical solution. If necessary, identify talking points where the reviewer's attention should be drawn to.

External Dependencies :four_leaf_clover:

Breaking API Changes :warning:

Simply specify none (N/A) if not applicable.

@niravnshah thank you! could you please sign the commit?
git commit -s --amend && git push


MultiPath TCP support

MultiPath TCP seems not include in netstack.  I don't know the how difficulty to implement it.
I see the mptcp implementation for Linux, and the code is not very long. Do you have any plan to include mptcp in netstack? Because many client machines does not support, use a  userspace tcp/ip stack is necessary

Apologies for the late response. We currently have no plans of implementing multi-path TCP as our current use cases do not require it. That said its certainly something we would be open to implementing if we had a clear use case for it in the short term.
Could you elaborate on your requirements for the same as that will help us prioritize working on it. That said we are always happy to accept contributions.


Feat: Implemented a memory adapter

Sounds good! Would be good to add some tests too

Will do that in a follow up PR!


Implement validators set handling

This issue should clarify that this isn't about "real" validators but simply using the validatorsetupdates from abci to determine the current block producer comittee (still no full fledged consensus necessary)

Resolved by #286.


DEPRECATION: Upgrade ember-cli-inject-live-reload version to 1.10.0 or above

I get the message DEPRECATION: Upgrade ember-cli-inject-live-reload version to 1.10.0 or above when I use the option "liveReloadOptions": {...} or "liveReloadJsUrl": "...".
When I remove liveReloadOptions the message goes away.
$ ember v
ember-cli: 3.10.1
node: 12.4.0
os: win32 x64
$ npm ls ember-cli-inject-live-reload ember-source
[...]
+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>
I solved it by now by changing my apache config to use mod_proxy_wstunnel. But how can I use liveReloadOptions without a deprecation warning?

I was running into this, in addition to a 404 I was getting. Browsing the addon code I found that it was actually watching an undocumented property liveReloadBaseUrl here.
Setting this in .ember-cli solved both the deprecation message and the 404 for us.


[FLINK-22914][connector/kafka] Use FLIP-27 KafkaSource in table connector

What is the purpose of the change
This pull request introduces FLIP-27 KafkaSource in the Kafka table connector, and migrate legacy table source using FlinkKafkaConsumer to another identifier "kafka-legacy"
Brief change log

Replace the runtime provider of KafkaDynamicTableSource to KafkaSource
Move legacy table source using FlinkKafkaConsumer to identifier "kafka-legacy" and "upsert-kafka-legacy"

Verifying this change
This change is already covered by existing tests for Kafka table connectors
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for the review @twalthr ! I just made some changes according to your comments and pushed again. Could you please have another round of review? Thanks a lot~

@flinkbot run azure

@flinkbot run azure

Thanks for the approval @lirui-apache !
Looks like the latest CI has passed except the pending E2E tests until now. I had other CI instances that has passed the E2E test:

My own Azure pipeline: https://dev.azure.com/renqs/Apache Flink/_build/results?buildId=65&view=results
A previous run which only diff in docs and a test case removal: https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=22244&view=results

Could you please help to merge the PR if you think it's good to go? Thanks!


Design tests

For now, there are some e2e tests in Python but we should consider testing as much as we can in Rust, especially when the rust CLI is ready: https://github.com/software-mansion/protostar/milestone/43

Is this still relevant with #2033? We could follow the same approach for testing cheatcodes as the tests in this PR.


better load-testing for checkboxes

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-5dc500c.modal.run


libc/tls: Change the default value of TLS_NELEM to zero

Summary
to ensure the default config as small as possible.
Impact
Testing
ci

@xiaoxiang781216 CI is failing with this error:
====================================================================================
Configuration/Tool: teensy-4.x/pikron-bb,CONFIG_ARM_TOOLCHAIN_GNU_EABI
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Disabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
  Enabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
  Building NuttX...
arm-none-eabi-ld: /github/workspace/sources/nuttx/staging/libc.a(exec_symtab.o):(.rodata.g_symtab+0xa5c): undefined reference to `pthread_getspecific'
arm-none-eabi-ld: /github/workspace/sources/nuttx/staging/libc.a(exec_symtab.o):(.rodata.g_symtab+0xa6c): undefined reference to `pthread_key_create'
arm-none-eabi-ld: /github/workspace/sources/nuttx/staging/libc.a(exec_symtab.o):(.rodata.g_symtab+0xa74): undefined reference to `pthread_key_delete'
arm-none-eabi-ld: /github/workspace/sources/nuttx/staging/libc.a(exec_symtab.o):(.rodata.g_symtab+0xb24): undefined reference to `pthread_setspecific'
make[1]: *** [Makefile:189: nuttx] Error 1
make: *** [tools/Unix.mk:527: nuttx] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 370: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize teensy-4.x/pikron-bb
HEAD detached at pull/9836/merge
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	boards/arm/imxrt/teensy-4.x/scripts/flash.ld.tmp

nothing added to commit but untracked files present (use "git add" to track)

@btashton is there some way to color in RED real errors to avoid those many "fake" errors?

@xiaoxiang781216 please include more info in the commit log message. Also why increasing from 32 to 255 is necessary?

@xiaoxiang781216 please include more info in the commit log message. Also why increasing from 32 to 255 is necessary?

tos field is uint8_t, so it doesn't make sense to limit the max value to 32.


[Feature request] Responsive style sheet

Hello guys! First of all, awesome lib @diego3g. I think it will be very helpful for us all!
One thing that can be a great addon is a ResponsiveStyleSheet (or ResponsiveSheet, if you prefer) that acts like the default rn's Stylesheet plus a parser to rem flags. It's something like the ScaledSheet from react-native-size-matters. It could be nice for the ones that often use the default Stylesheet instead of styled-components. So, for example, we can have something like:
import { ResponsiveStyleSheet } from 'react-native-size-matters';

const styles = ResponsiveStyleSheet.create({
  container: {
    width: '100rem',
    height: '200rem',
    padding: '2rem',
    margin: 5
  },
  row: {
    padding: '10rem',
    width: '50rem',
    height: '30rem'
  }
});

What do you think about it?

That would be great, but i think we should not replace StyleSheet from React Native but maybe we should make it work as a extension like:
import { StyleSheet } from 'react-native';
import { makeResponsive } from 'responsive-native';

const styles = makeResponsive(
  StyleSheet.create({
    container: {
      width: '100rem',
      height: '200rem',
      padding: '2rem',
      margin: 5
    },
    row: {
      padding: '10rem',
      width: '50rem',
      height: '30rem'
    }
  })
);

I don't know the better API for that for now, but I'll think a little bit more about it.

@diego3g nice man! Makes sense. Let me know if I can support you with ideas or the implementation part. I would be happy to help.

I'm working on this (dd80a35b17098f8e1a98e08efce1a2946ee3c5e3)

Hey guys, all right? How is the implementation of this feature going? It would be very important for the project I'm working on. Help us @diego3g <3


[FLINK-35708][Table SQL / API] Allow defining a schema in REPLACE TABLE AS (RTAS)

What is the purpose of the change
Allow defining new columns and/or overriding source column types in the CREATE clause on RTAS statements.
Syntax supported:
CREATE OR REPLACE TABLE  table_name
  [( <column_definition>[, ...n], [<watermark_definition>], [<table_constraint>])]
[DISTRIBUTED BY [HASH|RANGE|RANDOM](cols) [INTO n BUCKETS]]
AS SELECT query_expression;

Column definition supports adding new columns and/or defining columns already existing in the SELECT clause. Also, allows to pass PRIMARY KEY and WATERMARK columns.
Brief change log

Added support for column definition, watermarks, primary keys and distribution in the RTAS statement

Verifying this change
This change added tests and can be verified as follows:

Added unit tests to validation and converter classes
Added IT tests that verify data is written and read correctly

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs (will follow-up with another PR to update docs)

Closing this and replace this PR for https://github.com/apache/flink/pull/25247


Events Image from database

Uploading Screen_Recording_20221006_003454_GDSC IPSA.mp4…
Event details card UI redesigned and image from firebase implemented

@ismailbohra upload video link it not uploaded


Assetv2: Unprocessed asset meta defaults.

Objective
When processing assets, a meta file is produced per asset. By default, these meta files are redundant. If many assets are intended to be processed the same way, only one meta file should be needed per asset type.
This PR enables the creation of an "extension.defaults" file that will govern the processing behavior of all assets that do not have their own meta file.
An example where f.cool.ron will use cool.ron.defaults for processing because it has no meta file of its own.
-a---            6/9/2024  3:39 PM            174 a.cool.ron
-a---            6/9/2024  3:39 PM            433 a.cool.ron.meta
-a---            6/9/2024  3:39 PM            280 d.cool.ron
-a---            6/9/2024  4:51 PM            432 d.cool.ron.meta
-a---            6/9/2024  4:40 PM            114 f.cool.ron
-a---            6/9/2024  5:55 PM            464 cool.ron.defaults

Solution

Provide the ability to define an "extension.defaults" file which is used as a fallback if an unprocessed asset lacks meta.
Re-process assets if the meta defaults change (add/remove/modify) while a file watcher is active.

Questions

Do we want processed asset folders to have support for defaults also?

Testing

The asset_processing example has been modified to show fallback folder level meta being used by the f.cool.ron asset. This asset has no meta and so the folder's default cool.ron.defaults file is are used instead. This default processes the text in f.cool.ron to append "-modified_by_default_meta" to the contained string during asset processing.
Hot reloading can be tested by monkeying with the defaults (add / remove / modify) while running the example.

Changelog

Added support for .defaults files in the unprocessed asset folder. These files specify default metadata to be used per asset type during processing. For example, the asset alice.png will use png.defaults if alice.png.meta does not exist. Asset processing defaults support nesting, see the "asset_processing" example.

Migration Guide

...

If we're going to be settings defaults via file like this, I must mention being able to "override" them by placing a settings file deeper/more specifically in the hierarchy.
For example, the top level settings.default should be applied to everything in assets. However, the sprites dir has it's own settings which will override it. This enables you to perform tasks like "point filtering in the sprites directory but trilinear everywhere else".
assets/
├── settings.default
├── sprites/
│   ├── player.png
│   └── settings.default
└── logos/
    └── logo.png

I think it could get noisy having a settings file per extension. Would a map in a single settings file make more sense?

I think it could get noisy having a settings file per extension. Would a map in a single settings file make more sense?

I don't think so.

When we search up the parent hierarchy, we would have to search in all possible files for any matching defaults block.

It seems easier to just delete/add/remove the single file related to a particular type than have to search through existing files to find the one you want and remove the correct block.

We'd need to add support for a new type, which is the map of types to meta.

I did consider the map in a single file and actually started with that locally, but I think it is a lot more error prone. I'm thinking in terms of artist workflows for the most part, where dropping a "png.defaults" type file in a parent directory is pretty easy to figure out.

Great :) I like this design, the code quality is good, and there's new documentation addressing this. LGTM once CI is green and the typo I pointed out is fixed.

I need to investigate the other types of FileReader, which don't yet support this feature.

Was this closed this by accident?


Scrolling over code suggestions not working using macbook trackpad

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
macOS Ventura 13.4.1
Issue description
Scrolling suggestions list inside code editor scrolls the entire page instead if using macbook trackpad.
I tried it with regular mouse and find out that it works like expected so it is definetly trackpad-related issue.
Expected behaviour:

if mouse is over suggestions -> scroll suggestions
otherwise -> scroll the entire page

Attaching two videos. On the one I'm using mouse and everything works well, on another one I'm trying to do the same with trackpad.
https://github.com/godotengine/godot/assets/67281067/879534cb-36e7-4b26-a1d6-78cc3433380a
https://github.com/godotengine/godot/assets/67281067/4b7184a9-b301-4c6f-8cb6-6c1be36b16db
Even though it looks like a minor issue it is still super annoying and ruins experience using code editor, making suggestions almost useless if you are using macbook. Aiming that tiny little scrollbar as a fallback is one of the hardest things I've ever encountered.
Steps to reproduce

open godot project on macbook laptop
create gdscript
open it with built-in code editor
try typing anything to make suggestions appear
make sure suggestion list is long enough to scroll
hover mouse over it and try scrolling
see the entire page scrolling instead of suggestions

Minimal reproduction project
No setup required. Editor issue.

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/28149 and https://github.com/godotengine/godot/issues/49873.

Thanks for the report! Consolidating in #28149 and #49873.

Thanks! I also took a look at those issues and want to point out that this is not smoothness issue . In my case it just doesn’t  work, so hope you guys will fix it all at once at some point I guess.


Curie link to ALPS data not rendering

For reference, this is a Spring Data REST project.
The curie doc is linked to, in this case, http://localhost:8080/alps/customers.
Visiting that link directly leads to very valid ALPS data. However clicking the 'doc' link which opens the 'Inspector' view in the HAL browser shows no data at all.
Is this an issue or am I misusing the curie links?

have you exposed alps as a curie via a curieProvider ?


Updated api.md

There was a small error in the docs.

@RamonGebben Thanks for the PR! However, looking at it, I see that there is a mistake, but that is not it. It is supposed to be <NotAuthenticated> here, but the text is incorrect - the element will be shown only if the user is authenticated, but not as an administrator.

Ok sound good, I can recall that there were two blocks describing <NotAuthorized /> is the second one also correct?

@RamonGebben The basic example, without inGroup is 100% correct. The one you attempted to fix is almost correct, but the text should be something like: You are authenticated as something other than an administrator!


Docs cleanup

I fixed a handful of typos in this PR. Thanks for setting up some solid documentation!
You'll notice the diff is slightly hard to follow because of putting entire paragraphs on a single line. Personally, I prefer to use one sentence per line to make the diffs easy to follow, but if you look up "semantic linefeeds" you'll see plenty of other opinions out there on where to place the line breaks.

Thanks mark!


please update relay version on dockerhub!thanks

Describe the bug
if run wallconnect/monorepo 's newst version use older version of relay server on dockerhub that there is a error.

Describe the bug if run wallconnect/monorepo 's newst version use older version of relay server on dockerhub that there is a error.

hi, bro ~ solved it ?


Can not work with gridstack.js

Bug reports:
When i use the gridstack.js, the html2canvas can not work, when i remove gridstack,it's work fine! I don't know how to fix it. Please give me some advice,Thanks.
This is my screenshot with html:

This is my screenshot with html2canvas:

Specifications:

html2canvas version tested with: 1.0.0-alpha.4
Browser & version: Chrome( 62.0.3202.94)、Firefox(57.0.2)
Operating system: Win10(15063.726)

this is not gridstack's wrong.


Deepq train doesn't work

I tried to execute this code from custom_cartpole.py using stable-baselines and tf 1.14
import itertools
import argparse

import gym
import numpy as np
import tensorflow as tf

import stable_baselines.common.tf_util as tf_utils
from stable_baselines import logger, deepq
from stable_baselines.common.buffers import ReplayBuffer
from stable_baselines.deepq.policies import FeedForwardPolicy
from stable_baselines.common.schedules import LinearSchedule

class CustomPolicy(FeedForwardPolicy):
    def __init__(self, *args, **kwargs):
        super(CustomPolicy, self).__init__(*args, **kwargs,
                                           layers=[64],
                                           feature_extraction="mlp")

with tf.Graph().as_default():

  with tf_utils.make_session(8) as sess:
      # Create the environment
      env = gym.make("CartPole-v0")
      # Create all the functions necessary to train the model
      act, train, update_target, _ = deepq.build_train(
          q_func=CustomPolicy,
          ob_space=env.observation_space,
          ac_space=env.action_space,
          optimizer=tf.train.AdamOptimizer(learning_rate=5e-4),
          sess=sess
      )
      # Create the replay buffer
      replay_buffer = ReplayBuffer(50000)
      # Create the schedule for exploration starting from 1 (every action is random) down to
      # 0.02 (98% of actions are selected according to values predicted by the model).
      exploration = LinearSchedule(schedule_timesteps=10000, initial_p=1.0, final_p=0.02)

      # Initialize the parameters and copy them to the target network.
      tf_utils.initialize()
      update_target()

      episode_rewards = [0.0]
      obs = env.reset()
      for step in itertools.count():
          # Take action and update exploration to the newest value
          action = act(obs[None], update_eps=exploration.value(step))[0]
          new_obs, rew, done, _ = env.step(action)
          # Store transition in the replay buffer.
          replay_buffer.add(obs, action, rew, new_obs, float(done))
          obs = new_obs

          episode_rewards[-1] += rew
          if done:
              obs = env.reset()
              episode_rewards.append(0)

          if len(episode_rewards[-101:-1]) == 0:
              mean_100ep_reward = -np.inf
          else:
              mean_100ep_reward = round(float(np.mean(episode_rewards[-101:-1])), 1)

          is_solved = step > 100 and mean_100ep_reward >= 200

          # if args.no_render and step > args.max_timesteps:
          #     break

          if is_solved:
              if args.no_render:
                  break
              # Show off the result
              env.render()
          else:
              # Minimize the error in Bellman's equation on a batch sampled from replay buffer.
              if step > 1000:
                  obses_t, actions, rewards, obses_tp1, dones = replay_buffer.sample(32)
                  train(obses_t, actions, rewards, obses_tp1, dones, np.ones_like(rewards))
              # Update target network periodically.
              if step % 1000 == 0:
                  update_target()

          if done and len(episode_rewards) % 10 == 0:
              logger.record_tabular("steps", step)
              logger.record_tabular("episodes", len(episode_rewards))
              logger.record_tabular("mean episode reward", mean_100ep_reward)
              logger.record_tabular("% time spent exploring", int(100 * exploration.value(step)))
              logger.dump_tabular()

Describe the bug
I investigated the problem a bit.
Here most likely the problem is that we are trying to feed dones to network as input.
ValueError                                Traceback (most recent call last)
<ipython-input-5-97d09ee10f26> in <module>()
     77                   obses_t, actions, rewards, obses_tp1, dones = replay_buffer.sample(32)
     78                   print(obses_t.shape, actions.shape, rewards.shape, obses_tp1.shape, dones.shape)
---> 79                   train(obses_t, actions, rewards, obses_tp1, dones, np.ones_like(rewards))
     80               # Update target network periodically.
     81               if step % 1000 == 0:

2 frames
/usr/local/lib/python3.6/dist-packages/stable_baselines/common/tf_util.py in __call__(self, sess, *args, **kwargs)
    328         for inpt in self.givens:
    329             feed_dict[inpt] = feed_dict.get(inpt, self.givens[inpt])
--> 330         results = sess.run(self.outputs_update, feed_dict=feed_dict, **kwargs)[:-1]
    331         return results
    332

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    948     try:
    949       result = self._run(None, fetches, feed_dict, options_ptr,
--> 950                          run_metadata_ptr)
    951       if run_metadata:
    952         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1147                              'which has shape %r' %
   1148                              (np_val.shape, subfeed_t.name,
-> 1149                               str(subfeed_t.get_shape())))
   1150           if not self.graph.is_feedable(subfeed_t):
   1151             raise ValueError('Tensor %s may not be fed.' % subfeed_t)

ValueError: Cannot feed value of shape (32,) for Tensor 'deepq/double_q/input/Ob:0', which has shape '(?, 4)'

System Info
Describe the characteristic of your environment:

I use google collab
Tensorflow version 1.14

Good catch. The issues is in sampling (or storing) observations to the replay buffer. After sampling you should have obs of shape (32, 4), but for some reason it ends up with a vector of (32) elements.  Recent PR added "extend" version of the add, but that did not modify the original add, and on a quick glimpse I do not see where the error could be.

Well, I tried to investigate it a bit, obses_t, obses_tp1 have correct shape (32, 4)
The problem is in dones,  the train method tries to put dones as input to neural network.
A checker that by changing the size of dones, and exception text changed too.

Seems like the call signature has changed at some point. Looking at DQN code using that tf function, obses_tp1 are provided twice (for some reason?), while this code does not do that.

I think we should delete custom_cartpole it is an old code that does not follow the interface and the best practices from SB.

(for some reason?

I think it is a typo, I don't see any reason.

I will close this issue in favor of #812 as it has the fix in it.


Confusion on the batch_size in multi-gpu training

Thanks the authors for your amazing work!
I'm confused on the setting of batch_size during training and wanna ask for further clarification. Please see my question below.
In README, it says:

Now the batch size of a dataloader always equals to the number of GPUs, each element will be sent to a GPU. ...

So I understand the batch size during training is automatically set to be the number of GPUs. The line batch_size=len(args.gpus) in L171 of train.py seems also prove this.
But in L166 of train.py, I read batch_per_gpu=args.batch_size_per_gpu and the later has a default value of 2. What's the purpose of using batch_per_gpu if as said above, the batch size is automatically set to be #GPUs? Thank you!

if I understood it correctly the actual batchsize is the number of GPUs times the batch_per_gpu .
e.g. 8 GPUs, batch_per_gpu of 4 is an batch_size of 32.

@Michriko thanks for reply! Could the authors @hangzhaomit @Tete-Xiao @metalbubble please confirm? Thanks!

@Dav-Jay Yes, that is correct.


Add docker support

From @narate on April 5, 2018 15:43
Just add Dockerfile
Copied from original issue: ndidplatform/api/pull/16

From @dtinth on April 6, 2018 12:46
Thanks for the pull request, but I think the actual development of the platform is in this repository → https://github.com/ndidplatform/ndid

From @dtinth on April 6, 2018 12:48
I’m not sure though… @AnuchitO is this repository still being used? (I assume it is not because I also saw the API code in ndid project.) If not, should we archive this repository?

I see, 😎

Closed by #25.


Postgresql database

The traccar and traccar-web work with postgresql database?

How to create a DB in postgre  without sql ineqtion

If you have correct config, you just need to open web interface once and it will generate database schema.

If you are asking how to create database itself, you should be able to use any database client to do it (http://www.postgresql.org/docs/9.0/static/sql-createdatabase.html).

No, im create tables in db traccar on postgre9.3, launch traccar.bat. How a create a new device with ID in postgre database? in next time need actualise web interface for postgre. If this was done. Im have donate this project. In standart sheme Wharper+traccar+h2db im recive in port 5006 HEX packet but he dont show in a web.

I don't quite understand what the problem. Can you please give more details on what you are doing and what's going wrong.


Feature/mith cennz ela

Thank you for providing the SVGs for me! I made quite a few changes to the ela icon to clean up some of the vector points, and ensure its styling was consistent. I also included all the bitmap sizes and variants of these coins as part of https://github.com/cjdowner/cryptocurrency-icons/commit/4069b228f599851b4644f6adc0e1319a16e3ac47 so I'll close this, but thank you again. 😀


could be support all environment.

hi !
In this project , it's just support armeabi, could be support all environment? thanks a lot.

Currently, I only build for armeabi, of course, you can build it for other platform.

ye, i want to build for other platform, but it's always has some problem and i don't know how to fix it.  could you give me some script , thanks a lot.

What's the problem?


Running hyprswitch in special-workspace without --show-special-workspaces causes crash

Error: Failed to collect data with info Info { reverse: false, offset: 1, ignore_monitors: false, ignore_workspaces: false, sort_recent: false, filter_current_workspace: false, filter_current_monitor: false, filter_same_class: false, hide_special_workspaces: true }

Caused by:
    Active client not found in clients

fixed


[safety-rules] decouple waypoint update and epoch changes

Motivation
https://github.com/diem/diem/issues/9099
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land

:broken_heart: Test Failed - ci-test

/land


Add pep8 config, use pep8 in travis

also, fix E111-E113 errors:
E111 - indentation is not a multiple of four
E112 - expected an indented block
E113 - unexpected indentation

see https://github.com/sympy/sympy/pull/8538


STAT-21 ⁃ p2p sync pending transactions

(Replaces github issue #571)
Phil:
moving this over from #550 (github). This also relates to super-isssue #291 (github).
I believe I have a solution to the problem of effectively sending 1000s of pending transactions to a newly started producer node. I assume that a non-producing node has no need to cache pending transactions apart fromm possibly being able to forward transacitions on to other producer nodes.
The solution is it add a second queue on the connection object that would be filled with previously packed transactions. Then bundles of transactions are sent out at a lower priority than the notification and request messages.
The pending txn queue can be popluated with all cached packed transactions, or with all except those in a list, or only with those in the list.
Testing is proceeding.

➤ Joshua Lavin commented:
Phil:
This enhancement includes 2 extensions to the p2p protocol - I added a new message, "go_away" to convey that a compliant peer should not try to reconnect after an active connection closure. This allows for redundant remote connections in the node's config to facilitate quick reconnection after peer restart without worrying about duplication of messages.
Second, I extended the data passed with the notify and request messages, adding a "mode" for distinguishing between normal or catch-up modes, and an explicit indication that either the block or the transaction list is not supplied, rather than being supplied but empty. In catch-up mode, an empty id list in the request means send all pending blocks or transactions, whereas a nonempty list with a catchup-up messages indicates which block or transactions to not send, think of "send all except these"

➤ Phil Mesnier commented:
(from oct 20 gh #571) still in progress. Most recent push includes all the pieces necessary (IMO) to sort out initial sync once and for all. Still a little buggy in terms of determining which participant should do what, but I've got 10 hours of train time today to sort that all out.

➤ Phil Mesnier commented:
See PR #587

➤ Phil Mesnier commented:
[~dan.larimer] I have a remaining question on the block summaries + transaction ids. The issue is that the User Transactions in the block are "ProcessedTransactions" which includes output messages. Also GeneratedTransactions are included in the merkle calculation so they need to be included in the reassembled signed block.
So what should I be doing to reconstitute the processed transactions so that hash values and merkles match?

➤ Bezwada Satyapravin commented:
Please let us know how to QA this ticket.

➤ Phil Mesnier commented:
Marked this one done - Dan decided that for STAT we'll just go with sending full blocks making this issue redundant.

Sync'd the issue back to JIRA and closing.


Seeing Netty package version mismatch when using hoodie bundle jars

Cosmin reported an issue where he was seeing the below error:
java.lang.NoSuchMethodError: io.netty.buffer.PooledByteBufAllocator.metric()Lio/netty/buffer/PooledByteBufAllocatorMetric
with hoodie-spark-bundle.jar.
The bundle jars have netty classes included without getting shaded causing the error when used with Spark-2.3.1

netty is a transitive dependency coming from hadoop-hdfs and spark-core. Since both are provided packages, we need to exclude them from bundle jars

This is the actual start-up exception:
spark-worker                 |     at org.apache.spark.network.util.NettyMemoryMetrics.registerMetrics(NettyMemoryMetrics.java:80)
spark-worker                 |     at org.apache.spark.network.util.NettyMemoryMetrics.<init>(NettyMemoryMetrics.java:76)
spark-worker                 |     at org.apache.spark.network.client.TransportClientFactory.<init>(TransportClientFactory.java:109)
spark-worker                 |     at org.apache.spark.network.TransportContext.createClientFactory(TransportContext.java:99)
spark-worker                 |     at org.apache.spark.rpc.netty.NettyRpcEnv.<init>(NettyRpcEnv.scala:71)
spark-worker                 |     at org.apache.spark.rpc.netty.NettyRpcEnvFactory.create(NettyRpcEnv.scala:461)
spark-worker                 |     at org.apache.spark.rpc.RpcEnv$.create(RpcEnv.scala:57)
spark-worker                 |     at org.apache.spark.rpc.RpcEnv$.create(RpcEnv.scala:43)
spark-worker                 |     at org.apache.spark.deploy.worker.Worker$.startRpcEnvAndEndpoint(Worker.scala:785)
spark-worker                 |     at org.apache.spark.deploy.worker.Worker$.main(Worker.scala:755)
spark-worker                 |     at org.apache.spark.deploy.worker.Worker.main(Worker.scala)

Hoping this is fixed. Reopen if you still see the problem

I have reproduced the error with 0.4.5 - I managed to get around that by excluding netty and some other things from the spark-bundle ( which we are using )  .
   <artifactSet>
                <excludes>
...
                  <exclude>org.mortbay.jetty:*</exclude>
                  <exclude>io.netty:*</exclude>
                  <exclude>com.fasterxml.jackson.core:*</exclude>
                  <exclude>com.fasterxml.jackson.module:*</exclude>
...
                </excludes>
</artifactSet>


Javascript not included in Rails 7

I'm ahead of the game here - but in Rails 7 (at least with the setup used by JumpstartRails), the cocoon.js file doesn't get included.
Everything seems to work if I manually add it - but presumably the gem should be able to do that automatically.
thanks for a great project.

sorry - I got the wrong end of the stick on this.

How did you include it? @ConfusedVorlon

I import it manually


Revert "Add some more multivariate functionality"

Reverts kujaku11/mth5#228

There was an un-pushed commit for this PR that should be added, reverting, will tune up doc and then re-merge

I'd rather note revert, but just add the new stuff from my local


Add method to change the menu for HOC page

For pages created with zcl_abapgit_gui_page_hoc=>create, the menu is defined at the time of creation. It does not seem possible to update the menu afterward (or am I missing something @sbcgua?).
Add a method to update the menu - for example, to add items or switch checkmarks - and show the new menu after re-render. This will be essential when migrating more pages to the HOC model.

I have some custom pages where I need this but here's a similar example on our Diff page:

The menu options are handled by on_event and trigger an update of the menu:
https://github.com/abapGit/abapGit/blob/b03a70228f15979e7eac22e06518f9c413cf764d/src/ui/zcl_abapgit_gui_page_diff.clas.abap#L1332-L1337
Callback or an update_menu method on higher level would work.


[TestSupport] Unset the env variable that allows skipping tests

Since this effects the tests that use swift-test.

@swift-ci smoke test

@swift-ci test with toolchain

@swift-ci test linux


redux-logic + flow best practice?

Hi,
Currently I'm struggling with the use of Flow in combination with redux-logic because Flow does not understand that a given logic will only receive actions of a given type.
Given the following types:
export type Action =
          { type: 'SEARCH_LOCATION_SELECT', payload: string }
          | { type: 'GEOLOCATION_REQUEST' };

export type Dispatch = (action: Action) => any;

export type LogicContextT = {
  action: Action,
  getState: () => GlobalState
}

This is a problem for Flow:
const aroundMeSelectLogic = createLogic({
  type: 'SEARCH_LOCATION_SELECT',
  process({ action }: LogicContextT, dispatch: Dispatch): void {
    if (action.payload === 'AROUND_ME') {
      dispatch(geolocationRequest());
    }
  }
});

property payload. Property not found in object type

and this is NOT a problem for Flow:
const aroundMeSelectLogic = createLogic({
  type: '*',
  process({ action }: LogicContextT, dispatch: Dispatch): void {
    if (action.type === 'SEARCH_LOCATION_SELECT' && action.payload === 'AROUND_ME') {
      dispatch(geolocationRequest());
    }
  }
});

Does anyone have a suggestion on how to best handle this?

I believe I found a discussion of the same problem over in an issue on the redux repo, maybe there will be some ideas in there.
https://github.com/reactjs/redux/issues/290

It looks like this PR was the result of many discussions for the flow types for Redux. It has a union of actions, so maybe this provides some clues on how to proceed. I wish I understood Flow better to be able to provide some guidance, but I am not there yet.

Hi @jeffbski
Thanks for taking the time to reply.
Adding payload to all actions would not solve it since the payload might be different from action to action, it might solve the example I gave but then fail when I tried to access a property in a payload that was an object and not a string. Also, I might be listening for actions from 3rdparty libs which I don't control like the action shape (eg. react-router-redux).
I am not sure what code you are exactly pointing to, but indeed a union of actions + polymorphic types can solve the issue, although things get a bit more verbose.
This is not a problem for Flow and is the best approach I can come up with given my current knowledge of it:
export type SearchLocationSelectAction = { type: 'SEARCH_LOCATION_SELECT', payload: string };
export type GeolocationRequestAction = { type: 'GEOLOCATION_REQUEST' };

export type Action = SearchLocationSelectAction | GeolocationRequestAction;

export type Dispatch = (action: Action) => any;

export type LogicContextT<A> = {
  action: A,
  getState: () => GlobalState
}

const aroundMeSelectLogic = createLogic({
  type: 'SEARCH_LOCATION_SELECT',
  process({ action }: LogicContextT<SearchLocationSelectAction>, dispatch: Dispatch): void {
    if (action.payload === 'AROUND_ME') {
      dispatch(geolocationRequest());
    }
  }
});

@ivoanastacio Thanks for posting the polymorphic approach.
As I learn Flow better I'll post back any other ideas I come up with, but this seems to be the best way to go for the moment. I too wish it wasn't so verbose.


Refresh Grant returns RefreshToken with original Expiration

I'm working on configuring Spring Security OAuth JWTs using the versions included with Spring Cloud Camden SR6.
When I do refresh_grant requests, the response includes a new accessToken and refreshToken, however when I decode the refreshToken (JWT), I observe it to have the same expiration value as the original refreshToken submitted with the refresh_grant request itself.
How do I configure Spring Security OAuth to create a refreshToken with an extended expiration as part of the refresh_grant? I have already configured the ClientDetailService with a .refreshTokenValiditySeconds(1800).
Here is my current config class:
@Configuration
@EnableAuthorizationServer
public class MyConfig extends AuthorizationServerConfigurerAdapter {

	.... dependency injections omitted for brevity ....

  @Override
  public void configure(ClientDetailsServiceConfigurer clientDetailsServiceConfigurer) throws Exception {

    clientDetailsServiceConfigurer.inMemory()
        .withClient("client1")
        .secret("password")
        .scopes("scope1")
        .autoApprove(true)
        .authorizedGrantTypes("password", "refresh_token")
        .accessTokenValiditySeconds(60)
        .refreshTokenValiditySeconds(1800);

  }

  @Override
  public void configure(AuthorizationServerEndpointsConfigurer authorizationServerEndpointsConfigurer) throws Exception {

    TokenEnhancerChain tokenEnhancerChain = new TokenEnhancerChain();
    tokenEnhancerChain.setTokenEnhancers(Arrays.asList(jwtTokenEnhancer, jwtAccessTokenConverter));

    authorizationServerEndpointsConfigurer
        .authenticationManager(authenticationManager)
        .userDetailsService(userDetailsService)
        .tokenStore(new JwtTokenStore(jwtAccessTokenConverter))
        .tokenEnhancer(tokenEnhancerChain);

  }

Call 1
Using grant_type of password returns an accessToken and refreshToken. The refreshToken is decoded to this:
{
  "user_name": "user1",
  "scope": [ "scope1" ],
  "ati": "fe69f780-58f2-4787-8747-8bcc94aded87",
  "exp":<PHONE_NUMBER>,
  "authorities": [ "auth1" ],
  "jti": "ac604275-681e-4820-b47a-a2510560d2f0",
  "client_id": "client1"
}

Observe the exp claim is<PHONE_NUMBER>.
Call 2
Using grant_type of refresh_token, where I supply the refreshToken retrieved from the first call as part of the POST. This returns an accessToken and refreshToken. The new refreshToken is decoded to this:
{
  "user_name": "user1",
  "scope": [ "scope1" ],
  "ati": "c7bf8874-6dad-4d70-a17e-3058243361bf",
  "exp":<PHONE_NUMBER>,
  "authorities": [ "auth1" ],
  "jti": "ac604275-681e-4820-b47a-a2510560d2f0",
  "client_id": "client1"
}

Observe the ati claim changed, but the exp claim is still<PHONE_NUMBER>.
Call 3 (just to show it again)
Again using grant_type of refresh_token, where I supply the refreshToken retrieved from the second call. This returns an accessToken and refreshToken. The new refreshToken is decoded to this:
{
  "user_name": "user1",
  "scope": [ "scope1" ],
  "ati": "f4f3c732-e10b-4112-8c1d-6b3732d65000",
  "exp":<PHONE_NUMBER>,
  "authorities": [ "auth1" ],
  "jti": "ac604275-681e-4820-b47a-a2510560d2f0",
  "client_id": "client1"
}

Observe the ati claim is again changed, but the exp claim is still<PHONE_NUMBER>.
How do I configure Spring Security OAuth to produce extended refreshTokens on refresh_grant requests?

You should set .reuseRefreshTokens(false) in AuthorizationServerEndpointsConfigurer


Disallow black 22.10.0 because the initial macOS wheels had the wrong architecture

When black 22.10.0 was released, the initial macOS x86_64 wheels were actually amd64 wheels: https://github.com/psf/black/issues/3312
Apparently this has been corrected, but some of our GitHub agents still have the original, incorrect wheels in their Poetry caches.
Disallow black 22.10.0 to ensure that clients of ni-python-styleguide don't run into this problem.

Run actions/setup-python@v2
Version 3.6 was not found in the local cache
Error: Version 3.6 with arch x64 not found
The list of all available versions can be found here: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json

Run actions/setup-python@v2
Version 3.6 was not found in the local cache
Error: Version 3.6 with arch x64 not found
The list of all available versions can be found here: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json

‾_(ツ)_/‾, 3.6 has been eol for a while. I guess it's time to stop testing it here...

the initial macOS x86_64 wheels were actually amd64 wheels

Typo? I think it should be arm64 or aarch64.


Check for either ios-x86_64-simulator or ios-arm64_x86_64-simulator in module test

Prep for engine roll https://github.com/flutter/flutter/pull/85045 with recipe change https://flutter-review.googlesource.com/c/recipes/+/14780 that contains ios-arm64_x86_64-simulator instead of ios-x86_64-simulator
Passing: https://ci.chromium.org/p/flutter/builders/try/Mac build_ios_framework_module_test/7958?

build_ios_framework_module_test passed, merging ASAP to unblock the engine roll.

This test also successfully passed on the engine roll https://ci.chromium.org/p/flutter/builders/try/Mac build_ios_framework_module_test/7959?


Add support for rumble extension

Introduced by https://github.com/v1993/cemuhook-protocol/pull/7. This library should handle rumble command timeouts in addition to providing simple (yet optional) interface for server, as well as validate rumble requests for motor ID falling within bounds.

This extension doesn't seem to be really used by anything in the end and very basic support for it was implemented, so wontfix for the time being.


Please don't insert url_key attribute values if the url_key on a specific storeview is the same as in the default values

Hi!
This was tested on a Magento shop with 2 store views.
When you have a product with a url_key set to something on the default storeview but not overwriting it on a specific storeview scope, then after running this tool, it inserts the url_key value something on the 2 storeviews.
This makes it hard to update the url_key for a product, since you then need to manually go through all store views, and update all url_key's.
So if a url_key isn't filled in on a specific store view, it should just inherit it from the default values, and not save it separately in the database.
This might be related to https://github.com/olegkoval/magento2-regenerate_url_rewrites/issues/93
Thanks!

As explained in #pr78, this is default behavior. My PR was refused for this reason. However, I will submit soon a PR with the reverse option, that is not to regen URL_key.

Thanks for replying @werfu, I think you are talking about #93.
This ticket (92) is about not generating url keys on storeview level if it already exists on default level.
This is very related, but not exactly the same :)

Yeah, I mixed up. I replied in the other thread for clarity purpose.
About this one though, I'm not sure how, but we should check if the current storeview key is using the default or not. In some case, the generated url_key will not be the same as the attribute used for generating the key will be overridden.


Engineering Feedback FAQ

Initial draft of frequently asked questions (FAQ) for engineering feedback.
Related to #175

@nzthiago @paulbouwer @arsenvlad Would you please give this a review?

Can we add a link to this FAQ from the readme file?

@arsenvlad and @nzthiago Please have a look at the latest commit as it should address the majority of your earlier feedback.

@mcollier there's two notes from Vlad that you didn't change anything but I believe it's because you feel like they don't need any update, just want to confirm; if so you can resolve those conversations and I'll approve, everything else looks good to me!

@nzthiago Commented and resolved.  Please review.
@arsenvlad - FYI.


Is map's latest 1.15 ordered or unordered

Is map's latest 1.15 ordered or unordered

Unordered, as always.


Created new Get-PsakeScriptTasks

The function is now available along with it's xml documentation.

This relates to #153

@mabead thank you for your contribution!  I have merged this PR into the master branch here: https://github.com/psake/psake/commit/f472f9f88227d3095b20f3ef3f1b6d9ab37237f2  I did this so that I could remove the unnecessary merge commit that was in the Pull Request.
As a result of doing this, I need to manually close this PR, and not merge it directly.  The work that you have provided is available in the master branch though.  Thanks!

@gep13 Thanks Gary!


treeData has no property "children"

In OSF versions 0.119.0 and 0.120.28 I've been seeing an issue with storage services not being connected on the project page, including OSF storage. In the Javascript console it complains about "this.treeData.children" not existing in vendor.js
which is composed of the code from website/static/js/fangorn.js inside the function _loadTopLevelChildren(). It is called with this=tb.
If I add an if-statement to check whether the property "children" exists to allow the code to continue instead of crashing, I can verify that later on tb.treeData gets filled and has a "children" property. However at the stage where _loadTopLevelChildren() is called treeData is always empty and has no "children" property. I can't seem to find where treeData is supposed to be populated which might help to know what's causing the issue.

@jpkeith4 I believe this is fixed in our latest release. Feel free to re-open if the problem still exists.


Support context mananger in commit scheduler

I realized that having a context manager for the CommitScheduler makes a lot of sense while reviewing https://github.com/AnswerDotAI/fastdata/pull/9. It's a quick win, so let's do it. Benefit is to guarantee everything is committed correctly.
cc @AndreaFrancis the solution implemented in the PR is still correct, no need to change anything. This PR is more for convenience next time we want to integrate it.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


chore: fix argument order in Sum/Lemmas

I guess we really do need to make the matching Mathlib PRs before merging upstreaming PRs...

I kind of just want to say that we shouldn't be bothered about this in most cases... If you are giving the arguments you should use named argument style (α := Nat) anyway, so you can write them in whatever order you like. More precisely, I'm not sure we want to make implicit argument order a stable guarantee in the majority of lemmas and a decent chunk of definitions.

Okay! I will use \a := Nat in mathlib4#7126, and you can close or merge here as you please!


Suggestion to target .NET Standard 2.1

In the 2.x versions of IdentityServer4 and the corresponding libraries, .NET Standard 2.0 was a target. This allowed for me to include pieces of the library in .NET Standard 2.0 projects instead of making them all target .NET Core 2.x.
Now with 3.x, the libraries are solely targeting .NET Core 3, which would mean that I have to completely retarget my class libraries to .NET Core 3. Is this something that's willing to be considered or will this be the guideline moving forward?

We are dependent on ASP.NET Core 3 - which is a netcoreapp3.0 framework. That's why we need to target this as well.

Understood, I was  wondering if .netstandard 2.1 could be added as an extra target for the particular libraries that solely targeted .netstandard 2.0 in the 2.x versions. For instance, IdenttiyServer4.AccessTokenValidation works fine for the class library I'm using it in, since it still targets .netstandard.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Dominick Baier<EMAIL_ADDRESS>Sent: Thursday, September 26, 2019 11:31:08 AM
To: IdentityServer/IdentityServer4<EMAIL_ADDRESS>Cc: Jeremy Sinclair<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [IdentityServer/IdentityServer4] Suggestion to target .NET Standard 2.1 (#3680)
We are dependent on ASP.NET Core 3 - which is a netcoreapp3.0 framework. That's why we need to target this as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FIdentityServer%2FIdentityServer4%2Fissues%2F3680%3Femail_source%3Dnotifications%26email_token%3DAA6URJLNBRXKG67EJEQNB33QLTITZA5CNFSM4I23CSO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7V7YYY%23issuecomment-535559267&data=02|01||14529f1638c74c8a9cb608d742968e37|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637051086703475902&sdata=l%2FeYIAqo9qmiKinrRtbvPiDtjqLqT0PxOB%2FHIw7kbL8%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA6URJMGYBC6TWQDJXLTDP3QLTITZANCNFSM4I23CSOQ&data=02|01||14529f1638c74c8a9cb608d742968e37|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637051086703485907&sdata=RJE4p0Fr3VikxcwB5XDCQbTByEEIp5lP55W3TddVBIg%3D&reserved=0.

We already target the lowest possible. Everything would mean major refactoring and probably breaking changes.

Ah, gotcha. Thanks for the clarification!