[IMPROVEMENT] Webhook should always allow finalizer removal on deleting objects

Is your improvement request related to a feature? Please describe (👍 if you like this request)
In https://github.com/longhorn/longhorn-manager/pull/2425 we modified the validating webhook for node resources so that it is possible for Longhorn to remove the finalizer from a deleting node object regardless of other webhook checks. This was appropriate in that case because the node controller wanted the deletion to finish (hence its attempt to remove the finalizer) and could not proceed to any other logic that may have allowed the webhook checks to pass.
In general, I think it is appropriate for the webhook to allow the finalizer to be removed from a deleted object in all situations. It should be the responsibility of the controllers themselves to decide whether or not to remove the finalizer. Once the decision is made, the webhook should not prevent it.
Describe the solution you'd like
Evaluate the webhook checks for other resources and determine if there is any reason why similar behavior should not be adopted for them. If no reason can be identified, use the common code from https://github.com/longhorn/longhorn-manager/pull/2425 to make the change.

@ejweber Do you think we can improve in v1.7 or handle it later?

I think a v1.7.0 target is reasonable here. I think it should not take more than a day of work to evaluate it and make the change. The priority is low (IMO), so if it turns out to be more complicated (there are particular resources we're not sure about for some reason), we can change our minds.


PHP 8.1 compatibility

When I'm using the "from" and "renderImage" functions then I see different errors. One of that errors you can see below:
Deprecated function: Return type of Elphin\IcoFileLoader\Icon::offsetExists($offset) should either be compatible with ArrayAccess::offsetExists(mixed $offset): bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in include() (line 8 of /var/www/html/vendor/lordelph/icofileloader/src/Icon.php)

#0 /var/www/html/core/includes/bootstrap.inc(347): _drupal_error_handler_real()
#1 /var/www/html/vendor/lordelph/icofileloader/src/Icon.php(8): _drupal_error_handler()
#2 /var/www/html/vendor/composer/ClassLoader.php(571): include('...')
#3 /var/www/html/vendor/composer/ClassLoader.php(428): Composer\Autoload\includeFile()
#4 /var/www/html/vendor/lordelph/icofileloader/src/IcoParser.php(69): Composer\Autoload\ClassLoader->loadClass()
#5 /var/www/html/vendor/lordelph/icofileloader/src/IcoParser.php(55): Elphin\IcoFileLoader\IcoParser->parseICO()
#6 /var/www/html/vendor/lordelph/icofileloader/src/IcoFileService.php(126): Elphin\IcoFileLoader\IcoParser->parse()
#7 /var/www/html/vendor/lordelph/icofileloader/src/IcoFileService.php(111): Elphin\IcoFileLoader\IcoFileService->fromFile()

Because this repository is not active we did our own fork, compatible with PHP 8.0+ https://github.com/ProtonMail/icofileloader (protonlabs/icofileloader)

Good to know about it but I must support older PHP versions (5.6 and above). Any chance to have it in future versions of your fork?

I'll make a new release for php 8.x and bump the major version number. Older versions will still be available

The 1.x.x releases will be for php5
The 2.x.x releases will be for php7
The 3.x.x releases will be for php8+

I've pushed a new version 3.0.0 which declares the correct return types as well as generally upgrading the tests for more recent phpunit version - will close this PR but thank you very much for suggesting this.


TRINITY-53 Fixed Trinity js and sass files for the library build

https://sakaiproject.atlassian.net/browse/TRINITY-53

@ern and @mpellicer - I don't see this commit in the Trinity branch anymore. Was it removed this week?


Feature/logs-view-ui: Logs view implementation for Visualization

Description
Implemented the Config Options of Logs View as mentioned in #773 and #670

this functionality is addressed in https://github.com/opensearch-project/observability/pull/910.
Closing this one.


Have thread local context logging functionality

Closes #37: Add macro for setting thread-local context params for logs #38


Allow for React component methods to be used (requires support for React refs)

I'm trying to wrap the react-player component:
https://www.npmjs.com/package/react-player
Which already works well with the following code (minimal example):
import pynecone as pc

class ReactPlayer(pc.Component):
    library = "react-player"
    tag = "ReactPlayer"
    url: pc.Var[str]

However, the react-player exposes several methods like seekTo and getDuration.
Is there any way for me to wrap these methods and be able to call them on the Python ReactPlayer component?
I imagine something like:
react_player = ReactPlayer.create(url="https://www.youtube.com/watch?v=outcGtbnMuQ")

def index() -> pc.Component:
    return pc.box(
        pc.vstack(
            react_player,
            pc.button("Seek", on_click=react_player.seek_to(10)),
        ),
        padding="5em",
        border_radius="1em",
    )

Copying answer from Discord:

Alek99
We don't support the ref yet to add seekTo and getDuration but will add support for this you can implement the call backs though onReady see the color picker examples get_controlled_triggers code

Awesome thanks for making this ticket!

we do support custom refs now. Closing this ticket

we do support custom refs now. Closing this ticket

Amazing! Could you link to any resources regarding this? I wasn't able to find any. The PR will suffice.


[BUG] Banner ads are present in the scene, clickable, but are invisible.

Describe the bug
I am trying to show my banner ads in my app. When I call them using the show_banner() call, nothing seems to happen, but in reality, the ads do load and get in the place where they are supposed to be, but for some reasons they remain invisible. Upon tapping on the area where they are supposed to be, does open something related to a test ad - which means that it is there, present on the screen, just not showing up.
To Reproduce
Steps to reproduce the behavior:
I am right now using this to show the banner ads:
    MobileAds.load_banner()
MobileAds.show_banner()

Expected behavior
The banner ads should be visible, not invisible/transparent.
Screenshots
I have attached two videos that show the problem.
Smartphone (please complete the following information):

Device: Redmi POCO Phone F1
OS: MIUI Global
Version 12.0.3

I do not think that this problem is device/OS specific. I have tested my app on my brother's and parents' phones. All are different models and exhibit same behaviour.
Additional context
I have attached two videos here:

First video has my app in Aspect > KeepWidth mode. It has invisible ads.
Second video has my app is Aspect > KeepHeight mode. It has only partially visible ads at the bottom of the screen.

https://user-images.githubusercontent.com/90527174/189789607-d7412723-7f0f-4915-97af-e16aeaacc697.mp4
https://user-images.githubusercontent.com/90527174/189790058-87184df0-3c56-4f29-9410-b260dc30ff92.mp4

Hi! What Godot version and Android Plugin version are you using?

Godot V3.4.4 stable and for Admob version, I have attached the screenshot, I think it is 1.2.4 or 2.0.0 for Android.

@ajk2196 Can you provide me a sample project of this bug? I can't reproduce here
I find it strange that it seems that your UI is in front of the ads, are you using any other Plugin for Android besides our AdMob?

It is hard to reproduce this, I have tried this combination of plugin version and Godot version in some of my projects. Everything is normal in my other projects - the ads appear normally. Only this particular project is having this weird issue. I even tried loading it with latest plugin version and Godot version 3.5, but the bug still remained. It is hard to determine what is causing this bug. I will shortly be redoing this same entire project from scratch in about a month when I have time - everything step by step to pinpoint the change that triggers this bug. I will post about it when done. Hopefully, that shall help in solving this bug.
The interstitial ads show normally. Only the banner ads are getting overlayed by the app's scenes.

Thank you @ajk2196, if you have any news, please let me know

Do you have Per Pixel Transparency enabled ?

Yes. I don't have access to my project right now as I am on my phone, but I may have it on, I have it on by default in all my projects, so there are high chances that it may be on in this project as well. I will try toggling it and post the results once I am home. Thanks for the hint @patrick-wagner-0681

I have finally built my project from scratch and thankfully, have found no issues. Like @patrick-wagner-0681 suggested, I did not turn on Per Pixel Transparency, but also, I did not dare to mess with it with the fear of the problem getting repeated again. It took me almost a month to re-assemble my project from scratch.
Things I noted from this bug:

The bug was project specific. Making trial Android apps to test ads displayed the Ads just fine.

Removing / uninstalling Godot, Admob plugins, Android Build Templates and then adding / installing their same/different versions again did not help. The only thing that helped was re-making and a little bit of re-assembling the app from scratch.

To be sure that everything goes well, I 'Refreshed' my PC. It runs on Windows 10, where Refreshing a PC means keeping personal files and removing all installed softwares on the PC.

Nice i will close this Issue since seems to be solved! If this problem returns you can re-open, thank you!


Cannot Execute Command

Hello there.
I can not execute commands that i created with command.ts.
const command = await this.discordProvider       .getClient()       .application.commands.fetch(commandId, { guildId: interaction.guildId });
command.execute('message', args);
Execute is undefined.
So i used  .fetch function as a replace official command get method.

Sorry, but at the moment this package only works with version 12, 13 is in progress

Sorry, but at the moment this package only works with version 12, 13 is in progress

Okay. Btw thanks for this great package for nestJS.


Why is the address I get all lowercase from metamask? I want to get the original address

I am a novice. Hope to be helped.

That is probably related to MetaMask returning the addresses in lowercase.
I don't think this is related to the WC sdk.


Aggregate with join shouldn't select columns

Hi, I'm currently trying to do some aggregate queries that involve joining for filtering. The joining all works fine, but the problem is the moment you add the join, it selects all columns including the aggregate, which breaks the query (it doesn't literally break on sqlite, it returns incorrect data, but it does break in mysql).
I can't find any way of telling the query not to select those columns. I know I can use knex straight, but then I have to redefine all the joins, and ideally I can just make use of the existing relations in objection.
query:
  const catPeople = await Person.query()
    .joinEager('pets')
    .count({ count: '*' })
    .where('pets.species', 'cat')

  const expectedResult = Object.assign(new Person(), { count: 1 })
  chai.expect(catPeople[0]).to.equal(expectedResult);

output:
  actual:
   Person {
     count: 1,
     id: 1,
     parentId: null,
     firstName: 'Jennifer',
     lastName: 'Lawrence',
     age: null,
     address: null,
     pets: [ [Animal] ] },
  expected: Person { count: 1 }

full pastebin
Is there an easy way to do this, or should I just take the hit with a manual knex query?
EDIT: I previously tried .select({}) but that broke the query, doing .select('') mostly works, except it still returns the relation:
  actual: Person { count: 1, pets: [ [Animal] ] },
  expected: Person { count: 1 } }

So I think that'll still be broken on mysql

