How can I execute JfkWebApiSkills project.

I want to add my testing images in Blobstorage and after that I need to create tag and indexes. So How can I execute this project and add my personal images.

Hi sandipArchit,
Are you asking how to use this project on your own data files instead of the sample JFK Files we provided?  If so, then you can redirect the JfkInitializer to point to your own blob storage account by changing the provided keys "JFKFilesBlobStorageAccountConnectionString" and "JFKFilesBlobContainerName" in the App.config file to match your account that contains the files you would like to process.  Alternatively you can change this directly in the code in the "CreateDataSource" method (lines 156-157 of the Program.cs file in the JfkInitializer project).
Do be aware that this particular solution is set up to do things very specific to the JFK Files.  We create some resources that are open to the the internet for access (the website, the additional blob container for the imagestore skill) so if your data needs to remain private and secure you may need to remove these aspects.  The custom skills also might not be useful for your dataset.  I would recommend reading through the Cognitive Search quickstart tutorial to better understand how to implement your own skillset for your dataset, though you can also use the code provided here to help you make the necessary requests to get started.
-Carey MacDonald

Yes its work for me. But I want to execute other project JfkWebApiSkills and to create FacetGraphNodes, ImageStore because in my customized application everything is working but not grapth node functionality is not working.

https://jfk-site-cg6zgjvsynrik.azurewebsites.net/#/search?term=bassole

Hi sandipArchit,
You can change the JfkWebApiSkills project directly and then redeploy the Azure Function yourself with your changes.  Here is a guide on development with Azure Functions if you choose to do this.  Also, here is an example specific to Cognitive Search that shows you what the input and outputs would need to look like if you were to create your own custom skill from scratch, and how to deploy it and integrate it with your skillset.
-Carey MacDonald

Sorry I am bothering you. But How can I create Facet Graph. You can see in above screen. It shows only search string but realated search paramter related to entities. When I checked in Azure function. It runs following querystring https://jfk-function-app-cg6zgjvsynrik.azurewebsites.net/api/facet-graph-nodes?code=CJ8wXXFkw6ZptwBmZipGSt1QZ/4sPbXSayj6IpQrNWCplwvMWgscLg==&q=bassole&f=entities adn result is   {"edges":[],"nodes":[{"name":"bassole"}]}

Hi Carey,
I was wondering if you could explain a little bit more about resetting the frontend/.env file for each change to the App.config.
I am currently trying to change the connected blob storage account to one of my own test accounts and I want to make sure I'm changing everything that needs to be changed. So far, I have changed the blob storage account connection string and the container name in the App.config file on Visual Studio. I have found lines 156 and 157 and changed those as well, I have manually changed the blob storage connection string in the Functions Application Settings in Azure Portal, and now I'm just trying to figure out this last step so it will hopefully work.

Hi @BridgetLan
Observe lines 317-324 in the Program.cs file to see what I mean about needing to reset the .env file.  If you change any of the keys/values listed on those lines to something else after having already run the Initializer once, the .env file will need to be reset as the find and replace functionality there won't work properly the second time.  Assuming you cloned the repo vs. downloading it, you should be able to use git to reset the file and then run the Initializer with those changes.  Also, since you mention only changing the blob storage settings, it likely shouldn't affect the .env file, but you can confirm by looking at the lines I mentioned above.
One thing I do want to mention is that the blob storage connection string in the function application settings is for a different blob then the source data (which is what it sounds like you changed in the App.config).  You may not actually want that setting to be different, though I obviously don't know what your entire scenario is so I can't say for sure.  For clarity, if you change the "JFKFilesBlobStorageAccountConnectionString" and "JFKFilesBlobContainerName" (the source data) then you won't need to change anything in the Azure Function.  If you change the "BlobStorageAccountConnectionString" setting and the "BlobContainerNameForImageStore" constant in the Program.cs file then you will need to change the Azure Function to accommodate.  Hope that helps.
-Carey

Thank you @Careyjmac, I appreciate the clarification. I actually downloaded as a zip file, is there a way to change it from there? Is it possible to just copy a new .env file into the frontend folder? (I know, bad practice but I was hoping it might work). If not, I'll just clone it and start over.
Also, could you tell me what kind of changes could result in the need to reset the .env files? I am looking to change a few other things to match my particular Azure case needs like changing the http trigger to a blob trigger, etc.

@Careyjmac Apologies, I just saw the code answers me so you can definitely ignore that last question.

I keep on getting this error. The log file shows:

@Careyjmac I don't want to overload you, but do you have any idea what might be causing this? This is using everything with the original instructions and variables, not the ones I will change for my own use.

@Careyjmac That is really good to know. I actually cannot see the the files, so I've attached a photo of the Search Service index log file. I'm assuming it has something to do with the permissions.

  I would try to make sure you have the latest from this repository (there were some fixes recently for some issues with Azure Functions) and then either start from step one of the demo or redeloy your Azure Function app via Visual Studio  and then run the Initializer again.  Let me know if that helps.

It worked the second time around, thank you, @Careyjmac!
Do you happen to know if there would be a way to change this code to get the indexer to sort important information into a table on a SQL Database? Besides the standard documentation. There are many avenues that keep leading to roadblocks and I'm not sure what best practice would be.

You would likely need a custom skill in order to output the information elsewhere.  You can find documentation on that here.  Hope it is helpful for your scenario.

Closing as this seems to have been resolved and has not had discussion in several months.

Hi @Careyjmac . We have the same issue as the author, we do not have graph view working.
Can you please describe what specifically needs to be done in order to make graph view working?

If you changed the search service from the initial ARM template deployment, changing it in the App.config alone is not good enough as we put some of the configuration directly into the Azure Function implementation (which is used to create the graph view) as part of the ARM template.  See this comment above in order to change the configuration on the Azure function in order for the graph to work.  If there is some other reason it is not working, it may just be that your source files just aren't super compatible with the method we use for generating the graph, so there's not too much I can do to help.

Yes, I know that there is come config in the code. But when we change it and then redeploy - it has no difference at all

Although if we hardcode it into headers then it picked it up:

Can you clarify that?

Those aren't the configs I was referring to.  The configs I was referring to are for the service name and api key, which are in the settings.json file.  These can be changed on an existing Azure Function without a deployment via the Azure portal, following the instructions here.


Cannot load resources when using account with access to multiple tenants

Does this occur consistently? Yes
Repro steps:

Log on to the main tenant
Select a subscription

I use my azure account to log in to my main tenant. I use the same account to log on to another tenant, where my account is invited as guest account and made member.
When I connect to my main tenant, Logic Apps and Integration Accounts load by selected subscription as expected. But refreshing the Resources group generates this toast:

Version: 0.8.4
OS: win32
OS Release: 10.0.22621
Product: Visual Studio Code
Product Version: 1.87.1
Language: en

Closing as duplicate of #834


[build] Tell Kotlin compiler to generate default methods

I forgot about it, but by default Kotlin does not generate Java-compatible default methods in interfaces, so we want to force this to keep proper interop & API consistency

Weird. So this allows us to use Kotlin interfaces instead of Java interfaces?

Thanks !


[demo] Remove release name prefix from resources

Currently all Kubernetes resources created by the demo will have a name that is prefixed by the demo's release name. This pattern is typical for many, but not all Helm charts.
The demo is a rather large Helm chart in terms of the # of resources that it installs. Currently the demo will create:

1 Statefulset
23 Deployments
24 Pods
25 Services

All of these resources except the OpenSearch stateful set and related ones are installed with the Helm chart release name as a prefix. This causes several challenges when using the demo for its intended purpose, which is to demonstrate OpenTelemetry technologies.
When consumed in observability tooling, the long prefix on names for the pods causes clipping on charts and visuals associated with that resource. Deploying and validating the demo through bastion hosts or limited terminal interfaces can lead to line wrapping and a poor experience for the user. Additional complexity/tech debt is also associated with supporting the release name prefix for all demo components and subchart dependencies required by the demo.
I propose to remove the release name prefix from the names of all Kubernetes resources created by the demo.
To note, the Demo can only be installed a single time per namespace due to the variety of subcharts and RBAC dependencies.

I like the idea and have nothing against it. 🥳

Implementing this as an option seems reasonable.


espanso: init at 0.5.5

Motivation for this change
Add espanso to nixpkgs
Took this archlinux build script as reference: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=espanso

why doCheck = false; ? https://gist.github.com/kimat/c0b253dafef640da9cd5371af031a01d
why openssl ? https://gist.github.com/kimat/79bdd3507a2099c98e01a3714122f5e1

To test this without systemd : run ./result/bin/espanso then type :espanso anywhere and that text replaces itself.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

added a meta longDescription

I can confirm espanso packages : https://hub.espanso.org/ also work and can be installed like usual.

This is working great for me. I've created a module to be able to enable it. Once this is merged I'll open a PR for it.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/177

@kimat can you rebase please to fix the conflict? Also maybe update to v0.6.2.

@bbigras I just rebased, bumped to 0.6.3 and retested the app's behaviour with nixpkgs-review pr 86495 & ./results/espanso/bin/espanso daemon.

Reviewed points

[x] package path fits guidelines
[x] package name fits guidelines
[x] package version fits guidelines
[x] package build on amd64
[x] executables tested on amd64
[x] meta.description is set and fits guidelines
[x] meta.license fits upstream license
[x] meta.platforms is set
[x] meta.maintainers is set
[x] build time only dependencies are declared in nativeBuildInputs
[x] source is fetched using the appropriate function
[x] phases are respected
[x] patches that are remotely available are fetched with fetchpatch

Possible improvements
Comments

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/178

@numkem are you still planning to do that module PR?

@bbigras done!


EDL clip names as reel names

Clip names are often written/read(ex. Lwks, Flowblade, ShotCut and Blender) as reel names in EDL files like this(case sensitive):
001 4388aaBB V C 05:04:46:13 05:04:56:03 01:03:16:04 01:03:25:19
* FROM CLIP NAME: 4388aaBB.dpx

