ERR Cannot read properties of undefined

Sorry, no steps but have seen this right after reload
ERR Cannot read properties of undefined (reading 'set'): TypeError: Cannot read properties of undefined (reading 'set')
    at StickyScrollController._resetState (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/stickyScroll/browser/stickyScrollController.js:485:45)
    at StickyScrollController._renderStickyScroll (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/stickyScroll/browser/stickyScrollController.js:427:18)
    at StickyScrollController._onDidResize (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/stickyScroll/browser/stickyScrollController.js:422:14)
    at new StickyScrollController (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/stickyScroll/browser/stickyScrollController.js:61:14)
    at InstantiationService._createInstance (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/instantiation/common/instantiationService.js:130:24)
    at InstantiationService.createInstance (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/instantiation/common/instantiationService.js:101:27)
    at CodeEditorContributions._instantiateById (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/browser/widget/codeEditor/codeEditorContributions.js:125:57)
    at CodeEditorContributions._instantiateSome (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/browser/widget/codeEditor/codeEditorContributions.js:103:18)
    at vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/browser/widget/codeEditor/codeEditorContributions.js:45:18

duplicate of https://github.com/microsoft/vscode/issues/231448


Update xlsx.js

JSZip written with capitals in jszip.js and lower case in xlsx.js...

Je problem isn't that JSZip is or isn't in capitils, the problem is that in bower.json jszip isn't a dependency of js-xlsx. Therefore if you handle your dependencies automatic, the order of the files isn't correct, because it loads js-xlsx before jszip.

@lrdevries the bower entry point has been changed to a script that embeds the jszip dependency directly.
Closing because I think the underlying issue was resolved, please raise a new issue if it is still a problem


Added Reagent Grenades

About the PR

Adds Lube and Glue Grenades. Based of Cleanades
a bundle box for a common riff raff, and and disconted for the true hero's of the station the clown and mime
Why / Balance

prevent sec from following you. cut of area , slow down pursuers , mute your target and prevent it from escaping.
Technical details

smaller then the cleanade, about 1/4  so 1 cleanade can clean up 2 reagent granades.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Added fun!

grenades not granades, also the commit from your assault borg pr is mixed in here.

grenades not granades, also the commit from your assault borg pr is mixed in here.
yeah i commited it to the wrong branch , thats why the other branch had an errror

Not a maintainer here but, figured I'd mention this stuff anyways- not really enough for me to review here so comment it is!
Grenades, not Granades.  Also, somewhat, not somwat. A lot of spelling errors.
The grenades aren't really priced well. Clown's Lube grenades are way too cheap compared to the clustersoap, and the box is definitely not worth 40 TC. A freedom implanter costs less by the way, making this "Fuck with the janitor" for the same price tier of an actually useful item.
Also, do remember .ftl and the locale folder exist. Every uplink item uses them, even goob unique items.
To mention the cluster soap- Just buff it instead of adding new items that won't get used. 1 decent item's better than 3 unused items that are straight up useless.

And you might wanna throw a buff for cluster soap here or make another PR for that.

Prices also need tweaking.

Prices also need tweaking.

What prices need tweaking and what to?

lube is now 15, the box is still 40 to prevent over use.  glue can stay at 7 as it is not gonna be as problematic.


New Issue uses keyboard adjuster

Bug Report Dump (Auto-generated)

Version 1.13.0 (1191)
Device: iPhone X (iOS 11.1.1)
TestFlight: false

Nope, system handles it fine


Civic Tech Organizations Page currently includes "%20" in query when back button is used

Overview
Pressing back when navigating through the civic tech organizations page should take you to the same page you were previously on with the correct search terms.
Description

Replicate this issue

Open Civic Tech Organization's Tab: http://civictechindex.org/organizations?contrib=0&status=any

Search "Hack For LA"

Click on Hack For LA to open Organization Page

Press the back button

Observe search bar results

Action Items

[ ] Change how the search terms that are stored show up on the page

The intended outcome is to have the same page shown and for the page to show results if any

Resources/Instructions

Closing this issue as a duplicate of #1234


Internal Server Error - Services and Routes

Hello,
I am using the latest version and i am getting in most of times HTTP error 500, for service and routes.
The version of kong is 1.3.0

In the server the log of error is not so good also.
toJSON: [Function: responseToJSON],
| caseless:
  | Caseless {
  | dict:
  | { date: 'Tue, 19 Nov 2019 18:17:41 GMT',
  | 'content-type': 'application/json; charset=utf-8',
  | connection: 'close',
  | 'access-control-allow-origin': '*',
  | server: 'kong/1.3.0',
  | 'content-length': '42' } },
  | read: [Function],
  | body: { message: 'An unexpected error occurred' },
  | code: 500,
  | status: 500,
  | statusType: 5,
  | info: false,
  | ok: false,
  | clientError: false,
  | serverError: true,
  | error: { Error: got 500 response
  | at generateErrorMessage (/app/node_modules/unirest/index.js:470:23)
  | at Request.handleRequestResponse [as _callback] (/app/node_modules/unirest/index.js:473:13)
  | at Request.self.callback (/app/node_modules/unirest/node_modules/request/request.js:185:22)
  | at emitTwo (events.js:126:13)
  | at Request.emit (events.js:214:7)
  | at Request. (/app/node_modules/unirest/node_modules/request/request.js:1161:10)
  | at emitOne (events.js:116:13)
  | at Request.emit (events.js:211:7)
  | at IncomingMessage. (/app/node_modules/unirest/node_modules/request/request.js:1083:12)
  | at Object.onceWrapper (events.js:313:30)
  | at emitNone (events.js:111:20)
  | at IncomingMessage.emit (events.js:208:7)
  | at endReadableNT (_stream_readable.js:1064:12)
  | at _combinedTickCallback (internal/process/next_tick.js:139:11)
  | at process._tickDomainCallback (internal/process/next_tick.js:219:9) status: 500 },

For update some of the services, i am also getting the same error

I am also getting the same issue while creating a new service. Any solution to this problem?


chore(main): release 0.8.0

:robot: I have created a release beep boop
0.8.0 (2024-07-19)
⚠ BREAKING CHANGES