Use joinRelation. Don't eager load.

  const catPeople = await Person.query()
    .joinEager('pets')
    .count({ count: '*' })
    .where('pets.species', 'cat')

  const expectedResult = Object.assign(new Person(), { count: 1 })
  chai.expect(catPeople[0]).to.equal(expectedResult);


Error while building

I followed the steps and on execution of step 6 I receive the following error:
make[1]: *** [CMakeFiles/micro-service.dir/all] Error 2
make: *** [all] Error 2

I am using mc os with M1 processor (ARM).

I am using Mac Intel, but also have the same issue.
If you execute make directly under build/ directory without -j 8 flag. You will see the following error
Consolidate compiler generated dependencies of target micro-service
make[2]: *** No rule to make target `/usr/local/Cellar/openssl/1.0.2s/lib/libssl.1.0.0.dylib', needed by `micro-service'.  Stop.
make[1]: *** [CMakeFiles/micro-service.dir/all] Error 2
make: *** [all] Error 2

Try to do a global search on openssl in micro-service repository, you will find that under micro-service/CMakeLists.txt, there are some hard-coded paths & versions on openssl and zlib. But your local versions on Mac does not match those in micro-service/CMakeLists.txt. That's why we failed to build.
For my local mac openssl and zlib verions they are
<EMAIL_ADDRESS><EMAIL_ADDRESS>/usr/local/Cellar/zlib/1.2.12_1/lib/libz1.2.12.dylib

Change all those related path in micro-service/CMakeLists.txt, and I can re-build successfully

I get undefined reference to inflateReset' when I run make -j 8.


Turn into a parsing generator

I want to use the logic here to create an optimized parsing generator in that produces C (and rust bindings)

I can do the same as tree-sitter does: use JS/TS to create a grammar JSON, then use that to generate C code. But differently from tree-sitter, I want this C code to be zero dependency (no std even)


VRChat freezes when loading some worlds

Right, so the issue that I've encountered is that certain worlds will just freeze the game on a black screen with the "Hold V" overlay. It's not frequent, but definitely a bit irritating when it happens since I have to pkill the game and start it up again. I've confirmed that the same worlds cause the game to crash when running with or without wined3d, with and without eSync. The two worlds that I remember causing this off the top of my head are "The Great Pug" and "Sign Language" (created by MINT in case you find multiple by the same name).
Software information
VRChat 2019.3.1, launch settings: PROTON_LOG=1 %command%, launching through Steam Play
System information

GPU: Radeon RX480
Driver: media-libs/mesa-19.2.0_rc1 with:
USE: X classic dri3 egl gallium gbm gles2 llvm vaapi vulkan
VIDEO_CARDS: intel nouveau radeon radeonsi
ABI_x86: 32, 64
Wine version: Proton 4.11-3
DXVK version: app-emulation/dxvk-1.3.2-r1

Apitrace file(s)
Not sure I got the output correctly (just right clicked, "Select all", then pasted into Pastebin and ugh manually formatted it), but here we go: https://pastebin.com/3VFhBcNR
Log files

d3d11.log: https://pastebin.com/6JgG94jX
dxgi.log: https://pastebin.com/7xB44mtg

Does this work on wined3d?
fwiw the "apitrace" you posted isn't an apitrace but just some console output. I'm more interested in the .trace file, ideally recorded on Windows - though I'm not sure how useful that would be in this case.

