Add switch channel commands into activationEvents

I see some users want to switch daily channel for new features (e.g. Jupyter Notebook native support), but meet with errors: command 'Python.swichToDailyChannel' not found.
So we need to add switch channel commands into activationEvents to avoid this kind of issues.

Codecov Report

Merging #7635 into master will decrease coverage by 0.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7635      +/-   ##
==========================================
- Coverage   58.69%   57.93%   -0.76%
==========================================
  Files         493      493
  Lines       21995    21640     -355
  Branches     3533     3528       -5
==========================================
- Hits        12909    12538     -371
- Misses       8278     8286       +8
- Partials      808      816       +8

Impacted Files
Coverage Δ

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-75%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
0% <0%> (-66.67%)
:arrow_down:

...science/jupyter/jupyterKernelPromiseFailedError.ts
0% <0%> (-50%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-50%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

src/client/language/characters.ts
12% <0%> (-21.34%)
:arrow_down:

src/client/testing/codeLenses/main.ts
60% <0%> (-17.78%)
:arrow_down:

src/client/datascience/cellFactory.ts
72.34% <0%> (-17.32%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

src/client/linters/pydocstyle.ts
84.61% <0%> (-5.13%)
:arrow_down:

... and 32 more

Continue to review full report at Codecov.

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


docs(fields): fix link to union modes

Change Summary
The description of the union_mode parameter of the Field function is provided with a link to a nonexistent paragraph on the Standard Library Types page: https://docs.pydantic.dev/latest/api/standard_library_types/#union-mode
I believe that the correct link is: https://docs.pydantic.dev/latest/concepts/unions/#union-modes. It has been changed in this PR.

Related issue number
There is none.

Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

We're working on a fix for the typing extensions tests, and will merge this once that's fixed!

@antoni-jamiolkowski,
Could you please pull in the changes from main? That should allow tests to pass. Thanks!

@sydney-runkle,
Sure :slightly_smiling_face:


Seal safehandles to devirtualize

Seal the bottom level of safehandles to allow devirtualization of the safehandle calls https://github.com/dotnet/coreclr/pull/9230
Much as I don't like sealed; this does now have advantages and some of them are hot methods... And not sure they will be expected to be further derived?


New Resource SPOTenantCDNPolicy

Pull Request (PR) description
New Resource SPOTenantCDNPolicy to control tenant wide CDN settings
This Pull Request (PR) fixes the following issues
N/A

This change is

Issues with cmdlets preventing PR from running tests:
https://github.com/SharePoint/PnP-PowerShell/issues/2271

Codecov Report

Merging #206 into Dev will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              Dev     #206      +/-   ##
==========================================
+ Coverage   89.96%   90.08%   +0.11%
==========================================
  Files          52       53       +1
  Lines        4596     4650      +54
  Branches        8        8
==========================================
+ Hits         4135     4189      +54
  Misses        453      453
  Partials        8        8

Impacted Files
Coverage Δ

...urces/MSFT_SPOPropertyBag/MSFT_SPOPropertyBag.psm1
94.11% <ø> (ø)
:arrow_up:

...FT_SPOTenantCdnPolicy/MSFT_SPOTenantCdnPolicy.psm1
100% <100%> (ø)

Continue to review full report at Codecov.

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

@ykuijs Can you review? It introduces a new resource (even though the underlying cmdlets are broken:
https://github.com/SharePoint/PnP-PowerShell/issues/2271
And it also introduces the ability to do quiet Workload Specific extract which will help us get around the SPO Management Shell bug for the moment.


README does not contain prerequisites

Add prerequisites for using Google Vision API to README

Prerequisites added in pull request #27


Custom tab name allows blanks

During defining the new marketable service, the "Custom tab name" allows blank characters. The documentation and specification needs to be changed accordingly. Currently it says that the blanks are not allowed.

Corrected the relevant online help topics.
@misako0927 Please do the same in the Japanese topics
help/tasks/service_add.xml
help/tasks/service_edit.xml

@misako0927 is working on the custom tabs currently.
@redmond2683
Please could you care for the Japanese translations?

Done for Japanese.


[map refactor] Add data fetcher modules and a chart store

This is a no-op change to existing code and prepares for future updates of chart_loader.
The goal is to remove the dependency of data fetch and processing. Ideally, the tool only needs to fetch the needed data and will render the map when all necessary data is ready. The other data can be loaded on the background or on-demand, which can be controlled independently without touching the component logic.
Some design thoughts:

Each data load should be in a separate module, which fetches data and sends the raw data to the desired component. These modules are put in /fetcher folder. New data fetcher should be created in the same pattern.

All the raw data are used in various ways to construct the chart data, the construction depends on context (user input) and is dependent on data availability. Here I created a ChartStore to hold all the raw data and also the context when they are created. This has the following benefits:

Allow each fetcher to independently "dispatch" the data. The data is merged using "useReducer" hook in the UI component.
The UI component can check if the data is ready by comparing the context in ChartStore vs the current context.
In future change, the async data processing dependency can be resolved by "useEffect()".  By checking the readiness of the data, each useEffect() can exit early if needed.

Thanks for the detailed review!


Typo in the README.md file

in the section Getting Started there is a typo:

I would create a PR - but I do not have the needed access privilege.

Fixed in adriatic-1 PR


useRoute in nested Router with basepath cannot match absolute path

When you use useRoute in a nested Router with basepath specified, the path matching only matches the path portion after the base path. For example, in the following configuration,
<TopLevelRouter>
  <ChildRouter base='/test'>
    ...
    <Component />
    ...
  </ChildRouter>
  <ChildRouter base='/othertest'>
    ...
    <Component />
    ...
  </ChildRouter>
</TopLevelRouter>

The useRoute inside Component cannot match absolute paths like ~/test/:rest* (the way Links can), meaning there is no way to figure out if the current path is under /test or /othertest. Looking at the matcher source code, this is just a functionality that has not been implemented. But from a conceptual, functional point of view, is this considered a bug? I would think that there should be a way to match full path without resorting to window.location.href?

@waful  why not distinguish by path, you need to use base?

@cbbfcd
Elaborated contrived example:
<TopLevelRouter>
  <ChildRouter base='/userTypeA'>
    ...
    <Route path='/settings' component={SharedSettingsComponent} />
    <Route path='/otherSharedPage' component={SharedPageComponent} />
    <Route path='/featureA' component={FeatureAComponent} />
    <Route path='/' component={TypeAComponent} />
    ...
  </ChildRouter>
  <ChildRouter base='/userTypeB'>
    ...
    <Route path='/settings' component={SharedSettingsComponent} />
    <Route path='/otherSharedPage' component={SharedPageComponent} />
    <Route path='/' component={TypeBComponent} />
    ...
  </ChildRouter>
</TopLevelRouter>

I have 2 user types with some shared pages and some type specific pages nested in their respective routers. The settings page for both users are for the most part identical with minor styling/wording changes, so I want to use the same component with a switch flag instead of creating two completely distinct components. I want to be able to identify the nested base route I'm currently in from within SharedSettingsComponent, but the useRoute inside it can only match /settings and not /userType(A|B)/settings. No matter if you're in /userTypeA/settings or /userTypeB/settings you can only match the same path /settings. <Link> handles this today already, you can <Link href='/otherSharedPage'> from inside SharedSettingsComponent and it will go to /userType(A|B)/otherSharedPage based on which ChildRouter you're in, and you can link outside of the current nested base path for example with <Link href='~/userTypeA/otherSharedPage'> even if you're in /userTypeB. But useRoute does not have a similar capability to interact outside of its current nested router context.

i want made something similar, look exemple:
function Item({ label, href }) {
  const [match] = useRoute(href)
  return match ?
{label}
 :
{label}

};

function MainMenu() {
  return (

      <Item label="A" href="section-A" />
      <Item label="B" href="section-B" />

  )
};

function SubMenu() {
  return (
    <>
      <Item label="1" href="/option-1" />
      <Item label="2" href="/option-2" />
    </>
  )
};

function App() {
  return (
    <Router>
      <MainMenu />
      <Switch>
        <Route path="/section-A">
          <NestedRouter base="/section-A">
            <SubMenu />
            <Switch>
              <Route path="/option-1">

Option 1 from Section A

              </Route>
              <Route path="/option-2">

Option 2 from Section A

              </Route>
            </Switch>
          </NestedRouter>
        </Route>
        <Route path="/section-B">
          <NestedRouter base="/section-B">
            <SubMenu />
            <Switch>
              <Route path="/option-1">

Option 1 from Section B

              </Route>
              <Route path="/option-2">

Option 2 from Section B

              </Route>
            </Switch>
          </NestedRouter>
        </Route>
      </Switch>
    </Router>
  )
};

ReactDOM.createRoot(document.getElementById('root')).render(
  <React.StrictMode>
    <App />
  </React.StrictMode>
)

The NestedRouter is from example on docs.

So what i want is that my Item component can figure out if he is a selected item by useRoute, on first level it works, but on the others doest.
If i have selected a item on MainMenu, he become selected, but if i select another item on SubMenu, the previous selected item become unselected. Thinking that current selected item from SubMenu is inside a nested router, the item on MainMenu need be selected too,  but useRoute doesnt match.


Upgrade minimum Node version to 14 in netlify-plugin-next

Currently the "engines" property in the plugin's package.json is set to node v12.
Given node 12 was EOLed recently, this should be upgraded to 14.

For frameworks plugins we've been matching the minimum versions in the frameworks themselves. Last time I checked, Next.js still supports 12, but hopefully that will change soon. Gatsby moved to 14 last year, which is why we can require that.

With Next v13, the minimum node version is v14: Docs

Will need to hold off on updating since Next v12 still has node 12 as the minimum.

Superseded by v5.


I'm the author of emoxpand

I forgot to modify this when creating the bot

Codecov Report

Merging #169 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
- Coverage   44.87%   44.79%   -0.08%
==========================================
  Files          42       42
  Lines        3978     3978
  Branches      818      818
==========================================
- Hits         1785     1782       -3
- Misses       1726     1729       +3
  Partials      467      467

Impacted Files
Coverage Δ

prime/index.js
16.94% <0%> (-0.85%)
:arrow_down:

Continue to review full report at Codecov.

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

ついでに @naan4UGen をbetter-custome-responsesのオーナーにしました


Cloud Storage: list_blobs(delimiter='/', prefix="subdir/") includes subdir?

Specify the API at the beginning of the title: Cloud Storage
Python3, Container Optimized OS in GCE
Python 3.5.3
Library version: google-cloud-storage (1.8.0)

Code example (7):
for blob in bucket.list_blobs(delimiter='/', prefix='subdir/'):
    if blob.name = 'subdir/':
        print("should not happen")
    else:
        # real blob, proceed

Maybe I am not getting it, but after looking into the API and the issues I thought that above should not happen should not be printed when using delimiter and prefix. Is this the intended behavior, so that I must filter out the subdir itself in the loop to only operate on real blobs?

@frankyn FYI

HI @james-woods,
Looking for clarification. I'm assuming you're seeing "should not happen" as it found a blob with subdir/?
If this is correct, it's possible that you created a "folder" using the Cloud Storage browser. When a folder is created an empty object with the folder name is created to "simulate" a file structure even though it doesn't really exist.

Hi @frankyn - yes. I think you've nailed it. My question would be how I can remove the artificial folder with the minimum amount of steps as the "folder" keeps currently over 90k objects.

gsutil rm gs://bucket/subdir did the trick :)


Line missing in read.go

Queries of the type [Hh]ashTable fail to yield results, where hashTable and
HashTable each gets results.
It seems like a line is clearly missing in read.go in method:
  func (r *postReader) init(ix, trigram, restrict)
so that the restrict parameter is ignored.

The attached patch adds a test that shows the bug and fixes the problem.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 15 Apr 2012 at 3:59

Merged into: #8

Attachments:

query_bug_fix.diff

The [Hh]ashtable problem is actually an issue with the regex engine, reported
and fixed as issue #8.  The "restrict isn't initialized" bug is issue #16, also
with a patch.

I should poke Russ Cox to release a new version of codesearch, as there are a
number of patches sitting un-merged on the bug tracker.

Original comment by dgryski on 16 Apr 2012 at 8:53

Original comment by dgryski on 2 May 2012 at 8:54

Changed state: Duplicate

Thank you, that would be great!

Original comment by<EMAIL_ADDRESS>on 16 Apr 2012 at 6:26


Remove duplicate homu package

This explains why we were seeing such weird, old versions of homu being installed.

This change is

@bors-servo r+

:pushpin: Commit e6bbbbd has been approved by Manishearth

:hourglass: Testing commit e6bbbbd60243fa029529ced6f751336bc13c8509 with merge 4a60e467a239cb4da8ea397d28d95847c1462279...


Dark theme text issue

Text in dark theme is not inverted properly.

Assign me to this issue. I can fix it. Also, please add hacktoberfest label.

#9 fixed in this PR

Please don't remove hacktoberfest label.


Some tv2lorry.dk links fail to extract kaltura id

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.02.03. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Description
Some of the www.tv2lorry.dk links fail to download with this error message:
ERROR: Unable to extract kaltura id;
https://www.tv2lorry.dk/gadekamp/gadekamp-oerestad works ok
https://www.tv2lorry.dk/gadekamp/gadekamp-6-hoejhuse-i-koebenhavn does not work
Also posted on https://github.com/ytdl-org/youtube-dl/issues/30593 (not sure what is currently maintained).
Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.tv2lorry.dk/gadekamp/gadekamp-6-hoejhuse-i-koebenhavn']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.1 (CPython) - macOS-12.1-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.4.1, ffprobe 4.4.1, rtmpdump 2.4
[debug] Proxy map: {}
[TV2DK] gadekamp: Downloading webpage
ERROR: Unable to extract kaltura id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/tv2dk.py", line 93, in _real_extract
    kaltura_id = self._search_regex(
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract kaltura id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

See the patch at https://github.com/ytdl-org/youtube-dl/issues/30593.

50e93e03a7ca6ae35a319ea310104f7d6d91eee3


Nt/in app browser

web-view conversion to functional component
improved in-app browser interface

disabled dark mode in browser
added custom header
status bar dark content

Screenshots/Video

@feruzm updated....


Refactor examples

Codecov Report

Merging #37 (59c55e6) into main (7af11d3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   81.82%   81.82%
=======================================
  Files          34       34
  Lines        2405     2405
=======================================
  Hits         1968     1968
  Misses        437      437

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


card container overflows onto for on small screens

when viewing on small device, e.g. iphone 4, the card overflows into the form field.  easily reproducible with iOS simulator or real iphone!

Hi! You should be able to fix this using the width option or using a CSS selector to scale the container. Thanks!

hi @jessepollak!  thanks, sorry for the delayed response.  the card expanded outside of it's container element in this case.  is there anything requiring setting to make it fully self-contained/responsive?  perhaps from your point of view it is supposed to, it's just not what i experienced.  i did try and remove our own CSS to isolate it from our content.  again, it was only on iOS devices--it did scale per expectation on other devices!  thanks again!

@cdaringe where you able to fix this?

Sorry, I have long detached from that project to no longer have access to
the source code. Unfortunately I cannot recall
On Tue, Nov 17, 2020, 8:06 AM hounw<EMAIL_ADDRESS>wrote:

@cdaringe https://github.com/cdaringe where you able to fix this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jessepollak/card/issues/243#issuecomment-729029361,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHU57O7XKTCA7SVWQZRY4DSQKNPTANCNFSM4CBFFTBA
.


Correct API request url

Corrects API endpoint for request in README file.

@niranjan94 Please review. Thanks!

@mariobehling Please review. Thanks :smile:


Unscaling Wrapped optimizer making the gradient is unscaled twice

Hi, thank you for the useful tool.
https://github.com/huggingface/accelerate/blob/main/src/accelerate/accelerator.py#L416-L417
I'm referring to these lines of code where the wrapped optimizer is unscaled when gradient clipping. I think the inner optimizer  should be unscaled instead. Since the scaler will mark the optimizer state based on its id, giving wrapped optimizer and unscaling it would make the gradient get unscaled but the inner optimizer is not marked as unscaled. Then when calling accelerator.backward(), gradient get unscaled again.

That's right. Do you want to make a PR to fix this?

It is ok, you can fix it


Created Procedure for Setting Up a Scaler

Description of Changes
Created a procedure for setting up a scaler in KEDA.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Issue Fixed
Fixes kedacore/keda-docs#1368

@tomkerkhove please review this issue

@zroubalik can you please review

I think we should move this into Getting Started section: https://keda.sh/docs/2.15/
At least for 2.15

Thanks for the review. I will make the necessary changes as specified.

I guess I made a mistake while rebasing. I will open a new PR with the required commits


[Debug] "Run In Terminal" is broken in the 1.42.0-insider

Version: 1.42.0-insider (system setup)
Commit: 7c0095ee2d064033fc13184127a9adc603454729
Date: 2019-12-20T07:57:35.999Z
Electron: 6.1.6
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18363

Issue 1:
I already set my integrated terminal to cmd via "terminal.integrated.shell.windows": "C:\\Windows\\System32\\cmd.exe". But when i choose to run my Java program in the integrated terminal, it always opens powershell to run my program. This issue only appears in the insider version.
Issue 2:
The powershell in the insider doesn't escape the runInTerminal command line arguments well.
Here is the DAP request.
{
    "command": "runInTerminal",
    "arguments": {
        "kind": "integrated",
        "title": "Java Debug Console",
        "cwd": "c:\\spring-petclinic",
        "args": [
            "C:\\Users\\jinbwan\\.vscode-insiders\\extensions\\vscjava.vscode-java-debug-0.24.0\\scripts\\launcher.bat",
            "C:\\Program Files\\Java\\jdk1.8.0_231\\bin\\java",
            "-agentlib:jdwp\u003dtransport\u003ddt_socket,server\u003dn,suspend\u003dy,address\u003dlocalhost:51804",
            "-Dfile.encoding\u003dUTF-8",
            "-cp",
            "C:\\Users\\jinbwan\\AppData\\Local\\Temp\\cp_ahnlxjst7ispcqdhfngxir0if.jar",
            "org.springframework.samples.petclinic.PetClinicApplication"
        ]
    },
    "seq": 2,
    "type": "request"
}

Below is the powershell screenshot in the stable vscode 1.41.1, it works well.

Below is the powershell screenshot in the 1.42-insider, it doesn't work.

Have a try to set my integrated terminal to powershell directly, it works. Looks like VS Code sent the cmd-style arguments to powershell by mistake.

I would just like to note that I have faced issue 2 for several weeks now. Possibly since the beginning of December.

@testforstephen I'm guessing you are using extensions for Java. Can you list the extension versions that run in insiders and the versions that run in stable? If it's the same version, the code change is in the VS Code core.

/needsMoreInfo

In the insider, i just enable three Java extensions, and they are the same version as those running in the stable.

Anyway, i'm the committer of these Java extensions, i believe this issue is not caused by the Java extensions.

In VS Code 1.38 (August) we've introduced the notion of an "automationShell" setting which is used when VS Code runs tasks or debug targets in the integrated terminal (see https://code.visualstudio.com/updates/v1_38#_new-shell-setting-for-launching-a-debug-target and https://code.visualstudio.com/updates/v1_38#_automation-shell-setting).
So if you want to configure VS Code to use the command prompt ("cmd.exe") when launching the debug target from the integrated terminal, you must change
    "terminal.integrated.shell.windows": "C:\\Windows\\System32\\cmd.exe",

to
    "terminal.integrated.automationShell.windows": "C:\\Windows\\System32\\cmd.exe",

@Tyriar do you know why this problem surfaces only in the latest Insiders (1.42.0) and not in the August release (1.38.0), where this new concept was introduced?

@weinand there was a regression around terminal settings recently that I'm trying to work through with @sandy081 in https://github.com/microsoft/vscode/issues/87088, I would guess this is due to that.

@weinand you should probably revert your fix as getDefaultShell is meant to handle automationShell

@Tyriar thanks!
Another question:
I'm using this code to find value of the automationShell in the extension host:
const configProvider = await this._configurationService.getConfigProvider();
const terminalConfig = configProvider.getConfiguration('terminal');

let shell;
const automationShellConfig = terminalConfig.integrated.automationShell;
if (automationShellConfig) {
	if (env.isWindows) {
		shell = automationShellConfig.windows;
	} else if (env.isLinux) {
		shell = automationShellConfig.linux;
	} else if (env.isMacintosh) {
		shell = automationShellConfig.osx;
	}
}
if (!shell) {
	shell = this._terminalService.getDefaultShell(true, configProvider);
}

Is there a simpler way?
something like:
shell = this._terminalService.getShell(true, configProvider);

@weinand maybe it's badly named but getDefaultShell gets the "default" after considering all settings/environment factors. automationShell is meant to be considered here:
https://github.com/microsoft/vscode/blob/eaf427d740f7d3aea7c385d113c320956c8a85dc/src/vs/workbench/contrib/terminal/common/terminalEnvironment.ts#L248-L264

@Tyriar getDefaultShell does not work for me.
If I have this setting on macOS:
	"terminal.integrated.automationShell.osx": "/usr/local/bin/powershell",

then this code:
	const configProvider = await this._configurationService.getConfigProvider();
	let shell = this._terminalService.getDefaultShell(true, configProvider);

results in shell set to /usr/local/bin/bash.

@weinand yes I think it's broken because of #87088

Ok, then I will keep my current code as a workaround until #87088 is fixed.

I've removed my workaround for #87088 since #87088 is fixed now.


sliding window

Scenarios tested:

If syncPosition is currentTimeStamp - 1 hour, and I set an age restriction, then do not move the sync position back.

Halt the syncPosition until it is aligned with the threshold

If i activate a meteringpointcontract, while an age restriction is applied, then the sync position should not start at currentTimeStamp - ageRestrictionInHours

If a missing interval has a span ranging from before until after the age restriction, only the missing interval within the threshold, is processed.

It should not fetch measurements not within the threshold

It should not publish measurements not within the threshold

It should not process missing intervals not within the threshold

It should not register new missing intervals not within the threshold

Only publish measurements within the threshold

Only fetch measurements within the threshold

Only process intervals within the threshold

Only register new intervals within the threshold

An ageRestriction set to 0 is the same as no age restriction

Increasing or decreasing the ageRestriction does not affect the current syncPosition

Extra assertions needed:

Worker must not launch unless MinimumAgeThresholdInHours is explicitly set.

Not setting MinimumAgeThresholdInHours should not evaluate it to 0

Not setting MinimumAgeThresholdInHours,should throw a validation error, with a message saying "Option: MinimumAgeThresholdInHours must be set"

Setting MinimumAgeThresholdInHours to 0 explicitly should mean that CalculateMinimumAgeThreshold() method always returns a value matching UnixTimeStamp.Now.RoundToLatestHour() OR UnixTimeStamp.Now.Add(TimeSpan.FromHours(-0).RoundToLatestHour

Looks like it works:

{
    "federatedStreamId": {
        "registry": "energy-origin",
        "streamId": "fc9c54f5-0e03-45b8-93c8-dd28dd935086"
    },
    "quantity": 8515990,
    "start":<PHONE_NUMBER>,
    "end":<PHONE_NUMBER>,
    "gridArea": "DK1",
    "certificateType": "production",
    "attributes": {
        "energyTag_ConnectedGridIdentification": "DK1",
        "energyTag_Country": "Denmark",
        "energyTag_EnergyCarrier": "Electricity",
        "energyTag_GcFaceValue": "8515990",
        "energyTag_GcIssuanceDatestamp": "10/24/2024",
        "energyTag_GcIssueDeviceType": "Production",
        "energyTag_GcIssueMarketZone": "DK1",
        "energyTag_GcIssuer": "Energinet",
        "energyTag_ProducedEnergySource": "F01050100",
        "energyTag_ProducedEnergyTechnology": "T020000",
        "energyTag_ProductionDeviceCapacity": "12345",
        "energyTag_ProductionDeviceCommercialOperationDate": "N/A",
        "energyTag_ProductionDeviceLocation": "1 Producent Vej En by 9999",
        "energyTag_ProductionDeviceUniqueIdentification": "571313130083535430",
        "energyTag_ProductionEndingIntervalTimestamp": "1729692000",
        "energyTag_ProductionStartingIntervalTimestamp": "1729688400"
    }
}

Confirmed that reducing the threshold from 24 to 12 hours, correctly issues 12 hours worth of certificates:

Can also confirm that setting the threshold to 0, correctly fetches all remaining 12 certificates for the 3 meteringPoints.
12+12+12 = 36
105+36 = 141

Curious discovery: How can i have 150 fetched measurements & 150 certificates, but only 132 published measurements?:

Now one has to explicitly set the Options, and an undeclared variable will not be evaluated to a 0 int:


{packaging} Update argcomplete to 1.12.3

Description
Packaging azure-cli on Fedora requires patches since Fedora offers
argcomplete 1.12.3. Bump the version number to the latest argcomplete
release.
Testing Guide
Existing tests should continue working.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

packaging

Hi, what about updating this PR to package argcomplete 2.0.0 that was recently released? https://github.com/kislyuk/argcomplete/releases/tag/v2.0.0

Hi, what about updating this PR to package argcomplete 2.0.0 that was recently released? https://github.com/kislyuk/argcomplete/releases/tag/v2.0.0

Done! https://github.com/Azure/azure-cli/pull/21105


Observation: DELETE /targets returns a 500

Look for the general case and make sure that the mock matches

So does adding a target with empty content


The confiration email is missing link during signup

Describe the bug
When a tenant signs up, the email look as follow

Hi there,

Thank you for signing up for our services.

To get you started, please enter following code in verify screen.

137536

Alternatively you can click here to verify your account.

Thank you

the link is missing. Also, this email should probably be a template when one can change. Maybe also make the finish email as a template so one can change the body of the email

@CrestApps Email is a template that you can override

Link is fixed in latest RC


Precompile functions usage report

Problem
We should investigate how many times precompile functions are used in the current mainnet contracts, and how many transactions have interacted with the current contracts.
Solution
Disassemble contracts bytecode to find out calls to the 0x167, fetch the contract results for these contracts, and get the calls generated in the fetched transactions.
First part:

fetch all contracts on mainnet
disassemble their bytecode, look up for call, staticcall, delegatecall against 0x167 and list how many times each precompile function is used

Second part:

fetch all transactions that have interacted with these contracts
get their calls via MAPI /contracts/results/<tx_hash>/actions
humanize the data and add it to the report

Alternatives
No response

Here is the PoC branch https://github.com/hashgraph/hedera-smart-contracts/tree/precompile-functions-usage-report/tools/analysis/precompile-contract-functions-analyzer

@natanasow this was a great effort.
What's the next steps.
I would suggest

Paste the README result into this ticket for reference even if that branch gets deleted
Open a PR for any changes to existing tools you had to make
Open tickets capturing any features not supported that you had to hack together so that in future we don't have to hack anything together and we can just rerun checked in steps


Format of m_wheelSlip/m_wheelSpeed changed from array of floats to array of objects

Describe the bug
While still an array, the data format that the client sends for wheel slip and wheel speed changed the format. Previously the data was a simple array of float values. In v0.1.10 it changed to an array of objects.
// Before
m_wheelSlip = [0,0,0,0]

// After
m_wheelSlip = [
  { m_wheelSlip: 0},
  { m_wheelSlip: 0},
  { m_wheelSlip: 0},
  { m_wheelSlip: 0},
]

This change does not seem intentional to me and also makes the format a bit more complicated than it has to be IMO.
To Reproduce
const { F1TelemetryClient, constants } = require("f1-telemetry-client");
const { PACKETS } = constants;

const client = new F1TelemetryClient();
client.on(PACKETS.motion, m => console.log(m.m_wheelSlip));

Expected behavior
The data should be a simple array of floats

Hi @codegefluester ,
Thank you for your input. This was not changed in 0.1.10, but it has been like this for several versions now. I will make this change for the v0.1.11 release.

Cheers, I think I just put v0.1.10 since that is what I'm currently on and I didn't use the previous version as much as today.
Thanks for all your work on this!

Bjorn

No problem Bjorn. I am merging v0.1.11 right now, which fixes this behavior. Thanks again.


eciesjs ERR_PACKAGE_PATH_NOT_EXPORTED error

I'm running the Hello World example from the dotenvx README and I'm getting an ERR_PACKAGE_PATH_NOT_EXPORTED error:
 ~/dev/eciesjs-bug/hw$ ls                                                                                 ✔  13:42:01
index.js
 ~/dev/eciesjs-bug/hw$ cat index.js                                                                                 ✔  13:42:02
console.log('Hello ' + process.env.HELLO)
 ~/dev/eciesjs-bug/hw$ cat .env                                                                                 ✔  13:42:03
HELLO=world
 ~/dev/eciesjs-bug/hw$ npx @dotenvx/dotenvx run -- node index.js                                                                                 ✔  13:42:07
node:internal/modules/cjs/loader:598
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in /Users/deisner/.npm/_npx/fdacfd67f4da830d/node_modules/eciesjs/package.json
    at exportsNotFound (node:internal/modules/esm/resolve:303:10)
    at packageExportsResolve (node:internal/modules/esm/resolve:593:13)
    at resolveExports (node:internal/modules/cjs/loader:591:36)
    at Module._findPath (node:internal/modules/cjs/loader:668:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1130:27)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/Users/deisner/.npm/_npx/fdacfd67f4da830d/node_modules/@dotenvx/dotenvx/src/lib/helpers/decryptValue.js:1:21)
    at Module._compile (node:internal/modules/cjs/loader:1376:14) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

Node.js v20.11.1

The dotenvx dependency on eciesjs is resolving to the latest version (at the moment), v0.4.9, which was released a few hours ago:
~/dev/eciesjs-bug/hw$ (cd /Users/deisner/.npm/_npx/fdacfd67f4da830d && npm ls -a eciesjs)
fdacfd67f4da830d@ /Users/deisner/.npm/_npx/fdacfd67f4da830d
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
I think this is triggering the error:
 ~/dev/eciesjs-bug/hw$ cat<EMAIL_ADDRESS>const { decrypt } = require('eciesjs')

I'm not sure if this is an eciesjs bug, or an issue with the way dotenvx is importing it. Please let me know, thanks!

OK, probably eciesjs: https://github.com/ecies/js/issues/802

Perhaps you can pin the version to<EMAIL_ADDRESS>in package.json -- the only dev on eciesjs lives in Japan, where it's 3AM right now.

This broke our package also.  We pinned it through overrides in package.json for now, but ideally this package would pin its dependencies to avoid this in the future.

Same issue :(

For our package.json and npm we added
  "overrides": {
    "@dotenvx/dotenvx": {
      "eciesjs": "0.4.8"
    }
  }

and are running npx @dotenvx/dotenvx run instead of globally installing dotenvx and running it directly. Just make sure your do npm install or equivalent before running npx :)

@finack
It looks great when I install dotenvx globally, how can I solve this problem if I install dotenvx via curl -sfS https://dotenvx.sh | sh?

problem is fixed with new update on eciesjs

Hi Community,
This has been fixed in v0.4.10, sorry for the inconvenience.

I sent PR that fixes this: #421

thank you @kigawas for the amazing library. dotenvx would not be as great without it. 🙏
1.20.1 released everyone - https://github.com/dotenvx/dotenvx/actions/runs/11483050191

also big thanks to everyone here helping to unblock each other - while you waited on us maintainers.

It looks great when I install dotenvx globally, how can I solve this problem if I install dotenvx via curl -sfS https://dotenvx.sh | sh?

to clarify, it did not affect anyone using dotenvx as a binary, because those were already built and released with a working eciejs version. only affected those installing with npm/pnpm/etc


Upgrade org.checkerframework:checker-qual 3.27.0 -> 3.28.0

This PR contains the following updates:

Package
Type
Update
Change

org.checkerframework:checker-qual (source)
compile
minor
3.27.0 -> 3.28.0

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Checker Framework Annotations 3.27.0 -> 3.28.0

See:
- https://github.com/typetools/checker-framework/releases/tag/checker-framework-3.28.0
- https://github.com/typetools/checker-framework/compare/checker-framework-3.27.0...checker-framework-3.28.0


search_replace function

tl;dr
A community zync user had created a switch-based Zed program for a large search-and-replace task in their data transformation pipeline. They asked for some assistance in simplifying it for maintainability. We came up with some improvements using existing Zed building blocks, but in the end @mccanne had the following thought for something new we could write to nail this directly:

What if we used capture groups in Go's regexp library to create a search_replace function?  You could give it the map and we would translate the patterns to capture groups then use the capture group index to select the replacement string.

Details
At the time this issue is being filed, Zed it at commit 9766d17.
The original question from the community zync user was posed as:

do you know if there is way to declare a const with a regular expression ? ex: const myregexp=/myr.+n/1

Indeed, this is not currently possible in Zed. There's other examples of places where a user might expect to be able to use regexps but can't (#4917).
To illustrate the use case, the user shared their program, but it's confidential and can't be pasted here. However, here's a simplified program switch-grep.zed that uses their approach:
switch (
  case (grep(/foo|bar/, desc)) => category := "The Foobar category"
  case (grep(/a.*\-z.*/, desc)) => category := "The A-to-Z category"
  default => category := "The default category"
)

With input data descriptions.zson:
{"desc": "It's foo time"}
{"desc": "a is the first letter-z is last"}
{"desc": "Something else"}

Running it:
$ zq -version
Version: v1.17.0-1-g9766d17d

$ zq -I switch-grep.zed descriptions.zson
{desc:"It's foo time",category:"The Foobar category"}
{desc:"Something else",category:"The default category"}
{desc:"a is the first letter-z is last",category:"The A-to-Z category"}

The user's program actually had about 70 case statements.
In addition to its sheer size, a couple other challenges with maintainability are evident here:

There's a lot of code repeated in support of the actual strings that form the search/replace pairs (i.e., case (grep(..., desc)) => category := ...
Per the user's point, if the regexps could be defined as const, they could be more easily re-used in other contexts (e.g., define them in a separate file that's included with zq -I ... to be invoked in many programs)

After a little hacking, I found this could be improved to this switch-regexp.zed:
const changes = |{
  "foo|bar": "The Foobar category",
  "a.*\\-z.*": "The A-to-Z category"
}|

category := coalesce((over changes with desc | switch (case regexp(key, desc) != null => yield value)), "The default category")

Running it gives the same output we saw before.
$ zq -I switch-regexp.zed descriptions.zson
{desc:"It's foo time",category:"The Foobar category"}
{desc:"a is the first letter-z is last",category:"The A-to-Z category"}
{desc:"Something else",category:"The default category"}

Things to highlight:

By using over we're able to avoid the repeat of all the case clauses
By using the regexp function instead of grep we're able to leverage the former's unique ability to take a regular expression that's defined as a string, and strings can be defined via const. However, this does come with one caveat: Some escape sequences that aren't valid for a string but are for regex (such as the \- in the second pattern) now need a double backslash.

The user was satisfied with this improvement. However, having watched this unfold, @mccanne had the idea quoted above for a purpose-built search_replace function that would allow the user to avoid needing to know or look up the coalesce/over/switch/regexp combination shown here. If regexps in their / / form also became first class concepts at the same time that would surely be convenient to the user as well since they'd be able to avoid the "double backslash" overhead when creating the map (and hence more easily re-use regexps from other tools without modification), but this seems orthogonal.

Building further off what was shown previously, in the absence of the proposed purpose-built search_replace function written in Go as a core part of the Zed language, it could be written as a user-defined function such as this search-replace-func.zed.
func search_replace (s, change_map, default): (
  coalesce((over change_map with s | switch (case regexp(key, s) != null => yield value)), default)
)

I've also broken out the map of pairs for search/replace in this changes.zed.
const changes = |{
  "foo|bar": "The Foobar category",
  "a.*\\-z.*": "The A-to-Z category"
}|

Putting it all together at the command line:
$ zq -I search-replace-func.zed -I changes.zed 'category := search_replace(desc, changes, "The default category")' descriptions.zson
{desc:"It's foo time",category:"The Foobar category"}
{desc:"a is the first letter-z is last",category:"The A-to-Z category"}
{desc:"Something else",category:"The default category"}

This shows simple modularity at the CLI. We've also discussed the idea of more formally introducing "Zed modules" (#2599) and related concepts like having a standard library of functionality written as Zed to augment the rest of the core functionality implemented in Go, community users being able to share their own modules/libraries easily such as via GitHub, and so forth. This opens the door to being able to deliver enhancements in the short term as Zed that might be hacky at times but can be used like a black box. If perf problems or functional gaps in such an implementation limit usability, usage in its initial form would help validate the use cases for more ambitious core functionality later written in Go.

Also related: The original user later expressed an interest in being able to pull the search/replace pairings from a pool rather than defining them in a const. I was able to hack something together that does this using existing building blocks, but it's super ugly. A better approach would be if we had what's described in #3201.
Separately, @mattnibs has looked over everything shown here and had the impression that a "cross-product join" would be the way he'd approach this problem.


Shorthand v-bind

What problem does this feature solve?
If I want to use v-bind on a component where the value I want to use is the same as the name of the property, it is redundant. It will provide a better developer experience to remove this redundancy.
What does the proposed API look like?
Current:
<!-- example.vue -->

<script>
export default {
  data() {
    return {
      somedata: true,
    };
  }
}
</script>

<template>
  <SomeComponent :somedata="somedata" /> <!-- notice we have to use the same word twice, which is redundant and unnecessary -->
</template>

Proposed:
<!-- example.vue -->

<script>
export default {
  data() {
    return {
      somedata: true,
    };
  }
}
</script>

<template>
  <SomeComponent :somedata /> <!-- Automatically using the somedata value -->
</template>

Duplicate of #8178.


ng update fails to remove polyfills leaving broken code behind (8rc 4)

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [x] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?

Yes, previous ng updates that I have tried worked as expected.
Description
When running ng update @angular/cli @angular/core --next the process completes successfully, however the polyfills file has been left broken. It appears to have attempted to remove the core-js imports although failed half way through leaving half a line of code:
/***************************************************************************************************
* BROWSER POLYFILLS
*/
k-map';
import 'core-js/es6/set';

/** IE10 and IE11 requires the following for NgClass support on SVG elements */
// import 'classlist.js';  // Run `npm install --save classlist.js`.

All imports after the broken line are not removed. This obviously causes the build to fail.
🔬 Minimal Reproduction
Clone this repo: https://github.com/UXAspects/UXAspects-Seed
Perform the update: ng update @angular/cli @angular/core --next --force
CLI will report success, however the broken code now remains
🔥 Exception or Error
No exceptions during update, reports successful update
🌍 Your Environment

Angular CLI: 8.0.0-rc.4
Node: 10.15.1
OS: win32 x64
Angular: 8.0.0-rc.4
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, upgrade

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.0-rc.4
@angular-devkit/build-angular     0.800.0-rc.4
@angular-devkit/build-optimizer   0.800.0-rc.4
@angular-devkit/build-webpack     0.800.0-rc.4
@angular-devkit/core              8.0.0-rc.4
@angular-devkit/schematics        8.0.0-rc.4
@angular/cdk                      7.3.1
@angular/http                     7.2.3
@ngtools/webpack                  8.0.0-rc.4
@schematics/angular               8.0.0-rc.4
@schematics/update                0.800.0-rc.4
rxjs                              6.5.2
typescript                        3.4.5
webpack                           4.30.0

Anything else relevant?
No

Hi, I just download your repo and tried it locally and I am unable to replicate the issue
here's the polyfills.ts
/**
 * This file includes polyfills needed by Angular and is loaded before the app.
 * You can add your own extra polyfills to this file.
 *
 * This file is divided into 2 sections:
 *   1. Browser polyfills. These are applied before loading ZoneJS and are sorted by browsers.
 *   2. Application imports. Files imported after ZoneJS that should be loaded before your main
 *      file.
 *
 * The current setup is for so-called "evergreen" browsers; the last versions of browsers that
 * automatically update themselves. This includes Safari >= 10, Chrome >= 55 (including Opera),
 * Edge >= 13 on the desktop, and iOS 10 and Chrome on mobile.
 *
 * Learn more in https://angular.io/guide/browser-support
 */

/***************************************************************************************************
* BROWSER POLYFILLS
*/

/** IE10 and IE11 requires the following for NgClass support on SVG elements */
// import 'classlist.js';  // Run `npm install --save classlist.js`.

/** IE10 and IE11 requires the following for the Reflect API. */
import 'core-js/es6/reflect';

/** Evergreen browsers require these. **/
// Used for reflect-metadata in JIT. If you use AOT (and only Angular decorators), you can remove.
import 'core-js/es7/reflect';

/**
 * Required to support Web Animations `@angular/platform-browser/animations`.
 * Needed for: All but Chrome, Firefox and Opera. http://caniuse.com/#feat=web-animation
 **/
// import 'web-animations-js';  // Run `npm install --save web-animations-js`.

/***************************************************************************************************
 * Zone JS is required by default for Angular itself.
 */
import 'zone.js/dist/zone';  // Included with Angular CLI.

/***************************************************************************************************
 * APPLICATION IMPORTS
 */
(window as any).global = window;

Comment removed, as I managed to replicate this on windows.


Do not add Style as a feature property

This makes the KML format not add a Style property on features, as this may cause problems when serializing to another format such as JSON.
Close #3832

Also see https://github.com/openlayers/ol3/issues/3832#issuecomment-121174757 for a an idea of a simpler solution.

Thanks for the review @ahocevar.
@Jenselme, please add https://github.com/elemoine/ol3/commit/f10fb8172ce8a16b4de5ba4a6127b41adfcb0200 to your branch. When done, I'll merge this PR. Thanks.

PR updated.

delete object['Style'] causes some kind of disadvantage. When you create a custom function to create feature which needs its position and icon url to display, there's no way to get icon url from KML data. because it only has style functions. style function is suitable when style has some condition but in this case there's no condition.


Connection is currently waiting for an insert operation, check your previous InsertStatement.

Environment

OS version:
JDK version:
ClickHouse Server version:
ClickHouse Native JDBC version:
(Optional) Spark version: N/A
(Optional) Other components' version: N/A

Error logs

paste your error logs here

Steps to reproduce
Other descriptions

DO NOT insert currently, it's the limitation of ClickHouse native protocol. A single thread with a large batch size can achieve high performance.


Multiple concurrent requests

This middleware uses only one webdriver. If you have multiple requests running concurrently, probably you cannot get the correct result.
@clemfromspace please correct me if i'm wrong.

I guess you are right in the way of all the calls to the driver are synchronous.
The spider will handle the requests in a non-concurrently way (related to #2 ?)
One solution should be to remplace selenium with an asynchrone alternative, like https://github.com/HDE/arsenic or https://miyakogi.github.io/pyppeteer/
I am currently thinking about implementing a similar middleware using puppeteer, you can see some code here: https://gist.github.com/clemfromspace/bbe9a99da1e86a753baa00e394234b8c (not tested at all, no guarantee it will work)

I found the problem when the scrapy engine sent multiple SeleniumRequests to downloader middleware. In the spider callback function, response.meta['driver'].url is not the same as response.url.
I actually stick with selenium. I generate a webdriver for each request, and it will quit in the spider callback function. The only problem which bothers me is that multiple webdrivers are created, and they consume a lot of memory and can stay long time before being processed. However, it can be solved by setting a positive priority in the request.

I found this issue when trying to create a new one for concurrent requests feature.
I did look at async approach using pyppeteer mentioned in the link. And I don't think it solves the multiple requests simultaneously.
What I think of is every spider has max concurrent connection settings, then can we use it to start new driver with max being that value. And a queue mechanism to distribute the load for each request?

I still have a question about this issue. I'm developing a project where I necessarily need to use Scrapy with Selenium, due to some of its features. But my main bottleneck is concurrence. To get around this problem, I thought of distributing in the same spider part of the requests for Scrapy-Selenium with a single browser and another part for Scrapy-Splash, which allows concurrent requests using the CONCURRENT_REQUESTS setting. In this context where I have two middlewares handling concurrent requests, can I trust that the results returned by Scrapy-Selenium are consistent?


Remove Burrow from dep config

Burrow doesn't need to depend on itself. Thanks to @niels-s for the fix.

Coverage remained the same at 74.254% when pulling 76c514855866b698c1a36ad52686fa208a94097a on toddpalino:remove-self-dependency into cffd70eb72e35b28ab222306c82fe5fdb3d5dfb0 on linkedin:master.


Feature/composite interaction base

Description
Please include a summary of changes, motivation and context for this PR.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change)
[ ] New feature (non-breaking change which adds functionality)
[ ] API change (breaks downstream workflows and requires major semver bump)
[ ] Requires documentation updates

Testing
Please describe the tests that you ran to verify your changes and provide instructions for reproducibility. Please also list any relevant details for your test configuration.

[ ] Test Foo in /test/path/to/file_for_test_Foo.cpp
Description of test Foo
[ ] Test Bar in /test/path/to/file_for_test_Bar.cpp
Description of test Bar

Test Configuration:

OS:
SYCL implementation:
MPI details:
Hardware:

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any new dependencies are automatically built for users via cmake
[ ] I have used understandable variable names
[ ] I have run clang-format against my *.hpp and *.cpp changes
[ ] I have run cmake-format against my changes to CMakeLists.txt
[ ] I have run black against changes to *.py
[ ] I have made corresponding changes to the documentation
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

######### DELETE THIS AND BELOW #########
How-to:

Format C++ source code with clang-format

find ./src ./include ./test -iname \*.hpp -o -iname \*.cpp | xargs clang-format -i

Format cmake files with cmake-format

cmake-format -i CMakeLists.txt src/nektar/CMakeLists.txt test/CMakeLists.txt

Format python with black

find ./python -iname \*.py | xargs black

I have updated the NP side such that this points at the dev branch rather than an I think this is ready for another review/tests running by someone else.

I get some errors like this in the build phase for the dev branch of neso-particles using spack:
     509    In file included from /home/jedge/NESO/neso-particles/include/neso_particles/external_interfaces/common/../../compute_target.hpp:15:
     510    In file included from /home/jedge/NESO/neso-particles/include/neso_particles/containers/../loop/../sycl_typedefs.hpp:7:
     511    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl.hpp:16:
     512    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/backend.hpp:18:
     513    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/detail/backend_traits_opencl.hpp:26:
     514    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/queue.hpp:20:
  >> 515    /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-llvm/../include/sycl/CL
            /sycl/handler.hpp:1242:50: error: '__usmfill<RemotePoint>' is invalid; kernel name should be forward declarable at namespace scope
     516        kernel_parallel_for<KernelName, ElementType>(KernelFunc);

Am I using a compatible compiler?

Everything working now


python311Packages.conda-package-handling: 2.2.0 -> 2.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.conda-package-handling is: Create and extract conda packages of various formats
meta.homepage for python311Packages.conda-package-handling is: https://github.com/conda/conda-package-handling
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.0 with grep in /nix/store/rwlwk2vra70idx1fk6p8d3gymkci4xnv-python3.11-conda-package-handling-2.3.0
found 2.3.0 in filename of file in /nix/store/rwlwk2vra70idx1fk6p8d3gymkci4xnv-python3.11-conda-package-handling-2.3.0

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.conda
python311Packages.conda-package-handling
python312Packages.conda
python312Packages.conda-package-handling

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rwlwk2vra70idx1fk6p8d3gymkci4xnv-python3.11-conda-package-handling-2.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.conda-package-handling https://github.com/r-ryantm/nixpkgs/archive/eda9d6ea92f7ada4f2c787782b3df41d7459f585.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/eda9d6ea92f7ada4f2c787782b3df41d7459f585#python311Packages.conda-package-handling

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rwlwk2vra70idx1fk6p8d3gymkci4xnv-python3.11-conda-package-handling-2.3.0
ls -la /nix/store/rwlwk2vra70idx1fk6p8d3gymkci4xnv-python3.11-conda-package-handling-2.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

8 packages built:

python311Packages.conda
python311Packages.conda-package-handling
python311Packages.conda-package-handling.dist
python311Packages.conda.dist
python312Packages.conda
python312Packages.conda-package-handling
python312Packages.conda-package-handling.dist
python312Packages.conda.dist

Maintainer pings
cc @EricTheMagician for testing.

@nixpkgs-merge-bot merge

@NixOS/nixpkgs-merge-bot merge


[Contribution] Graph History (Undo/Redo)

Hey, I've added this feature to my Diagram implementation which allows me to Undo & Redo some actions (only supports Node & Link Add/Remove for now).
Since this can be possibly useful for anyone in the community, I'm leaving here the code too:
The action object (separate class)
 public class GraphAction
    {
        public enum ActionType { AddNode, RemoveNode, AddLink, RemoveLink }

        public ActionType actionType;

        public object actionData;

    }

Private fields necessary:
private List<GraphAction> undoActionList, redoActionList;
        private int actionHistoryMaxSize = 200;
        private bool isTrackingHistory, undoActionFlag, redoActionFlag;

The specific functions to handle the feautures (NodeData here is my NodeModel implementation):
#region HistoryManagement

        private void InitHistoryTrack()
        {
            undoActionList = new();
            redoActionList = new();
            isTrackingHistory = true;
        }

        public void UndoLastAction()
        {
            if (!undoActionList.Any()) return;

            undoActionFlag = true;
            RevertAction(undoActionList[^1]);
            RemoveLastUndoAction();
            diagram.UnselectAll();
        }

        public void RedoLastAction()
        {
            if (!redoActionList.Any()) return;
            redoActionFlag = true;
            RevertAction(redoActionList[^1]);
            RemoveLastRedoAction();
            diagram.UnselectAll();
        }

        private void RevertAction(GraphAction action)
        {
            switch (action.actionType)
            {
                case GraphAction.ActionType.AddNode:
                    diagram.Nodes.Remove((NodeData)action.actionData);
                    break;
                case GraphAction.ActionType.RemoveNode:
                    diagram.Nodes.Add((NodeData)action.actionData);
                    break;
                case GraphAction.ActionType.AddLink:
                    diagram.Links.Remove((LinkModel)action.actionData);
                    break;
                case GraphAction.ActionType.RemoveLink:
                    diagram.Links.Add((LinkModel)action.actionData);
                    break;
            }
        }

        private void RemoveLastUndoAction()
        {
            if (undoActionList.Any())
                undoActionList.RemoveAt(undoActionList.Count - 1);
        }

        private void ClearRedoList() => redoActionList.Clear();
        private void RegisterRedoHistoryAction(GraphAction.ActionType _actionType, object _data)
        {
            var action = new GraphAction { actionType = _actionType, actionData = _data };
            redoActionList.Add(action);
        }

        private void RemoveLastRedoAction()
        {
            if (redoActionList.Any())
                redoActionList.RemoveAt(redoActionList.Count - 1);
        }

        private void RegisterUndoHistoryAction(GraphAction.ActionType _actionType, object _data)
        {
            if (!isTrackingHistory) return;

            if (undoActionFlag && !redoActionFlag)
            {
                RegisterRedoHistoryAction(_actionType, _data);
                undoActionFlag = false;
                return;
            }
            if (redoActionFlag)
                redoActionFlag = false;
            else
                ClearRedoList();

            var action = new GraphAction { actionType = _actionType, actionData = _data };
            if (undoActionList.Count > actionHistoryMaxSize)
                undoActionList.RemoveAt(0);

            undoActionList.Add(action);
        }
#endregion HistoryManagement

The history track initialization [1st function] w/ its calling context [2nd function], In my implementation I only begin the
history tracking after importing/building the graph for logic reasons:
private void InitHistoryTrack()
      {
          undoActionList = new();
          redoActionList = new();
          isTrackingHistory = true;
      }

public void InitGraph()
      {
         // diagram.RegisterModelComponent<NodeData, VisualNode>();
          //graphBuilder.BuildGraph();            Own code serving as context example
          InitHistoryTrack();

      }

Subscribing to the necessary events (this can be on the diagram Init even before diagram build/import) & the actions registering :
 private void EventsSubscription()
        {
            diagram.Links.Added += Links_Added;
            diagram.Links.Removed += Links_Removed;
            diagram.Nodes.Added += Nodes_Added;
            diagram.Nodes.Removed += Nodes_Removed;
        }

private void Links_Added(Blazor.Diagrams.Core.Models.Base.BaseLinkModel link)
        {
            if (link.TargetNode is null)
                    link.TargetPortChanged += Link_Connected; //In case its a empty link being dragged (listen for its connection)
            else
             //In case it was connected instantaneously (via code)
                RegisterUndoHistoryAction(GraphAction.ActionType.AddLink, link);
        }

         private void Link_Connected(Blazor.Diagrams.Core.Models.Base.BaseLinkModel arg1, PortModel _, PortModel outPort)
        {
            arg1.SourcePortChanged -= Link_Connected;
            RegisterUndoHistoryAction(GraphAction.ActionType.AddLink, arg1);
        }

 private void Links_Removed(Blazor.Diagrams.Core.Models.Base.BaseLinkModel link)
        {
            if (link.IsAttached)
                RegisterUndoHistoryAction(GraphAction.ActionType.RemoveLink, link);
        }

 private void Nodes_Added(NodeModel obj)
        {
            RegisterUndoHistoryAction(GraphAction.ActionType.AddNode, obj);
        }

        private void Nodes_Removed(NodeModel obj)
        {
            RegisterUndoHistoryAction(GraphAction.ActionType.RemoveNode, obj);
        }

The functions below (UndoLastAction & RedoLastAction are exposed allowing user to undo & redo the actions as intended)
Keyboard handle context and functions (Undo w/ Ctrl+Z &  redo w/ Ctrl+Y):

diagram.KeyDown += KeyboardHandle; //This should be on Init too

  private void KeyboardHandle(KeyboardEventArgs e)
        {
            if (e.CtrlKey && e.Key.Equals("z"))
            {
              UndoLastAction();
            }
            else if (e.CtrlKey && e.Key.Equals("y"))
            {
                RedoLastAction();
            }
        }

Hello, thanks for your contributions!
I'll leave this as is in this issue for now as I can't add it to the library at the moment, feel free to create a PR for it so we can discuss this, otherwise users can just use it from here (for now).

I took a slightly different, and perhaps, a more aggressive approach to implement undo/redo in that I converted the models within Diagram Core over to ReactiveUI, turning the Model class into a ReactiveObject and thus making all derived models Reactive as well.  I changed all properties to include a private backing field and then added an Observable pipeline for each property within it's respective model.
public abstract class Model: ReactiveObject, IDisposable
{
    private bool _isTracking;
    private bool _locked;
    private bool _inhibitMove;

    [JsonIgnore]
    public CompositeDisposable? Disposables { get; set; }

    public Model() : this(Guid.NewGuid().ToString()) { }

    public Model(string id)
    {
        Id = id;
    }

    public void TrackHistory()
    {
        if (UndoRedoHistory != null && !_isTracking)
        {
            _isTracking = true;
            Disposables = new CompositeDisposable();

            // This will track all undo/redo changes associated to Locked
            Disposables?.Add(this.WhenAnyValue(x => x.Locked).ObserveWithHistory(x => Locked = x, Locked, UndoRedoHistory, "Model Lock", "Model Lock", postUndoAction: Refresh, postRedoAction: Refresh));

            // This will track all undo/redo changes associated to InhibitMove
            Disposables?.Add(this.WhenAnyValue(x => x.InhibitMove).ObserveWithHistory(x => InhibitMove = x, InhibitMove, UndoRedoHistory, "Model InhibitMove", "Model InhibitMove", postUndoAction: Refresh, postRedoAction: Refresh));
        }
    }

    [JsonIgnore]
    public IHistory? UndoRedoHistory {  get; set; }

    public event Action? Changed;

    [JsonProperty]
    public string Id { get; }

    [JsonProperty]
    public bool Locked
    {
        get => _locked;
        set => this.RaiseAndSetIfChanged(ref _locked, value, nameof(Locked));
    }

    [JsonProperty]
    public bool InhibitMove
    {
        get => _inhibitMove;
        set => this.RaiseAndSetIfChanged(ref _inhibitMove, value, nameof(InhibitMove));
    }

    public virtual void Refresh() => Changed?.Invoke();

    public void Dispose()
    {
        Disposables?.Dispose();
    }
}

For Undo/Redo, I added a ReactiveHistory instance within the Model.
Reactive History provides both a Property and Collection RX extension to capture and track all property and collection changes.  It also provides Undo/Redo/Clear integration support for all operations such as PAN, Zoom, Move, Resize as well as all other property changes.  With those behavior operations that are mouse related such as Pan, Zoom, Move and Resize, I add a RX Throttle filter operation to the observable pipelines so as to only capture an undo/redo state once all property changes have ceased.  The RX Throttle filter will only pass through the last property change, at the expiration of a specified time delay.
See the use of the Throttle filter in the Track History method below...
public abstract class MovableModel : SelectableModel
{
    public MovableModel()
    {
    }
    public MovableModel(Point? position = null ):base()
    {
        Position = position ?? Point.Zero;
    }

    public MovableModel( string id, Point? position = null) : base(id)
    {
                  Position = position ?? Point.Zero;
    }

    private bool _isTracking;
    public new void TrackHistory()
    {
        if (UndoRedoHistory != null && !_isTracking)
        {
            _isTracking = true;
            base.TrackHistory();
            // This will track all changes associated to Position
            Disposables?.Add(this.WhenAnyValue(x => x.Position)
                   .Throttle(TimeSpan.FromMilliseconds(500))
                   .Where(x => x != null)
                   .ObserveWithHistory(x => SetPositionUndo(x), Position, UndoRedoHistory, "Model Position", "Model Position", postUndoAction: Refresh, postRedoAction: Refresh, IsThrottled: true));

        }
    }

    private Point? _position;
    public Point? Position
    {
        get => _position;
        set => this.RaiseAndSetIfChanged(ref _position, value, nameof(Position));
    }

    private void SetPositionUndo(Point newPosition)
    {
        if(newPosition != null)
        {
            _position = newPosition;
        }
    }

    public virtual void SetPosition(double x, double y) => Position = new Point(x, y);
}

For Resize, I integrated with InteractJS .  I had to disable the Move Behavior whilst a ReSize operation was in flight as it conflicts with the Move operation.
I also added a copy/paste clipboard manager into the Diagram which saves selected diagram model elements into the clipboard within the Browser.
I notice that I am not able to serialize the ShapeDefiner so I had to Ignore it when serializing for copy/paste and also, upon paste I have a few small clean up tasks were required to allow the newly pasted instances to become fully adopted within an existing diagram.  With that I am able to copy and paste between diagrams as well as separate instances of the application since I am using the Browser's Clipboard API.
Finally my core designer interface is hosting a Docking Framework in which documents have their own Content View and it's own Diagram instance.
Here is an example of two diagrams within the Docking Layout control, Sample 1 and Sample 2...

Here is Redo Stack if I were to completely Undo the diagram changes...

I had to make a few minor adjustments to both the Diagram and Reactive History projects in order for this to work.
I plan on adding a Property Editor Tool which will provide the means to edit the state of customized Node Renderers and is why I chose the ReactiveUI and Reactive History approach so that Undo/Redo is inclusive of state changes within related nodes.
The Blazor Diagram project is very well designed and well coded.  This made it relatively easy to understand the underlying framework and to integrate the above changes.
Cheers


Add term allelic_frequency

HI
I'm not sure what kind of thing an allelic frequency is?
I can see it being the property, but it is not a sequence feature. Any thoughts, suggestions for a definition?
It is always going to be dependent on a particular population right?
--Karen

@gafiore,  please follow up with an explanation, definition, and an example of how you would use allelic_frequency.

Given the presence of allelic state as a separate issue, I'd suggest that allelic frequency is similar to variant frequency. http://www.sequenceontology.org/browser/current_svn/term/SO:0001763
Under sequence attribute/variant quality.  Variant frequency's definition is
"A physical quality which inheres to the variant by virtue of the number instances of the variant within a population."
I'm not sure allelic frequency is really different from variant frequency. Or is Allelic frequency more about the occurrence of an allelic state....I'll have to double check the resource to look for implied meaning.

Hi Bret
thanks,
It looks like HL7 copied the definition from SO :o)
Does allele frequency need to be a separate entity from allele frequency or should it be a synonym.
--K

I have added this terms as a child of variant frequency.
The ID is:
SO:0002119
The definition is:
A physical quality which inheres to the allele by virtue of the number instances of the allele within a population. This is the relative frequency of the allele at a given locus in a population.

This is checked into the latest revision. I'll keep this open for comments for a couple of days.

Thank you! I'm including the HL7 Clinical Genomics Work Group for feedback.
On Thu, Jul 21, 2016 at 2:14 PM Karen EIlbeck<EMAIL_ADDRESS>wrote:

I have added this terms as a child of variant frequency.
The ID is:
SO:0002119
The definition is:
A physical quality which inheres to the allele by virtue of the number
instances of the allele within a population. This is the relative frequency
of the allele at a given locus in a population.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/The-Sequence-Ontology/SO-Ontologies/issues/357#issuecomment-234337689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAU5jrgkHMoww433ttJlke5htV3lXMY7ks5qX7cQgaJpZM4IxZlI
.


how to use docker？

Dockerfile
FROM node:6.10.2
ENV HOST <IP_ADDRESS>

RUN mkdir -p /app

EXPOSE 3000

COPY . /app
WORKDIR /app
RUN npm install
RUN npm run build

CMD [ "npm", "start" ]

npm install
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm info ok
---> 1242ffb8765c
Removing intermediate container 26306d862022

npm build  happen error
npm info lifecycle<EMAIL_ADDRESS>Failed to exec build script
npm ERR! Linux 3.13.0-100-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build"
npm ERR! node v6.10.2
npm ERR! npm v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: `nuxt build && backpack build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'nuxt build && backpack build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the spr package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! nuxt build && backpack build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR! npm bugs spr
npm ERR! Or if that isn't available, you can get their info via:
npm ERR! npm owner ls spr
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR! /app/npm-debug.log
Removing intermediate container 2e1d415c77af
The command '/bin/sh -c npm run build' returned a non-zero code: 1

This question is available on Nuxt.js community (#c1046)

I was struggling with docker yesterday and this is my working settings 👍 hope this helps
Dockerfile
FROM node:8.2.1

ENV NODE_ENV=production
ENV HOST <IP_ADDRESS>

RUN mkdir -p /app
COPY . /app
WORKDIR /app
# Expose the app port
EXPOSE 3000

RUN npm install
RUN npm run build
CMD ["npm", "start"]

.dockerignore  // Since you copy the whole folder, including node_modules could leading to problems.
node_modules
.nuxt

i try
result
@wushan

You can basically ignore the fsevents warning but that eslint-loader isn't seems right, can you build the project without  docker successfully ?

my packpage.json   There are  "eslint-loader": "^1.6.1",  Don't know why no success
{
  "name": "nuxt_base",
  "version": "1.0.0",
  "description": "Nuxt.js project",
  "author": "wangshudong<EMAIL_ADDRESS>  "private": true,
  "dependencies": {
    "nuxt": "latest"
  },
  "scripts": {
    "dev": "nuxt",
    "build": "nuxt build",
    "start": "nuxt start",
    "generate": "nuxt generate",
    "lint": "eslint --ext .js,.vue --ignore-path .gitignore .",
    "precommit": "npm run lint"
  },
  "devDependencies": {
    "babel-eslint": "^7.1.1",
    "eslint": "^3.15.0",
    "eslint-config-standard": "^6.2.1",
    "eslint-loader": "^1.6.1",
    "eslint-plugin-html": "^2.0.0",
    "eslint-plugin-promise": "^3.4.1",
    "eslint-plugin-standard": "^2.0.1"
  }
}

do  you  use  qq or  wechat?

Try wrap your app in express for production might solve the issue, and no, I don't have qq nor wechat.express-template

@wsdo For your problem I think you are setting NODE_ENV=production (or already set by base image) just before build STEP and Eslint is always enabled on nuxt.config.js....
Simple but not best solution is changing build step to : RUN NODE_ENV=build npm install
Also as a side note, we are internally working to investigate best practices for high quality and production ready docker/swarm based deployments. Stay tuned :)

@pi0 Any news/link on the docker/swarm based Nuxt.js deployments?

@pi0, I'm also interested in running Nuxt with Docker. Any update on this?

@pi0 any update?

@pi0 hey there, any updates?

Same error for me, any update for this issue? :(

After a long time of testing I finally was able to use docker with Nuxt
Dockerfile
FROM node:8.9.1

RUN mkdir -p /app
COPY . /app
WORKDIR /app

COPY package.json /app
COPY package-lock.json /app
RUN npm install

ENV NODE_ENV=production

COPY . /app
RUN npm run build

ENV HOST <IP_ADDRESS>
EXPOSE 3000
CMD ["npm", "start"]

then I run:
sudo docker build -t <image-name>
fater build is done:
sudo docker run -it -p 3000:3000 <image-name>
After that I was abel to successfully go to localhost port 3000 :)
I hope this helps

I get an error as a result of having nuxt build inside Docker. I did not write NODE_ENV, but what else should I be aware of?
I can not understand why errors are generated because npm run build on Mac OS is properly built.
Please help me.

Dockerfile
FROM node:9.11.1

RUN mkdir -p /app
COPY . /app
WORKDIR /app
# Expose the app port
EXPOSE 3000

RUN npm install && npm cache verify
RUN npm run build
CMD ["npm", "start"]

@besnikh Your Dockerfile doesn't work for me.
the RUN npm run build step errors when looking for '/package.json' (npm can't find this file)

@ChrisDillinger Happy that it helped.
I am using the following on production server with kubernetes
FROM node:10.9.0-alpine

RUN mkdir -p /app
COPY . /app
WORKDIR /app

RUN apk update && apk upgrade && apk add git

COPY package.json /app
#RUN npm rebuild
RUN npm install

ENV NODE_ENV=production

COPY . /app
RUN npm run build

ENV HOST <IP_ADDRESS>
EXPOSE 3000

CMD ["npm", "start"]

you can notice #RUN npm rebuild that is to be run first time after changing node version than after that you do not need to rebuild.
Cheeers

Have you guys managed to run it in a local environment with hot reload from Docker?


Update URL for Metacontroller

https://github.com/kubernetes/website/pull/28685

Thanks!
/lgtm
/approve


Provide TextMate scope for static functions in PHP language grammar

I'm trying to highlight static PHP functions slightly differently than non-static ones. Using a custom TextMate modifier rule.
"editor.tokenColorCustomizations" : {
    "textMateRules": [{
          "name": "Highlight static functions",
          "scope": ["meta.function.php storage.modifier.php", "storage.type.function.php"],
          "settings": {
                "foreground": "#FF0000"
          }
     }]
}

The TextMate manual 12.4 Naming Conventions suggest that:

For example if you are matching the static storage modifier, then instead of just naming it storage.modifier use storage.modifier.static.«language».

However, using the Inspect TM Scope I'm unable to find any such modifier for static functions in PHP.
(Link to my original question on stack overflow)

VSCode Version: latest 1.16.1
OS Version: OS X 10.12.6

Steps to Reproduce:

Open a file and add this php content:

<?php
class Test {
     public function Run() {}
     static public function Walk() {}
}

Open the 'Developer: Inspect TM Scope' window and look at the available modifyers for public, static,  function, Run and Walk.

Right, that won't be possible right now, because the grammar don't apply any special scopes to static vs non-static functions. That would be a feature request on the upstream grammar: https://github.com/atom/language-php


migrate Google API authentication notes from cancerDHC/ccdhmodel

There are some brief notes at https://github.com/cancerDHC/ccdhmodel#automated-generation-of-yaml
I have some very verbose notes at https://github.com/cancerDHC/ccdhmodel/issues/98

I've transferred cancerDHC/ccdhmodel#98 to https://github.com/cancerDHC/sheet2linkml/issues/6.

And #6 was closed, so I'm closing this!


Load shell environment when launching from GUI

When lapce attempts to launch sub-processes (i.e. a LSP) that depend on environment variables like a customized PATH it will fail as long as it has been launched from the GUI (at least on macOS).
This is due to the fact that i.e. macOS does not load the user's shell environment when launching an app. This can be mitigated by starting a login shell and extracting all environment variables from it. A strategy described in this comment on the Atom Editor project.
I have tested this on macOS, and it resolved all issues of lapce not being able to find LSP executables like pylsp. For Linux, I do not know if it is actually necessary or if the correct environment is available to lapce there.

@panekj
a. What do you not like about it?
b. what would you suggest as an alternative solution?

a) the fact that it's messing with environment
b) not doing it at all, plugins should correct their behaviour

a) the fact that it's messing with environment
Fair point. What about setting env variables on the Command when launching the subprocess?

b) not doing it at all, plugins should correct their behaviour
I don't see how plugins can resolve this issue. Even when a plugin accepts a serverPath to, i.e. /opt/homebrew/bin/pylsp the LSP will then not be able to locate the python interpreter, as it has also been installed via Homebrew and is not in the macOS standard PATH.

Lapce not having access to Homebrew installed executables is a high inconvenience for users on macOS.

Codecov Report

Merging #1461 (155df0e) into master (7c993f9) will increase coverage by 0.03%.
The diff coverage is 0.00%.

 Learn more


Error: ffmpeg exited with code 126

Version information

fluent-ffmpeg version: 2.1.0
ffmpeg version :ffmpeg-3.2.4
OS:  CentOS release 6.8 (Final)

Code to reproduce
fs.chmod(config.ffmpegPath, 751, (err) => {
      if (!err) {
        ffmpeg.setFfmpegPath(config.ffmpegPath)
        let stream = ffmpeg(request.get(`${config.down_voice_url}access_token=${token}&media_id=${media_id}`))
          .noVideo()
          .audioCodec('libmp3lame')
          .on('start', function (commandLine) {
            console.log('Spawned FFmpeg with command: ' + commandLine);
          })
          .on('error', (err) => {
            console.log('error when transcoding audio: ', err)
          })
          .on('end', () => {
            console.log('transcoding to mp3 success')
          })
          .pipe()
        request.post({
          url: config.storeFileUrl,
          formData: {
            value: stream,
            contentType: 'audio/mpeg'
          }
        },
          (err, res, body) => {
            callback(JSON.parse(body).data.url)
          })
      }
    })

(note: if the problem only happens with some inputs, include a link to such an input file)
Expected results
Observed results
Checklist

[ ] I have read the FAQ
[ ] I tried the same with command line ffmpeg and it works correctly (hint: if the problem also happens this way, this is an ffmpeg problem and you're not reporting it to the right place)
[ ] I have included full stderr/stdout output from ffmpeg

this is the error
error when transcoding audio:  Error: ffmpeg exited with code 126
at ChildProcess. (/root/webapp/wenda/node_modules/fluent-ffmpeg/lib/processor.js:177:22)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:191:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)

any advice ?

i get the same error. what is 126 code stand for  ?

Hi guys getting this error in the AWS Lambda too, maybe is related to the fact that the node library can't find ffmpeg installed on the computer ?

Guys, the checklist is not there just for the looks. Please do what it says.

For me this error was because of using wrong static build of Ffmpeg on Lambda, I had to use x86 instead of arm64 but you can also configure your lambda which architecture to use, for fluent-ffmpeg ffprobe is also required so don't forget to put it beside your ffmpeg.


Fix path for yoga in Podfile

Does any other open PR do the same thing?

No
What issue is this PR fixing?
#2380
How did you test this PR?

At first I had the same problem as issue #2380. After the path was changed, I am able to build example project successfully and run on simulator.

Did not fix #2380 for me

Thanks for your work. Will this be merged any soon? I still cannot make examples work.


Leak in http code on Android?

https://github.com/gregsimon/network_stress
Is the repro.  Crashes OOM pretty quickly on Android.
We can turn it into a single test.dart file to run in the Android dart cli if necessary too.
@zanderso probably knows what might be going on?

I'm told this repros on fuschia as well, so I guess this isn't Android-specific.

Correct this is not android-specific.

Thanks for the report. I have a local reproduction, and I will investigate.

Great! It happens with both http and HttpClient so issue is lower, FWIW

I ran with ASAN and did some printf debugging. The requests go through Dart's secure sockets implementation, which uses BoringSSL on Android and Fuchsia. For each connection, we allocate an SSL filter and a fresh SSL_CTX object, and load the SSL_CTX up with the trusted root certs. These objects are properly deallocated when the corresponding objects in the Dart heap are GC'd. These native objects are also correctly set up to increate GC pressure so that they don't pile up too much. The problem seems to be that allocating a fresh SSL_CTX for each connection causes BoringSSL to do many small-to-medium heap allocations, which I suspect leads to heap fragmentation.
These two things together (delayed deallocation of external objects, and heap fragmentation) look like a memory leak.
I spoke with @aarongreen who indicated that the SSL_CTX objects can be reused, so I'm going to start working on that.

It turned out that we were already re-using/sharing the SSL_CTX context, but in the referenced CL above, I switched to eagerly deallocating our SSLFilter objects, which leads to a reduction in memory footprint as the objects no longer pile up waiting for GC.
It appears that this program blowing up on Fuchsia is due to an issue with the devhost process: See MG-613 and MG-506.
@fsc8000 and I have also been running the above Flutter app for the past couple of hours (in builds w/o the eager SSL Filter deallocation) and haven't been able to reproduce any blowup. @fsc8000's debug build was stable at ~162MB for about 45 minutes, and my profile build at engine/framework ToT has been stable for >1 hour.
If someone has a local repro on Android that we can attach to with the Observatory, that would be helpful.


Docs: Nesting of  within  is not currently supported.

The React Native website is hosted on a separate repository. You may let the
team know about any issues with the documentation by opening an issue there:

https://github.com/facebook/react-native-website
https://github.com/facebook/react-native-website/issues

Please do not open a documentation issue in the core React Native repository.
###View
###     View
###  Text>{data.title}</Text
View
###View
My Code. Error Docs: Nesting of  within  is not currently supported. For android emulator.### etiketindeki  etiketi bir hata döndürür.

Nesting of <View> within <Text> is not currently supported, the documentation has been recently updated to reflect that change on iOS (before the warning was only on Android).
Closing this issue, if you find any issues within the documentation, please open an issue here instead: https://github.com/facebook/react-native-website.


Add CommandGetXdbcTypeInfo to Flight SQL Server

Which issue does this PR close?
Closes #4054

Before we merge, I noticed that the same method is also missing for the client. I am adding it as well.

IMO we should try stuff with the JDBC / ODBC drivers, record what happens

I agree -- I think this testing is covered by https://github.com/apache/arrow-rs/issues/2409. I agree it would be a fantastic additio


RemoteTaskActionClient: Fix statusCode check.

It was supposed to be checking for 2xxs. It was a little too generous and accepted 3xxs as well.

Please file an issue about RemoteTaskActionClient not having UTs and I'm :+1: after travis

Failed on DruidCoordinatorTest.testCoordinatorRun:349 expected:<1> but was:<0>

#1918

do we have other places where we do those kinds of checks?

@xvrl not that I could find with a string search


No builder for Device

Description
Currently I am using v6.2.5 but was looking to upgrade to the most recent versions.  However it seems with the change to use AutoValue it's no longer possible to construct a Device?
I currently use HostConfig.builder().devices(new Device(...), new Device(...)) to run a container with an attached device.  With the AutoValue'd version Device is abstract so can't be initialized, and the AutoValue_Device isn't accessible.  Looks like a builder needs to be setup as well but I wasn't sure if this was intentional and in new versions should be adding devices some other way.
https://github.com/spotify/docker-client/blob/master/src/main/java/com/spotify/docker/client/messages/Device.java
https://github.com/google/auto/blob/master/value/userguide/builders.md

@pmcq Thanks for notifying us. Sorry, this is a regression. We'll make a fix.

You can use the static creator method Device.create(...).

That one's package private :(

Ah, ok, I did not notice that.

I'll make a PR.

Thanks for the quick turnaround!


Update managed-identity.md

Change the name of exception as per current code

Thanks @neha-bhargava


Organized variables

For the example went ahead and organized some of the variables, making
it easier to edit for people who may want to test it out.

The test file is going to be completely changed anyway in another PR, so this is not needed.