[NB. spacing is truncated in example. File name as Reel name is case sensitive(in Lwks), 8 characters long, no empty space characters in file name(Blender) and with no extention]
Looking through the OTIO EDL code, this practice seems to be ignored i/o in EDL files.
[Btw. really great project. Hopefully, the open source A/V projects will embrace it. If ex. MLT was added as a format then Kdenlive, ShotCut, and Flowblade could all benefit from it. It would also be great to see Blender, Ardour, Audacity, Synfig, FFmpeg, OpenToonz, Storyboarder etc. sharing projects with Lwks, Resolve, Premiere, Final Cut and Avid and vice versa. ]

You're right, the EDL adapter doesn't really pay much attention to the reel, which is an oversight. In the case you mention, it appears to be also covered by FROM CLIP NAME. If the EDL output from the adapter put the truncated/abbreviated clip name into the reel would that be sufficient, or does it need to read the reel name and carry that around too?

"* From clip name: " is an addition to the original CMX 3600 EDL format. Ex. Lwks and Blender can't parse these comment lines, so the only way is to use the reel name to identify the files in those softwares. But I think most pro NLE's will look in the comments for a file-name with extension, after "* From clip name:". I know Resolve will do that if you check the right boxes for the import settings. I don't know which NLEs will use the full path "* From file:", though that would be the logic - I haven't tested those.
If you can provide a short OTIO edl file and a video clip with cuts at the same frames as there are cuts in the video clip I can test it against Resolve, Blender and Lwks. [Like I did for ShotCut: https://forum.shotcut.org/t/mlt2edl-script-error-fix/55/40]

Hi!
I ran into this "limitation" today when going from xml to and edl via otioconvert and thought I'd add my two cents.
Quite a few applications base clip names on the reel name and in those cases AX make them hard to conform and so on.
I prefer when reel names are the same as FROM CLIP NAME, but to be safe it makes sense to keep reel names to 8 characters like the original spec. (optional perhaps?)
What we could do is something similar to what FCP7 did/does, which is to truncate the reel name to spec and add a comment letting people know what they did.
From an FCP7 EDL: FINAL CUT PRO REEL: YL600_COMP_MASTER_V001_TMA REPLACED BY: YL600COM
FCP's comment is a bit strange not prefixing the line with an * so I don't suggest copying this directly. We could however create our own comment which follows the * convention and informs users of the truncation like: * OTIO BASED REEL NAME ON: YL600_COMP_MASTER_V001_TMA and clip.metadata['cmx_3600']['reel'] then yields: YL600COM
The resulting lines in an EDL:
453  YL600COM V     C        14:09:34:07 14:09:40:18 02:14:49:09 02:14:55:20
* FROM CLIP NAME:  YL600_COMP_MASTER_V001_TMA_PRORES
* OTIO BASED REEL NAME ON: YL600_COMP_MASTER_V001_TMA_PRORES

Please let me know if none of this makes any sense and I'll try to express myself clearer.
I can try to implement this (or another) solution if you like.

@apetrynet feel free to change the reel name based on the clip name. The hard coded "AX" was based on our limited EDL experience. There is a style parameter to the adapter which could be used to control the specifics if needed, but I doubt that anyone really needs the AX reel name.

Cool!
Should I add some logic revolving truncating reel name as well? Could be a trunc_reelname=True or strict_reelname=True argument in the write_to_string() for instance?

Hi!
I began working on this and have a first implementation in place, but need to tweak some of the unit tests to support the new reel names.
The tests checking against a string representation of an EDL are OK, but I'm having some difficulties deciding how to approach the test_edl_round_trip_mem2disk2mem which compares JSONs.
They won't be equal anymore since I've added a comment and reel name to the clips metadata.
I can of course get rid of those metadata entries, but find them a bit useful. Has anyone got any thoughts on this?
I haven't created a PR yet, but have a fetaure branch setup.

I would just like to add that I've noticed the Davinci Resolve edl export adds the full file name no matter the number of characters, so if the normal 8 characters limit can't be read succesfully - then maybe the parts before and after the name could be cut because the length of those should be a consisten number of characters.
Ex.
001  WildlifeInAfrica V     C        00:00:00:00 00:00:04:05 00:00:00:01 00:00:04:06

We'd like OTIO to be flexible in reading EDLs that don't 100% match these limitations, since there is quite a lot of variation out there. When writing an EDL out from OTIO, there is an optional "flavor" argument to the EDL adapter to help with differences like this. Ideally the reader won't need an argument, but could discover it from the file itself.

@tin2tin My proposed changes don't touch the reel name when reading an EDL. It adds it to the metadata and goes on with it's business, so the adapter should be able to read an EDL from DaVinci. The truncate option is for writing EDLs.
@jminor Was your last comment aimed at my code or @tin2tin's comment?
I'll submit a  [WIP] PR.
Thanks!

My comment above was mostly related to @tin2tin's comment about Resolve, and to just point out the "flavor" concept. I'll take a look at the PR :)

Hello all, I just pushed up a PR ( #434 ) to that does some work on the cmx_3600 adapter
Although this PR doesn't address much (if any) of the direct concerns mentioned here it does work to maintain the SOURCE_ID (reel) when round tripping an EDL
My thinking is if we are maintaining and storing the reel in the metadata one could easily add a hook to truncate/clean/substitute reel names or clip names.  That is where we are doing our path mapping between Linux and Windows as well

@ssteinbach I guess we can close this with #434 and #392 merged into master?

Yes ! I was hunting around for this issue.  Thanks!


Update defmt, dependencies

I wanted to use the fancy features in Knurling-rs changelog #13 but this wasn't updated to allow that.

Thanks!
bors r+

Thanks!
bors r+


TabBarView swiping interruptions lead to TabBar animation bug

Reproducable with
fluttter_gallery scrollable_tabs.demo.gart
If you swipe through the pages and stop the TabBarView in the middle of the animation, the TabBar animation will finish and not stop like the TabBarView does.
This is especially bad if a TabBar with a custom animation is used.
video
[✓] Flutter (Channel dev, v0.8.2, on Mac OS X 10.13.4 17E202, locale de-DE)
    • Flutter version 0.8.2 at /Users/valauskasmodestas/Desktop/flutter
    • Framework revision 5ab9e70727 (8 days ago), 2018-09-07 12:33:05 -0700
    • Engine revision 58a1894a1c
    • Dart version 2.1.0-dev.3.1.flutter-760a9690c2

[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/valauskasmodestas/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 27.0.3
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk1.8.0_152.jdk/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_152-ea-b05)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.2

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] IntelliJ IDEA Ultimate Edition (version 2018.1.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 24.2.2
    • Dart plugin version 181.4892.1

[✓] Connected devices (1 available)
    • iPhone X • B07F3B8D-2BAD-4E9A-9230-CEA38DED81FA • ios • iOS 11.4 (simulator)

Sounds a bit like #15126 but that was fixed a while ago.
Please add the output of flutter doctor -v.

added

I have the same issues.
[✓] Flutter (Channel dev, v1.2.0, on Mac OS X 10.14.3 18D109, locale zh-Hans-CN)
    • Flutter version 1.2.0 at /Users/xxx/git/dart/flutter
    • Framework revision 06b979c4d5 (2 weeks ago), 2019-01-25 14:27:35 -0500
    • Engine revision 36acd02c94
    • Dart version 2.1.1 (build 2.1.1-dev.3.2 f4afaee422)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/xxx/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 32.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.3.4)
    • IntelliJ at /Users/xxx/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 30.0.2
    • Dart plugin version 182.5124

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

[✓] Connected device (2 available)
    • MI 4LTE • 9121afb0                                 • android-arm • Android 6.0.1 (API 23)

• No issues found!

My solution:
  // Called when the PageView scrolls
  bool _handleScrollNotification(ScrollNotification notification) {
    if (_warpUnderwayCount > 0) return false;

    if (notification.depth != 0) return false;

    _warpUnderwayCount += 1;
    if (notification is ScrollUpdateNotification &&
        !_controller.indexIsChanging) {
      if ((_pageController.page - _controller.index).abs() > 1.0) {
        _controller.index = _pageController.page.floor();
        _currentIndex = _controller.index;
      }
      _controller.offset =
          (_pageController.page - _controller.index).clamp(-1.0, 1.0);
    } else if (notification is ScrollEndNotification) {
      if (_controller.animation.isCompleted) {
        _controller.index = _pageController.page.round();
        _currentIndex = _controller.index;
      } else if (!_controller.indexIsChanging) {
        if ((_pageController.page - _controller.index).abs() > 1.0) {
          _controller.index = _pageController.page.floor();
          _currentIndex = _controller.index;
        }
        _controller.offset =
            (_pageController.page - _controller.index).clamp(-1.0, 1.0);
      }
    }
    _warpUnderwayCount -= 1;

    return false;
  }

Seems to be fixed.
I can't see the behaviour what is shown on the video.
With code from https://flutter.dev/docs/catalog/samples/tabbed-app-bar or in gallery web-site https://flutter.github.io/gallery/#/demo/tabs. Closing
@modulovalue If you disagree please write in the comments and I will reopen it.
Thank you


DOCS-Add transaction signatures documentation

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer


Forget password

Hi, could you also develop forget password section if its possible for you?
Kind regards

Hi @uskudaristanbul,
Sorry, I am going to close this issue... There are a few different ways to handle "Forgot Password" functionality with Supabase (deep linking, OTP) and I believe it is best to allow developers to choose and implement their own solutions.
However, here is the documentation from Supabase's JavaScript API Reference for implementing "Forgot Password" functionality: https://supabase.com/docs/reference/javascript/auth-resetpasswordforemail


updates(company roles): update title. fix scroll issue

Description
change title. fix scroll to the section issue
Why
scroll issue
Issue
NA
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally

Closing this PR. These changes are already included in PR - 72

Closing this PR. These changes are already included in PR - 72


Tests are ignored by eslint