If you're referring to running this with the PROTON_USE_WINED3D=1 option, then no, it doesn't work on wined3d (and in fact has quite a few graphical glitches that for some reason don't show up when I use apitrace). Here's a link to an Imgur gallery containing a few examples of such graphical issues: https://imgur.com/a/X3rg2Qx.
Apologies about grabbing the wrong .trace file. Here's the right one: (good for 1 download and for 7 days): https://send.firefox.com/download/0f82e022e97d95f3/#h1aMuCkftA_awlpnnnf1WA
Or at least I hope it is. Uploading that took literally hours :x

VRChat usually freezes  when a world use  youtube-dl.exe.  You can remove/rename this file from VRChat to disable the feature.  It usually resolve the freezing issue.
It's far from a perfect solution but it make the game playable.

Was this issue solved?
I have not been able to reproduce it myself when playing some none VR rooms including the great pug and some others.
As said above it seems people had trouble in worlds with youtube-dl in the past? But seems like a proton issue maybe and isn't a problem anymore?

@AuroransSolis @leucome Friendly ping.


Dispel Magic of rm /tmp/pach

See this issue for an example of such magic

Can this be closed? I think the magic of rm /tmp/pach is just that you were assuming that make launch blew away database state when it actually doesn't. So it makes perfect sense that schema changes would require rm /tmp/pach

Yup! After our slack discussion this makes sense


[TO BE UPDATED] Add Query String Support to Get_shopcarts Function

As a System Administrator
I need  to add query string support to the GET /shopcarts/{user_id} route
So that I can query the items based on the specified query parameter
Details and Assumptions

Given the URI in format /shopcarts/{user_id}/items?name=x&hold={true or false}
or /shopcarts/{user_id}/items?hold={true or false}
or /shopcarts/{user_id}/items?name=x
with "GET", we should get a list of items which each item in the list having item_name "x", with hold attribute set to {true or false}
And 200 OK if success, 400 if user_id not valid or hold= {something that's not either true or false}; 404 if the given user id not found
UPDATE: per prof: 'yes its fine if you only implement 1 query. Probably the most useful query would be to return all of the shopcarts that contain a particular item id.  This would be helpful for store management to notify all customers that something in their cart just went on sale, or is out of stock.'

Acceptance Criteria
Given we have item_id 100 with item_name "Ring100" in user_id 300 with hold = false
And we have item_id 100 with item_name "Ring100" in user_id 400 with hold = false
And we have item_id 200 with item_name "Ring" in user_id 400 with hold = true
And we have item_id 300 with item_name "Ring" in user_id 400 with hold = true
When we call GET on shopcarts/400/items?name=Ring&hold=true
We get 200OK and receive a list of two items, [item_id 200 and item_id 300]
When we call GET on shopcarts/400/items?name=Ring&hold=t
Then we receive 400 BAD REQUEST
When we call GET on shopcarts/300/items?name=Ring
We get 200OK and receive an empty list

To be updated per Professor's comment on slack (copied above)


Reformat based on clang-format's LLVM style with 4 spaces

I'm working on updating the version of the patricia code we use in https://github.com/zeek/zeek. I noticed that the formatting in the code is all over the place. It's a mix of tabs and spaces and varying indentation styles. This PR reformats it using clang-format and --style="{BasedOnStyle: llvm, IndentWidth: 4}". There aren't any functional changes in this PR at all, it's just whitespace fixes and minor reordering of lines based on the style (the #include lines for example).

Thanks, this is much needed.
I have a slight preference for sticking with the clang format spec that's used in other CAIDA repos. E.g., https://github.com/CAIDA/libbgpstream/blob/master/.clang-format

Oh! I looked at the top-level and didn't see one, so I went with LLVM since it was sorta close to what existed. I'll update the PR using that config.

Ping on merging this. I'd like to rebase our version on this upstream one, but I want to wait until it's been officially merged.

@kenkeys could you take a look at this PR?

Any word on merging this?

One more ping on this.

Is there something still holding this up from getting merged?


cleanup: fix race condition

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
First part of the work for https://github.com/kubernetes-sigs/kueue/issues/67
Second part: https://github.com/kubernetes/test-infra/pull/25443
Which issue(s) this PR fixes:
Fixes https://github.com/kubernetes-sigs/kueue/issues/67
Special notes for your reviewer:
None

hi @alculquicondor , thanks for review! please take another look ＾ ＾

/retest

/approve
/lgtm

/retest

@BinacsLee it seems that we have another race in a log line somewhere...
could you take a look?
https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/93/pull-kueue-test-unit-main/1499496533626720256

@BinacsLee it seems that we have another race in a log line somewhere... could you take a look?
https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/93/pull-kueue-test-unit-main/1499496533626720256

sure, I'll deal with it as soon as possible


check connection in beta channel fails

in flutter master channel I was using:
Future isInternetConnected({
  @required Function toDo,
  @required bool showWaitingDialogInToDo,
  @required BuildContext context,
}) async {
  var result;

  toDoOnErrors() async {
    await showCustomDialog(
      context: context,
      massage: tr("errors_and_dialogs.connection"),
    );
    await Future.delayed(
      Duration(seconds: 1),
    );
    return isInternetConnected(
      context: context,
      showWaitingDialogInToDo: showWaitingDialogInToDo,
      toDo: toDo,
    );
  }

  try {
    result = await InternetAddress.lookup('google.com');
  } catch (_) {
    return toDoOnErrors();  // here  ********************
  }
  if (result.isNotEmpty && result[0].rawAddress.isNotEmpty) {
    if (showWaitingDialogInToDo) showWaitingDialog(context);
    return toDo();
  } else
    return toDoOnErrors();
}

to see wheater there is internetConnection or not.
and today I go to channel beta for web support but:
it throws exception.

Hi @mohammadne
Can you please provide your flutter doctor -v and flutter run --verbose?
Thank you

actually :
[√] Flutter (Channel beta, v1.14.6, on Microsoft Windows [Version 10.0.18363.657], locale fa-IR)
    • Flutter version 1.14.6 at C:\flutter
    • Framework revision fabeb2a16f (5 weeks ago), 2020-01-28 07:56:51 -0800
    • Engine revision c4229bfbba
    • Dart version 2.8.0 (build 2.8.0-dev.5.0 fc3af737c7)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at C:\Users\Nasr\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • Java binary at: C:\Users\Nasr\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\191.6010548\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.5)
    • Android Studio at C:\Users\Nasr\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\191.6010548
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.42.1)
    • VS Code at C:\Users\Nasr\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.1

[√] Connected device (2 available)
    • Chrome     • chrome     • web-javascript • Google Chrome 80.0.3987.122
    • Web Server • web-server • web-javascript • Flutter Tools

• No issues found!
PS D:\Mohammad\QC>

[  +24 ms] executing: [C:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +49 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] fabeb2a16f1d008ab8230f450c49141d35669798
[        ] executing: [C:\flutter/] git describe --match v*.*.* --first-parent --long --tags
[  +34 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[        ] v1.14.6-0-gfabeb2a16
[   +6 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +28 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/beta
[        ] executing: [C:\flutter/] git ls-remote --get-url origin
[  +25 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +60 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref HEAD
[  +27 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[  +36 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +35 ms] executing: C:\Users\Nasr\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[  +22 ms] Exit code 0 from: C:\Users\Nasr\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[        ] List of devices attached
[  +12 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +5 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +54 ms] More than one device connected; please specify a device with the '-d <deviceId>' flag, or use '-d all' to act
on all
           devices.
[  +30 ms] Chrome     • chrome     • web-javascript • Google Chrome 80.0.3987.122
[   +1 ms] Web Server • web-server • web-javascript • Flutter Tools
[   +8 ms] "flutter run" took ۲۴۹ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:27:3)
#1      RunCommand.validateCommand (package:flutter_tools/src/commands/run.dart:325:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:619:11)
<asynchronous suspension>
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:537:33)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#5      _rootRun (dart:async/zone.dart:1126:13)
#6      _CustomZone.run (dart:async/zone.dart:1023:19)
#7      _runZoned (dart:async/zone.dart:1518:10)
#8      runZoned (dart:async/zone.dart:1465:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:527:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#12     FlutterCommandRunner.runCommand.<anonymous closure>
(package:flutter_tools/src/runner/flutter_command_runner.dart:338:21)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#14     _rootRun (dart:async/zone.dart:1126:13)
#15     _CustomZone.run (dart:async/zone.dart:1023:19)
#16     _runZoned (dart:async/zone.dart:1518:10)
#17     runZoned (dart:async/zone.dart:1465:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:288:19)
#20     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:112:14)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:231:18)
#24     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:64:22)
#25     _rootRun (dart:async/zone.dart:1126:13)
#26     _CustomZone.run (dart:async/zone.dart:1023:19)
#27     _runZoned (dart:async/zone.dart:1518:10)
#28     runZoned (dart:async/zone.dart:1502:12)
#29     run.<anonymous closure> (package:flutter_tools/runner.dart:62:18)
<asynchronous suspension>
#30     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#31     _rootRun (dart:async/zone.dart:1126:13)
#32     _CustomZone.run (dart:async/zone.dart:1023:19)
#33     _runZoned (dart:async/zone.dart:1518:10)
#34     runZoned (dart:async/zone.dart:1465:12)
#35     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#36     runInContext (package:flutter_tools/src/context_runner.dart:64:24)
#37     run (package:flutter_tools/runner.dart:51:10)
#38     main (package:flutter_tools/executable.dart:65:9)
#39     main (file:///C:/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#40     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

I found same issue : #49482
but what is the solution ?

Duplicate of 49482
Please follow up there
Thank you


BUG: "Add to Obsidian" does nothing on Linux Firefox

Version

OS: Ubuntu 22.04
Browser: Firefox
Web Clipper version: 0.9.4
Obsidian version: 1.7.5

Describe the bug
When clicking "Add to Obsidian", the Firefox web console is cleared and then nothing happens. Notably Obsidian never logs the Received URL action in its console.
To reproduce
Steps to reproduce the behavior:

Go to any URL
Click on the web clipper
Click "Add to Obsidian"
Observe that the article wasn't added to Obsidian

Expected behavior
The article to be added to Obsidian.
Your template file
I'm using the "Default" template.
Other things I've noticed

I've tested this in Brave (chromium-based) and it works correctly, which maybe eliminates my .desktop file being incorrect. (xdg-open 'obsidian://open?vault=main works on the command line as well).
There are no errors of note in the Firefox web console or the Obsidian console.
I'm not running Firefox as a snap.
I'm using the Obsidian AppImage and the error occurs whether or not I point my .desktop file to the extracted squashfs binary.
I've tried using legacy mode, the error occurs with and without it.
I've disabled all other extensions in Firefox and the problem persists.

I assume this is some form of Firefox being Firefox, but I'm totally out of ideas. Thanks for taking a look :)

Did you try the troubleshooting steps?
https://help.obsidian.md/web-clipper/troubleshoot

Did you try the troubleshooting steps?
https://help.obsidian.md/web-clipper/troubleshoot

Yes, I've tried all of them and confirmed that none of them are the issue.

Are you sure you are using the official Obsidian Web Clipper?
You are saying Web Clipper version is 2.10.0 but the latest version of Obsidian Web Clipper 0.9.4

Sorry, I must have copied the version number of a different extension. I am using 0.9.4.

Which version of the Linux Obsidian release are you using? Eg AppImage, Deb, Snap, Flatpak

Are you on Wayland?

Which version of the Linux Obsidian release are you using? Eg AppImage, Deb, Snap, Flatpak

I'm using the AppImage, though I tried pointing at the extracted AppImage as well.

Are you on Wayland?

No, X.

Hmmm... I'm running out of ideas. The only other thing I can think of to check would be the inspector for the Firefox extension itself and/or the service worker.

Yeah, I've looked at the inspector and seen no issues. Since it works in Brave I'm not sure there's a benefit to me building the extension locally and debugging, I'm guessing it's more likely a problem with the link between Firefox -> Obsidian URI handler?

I loaded this into the debugger and it does appear to be some sort of bug where Firefox just isn't correctly opening the obsidian:// link. I think this issue can be closed since there's nothing actionable on your end. I'll update it when/if I figure it out for posterity.
Random question I noticed when debugging, is this line of code intended to be in the code path when writing to the clipboard is successful?

Okay your change was added to 0.9.5


Exporting DAOTests root and simple changed in FlashcardDAO

Tests now extends one abstract class, which contain the common methods for all DAOTests.
FlashcardDAO is changed, so that Flashcard entity now contain deckID

First: it should compile. :)

Rebase this PR, please - it cannot be merged cleanly anymore.

Thanks!


Support of PGP signed commits and tags

Just showing, who signed them will be great

+1. Would be awesome to have this in Gogs now that it's been deployed as a GitHub feature!

+1

Agree that this should be a thing.

I would very much so enjoy this feature, as well!

I'm claiming this :)

I am adding my "voice" to this request. I love Gogs!

I would love this too!

I'd really wanna do this in the frontend only, since Gogs as a service don't really care. Anyone against that?
Note to Self: https://github.com/openpgpjs/openpgpjs

It will be an opportunity to implement this feature?


fix: Switched isOfferableFromInquiry check to isOfferable

This PR solves NX-3114
The type of this PR is: Bugfix.
Since we moved all offers to go through conversations, we switched the check for the counteroffer modal from isOfferableFromInquiry to isOfferable.
cc @artsy/negotiate-devs
Co-authored-by: Carlos Barreto<EMAIL_ADDRESS>Co-authored-by: Lidiane Taquehara<EMAIL_ADDRESS>

I love the teamwork for this ticket! 💛


Update tutorial 7 and related code

Update tutorial 7 and related code

Update explanations and expand function calls so that people see all the options
clean up tutorial by proofreading and removing importlib calls from when I was developing
Update new code for multitask model comparisons by making the column names the same between different compare_models functions
Update plotting code to count colors correctly, and not create a regression plot for a classification model

I think the ad hoc tutorials are more of a place holder right now and aren't meant to be rigorous. They're also out of order, the numbers aren't meaningful.

Defer the change request for later release. Thanks.


Fix deadlock on init

This should always initialise the library correctly without dead-locking when nesting LIBRARY_INIT and LOCK
Fixes #174

That's great! Yea, that should fix it. Maybe update the changelog as well (in the 'fixed' section)? Or is it sometime we have introduced past 0.7.1?
(Few cosmetic comments aside, looks good to go if it's all green)

👍 Great. I'll rebase my loc branch on that then and push it shortly


macs: add apple-virt, nixos-test system features

I think this was tested, and you discussed it in the infra room, but I missed the outcome. Can someone bring me up to speed?

It was applied to one machine (cosmic-stud) and it "worked" in that they weren't being cancelled / aborted / whatever'd anymore, but the tests taking advantage of this would just hang (c.f. https://github.com/NixOS/nixpkgs/pull/303597#issuecomment-2130121108 and https://github.com/NixOS/nixpkgs/pull/303597#issuecomment-2135345269)

As it was always hanging, I rolled back the rhea's machines file, so that scheduler wouldn't try to run it.

Is this good to merge?

Running those jobs doesn't seem to work.  They hang and time out.

Closing. Feel free to reopen if we want to spend time figuring out why they're hanging.


Rotations for map tilesets not triggering

only one corner gets the corner prefabs.

fixed in https://github.com/hunterdyar/daily-golf-project/commit/39793f5779f1220de062edb6fef00cfa33f9ee1d
I was rotating the face check. oops.


DropItemEvent.Destruct is called when inventory is kept

I am currently running

SpongeForge version: spongeforge-1.12.2-2838-7.1.9-RC3980

Issue Description
Gist
The DropItemEvent.Destruct is still called when keepInventory is true. And the event.getEntities() size is 0.

https://github.com/SpongePowered/SpongeCommon/blob/e03eff307a7a51f57d1e386faeb06cca2925d338/src/main/java/org/spongepowered/common/event/tracking/phase/entity/EntityDeathState.java#L112-L113

https://github.com/SpongePowered/SpongeCommon/blob/e03eff307a7a51f57d1e386faeb06cca2925d338/src/main/java/org/spongepowered/common/event/tracking/phase/entity/EntityDeathState.java#L112-L113

But it still shouldn't be called. It is not a mod event

It's tied to the mod event though. With the way Sponge works, in a modded environment, it strives to call events that have a Forge equivalent. In Forge, the LivingDropsEvent is equivalent. And, if we have that behaviour in SF, then we really should make it consistent and do it in SV too. Besides, the explanation why is given right in that comment.
Working as intended - gives flexibilty in case mods add to the drop list and it gives plugins a sane time to drop items itself - no items might be dropped but a plugin might want to drop some sort of skull, this is the time to do it. So it should be fired.
For your case, an "empty" event is easy to detect and ignore - that's what you're going to have to do.


Scheduled Value (in ComfyUI node) seems to do nothing?

Thanks for making this node for Comfy.  Maybe more of a question than a bug...   It's super hard to find detailed info on what some of the features do so I've been doing XY testing.   In that testing, it appears that the sched_val  setting does not change anything.   Is it there just for organizational purposes ie to keep track of what the user intends to set CFG in the sampler? or was it there to do something? Or did it come from the A1111 port?

[DT] Scheduler Value if that's what you're referring to, is part of Dynamic Thresholding, which is documented in detail @ https://github.com/mcmonkeyprojects/sd-dynamic-thresholding/wiki
If you view the parameter in the normal param list (under Advanced in this case), there's a ? you can click for in-UI documentation

If you meant the Scheduler Type, that's a sub-parameter of Sampler. Auto WebUI combines Sampler+Scheduler into one (eg "DPM++ 2M Karras" is the sampler "DPM++ 2M" with the scheduler "Karras"). Sampler specifics are fairly minor and don't need to be fiddle with much.

Hey, I read the wiki first, didn't find any info on that setting but thank you for the quick reply.      Is it perhaps only use when you have Power Up selected?  Power-up needs a variable?

sched_val is Scheduler Value, as explained in the screenshot above and also at https://github.com/mcmonkeyprojects/sd-dynamic-thresholding/wiki/Schedulers


missing mountpoints on some drives on linux

same issue as #68
here is what is returned for my root drive:
description:"Hitachi HDT72101"
device:"/dev/sda"
mountpoint:null
name:"/dev/sda"
size:"931.5G"
system:true
Mountpoint is null. This drive is mounted via UUID:
UUID=7af69416-051a-4149-a3e8-9b59becb56f7 /               ext4    errors=remount-ro 0       1
I thought this was fixed recently?  now it is back
I have 3.1.0 installed...

Hi @squirrelpiles ,
That's weird. Maybe there is a corner case we didn't consider.
What's the output of blkid -s UUID -o value /dev/sda* and grep "^/dev/disk/by-uuid/7af69416-051a-4149-a3e8-9b59becb56f7" /proc/mounts | cut -d ' ' -f 2 | tr '\n' ','?

dave@lucas:~$ blkid -s UUID -o value /dev/sda*
7af69416-051a-4149-a3e8-9b59becb56f7
4139d758-2cdd-4cd8-8455-e101ccadc2ca  <-- this is my swap
dave@lucas:~$ grep "^/dev/disk/by-uuid/7af69416-051a-4149-a3e8-9b59becb56f7" /proc/mounts
/dev/disk/by-uuid/7af69416-051a-4149-a3e8-9b59becb56f7 / ext4 rw,relatime,errors=remount-ro,data=ordered 0 0
/dev/disk/by-uuid/7af69416-051a-4149-a3e8-9b59becb56f7 /var/lib/docker/aufs ext4 rw,relatime,errors=remount-ro,data=ordered 0 0
it appears to work.  but here is another piece of info:
when I first entered blkid -s UUID -o value /dev/sda*, it returned nothing.  So I figure I need to sudo it.  Then it worked.  But then it also then worked without sudo.
Googling and reading man page shows that blkid has a cache file.  I am thinking that there are times when the cache is out of date.   The cache on ubuntu is at /etc/blkid.tab
Also, many places seem to indicate that blkid should be run as root, so I think that is probably the issue.
Do this:

delete /etc/blkid.tab
as a non-root user, do:  blkid -s UUID -o value /dev/sda*
you will get no results
now run:  sudo blkid -s UUID -o value /dev/sda*
you get results.
as a user again: blkid -s UUID -o value /dev/sda*
magic - you now get results.

I played around and it also seems that things like USB sticks etc will show with a non-root blkid command
I tried to use lsblk -O MOUNTPOINT,UUID   which returns the right info when run as root.  If you run as user, you get mountpoints but not UUIDs.   Which is sort of indicating that the UUIDs are not visible to non-root users?
Couldn't you derive all the info from a df command?

Great observations!
I think df could be a pretty good alternative, I'll give it a go when I have access to a Linux box.
What happens if you pass /dev/null as a cache for blkid? From the man page: balked -c /dev/null -s UUID -o value /dev/sda*.

Hm, I just tried the steps you mentioned above, and couldn't reproduce. blkid seems to return the correct UUIDs as root and as non-root, with any cache file. Maybe you can custom sudoers settings or something like that?
In any case, I'll try to recreate my work using df, so it doesn't bite any other user.

Turns out I can reproduce by forcing the tool to create a cache file in a custom location, and trying to re-use it back.

Using something like df --output=source,target | grep "^/dev/sda*" and joining the values of the second column should be good enough.

For me it doesn't find drive when /proc/mounts contains something like:
/dev/disk/by-label/DOROOT / ext4 rw,relatime,errors=remount-ro,data=ordered 0 0

Hi @ilyabreev ,
What happens when you run the following command: df --output=source,target? Does that lists your drive with the corresponding mount point?

Yeah, it perfectly does.
As a temporary workaround I added search by label.

Awesome, thanks lot, I'll publish a fix soon, and probably ping you to test it out if you don't mind!

@ilyabreev Let me know if https://github.com/resin-io-modules/drivelist/pull/80 fixes the issue for you! If so, I'll merge and publish a new version with it right afterwards :)

I did a slight change to the PR to account for multiple mount points.

Thank you for such a quick fix.
Now linux.sh returns:
device: /dev/vda
description:
size:<PHONE_NUMBER>2
mountpoint: /,
name: /dev/vda
protected: False
system: True

It seems ok but look at the mountpoint. It has extra comma at the end of string.

I think it's because of tr behaviour of replacing \n to ,. It'll be better to strip that trailin comma.

Hi @ilyabreev ,
You should be able to ignore it. The actual parser that transforms linux.sh output to JSON takes care of stripping it out. We were previously doing in the script itself, but was way easier to do it in the JavaScript side. You can run example.js in the top-level directory to confirm this.

Ah, that's my fault, sorry. Got it.

@ilyabreev I've released v3.2.1 to NPM with the fix! Thanks a lot for reporting and testing!

@jviotti Thanks a lot! Updated and it works like a charm!


Allow moderators to add content warnings

Todo: need list of content warnings

My understanding is that we are doing a single label, no additional text now. Content will be added to the FAQ for more specific content and phone numbers to call.


Home Assistant doesn't detect ha-bridge

I'm trying out Home Assistant and it does not detect ha-bridge. I've tried unplugging my real Hue hub, during the process, as well as reinitializing ha-bridge during HA's attempts to find it; with no luck.
HA probably has some manual ip entry, but I haven't found it yet.

Is your habridge running on port 80? In 2 seperate homes I have automated, more than half of the devices and services would not find the habridge unless it was running it on port 80. Now I just set them all up on port 80 by default.

@jbardi this habridge runs on port 80

@pr3sidentspence here's a guess blog that got home assistant running with the ha bridge.  Check it out http://fearoffish.com/home-automation/home-assistant-and-ha-bridge/

Yes, port 80.
On Tue, Jan 24, 2017 at 1:02 AM Matt8119<EMAIL_ADDRESS>wrote:

@pr3sidentspence https://github.com/pr3sidentspence here's a guess blog
that got home assistant running with the ha bridge. Check it out
http://fearoffish.com/home-automation/home-assistant-and-ha-bridge/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/385#issuecomment-274725635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArbCcoAwcZyD_97ZmLc_q4n6Et6aQhKks5rVaIcgaJpZM4Lr3oR
.

Yeah, I found that blog post, too. However he was going the other way,
discovering hass devices in ha-bridge.
On Tue, Jan 24, 2017 at 1:55 AM Mike Ellis<EMAIL_ADDRESS>wrote:

Yes, port 80.
On Tue, Jan 24, 2017 at 1:02 AM Matt8119<EMAIL_ADDRESS>wrote:
@pr3sidentspence https://github.com/pr3sidentspence here's a guess blog
that got home assistant running with the ha bridge. Check it out
http://fearoffish.com/home-automation/home-assistant-and-ha-bridge/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/385#issuecomment-274725635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArbCcoAwcZyD_97ZmLc_q4n6Et6aQhKks5rVaIcgaJpZM4Lr3oR
.

I have this working. I had to re install Home Assistant with the latest version.

Hmm.. I just installed Home Assistant yesterday, using pip3. Did you get it some other way?

Sorry, I was wrong on that...I added a vera...

so, is this anjust for fun exercise or a need....

Was a need. But I've switched to using hass + fauxmo to control my old legacy x10 stuff and am no longer using ha-bridge.

Will this actually work? I have some mqtt devices setup on ha-bridge  I want hass to discover ha-bridge  so I can use the devices, else I'll have to duplicate mqtt calls in hass
hass is currently giving me error messages


Azure Databricks Access Connector Resource Onboarding Request

Resource Provider: Microsoft.Databricks

Description of Feature or Work Requested: We are onboarding a new resource under our namespace, we want to enable Az CLI operations for CRUD and Add/Remove Identity to the resource.

Minimum API Version Required: 2022-10-01-preview

Swagger Link : https://github.com/Azure/azure-rest-api-specs/pull/19759

Target Date : 2022-10-31

This feature is required by Walmart

Databricks

@mvvsubbu,

When will the swagger be merged into public swagger repo?

What's the release phase for the service? private preview? public preview? GA?

We are doing Private Preview by 5/30
Public Preview

System Assigned Identity Support by 6/30
User Assigned Identity Support by 9/30

Merging of Swagger to Public Repo will be done ~6/30
For User Assigned Identity I will add new API version after 6/30 in Public Repo
We are looking for Az CLI release by 9/30 milestone for both System Assigned and User Assigned Identities Support
GA would be by end of the calendar year

We have completed milestone 1 targeted at 6/30 and published swagger for that milestone in Public Repository with API version 2022-04-01-preview: https://github.com/Azure/azure-rest-api-specs/tree/main/specification/databricks/resource-manager/Microsoft.Databricks/preview/2022-04-01-preview
Working milestone 2 for which is targeted to be released by 10/31.

Can you help get this CLI released as well for 2022-04-01-preview version, we have PS already got released. So it would be good to have this released as well to be on par experience for customers.

We'll plan it in this sprint and try to release it early next month.

Close this one as this feature was supported in https://github.com/Azure/azure-cli-extensions/pull/5600

Thanks for helping us with onboarding Az CLI for this feature.


Fix lint breaks

My commits weren't captured in the last pull request. I've put them here for your review. Thanks!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jake Neal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Yes, there is plenty more red to squeeze out of our Typesafe config

This patch introduces a new trait WellcomeApp, which is extended by the Main object in all of our apps. It cleans up a bit of boilerplate and repetitive code, and makes simpler Main objects in each app.
It also removes a couple of now-defunct stop() methods, and fixes the Sierra transformer.

And hey, we just kicked Scalaz and cats out of our common classpath. 😜

I think the dependencies changes might be too restrictive: registrar_http, progress_http and notifier use circe optics in the test assertions, I believe. Also, archivist and registrar need cats definitely, not sure if that comes from somewhere else

Yo @alexwlchan, what does this need to be closed?

Closing as abandoned.


Problem inserting Unicode into MSSQL with non-parameterized queries

Setup

Python: 2.7
pyodbc: 4.0.16
OS: Debian 8
DB: MSSQL2014 (Latin1_General_CI_AS encoding), but the issue was also present on MSSQL2005
driver: FreeTDS 7.1

Issue
If I insert unicode characters passed as parameters, everything works as expected:
cur = cnxn.cursor()
q = cur.execute(u'insert into person (surname) values (?)', u'Ż')
q.commit()
cur.execute('select top 1 surname from person order by id desc').fetchall()
[(u'\u017b', )]
But if the value is included inline, the character gets corrupted:
q = cur.execute(u'insert into person (surname) values ('Ż')')
or
q = cur.execute(u'insert into person (surname) values (N'Ż')')
q.commit()
cur.execute('select top 1 surname from person order by id desc').fetchall()
[(u'{', )]
Decoding Unicode is always fine, and inserting such characters via FreeTDS' tsql tool also works fine

Is your Python source file saved as a UTF-8 encoded text file? Have you included
# -*- coding: utf-8 -*-

as the first line of that file?

Yes, it's a utf-8 file with the coding string at the top

Please post the results of tsql -C.

Compile-time settings (established with the "configure" script)
Version: freetds v0.91
freetds.conf directory: /etc/freetds
MS db-lib source compatibility: no
Sybase binary compatibility: yes
Thread safety: yes
iconv library: yes
TDS version: 4.2
iODBC: no
unixodbc: yes
SSPI "trusted" logins: no
Kerberos: yes
Since it wasn't compiled with a higher TDS version as default, I connect with "TDS_VERSION=7.1"

I was able to reproduce your issue with FreeTDS 0.91, the version that was installed via the Ubuntu 16.04 repositories.
The problem did not occur after I downloaded the latest stable version of FreeTDS, currently
ftp://ftp.freetds.org/pub/freetds/stable/freetds-1.00.40.tar.gz
built it using
./configure --with-tdsver=7.3 --enable-msdblib
make
sudo make install

and then replaced my old entry in /etc/odbcinst.ini ...
[FreeTDS_ODBC]
Description=FreeTDS ODBC
Driver=/usr/lib/i386-linux-gnu/odbc/libtdsodbc.so

... with this:
[FreeTDS_ODBC]
Description=FreeTDS ODBC
Driver=/usr/local/lib/libtdsodbc.so

And it works!
Thank you so much.  I'd been trying to deal with this issue for an absolute age.


Playlist Problem

Bug description
So there is this thing or problem, not on the main bot, but when i forked the code and tried it i tried to play a spotify playlist, it worked but instead of playing the same song on the playlist it would play that song but a different version that it found or another song from that artist if there were some songs from that same artist.
This wouldn't happen on every song but on some of them.
I don't know if this is only happening for me or for all, but i haven't seen anyone talk about it so i don't know.
Step to reproduce
Im not very sure, i tried to find a solution but no luck.
Operating system
Linux (replit)
Nodejs version
The version that comes with the bot.
Errors / Logs
No errors show in the console or the error channel.

its just the lavalink issue... try restaring ur lavalink server or use another lavalink server.


HTML string inside TypeScript and JavaScript files?

When using Angular 2 and TypeScript I noticed that editing template strings of components in TypeScript files are not working as one expect. There is not autocomplete for tags and other usual HTML editing features.
For example
import {Component} from 'angular2/core';

@Component({
    selector: 'my-app',
    template: '
My First Angular 2 App
'
})
export class AppComponent { }

Of course it's because the type definition has declared template as a String and no part of VSCode knows that it's an HTML string.
My proposal is to define a HTMLString type where it's basically just string but helps tooling in cases like this.
If it was possible to do this, then we can define CSSString, JavaScriptString, SCSSString and so on...

:+1:

I copied the item to the TS repository to hear their opinion: https://github.com/Microsoft/TypeScript/issues/6778

Consider using templateUrl, and specifying a HTML file name.  You can then get all the autocomplete in that HTML file. I think that is better than mixing code and HTML like Classic ASP.
Example:
https://github.com/sheldonbarnes/Angular2Learning/blob/master/app/app.customerdetailscomponent.ts

#1751

I don't know if this would help in VSCodes case, but @GregOnNet came up with a brilliant solution for adding HTML highlighting to the ` ` back-tick section here for Atom-typescript
https://github.com/TypeStrong/atom-typescript/pull/948
Hoping that might help potentially add this ability to VSCode!

@aeschli please see the latest comment. With what we discussed last week can we achieve them?

@dbaeumer @egamma The solution mentioned above adds the html/angular syntax highlighting directly to the TypeScript grammar. That's one of the options we discussed last week. It would be up to the TypeScript team to decide that. Alternatively we work on adding injection support: https://github.com/Microsoft/vscode/pull/6331.

In the meantime looks like this extension works pretty well:
https://marketplace.visualstudio.com/items?itemName=nwallace.language-vscode-javascript-angular2

@dbaeumer @egamma @juliusl @aeschli
Well the color works with the extension, but the autocomplete of
-> custom html tags, in a angular2 component we call it a selector:.
template: `
          <color-pi...

will not complete.

True, it doesn't autocomplete html but at least it'll help you with variables you have within that file.
Let's say you had public someString : string; in your class, when you do {{ som...(it'll show someString). Better than nothing I suppose ;)

https://marketplace.visualstudio.com/items?itemName=natewallace.angular2-inline

Just for other people finding this thread the link above is the working one. The other extension from the same author no longer works with the current version of vscode as far as I know.
The extension does provide:

intellisense
and coloring

But sadly NO formatting. There is an open feature request for formatting, give it some love if you also wanne see it included too: https://github.com/natewallace/angular2-inline/issues/6

Is it desirable to turn this into a more generic solution? What if my backtick contained some JS code for evaulating (yuck), or more likely, I'm providing some inline scripting to an Elasticsearch query?

https://github.com/Microsoft/vscode/issues/1751

@robhaswell Also SQL would also be awesome

This would be quite useful for hyperHTML/viperHTML and lit-html as well
https://github.com/PolymerLabs/lit-html
https://viperhtml.js.org/

Meanwhile I found this. https://github.com/Microsoft/typescript-lit-html-plugin

Yes for angular, try the angular-inline extension and for lit-html, try:

lit-html syntax highlighting extension: https://marketplace.visualstudio.com/items?itemName=bierner.lit-html
Lit-html IntelliSense TypeScript plugin: https://github.com/Microsoft/typescript-lit-html-plugin

I posted more information about embedded language support in https://github.com/Microsoft/vscode/issues/5961#issuecomment-335964652 as well. To sum up, you can already write extensions and TypeScript plugins that contribute support for embedded languages. The main remaining work is to polish the experience so that you only need to install a single vscode extension to get syntax highlighting and intellisense for your template strings. This work is now being tracked by #5961
Closing this issue in favor of #5961

A bit more context. The feature used here is called Tagged_template_literals
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals#Tagged_template_literals
It actually gives the ability to effect replacements on the fly like (null values etc).
Since it tags the following code it can than easily be used to switch to the right language for formatting. Detecting the tag and mapping that to a formatting is all the extension does.
There are literal tags for SQL already, not sure about T_SQL though:
https://www.npmjs.com/package/sql-tagged-template-literal
https://www.npmjs.com/package/sql-template-strings
So all one has todo for SQL formatting one is to create a simple extension detecting the tag and using one of the available sql formatters.
Instead of building one extension for each tag I would suggest todo one of the following:

Create a generic extension that allows a to configure a mapping of file extension + tag to vscode formatter  so that we don't need tens of extension for inline formatting.
Or instead of a generic extension build this config based mapping capability into VSCode directly.

This is basically an extension of the existing approach as vscode already allows to specify formatting for unknown file types via config.
It should be off by default to not impact perf.

I personally vote for the second option as I feel:

the IDE should handle all cases of formatting detection if it already handles file extensions and
it being very straight forward and not bloating

Here is a super simple reference to build your own literal tag:
http://exploringjs.com/es6/ch_template-literals.html#sec_implementing-tag-functions


Regex Find/Replace works with numbered capture groups but fails with named capture groups

Type: Bug
The replace field in the tool fails to parse most syntax related to regex substitutions beyond the basic numbered capture groups.  There are several issues opened for closely related problems with find replace that are probably the same issue (incomplete/broken implementation of regex in the replace field) but they're not an exact match.
To Reproduce:
Create a find regex with a named capture group that should match with the find/replace tool bound to ctrl+f.  E.g.:
text in file:    beforetextToMatchAndCaptureafter
find:    before(?textToMatchAndCapture)after
replace:    $
expected:    the captured text between the (), literally "textToMatchAndCapture"
actual:    the literal $
related: $1 does the substitution correctly.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.83.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

No change on 1.83.1

See (Backlog Cnadidate): https://github.com/microsoft/vscode/issues/127657

I see that issue is closed, but don't see when/whi/why.
Is it still really in the backlog of closed?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@joshuaking-madtech
You should use the following replace (curly braces)
text in file:    beforetextToMatchAndCaptureafter
find:    before(?<myNamedGroup>textToMatchAndCapture)after
replace:    ${myNamedGroup}

This used to work 2 weeks ago, however it seems something went wrong in the past weeks

@PieterjanDeClippel I don't remember named capture groups ever working in vscode.
You can find the commit that botched the feature by doing this:
Finally, to help us narrow down what change caused the issue, please run the following command
to find the build. It will start previous releases of VS Code insiders to see where the
regression was introduced:

* npx --yes vscode-bisect@latest
* follow the instructions until you found the offending build
* report back the commit range in this issue

from https://github.com/microsoft/vscode/issues/182758#issuecomment-1551500597


A quick question - Set Mac address (iBeacon)

A quick one; Is it possible to set the mac address when using this iBeacon example?
I have a peculiar case where I have no choice but to change the mac address. In short the other scanners should see the mac address that I have set rather than the real mac address of the iBeacon. Any pointers? Thanks.

I have not implemented this as a library function yet but it is possible. Do you need a random address that changes often like on phones or just a different address that you set yourself?

Thanks for the quick reply. I have a fixed address that I need to set.

For that purpose you’ll have to refer to this document as it involves esp function calls.
When I can get on my pc I’ll check how it is set via NimBLE when it sets a random address.

OK cool thanks. I'll check out the documentation and see what needs to be done ... but I am guessing I'll need to change the base_mac ... probably something like
uint8_t new_mac[8] = {0x01,0x02,0x03,0x04,0x05,0x06};
esp_base_mac_addr_set(new_mac);

Gona try in a while. Thanks once again @h2zero

On the first look, it seem like esp_base_mac_addr_set works ... just gota remember that for the Bluetooth mac (4 universally administered) it adds base_mac, +2 to the last octet.

If the number of universal MAC addresses is two, only two interfaces (Wi-Fi Station and Bluetooth) receive a universally administered MAC address. These are generated sequentially by adding 0 and 1 (respectively) to the base MAC address. The remaining two interfaces (Wi-Fi SoftAP and Ethernet) receive local MAC addresses. These are derived from the universal Wi-Fi station and Bluetooth MAC addresses, respectively.
If the number of universal MAC addresses is four, all four interfaces (Wi-Fi Station, Wi-Fi SoftAP, Bluetooth and Ethernet) receive a universally administered MAC address. These are generated sequentially by adding 0, 1, 2 and 3 (respectively) to the final octet of the base MAC address.
When using the default (Espressif-assigned) base MAC address, either setting can be used. When using a custom universal MAC address range, the correct setting will depend on the allocation of MAC addresses in this range (either 2 or 4 per device.)

Number of universally administered MAC address

Yes that would be the case, so if you wanted to specify a MAC address exactly you would have to -2 on the last octet so if you wanted 11:22:33:44:55:66 you would have to set it to 11:22:33:44:55:64.

Okay cool ... now messing with one little thing; the name of the device does not appear in the scanner application. I can see the mac address and the advertisement string but the name does not appear ... it's empty
From the scanner app
Advertised Device: Name: , Address: 62:c1:7a:74:27:69 type: 1, manufacturer data: 4c001005XXXXXXXXXXXXXXXXXX, txPower: 12
I tried setting the iBeacon name in the BLEDevice::init("NAMETEST"); ... as well as using setName() , setShortName() (for BLEBeacon()) etc.
I might be missing a tiny something here.

You'll need to set the name in the scan response data, the advertising packet doesn't have room.

Ok cool. I'll experiment with the scan response and see how it goes. A big thanks @h2zero.

Okay cool ... now messing with one little thing; the name of the device does not appear in the scanner application. I can see the mac address and the advertisement string but the name does not appear ... it's empty
From the scanner app
Advertised Device: Name: , Address: 62:c1:7a:74:27:69 type: 1, manufacturer data: 4c001005XXXXXXXXXXXXXXXXXX, txPower: 12
I tried setting the iBeacon name in the BLEDevice::init("NAMETEST"); ... as well as using setName() , setShortName() (for BLEBeacon()) etc.
I might be missing a tiny something here.

Thought I'll mention this; the below worked for me. Earlier I might have missed something and that's why it didn't work.
Example: https://github.com/h2zero/NimBLE-Arduino/blob/master/examples/Refactored_original_examples/BLE_iBeacon/BLE_iBeacon.ino
One can use either setName() or setShortName() depending on what's needed on the other end.
void setBeacon()
{
  const char deviceName[] = "TestDevice";
  const char deviceShortName[] = "TestDvc";
  BLEBeacon oBeacon = BLEBeacon();
  oBeacon.setManufacturerId(0x4C00); // fake Apple 0x004C LSB (ENDIAN_CHANGE_U16!)
  oBeacon.setProximityUUID(BLEUUID(BEACON_UUID));
  oBeacon.setMajor((bootCnt & 0xFFFF0000) >> 16);
  oBeacon.setMinor(bootCnt & 0xFFFF);
  BLEAdvertisementData oAdvertisementData = BLEAdvertisementData();
  BLEAdvertisementData oScanResponseData = BLEAdvertisementData();

  /* Set name */
  oAdvertisementData.setName(deviceName);
  /* Set Short name */
  oAdvertisementData.setShortName(deviceShortName);

  oAdvertisementData.setFlags(0x04); // BR_EDR_NOT_SUPPORTED 0x04

  std::string strServiceData = "";
  strServiceData += (char)26;   // Len
  strServiceData += (char)0xFF; // Type
  strServiceData += oBeacon.getData();
  oAdvertisementData.addData(strServiceData);

  pAdvertising->setAdvertisementData(oAdvertisementData);
  pAdvertising->setScanResponseData(oScanResponseData);
  /**  pAdvertising->setAdvertisementType(ADV_TYPE_NONCONN_IND);
  *    Advertising mode. Can be one of following constants:
  *  - BLE_GAP_CONN_MODE_NON (non-connectable; 3.C.9.3.2).
  *  - BLE_GAP_CONN_MODE_DIR (directed-connectable; 3.C.9.3.3).
  *  - BLE_GAP_CONN_MODE_UND (undirected-connectable; 3.C.9.3.4).
  */
  pAdvertising->setAdvertisementType(BLE_GAP_CONN_MODE_NON);
}


Update token_handler.py

Type
enhancement, bug_fix

Description

Added a try-except block in _get_system_user_tokens to handle potential exceptions by returning -1. This enhancement improves the robustness of the token handling mechanism by gracefully handling errors instead of raising exceptions to the caller.
Fixed the missing newline at the end of the file to adhere to Python file formatting standards.

Changes walkthrough
Relevant filesError handling

token_handler.pyEnhance Error Handling and Fix Newline in token_handler.py

pr_agent/algo/token_handler.py
Added error handling in _get_system_user_tokens method to return -1 on exception.  Ensured newline at the end of the file.

+10/-7

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/describe


[v118] After multiple reboots, the system progressively becomes more unusable

On the first reboot, everything is still working correctly - on the second, it performs the 5-minute wait sequence, and on the third, it refuses to boot altogether with the "Developer mode disabled by your administrator" screen.

Fix currently in testing.

Everything seems to be working, I'm closing this issue for now but will reopen it if needed.


[Storage][Blob] Hide DataLakeStorageError in code gen

We are not using it yet.
See .NET swagger readme.md for the transformation example

Link to .NET swagger readme: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Blobs/swagger/readme.md
Might need private swagger files from me though as there might be slight differences.

This issue was addressed by PR #5567 and DataLakeStorageError is no longer exported
/cc @HarshaNalluru @ramya-rao-a


Capitalization error in navigation pane

Description
"Drive" in Network drive is not capitalized. All other titles in the navigation pane have both words properly capitalized including Cloud Drives and Recycle Bin as illustrated in the screenshot below.
Steps To Reproduce

Open the app
Look at the navigation pane
See that the "drive" in Network drive isn't capitalized

Expected behavior
Proper proofing.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.469
Windows Version
Windows 11 Pro 21H2 22000.469
Relevant Assets/Logs

@JacksonCampbell99 would you be intrested in trying to fix this?

@yaichenbaum It would be fun if I knew how to code.

Closed with #8357


'light-webapp used JavaScript to provide functionality to the turn ON/OFF buttons.

light-webapp used JavaScript to provide functionality to the turn ON/OFF buttons.
Used html and css to give the basic layout

can you tell me what you have done @Mahak129

Yeah sure it's just a webpage in which I had used Javascript to provide
functionality to the buttons (ON/OFF) .As one click on the button it will
change accordingly.
On Thu, 6 Oct 2022, 01:59 Intellimixio PCE Purnea, @.***>
wrote:

can you tell me what you have done @Mahak129 https://github.com/Mahak129
—
Reply to this email directly, view it on GitHub
https://github.com/Intellimixiopce/intellimixio/pull/18#issuecomment-1268938917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUDP22KKTZFKPKBABSTSURTWBXQLBANCNFSM6AAAAAAQ44QF7E
.
You are receiving this because you were mentioned.Message ID:
@.***>

means dark mode? @Mahak129

Would be glad if you will merge my PR.
On Thu, 6 Oct 2022, 02:21 Mahak Soni, @.***> wrote:

Yeah sure it's just a webpage in which I had used Javascript to provide
functionality to the buttons (ON/OFF) .As one click on the button it will
change accordingly.
On Thu, 6 Oct 2022, 01:59 Intellimixio PCE Purnea, <
@.***> wrote:

can you tell me what you have done @Mahak129
https://github.com/Mahak129
—
Reply to this email directly, view it on GitHub
https://github.com/Intellimixiopce/intellimixio/pull/18#issuecomment-1268938917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUDP22KKTZFKPKBABSTSURTWBXQLBANCNFSM6AAAAAAQ44QF7E
.
You are receiving this because you were mentioned.Message ID:
@.***>


examples/custom_3d_three-d project, I wasn't able to get to compile for wasm

This doesn't seem to be a bug - just a compilation error
In the examples/custom_3d_three-d project, I wasn't able to get to compile for wasm using the prescribed

wasm-pack build examples/custom_3d_three-d --target web

instruction because it complained of missing
).await?
Once I added it, it worked.  If appropriate, I can make the PR with the modified code.

This example is no longer there


add unified query interface guide

Love this tutorial! I think this really emphasizes the power of llama index, and gives some examples that are really needed (like building a graph index that includes other graphs!)

Love this tutorial! I think this really emphasizes the power of llama index, and gives some examples that are really needed (like building a graph index that includes other graphs!)

yes! i think it's much needed :)


Ajax - Signin Not working

Started POST "/users/sign_in" for <IP_ADDRESS> at 2017-09-26 17:25:15 +0530
Processing by User::SessionsController#create as JS
Parameters: {"utf8"=>"✓", "authenticity_token"=>"Fozvt4xIkxj+IuOZg56YU6/LBHGuAbjVbjE+xN8CcixZ+rf/mvzIT71pW2tB/6DstQ0KgYufG9e2sKKGbpBRKQ=="<EMAIL_ADDRESS>"password"=>"[FILTERED]"}, "button"=>""}
User Load (0.4ms)  SELECT  "users".* FROM "users" WHERE "users"."email" = $1  ORDER BY "users"."id" ASC LIMIT 1  [["email"<EMAIL_ADDRESS>Completed 401 Unauthorized in 4ms (ActiveRecord: 0.4ms)
I want to call create.js.erb but always ingnored

Please use StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only. Thank you!


Update to null safety

I fixed it for null safety.
#49

Thanks a lot. I fear that the developer here has gone dark.
Does your fork also contain the bufixes (#45 and #43) from the other fork #46?.

Yes, I was not aware of that issue, but I fixed it.

You can check it by writing it in pubspec.yaml.
calendarro:
  git:
    url: https://github.com/kitwtnb/calendarro.git
    ref: master


⚠ Bofrost.es has degraded performance

In 5ae7126, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1603 ms

Resolved: Bofrost.es performance has improved in be6dca0 after 6 minutes.


Add Security-Bot

There are entry points for code injection. Code in repo needs to be checked to not be malicious.

See https://github.com/janelia-cellmap/dacapo/settings/security_analysis

We looked into this and we have dependabot check dependencies for vulnerabilities and open patch PRs. We also have github code scanning detect vulnerabilities in our code. It would be hard to check user supplied functions for maliciousness - probably not within scope at this time.


add link to best practices for pull_request_target

I was initially confused by the intended use of pull_request_target and its capabilities, but this blog post in the Security Lab laid out the caveats and solutions well.

Why:
Closes #2264
What's being changed:
A single sentence being added to the documentation.
Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Thanks so much for following up and opening a PR so quickly @zkamvar! I'll get this triaged for the team to take a look 👀


Allow React 18 peer dep

This would be nice!

Maybe if we ping one of the mainteners @martijnrusschen 🙏

+1

1 test is failing

Sorry for the delay on this. The failing test seems unrelated but I'd love to get that fixed. If someone has time to look into that I'd really appreciate it!

New to the project but going to dig it to try to help resolve. Important to get this working with React ^18.1.0

Merging in the latest master branch should fix the build for this pull request.


nvm command not found using github actions, works in shell

Operating system and version:
Debian GNU/Linux 11 (bullseye)
nvm debug output:

nvm --version: v0.39.1
$SHELL: /bin/bash
$SHLVL: 1
whoami: 'github-actions'
${HOME}: /home/github-actions
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v16.16.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 5.1.4(1)-release (x86_64-pc-linux-gnu)'
uname -a: 'Linux 5.10.0-15-cloud-amd64 #1 SMP Debian 5.10.120-1 (2022-06-09) x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Debian GNU/Linux 11
curl: /usr/bin/curl, curl 7.74.0 (x86_64-pc-linux-gnu) libcurl/7.74.0 OpenSSL/1.1.1n zlib/1.2.11 brotli/1.0.9 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.3.0) libssh2/1.9.0 nghttp2/1.43.0 librtmp/2.3
wget: not found
git: /usr/bin/git, git version 2.30.2
grep: /usr/bin/grep, grep (GNU grep) 3.6
awk: /usr/bin/awk, awk: not an option: --version
sed: /usr/bin/sed, sed (GNU sed) 4.7
cut: /usr/bin/cut, cut (GNU coreutils) 8.32
basename: /usr/bin/basename, basename (GNU coreutils) 8.32
rm: /usr/bin/rm, rm (GNU coreutils) 8.32
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 8.32
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.8.0
nvm current: v16.16.0
which node: ${NVM_DIR}/versions/node/v16.16.0/bin/node
which iojs:
which npm: ${NVM_DIR}/versions/node/v16.16.0/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v16.16.0
npm root -g: ${NVM_DIR}/versions/node/v16.16.0/lib/node_modules
nvm --version: v0.39.1
$SHELL: /bin/bash
$SHLVL: 1
whoami: 'github-actions'
${HOME}: /home/github-actions
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v16.16.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 5.1.4(1)-release (x86_64-pc-linux-gnu)'
uname -a: 'Linux 5.10.0-15-cloud-amd64 #1 SMP Debian 5.10.120-1 (2022-06-09) x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Debian GNU/Linux 11
curl: /usr/bin/curl, curl 7.74.0 (x86_64-pc-linux-gnu) libcurl/7.74.0 OpenSSL/1.1.1n zlib/1.2.11 brotli/1.0.9 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.3.0) libssh2/1.9.0 nghttp2/1.43.0 librtmp/2.3
wget: not found
git: /usr/bin/git, git version 2.30.2
grep: /usr/bin/grep, grep (GNU grep) 3.6
awk: /usr/bin/awk, awk: not an option: --version
sed: /usr/bin/sed, sed (GNU sed) 4.7
cut: /usr/bin/cut, cut (GNU coreutils) 8.32
basename: /usr/bin/basename, basename (GNU coreutils) 8.32
rm: /usr/bin/rm, rm (GNU coreutils) 8.32
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 8.32
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.8.0
nvm current: v16.16.0
which node: ${NVM_DIR}/versions/node/v16.16.0/bin/node
which iojs:
which npm: ${NVM_DIR}/versions/node/v16.16.0/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v16.16.0
npm root -g: ${NVM_DIR}/versions/node/v16.16.0/lib/node_modules

nvm ls output:

->     v16.16.0
         system
default -> lts/* (-> v16.16.0)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v16.16.0) (default)
stable -> 16.16 (-> v16.16.0) (default)
lts/* -> lts/gallium (-> v16.16.0)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.20.0 (-> N/A)
lts/gallium -> v16.16.0

How did you install nvm?
sh install script.
Issue
Running nvm use or any nvm command using GitHub actions in container (appleboy/ssh-action) returns: err:  nvm: command not found
If I however ran this in shell normally it is functional. I suspect this is an issue with the way nvm is being installed on the machine, and not being accessible by docker?
Is there anything in any of your profile files that modifies the PATH?
No

I had a similar problem - answered here: https://askubuntu.com/a/1041348/521202
Ubuntu's default .bashrc is configured NOT to do anything when sourced on non-interactive terminals by default. The .bashrc file must be updated before it can be sourced by non-interactive logins.

I had a similar problem - answered here: https://askubuntu.com/a/1041348/521202
Ubuntu's default .bashrc is configured NOT to do anything when sourced on non-interactive terminals by default. The .bashrc file must be updated before it can be sourced by non-interactive logins.

What did you do concretly ?

@amapic
If I remember correctly, I looked for the below code in .bashrc and removed it:
# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

@amapic
If I remember correctly, I looked for the below code in .bashrc and removed it:
# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

Thanks for answering. Here's actually what I did after which seems simpler : adding
script: |
  export NVM_DIR=~/.nvm
  source ~/.nvm/nvm.sh
  npm --help

in the github actiion file. It comes from here : https://github.com/appleboy/ssh-action/issues/21#issuecomment-574050424

@amapic
If I remember correctly, I looked for the below code in .bashrc and removed it:
# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

Thanks for answering. Here's actually what I did after which is perhaps simpler : adding
script: |
  export NVM_DIR=~/.nvm
  source ~/.nvm/nvm.sh
  npm --help

in the github actiion file. It comes from here : appleboy/ssh-action#21 (comment)

This has worked for me. Thanks a bunch.


How to close expandable row on scroll

I am Using Expandable Row Functionality to expand/collapse the Row.
Is there a way to collapse the Row on Scroll as well

You want to close once the element is not in the viewport?

I want to close it, as soon as user starts scrolling

I want to close it, as soon as user starts scrolling

Can you post your code? I can add the necessary code to the top of it


V1.4 DG

Codecov Report

Merging #268 (9bb9f6f) into master (b90e051) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #268   +/-   ##
=========================================
  Coverage     73.51%   73.51%
  Complexity      905      905
=========================================
  Files           105      105
  Lines          2919     2919
  Branches        397      397
=========================================
  Hits           2146     2146
  Misses          662      662
  Partials        111      111

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 b90e051...9bb9f6f. Read the comment docs.


stdout not captured on Windows 8

I was originally facing ricardofbarros/linter-js-standard#30, but I've further debugged the issue and it now looks like there is something awry in atom-linter 2.0.1. More specifically, on Windows 8 at least, Atom 1.0.2 seems unable to capture the child process's stdout.
I instrumented helpers.coffee to log the various stages of process creation. Basically, I see linter-js-standard invoke atom.exe with ATOM_SHELL_INTERNAL_RUN_AS_NODE=1. If the file has errors, then the status code differs, so it is linting. However, there is never any standard output and hence no errors.
In a Windows Command Prompt, I did a set on the variable above and then ran atom.exe with the same arguments. Interestingly, it immediately went to a blank prompt and then showed the output of the linter. It doesn't exhibit this behavior with a regular node.exe.
Additionally, I tried overriding command with node.exe (which is in my PATH), and that actually fixed the issue. Hence, it seems as though Electron run as Node behaves differently than a regular Node. It might very well be that the issue is with Electron instead. In that case, atom/electron#1463 should probably be reopened. However, since I've only seen it occur with atom-linter, I wanted to report it here first.

I have tried linter-jshint on windows 10 beta, and it worked perfectly fine there. Might be a problem with that package :)

That's not necessarily relevant. The problem seems to be with atom.exe running as Node, which is something that linter-jshint doesn't do. That one just invokes jshint directly.

@timdp I think you might not have observed the latest version of it, in the latest version of it it runs atom.exe with ATOM_SHELL_INTERNAL_RUN_AS_NODE=1 passing the jshint bin file as an arg. which works.

Ah, I wasn't aware of that, but you know best, of course. I was looking at main.coffee on master, which currently references the path to jshint pretty clearly.

Oh, wait. I see now that it calls execNode rather than just spawning it. Sorry about that.

Confirmed as a linter-js-standard issue, so I'll close this one. Thanks!


Wrong begin date of DST

Hello,
I use the library without a network connection. Therfore I defined the POSIX TZ rule like this:
#define CET_POSIX	"CET-1CEST,M3.5.0/02:00:00,M10.5.0/03:00:00"		// Time in Berlin
LOC.setPosix(CET_POSIX);

The last Sunday in March this year will be the 31st. Unfortunatly the DST begin date seems to be already on 24th of March. This could be a similar problem with the POSIX day number like #28

In addition to that I figured out that the DST starting time is not at 2:00 as defined in the rule. The DST starts at 3:00, and it seams that the rule applies to UTC instead of CET.

I believe this is fixed by the fix for #38, my makeOrdinalTime function was off when the last of a given weekday in a given month was requested. Please reopen if problem reoccurs.


Multiple nodes on same (LXC) hostserver results in collision

What you expected to happen?
Weave network can setup the network connection between nodes.
What happened?
Connection fails:
local "4a:ef:c1:7c:c3:26(kube-controller1)" and remote "4a:ef:c1:7c:c3:26(kube-worker1)" peer names collision, retry: never
How to reproduce it?
Setup Kubernetes Cluster:
3 bare metal with Debian Stretch.
Every bare metal has 2 nodes:

kube-worker
kube-controller

Because if the same /sys/class/dmi/id/product_uuid on both containers, the weave mac address is the same.
Anything else we need to know?

Versions:

$ weave version
weave version
weave script 2.4.0
weave 2.4.0
$ docker version
docker version
Client:
 Version:           18.06.1-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        e68fc7a
 Built:             Tue Aug 21 17:23:18 2018
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.1-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       e68fc7a
  Built:            Tue Aug 21 17:22:21 2018
  OS/Arch:          linux/amd64
  Experimental:     false

$ uname -a
Linux kube-controller1 4.17.0-0.bpo.1-amd64 #1 SMP Debian 4.17.8-1~bpo9+1 (2018-07-23) x86_64 GNU/Linux

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.1", GitCommit:"4ed3216f3ec431b140b1d899130a69fc671678f4", GitTreeState:"clean", BuildDate:"2018-10-05T16:46:06Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:08:19Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

The need for a unique identity on each machine is described in the docs.


fix: now with more release branch

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


fix: refactor the internal writes

Introduce the "uncommited state" - each contract on the call tree for a given interaction should have its own uncommited state. This state should be used during the evaluation of the internal writes.
e.g. if Contract A writes to Contract B - and we're reading the state of the Contract B:

Read the state of the Contract B up to the internal write interaction
for the internal write interaction - set the uncommited state for contract B to current state
read the state of the contract A (the one that performs write on contract B)
at the time the Contract A tries to perform a write on Contract B - read Contract B state from the contract.parent().uncommitedState, where
contract = Contract A
contract.parent() = Contract B
and perform the write.
The result of the write should be stored in contract.parent().uncommitedState
After returning with evaluiation to Contract B - load its state from contract.uncommitedState

Watch out while evaluating the writes from the 'caller' (i.e. the one that is performing a write) contract perspective.
At the end of the evaluation of each interaction of the 'root' contract (i.e. the that user is trying to read via contract.readState() - commit all "uncommited states" in the root contract's call tree (i.e. store the results in the SDK cache).
This change should also allow to remove the "inf. loop fix" (which protects agains falling into inf. call loop when contracts are interacting with each other (Contract A reads/writes to Contract B, where Contract B reads/writes to A, etc...) and simplify the evaluator code.
This should also allow to achieve https://github.com/warp-contracts/warp/issues/195

since 1.2.49


Update path

What is be the best way to update when there is a new release?
composer update pattern-lab/core? or do I need to do composer update pattern-lab/edition-development-twig?

@TxHawks -
We will have updates to components as opposed to one monolithic update. For example, pattern-lab/core and pattern-lab/patternengine-twig are both components. The Editions should be considered projects and their (very thin) code is not updatable via Composer. You have two options:

run composer update and let every component in your project check for updates
run composer update pattern-lab/[component-name] and update a specific component

For the most part I believe the project dependencies (including what we will do with Pattern Lab) use semantic versioning. This means running the first option should be fairly safe. For example, I've done a lot of work in the Twig PatternEngine. To update it specifically you'd run:
composer update pattern-lab/patternengine-twig

Since you're running the Development Edition you're going to get whatever is in the dev branches for components. This should be considered unstable. I try to be good and only push working code to dev but that doesn't mean it's completely vetted, documented, or finished. As you've seen ;)
The Standard Edition for Twig is my attempt to release a more stable, based-on-real-releases Edition of Pattern Lab. Fingers crossed I finish the documentation for that and its related components today :)

If it's useful to you, for the time being I'm happy to keep using the dev edition for testing. Up to now nothing has been broken in a major way and you've been super responsive and helpful, so it's not a problem

@TxHawks -
Use what you like :) I'm very appreciative of the feedback.


Add missing Z in SubmarineExploration coordinates

Is there anything specific why you thought this might be Z on a 2d map ?

Is there anything specific why you thought this might be Z on a 2d map ?

I only ask because I don't do much subs so not sure, as far as I know it's a 2d map anyway

They went with X,Y,Z for Submarines. The distances between some sectors that may look the same if you only look at them in a 2D pov, aren't. The reason behind that is because of the Z axis, it takes into account the depth. It only applies to submarines. Airship sectors are in a 2D plan, so only X and Y axises.
I use them to compute the estimated time for a trip on mogship.com
If you wanna know more about it, lmk

@IcarusTwine they actually use a 3d map coord system in the time calculation from what I can decode alone and this is what I have figured out so far
TimeToBeDone = 720 + Duration<min> + round((sqrt((x1 - x2) ^ 2 + (y1 - y2)^2 + (z1 - z2)^2) * (j / s)) * 1440)
s = speed of the airship or submarine
j = 0.04143 for airships (59m40s)
j = 0.02765 for submarines (39m49s)
NOTE: this is only an approximation of the time used, as it can deviate from what the game shows.

This does look correct for the Z reference @IcarusTwine.

Perfect ! Thank you for the explanation and great find!

That's almost correct but it's missing a few things.
Each map has a starting sector (airship and submarine).
Airship:  sectors #127
Submarine: sectors #0 (Deep sea site), #31 (Sea of ash), #52 (Sea of jade)
The correct way to get the time is as following:
Trip is A -> B -> C
A: The starting sector of the map, the game doesn't show it
B: The first sector you select
C: The second sector you select
TotalDistance = range A->B + range B->C + distance A->B + distance B->C + timeForSurvey B + timeForSurvey C + rangeForSurvey B + rangeForSurvey C
timeForSurvey = Duration< min > * 70
rangeForSurvey = DistanceForSurvey
distance A->B = floor(sqrt((x1 - x2)^2 + (y1 - y2)^2 + (z1 - z2)^2) * n)
n = 60 (airship)
n = 39.9 (submarine)
range A->B = floor(sqrt((x1 - x2)^2 + (y1 - y2)^2 + (z1 - z2)^2) / n)
n = 19.2265 (airship)
n = 28.5716 (submarine)
days = ((TotalDistance  / speed) / 1440) + 0.5;
hours = ((TotalDistance  / speed) / 60 + 12) % 24;
minutes = (TotalDistance  / speed) % 60;

Didn't have time to comment on this, but some few days after the previous comment me and some other people dug through the in-game code to figure out how the calculation was actually done, documented here. It's somewhat a weird set of values, but this is actually how things are done through the game.


test: find all available deliveries

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+5.2%) to 66.948%

Totals

Change from base Build<PHONE_NUMBER>:
5.2%

Covered Lines:
1135

Relevant Lines:
1751

💛  - Coveralls


Add generators for forms and decorators

Please check TODOs in code

@kalashnikovisme Updated

@haukot one last update, please!!! :)

@kalashnikovisme updated :)

Watch your tongue, young man!

@kalashnikovisme I don't have the access to merge it. Just saying

DONE!

🎉


Make Register a normal metaclass, not ABCMeta

I couldn't find any support (in nearby commented code) for why we needed
the abc-stuff.
https://blog.ionelmc.ro/2015/02/09/understanding-python-metaclasses
Have you tested this? If so, how?
I'm quite skeptical to this myself. I will try it on my setting and I suspect it will break. Don't merge until I self-approve. ^^

It will break any user code (we have some) that depends on the fact that Task is subclassed from ABCMeta. Not sure what you get in return.

@themalkolm yes. I'm thinking of not merging this. Thanks for your opinion.
However, we get quite some things in return. (1) Less complex luigi and (2) no random arbitrary choice that Tasks may if they so desire ABCMeta, but all other metaclases are not allowed.

I think it is released in this state for way too long and way too late to fix.


Random Testing on main and version branches

OpenSearch org needs random testing on the main and supported unreleased minor version branches (e.g., 1x). Without it, PRs bear the burden of finding corner case bugs introduced in merged features.
This just occurred in https://github.com/opensearch-project/OpenSearch/pull/1073  where SimpleFS deprecation PR failed twice w/ a corner case bug not caught in the original PR. Without random testing to catch corner cases like this the project is at the mercy of the law of large numbers where situations like these will occur more frequently and very few PR checks will succeed.
Outstanding tasks:

[ ] I will utilize the core team channel and @channel in slack msg, which already have the fork Jenkins links to run
[ ] In Git Issues I will include the error msg stacktrace, the seed, and the build number for now

I'm a little worried that we are trying to include this in scope for 1.1, given that we're still not running any bundle tests (integ, perf, bcw) daily. Definitely can come next after those.

This is independent of any release. We don't have any random test coverage leading to PRs as the first (and only) line of defense for catching random seeded failures. We can't continue as a project like this. We need random testing on main, 1.x, (and 1.1 when we cut the branch) so we have adequate test coverage for "corner cases".

Case in point:
./gradlew ':qa:repository-multi-version:v7.10.0#Step3OldClusterTest' --tests "org.opensearch.upgrades.MultiVersionRepositoryAccessIT.testCreateAndRestoreSnapshot" -Dtests.seed=7303D305178F6209 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=hi -Dtests.timezone=Pacific/Apia -Druntime.java=15

I was able to reproduce this test failure on all three branches (including 1.0 release branch). Which means it was introduced sometime back before 1.0 GA. It was only just now caught because of the version bump PR. Random testing should've caught this far earlier than a PR.

[Triage] @nknize do you have an approach of how this testing would be conducted - could this be done in a GHA or does this need support for a new kind of execution / reporting?

I was originally thinking like a temporal trigger (say every 4/6 hours or so; 4 times a day)? Perhaps that's too often? Maybe we run it twice a day? I would say we could be more conservative and set it up as an intake check (e.g., every merge)? But we don't have enough merged contributions for that to give the coverage needed, so maybe a 2/4 times a day random testing is good until we have a lot of merge activity and we switch to an intake check?

Could these tests be run in GHA?  It supports cron syntax as we use it in this repository.

There's just not enough hardware in GHA IMO, and Jenkins can do cron just fine too.

[Triage] Many concerns of this task will be resolved when Jenkins in public https://github.com/opensearch-project/opensearch-ci/issues/3, we will be prioritize that work first afterward this can be reopened if it comes up.


Add message recovery support, version 2

Started from this PR #32, rebased to the current master, made changes according to this PR centrifugal/centrifuge-java#19

@antonselyanin thanks a lot – looks really good 👍

@antonselyanin thanks a lot – looks really good 👍

🚀  SwiftCentrifuge (0.3.0) successfully published

🚀  SwiftCentrifuge (0.3.0) successfully published


Change discv5 crate source

What was wrong?
Discv5 dependency used a patched source pointing to a forked branch of discv5 repo:
discv5 = { git = "https://github.com/ogenev/discv5.git", branch = "public_kbucket_stable" }

How was it fixed?
Our patch was merged upstream a while ago. Switching to the main discv5 branch so we can downstream the latest changes. Bumps also delay_map crate version because of dependency issues.
To-Do

[x] Add entry to the release notes (may forgo for trivial changes)
[x] Clean up commit history

Should we go with latest version as opposed to master? The only concern is upstream breaking changes.

I think the latest version is lagging a bit and I'm not too worried about breaking changes because at this stage we can quickly adjust. I think until we have a stable Trin release, it makes sense to always use the latest dicv5 updates.