Ensure that nested structs will deeply serialize (#118)

Bug Fixes

Ensure that nested structs will deeply serialize (#118) (9216d02)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/maxveldink/sorbet-schema/releases/tag/v0.8.0 :sunflower:


hotkey.js: Add hotkey for drafts.

Added 'd' as the hotkey for opening drafts. Also, pressing enter while in drafts to edit the most recent draft.

Automated message from Dropbox CLA bot
@JPJPJPOPOP, it looks like you've already signed the Dropbox CLA. Thanks!

Hey @JPJPJPOPOP, thanks for adding this hotkey! Do you mind adding your hotkey to the keyboard shortcuts modal in templates/zerver/keyboard_shortcuts.html and documenting your hotkey in templates/zerver/help/keyboard-shortcuts.md?

@timabbott need help with node tests. Do I need to extract the functions I have in hotkey.js to drafts.js?

@timabbott ready for final review

This is awesome work, and a huge improvement!  I renamed a few variables that had abbreviations in their names and merged this.  I'm going to open several follow-up issues for things that I think we could further improve with the drafts-related hotkeys.
Huge thanks @JPJPJPOPOP!

OK opened #4333, #4334, and #4335 as follow-ups.


"Desktop composition is disabled" error when exiting VR

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

Play ETS2 in VR mode using a WMR headset (I have the HP Reverb G2)
Exit the game
Exit SteamVR
Exit the Windows Mixed Reality.

I already had this problem with version 0.78.
This happens around 1/6 since February.
Edit: Not sure that you need to play a game in VR. Maybe you just need to launch WMR and close it, idk.
✔ Expected Behavior
No crash when I quit VR mode
❌ Actual Behavior
While Windows was shutting down my headset I had this crash:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22635.0
IntPtr Length: 8
x64: True
Date: 21/03/2024 22:59:05
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealC
all(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
PowerToysReport_2024-03-21-23-49-11.zip
Other Software
SteamVR beta <IP_ADDRESS>
Mixed Reality Portal
Windows 11 Pro Insider 23H2 Build 22635.3350

/dup https://github.com/microsoft/PowerToys/issues/30507


provide cleanup guidance

Proposal
We should provide cleanup guidance to delete all google cloud resources deployed when deploying Emblem.  This should be a bash script provided alongside setup.sh
Problems this will solve
This will help:

Emblem developers cleanup resources when re-using a Google Cloud project
Allow us to create testing of setup using one consistent Google Cloud project
Ensure users are able to cleanup resources that they may be billed for.

Alternatives
At minumum we should provide users with a step by step guide to cleaning up Emblem.

cleanup.sh might be a good starting point.


Can't run pack due to Pydantic Error: RootModel does not support setting model_config['extra']

empack version: 3.1.0
Operating System: ubuntu 22.04
Compiler: emscripten

Description
Packing an environment in a GitHub action fails due to a Pydantic error from https://github.com/emscripten-forge/empack/blob/d6ff060fda66306ffb1367ac0e7f04ccad5f4233/empack/file_patterns.py#L33
The error is:
pydantic.errors.PydanticUserError: `RootModel` does not support setting `model_config['extra']`

Traceback

Traceback (most recent call last):
  File "/opt/conda/bin/empack", line 7, in
    from empack.cli.main import app
  File "/opt/conda/lib/python3.9/site-packages/empack/cli/main.py", line 2, in
    from .pack import *  # noqa: F403
  File "/opt/conda/lib/python3.9/site-packages/empack/cli/pack.py", line 6, in
    from empack.file_patterns import pkg_file_filter_from_yaml
  File "/opt/conda/lib/python3.9/site-packages/empack/file_patterns.py", line 33, in
    class FilePattern(RootModel, extra="forbid"):
  File "/opt/conda/lib/python3.9/site-packages/pydantic/_internal/_model_construction.py", line [12](https://github.com/qutip/qutip-virtual-lab/actions/runs/5914370171/job/16039628729#step:9:13)3, in __new__
    cls: type[BaseModel] = super().__new__(mcs, cls_name, bases, namespace, **kwargs)  # type: ignore
  File "/opt/conda/lib/python3.9/abc.py", line 106, in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
  File "/opt/conda/lib/python3.9/site-packages/pydantic/root_model.py", line 50, in __init_subclass__
    raise PydanticUserError(
pydantic.errors.PydanticUserError: `RootModel` does not support setting `model_config['extra']`

What I Did
You can see the failed CI run log here and the workflow config file here, but in summary, I tried running
        empack pack env \
        --env-prefix=/opt/conda/envs/web \
        --relocate-prefix=/ \
        --no-use-cache \
        --outdir=./public/empack \
        --config=empack_config.yaml

Oddly enough, this was working just 3 days prior (without any changes to the workflow file) as you can see in this successful run. I tried pinning the empack version to 3.1.0 but that didn't help.
I don't know too much about Pydantic, but willing to submit a patch if it's an easy fix.

Probably some change in pydantic.
I have a look once I am back from vacation next week.
Thanks for reporting

Fixed in #79


Setup CI test matrix

I don't want to over do the jobs, so I think 4 runs on different
Elixir/OTP versions should be about right?

Ended up going for 5 versions since that covers all stable Elixir versions.


Issue: Get Text getting Placeholder text

when i am getting text of textview which is black, it returns placeholder Text.
is there any solution for that??
thanks in advance.

Please provide me with a sample project with comments describing what's wrong and what should actually happen.


Keypoint Numbering Messed up for COCO Keypoint export

My actions before raising this issue
Read/searched the docs
Searched past issues
When exporting keypoint annotations using the COCO keypoint annotation and then importing them back into CVAT, the keypoints are in the wrong order. I don't know if this is useful, but the annotation was done using the keypoint skeleton.
Expected Behaviour
The annotations shouldn't change order when exported. It occurs on the export as I open them on another annotator to check.
This does not occur when exporting in CVAT format.
Current Behaviour
The key points are incorrect order. E.g Square is annotated as below.
1--2
|  |
4--3

When importing this back into CVAT the following may occur.
3--2
|  |
1--4

Possible Solution
There doesn't seem to be a pattern in the changes I can see, but I assume this is happening in the conversion to COCO keypoint.
Steps to Reproduce (for bugs)

Annotate images in CVAT using a keypoint skeleton
Export annotations
Import annotations onto a copy of the dataset to visualise it.

Context
I need this for a project, but this is also a bug in the performance.
Your Environment
I am using the web browser version of CVAT on a linux ubunutu 20.04 OS, with Chrome as my browser.

Hi, @jitaxis
If you are using app.cvat.ai, please share a task/job ID with us.

The task ID is:
#15200
The Job ID is:
#15642
Let me know if there is anything else required.

@jitaxis
Looks like I was able to reproduce it.
Is it something like this?
From:

To:

@yasakova-anastasia Could you please look at the problem?
@jitaxis Thank you for the report.
You can use CVAT for Images format as a temporary workaround.

@jitaxis
Looks like I was able to reproduce it. Is it something like this?
From:
To:

Yes, that is exactly it.
I need to have my annotations in COCO format for other work, so I won't be able to use CVAT for Images as a workaround for that. Do you know of a conversion script that I could use or how long this issue may take to be fixed?

I was able to reproduce this on app.cvat.ai, but not on a development server (with the latest commit). It's possible this has already been fixed.

@SpecLad
I am not sure about it.
Try to restore a task from a backup on your local instance.

Is the fix able to be used in the docker image of cvat?

@jitaxis
Just pull the changes from the repository and:

Probably --build argument is missed.

@jitaxis
Just pull the changes from the repository and:

I'm still getting the same issue with the key points as before. I pulled the latest "develop" branch and ran the line you mentioned above.
My cvat information is below:
Server version: 2.3
Core version: 7.0.1
Canvas version: 2.15.4
UI version: 1.42.2
Is there something I am missing?

@SpecLad Could you please have a look?

@jitaxis Did you use the --build option to docker-compose? At the time you posted your comment, the latest UI version should've been higher than 1.42.2, so you might have used outdated Docker images.


fix(remix-dev/vite): import hmr runtime from main script to ensure script ordering

Closes: https://github.com/remix-run/remix/issues/7863
I saw that this issue seems to be addressed by https://github.com/remix-run/remix/pull/7842, but I still observed @vitejs/plugin-react can't detect preamble after reloading repeatedly like 10 or 20 times (see reproduction in https://github.com/remix-run/remix/issues/7863#issuecomment-1793748142), so it might be that "async" attribute doesn't necessarily guarantee execution order.
I was looking at the code and it seems pre-vite HMR had already some logic to inject extra import in <Scripts /> before running client entry, so I was wondering if there is any downside to use this for vite plugin as well:
https://github.com/remix-run/remix/blob/06856da8f1d2f215e55b5adbe66dfcbf07c94e3a/packages/remix-react/components.tsx#L776-L777
One clear difference is that this would setup HMR regardless of LiveReload, but currently if users didn't include LiveReload then they will get the same error @vitejs/plugin-react can't detect preamble anyway, so it seems this won't do any harm at least (though it may not be ideal eventually).
Testing Strategy:
Using the local build of this branch in https://github.com/hi-ogawa/test-remix-vite-hmr-runtime/pull/2, I verified the same error doesn't happen on CI anymore (testing 1000 times reload).

@hi-ogawa Thanks for the PR! I had actually independently arrived at the same solution as part of #7919 which is pretty reassuring. I'm closing this PR in favour of that one since it goes a bit further, also fixing the issue where <LiveReload /> would cause an error if rendered after <Scripts>, something which isn't the case in the existing Remix compiler.


What should I do when CLS is reported but no source nodes included? How do I reduce them?

I was reading Debugging Web Vitals in the field and trying to reduce my CLS to 0 at my best.
However, 99% of my shift bugs are fixed, only remaining this weird (not set) issue, which could have a high CLS of 0.7.

(not set) is the string set by the code (created by Google, see "2" below) - It means there's no source attached to the CLS report.
I have also tested my pages during a very long enough interaction. And CLS are all fine.
1. Code I used to test my CLS on my own machine. It's injected via a self-developed Chrome extension. No CLS was reported.
let cls = 0;
new PerformanceObserver((entryList) => {
  for (const entry of entryList.getEntries()) {
    if (!entry.hadRecentInput) {
      cls += entry.value;
      console.log('Current CLS value:', cls, entry);
    }
  }
}).observe({type: 'layout-shift', buffered: true});

2. Code I used on my page. It says some (not set) source has a very high level CLS:
import {getLCP, getFID, getCLS} from 'web-vitals';

function getSelector(node, maxLen = 100) {
  let sel = '';
  try {
    while (node && node.nodeType !== 9) {
      const part = node.id ? '#' + node.id : node.nodeName.toLowerCase() + (
        (node.className && node.className.length) ?
        '.' + Array.from(node.classList.values()).join('.') : '');
      if (sel.length + part.length > maxLen - 1) return sel || part;
      sel = sel ? part + '>' + sel : part;
      if (node.id) break;
      node = node.parentNode;
    }
  } catch (err) {
    // Do nothing...
  }
  return sel;
}

function getLargestLayoutShiftEntry(entries) {
  return entries.reduce((a, b) => a && a.value > b.value ? a : b);
}

function getLargestLayoutShiftSource(sources) {
  return sources.reduce((a, b) => {
    return a.node && a.previousRect.width * a.previousRect.height >
        b.previousRect.width * b.previousRect.height ? a : b;
  });
}

function wasFIDBeforeDCL(fidEntry) {
  const navEntry = performance.getEntriesByType('navigation')[0];
  return navEntry && fidEntry.startTime < navEntry.domContentLoadedEventStart;
}

function getDebugInfo(name, entries = []) {
  // In some cases there won't be any entries (e.g. if CLS is 0,
  // or for LCP after a bfcache restore), so we have to check first.
  if (entries.length) {
    if (name === 'LCP') {
      const lastEntry = entries[entries.length - 1];
      return {
        debug_target: getSelector(lastEntry.element),
        event_time: lastEntry.startTime,
      };
    } else if (name === 'FID') {
      const firstEntry = entries[0];
      return {
        debug_target: getSelector(firstEntry.target),
        debug_event: firstEntry.name,
        debug_timing: wasFIDBeforeDCL(firstEntry) ? 'pre_dcl' : 'post_dcl',
        event_time: firstEntry.startTime,
      };
    } else if (name === 'CLS') {
      const largestEntry = getLargestLayoutShiftEntry(entries);
      if (largestEntry && largestEntry.sources && largestEntry.sources.length) {
        const largestSource = getLargestLayoutShiftSource(largestEntry.sources);
        if (largestSource) {
          return {
            debug_target: getSelector(largestSource.node),
            event_time: largestEntry.startTime,
          };
        }
      }
    }
  }
  // Return default/empty params in case there are no entries.
  return {
    debug_target: '(not set)',
  };
}

function sendToAnalytics({name, value, entries}) {
  navigator.sendBeacon('/analytics', JSON.stringify({
    name,
    value,
    ...getDebugInfo(name, entries)
  });
}

getLCP(sendToAnalytics);
getFID(sendToAnalytics);
getCLS(sendToAnalytics);

Prior to Chrome 84, the LayoutShiftAttribution interface was not available, so my guess is these reports are either:

Coming from a Chrome version 83 or lower, or
Coming from another browser that uses an older Chromium version (I believe Samsung Internet doesn't always auto update).

If you look at the browser and browser version dimensions in your report, you should be able to confirm whether or not that is the case, but either way, if the LayoutShiftAttribution interface is not available in that browser, there's not really a good way to determine the source of the layout shift. Hopefully it should be very few users though, so it shouldn't affect the 75th percentile value.

@philipwalton
Thanks very much for your fast reply. It sounds quite reasonable.


(AzureCXP) Updating epsilon symbol

Based on customer feedback on below GitHub thread , correcting epsilon symbol
https://github.com/MicrosoftDocs/azure-docs/issues/51697

@jasonwhowell for review. Thank you

#sign-off

@JasonWHowell: I'm sorry - only the author of this article, @jseb225, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

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


There are more actual tables / views in the workbench, but there are some things that are not displayed in Page Nation.

Description

There are more actual tables / views in the workbench, but there are some things that are not displayed in Page Nation.
Related Issue :

How Has This Been Tested?

Create a workbench. ( test connection : PostgreSQL-sql.improvado.io-5432 )
Verify that the table list is displayed correctly.

Need additional checks?
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 5

@eltriny
it is not a front logic issue.
it is a server-side bug.


Flows Created with new Editor not appearing in Surveyor App

Based on reports from a user with access to the new RapidPro editor, they have not been able to access flows created in the editor in the Surveyor application.
I have not tried to reproduce the bug, but will do so and provide additional details.

This was due to Rapidpro not yet knowing how to export flows made in the new editor. I've pushed a update to RapidPro (https://github.com/nyaruka/rapidpro/pull/2409) and also made a new 13.0.1 release of Surveyor which I can confirm now supports new flows.


Not sure how to get a hold of you

How are you getting your events from the background process to the main thread? I have been having a hard time wrapping my head around this and it looks as though your doing just this.

Sorry for not responding, I have been fairly busy lately.
As far as getting events over to the main thread, I've always just used a thread-safe message pump pattern for this.
A thread-safe queue lies at the core of this, so that your networking thread enqueues messages it receives onto this queue, and then your main thread at some interval dequeues all messages on this queue and processes them.

I figured it out after some time. Ill have to show you my solution once its finished sometime over the weekend. Thank you for your response.
-levon
Sent from Outlook

From: KillaMaaki<EMAIL_ADDRESS>Sent: Wednesday, November 1, 2017 10:48:58 AM
To: KillaMaaki/Netcode.IO.NET
Cc: levonravel; Author
Subject: Re: [KillaMaaki/Netcode.IO.NET] Not sure how to get a hold of you (#10)
Sorry for not responding, I have been fairly busy lately.
As far as getting events over to the main thread, I've always just used a thread-safe message pump pattern for this.
A thread-safe queue lies at the core of this, so that your networking thread enqueues messages it receives onto this queue, and then your main thread at some interval dequeues all messages on this queue and processes them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/KillaMaaki/Netcode.IO.NET/issues/10#issuecomment-341184915, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALOAc1kX3vlV02yhky12S8Kia5iLCxNQks5syK8FgaJpZM4P8xZL.


update comment to match config

Summary
Updating a outdated comment.
Test plan
The YAML still parses the same, and still validates.  Comment is updated without regressions.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #9688      +/-   ##
==========================================
- Coverage   65.11%   65.09%   -0.02%
==========================================
  Files         287      287
  Lines       12144    12144
  Branches     3009     3009
==========================================
- Hits         7907     7905       -2
- Misses       3603     3604       +1
- Partials      634      635       +1

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
94.96% <0.00%> (-1.26%)
:arrow_down:

Continue to review full report at Codecov.

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


see how many beasts collected against someone's name on the leaderboard

Is this still open? If yes, I would love to work on it.


Update with pipeline schedule support

Hopefully doing this the right way, but I figured if I open the issue I might contribute to the solution.
Tested it against our on premise gitlab instance and it seems to work. Hopefully should help implement the support for the pipeline schedules.

@gmessner,
I believe this covers it. I did notice that if I copy the exact json from the site any timestamp with a millisecond field of 000 fails the test (if it has any millisecond value it works) . But this was also the case for the existing tests.
The json converter strips the trailing 0's and then the generated and input json don't match.
ex. "next_run_at": "2017-05-19T13:41:00.000Z".
PS
I'll have one or two extra pull requests up next week, working through the functions I need for the role out at work and adding where needed.

@lpiet
There are caveats when testing the JSON, this being one of them, I should have mentioned this, my apologies if you spent much time on this.  The other big caveat is that values in the test file cannot have null as a value.

@lpiet
I am assuming that you are done with this PR?

correct

@lpiet
This has been merged, A release will happen later this week.  WIll let you know when the release is completed.
Thanks again for your contribution.


Request: Convert to tile

I'd often want to draw up some rooms on-the-fly but my dungeon maps are gigantic, so I can't really see a way to get my new room into place without it being a tile.

hi, sorry for the delay responding. Can you help me understand what exactly you're trying to do? It sounds like you have an existing map image for the scene background, but you want to use Dungeon Draw to just draw a room or two to show with the existing non-Dungeon Draw map?

yes, using dungeon draw to make smaller segments on a layer above the background so I can reposition, sort of modular.


__wbindgen_add_to_stack_pointer is not a function

 const wasm = await import('lz4-wasm');
 const data = wasm.compress(new Uint8Array([1, 2, 3]));

Uncaught (in promise) TypeError: wasm.__wbindgen_add_to_stack_pointer is not a function

You can check the usage example here: https://github.com/PSeitz/lz4-wasm/blob/main/example_project
Maybe this also helps
https://stackoverflow.com/questions/72618553/how-to-import-a-wasm-module-in-wasm-rust-and-pass-a-string-parameter


Minicap install fails

Nox (debloated) > Samsung Galaxy S9+
Android 5

Looks like it is actually not liking something on the windows file system
Can you make sure adb is connected to the device then run the following command in the command line (in the bot directory):
adb shell "getprop ro.product.cpu.abi | tr -d '\r'"

ah the tr command is not on that device.  I should just trim it after I get the result back instead

Yeah options list offers

Just to make sure see what the output of, adb shell "getprop ro.product.cpu.abi" is

Gah
Well, this will be fixed in 7.2

In the meantime, would it be bad if I ran it without the tr?

You can manually push these 2 files with adb push command:
{abi} is x86
{apiLevel} is 21 or 22 depending on if you are on android 5.0 or 5.1
left hand path is windows file system, right hand side is android
.\minicap\{abi}\bin\minicap to /data/local/tmp/minicap
.\minicap\{abi}\lib\android-{apiLevel}/minicap.so to /data/local/tmp/minicap.so

Whoops forgot to set the permissions of those two files
Need to adb shell chmod 777 those 2 files

That worked!  Well, sort of: LabMem now doesn't seem to recognize any buttons (at least not check inventory for auto start, or the OK button on a portal confirm dialogue).

The only clue I have is your frame capture delay is 30-40ms where as mine is typically 80-90ms, dipping down to 40-50ms only when the screen is mostly black.  I suspect something with minicap isn't playing well with Nox and it is returning a blank screen or something.
To test this we can run minicap with the -s switch to do a single capture to a file:
adb shell LD_LIBRARY_PATH=/data/local/tmp/ /data/local/tmp/minicap -P 1080x1920@1080x1920/0 -s ">/data/local/tmp/minicap.jpg"
And then pull it to the bot directory:
adb pull /data/local/tmp/minicap.jpg

Output from first command:

And resulting screencap:

Yup, it's getting blanks.  Alas.

Well, I guess it was kinda good the install fails on Nox since that would cause a lot of confusion.
The only thing I could do at this point is fork minicap and see if I can get it to work with Nox which would probably be a ton of work.  Thankfully, from what I can tell, the number of users running Nox is small.
Fixing the install causes another problem since I would have to check if getting a blank screen when verifying minicap is installed.  I'll at least do this and put a warning on screen the device is incompatible and auto-switch to Adb.

Yup, it definitely sounds like we should just stop using Nox.  Ah well!


Introduce rules to enable/disable features

Add carpet-style rules to the mod to enable, disable, or configure the features it offers.
Rules to be modifiable with the command /rug. Rules should be contained within a single class as static fields, which is reflected upon on startup to build valid usable arguments for the rug command. A rule change is not persisted however the default value can be configured.
Proposed rules:

/rug antiCheatDisabled <true/false> - Disables some anti-cheat checks
/rug commandCamera <true/false/op> - Enables the use of camera mode via /cs
/rug commandPlayer <true/false/op> - Enables the use of /player
/rug cameraModeDisableDamageCooldown <true/false/op> - Disable the damage cooldown that prevents players entering camera mode after recently taking damage
/rug carefulBreak <true/false> - Allows careful break to be enabled via /log carefulBreak
/rug setDefaults <rule> <value> - Sets the default value for a rule
/rug sleepPercentage <0-100> - Percentage of non-bot, non-afk¹ players that must be sleeping to skip the night.

¹ Marking players as AFK is not yet a feature

why yes this does seem like a good idea 🚀


Do we have a plan to support webTransport?

https://datatracker.ietf.org/doc/html/draft-ietf-webtrans-http3-02.txt

No plan yet, but pull requests are welcome.

So whenever this spec releases, we want to support it? It looks very slim over HTTP3, so the interfaces shouldn't change that much or at all.


unbind click event when modal closes

Need to unbind the click event on the "crop" button when the modal closes so it doesn't fire multiple times when the modal re-opens.
@DoSomething/front-end

Sneaky little events! :+1:


issue-107 : replaced load file with ExecutionAndTraceLog+config

this also includes Elasticsearch configuration

Fixes #107
Signed-off-by: Dana Sunalova<EMAIL_ADDRESS>

Reviewer is @vanithavalluripalli9


[Program]: Write a GO program to convert time from 12-hour to 24-hour format

Description
Write a GO program to convert time from 12-hour to 24-hour format
Input  : 12:05:45PM
Output : 12:05:45

How to contribute

Save the solution in program/program/convert-time-from-12-hour-to-24-hour-format/program/convert_time_from_12-hour_to_24-hour_format.go file

!assign


Option to exclude files / directories

Lsp needs to parse dependent libraries (e.g. vendored modules in go), but any issues reported with the code are most likely not relevant to the Trouble user. It would be nice if files / directories can be excluded based on e.g. a glob pattern.

I would like to exclude *.min.js. I have some embedded in a project and having them appear in the preview window then get highlighted causes nvim to take minutes to become responsive again.

Trouble just shows Neovim diagnostics. I'm not planning to add any filtering capabilities. You should properly configure your lsp server


Test code-signing with self-signed certificates

Summary
Test code-signing with self-signed certificates.
This PR is only for testing code-signing on jenkins. So this should NOT be merged.
Issue link
#852

v4.2.0 was directly code-signed by electron-builder instead of osslsigncode. However the problem was not resolved.


fix(tree-sitter): mark helper functions static

The scanner is not generated, and this resolves conflicts with other bundled files/projects (e.g. other parsers) that use functions w/ the same names, it should only be available in this file.


fix: use a default value when Concurrency is not specified

Fixes #289
Signed-off-by: Sylvia Lei<EMAIL_ADDRESS>

Codecov Report

Merging #296 (ad7cbd1) into main (6d3ce22) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
- Coverage   71.20%   71.19%   -0.01%
==========================================
  Files          37       37
  Lines        3480     3479       -1
==========================================
- Hits         2478     2477       -1
  Misses        749      749
  Partials      253      253

Impacted Files
Coverage Δ

copy.go
57.69% <100.00%> (-0.21%)
:arrow_down:

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


No active package flutterfire_cli

I am trying to use the CLI as stated in the doc, however I am stuck at the step flutterfire configure.
My output is as below:
bash-3.2$ flutter pub global activate flutterfire_cli
Resolving dependencies...
+ ansi_styles 0.3.1
+ args 2.3.0
+ charcode 1.3.1
+ cli_util 0.3.5
+ collection 1.15.0
+ dart_console 1.0.0
+ deep_pick 0.10.0
+ ffi 1.1.2
+ file 6.1.2
+ flutterfire_cli 0.1.1+2
+ interact 2.1.1
+ matcher 0.12.11
+ meta 1.7.0
+ path 1.8.1
+ petitparser 4.4.0
+ platform 3.1.0
+ pub_semver 2.1.0
+ pubspec 2.0.1
+ quiver 3.0.1+1
+ source_span 1.8.1
+ stack_trace 1.10.0
+ string_scanner 1.1.0
+ term_glyph 1.2.0
+ tint 2.0.0
+ uri 1.0.0
+ win32 2.3.3
+ xml 5.3.1
+ yaml 3.1.0
Installed executable flutterfire.
Warning: Executable "flutterfire" runs "bin/flutterfire.dart", which was not found in flutterfire_cli.
Activated flutterfire_cli 0.1.1+2.
bash-3.2$ flutterfire configure
No active package flutterfire_cli.

This is my flutter doctor output:
flutter doctor output
bash-3.2$ flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 11.6 20G165 darwin-x64, locale
    fr-FR)
    • Flutter version 2.8.1 at /Users/seb/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (3 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/seb/Library/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/seb/Library/Android/Sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.5.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.0.0.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.32.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.110

! Doctor found issues in 1 category.

I tried downgrading the version of the flutterfire_cli package but with no luck.
Any idea on how to resolve this?

Use dart pub global activate flutterfire_cli, instead of flutter pub global activate flutterfire_cli

Thank you it worked!
I used the flutter command instead of dart sometimes ago when I faced issues using dart (problems with my PATH that I fixed recently).
I thought that running flutter was somehow equivalent to running dart. Apparently it is not.


Warning: rename(,.pdf) / unlink(.pdf): No such file or directory

Warning:
Warning: rename(,.pdf): No such file or directory in /Applications/XAMPP/xamppfiles/htdocs/p/vendor/mikehaertl/php-tmpfile/src/File.php on line 41
Warning: unlink(.pdf): No such file or directory in /Applications/XAMPP/xamppfiles/htdocs/p/vendor/mikehaertl/php-tmpfile/src/File.php on line 52

Code:
<?php

// load Composer
require __DIR__ . '/vendor/autoload.php';

use mikehaertl\wkhtmlto\Pdf;

$options = array(
    "binary" => "/usr/local/bin/wkhtmltopdf",
    'no-outline'
);

$pdf = new Pdf($options);
$pdf->addPage('/index.html');
$pdf->send();

?>

Weird. Can you do some debugging? The relevant lines are in vendor/mikehaertl/php-tmpfile/src/File.php here:
https://github.com/mikehaertl/php-tmpfile/blob/master/src/File.php#L30
This line is called from here:
https://github.com/mikehaertl/phpwkhtmltopdf/blob/master/src/Pdf.php#L239
As you see, there is a call to tmpfile() in the constructor of File. Something seems to go wrong in your setup.

Ok, it's probably that you don't have write permission to the tmp directory that sys_get_temp_dir() returns. Check the output of echo \mikehaertl\php-tmpfile\File::getTempDir() and if PHP has write permission to that directory.

Sadly im not really familiar with PHP, usually developing in Java. Other infos to my setup: OSX 10.11, Apache running under XAMPP, CHMOD 777 set to the whole htdocs folder.

Where's the temp directory located, which is used by PHP?

Check the output of echo \mikehaertl\php-tmpfile\File::getTempDir() and if PHP has write permission to that directory.

Ok, i'll try that and update the issue with the relevant information a litte later.

Please check the documentation of tempnam() here: http://de2.php.net/manual/en/function.tempnam.php

Creates a file with a unique filename, with access permission set to 0600, in the specified directory. If the directory does not exist or is not writable, tempnam() may generate a file in the system's temporary directory, and return the full path to that file, including its name.

So usually this method should never fail. Because if the provided directory is not writeable, it will fall back to the system's temp dir, which should always be writeable. So something must be wrong with your setup. Unfortunately I can not help you with that.
I'm closing this for now. Feel free to reopen, if you still think, this is a bug in this library.


Update README.md

I am sorry, but I am not able to find your Contributions, i.e. adding a Linear Search code. Please post the link of the PR.
In case if you haven't you can always add the code. :)

I am adding it in a min or two
On Tue, 2 Oct 2018, 10:07 pm Ayan Banerjee<EMAIL_ADDRESS>wrote:

I am sorry, but I am not able to find your Contributions, i.e. adding a
Linear Search code. Please post the link of the PR.
In case if you haven't you can always add the code. :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ayan-b/Linear-Search/pull/76#issuecomment-426343531,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVTrO9KJU5tkD6scDxNmxDId_EeL5YBAks5ug5ZZgaJpZM4XEafa
.


Compile changes if stream changes

If the default stream or device change then we recompile the function.
I looked into the alternative of updating the stream dynamically but it was quite a bit more complicated, especially considering the fact that we often use different primitives for different devices.
Another option is to update the compiled primitive only if the default device changes and dynamically update the stream. This is still complex since we need to track if the stream was set as a default or set explicitly. Presumably if you set it explicitly then you don't want it to change under your feet.
I also removed a few constructors for bound types that I think don't make sense to be explicitly bound.
Closes #1624

Just FYI there is a flaky test on linux. It's unrelated to this PR. I'll look into it.


Do not wipe (HTML) reports directory when creating a new mutation report

Is your feature request related to a problem?
I sometimes want to compare two mutation reports (e.g. to see which mutants I killed after adding a test) but unfortunately I can't put the copied report in the same directory because the HTML reporter wipes the directory before writing the new report (I don't use nor did I check other reporters that write to disk).
Describe the solution you'd like
Don't clean the reports directory, leave any files in the directory and only replace the report file(s). Whether that's overwriting (in place) or replacing (removing-then-adding) is irrelevant to me.
Describe alternatives you've considered
None.
Additional context
I did some minor investigation before opening this issue. All I was able to find is that this was written (effectively) as it is currently found in the source code, in https://github.com/stryker-mutator/stryker-js/pull/290 and (the behaviour) hasn't changed since. Unfortunately I didn't see an explanation for why the entire folder is wiped in that Pull Request or any of its commits. If I had to guess why it is this way I'd guess it is because this is the most straightforward way to implement this.

Good point. The reason for it to delete the entire dir is for historic reasons, since we used to have an HTML report that existed out of multiple HTML files.
For v6, I want to get rid of the directory entirely. Instead writing directly to reports/mutation/mutation.html (or something like that).
Timeline for v6 is March / April. Feel free to send in a PR if you want it fixed sooner.

It's implemented and ready to go with StrykerV6 release. Closing in order to keep track.


Access to language server in extension

I want to write an extension for JS files. Is it possible to access the output of the language server like knowing if the given word is a variable.

We do not expose this currently. Existing plugins get around this by using their own version of TypeScript to parse files: https://github.com/Microsoft/vscode/blob/master/extensions/extension-editing/src/extension.ts#L9
If you are writing an extension that augments JavaScript intellisense , consider creating a TypeScript server plugin instead: https://github.com/Microsoft/TypeScript/wiki/Writing-a-Language-Service-Plugin This is the preferred approach to add new editor functionality to javascript files
Let me know if you have any questions about these options


"pause" command not available on linux

Line 29 in console_exec.py uses "pause" command.
Use "read" instead on linux.

Revisiting this project today. I'm getting it up on ST2 and ST3.
Did you ever end up adding support? If so, mind sharing what you did?
Thanks!

Actually, I went ahead and added the read fix. Mind looking it over?
Let me know if it still doesn't work for you!

read is not be available if BASH is not the default shell.
I just try this plugin on my Lubuntu 15.10 with TCSH as the default shell.
The console just blinks and gets read: Command not found.

@jfcherng Go figure. Any ideas?

On my case, it seems that xterm -e python3 -u /path/to/test.py ; read -p "Press [Enter] to continue..." will be interpret as xterm -e python3 -u test.py ; and read -p "Press [Enter] to continue...".

xterm -e python3 -u /path/to/test.py ; runs in xterm.
read -p "Press [Enter] to continue..." runs in a background process.

So currently, even if the default shell is BASH, xterm won't pause...
(I change my default shell to BASH and it doesn't pause.)
I am thinking that writing a script pause.sh like this:
#!/bin/env bash
read -p "Press [Enter] to continue..."

And construct this string xterm -e "python3 -u /path/to/test.py ; bash '/path/to/pause.sh'"
which makes python3 -u /path/to/test.py ; bash '/path/to/pause.sh to be the command passed to xterm.
But /path/to/test.py and /path/to/pause.sh may need to do escapes?
=========
By the way, on line 28 pause = [';', 'read' '-p' '"Press [Enter] to continue..."'] misses two commas.
It's pause = [';', 'read', '-p', '"Press [Enter] to continue..."'], still not working, though...

Oh nice, I really like that solution. Do you think that could be run without the new file? Say, wrapped in bash -c ""? Or is #!/bin/env bash what's important here?

misses two commas

Ugh! How'd that get by? Haha, thanks for pointing that out. Fixed in be1e38401a91b1ff0664c9a1582b8bd5605437c0.

#!/bin/env bash is not important here if the script is not used as a executable.
It states what executable should be used to run the script.
Since we use bash /path/to/the/script, it can be removed.
bash -c "blablabla" should work too,
But I am not sure will quotes make the escape complicated in blablabla.

Ok cool. Thanks for clarifying all of that.

But I am not sure will quotes make the escape complicated

Yeah, that's why I didn't start out that way. Based on what you're saying, it'd be a good addition.
I just opened #4 for this. Want to take a shot at it?


could you please add a compact mode for pc mouse user, begging you my dear M$

please

the title bar height

side bar width

the editor tab height

the panel title height

bump

@heartacker does zoom level not satisfy your requirements?

@heartacker does zoom level not satisfy your requirements?

no my  pc is  with  small  screen
and i find  that  the ui is Overstretch，like vs 2019,we could  found that there  is more space for  code  editor

cc @misolori

Also, please subscribe to https://github.com/microsoft/vscode/issues/115641 as that explores a density option


Expose subnet_ids var to root module

Description
This PR exposes the subnet_ids optional var to the root module, so that it can be specified if necessary.
Documentation
Documented in the variable descriptions.
TODOs
Please ensure all of these TODOs are completed before asking for a review.

[x] Ensure the branch is named correctly with the issue number. e.g: feature/new-vpc-endpoints-955 or bug/missing-count-param-434.
[x] Update the docs.
[x] Keep the changes backward compatible where possible.
[x] Run the pre-commit checks successfully.
[x] Run the relevant tests successfully.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.


Use ninja build system.

Unclear why, but using ninja might help with the flakyness we've been seeing on the large VMs.  It may also speed up builds too.  As a side affect, ninja logs can be used to easily profiler the build times (see https://buildbloat.neocities.org/)

I'm going to re-run kokoro a couple times to see if it is more stable, or just a coincidence.  Ninja does appear to run faster than the Makefiles, so this is probably worth applying to the other jobs too.

Generally LGTM.
Is Travis is still running with make?

Right now only kokoro xc7 is running with ninja, everything else is Make.

(BTW I do think we want to have something continuing to test with just make.)


L7 bugfixes

When wallis_fill filtering images, the variance can end up less than zero (but very close to zero), leading to NaNs in the valid data, which blows up downstream and result in zero valid pixels
fix the boolean operator used when combining image zero masks when wallis_fill filtering images

@alex-s-gardner I did confirm we'll want these changes in production. Ideally, we'd cut a v1.5.1 release with these changes, and then hyp3-autorift can use the "canonical" version from conda-forge. I can move faster than that, however, with just these changes.

I suggest reviewing the code related to this operator and implementing a more robust check to prevent negative variances. Additionally, ensure that the valid data is handled correctly, avoiding zero valid pixels. This fix will enhance the stability and reliability of the wallis_fill filter.
Feel free to share any specific code snippets or files related to this issue, and we can work together to make the necessary adjustments. Collaboration is key in open source, and your contribution in identifying this problem is greatly appreciated. Let's work together to resolve it and improve the project's overall quality.

I suggest reviewing the code related to this operator and implementing a more robust check to prevent negative variances. Additionally, ensure that the valid data is handled correctly, avoiding zero valid pixels.

@itstalmeez I'm happy to review a pull request if you do any of the above or have further suggestions.


how to use multi gpu on one machine?

I got two M40 gpu on one machine, however, while I set batch_size=2048, it always report GPU out ot memory, but the another gpu still got  11443MiB unused

Unfortunately, Neural Monkey does not support multi-gpu setup just yet. You can track issue #491


Warnings when using blt in cmake projects that use a new style project command

When you add a project using this style of project command:
project(project_name VERSION "x.y.z")
Our google test and google mock sub projects trigger the following warnings
CMake Warning (dev) at blt/thirdparty_builtin/googletest-release-1.8.0/CMakeLists.txt:3 (project):
  Policy CMP0048 is not set: project() command manages VERSION variables.
  Run "cmake --help-policy CMP0048" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.

  The following variable(s) would be set to empty:

    PROJECT_VERSION
    PROJECT_VERSION_MAJOR
    PROJECT_VERSION_MINOR
    PROJECT_VERSION_PATCH
This warning is for project developers.  Use -Wno-dev to suppress it.

We can update the project commands in those files, but would that complain when people use the old style project command?

Thankfully, it turns out there is a simple fix:
https://github.com/google/googletest/pull/782/files
(this PR hasn't been pulled into google test yet)
I was trying to add the policy commands in the wrong places (I missed the top level project)


Wrong tests for search

Tests should use values from sample_metadata_list, not "fish attack painting"
For a generated model with "title: type: fulltext+keyword", the search tests (both resource and service) fail in search.
The generated test should not have the correct alternative hard coded, but should use the value from sample_metadata_list
(and check that the correct number of records is returned, if the value in the list is duplicated)

Also, the paths for the tests should be generated from the model, ignoring non-searchable stuff


After above command just enter path of combo file explain please

Drag and drop your combo file, when asked to enter combo file path. Or Right click on file and find option to 'copy as path', now paste it.


Create unit tests

As a Developper it is preferable to make Unit tests, to make sure the logic behind the functions and methods, are not broken during development.
acceptance criteria:
Make Unit tests, that tests the methods just implemented in the former issue

Added the desired unit tests.


Disable added llvmjit tests with github issue

These tests were added after https://github.com/google/iree/commit/4e47a49bb3ab7babe00e90b5f4caefa26aff8320 and weren't disabled by this workaround.

./integrations/tensorflow/e2e/math/BUILD:547: unused-variable: Variable "LLVM_FAILING_MULTIPLE" is unused. Please remove it.
To disable the warning, add '@unused' in a comment. (https://github.com/bazelbuild/buildtools/blob/master/WARNINGS.md#unused-variable)


Write a simulator to test different storage formats and cleaning strategies

Something like the simulator described here: https://arxiv.org/pdf/2005.00044.pdf.
Per discussion at: https://discord.com/channels/869525774699462656/869526396878323733/873033240271810560

@skyzh just for FYI
@hlinnaka any further work needed here ?

the thing I have in the compaction PoC is a compaction simulator. https://github.com/neondatabase/neon/pull/4539 As tiered compaction make decisions purely based on sizes, we can estimate write / read / space amplification easily with such compaction simulators.

PR #6830 / #5234 has added a simulator, can this be closed now?


Get-AdminServiceEndpointType returning Internal

Currently testing the 4.0.2 script out with MECM 2006 through the CMG. Using the script internally I have no issues but I have a client on the internet that is returning its AdminServiceType as Internal and therefore cannot download the driver package.
Going through the script and looking at how it is determined I see what is going on. I see that the script gets all the active MPs with this line
$ActiveMPCandidates = Get-WmiObject -Namespace "root\ccm\LocationServices" -Class "SMS_ActiveMPCandidate"
and then counts both the Internal and External mps with the next two lines
$ActiveMPInternalCandidatesCount = ($ActiveMPCandidates | Where-Object { $PSItem.Type -like "Assigned" } | Measure-Object).Count
$ActiveMPExternalCandidatesCount = ($ActiveMPCandidates | Where-Object { $PSItem.Type -like "Internet" } | Measure-Object).Count
On the client I'm working with, I get counts for both Internal and External. The scripts appears to default to "Internal" if the "$ActiveMPInternalCandidatesCount" is not 0.
				switch ($ActiveMPInternalCandidatesCount) 				{ 					0 { 						if ($ActiveMPExternalCandidatesCount -ge 1) 						{ 							$Script:AdminServiceEndpointType = "External" 						} 						else 						{ 							Write-CMLogEntry -Value " - Unable to determine AdminService endpoint type, bailing out" -Severity 3 							$ErrorRecord = New-TerminatingErrorRecord -Message ([string]::Empty) 							$PSCmdlet.ThrowTerminatingError($ErrorRecord) 						} 					} 					default { 						$Script:AdminServiceEndpointType = "Internal" 					} 				}
The client appears to function, I can install applications, CBs run, etc. Is it possible I have something configured incorrectly on my client or is this a possible expected behavior for a client?

I have the same problem...

I am seeing this same issue.


Unsupported state transitions

In extractPTP4lEventState() we are not handling some of the state transitions, even though it is mentioned in the comments.

// ExtractPTP4lEvent ... extract event form ptp4l logs
//
//	    "INITIALIZING to LISTENING on INIT_COMPLETE"
//		"LISTENING to UNCALIBRATED on RS_SLAVE"
//		"UNCALIBRATED to SLAVE on MASTER_CLOCK_SELECTED"
//		"SLAVE to FAULTY on FAULT_DETECTED"
//		"LISTENING to FAULTY on FAULT_DETECTED"
//		"FAULTY to LISTENING on INIT_COMPLETE"
//		"FAULTY to SLAVE on INIT_COMPLETE"
//		"SLAVE to UNCALIBRATED on SYNCHRONIZATION_FAULT"
//	     "MASTER to PASSIVE"
func extractPTP4lEventState(output string) (portID int, role types.PtpPortRole, clockState ptp.SyncState) {}

handled ones are below
clockState = ptp.FREERUN
	if strings.Contains(output, "UNCALIBRATED to SLAVE") ||
		strings.Contains(output, "LISTENING to SLAVE") {
		role = types.SLAVE
	} else if strings.Contains(output, "UNCALIBRATED to PASSIVE") ||
		strings.Contains(output, "MASTER to PASSIVE") ||
		strings.Contains(output, "SLAVE to PASSIVE") ||
		strings.Contains(output, "LISTENING to PASSIVE") {
		role = types.PASSIVE
	} else if strings.Contains(output, "UNCALIBRATED to MASTER") ||
		strings.Contains(output, "LISTENING to MASTER") {
		role = types.MASTER
	} else if strings.Contains(output, "FAULT_DETECTED") ||
		strings.Contains(output, "SYNCHRONIZATION_FAULT") ||
		strings.Contains(output, "SLAVE to UNCALIBRATED") ||
		strings.Contains(output, "MASTER to UNCALIBRATED on RS_SLAVE") {
		role = types.FAULTY
		clockState = ptp.HOLDOVER
	} else if strings.Contains(output, "SLAVE to MASTER") ||
		strings.Contains(output, "SLAVE to GRAND_MASTER") {
		role = types.MASTER
		clockState = ptp.HOLDOVER
	} else if strings.Contains(output, "SLAVE to LISTENING") {
		role = types.LISTENING
		clockState = ptp.HOLDOVER
	}

So is this intentionally left out or need to be implemented?
Any reasons for not adding those cases? if left out I can add those cases in a patch.
Please let me know.
Thanks

I think there is more logic to understand, my problem was because of a wrong ptpconfig.
Closing the issue.
Thanks


seq2ffv1.py - check rawcooked process before run

add clean_start() for every subprocess.call('rawcooked',...) to avoid occasionally RawCooked freezing during script running.
It checks whether 'rawcooked' is in the pid list and kill it every time before run subprocess.call().
This has been tested to work fine.

Changes made as it didn't work on Window due to different process name.

The latest script has been test on both Mac and Windows. Ready to be merged.


SideMenu is visible below the component it is rendered on

Hi,
I have created the SideMenu as per the example here, however, the SideMenu is visible beneath the component I am rendering it on, even though the toggle state may be closed. I can even see it between screen transitions, until the main component gets rendered over it.
How can I fix this?

@aejaex I am facing the same issue. I have found a work around. I hope it helps you.
I added props isVisible to the Menu component which will control if the menu should be displayed or no.
I am passing the this.state.showSideMenu as the value for isVisible props which controls the SideMenu's isOpen props.
render() {
   const menu = <Menu isVisible={this.state.showSideMenu} />;
   return (
     <SideMenu
       menu={menu}
       menuPosition="right"
       isOpen={this.state.showSideMenu}
     >
        <Text>Welcome to Side Menu Demo</Text>
    </SideMenu>
);

###Menu.js
import React, { Component } from 'react';
import { View, Text } from 'react-native';
export default class Menu extends Component {
  constructor(props) {
    super(props);
    this.state = {
      isVisible: props.isVisible
    }
  }

  componentWillReceiveProps(nextProps) {
    if (nextProps.isVisible != this.props.isVisible) {
      this.setState({isVisible: nextProps.isVisible});
    }
  }

  render() {
    return (
      <View style={{flex: 1}}>
        {
          this.state.isVisible ?
            <View style={{flex: 1}}>
              <Text>I am side menu content</Text>
            </View> :
          null

        }
      </View>
    );
  }
}

Does someone have a solution to this? I have my side menu wrapped around my App.js which makes it accessable in other files. When i swipe in my app, the menu is visible below the components

I switched over to NativeBase's Drawer, which I found easier to use and more robust.

@aejaex
You should set your content with background color like white or another solid color to hide the side menu when it doesn't open.

Any solution? I don't understand because the menu is show when the component is rendering. I've isOpen = false. I think is a terrible bug!

I get SideMenu by ref,then invoke the ”openMenu" function of the ref

@aejaex I am facing the same issue. I have found a work around. I hope it helps you.
I added props isVisible to the Menu component which will control if the menu should be displayed or no.
I am passing the this.state.showSideMenu as the value for isVisible props which controls the SideMenu's isOpen props.
render() {
   const menu = <Menu isVisible={this.state.showSideMenu} />;
   return (
     <SideMenu
       menu={menu}
       menuPosition="right"
       isOpen={this.state.showSideMenu}
     >
        <Text>Welcome to Side Menu Demo</Text>
    </SideMenu>
);

Menu.js
import React, { Component } from 'react';
import { View, Text } from 'react-native';
export default class Menu extends Component {
  constructor(props) {
    super(props);
    this.state = {
      isVisible: props.isVisible
    }
  }

  componentWillReceiveProps(nextProps) {
    if (nextProps.isVisible != this.props.isVisible) {
      this.setState({isVisible: nextProps.isVisible});
    }
  }

  render() {
    return (
      <View style={{flex: 1}}>
        {
          this.state.isVisible ?
            <View style={{flex: 1}}>
              <Text>I am side menu content</Text>
            </View> :
          null

        }
      </View>
    );
  }
}

It worked for me with small glitch. It will hide the sidemenu once you click the close, then animation goes in blank area.


Use links in test meta description

Is your feature request related to a problem? Please describe.

Currently, links don't work in the test meta description. When a test fails. I want to have a link to a data source directly in the test description to fix the error in source data ASAP.
Describe the solution you'd like
Allow to properly render HTML <a> tag in test meta description
Describe alternatives you've considered
Support markdown instead of HTML.
Additional context

slack request 1
slack request 2

Would you be willing to contribute this feature?
yes

@oravi @Hadarsagiv
We have 2 different requests for URLs in sources node info.
Maybe just like we are working on a dedicated node info for exposure we should have one for source?
I can imagine this expanding to links that we capture via integrations, but start with giving the users option to configure (like in exposures)

Hi all,
I am also interested in this feature, but exposing URLs to a Google Sheet that will contain the results in a human-readable way.

I was also directed to this issue to make a related request.
On test failures I want to embed links in the test failure row data (through the data that's selected from the test failure itself).
E.g I would like something like this image to show a clickable link (either automatically on an https:// link or by me embedding HTML like the second, I don't mind!)


Update 3 Casks: depends_on macos

Use :symbol

==> 3 modified casks: camtwist-beta sourcetree-alpha sourcetree-beta
audit for camtwist-beta: passed
audit for sourcetree-alpha: passed
audit for sourcetree-beta: passed

>>> brew cask style Casks/camtwist-beta.rb Casks/sourcetree-alpha.rb Casks/sourcetree-beta.rb
3 files inspected, no offenses detected


Can't update account information on Oauth connected account

Just getting this all setup for the first time. I have the app up and running.
I just created an Oauth app for Github and was able to sign in. However, when I got to Account I can't update the name or email address. The text fields can be typed in, but the "Update Name" and "Update Email" buttons are disabled.
I did a quick scan through Supabase but I didn't find any settings that seemed to address this.
Any idea how to fix this?

https://github.com/vercel/nextjs-subscription-payments/pull/278


Change dashboard_url at runtime

We can change menu at runtime, but it would be great if we could change other options, like dashboard_url.

This sounds a bit like an X-Y Problem. Why would you want to change the dashboard uri? The dashboard is always the dashboard right?
In the meantime, if you must, you can always publish the views and create this functionality yourself.

Hey @gpressutto5, you can override the template and add logic to change the dashboard URL at runtime.


Add Repo.all_by as a mirror to Repo.get_by

I'm afraid we will soon need Repo.update_by and Repo.delete_by.
Wouldn’t it be better to have a Ecto.Query.by_fields/2 function to generate a query, and use regular Repo.one, Repo.all, etc. with it?
It would be used like this:
Ecto.Query.by_fields(MyModel, fields) |> Repo.all
Ecto.Query.by_fields(MyModel, fields) |> Repo.one

This would also remove the need for Repo.get_by. It also promotes using composable queries, and allows easy use of preloads, etc. if needed.

The Ecto.Query.by_fields/2 option sounds great! Instead of by_fields/2, how about we overload Ecto.Query.where with this behavior?

I'm not sure that's doable, because where is a macro, and I think a normal function would be enough here.

Is Ecto.Query.by_fields(MyModel, foo: 123) really that much better than from(m in MyModel, where: m.foo == 123)?

@ericmj it is not the point of which one is better but in allowing things to be dynamic. Imagine I want to convert params data in Phoenix to a query. If we can do it dynamically i can do:
attrs =
  for {k, v} in params,
    k in ~w(age title gender),
    do: {String.to_atom(k), v}
Profile |> by_fields(attrs) |> Repo.all

I don't like by_fields as a name but the idea is definitely worth exploring.

:heart: this!
Names are hard.
Profile |> matching(attrs) |> Repo.all
Profile |> filter_for(attrs) |> Repo.all
Profile |> filter_by(attrs) |> Repo.all
Profile |> query_for(attrs) |> Repo.all

eh?

Right, we can't overload where with this.
Of the options @parkerl suggests, filter_by/2 sounds the most suitable to me.

@gjaldon I like that one too but is it confusing that it won't be running the query and returning results but rather creating a query?

Is anyone picking this one up? I'd love to take a stab.

We haven't decided an API yet, so let's wait a bit :)
--
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Director of R&D

@josevalim ok I'll be patient. ;-) Was just wondering if some code might facilitate the conversation?

It seems that you would be able to use where As a regular function. I am fairly certain that would work, especially since it's a different arity than the current where/3 macro. Since this is doing what the where function is doing, but with different parameters, I think it makes sense to keep the function name. It's clear and simple IMO.
def completed(query) do
  # a lot cleaner and super easy to understand what's going on
  query |> where(completed: true)
end

And of course it allows for the types of things @josevalim mentioned above

@paulcsmith you are right, that's a great pick. The question is if we still need get_by?

Ok I  like the idea but maybe I'm missing something. Without the binding how do we know what model to build the where clause against?
Also, would this expand in keyword syntax?

@josevalim speaking of get_by can I ask a potentially stupid question? Why does it throw an exception when multiple results return? If it just returned all the results (as I expected it to do the first time I used it) then we are done here by just taking two lines out of a case statement.

@parkerl it will always apply to the model in "from". Btw, implementing this feature using where is actually quite complicated because it requires changes to the query builder but I think it is the way to go.
Regarding the second question: the goal of get functions is to return a single result, so we should not change its behaviour.

@josevalim ok that makes sense.
On the second point, I was expecting get and get_by to function like AR's find and find_by`. I need to stop doing that. ;-)

@josevalim @paulcsmith I'd still be happy to take a stab at this unless you already have code in the works.

I don't have anything in the works. :)

Ok I will see what I can come up with.

@josevalim looking for feedback on the wip above. Again, I must be missing something because this naive implementation seems to work. I spent some time digging around looking for more complexity. Can you expound on:
"Btw, implementing this feature using where is actually quite complicated because it requires changes to the query builder but I think it is the way to go."
I drove this out with integration tests in the repo cases but I know that they don't belong there. Where should this be tested? In query_test.exs I could test that the query returned has the correct wheres.

@parkerl if we are adding it to where, I should look like this:
query |> where([], [foo: 1]) # i don't care about the bindings
query |> where([p, q], [foo: q.foo])

If we support the syntax above, then we can do:
(from Post, where: [foo: 1])

Or:
(from Post, ..., where: [foo: 1.foo])

But we should also support:
where = [foo: 1.foo]
(from Post, ..., where: ^where)

And all of those require changing the query builder API.

@josevalim that is a cool idea! That'll be really nice. I assume that what @parkerl did should also supported? So you can leave the binding parameter out completely?

@paulcsmith it should not be support. We generally don't support versions without binding to keep the API simple, specially since the keywords syntax is prefered.

@josevalim I see your point, but I think that query |> where([], [foo: 1]) reads very strangely. I think it would be weird to try to explain to someone why you still need to pass in an empty list if you don't care about the binding? Anyway, just a thought :) Even with the binding I will be quite happy to try this out in some of our apps!

All macros need the binding, so at the moment you go into the macro syntax, you need to explain that each macro operation requires the binding. :)

I'm confused, @parkerl wrote a version that seems to work without passing an empty list? I'm probably just misunderstanding though. I admit I don't know the internals of how queries work very well

All I am saying is that all operations where, order_by, select today expect the binding so I don't see why we should have a special variation of where that does not. Doing so would be the exception and the one likely confusing.

Ah I see what you're saying :) That makes sense. I thought you meant it wasn't possible. Thanks for the explanation!
One thing that could be done is to allow where, order_by, select and friends to also have an optional binding, but maybe that's a bit too crazy :)

@josevalim here is my wip on this. However, I think I'm still not understanding your vision here. I ended up with this in Filter so that I could iterate to build multiple "wheres". I attempted to push this down in the builder but escape doesn't seem want to return multiple statement. Is that a part of what you see changing?
Another question, when you say this above
(from Post, ..., where: [foo: 1.foo])

are you saying where the binding "foo" has the value "1"? Or is it that the binding in position "1" has the value "foo"?
Thanks,

@parkerl sorry, I have updated the code samples above. The 1 was a mistake.
Your WIP is a very good starting point! I would do the following changes next:

you don't really need the first binding. The first binding is always {:&, [], [0]} in the AST
instead of reducing on the query by calling build multiple times, you can just join the different fragments with and. Something like Enum.reduce parts, &{:and, [], [&1, &2]}
Move that code out of build and into escape (still in Filter)

Good work!

Thanks for those updates. That makes it more clear. I will make those changes asap.

@josevalim it had my back against the wall for a time but I have it working with your edits above. Care to take a look? What's next?

Thanks for working on this @parkerl. I'm stoked to try this out.
I've been thinking a lot about this and I really believe that the binding should be truly optional. If you don't want it, remove it completely. Trying to explain that you have an empty list there because you don't need a binding doesn't make sense to people. Maybe I'm explaining it wrong, but IMO it is not intuitive.
To make this work well across the Query API, I think that making the binding optional for order_by, limit, where, and whatever other query functions are needed would make writing queries so much simpler.
To me this makes total sense and is really easy to explain to people, but maybe that's just me
User |> where(admin: true) |> order_by(:name) |> offset(10) |> limit(10)
I think it would be worth it to do this for all of the query functions and would be happy to help with a PR :)

Thanks @paulcsmith. I will defer to @josevalim to respond to your comments.

I am ok with dropping the bindings for where, having, offset and limit. @ericmj?

@josevalim I might finally have a lock on this. Tests pass. Take a look?

Looks good! Please send a PR when you are ready!

@josevalim updated.

Perfect, please send a PR! :)

@parkerl, thank you! Are you ready for the next and final step? We also need to allow the following:
list_of_values = [title: "hello"]
from Post, where: ^list_of_values

The bad news is that we cannot re-use the code we just wrote, because the interpolation happens at runtime, and everything we did was compile time. The good news is that the runtime version should be simpler.
In order to implement this, we have roughly those steps:

on escape, we need to match on anything like {^, _, [var]} and delegate the work to runtime. Something like this:

def escape(kind, {^, _, [var]}, _, _) do
  {quote(do: Ecto.Query.Builder.Filter.runtime!(unquote(kind), unquote(var))), %{}}
end

At runtime, you are now going to convert the keyword list to ands. The good news is that we no longer need to worry about escaping and what not. Everything is happening at runtime. Here is a sample input/output of the runtime! function called above:

runtime!(:where, [title: "hello", id: 3])
#=> {:and, [],
   [{:==, [],
     [{{:., [], [{:&, [], [0]}, :title]}, [], []},
      %Ecto.Query.Tagged{tag: nil, type: {0, :title}, value: "hello"}]},
    {:==, [],
     [{{:., [], [{:&, [], [0]}, :id]}, [], []},
      %Ecto.Query.Tagged{tag: nil, type: {0, :id}, value: 3}]}]}

Notice each value is wrapped in Ecto.Query.Tagged and the type is the same we pass in your latest PR.

Write tests, change Repo.get_by to use where: ^where instead of the loop it uses today and we are ready to ship it!

We are almost there! Let me know if you have any questions!

@josevalim great! Thanks for the detailed task list. I will dive in!

@josevalim how's that look so far?

To make this work well across the Query API, I think that making the binding optional for order_by, limit, where, and whatever other query functions are needed would make writing queries so much simpler.

@paulcsmith could you please send a pull request for this?

@josevalim Yeah I can do that. Is there any time limit on this? It might take a week or two before I have time. if that's ok I'm happy to work on it!

@paulcsmith that would be completely fine. I will open up a new issue and ping you on it. :)

Closing this in favor of the latest PR. :)


foreign_key of two column

I have an entity which consists of a primary key on two columns.
It would have to be able to translate this entity but when I use the command "doctrine: schema: update" I have an error message:
Translatable does not support composite identifiers in class.
do you know the solution to use "gemo translatable" with an entity that has a primary key of two columns?
thank you

Closing due to age and lack of activity. If you are interested in this issue or are still experiencing any problems described here, please add a comment!


Merge to live!

Add sharing link to See Also

Feedback

Docs Build status updates of commit 7b851d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/overview/excel.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Not able to build the project

I'm trying to build the project. It says build/src folder not found. Kindly help

Sorry!, Managed to find the problem


Deno babel helpers import fails

Something wrong is going on with the server
GET https://cdn.esm.sh/v58/<EMAIL_ADDRESS>results in
500 Internal Server Error
and the following body:
/* esm.sh - error */
throw new Error("[esm.sh] " + "parseCJSModuleExports: Can't resolve '@babel/runtime/helpers/esm/extends' in '/tmp/esm-build-c7210890451e520cc3e3082a95465dc682656511-0e7bcf98'");
export default null;

My build works just fine with local cache, but my CI fails with this error. Trying to add other dependencies locally causes this error during local build. Seems that something's affected the way esm.sh bundles libraries.
deno bundle prints the following line and exits
error: Import 'https://cdn.esm.sh/v58/<EMAIL_ADDRESS>failed: 500 Internal Server Erro

Additional info

esm.sh version: v58
Deno version: 1.16.4

I also have this error when trying to import react-router-dom! Hopefully, it get fixed soon.

The latest working version is https://cdn.esm.sh/v58/<EMAIL_ADDRESS>I encountered this issue when trying to use react-router-dom. I resolved it for my project by manually overriding the<EMAIL_ADDRESS>which used the broken babel runtime dependency with<EMAIL_ADDRESS>which used the latest working babel runtime.
https://github.com/alephjs/esm.sh/issues/235

@KyleJune I am having the same issue! So how do I import react-router-dom? I didn't understand.

You can copy my deps folder that has the fixed versions of react, react-dom, react-router-dom, and react-router then import from them instead of importing from esm.sh. I just took the esm.sh files and modified them to get them working. In the below deps.ts file, you can see it imports from my local deps directory instead of from the esm.sh urls. I plan on switching back to esm once issues are resolved.
https://github.com/KyleJune/deno-tailwind-ui-react-example/blob/406f553efad8f9f2d48920bef987e9d71c4aab7a/components/deps.ts
For a complete example of using react with deno deploy, check out that example repository I made.
https://github.com/KyleJune/deno-tailwind-ui-react-example/

guys, the issue seems to be related to infrastructure, not to the code of esm.sh. i have setup esm.sh locally on my dev machine and directed deno to look at it using import maps. i can import everything without a problem now and it works much faster than through hosted service.
i can suggest the following:

run docker containergit clone https://github.com/alephjs/esm.sh.git
cd esm.sh
docker build . -t esm
docker run -d \
           -v esm:/esm.sh/.esmd \
           -p 8080:8080 \
           esm esm --port 8080 --cdn-domain localhost:8080

and --import-config imports.json, where imports.json contains:{
  "imports": {
    "/esm/": "http://localhost:8080/"
  }
}

then in your code you can point to dependencies likeimport React from '/esm/react'

this might require some enhancement for CI build if you have it, but basically you'll need to spawn esm container and provide imports with /esm/ entry pointing to that container.
if you use docker to build your apps, consider adding --network host to allow docker access esm container.

guys, the issue seems to be related to infrastructure, not to the code of esm.sh. i have setup esm.sh locally on my dev machine and directed deno to look at it using import maps. i can import everything without a problem now and it works much faster than through hosted service.

It's still an issue that esm.sh can end up in a bad state like this that it doesn't recover from on its own. I don't think most people would host their own instance of esm.

This should be fixed in the latest update


test: data-driven tests for partial evaluation

Closes partially #433.
Summary of Changes

Implement a data-driven way to test the partial evaluator
Add a very basic port of our implementation from Xtext and lay out the general structure of the full implementation

:tada: This PR is included in version 0.2.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


ignoring built-in table schemas on capture

From https://issues.redhat.com/browse/DBZ-1939:
When table.ignore.builtin is set to true (default), one might expect the tables under the mysql schema to be completely ignored from both data capture and schema capture.
Nevertheless those tables get registered in the createTablesMap map because the condition createTableFilters == filters && shouldRecordTableSchema is true whenever isStoreOnlyMonitoredTablesDdl is set since !schema.isStoreOnlyMonitoredTablesDdl() will be true by default and therefore shouldRecordTableSchema(schema, filters, id) will return true, even for builtin table.
It's not really clear if these is any added value of capturing builtin tables schema, if not, probably this behavior needs to be changed to fully and completely ignore built-in tables.

I'm going to send a new PR to that logs a warning if a table does not exist instead of ignoring built-in tables.


Referencing detector logics by component id or index?

As far as I understand, activating detector logics is done with M0008 by referencing the component id. But when you request the status of detector logics activations with S0021, you get a string with the status for each, referenced by index.
If this is correct, I think it's unfortunate that different ways of referencing is used..

It is correct. I think it is better to reference by component-Id (as in M0008), since we need to use component-id anyway for other purposes (e.g.) alarms and there are some inherent problems with simply using a series of ones and zeroes in a string. All detector logics must be placed in sequence and there cannot be any gap in the series.
But the format of S0021 (similar to S0002) is needed for performance. And also, in the real world it is very likely that you'd only like to activate a single detector logic at a time but you'd still like to know the status of all detector logics at the same time. Subscribing to each detector logic individually would cause massive overhead.

It seems we have two unrelated referencing systems. Regarding overhead, is it the size of the status response you wrory about, or requiring additional messagse?
An ideas is to keep a numbered list of detector logic components, eg:
1: DL_A
2: DL_C
3: DL_F
4: DL_G
And make it possible to subscribe to this list, so you get the updated list when ever a component is added/removed.
That way you will be able to tell what compoent-ids the statuses you get back with e.g. S0021 refer to.

The overhead I'm worried about is that there would be a need for a separate status response message - one for each component. And there could easily be more than 40 detector logics in many intersections. Updated status of all detector logics is needed, at least upon connection establishment.
If one would like to know the status of 40 detector logics, a status response message like the one below would need to be sent by the TLC 40 times.
{
	"mType":"rSMsg",
	"type":"StatusResponse",
	"mId":"e5688e53-c51e-408a-8075-c3c018a67f56",
	"ntsOId":"KK+AG0503=001DL001",
	"xNId":"",
	"cId":"KK+AG0503=001TC000",
	"sTs":"2019-09-26T13:31:32.114Z",
	"sS":[{
		"sCI":"S0021",
		"n":"detectorlogics",
		"s":"False",
		"q":"recent"
	}]
}

The idea to use a string with ones and zeros may not be perfect, but it was commonly used in TLC's for other purposes since earlier (eg. RS232 terminal interface, which we require all TLC's to have) and at least all manufacturers agreed upon using it.
A numbered list of detector logics as you mention would be a way to solve some of the shortcomings with using a plain string with ones and zeros, but then we need to get the manufacturers involved and convince them to change the format. We need to ask them if they see a problem with the current format and if it is worth the trouble to change it.

The problem is more on the supervisor side. You need to know what component ids match which indeces., which is more complicated than it needs to be.
The suggestion with subscribing to the index<->component id mapping was to keep the existing format, but just add a way to see how compoent ids map to indeces.

could perhaps be solved by https://github.com/rsmp-nordic/rsmp_core/issues/13

Would be good to solve, but perhaps it has complications. Perhaps best solved in RMSP 4.


DbInterface.java uses sequence name dynamically in queries.

Describe the bug
For simplification, getKey and resetKey pass the table/sequence name as a parameter and builds the sequence query based on that variable.  We might want to verify that the sequence actually exists to prevent a possible sql injection.
NOTE - Currently, the only use of this is to not ever pass user input into this function.  This might be a better solution, as getting all of the sequence names is inefficient.
To Reproduce
Steps to reproduce the behavior:
On DbInterface.java, you can see in the resetKey and getKey functions, it is building a dynamic sql query, passing in the sequence name.
String q = isOracle ? ("SELECT " + seqname + ".nextval from dual")
Expected behavior
Solutions:

Check that the sequence name exists before running the query.

Make sure user input is never passed into this function.  (preferred method)

Create new functions for each sequence.

We really should start using auto sequence and the available insert ... <table> values(...) returning <key column>  that should be able in all databases these data.
But that would require getting the main code to do the same thing at the same time so it'll probably have to wait.

For now, would you recommend we should run the check to verify each sequence exists, or just rely on the code only passing non -user input?
Both have their drawbacks, but I think querying for all sequences is a little inefficient.
As a side note - I can create a new issue to track your comment above.

Yeah, I think that it's a reasonable sanity check. especially in case another developer violates the rule of non-user input.

After speaking with Peter and Adam, we thought it might be a good idea to use a similar approach, but create an enum (of valid sequences) that forces the developer to adhere to the rule of non-user input.

Hmm, that sounds better; certainly less possible performance issues waiting for a database roundtrip.


Support Hive bucketing v2

Fixes https://github.com/prestosql/presto/issues/538

Thanks @kokosing for your review.
I remved the DEBUG commit and will merge when Travis is green.


[CLI] Script Killed using DDP with Pytorch lightning and WANDB

Hi all, appreciate the help on this! I've been having some issues with DDP in pytorch lightning while also using wandb. As a preface, I tested my script on a development node with a small batch size using only one GPU and it ran just fine (just slow) - the issues come up when I start to use multiple GPUs (4) using DDP with pytorch lightening. Unfortunately, I don't have much of an error to give - the script runs but ends quickly with the log file only reporting 'Killed barlow.py'. I thought initially this might be a memory issue and have since decreased my batch size significantly - from the gpu memory usage report, however, I don't think it's a memory issue. Is there something that pops at as wrong with how I'm initializing/using wandb along with pytorch lightening in these runs? Any thoughts on what exactly is going on? Also of note, when I log validation losses, I use the flag 'sync_dist' = True. A portion of my code is reproduced below. Happy to show any other parts of my code if helpful as well.
default_config = {'LR': 1e-4,
            'WD': 1e-6, 'Z_DIM': 8192, 'LAMBDA': 5e-3
            }
#added start method 'thread'
wandb.init(config=default_config, settings=wandb.Settings(start_method='thread'), mode="offline", project="Barlow")
# Config parameters are automatically set by W&B sweep agent
config = wandb.config

def main(config):
    # Zero-initialize the last BN in each residual branch,
    # so that the residual branch starts with zeros, and each residual block behaves like an identity.
    # This improves the model by 0.2~0.3% according to https://arxiv.org/abs/1706.02677
    encoder = resnet50(zero_init_residual=True)

    # replace classification fc layer of Resnet to obtain representations from the backbone
    encoder.fc = nn.Identity()
    encoder_out_dim = 2048
    BATCH_SIZE = 256
    MAX_EPOCHS = 1000

    run_tags = [
        f'ENCODER OUT DIM: {encoder_out_dim}',
        f'Z DIM: {config.Z_DIM}']

    # ------------------------
    # 2 WANDB LOGGER
    # ------------------------
    wandb_logger = WandbLogger(tags=run_tags)

    model = BarlowTwins(
        encoder=encoder,
        encoder_out_dim=encoder_out_dim,
        learning_rate = config.LR,
        weight_decay = config.WD,
        num_training_samples=262144,
        batch_size=BATCH_SIZE,
        z_dim=config.Z_DIM,
        lambda_coeff = config.LAMBDA,
        max_epochs=MAX_EPOCHS
    )

    checkpoint_callback = ModelCheckpoint(dirpath='/wynton/protected/home/ichs/dmandair/BRCAness/datasets/train/pcam/', monitor = 'val_loss', every_n_epochs=50, save_top_k=1)

    trainer = pl.Trainer(
        max_epochs=MAX_EPOCHS,
        precision=16 if torch.cuda.device_count() > 0 else 32,
        callbacks=[checkpoint_callback],
        logger = wandb_logger,
        gpus=4,
        strategy='ddp'
    )

    trainer.fit(model)

if __name__ == '__main__':
    # print(f'Starting a run with {config}')
    main(config)
    wandb.finish()
    #added wandb finish so job doesn't lag

Just so you have it, the debug.log file from wandb is here:
          2022-02-03 00:03:10,481 INFO    MainThread:35223 [wandb_setup.py:_flush():71] setting env: {}
          2022-02-03 00:03:10,542 INFO    MainThread:35223 [wandb_init.py:_log_setup():371] Logging user logs to /wynton/protected/home/ichs/dmandair/BRCA/wandb/offline-run-20220203_000310-eikdsoq4/logs/debug.log
          2022-02-03 00:03:10,543 INFO    MainThread:35223 [wandb_init.py:_log_setup():372] Logging internal logs to /wynton/protected/home/ichs/dmandair/BRCA/wandb/offline-run-20220203_000310-eikdsoq4/logs/debug-internal.log
          2022-02-03 00:03:10,543 INFO    MainThread:35223 [wandb_init.py:init():404] calling init triggers
          2022-02-03 00:03:10,544 INFO    MainThread:35223 [wandb_init.py:init():409] wandb.init called with sweep_config: {}
          config: {'LR': 0.0001, 'WD': 1e-06, 'Z_DIM': 8192, 'LAMBDA': 0.005}
          2022-02-03 00:03:10,544 INFO    MainThread:35223 [wandb_init.py:init():460] starting backend
          2022-02-03 00:03:10,544 INFO    MainThread:35223 [backend.py:_multiprocessing_setup():99] multiprocessing start_methods=fork,spawn,forkserver, using: fork
          2022-02-03 00:03:10,550 INFO    MainThread:35223 [backend.py:ensure_launched():216] starting backend process...
          2022-02-03 00:03:10,559 INFO    MainThread:35223 [backend.py:ensure_launched():221] started backend process with pid: 35314
          2022-02-03 00:03:10,561 INFO    MainThread:35223 [wandb_init.py:init():469] backend started and connected
          2022-02-03 00:03:10,562 INFO    MainThread:35314 [internal.py:wandb_internal():87] W&B internal server running at pid: 35314, started at: 2022-02-03 00:03:10.561082
          2022-02-03 00:03:10,568 INFO    MainThread:35223 [wandb_init.py:init():533] updated telemetry
          2022-02-03 00:03:10,569 INFO    MainThread:35223 [wandb_init.py:init():606] starting run threads in backend
          2022-02-03 00:03:10,570 INFO    WriterThread:35314 [datastore.py:open_for_write():77] open: /wynton/protected/home/ichs/dmandair/BRCA/wandb/offline-run-20220203_000310-eikdsoq4/run-eikdsoq4.wandb
          2022-02-03 00:03:15,574 INFO    MainThread:35223 [wandb_run.py:_console_start():1810] atexit reg
          2022-02-03 00:03:15,600 INFO    MainThread:35223 [wandb_run.py:_redirect():1684] redirect: SettingsConsole.REDIRECT
          2022-02-03 00:03:15,600 INFO    MainThread:35223 [wandb_run.py:_redirect():1689] Redirecting console.
          2022-02-03 00:03:15,602 INFO    MainThread:35223 [wandb_run.py:_redirect():1745] Redirects installed.
          2022-02-03 00:03:15,603 INFO    MainThread:35223 [wandb_init.py:init():633] run started, returning control to user process

Hi @dmandair,
Thanks for reporting the issue. From a first glance your script seems to be correct.
We are currently working on a new feature which provides a better support for multiprocessing, could I suggest you give it a try: https://github.com/wandb/client/blob/master/docs/dev/wandb-service-user.md
Also could you share your debug-internal.log (found in the same folder as the debug.log this log should give us more information.

Yes, here's the debug-internal.log:
2022-02-03 00:03:10,562 INFO    MainThread:35314 [internal.py:wandb_internal():87] W&B internal server running at pid: 35314, started at: 2022-02-03 00:03:10.561082
2022-02-03 00:03:10,571 DEBUG   HandlerThread:35314 [handler.py:handle_request():130] handle_request: run_start
2022-02-03 00:03:10,570 INFO    WriterThread:35314 [datastore.py:open_for_write():77] open: /wynton/protected/home/ichs/dmandair/BRCA/wandb/offline-run-20220203_000310-eikdsoq4/run-eikdsoq4.wandb
2022-02-03 00:03:10,589 DEBUG   HandlerThread:35314 [meta.py:__init__():40] meta init
2022-02-03 00:03:10,590 DEBUG   HandlerThread:35314 [meta.py:__init__():54] meta init done
2022-02-03 00:03:10,590 DEBUG   HandlerThread:35314 [meta.py:probe():214] probe
2022-02-03 00:03:10,596 DEBUG   HandlerThread:35314 [meta.py:_save_pip():58] save pip
2022-02-03 00:03:10,617 DEBUG   HandlerThread:35314 [meta.py:_save_pip():72] save pip done
2022-02-03 00:03:10,618 DEBUG   HandlerThread:35314 [meta.py:_save_conda():79] save conda
2022-02-03 00:03:28,540 DEBUG   HandlerThread:35314 [meta.py:_save_conda():89] save conda done
2022-02-03 00:03:28,541 DEBUG   HandlerThread:35314 [meta.py:probe():252] probe done

hi @aidanjdonohue @kptkin - I haven't been able to test it out yet (gpu job schedueling here just takes a while). Are the logs suggestive of any issue?

Hi @dmandair, you should put it in your __main__  before the main method. This should come in the beginning of your script to set it up.
The log suggest that the run is cut off after the call to meta, it seems indeed that it doesn't complete the run.

Hi @dmandair , we just introduced a tentative fix.
Could you update Pytorch Lightning from master branch and install the most recent version of wandb?
pip install --upgrade wandb
pip install --upgrade git+https://github.com/PytorchLightning/pytorch-lightning.git

hey @borisdayma, thanks for letting me know! I assume using these pip install inside my conda environment should work fine? And I shouldn't have to use wandb-service once I run the above? Thanks again!

Yeah I think it should work (not been using conda in a while).
And yes, you won't have to use wandb-service.

hi everyone - thanks so much for the help with this. I just wanted to circle back and let you know that the upgrades seem to take care of the issue. @borisdayma @kptkin

hi everyone - thanks so much for the help with this. I just wanted to circle back and let you know that the upgrades seem to take care of the issue. @borisdayma @kptkin

This is great to hear! :)
So I'm closing this issue


Listsnapshots

Show container snapshots in lxc list output.
Also fix the urls returned over the REST get for /1.0/containers/name/snapshots.

Doesn't match the spec. The spec for "lxc list" says that container snapshots are shown on "lxc list " not in the main list. Though once we have fancy rendering, I wouldn't be opposed to having a column saying how many snapshots a container has.

Hm, I misunderstood the spec then.