we are only linting index.js and ./lib/* - we should also be linting the tests

linting has been added, as of a few months ago, closing.


importlib.metadata.PackageNotFoundError: No package metadata was found for aws-mfa

Just rubber ducking this here, seems like some Python package updates maybe broke aws-mfa.
importlib.metadata.PackageNotFoundError: No package metadata was found for aws-mfa

I am on Python 3.10.1 but just installed 3.9.9 with pyenv install 3.9.9; pyenv global 3.9.9 and still same error. Maybe it isn't taking right.

Looks like I need to rebuild all my AUR packages per this PSA.
https://www.reddit.com/r/archlinux/comments/rf6c84/psa_python_310_is_in_core_rebuild_your_aur/ (PSA: Python 3.10 is in [core]. Rebuild your AUR packages.)

K, finally figured this out.
I just had to re-run yay python-aws-mfa BUT choose an option to rebuild it. My other issue was that pyenv then installed it to ~/.pyenv/* subfolder and it wasn't in my path. So I had to pyenv global system so it went back to using 3.10.1, BUT I had to clear my ~/.cache/yay/python-aws-mfa as it was build against the Pyenv version etc.
All is well now!


Keep a consistent behavior between file shares and ADLS Gen2 blob containers when using the name 'a/b/c' to create a new folder

Storage Explorer Version: 1.15.0-dev
Build: 20200811.2
Branch: master
Platform/OS: Windows 10/ CentOS 7.6.1810 (Core) / MacOS Catalina
Architecture: ia32/x64
Regression From: Previous release (1.14.2)
Steps to reproduce:

Expand one ADLS Gen2 storage account -> File Shares and Blob Containers.
Create a file share and a blob container.
Click 'New Folder' both in the file share and blob container -> Input 'a/b/c' as the new folder name -> Click 'OK'.
Check the results.

Expect Experience:
Show a consistent behavior.
Actual Experience:
Show inconsistent behaviors.

More Info:
The screenshot on version: 1.14.2.

@craxal Verified this issue on build master/20200816.2. Fixed.
Could you help to take a look a this scenario?
When using name 'a\b\c' to create a new folder.

For ADLS Gen2 blob containers, succeed to create a folder.

For File shares, fail to create a folder.

@v-xianya This looks like a separate issue. Can you open a new bug, try it with forward slashes, try it on multiple platforms, etc.?

@craxal Verified the issue on 3 platform, the behaviors are the same.
The new opened bug: https://github.com/microsoft/AzureStorageExplorer/issues/3415


What is wrong with my model? + summary & solutions to F.A.Q

Hi everyone!
I was already having a discussions about my issues in issue-68, but decided to open a separate ticket anyway for completeness towards other people. As of now. I am clueless what is wrong with my model. My workflow and solved issues are as follows:

Using Unity3D, I created a data-set with around 240 training-images and 60 test-images for a custom model: a cube with 4 different colors (for testing).

I came across multiple issues regarding the following:

I set my camera calibration intrinsic parameters as follows:

    K = np.zeros((3, 3), dtype='float64')
    K[0, 0], K[0, 2] = 320, 320
    K[1, 1], K[1, 2] = 320, 240
    K[2, 2] = 1.

Annotated labels are created automatically in Unity3D of which I expect there to be no camera distortion (for the intrinsic camera calibration). The original author's of the LINEMOD dataset use a Kinect camera that does have such a camera distortion. This internal camera calibration is necessary for among others the PnP-algorithm.

Initially, I had many problems with creating my .PLY files:

Scaling them into meters. I think I managed this my .PLY file here:
Apparently models have to be centered at their volumetric point of mass at the origin (0,0,0). I centered my objects at the bottom-surface, giving them an unwanted offset.

Bounding-box coordinates need to be in a specific order in their respective annotated label file. If this is not the case, the bounding-box that gets generated form the .PLY file and predictions from singleshotpose will be distorted completely. There are multiple issues on Github about this (among which issue-49

Correct order of boundingbox coordinates in label file (green border):

Incorrect order of boundingbox coordinates in label file:

Find here, an example of an image and a label file from my trainings-set.
HOWEVER.
I am still not obtaining correct results and I am unsure about how long to train my models for. In the implementation 700epochs are stated, yet if I train for 4000 epochs, my results are still not good:

How many epochs should a new object be trained for?
NOTE: I am using benchvise/init.weights as my initial weights for my new model of the custom data-set.
This while my loss function goes down properly, but my accuracy measurements stay at 0%:

Could there still be a problem with how I created the annotation files, camera intrinsic parameters or .PLY model. Or could there be another problem that I am not considering?
@btekin Would it be an idea to add a F.A.Q section to the README, using my findings? I think the section about training on a custom data-set could use a lot more elaboration.
Moreover, I am curious as to what people are doing with singleshotpose. Anyone experimenting with some interesting use-cases?
Many thanks for anyone that can help!

I managed to get my "Acc using x vx 3D transformation up to 20 - 30%" using a more appropriate diameter in the .data file. I calculated this diameter using a piece of code that pair-wise compares all vertices and takes the pair with the greatest distance.
Unfortunately I am not able to get any accuracy at the other 2 metrics (Acc using 5px 2D projection and Acc using 5 cm 5 degree metric). It seems that the model is unable to learn from my data? Might the data be too monotonous? If so I would expect to at least see some accuracy metrics go up (because the model would just over-fit).

Too bad that nobody seems to be able (or willing) to help. I have a little update:

Instead of 240 training-images, I now used 4000 training-images.
I also added random-background colors and added more colors to the cube (since I thought that the model was unable to learn about my cube object due to insuccifient features in texture).

My accuracy after 1000 epochs is:

What I find weird is that LINEMOD has only ±200 images per class for training (and ±1000 for testing). Hence, their training-set is very small, yet @btekin was able to reach high accuracy >90%. @btekin, I would love to hear your input about this. Is this due to pre-training? Can you tell us a bit more about how long your pretraining took and what parameters you used in your training/test set?
Sidenote:
I do believe that the trainings-images of LINEMOD are selected in such a way that together they represent a good 3D presentation of an object. In other words, every image in the trainigsset features the object from a unique POV (unique camera angle, position etc). Could this be of influence on why my model cannot seem to learn with only 240 images? On the other hand, I do expect the model to at least learn something with a small datset, when increasing the amount of epochs to train for; the model would overfit but still should learn and return high accuracies on the training-set.
Any thoughts?

Hello, thanks for your interest in our code and sorry for the late reply, I didn't have the time to reply as I had to deal with my other work related projects.
We follow the same training/test splits with earlier work, e.g. that of the BB8 paper by Rad & Lepetit, ICCV'17. Indeed, the training examples are sampled such that they cover a wide variety of viewpoints around the object. Having a more representative training set should, in principle, increase the accuracy on test examples.
Instead of using initialization weights for another object, you could also pretrain the network on the same object by setting the regularization parameter for the confidence loss to zero, as explained in the readme file. See also the discussion in the paper and in #79 why such pretraining could be useful.
About your findings: We already mention what order should be used for the keypoints in this link along with a step-by-step guide and this was also discussed in the duplicate issue #68. Custom datasets have different camera intrinsics matrix and might have different object models/scales. The scale of the object model should certainly be consistent and be set appropriately for a new dataset.

Hello @btekin. Thank you for your answer, yet I would like to ask you to be more specific.
The LINEMOD dataset uses ±200 images for training (while having 1200 images per class: 1000 remain for testing). How can ANY model learn from only 200 images? Aren't NNs like YOLO supposed to have thousands of images per class?

My model cannot learn anything when using 200 images in my custom data-set; loss-functions go down (<0 for x,y loss and confidence) yet accuracy metrics stay at 0.
When I use 5000 images per class, accuracy maxes out at 20% (note that I am using synthetic data)

Sidenote: I am using lower batch-size of 8
You speak about pre-training being necessary. Can you be more elaborate about this? how long did you train for? How many images per class? 200 again? How many epochs.
At this point I am distraught and about to give up...

Same problem here. I raised an issue #85  mentioning that my model also is not learning.
And the thing is that, while validating, I can only use the first trained model which is after epoch 11. Even if my model keeps on training, it never updates the weights means that its never getting any better. It just save the summary in costs.npz file
I used a very good source for generating my synthetic data, I have masked images too that are corresponding to my RGB images, correct intrinsics, precise labelling files, exact diam value and rightly scaled .ply file. I am using 1170 images with 65 different orientation of the object and different background and just using one class (object). I also previously tried the 15% training/85% testing ratio split but also didn't learn.

After comparing my inputs with that of ape.data and finding everything is matching yet not learning for custom data, I am also about to give up...

Hi @MohamadJaber1
Your green ground truth box does seem to be an incorrect bounding-box though. Maybe there still is something wrong with the labeling for your case? If you see my green ground-truth bounding boxes, they are exactly matching the object. It is indeed the case that the code is written in such a way that it will not save weights when there is no increase in accuracy. You can add a line of code to make it save weights after every 10 epochs or so. I tried this as well, but it is not helpful; if the accuracy does not increase, you can save weights all you want, but they are useless.
I have just now tried to train a model on 16.000+ images. I stopped the training at around 310 epochs, because I don't have the time to continue training this model, since I am only using a Nvidia 1080ti. Interesting enough, the loss function is pretty low, yet accuracy does not rise.

At this point I think that in general, 1000 images for training should do the trick. Because I can only use a batch-size of max 8, I think that I have to train for far more epochs than proposed in the code (700 epochs). It would be nice to hear @btekin input 😄

Hi @jgcbrouns, good to hear from you and hope to hear  also from @btekin soon 😄
Thats true, even my ground truth labels aren't matching which is quite strange. I forked @juanmed singleshotpose as he used the same source for generating the data I used and he also created a script called ndds_dataset_creator.py that inputs your 3D bounding box configurations and output a label text file that is compatible with singleshotpose. You can also visualize the points (labels) at each of your images. If you can write the way you created your label files.
For saving the model, I know I can modify the script by saving whatever but as you said it is useless if the model isn't getting any better. But my question was, why from the first place my model isn't getting any better? Why accuracy is 0?
By CUDA out of memory, I meant I reduced the batch size and subdivision to 4 as the yolo-pose-pre.cfg was having a batch size of 32.
I think that the original singleshotpose using LINEMOD dataset was trained over than 700 epochs in way that they kept on updating the initialization weights with the trained model and trained allover again to improve accuracy or changed the 700 epochs to any other value in thousands.

I also think that # epochs was way in the thousands.

"But my question was, why from the first place my model isn't getting any better? Why accuracy is 0?"

For your case it is pretty straightforward I think: fix the bounding box corners (check for ground truth box correctness) and your model will learn at least something (like mine). Your other settings seem to be correct: .ply file, diameter of ply vertices, camera intrinsics. Another tip: check if your .ply file has multiple vertex points. I see that your object is a lego-block. A cube in general can be modeled as a parametric 3d model. The linemod objects all have many vertices and edges in their model. @btekin uses the individual vertices to calculate accuracies against. In my hypothesis, more vertices adds more change for higher accuracy. What you could try is add more vertices in your model via blender:

open 3d .ply model -> select all vertices and edges -> go into edit mode -> press 'w' --> click 'subdivide'

But again, before that, fix the ground truth boundingbox label coordinates :P

@jgcbrouns @MohamadJabar1 Thank you for your kind feedbacks 😄

Hello, a maximum number of 700 epochs were set for training but the model always converged much earlier than that.

The model was trained and validated on the LINEMOD dataset. Depending on your custom data, training might proceed in a different way and might require different number of epochs, learning rates, batch sizes learning schedule etc. I would be interested in looking at your own custom synthetic datasets if you could share it to better understand what problems you are having.

For LINEMOD, we use the standard training/test splits and apply extensive data augmentation by changing the background with segmentation masks, random scaling/translation, etc. Without using segmentation masks the accuracy might not be good enough because of the lack of generalization ability. To increase generalization, you could change the background of your images using segmentation masks or increase the number of your training samples.

Please also check  #84 to see if your problem comes from this. @jgcbrouns it seems that your corner predictions seem accurate, however solvepnp method of the opencv version you are using might be returning inaccurate results. Could you visualize your predictions before pnp and let me know if they are accurate?

@MohamadJaber1 Symmetric objects with uniform texture generally brings additional challenges because of the pose ambiguity. Could you also try with non-symmetric or well textured objects and let me know how it performs.

hi @btekin . Thanks for your respons!
I think that the images that I posted above visualize the individual corners before PnP, straight from the predictions (red) and ground truths (green):
    ax.scatter(corners3D[0], corners3D[1], -corners3D[2], zdir='z', c= 'red')
    ax.scatter(vertices[0], vertices[1], -vertices[2], zdir='z', c= 'red')
where vertices and corners3D is:
vertices      = np.c_[np.array(mesh.vertices), np.ones((len(mesh.vertices), 1))].transpose()
corners3D     = get_3D_corners(vertices)
It would be AWESOME if you could take a look at my dataset!!
I am breaking my head about this every day 😅 🔫
Dataset: Google Drive link ~8MB, 300 images
I will look at the ransac replacement algo for OpenCV right now.

Thank you both for your replies @jgcbrouns @btekin for your reply. I will follow your comments and try them on Tuesday - Wish you both Happy Easter 😄)
@jgcbrouns Reason why I am using NDDS is that I want later to validate my model with an image taken from a robot software environment and NDDS provided me with all the necessities.
@btekin It would be really great if you can take a look at our dataset (I will provide a sample)

@jgcbrouns Thank you for providing your dataset. After inspecting examples from your dataset, I would suggest you to do the two following things and see if they help:

Reduce color data augmentation. This could be useful because your object does not have any texture and the only cue that is useful to predict the pose is color. When you apply color data augmentation (changing hue, saturation values etc.) during training, the model has difficulty in distinguishing between different colors and hence estimating the pose. You can change the values for color data augmentation at the following lines: https://github.com/Microsoft/singleshotpose/blob/master/dataset.py#L67:L69

Randomly change the background. This could be useful as you have a small number of training examples and although you use different backgrounds for different training images, having static backgrounds for the same examples might result in overfitting.

Hope this could help your problem. Please let me know how it goes.
@MohamadJaber1 As @jgcbrouns pointed out, I think you would need to fix the bounding box label coordinates in order to for the model to start learning. If you provide a sample, I could also take a look at your data.

Hi @btekin and @MohamadJaber1
Thank you @btekin for looking at my dataset.
Coincidentally I managed to gain some results this morning by converting my images (which are .png) to .jpg. I suspected that this could be of influence since .png can have transparent pixels in its images. Moreover I extended my tool with a mask over objects. Every object now has a mask as well. Either one of these changes fixed the problem for me.

@jgcbrouns good to hear that, I will also convert the types of my images to jpg later
I tried your dataset and checked the ground truth labels - they were matching fine!

At the cmd window, I am writing
python train.py cfg/cube.data cfg/yolo-pose.cfg backup/benchvise/init.weights
But the problem is that after epoch 11, the model.weights is saved once and never updated. I trained the model for over than 140 epochs but even though never updated.

@btekin Thank you so much for offering this help.

Google driver link 400 RGB images, 400 masked images, 400 labels files, Lego.ply, test.txt, train.txt, training_range.txt (.json files from NDDS). So originally the images are 18 same pose images*65 different orientations = 1170 total images.

Lego.data

yolo-pose.cfg

Please let me know what can you observe and how can I solve it

Hi @MohamadJaber1
0% acuracies after 100 epochs implies that there is something wrong for sure. The model is supposed to converge rather quickly (on average at epoch #30, the model starts showing accuracy increases. Before that, its stays at 0%).
Here is my final dataset including masks and .jpg files (converted from .png):
Google Drive link
You can try training with this dataset. It should work 😄
I'll take a look at your dataset now.

About your quick update:
labels are indeed ordered with a specific coordinate system in mind. Unity (where I create my dataset has a different coordinate system than blender for example). The order is important because in @btekin code, this order gets interpreted and adhered to: link

You can validate if your order is correct:

Open your .PLY model in blender via import.
Your first x,y coordinate in your label file is the center, subsequently it should be the red dot as visible in the picture. This first cornercoordinate has the minimum values for x, y z. In blender you can see that this indeed matches.

Hi @jgcbrouns
Thank you for this clarification, I am so glad to know that your dataset is working with the model. This gives me hope and motivation to dig more and fix mine also :smile:
Yea sure, it would be nice to contact you. Expect a mail soon :sunglasses:
I am currently training on your dataset to see if it will converge with me. Also, I will go few steps backwards to check all my own custom data (mostly the labels). For the zipped version of my data, please find it here
Google drive link for my 1170 images
Thank you very much

Hi @jgcbrouns
Your dataset is learning with the model pretty good :smile:

This was trained for almost 60 epochs and its already showing good results :+1:

@jgcbrouns thank you for all the info you've shared in this thread. I am currently trying to figure out what's wrong with my dataset (or dataset generating tool). I am trying to learn on the dataset you've provided in this thread. After that I would try to generate the same dataset with the tool I am using to see if it's working as expected.
Could you please provide the texture that you've used for your object and also if possible your data generating tool?


XML log append

Hello
Could you tell me how to append in a XML log file?
I would like to add a new element attached to the root node element.
Thanks

If you mean,
mylogfile.txt
<root>
   LOGS WRITTEN HERE
</root>

Then you cant, as we can only append to a file, not insert.
Example that work, see here: https://github.com/NLog/NLog/wiki/XmlLayout

Duplicate of  #3537

Rewrite of Footer-Layout on file-append discussed here: https://github.com/NLog/NLog/issues/1658#issuecomment-604589881


Update with latest base

If you ever hope to merge it in base, you should keep it updated and open a PR in the base repo, don't you think?

This repo was more of a POC to prove out the value in abstracting the browser runner, as described in https://github.com/Polymer/web-component-tester/issues/583.
I am of course, happy to take PRs to keep it updated! Though the actual code lives on the custom-runner branch, I'd need to merge that into this repo's master

Oh man I think this overwrote your changes

Nah, my changes are in a separate branch and you made the PR to master. I'll put my changes onto master as well

@pdesjardins90 you should be able to install https://www.npmjs.com/package/web-component-tester-custom-runner 6.5.0 now that has my CustomRunner along with these updated changes


Remove extra print statement

Issue #, if available:
Description of changes:

Fix print in route53 rule.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

:facepalm:


Feature Request: Streamed code completions in the VS code editor

I realize this is a tall ask, but I looked around and didn't see any other discussions about this feature. I could have missed a previous discussion somewhere, or maybe it's just impossible...
However, I just noticed that an example of streaming completions was recently published by Open AI. Maybe this makes it a possibility, or perhaps not. I but wanted to at least ask so you can give your thoughts on the possibilities.
https://github.com/openai/openai-cookbook/blob/main/examples/How_to_stream_completions.ipynb
Thanks,
Nick

Hi @lotekjunky
sorry for the delayed response. I've changed country recently, and that's taken up most of my time.
It's a good suggestion. I want to switch to streaming, but how vscode interacts with webviews makes the implementation tricky. I'll look into this feature and see if it's feasible.

Hi @lotek4u
Streaming isn't feasible with the way that vscode and webview work (currently). I've added a response indicator to help know that the request is still being processed.


A11y_Azure Data Studio_Nov2023_Database Properties Preview_Files_Keyboard: When navigating to under the Files tab "Database files" grid column headers are not accessible with any keyboard keys.

"Please do not close this bug. This bug should only be closed by Trusted Tester after verification."
Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
GitHub Tags:
#A11yTCS; #A11yMAS; #A11ySev1; #DesktopApp; #WCAG2.1.1; #FTP; #SH_Azure Data Studio_Nov2023_Linux_Dec2023; #SQL Azure Data Studio; #Linux; #Keyboard; #NonBenchmark;
Environment Details:
Application Name: Azure Data Studio
OS Name: Ubuntu 22.04.2 LTS
Azure data studio 1.47.0
Repro Steps:

Install Azure data studio application and open it
Make sure to enable "Preview" feature in settings.
Navigate by using TAB key till "Database" and press enter to expand the tree.
Navigate by using TAB key till any database and pressF10 to open context menu.
Navigate by using TAB key till "Properties Preview" and press enter.
Navigate by using TAB key till "Files" tab and press enter.
Navigate by using TAB key till "Database files" table and try to change column size.
Observe that, unable to change column size with keyboard keys.

Actual results:
When navigating to under the Files tab "Database files" grid column headers are not accessible with any keyboard keys.
Note: Issue seen throughout the application for similar type control.
Expected results:
When navigating to Files "Database files" grid column headers should be accessible with keyboard & also should be able to adjust the column header with keyboard.
User Impact:
Keyboard only users will face difficulties if the focus does not go to column header and cannot resize the column header.
Attachments:
25208.zip

@Raj-ghuge This is a duplicate of another accessibility issue here: https://github.com/microsoft/azuredatastudio/issues/25132

@corivera
https://github.com/microsoft/azuredatastudio/issues/25132 the bug is on Win32 platform and above bug is of Linux.

@Raj-ghuge resizing columns is provided via the keyboard shortcut Shift+Alt+S

@Raj-ghuge Can you verify that the issue is fixed using that shortcut? You can find the latest Ubuntu build of ADS here.

@corivera
The keyboard shortcut Shift+Alt+S is not working for "Database files" grid table not working.

@Raj-ghuge the keyboard shortcut might be slightly different on linux. To find the key mapping

open the command palette (View->Command Palette):

Type "keyboard shortcut" and select "Preferences: Open Keyboard Shortcuts"

Search for "grid.resizeColumn" and see the shortcut (ex: on my mac, it shows shift+option+s):

@Raj-ghuge what's not working here? I saw the size prompt popped up at the top of the ADS window in your video clip. The column size is set there and then after hitting Enter the new size is applied to the column.

Verified the issue in the environment -
Application Name: Azure Data Studio
OS Name: Ubuntu 22.04.2 LTS
Azure data studio 1.49.0
The issue is fixed, hence closing the bug. Please find attachment.
#Closed;
#Regressed:03-14-24
https://github.com/microsoft/azuredatastudio/assets/130441232/05fdfc27-e045-4036-ac75-dc451ac2d5ed


add enable_verification argument

This PR makes the fuzzer run with enable_verification, when it passed as an argument, e.g. from the duckdb-fuzzer-ci.

Thanks!


style:  change unit test scripts for flake8

This PR closes #120 .
All the unit test scripts are now passing the flake8 analysis.
Moreover, the call for ''flake8'' in the CI has been changed.
For each unit test script,  the error  E402 module level import not at top of file appears. Given that the modules are found in the parent directory, to set the path there is required and done after importing sys.  Hence, the addition of --ignore=E402.

By ignoring the error E402, all python scripts failed the static code analysis.
After fixing them, an extra error is add to be ignore, error W504.
This error does not allow to break a line after/before an arithmetic operator enabling the possibility of concatenating strings in different lines (which avoids having >79 char long lines).


TP: Removes the UI screens that allow CRUD of api_capabilities

What does this PR (Pull Request) do?
Removes the ability to create, update or delete an api endpoint from the api_capability table. Doing this depends on endpoints that are to be deprecated.

[x] This PR is related to #4235

Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?
These screens were somewhat hidden in the UI as roles/capabilities has yet to implemented so you have to manually navigate to https://tp.domain.net/#!/endpoints. Ensure that the table is populated and does not include a way to edit a row or create a new row.
Also run the UI tests to ensure that TP builds and any existing functionality was not broken.
This UI functionality was never documented, therefore, there are no docs to change nor were there any UI tests to delete/change.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4976/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4977/

You've got a conflict though

You've got a conflict though

resolved


Resolves #145: add particle offset (gapSize)

Hi, I found a simple but working solution to this problem. Czech it out.
Just add gapSize={} to the Carousel component, type a number in pixels and a simple calculation will take care of the rest.

Hello @vadimkorr, I would greatly appreciate it if you could consider releasing this feature. Please

Hopefully someday.


Incorrect Output in Multi-Tile Processing When Using Mixed uint8 and bfloat16 Formats in Compute Kernel

Describe the bug
We are experiencing issues with using different types of CBs (bfloat16 and uint8) in the compute kernel.
When processing a single tile in a loop, everything works as expected, but when iterating and handling two or more tiles, specific faces in the resulting tiles are incorrectly stored as 0.
The strange part is that the issue doesn’t occur when I add a dprint in the reader.
Because of this, I suspect something is missing in the kernel or host code that I wrote.
Could you please help check for any potential issues?
To Reproduce
Steps to reproduce the behavior:
This was detected in this bug https://github.com/tenstorrent/tt-metal/issues/11000 and this example can be used.
Expected behavior
The expected output should be produced.

Note:
Currently the highest priority issue among LLK related issues.

@dongjin-na I managed to solve the issue locally by adding a stall on the unpack_reconfig call. Can you please test it out on your branch to confirm the fix works. The fix is found on the rd/stall_unpack_reconfig branch for the wh-b0 submodule, so you can just check that commit out.

Dear @rdjogoTT,
I tested the script and other related implementations using the patch you provided.

The test script I shared passed successfully.
All other implementations, except one, also passed.
The kernel of the failed implementation is similar to the test one. It originally used one SFPU and two FPU compute APIs. After I modified it to use one SFPU and one FPU compute API, like the test kernel, it passed.

Based on these results, I have the following requests and questions:

Could you please proceed with merging this patch into the main branch?
Is this patch considered a complete solution? As mentioned earlier, it seems that when a kernel uses multiple SFPU and FPU compute APIs, the issue can still occur even after applying the patch.

Dear @rdjogoTT,
Additionally, I'd like to inform you that I’ve updated the test kernel, which still fails even after applying the patch you provided.
Could you please check if there might be an issue with the kernel implementation or if the patch might need further improvement?
Thank you in advance for your insights, and I look forward to your response.
test kernel

I’ve force pushed the test kernel to the uint8_and_bfloat16_ver2 branch.
 I am investigating deeper as to why, as this is a more extreme measure than should be necessary.

Looks like we've found the problem, working on a the PR now. It's a bit of a bigger one

@dongjin-na The fix is on the updated rd/stall_unpack_reconfig branch. Please try to run the test again to see if it works for you, I was able to get all tests to pass locally.

@rdjogoTT Hello, I have confirmed that the previously failing tests now pass with the provided patch.
However, I found a new issue related to this in another use case. Since this issue became too lengthy, I have separated it into a new issue. Please refer to https://github.com/tenstorrent/tt-metal/issues/12963. Thank you

@rdjogoTT
Please close this issue as completed once the changes in rd/stall_unpack_reconfig branch are pushed to the main branch.
Thanks!

@rdjogoTT has the fix been merged into main?

@rdjogoTT has the fix been merged into main?

Not yet, investigating the new issue #12963 to make sure it is not related in some way.

Dear @rdjogoTT ,
What I remember from yesterday's meeting with Milos is that this fix (= work for uint8) will be added soon and the issue mentioned in #12963 will need more time.
I agree that using uint8 rather than uint16 can be work-around but it forces us to change all our compiler logics and corresponding op implementations (more memory consumption and performance degradation is another problem). Since it is the only issue that is labeled p0 currently, I respectfully request that you reconsider your choice above.

@razorback3,
You are correct, but the problem we faced while to merge the fix in main is very persistent and we would need more time to find out what is the problem. @rdjogoTT tried the same workaround applied to other problem and it works consistently, which was only to enable you to make forward progress reliably. Uint16 vs uint8 perf delta should be negligible, but if your infrastructure needs lots of changes we will continue to work on this with P0.
Do you think you could use int8 instead of uint8? We’re handling int8 in a different way and it may be easier for us to enable. I am not sure yet though, just asking if it’s worth investigating on our end.
Milos

@razorback3
We have thought of a new approach for how to tackle this problem, and are working on testing it out now. The plan is that I will create a new reconfig API specific to cases where floating point an integer datatypes are mixed within a compute kernel (like this one). The API will call reconfig functions in the math and unpack thread to eliminate the race issue we were seeing, while maintaining compatibility for current use cases of dataformat reconfig. The only limitation is that Dest will need to have fp32_dest_acc_en==true, and the number of usable tiles will therefore be reduced to 4 at once.
This solution would also deal with the issue raised in https://github.com/tenstorrent/tt-metal/issues/12963, because Dest will always be in fp32_mode so the failing case won't be encountered, so we won't need to use the UInt16 workaround proposed there.

@razorback3
We have thought of a new approach for how to tackle this problem, and are working on testing it out now. The plan is that I will create a new reconfig API specific to cases where floating point an integer datatypes are mixed within a compute kernel (like this one). The API will call reconfig functions in the math and unpack thread to eliminate the race issue we were seeing, while maintaining compatibility for current use cases of dataformat reconfig. The only limitation is that Dest will need to have fp32_dest_acc_en==true, and the number of usable tiles will therefore be reduced to 4 at once.
This solution would also deal with the issue raised in #12963, because Dest will always be in fp32_mode so the failing case won't be encountered, so we won't need to use the UInt16 workaround proposed there.

That would be great :)

https://github.com/tenstorrent/tt-metal/pull/12108 finally looks ready for review. The new API is in tt_metal/include/compute_kernel_api/reconfig_data_format.h. If reconfig to/from UInt8 is needed, the API has a template param to_from_int8 to enable this which is false by default.

@razorback3 @dongjin-na can one of you approve? Moreh files were touched https://github.com/tenstorrent/tt-metal/pull/12108

@razorback3 @dongjin-na can one of you approve? Moreh files were touched #12108

Dongjin is on it. Please give us a few days.

Dear @rdjogoTT,
Review of #12108
Regarding the changes you touched to Moreh code, everything looks good on my end.
Test Results
All tests have passed successfully. However, I need to confirm if the to_from_int8 template parameter was used correctly.

When settingto_from_int8 parameter to true only for the UInt8 CB, the tests fail with incorrect results.
Tests only pass if I either set to_from_int8 to true for all reconfig function calls or ensure it is set to true for subsequent reconfig function calls as follows:

        // ......
        tile_regs_acquire();
        reconfig_data_format_srca(cb_input1); // bfloat16 CB
        copy_tile_to_dst_init_short(cb_input1);
        copy_tile(cb_input1, 0, input1_dst_reg);

        reconfig_data_format_srca<to_from_int8=true>(cb_cond); // UInt8 CB
        copy_tile_to_dst_init_short(cb_cond);
        copy_tile(cb_cond, 0, cond_dst_reg);

        // Subsequent reconfig function
        reconfig_data_format_srca<to_from_int8=true>(cb_input2); // bfloat16 CB, to_from_int8 must be true
        copy_tile_to_dst_init_short(cb_input2);
        copy_tile(cb_input2, 0, input2_dst_reg);
        // ......

I would appreciate your confirmation on whether this is the intended behavior.
Other
You can run the test code on the [moreh/stall_unpack_reconfig_test](moreh/stall_unpack_reconfig_test
metal/tree/moreh/stall_unpack_reconfig_test) branch.
 pytest tests/tt_eager/python_api_testing/unit_testing/misc/test_moreh_test2.py

@dongjin-na My apologies, I should have explained better. The to_from_int8 must be set for all reconfigs that change the hw TO or FROM working with the uint8/int8 dataformat. So if you go fp16b -> uint8 it needs to be set, and if you go uint8 -> fp16b it also needs to be set. If you go fp16b -> bfp8b or something like that, it does not need to be set.
So it looks like it is indeed working as intended. So, you only need to set it for subsequent calls as you have shown in your code above.

https://github.com/tenstorrent/tt-metal/pull/12108 is merged, can we close this now?


⚠ Bofrost.at has degraded performance

In 293e224, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 938 ms

Resolved: Bofrost.at performance has improved in 29210b9 after 20 minutes.


Add VideoFrame

Signed-off-by: junjie.jiang<EMAIL_ADDRESS>

Codecov Report

Merging #1272 (b336a1e) into main (a54463a) will increase coverage by 0.33%.
The diff coverage is 75.75%.

:exclamation: Current head b336a1e differs from pull request most recent head 9ee0b23. Consider uploading reports for the commit 9ee0b23 to get more accurate results

@@            Coverage Diff             @@
##             main    #1272      +/-   ##
==========================================
+ Coverage   69.55%   69.89%   +0.33%
==========================================
  Files         262      266       +4
  Lines       12962    13245     +283
  Branches     2138     2184      +46
==========================================
+ Hits         9016     9257     +241
- Misses       3379     3406      +27
- Partials      567      582      +15

Impacted Files
Coverage Δ

towhee/types/video_frame.py
75.00% <75.00%> (ø)

towhee/types/__init__.py
100.00% <100.00%> (ø)

towhee/functional/entity.py
92.85% <0.00%> (ø)

towhee/functional/mixins/column.py
86.95% <0.00%> (ø)

towhee/models/mdmmt/mmt.py
87.12% <0.00%> (ø)

towhee/functional/entity_view.py
72.72% <0.00%> (ø)

towhee/functional/data_collection.py
84.83% <0.00%> (+0.05%)
:arrow_up:

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 a54463a...9ee0b23. Read the comment docs.

/lgtm
/approve


[RFC] Add ability to explicitly disable loaders in object construction

Problem:
We're running into an issue (related to #751) where we're unable to disable an external loader in a multi-project environment. One project uses vault, while the other doesn't, and loading vault in that second project causes config conflicts.
Describe the solution you'd like
What I'd like to be able to do is to explicitly disable one or more loaders at the class instance level. For example:
settings = Dynaconf(..., disable_loaders=["dynaconf.loaders.vault_loader"])

Describe alternatives you've considered
Unfortunately, there don't seem to be many good alternatives. We've used this workaround for a while, but now are seeing issues. This may be due to the attribute access settings._loaders triggering the loading itself.
settings._loaders = [loader for loader in settings._loaders if "vault" not in loader]

An example of this failure can be seen in the checks here:
https://github.com/SatelliteQE/broker/actions/runs/5489518057/jobs/10003774330?pr=216
I have an idea of how this could be minimally implemented but haven't dug into all areas where this new attribute may need to be changed.

Will be part of #1150 for dynaconf 4.0.0


[docs][java-client] Add sections for interceptor usage

Motivation
The Pulsar documentation is missing information and usage examples about ProducerInterceptor and ConsumerInterceptor.
Modifications
Added sections for both interceptor types to the java-client documentation.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[x] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

I just updated the PR accordingly. The feature matrix will be updated as soon as my access request is accepted.
Please find the screenshots of the changes below:


Handle errors in pin-bundle-images

Even if there are errors in pin-bundle-image a broken index image get build and pushed to registry. Handle such errors to avoid push for broken index image.
Resolves: OSPRH-5354

/test openstack-operator-build-deploy-kuttl


Rust Tools Missing

VSCode Version: 1.8.1
OS Version: OS X El Capitan 10.11.5

Steps to Reproduce:

Rust tool is missing, (Racer not found)x2
All the tools are already installed
cargo install racer && cargo install rustfmt && cargo install rustsym [uacfw@uacfw ~/Desktop/Rust/projects/guessing_game]$ cargo install racer && cargo install rustfmt && cargo install rustsym  Updating registry https://github.com/rust-lang/crates.io-index`
error: binary racer already exists in destination as part of racer v2.0.2
Add --force to overwrite
[uacfw@uacfw ~/Desktop/Rust/projects/guessing_game]$`

@htory327 I don't see anything related to VS Code, can you help understand what's your expected behavior of Code?

So just install them with --force, e.g. cargo install racer --force. It's none of VSC's business...
And you have to ensure the PATH environment variable is correctly set. Test it with type racer. I once used OS X for a while and the env vars of BASH was correctly read by VSC, however, if you've changed your (login) shell, make sure you don't mess it up.

@Frederick888 thanks for your help!


Fabricator reloading functionality is missing

When developing using a tool like spork, it's very messy to reload the full rails stack to update the fabricator's definition, can you please add some kind of reloading mechanism?

It is supported, it just wasn't in the documentation. I added it to the bottom of this page:
http://fabricationgem.org/#!defining-fabricators
The short story is that you can call Fabrication.clear_definitions anytime to reset it back to its original state.
Thanks for pointing that out!

Thank you for the update.


Fix "Cannot find module" e2e test failure

A Node.js test in the Core tools e2e integration build is currently failing with an error like this:

Cannot find module '/home/vsts/work/1/s/test/end-to-end/testFunctionApp/QueueTriggerAndOutput/index.js'

The bug only happens for the webpacked version of the worker and it came about because the webpack config hard-coded a file name and I moved some code out of that file in https://github.com/Azure/azure-functions-nodejs-worker/pull/576. The code in question does a 'require' on the user's scriptFilePath which is a dynamic value that webpack can't/shouldn't handle. I improved the webpack config so that we can use a simple comment in the code to tell webpack to ignore that 'require'. This should prevent similar bugs from happening in the future regardless of where we put this code.

It looks like some E2E tests are still failing?

The existing errors are "Condition not reached within timeout." and timeout errors are usually transient


Where is the archive generator?

Could someone point me to the code that is responsible for generating https://leanprover-community.github.io/archive/ ?

https://github.com/robertylewis/zulip-archive, but it’s a lot less useful now that Zulip offers read access without creating an account.


he argument type 'Obx' can't be assigned to the parameter type 'int'.

use something like this ,
RxInt number = 12.obs ;,
and you can't assign obx to an integer , Obx is a widget

use .value
final number = 12.obs;

int secondNumber == number.value \\ true


Find a home for Green

@JohanObrink feels like it has found a home in Digital Services CX right? Close this one?

@vsjolander Right you are 😸 Green has a home and a dad (in @vsjolander). And now it seems the family is growing.


My world map, run energy, spec attack, and insta prayer icon has disappeared

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'Runelite'
Click on ' Any Shattered Relics World '
Scroll down to 'look at my world map'
See error

Expected behavior
These Icons to appear
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.8.10
Launcher version: 1.6.0

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

Turn on the "Show data orbs" option found under the "All Settings" option ingame.


fix: use abort() before request sent cause an exception when using useRequest (#314)

Fixing type annotations and adding an optional chaining (?.) operator. (close #314)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


etcd needs to be secured

Today, anyone can access etcd and access/modify the accounts storage information.

won't fix (will be obsoleted soon)


Why do you handle the dataset in this way？

https://github.com/casper-hansen/AutoAWQ/blob/79547665bdb27768a9b392ef375776b020acbf0c/awq/utils/calib_data.py#L59
why do you concatenate all samples and split according to max sequence length rather than without concatenate?
I think it is very weak in logic,

if there is no correlation between two paragraphs, will the understanding ability of the quantified model deteriorate? for example, will there be a situation where it doesn't end in the correct position？
after concatenate and split, only the first sample with right begin token, other sample with wrong begin token, will this have an impact on the activation value?

Please re-examine whether concatenate and split dataset is reasonable.
Or is there a better way to process the dataset?

quantisation calibrations are generally and ideally performed on corpora datasets; examples c4, wikitext, pileval, etc. In which case,
[1] concatenate and split by max seq. length is essentially just making sure all samples are of a common sequence length.
[2] since its a text corpus, there is no intermediary special token (eol, etc.)
If in case you wish to calibrate with an instruction dataset; you would rather want to pass in the tokenized examples from your dataset directly to the quantizer with each sample being one complete instruction.
something like

# tokenizer := some tokenizer
apply_func = lambda x: tokenizer(x)

tokenized_data = [apply_func(sample) for sample in dataset["message"])]

awq_model.quantize(
    ..., # other arguments
    data_samples=tokenized_data,
)

or refer to this notebook (for a real use-case) that i used sometime back to quantise on instruction datasets;


Make gist parameter optional in Owl_zoo_cmd.load_file

@jzstark Can we make the gist parameter optional Owl_zoo_cmd.load_file
? If not passed in, the load the file from current gist, the reason is that if a gist script want to load something from its own gist folder, it is difficult to what will be the gist id since it is not allocated yet.

@jzstark thanks for fixing this 😄


Remove origin_site_name conversion from SE to PNN

As double-checked with Yuyi, origin_site_name reports the PNN instead of SE name for quite a while now. Thus I'm making these changes now before we forget it again.
Needs to be tested in VM and agent.
To be merged after #7457

+3/-81 Awesome.

I'm getting good at these deletions LoL


docs: Get caller IP

Description
How can I get the caller's remote IP from Request ?

Hi @rsc1975 👋
Thanks for opening an issue!
This is a duplicate of #203 and should be addressed shortly 👍


Jest: TypeError: React.Component is not a constructor

I'm getting an error while running Jest with Cleave:
TypeError: React.Component is not a constructor`

      1 | import styled, { css } from 'styled-components'
    > 2 | import Cleave from 'cleave.js/react'
        | ^
      3 | import { isEmpty } from 'utils/array'
      4 | import { InputProps, ContainerProps, MessageProps } from './types'
      5 |

A different method I tried is to uninstall @types/cleave.js and create my own module like so:
declare module 'cleave.js/react' {
  const Cleave: any
  export default Cleave
}

However this did not work as well. I couldn't find a solution anywhere either so any help on this is greatly appreciated!

Same here.


Legend on KEGG pathway map is wrong

The legend shows that the olive drab color is listed as "Not annotated" but it should say that only some of the genomes have this EC number.  Language could be "Present in some, but not all genomes"

Also, when you are looking across groups of genomes, none are selected.  This legend should read as follows
White box: Not annotated
Green box: Annotated
Olive box: Present in some genomes
Blue box: Selected genes (or something like that....)

I forgot one:
Red box: Selected from EC table

Would you check whether this is correct now?
https://www.patricbrc.org/portal/portal/patric/CompPathwayMap?cType=taxon&cId=234&dm=&feature_id=&ec_number=&map=00010&algorithm=PATRIC&pk=

Looks good except for the blue box.  What if we said something like..."From selected feature group".  Or "From selected gene group"
Does anybody else have any ideas?
----- Original Message -----
From: "Harry Yoo"<EMAIL_ADDRESS>To: "PATRIC3"<EMAIL_ADDRESS>Cc: "Rebecca Wattam"<EMAIL_ADDRESS>Sent: Thursday, August 27, 2015 5:30:19 PM
Subject: Re: [patric3_website] Legend on KEGG pathway map is wrong (#420)
Would you check whether this is correct now?
https://www.patricbrc.org/portal/portal/patric/CompPathwayMap?cType=taxon&cId=234&dm=&feature_id=&ec_number=&map=00010&algorithm=PATRIC&pk=

Reply to this email directly or view it on GitHub:
https://github.com/PATRIC3/patric3_website/issues/420#issuecomment-135560424

I like "Selected" or "Selected Genes" better, because genes can be selected from multiple places in multiple ways, in addition to gene group/feature group.
-Maulik

So what are we going with?  I don't like Selected.  Selected Genes would be better.

I had the same thought at first. But, then, given that we already have have “Annotated” / “Not annotated” and
“Annotated in some genomes” as legend, without qualifying it with genes or EC numbers, I thought
“Selected” would fit the pattern, no?
…I am fine with either though. Your call.

On Oct 30, 2015, at 12:31 PM, ARWattam<EMAIL_ADDRESS>wrote:
So what are we going with? I don't like Selected. Selected Genes would be better.
—
Reply to this email directly or view it on GitHub https://github.com/PATRIC3/patric3_website/issues/420#issuecomment-152595029.


Custom article permalinks

I hope this is the right place to post this question.
I assign a category for each post I create. It would be nice if I could display the category in the url so it would look something like:
http://examplewebsite.com/java-programming/my-post-title
Is it possible to change this?

Should be possible using https://github.com/jnordberg/wintersmith/wiki/Page-Plugin#filename-templating


Check if colorCache dictionary < 2048

I have a problem with animations which affects color changing. It leads to massive RCTConvert calls and app crashes at the line RCTConvert.m:600.
This is a super simple hack to help the app stays out of crashes.

Seems like we'd want a different cache policy than taking just the first 2048 values too.

I agree. This is OH_I_NEED_TO_SEND_APP_TO_APP_STORE_RIGHT_NOT level.


Add disable ssl verification to client instantiation

Initial work.
Still need to figure:

a way to pass disable_ssl_verification to aopen in __batch_send method of _BatchBase class.
Write some tests or at least check if the httpx client was initiated with the correct verify parameter

The idea is to print this message on exception with CERTIFICATE_VERIFY_FAILED
        We have identified a SSL CERTIFICATE_VERIFY_FAILED error.

        This error could be due to one of several reasons:
        - Weaviate client is under a corporate network that terminates ssl and issues it's own certificates.
        - You have a self signed certificate

        Weaviate python client uses certifi, and because of that, it will not be able to trust
        Potential fixes:
        - disable ssl verification by setting using `disable_ssl_verification=True` in client initialization
            - note that Weaviate will trust any certificate
        - Replace certifi cacert with the same cacert that is issued by your corporate network.
            - for example: cat MyCompanyRootCA.pem >> $(python -m certifi)

To test this, you can rename the cacert.pem inside certifi package, like so:
import certifi
right_path = certifi.where()
wrong_path = right_path + "renamed"

import os
os.rename(right_path, wrong_path)

Closing in favour of https://github.com/weaviate/weaviate-python-client/pull/1262


Typo in error message: "occurres"

The current stable version of now cli has an error message when you're messing with URL routes that says:

The segment "[something]" occurres more than one time in your path

"occurs" is spelled wrong in detect-routes.ts (and also in a comment elsewhere in that file)
This is kinda related to #2761 :)

Thanks for creating this issue! Would you like to submit a PR to fix it?


Implementing hpx::functional::tag_invoke

This PR introduces hpx::functional::tag_invoke which is an implementation
of https://wg21.link/p1895. This is needed to implement https://wg21.link/p0443.

@sithhell what's the status of this?

@sithhell could you rebase this branch onto master, please?

Rebased it onto master and should be ready to go.

The review comments have been addressed. The failing tests should be unrelated to this PR.

@sithhell this needs clang-formatting and some of the pycicle builders are not happy (mostly the gcc builds, but also the cray clang build). Let me know if you need help reproducing those, it's not immediately clear to me why they're failing.

@sithhell I'll go ahead with this now and hopefully things will be okay with most other compilers. There is still no ETA for when we will get Piz Daint back for testing so let's not block things from going in because of that.


Copy with Headers

Issue Type: Bug
When you select copy with headers it puts the headers before each row as opposed to just having the headers once.  So if I have 58 results it will provide the headers 58 times.
SQL Operations Studio version: sqlops 0.32.9 (8004776eff330e8980b473b6a2f6faf607164005, 2018-09-19T05:11:56.160Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz (4 x 2394)

GPU Status
2d_canvas: unavailable_softwarechecker_imaging: disabled_offflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabled_readbackwebgl2: unavailable_off

Memory (System)
16.00GB (8.60GB free)

Process Argv
C:\Program Files\SQL Operations Studio\sqlops.exe

Screen Reader
no

VM
100%

Extensions (3)

Extension
Author (truncated)
Version

ssmskeymap
kev
0.2.0

whoisactive
Mic
0.1.1

sql-search
Red
0.2.1

@JCarden09 this should be fixed in the latest insiders build available at https://github.com/Microsoft/azuredatastudio/releases and will be fixed in the next stable build later this week.  You can work-around the issue by "Select All" either through the context menu or Ctrl-A (instead of clicking on top-left grid cell).


CTE queries binding wrong values

Environment
Knex version: 0.15.2
Database + version: PostgreSQL 10.4
OS: macOS 10.13.6
Bug
When using the with method to generate data-modifying CTEs, there seems to be a problem with the value binding; the values passed into the query builders are being bound to the wrong placeholders.
I've got a minimal example:
const knex = require('knex')({
  client: 'pg',
  connection: 'psql://postgres@localhost:5432/<DB_NAME>'
});

const q1 = knex
  .table('user')
  .update({ name: 'foo' })
  .where({ group_id: 1 })

const q2 = knex
  .with('updated_group', (qb) =>
    qb
      .table('group')
      .update({ group_name: 'bar' })
      .where({ group_id: 1 })
      .returning('group_id')
  )
  .table('user')
  .update({ name: 'foo' })
  .where({ group_id: 1 })

console.log('WITHOUT CTE')
console.log(q1.toString())

console.log('WITH CTE')
console.log(q2.toString())

This gives me the output:
WITHOUT CTE
update "user" set "name" = 'foo' where "group_id" = 1

WITH CTE
with "updated_group" as (update "group" set "group_name" = 'foo' where "group_id" = 1 returning "group_id") update "user" set "name" = 'bar' where "group_id" = 1

You can see that in the second example (WITH CTE), the bindings of foo and bar are the wrong way round.
If I use toSQL to inspect these queries, the following is the bindings array for the second query:
bindings: [ 'foo', 1, 'bar', 1 ],

So seems like these are being substituted into the query sequentially, and since the WITH clause will always come first, the bindings are in the wrong order (should be [ 'bar', 1, 'foo', 1 ]). Obviously changing the order of the method chain doesn't affect the final query.
Am I just misunderstanding the with API, or is this actually an issue? Thanks.

Seems to be an issue where knex prepares bindings for .update prior to compiling the .with statement which results in reverse order of bindings.
Compiling the .with call first seems to fix the issue. Wonder if its the same for select and insert..
Marking as bug.


Add a ViewModel to Dessert Clicker: Android Basics in Compose

In 3. Set up dependencies, the dependency script is build.gradle and therefore in Groovy rather than build.gradle.kts in the Kotlin DSL. The project code uses build.gradle.kts so this part of the content should be updated to reflect that.

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


fix(BigQuery.Storage):Removed ToProto Extension method.

Errors are unrealted and tracked in #1690. Force merging...


With optional it does not work with null

Using null is a TypeScript nightly only feature and breaks everyone using a stable release.

Yeah, any alternative solution?

Not that I know of, I've worked around it in other places by using void, but that doesn't allow null - just undefined.

AFAIK null = any > void. So it should work.

@louy Sorry, I didn't understand that notation?

Sorry. I meant any contains void. Couldn't find the symbol ⊃ on my keyboard.
In TS 1.8 null has type any, meaning that if you pass null as void it should work fine.

I've just realised that the problem @unional is having is with TS nightly as null should work fine with the current typing (there wasn't much context for this PR).
I don't think we can do anything about this for now to make it work with both versions. Anyway non-nullable types are a version 2.0 feature so I think we don't have to worry about them for now.

@louy yeah, notice that it is working fine when I actually open this project and run the tests.
Wouldn't it be a backward compatibility issue with nullable type? Seems like this will break a lot of typings when TS 2.0 is released.

Absolutely, it kind of does - but that's what a 2.0 is for, I suppose. I've been having a pretty good run using strictNullChecks and the nightlies for around 3 months on the current project. The only thing I've changed is typings where I used void (which is really the only backward-compatible way to partially fix this).


Drop ros-operator-install and rename populate-node-labels script

Fixes: https://github.com/rancher/elemental/issues/166
Signed-off-by: Ettore Di Giacinto<EMAIL_ADDRESS>

mmh. ros-operator-install seems to be used here:
https://github.com/rancher/elemental/blob/c3e62ce672b1f3cf3babd41df9cac9426df84326/framework/files/usr/share/rancher/rancherd/config.yaml#L2
but as rancherd is no more.. I guess we shouldn't care?

Right.
That's probably worth an issue in rancher/elemental.

Ros-operator-install was dropped as part of #178

I think this is still being called somewhere in the systemd services and the systemd services are also names wrongly 🙂


How can I use hosts file to my router?

Currently, I am using the hosts file on my Windows PC.
But I would like to expand that to all the devices connected to the router.
How can I do that?

This is the first result when I Google hosts file on router.  This is your answer.
https://security.stackexchange.com/questions/133043/modify-the-hosts-file-of-the-router-and-redirect-trafic
Closing.


Unload karabiner_grabber when Quitting Karabiner

Hi, when we quit Karabiner, karabiner_grabber remains loaded. This breaks the use of keyboard logging in MATLAB, so I have to both quit Karabiner, then sudo launchctl unload the karabiner_grabber.plist.
Would it be possible if Karabiner could load/unload karabiner_grabber when starting and stopping?
Thanks for such an amazing utility.

It seems there is a new daemon karabiner_observer and this also interferes with MATLAB lowlevel keyboard access too, so I need to unload both manually, here are aliased commands to start and stop karabiner for the moment:
alias karabinerLoad='launchctl load /Library/LaunchAgents/org.pqrs.karabiner.karabiner_console_user_server.plist; \
sudo launchctl load /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_grabber.plist; \
sudo launchctl load /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_observer.plist;'
alias karabinerUnload='launchctl unload /Library/LaunchAgents/org.pqrs.karabiner.karabiner_console_user_server.plist; \
sudo launchctl unload /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_grabber.plist; \
sudo launchctl unload /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_observer.plist; \
sudo killall karabiner_grabber; \
sudo killall karabiner_console_user_server; \
sudo killall Karabiner-Menu; \
sudo killall Karabiner-Elements'

There is still a significant incompatibility between Karabiner and MATLAB toolboxes used for neuroscience and psychology research...

This is getting worse for me, now I need to also unload the kext — but often the kext fails to unload even if karabiner and all its daemons and services are fully quit...
alias karabinerUnload='launchctl unload -F /Library/LaunchAgents/org.pqrs.karabiner.karabiner_console_user_server.plist; \
	sudo launchctl unload -F /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_grabber.plist; \
	sudo launchctl unload -F /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_observer.plist; \
	sudo killall karabiner_observer; \
	sudo killall karabiner_grabber; \
	sudo killall karabiner_console_user_server; \
	sudo killall Karabiner-Menu; \
	sudo killall Karabiner-Elements; \
	sudo kextunload /Library/Application\ Support/org.pqrs/Karabiner-VirtualHIDDevice/Extensions/org.pqrs.driver.Karabiner.VirtualHIDDevice.v060800.kext;'

Well.........same issue here.

Hi, in recent karabiner-elements, deselecting the keyboard in the preferences:
https://pqrs.org/osx/karabiner/document.html#configuration-devices
...allows the HID keyboard driver to work without issue (MATLAB 2018b), so I think this is the best solution as we just create a profile with keyboard disabled and toggle profile as needed (no need to load/unload kexts manually).

Hi, in recent karabiner-elements, deselecting the keyboard in the preferences:
https://pqrs.org/osx/karabiner/document.html#configuration-devices
...allows the HID keyboard driver to work without issue (MATLAB 2018b), so I think this is the best solution as we just create a profile with keyboard disabled and toggle profile as needed (no need to load/unload kexts manually).

Thank you for your work. I also encountered a similar issue, as setlds does not work while Karabiners driver is active. Using your proposed profile-based approach (switching to an empty profile via karabiner_cli), everything works flawlessly.
Cheers!


Cypress waiting functions revision for basic tests

Investigate, and check if cy.wait() functions across our tests can be safely converted to timeouts. Check whether people use this, what is the outcome, etc..
Useful link: https://filiphric.com/8-common-mistakes-in-cypress-and-how-to-avoid-them (First point)

https://github.com/cortezaproject/corteza-e2e-cypress/pull/65


feat: login update with wallet-connect v2

https://github.com/valory-xyz/governatooorr-frontend/assets/22061815/a077eb32-e0ba-4736-b875-e7dd839d3459

Code looks fine but can we match the colours to the primary colour of the app?


lol fixed playerhostiles

/facepalm
Pull request summary
Description:
Fixes what appears to be a mistype of isPlayer vs isHuman in the prototypes for creeps / room tracking.
Fixed:
Defenses / safe mode wasn't operating correctly due to not seeing hostile player entities.

None

Testing checklist:

[x ] Changes are backward-compatible OR version migration code is included
[x ] Codebase compiles with current tsconfig configuration
[x] Tested changes on {choose PUBLIC/PRIVATE} server OR changes are trivial (e.g. typos)

Hi, I've noticed you've made a couple of pull requests recently; thanks for the interest in contributing. The next two of weeks are pretty busy for me so I don't have a lot of time to work on Screeps, but I'll try to review and merge all your pull requests as soon as I can!


some producer ops using the new send operation with batches

Does this want picking up and finishing off?

Turns out I can't add to this branch (presumably because it's Zach's) so I've continued here: #906


Want to add project pokemon match cards Game

Is your feature request related to a problem? Please describe.
It will be a memory pokemon match card game with different pokemon hidden behind the card you have to memorize and have to find the right match
Describe the solution you'd like
It will be card game make through html,css and js
Additional context
Stack:
Html
CSS
JavaScript
Steps Followed✅
Check the box with [x] If you followed all the steps.

[x] Is your feature request related to a problem? Please describe.
[x] Describe the solution you'd like
[x] Describe alternatives you've considered
[x] Additional context

Code of Conduct✅

[x] I agree to follow this project's Code of Conduct.
[x] I checked the current issues for duplicate problems.
[x] I have gone through project README.

As a participant of GSsOC '22, I would like to work on it.


Several issues with Angular 6 "Migration"

I followed this tutorial to upgrade to angular-token: https://github.com/neroniaky/angular2-token/wiki/Migrate-to-Angular-Token-6.0
However, I verify that there are missing steps and also the integration is not that "smooth".
Let me point out some issues Im facing:

Login and Register interface changed from "email" to "login", I was forced to make the following change and also similar on registration component to keep things working

    this.signInUser =
      {
        //email: this.email.value,
        login: this.email.value,
        password: this.password.value
      }

The signin remains making successful calls with backend because it doesn't change: example request {"email":"xxxx","password":"yyyy"}, however as stated on the previous step, I was forced to change email from login somehow it still sends the request with email, really weird...

Another inconsistency with regular devise-token backend on the registration. Contrary with login that still sends the email, the register does not work because no email field is sent:

Example request:
{"login":"dadsa@fad.da","password":"123456","password_confirmation":"123456","confirm_success_url":"http://localhost:4200/home"}
Response:
//422 Unprocessable Entity
    "errors": {
        "email": [
            "can't be blank"
        ],
        "full_messages": [
            "Email can't be blank"
        ]
    }

Hey. Thanks for bringing issues to our attention. We're in the process of moving to HttpInterceptors, but we're not done yet. Trying to get as many kinks out of the cable as possible.
The change in naming to the backend of the login field can be changed by the loginField configuration. Please refer to the Configuration. So basically, your application should always communicate to the angular-token library through login-field and the library will make sure it uses the correct loginField to communicate with the backend.
Which angular-token package version did you use? Because the problems you describe in the registerflow are indeed present in version 6.0.0-rc.0, but should be solved in version 6.0.0-rc.2.

Can you please elaborate a complete tutorial for the new version?
I'm having issues putting this new version working. The registration and login works but the login never returns true with this.authToken.userSignedIn(); .
I used the angular2-token version without issues, this update was the worst idea I had..
Thanks for your quick reply btw 👍

With HttpClient, you no longer automatically get the status back. Why would you want to check the status?
If you want to handle errors, you should resolve it like this:
  public logoutx(){
    this.authService.signOut().subscribe(
      res => {
        console.log(res)
        this.router.navigate(['/']);
        window.location.reload();
      },
      err => {
        console.log('Oops! Something went wrong!', err);
      }
    )
  }

You can still check res['success'] if you want to check whether it was succesfull.
Do I understand from your previous post that your issues have been fixed?

@acegilz I added an additional step for the migration tutorial, which explains the 'login' field.

@arjenbrandenburgh thanks for the clarification. In fact I don't need to check the status that way, I just saw it many times, including on templates and also several tutorials and the truth is that it worked there (registration module) until I updated this package to the new version and angular to 6, so I don't know why exactly it stopped worked but no problem because the res already passes the success responses..
Regarding your lasts post it was a great help to put things working.. Everything is fine now 👍 thanks
@neroniaky Great step, thanks and keep going with the wonderful work guys