HTTP and HTTPS mode, Document array support

Not sure if you're interested in taking PRs as this is a fair bit different than your service.
This allows the container to run under HTTPS or HTTP (good for local testing) and accept HTML/CSS directly for rendering. We also added document array support so you can pass in multiple documents/URLs, each with their own configs, and get back a single PDF to build more complex documents. For images, it just uses the first element in the array.

Hi, thanks for that.
Looks interesting, I'll definitely have a look.
When passing in the HTML, how do you deal with relative links?

We aren't currently dealing with relative links so we haven't tried to solve it. However, I would probably try inserting a <base href="..."> in the had before I posted the HTML (with the expectation that wkhtmltopdf understands that).


Cicero 0.21: Latest Ergo / Markdown transform, support for formulas and timezones

Changes

Formulas are back!
Timezones are back!

Coverage increased (+0.08%) to 95.683% when pulling 600e3d5d5df072a7033e13449da5c4b22865d871 on jeromesimeon:parserv2 into 0d774fe4bb98847beb1292a40af7b43cc3ceb16c on accordproject:master.


Add favicon to the generated app

Seems like a nice thing to do + we need to figure out how to do it with webpack 😄

The app doesn't build if favicon.ico is missing:
> react-scripts start
Starting the development server...
Failed to compile.

Error in   Error: HtmlWebpackPlugin: could not load file /Users/modernserf/work/src/github.com/modernserf/c  ritter/favicon.ico

  - index.js:303
    [critter]/[react-scripts]/[html-webpack-plugin]/index.js:303:27

@modernserf Good point. Let’s discuss in https://github.com/facebookincubator/create-react-app/issues/96.


Add and annotate root Facet classification; align core:hasFacet

@plbt5 , this PR is ready for your review.
I have separately caught PR #5 up with this PR, so if you merge 5, 7 will be merged automatically.

@plbt5 I separated the alignment in this PR because I realized there is at least one Facet subclass that could complicate where the alignment lands.  I think it is safe to merge this PR and then hold the discussion with the OCs.


GET list endpoints require ids

Describe the bug
For the get endpoints of:

Settings
Services
TechnicalServices

The endpoints does provide a list for existing services. Nevertheless, the id is mandetory. It doesn´t matter which id is provided, the result ist the same list of results
Details
The get endpoints are supposed to serve a list of existing services in the current organization without providing a id
To Reproduce
Steps to reproduce the behavior:
Invoke <BASEURL>/oscm-rest-api/#/users and login with user key 10000
Click 'GET' and 'Try Out'
See the field 'serviceId'

Expected behavior
'id' is optional and when omitted execute leads to returning a list.
Observed behavior
'id' is mandatory and can't be omitted for execution.

This task will be done within https://github.com/servicecatalog/oscm-rest-api/issues/194


Support Java 8 Datetime API

The Java client is already packed up for Java 8, so would you consider supporting the Java 8 Datetime API instead of or in addition to Joda Time?

Hey @Chessray, sorry for the outrageously late getting back to you on this issue. I'm just doing something housekeeping on our open issues. Does this issue still stand and is still something you'd like to see changed?

Sorry, I've moved projects (and employers and type of employment) long ago and haven't done any governmental work in more than two years. I still think it would be a good addition, but I won't be using it anytime soon.

Thanks :)
For others reading, I know this won't reach very high up our priority list (unless we hear from more users of the client saying this would be useful) and is unlikely that we will action any time soon. We always welcome PRs though if anyone does want to contribute.

Hi @idavidmcdonald I raised #202 for this


Deployment of the web-app on linux server

Hello all!
I am developing a web application with Python and Cortex API.
At the moment my application is only running locally and I am looking for a way to deploy the application on the Internet on a Linux server. Can you please explain me if this is possible and how?
Thanks a lot.

Hi @nigoshaya
You can install Cortex on your Linux server. But this Cortex won't be able to connect to your Emotiv headset. You must install Cortex locally so it can connect to you headset via Bluetooth or USB dongle.


Update Cloudinary

Cloudinary supports 2FA.
See #962.

+looks-good

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from the review on Reviewable.io

The day when @jamcat22 spammed Carl's inbox.
It's SPAMCAT22 The movie
Starring Arnold Schwarzenegger as Jamcat, Chuck Norris as Carl and Tom Cruise as @mxxcon.

I'm not crazy. :-1:

So who do you want to play you? Morgan Freeman? Oh I want Morgan Freeman! Want to trade Chuck Norris? :smile:

@Carlgo11 Well I had to go to sleep at some point. :smile:
Also, remember when I did this? #448
I did one of the worst things I could have possibly have done: inserted 20 sites into one pr.

Nope I don't. Thanks for reminding me :wink:

Just did a search in my inbox. 276 mails from you :smile:

Well I have 472 from @mxxcon and 203 from Josh. (out of 2165 sent by github)


Remove horizontal lines which breaks headings

no clue what those are for... if it looks right, 👍

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


chore(master): release 1.7.0

:robot: I have created a release beep boop
1.7.0 (2023-01-15)
Features

deps: update pnpm to 7.25.0 (b6e28b3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-node/releases/tag/v1.7.0 :sunflower:


jekyll help does not show jekyll-compose commands; Mac OSX: gem 2.4.5: jekyll 2.5.3: jekyll-compose 0.2.1:

I am unable to get the jekyll-compose commands to run. Specifically I tried "jekyll draft" and I got:
Invalid command. Use --help for more information
When I look up jekyll help, I do not see the "draft", "post" or "publish" commands.
gem list --local shows me the following versions of relevant gems:
jekyll (2.5.3, 2.4.0)
jekyll-coffeescript (1.0.1)
jekyll-compose (0.2.1)
jekyll-gist (1.1.0)
jekyll-import (0.5.3)
jekyll-mentions (0.2.1)
jekyll-paginate (1.1.0)
jekyll-redirect-from (0.6.2)
jekyll-sass-converter (1.3.0, 1.2.0)
jekyll-sitemap (0.6.3)
jekyll-watch (1.2.1)
Not too sure, what I am doing wrong.

p.s. I installed jekyll-compose with:
gem install jekyll-compose

same here. I also tried to add the group to the Gemfile as suggested:
group :jekyll_plugins do
    gem 'jekyll-compose'
end

but it did not help.

Same here

you need to add jekyll-compose to your gemfile as @elipapa indicated, then use bundle exec jekyll <command>.

Thanks.
Already added jekyll-compose to my Gemfile,
and invoked the command with bundle, still getting
Invalid command. Use --help for more information

using Jekyll 2.5.3

same here. unfortunately bundle exec jekyll <command> does not help.

Oh, sorry guys. I think this only works with the new Jekyll 3 beta.

Would it not be sensible to note the dependance on the Readme to prevent more people from falling at this hurdle?

Would it not be sensible to note the dependance on the Readme to prevent more people from falling at this hurdle?

Yes, it would be very sensible. Would you propose an update with a pull request, please?

I will indeed

#14 is the pull request with the updated readme.
Is there anything we can do for jekyll < 3 beta users?

No, the code that brings in these commands isn't in 2.x.
Thanks for the PR!


Add Parameter serialization

Summary
Parameter is a sub class of ParameterExpression and that's why Parameter was getting converted to ParameterExpression.
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 69.106%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2049

Relevant Lines:
2965

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 69.106%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2049

Relevant Lines:
2965

💛  - Coveralls


TypeError: overrideId is not a function

Hi, based on the API docs, there are 2 params on function callback

annotation
overrideId

I was trying to use overrideId and it give TypeError: overrideId is not a function . here my code
this.anno.on('createAnnotation', async function(annotation, overrideId) {
    const slideId = self.slide.id
    const savedAnnotation = await self.saveAnnotation(slideId, annotation)
    overrideId(savedAnnotation.id)
});

I use @recogito/annotorious-openseadragon: 2.6.0 which using annotorius v 2.6.0 and client core 1.6.0
Is it my implementation is wrong or something else?
Thanks in advance

Hi @sunupf,
weird - your code looks ok. I just checked & can't seem to reproduce the problem. OverrideId is correctly passed to the createAnnotation event handler, both in AnnotoriousOSD 2.6 and the latest 2.7. I also tried with an async/await example and that seemed to work just fine, too.
Perhaps there's something additional going on in your code. Do you have a full code sample you can post? Can you try to log the value of overrideId to see if something else is being passed?

Hi thanks for the quick reply,
ok let me explain my code.

We user headless mode, so we can show custom popup to input label, (why not using widget, well I guess we try something that comfort for my team :)) . it's something like this :

When user insert the label, we update the annotation using

await this.anno.updateSelected(this.selectedAnno);
this.anno.saveSelected();

Because it will trigger createAnnotataion event, I code the save and try to override the Id using previous code

this.anno.on('createAnnotation', async function(annotation, overrideId) {
    const slideId = self.slide.id
    const savedAnnotation = await self.saveAnnotation(slideId, annotation)
    overrideId(savedAnnotation.id)
});

but it throw error when the code hit overrideId. Here's the screenshoot for error an I try console.log before the call

FYI : I use annotorious plugin called @recogito/annotorious-selector-pack , it seems use different version of annotorious

Thanks
Sunu

Thanks for this! Yikes - that is indeed a bug in Annotorious. Works in normal mode (where I tried), but is broken in headless mode. I have already added a fix to the main branch in annotorious-openseadragon. Will do a bit more testing & check if the same problem also exists in Annotorious' normal version (non-OSD). The next release will definitely work.

This is now fixed on the main branches of both Annotorious standard and Annotorious OSD. Will be included with the next releases.


Data for segmentation

I wonder which one is the file of  you mentioned(the dense point clouds with segmentation labels)

I want to ask you whether there are labels in these two files?

This parameter of "segmention = 1" is not in "octree.exe"

This parameter of "segmention = 1" is not in "octree.exe"

The segmentation.zip is a lmbd database, which can be directly used by caffe.
The zip files, like 04379243.zip, contain raw point files.
Ignore the segmentation parameter. I will update the readme soon.

很感谢您光速的回复，但是我现在想用tensorflow版本，也就是说我不能直接用lmbd 格式，我需要原始点云加标签的数据集，然后按照你的readme 去运行，然后在运行octree.exe时会产生带有标签的octree file ,但是现在没有也就是说我用的点云文件里面是没有标签的，是这样吗？ 如果是这样您这边有带有标签的点云吗？  我看readme里面说的是有的你？  谢谢！
---------- 原始邮件 ------------------
发件人: "Peng-Shuai<EMAIL_ADDRESS>发送时间: 2020年3月14日(星期六) 上午10:40
收件人<EMAIL_ADDRESS>抄送: "可爱男孩"<2232661644<EMAIL_ADDRESS>主题: Re: [microsoft/O-CNN] Data for segmentation (#79)
The segmentation.zip is a lmbd database, which can be directly used by caffe.
The zip files, like 04379243.zip, contain raw point files.
Ignore the segmentation parameter. I will update the readme soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Feat/top app bar/iss 499

A Successful merge would close issue #499 and introduce a slew of breaking changes

[x]   Attempts to create a more declarative syntax for top-app-bar.
[x]   implements missing foundation methods initScrollHandler and destroyScrollHandler.
[x]    updates component mdc-top-app-bar to 0.44.0

note: golden.json has not been updated.

I signed it

Codecov Report

Merging #688 into rc0.11.0 will increase coverage by 0.09%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##           rc0.11.0     #688      +/-   ##
============================================
+ Coverage     94.88%   94.97%   +0.09%
============================================
  Files            68       73       +5
  Lines          2853     2905      +52
  Branches        432      443      +11
============================================
+ Hits           2707     2759      +52
  Misses           50       50
  Partials         96       96

Impacted Files
Coverage Δ

packages/top-app-bar/Title.tsx
100% <100%> (ø)

packages/top-app-bar/Icon.tsx
100% <100%> (ø)

packages/top-app-bar/Row.tsx
100% <100%> (ø)

packages/top-app-bar/Section.tsx
100% <100%> (ø)

packages/top-app-bar/constants.ts
100% <100%> (ø)

packages/top-app-bar/index.tsx
100% <100%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

created https://github.com/material-components/material-components-web-react/issues/694

prominentToShortCollapsed: 203d617f5358eaa406fafbb41b3adce69bd4c3939791dbf85738b8e49e93dfa6
shortCollapsed: a711abff27747da6c91a09525a2a17ae889ce1a0ff81d513b0b3b067b66abac3
short: 90dba9623f16d58cfc4a24b2a3ab652c7e0cc6d5ccfd030566a170a55d6bce0c
twoRows: a7c025c8c04123d377f53a3e3e888eb4de78439b9dcd76c306e67712934f9ed1

merging! Signed in https://github.com/material-components/material-components-web-react/pull/688#issue-255115222

created https://github.com/material-components/material-components-web-react/issues/698 for 0.13.0


2.x BaseMessageConnection fix

NetflixOSS » eureka » eureka-pull-requests #96 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » eureka » eureka-pull-requests #97 SUCCESS
This pull request looks good


Intermittent strange png result

Using this svg file
ba585957575f9f4b5b190bbd92b13b9c.zip
I sometimes get a good png (better than any other converter I can find) (after the fix to allow 'pt' units)
but sometimes I get this:

Since this is intermittent it could work differently on other machines. Any suggestions to help track it down would be welcome.

I'm sorry for the incredibly long delay in replying in responding to you but life sometimes gets in the way of OSS development. I thank you for your patience and, although I'm closing this issue out, you may want to try using today's 0.6.0 release which includes your pt unit support etc. There's a chance the upgrade to the latest version of Puppeteer may have fixed some bugs relating to inconsistency.
If you continue to have this problem, please raise a new ticket with an SVG to reproduce the problem and, where possible, an expected and actual file of the output. I hope you can understand my delay.


add new sample sample for polyglot notebooks using llamaindex and hugging face

The new sample shows polyglot notebook using python to access local models via hugging face and advanced rag patterns using llamaindex. finally .NET is used to expose those objects and create an api that can be consume in c# code

LGTM. Thanks @colombod


Allow passing custom cmp.Options to TableTest.

We're playing around with writing TableTests for the Operator (which uses manifestival) and would
like to see if we can get a good comparator for the automatically added last-applied-configuration annotation.
If that ends up being general purpose we can upstream it here, but in general it's nice to be allowed to do
some application specific filtering of the desired state changes.

/assign @vagababov


The Rancher Windows related images should be in release image list

The Rancher Windows image list is missing in rancher-images.txt and the related tool scripts for Windows images are missing also. We need to add them.
Since the images in rancher-images.txt will be used bash shell script, we can not just add Windows images into rancher-images.txt. We need to create the rancher-windows.images.txt to store the Windows image list. And we would have rancher-load-images.ps1, rancher-save-images.ps1 and rancher-mirror-to-rancher-org.ps1 powershell scripts for that list.

@thxCode please validate it when we have a new RC release. Low risk issue as it's for windows air-gapped env

Verified in rc4
rancher-load-images.ps1
rancher-mirror-to-rancher-org.ps1
rancher-save-images.ps1
rancher/calico-cni:v0.0.1-nanoserver-1803
rancher/canal-cni:v0.0.1-nanoserver-1803
rancher/flannel-cni:v0.0.1-nanoserver-1803
rancher/hyperkube:v1.11.8-nanoserver-1803
rancher/hyperkube:v1.12.6-nanoserver-1803
rancher/hyperkube:v1.13.4-nanoserver-1803
rancher/kubelet-pause:v0.0.1-nanoserver-1803
rancher/nginx-proxy:v0.0.1-nanoserver-1803
rancher/rancher-agent:v2.2.0-rc4-nanoserver-1803


[HOTFIX] disable ctest for ConvAsmImplicitGemmGTCDynamicFwdXdlops

in PR #1619 we disable ConvAsmImplicitGemmGTCDynamicFwdXdlops, but not close it in ctest. This will cause issue #1638 .This PR is a hot fix for issue.

in PR #1619 we disable ConvAsmImplicitGemmGTCDynamicFwdXdlops, but not close it in ctest. This will cause issue #1638 .This PR is a hot fix for issue.

@carlushuang  thanks for the W/A. Could we also establish a new issue so that we can track progress on fixing this kernel? I assume we need to remove this workaround in a later time.

in PR #1619 we disable ConvAsmImplicitGemmGTCDynamicFwdXdlops, but not close it in ctest. This will cause issue #1638 .This PR is a hot fix for issue.

@carlushuang thanks for the W/A. Could we also establish a new issue so that we can track progress on fixing this kernel? I assume we need to remove this workaround in a later time.

I need to merge this to unblock develop branch, while please keep an eye on the post-merge action items.

It looks some tests for non 1x1 filters also removed by this P.R


README.md documentation.

Move to the discussions for detailed information on how to go about it

@AjeetSingh2016 yeah sure.
I'll share more details soon on this comment thread.


Broken link

Azure AD V1 samples -> Integrating Azure AD Into an ASP.NET Core Web App: results in 404 Not Found.

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

ID: 7bdae773-368b-a0c1-64d0-2a7b8c8fb134
Version Independent ID: 2816eb27-7fc5-19e6-7f79-7e81a388647f
Content: Azure Active Directory with ASP.NET Core
Content Source: aspnetcore/security/authentication/azure-active-directory/index.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks @CJEdgerton ... I'll fix that up.

The link now goes to the v2 sample. :(

@fowl ... Open a new issue using the This Page feedback button+form at the bottom of the topic and state which link you're referring to and where you think it should go.


Free: Invalid pointer on start-up

It is reproducible with a new project immediately following compose release v1.0.0-beta6-dev474.
The app crashes Before the debugger can connect to it.
It only occurs in my colleague's system. We are both using Ubuntu 20.04 LTS-3.
Exact same dev environment,

Should be fixed in 1.0.0-rc6

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Change color of footer link on hover

Issue Title:  [[Bug]: Footer hover effects improvement #447]
closes #issue_number : #447
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
Describe how have you verified the changes made.

@gaurav1603 you're not assigned to work on this issue (#447), @aanchalpahuja18 have already been assigned.
I'm closing this PR, thank you!


I wish Serial.flush() was implemented...

Like was done for OpenCR in the Issue:
https://github.com/ROBOTIS-GIT/OpenCR/issues/92
While trying to debug another issue, later Serial.prints did not show up to the debug terminal as the system faulted as the user was calling through a NULL pointer.
Side note: Wish list.  Wish that when the processor does fault like this, that maybe the fault handler tried to do some similar things to what Teensy tries to do and that is to try to service Serial (both USB and some of the HardwareSerial interrupts, such that maybe any pending messages print out.

Hi, @KurtE.
I just updated to the develop branch. Could you check it out?

Hi @OpusK,
Looks like your change helps out.  If I try running a stripped down version of #31 which referenced a NULL pointer before nothing would show up on the screen.  Sample sketch:
#include <DynamixelSDK.h>
dynamixel::PortHandler *portHandler;
dynamixel::PacketHandler *packetHandler;

void setup() {
  // put your setup code here, to run once:
  Serial.begin(115200);
  while (!Serial);
  Serial.println("Start..");

  Serial.println("Before BOOM");
  Serial.flush();

  portHandler->openPort();
  Serial.println("After Boom, should never get here");
}

void loop() {
  Serial.println("How did I get here?");
  delay(1000);
}

When I run this on the current released version, nothing shows up on the screen, then everything hangs...
When I run it using your update, I get:
Start..
Before BOOM

Then it dies as expected!
Notes: In both this and the OpenCR implementations, I don't know how CDC_Itf_TxISR is triggered/called.  That is, is it called by a timer?  Or everytime the USB timeframe for this unit is checked...  If it is like FTDI with some timeout value, than maybe would be nice to somehow trigger it NOW instead of wait for timeout... Not sure if that makes sense?
Also currently when I am building using the develop branch I am still getting debug warning on every compile line...
The last couple of files compiled:
"C:\Users\kurte\AppData\Local\Arduino15\packages\OpenCM904\tools\opencm_gcc\5.4.0-2016q2/bin/arm-none-eabi-gcc" 	-c -g -O2 -std=gnu11  -MMD -ffunction-sections -fdata-sections -DARM_MATH_CM3 -DUSE_HAL_DRIVER -DBOARD_OpenCM904 -mcpu=cortex-m3 -DF_CPU=72000000L -DARDUINO=10805 -DARDUINO_OpenCM904 -DARDUINO_ARCH_OPENCM9.04    -mthumb  -DSTM32F103xB -D__OPENCM904__ "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm/include" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/driver" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/usb_cdc" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/lib/STM32F1xx_HAL_Driver/Inc/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\cores\arduino" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904" "C:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904\lib\STM32F1xx_HAL_Driver\Src\stm32f1xx_ll_usb.c" -o "C:\Users\kurte\AppData\Local\Temp\arduino_build_880127\core\lib\STM32F1xx_HAL_Driver\Src\stm32f1xx_ll_usb.c.o"
<command-line>:0:21: warning: ISO C99 requires whitespace after the macro name

"C:\Users\kurte\AppData\Local\Arduino15\packages\OpenCM904\tools\opencm_gcc\5.4.0-2016q2/bin/arm-none-eabi-g++" -c -g -O2 -std=gnu++11 -Wno-write-strings  -MMD -ffunction-sections -fdata-sections  -fno-rtti -fno-exceptions -DARM_MATH_CM3 -DUSE_HAL_DRIVER -DBOARD_OpenCM904 -mcpu=cortex-m3 -DF_CPU=72000000L -DARDUINO=10805 -DARDUINO_OpenCM904 -DARDUINO_ARCH_OPENCM9.04     -mthumb  -DSTM32F103xB -D__OPENCM904__ "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm/include" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/driver" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/usb_cdc" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/lib/STM32F1xx_HAL_Driver/Inc/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\cores\arduino" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904" "C:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904\main.cpp" -o "C:\Users\kurte\AppData\Local\Temp\arduino_build_880127\core\main.cpp.o"
<command-line>:0:21: warning: ISO C++11 requires whitespace after the macro name

"C:\Users\kurte\AppData\Local\Arduino15\packages\OpenCM904\tools\opencm_gcc\5.4.0-2016q2/bin/arm-none-eabi-g++" -c -g -O2 -std=gnu++11 -Wno-write-strings  -MMD -ffunction-sections -fdata-sections  -fno-rtti -fno-exceptions -DARM_MATH_CM3 -DUSE_HAL_DRIVER -DBOARD_OpenCM904 -mcpu=cortex-m3 -DF_CPU=72000000L -DARDUINO=10805 -DARDUINO_OpenCM904 -DARDUINO_ARCH_OPENCM9.04     -mthumb  -DSTM32F103xB -D__OPENCM904__ "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/bsp/opencm/include" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/driver" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/hw/usb_cdc" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/lib/STM32F1xx_HAL_Driver/Inc/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904/" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\cores\arduino" "-IC:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904" "C:\Users\kurte\Documents\Arduino\hardware\opencm904\opencm9.04\variants\OpenCM904\variant.cpp" -o "C:\Users\kurte\AppData\Local\Temp\arduino_build_880127\core\variant.cpp.o"
<command-line>:0:21: warning: ISO C++11 requires whitespace after the macro name

Hi, @KurtE.

In both this and the OpenCR implementations, I don't know how CDC_Itf_TxISR is triggered/called.

For OpenCM9.04, TxISR() is registered with the software timer when CDC_Itf_Init() is called.
For OpenCR, it is called by HAL_PCD_IRQHandler().
HAL_PCD_IRQHandler() called HAL_PCD_SOFCallback().
HAL_PCD_SOFCallback() called CDC_Itf_SofISR().
CDC_Itf_SofISR() called CDC_Itf_TxISR().

I am still getting debug warning on every compile line...

In this regard, we will commit later.

Thanks, my earlier search on OpenCM did not find some information, probably had query options set bad...
So it looks like, CDC_Itf_TxISR  is called every 5ms, and if there is anything in the TX buffer it is sent out at that time.   Minor note: looks like if the in pointer wraps around, the Output will only output that part from the current out pointer to the end of buffer and then wait for the next timer to output from start of buffer to whatever the in pointer is at that time...
As for the Warning, I assumed you already was aware of it, but thought I would mention it, in case...

You are right.
Since TxISR() is executed every 5ms, it may happen that depending on the situation, it may wait for 5ms.
We could change this to 1ms, or use another timer or change it like OpenCR.

As for the Warning, I assumed you already was aware of it, but thought I would mention it, in case...

I have not used OpenCM 9.04 very often, so I have not noticed the warning problem!
Your report has helped me :) Thanks again for your contribution!

It looks like the the OpenCR version is more advanced.  Hard for me to say if it is worth it for you to do it on the lower end boards.  Especially since I am not very experienced with the STM32 layers and hardware.
I know on the Teensy boards, the USB code works a lot differently.  Maybe partially as the USB can be configured to act like a Serial port, or Mouse, or Keyboard, Joystick, MIDI, RAW and several combinations of these.   With the Serial write code on these devices.  The Serial.write code is setup to start filling a buffer, if it has enough information to fill a USB Packet (I believe your code is setup for 64 data bytes?), it then allocates a USB data buffer, and schedules it to output on the next USB slot,  if there are more than one full packet available to output, the ISR code will see that there is additional packets, if so will schedule the next one into the next free USB frame...
With each write, if there is still a partial frame of data, it will reset the timeout counter, back to the initial value.  Only after there are no more calls to Serial.write for the timeout period or the user calls Serial.flush , will it output a non full frame.   But again very different setup done by Paul of PJRC...   Sorry for being slightly off topic 8)
For this one, what you put in is probably good enough.  As it solved the problem of not being able to force the print statements to show up before code crash...
But wonder if maybe it would make sense?  to potentially have a swTimer function like:
swtimerTriggerNext(uint8_t TmrNum),

Which maybe sets: wtimer_tbl[TmrNum].Timer_Ctn  = 1;
So it forces it to call of on the next counter... Maybe need to protect the setting of variable, although I suppose that even if the ISR happened before we set this, then we set it to 1, it would simply call our function and see there is nothing to do... So maybe not too bad.   But again maybe not worth the risk or effort.

I will discuss this with the people involved.
As you said, we will need to first determine the value, opportunity cost, and priority of this update, considering our resources, the limitations of the OpenCM 9.04 CPU, the nature of the firmware, and so on.

Understood, I might play around some, to see if it can help.  Where I can see it helping out, is if for example I wish to use an OpenCM board like you do the OpenCR board talking with other board like RPI and working like the servo controller, like the R++2 code (Restore firmware), Is the Source for this some place?
Where I can see that when the 904 sees the response packet from servos and forwards to USB, it would be good to flush it out after the full packet has been received from Serial1 or Serial3...
I played around with test Example program, where I output when servo revolution was seen, or if I turn on output position data.  To the app, I added a Serial.flush() after these messages.  You can see the result of this on the Logic Analyzer...

The top two lines are the USB traffic.  The third line is data on DXL of Serial 3...  You can see the Serial packets going one after another until the Serial.print()... Serial.flush() and then you see up to the 5ms delay.   During that time you can see all of the USB traffic, each gap there is a USB frame.  Maybe does not show up easily in printout but the last one before the next data shown on DXL is wider as it shows the actual packet showing the text...
Minor NIT probably not worrying about is it is showing new data actually going out Serial3 before the actual USB output has been fully output.  That is we actually only wait until we have called: USBD_CDC_TransmitPacket, not when the hardware has actually output the packet.  Again not worth trying to fix this.

@OpusK ,
Quick update on my playing around:
I tried making the code more similar to the OpenCR board.  And instead of using the 5ms timer to flush the Serial data back from the OpenCM board to the host, instead tried turning on the Start Of Frame(SOFM) interrupt and then do like OpenCR and output on the next frame... First tried turning on and leaving it on... It appeared to interfere with the code reading results back from servos (or maybe I screwed something up...), Tried turning the SOFM on and Off.  That is anytime you called: Serial.write() it turned it on.  The code then turns it back off after the SOF interrupt is processed... Still had issue.
So for now will punt using this technique.  I put this in a new local branch on my machine, which I have not published back to github... Can if anyone is interested...
Next up, maybe see what happens if maybe change 5ms to maybe 2 or 3ms.    And/Or maybe have flush update the timer counter to 1, so the write would happen on the next ms...
Again all of this is probably out of the scope of this issue.  So you could easily close it if you like and if anything looks promising, could either reopen, or open new one, or simply do a Pull Request.

@KurtE,
This issue is currently underway to PR, so can I close it?

Sure!
From: Kei<EMAIL_ADDRESS>Sent: Sunday, August 26, 2018 5:27 PM
To: ROBOTIS-GIT/OpenCM9.04<EMAIL_ADDRESS>Cc: KurtE<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ROBOTIS-GIT/OpenCM9.04] I wish Serial.flush() was implemented... (#33)
@KurtE https://github.com/KurtE ,
This issue is currently underway to PR, so can I close it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ROBOTIS-GIT/OpenCM9.04/issues/33#issuecomment-416084400 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABfGQEHYafymTjH75WgsIw_p0cjy5ORBks5uUzy9gaJpZM4WAUPF .


Comparisons of variables with mismatch in signedness in model.h

Hello, and thanks for the excellent work on cppflow!
In model.h there are a few spots where there are signed and unsigned variables being compared. I am integrating cppflow into a build which is configured to error out on compiler warnings, meaning that I need to fix these up before I can complete my build. Below are the locations where this error is occurring:
https://github.com/serizba/cppflow/blob/883eb4c526979dae56f921571b1ab93df85a0a0d/include/cppflow/model.h#L119
https://github.com/serizba/cppflow/blob/883eb4c526979dae56f921571b1ab93df85a0a0d/include/cppflow/model.h#L127
https://github.com/serizba/cppflow/blob/883eb4c526979dae56f921571b1ab93df85a0a0d/include/cppflow/model.h#L143
https://github.com/serizba/cppflow/blob/883eb4c526979dae56f921571b1ab93df85a0a0d/include/cppflow/model.h#L162
While this is not urgent, these minor issues should probably be fixed.

@gth828r
Thanks for pointing this. If you changed it feel free to open a PR.

I won't be around today, but I should be able to submit a PR early next week. Thanks for the nudge!

Fixed with #180


Query String in Paths

The spec doesn't seem to define, if query strings are allowed in paths or not.
According to the docs on swagger.io (section Query String in Paths) this is not allowed.
There are some related issues that doesn't seem to have a definitive answer either, e.g. https://github.com/OAI/OpenAPI-Specification/issues/123.
Are there official specs regarding this problem?

The phase path is based on the RFC3986 definition.  Therefore, paths can't contain query parameters.

The phase path is based on the RFC3986 definition.  Therefore, paths can't contain query parameters.

Is this something that should be made more explicit in the specification?
When reading the specification regarding the paths object it states Templated paths with the same hierarchy but different templated names MUST NOT exist as they are identical. which would be the case for paths which only differentiate in their query params, however, the next sentence states In case of ambiguous matching, it's up to the tooling to decide which one to use. which seems to nullify the previous sentence or am I misunderstanding this?

Is this something that should be made more explicit in the specification?
When reading the specification regarding the paths object it states Templated paths with the same hierarchy but different templated names MUST NOT exist as they are identical. which would be the case for paths which only differentiate in their query params, however, the next sentence states In case of ambiguous matching, it's up to the tooling to decide which one to use. which seems to nullify the previous sentence or am I misunderstanding this?

I think you're overlooking the distinction between paths and query strings, which is pretty explicit in RFC3986. The path stops where the query string starts; even if a request contains query strings, they are not considered part of the path. The description of the paths object in the specification is about paths only; it should not be understood to apply to query strings at all.
Your statement about "paths which only differentiate in their query params," confuses the two concepts. Two paths which only differentiate in their query params would be one and the same path, not two different paths.

which would be the case for paths which only differentiate in their query params

In

Templated paths with the same hierarchy but different templated names MUST NOT exist as they are identical.

The key word is templated. The guidance refers to paths such as /users/{name} and /users/{id}.
Closing, as the spec currently as written conveys the intent.


Can i use in Xamarin.Forms?

I suspect that question is out of scope for this repo.  You may want to try http://stackoverflow.com/

sorry

why is it out of scope? it is a very basic question as Xamarin belongs to Microsoft? Are you not dotnet and microsoft? why should we ask a question to other users instead of creators?
if you arent accepting any questions please change your documentation with comments below

Feedback and Requests
Please use the issues page to submit any bugs, comments or questions, and feature requests.

@EmilAlipiev  to my knowlage Xamarin is not supported


fix(android): TabStrip style for the unselected item is not working

TabStrip style for the unselected item is not working.
Example CSS file:
MDTabStrip {
  background-color: #79d2a6;
  selected-item-color: red;
  un-selected-item-color: yellow;
  highlight-color: red;
}

TabStripClassStyle MDTabStripItem {
  background-color: #79d2a6;
}

Before bugfix:
https://user-images.githubusercontent.com/5738416/219018760-5b8756a3-85e1-4f3b-80cb-fb21084a0c8f.mp4
After bugfix:
https://user-images.githubusercontent.com/5738416/219018836-c5ee6a6a-a258-4b95-98b0-dc6f3acde74c.mp4
Resolution:
The mUnSelectedItemColor filled up on the CSS file should have greater priority than the default color.

@dgmachado thanks!

@dgmachado thank you


Unreadable UI text (4.X)

Describe the bug
I tried the launcher, but I couldn't see the UI after running it (no errors), so I'm running recaf directly:
java -cp recaf-ui-4.0.0-SNAPSHOT-all.jar;dependencies/* software.coley.recaf.Main

And the rest is history. No matter what I do, I just can't get the text to work.
Info about JDK:
openjdk 23.0.1 2024-10-15
OpenJDK Runtime Environment Temurin-23.0.1+11 (build 23.0.1+11)
OpenJDK 64-Bit Server VM Temurin-23.0.1+11 (build 23.0.1+11, mixed mode, sharing)

Maybe I can change the font? I just don't know how to do it.
You can download the build from here. I have also included the dependencies.
EDIT: I tested this in a VM, and there's no such issue. It's only a problem on my current machine. I have tried reinstalling the fonts, but issue persists. If you have an idea how to fix it (like maybe I need to install some fonts?), it would be nice, otherwise you can close the issue. Thank you!
Screenshots

I'm closing this because the issue is probably from my machine, since it works fine on a VM. Although, I haven't found a fix for it yet, but I can spend some time on this get it fixed later.

Possibly related to this? https://stackoverflow.com/questions/66747171/why-javafx-application-and-scene-builder-is-showing-garbled-text

Possibly related to this? https://stackoverflow.com/questions/66747171/why-javafx-application-and-scene-builder-is-showing-garbled-text

Sorry, I forgot to respond. Yes, that's the reason. I had the same issue with Eclipse because of Segoe UI font. Thank you for your help!


Align lines of eval output to the 1st line in repl

Hi,
Currently the lines in the Out blocks are not aligned, causing of an object with multi-line repr appear like this:
In [1]: import numpy

In [2]: numpy.arange(16).reshape(4,4)
Out[2]: array([[ 0,  1,  2,  3],
       [ 4,  5,  6,  7],
       [ 8,  9, 10, 11],
       [12, 13, 14, 15]])

This PR makes the lines in the Out blocks aligned to the first one.
In [1]: class MultilineRepr:
     2.     def __repr__(self):
     3.         return "three\nline\nrepr"

In [2]: MultilineRepr()
Out[2]: three
        line
        repr

Thanks for the pull request! I appreciate it.
I'll merge it soon. (A bit busy right now.)

Merged in: https://github.com/jonathanslenders/python-prompt-toolkit/commit/28711f90c29e41322a36553da0d9a06291333fd1
(nice patch!)

Thank you, but I've just spotted a bug in it :disappointed: . ('%r\n' % result should be '%r\n' % (result, ) otherwise it'll break with tuples.) Later today or tomorrow I'll fix it.


Fixes flipped condition in scene tree dock disable checks

Without this the scenes are uneditable, at least for me.
Might have misunderstood something. That whole line should be split off and made
clearer but I didn't want to poke the bear just yet.
Signed-off-by: Saggi Mizrahi<EMAIL_ADDRESS>

Hi @ficoos, there is now a merge conflict due to 35959f9. Is your PR still relevant? (if so you'll have to rebase it to resolve the merge conflict).

The bug has been fixed in master


[Summer 2021] [SAPP] Feature Request - Reduce number of interactions in Web UI required to re-save a filter

Scenario
Suppose a user has a saved filter named Test Filter 1 that filters for Code == 5008, but the user wants Test Filter 1 to filter for Code == 5008 and Code == 5012.
Expected workflow:
The following is the flow a user would expect to accomplish the scenario above:
Ensure you have a Pysa run in a SAPP db that contains issues with code 5008 and 5012

sapp --database-name test.db server
Click the Filter button
Click the saved filter field
Click Test Filter 1
Add 5012 to Codes field
Click the menu button and click Save. Notice on this step that the Save button is grayed out and the user isn't able to save the edited filter.

Actual workflow:
But the current flow to accomplish the scenario above:
Ensure you have a Pysa run in a SAPP db that contains issues with code 5008 and 5012

sapp --database-name test.db server
Click the Filter button
Click the saved filter field
Click Test Filter 1
Remember all the field values populated or screenshot the field values populated
Click the Clear button
Click the Filter button
Fill in the fields from your memory or a screenshot from Step 5
Add the additional things you wanted to filter for - Add 5012 to Codes field
Click the menu button and click Save
Type in the name of the filter you want to save as - Test Filter 1

I would like to work on it. But this touches functionality in https://github.com/facebook/sapp/pull/27
I am also not liking the position of save and load, what do you feel we move them outside to the form itself and then, implement collapasables and add update filter and save filter options there near clear and apply buttons?

I though it would be less work if https://github.com/facebook/sapp/pull/27 is merged, but it's taking time as it has evolved a lot, thereby, taking this up. I can rework in order of merge if needed :)

Closed via https://github.com/facebook/sapp/commit/183bff992d5e4217f4246a67a1fe42025dd5db91


feat(gen ai): add samples for aiplatform batch predict api with gemini

Description
Adds new samples for the Vertex AI "Batch Prediction" page sections:

"Batch Prediction for Cloud Storage" section
"Batch Prediction for BigQuery" section

Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[x] Tests pass
[x] Code formatted
[x] Vetting pass
[x] Please merge this PR for me once it is approved

@telpirion , @msampathkumar , I believe that all issues with this PR have been resolved. Can you please take a final look? Thanks!


Getting this pesky error after updating to latest version of the package

also geeting a blank screen on table

Could you please provide the output of flutter doctor -v, the version of the package you have (you can check pubspec.lock) and the target platform you're building your app for

Also a snippet of code how you add DataTable2 and what kind of widgets wrap it is helpful - I suspect there might be some scrollable above the table which messes with the ScingleChildScrollView's used by the widget

more Clear: page 1

this is Expercted Result:

You can't put DataTable2 inside SingleChildScrollView and Column, it won't work. Those parents don't have dimension constraints.

If you may, please remove these from the following file for understanding
https://drive.google.com/drive/folders/1Nwscj8gIjvDT9X6gSgZRdmwB6tf-6RJD?usp=sharing.
.:
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
06/09/22,
11:59:30 AM
On Thu, Jun 9, 2022 at 8:13 PM Maxim Saplin @.***>
wrote:

You can't put DataTable2 inside SingleChildScrollView and Column, it won't
work. Those parents don't have dimension constraints.
—
Reply to this email directly, view it on GitHub
https://github.com/maxim-saplin/data_table_2/issues/98#issuecomment-1151252712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSHZ4NE67LHVAE6Y473VOICX5ANCNFSM5YG6N5DQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

What should be done with the column? i have already removed Singlechildscroll

could you please tell what should be used alternative to column

Either reome column and leave just the table widget OR try putting DataTable2 into Expanded or Flexible

Brother, I am trying but still having the same issue, could you please help a bit, This problem never occured on previous versions though. I am a beginner and dont know much about this...

Try this:

Instead of this:

I changed the code to this:
[image: image.png]
but i stil get this error:
[image: image.png]
On Mon, Jun 13, 2022 at 7:08 PM Maxim Saplin @.***>
wrote:

Try this:
[image: image]
https://user-images.githubusercontent.com/7947027/173372300-f4a37659-09e7-4a58-9259-feda4653e0d1.png
Instead of this:
[image: image]
https://user-images.githubusercontent.com/7947027/173371899-e7b1ae51-6ffc-48cf-bc10-5c36841702df.png
—
Reply to this email directly, view it on GitHub
https://github.com/maxim-saplin/data_table_2/issues/98#issuecomment-1153965205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSFJTULUQDHYE6EJ25DVO46FTANCNFSM5YG6N5DQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

i mean could you please highlight what should i remove to fix the issue?
On Fri, Jun 10, 2022 at 12:03 AM Maxim Saplin @.***>
wrote:

Closed #98 https://github.com/maxim-saplin/data_table_2/issues/98 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/maxim-saplin/data_table_2/issues/98#event-6780402776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSASINALZA2GZ7S5TNDVOI5YHANCNFSM5YG6N5DQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Pic2card blog post

Microsoft Reviewers: Open in CodeFlow

the images are showing as empty... github bug, or zero-length files?


HP ALM (QC) integration

Hi
This framework is powerful, is it possible to add another feature to connect HP ALM (Quality Center) to run or update back results using ALM API.
Thanks
Thulasi

Hi,
we have no plans to develop HP ALM (QC) integration, you can develope it by your own and we will merge your pull request.
best regards
Jakub


Correction for raw affiliation Aeres University of Applied Sciences (Netherlands); Wageningen Livestock Research (Netherlands)

Correction needed for raw affiliation Aeres University of Applied Sciences (Netherlands); Wageningen Livestock Research (Netherlands)
raw_affiliation_name: Aeres University of Applied Sciences (Netherlands); Wageningen Livestock Research (Netherlands)
new_rors: 04qw24q55;03jnx2c74
previous_rors: 04qw24q55;015d5s513
works_examples: W4288831199
contact: 69c20abb1614d98eea599f264009251d:cd578141fb5aba657b7a @ hotmail.com

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Remove helm chart version to always install latest 2.2

Signed-off-by: Feng Xiang<EMAIL_ADDRESS>
Remove helm chart version to always install the latest

@chenz4027 Wondering if you're ok to merge this without the e2e passing since we don't have the cluster anymore?

@fxiang1 yeah sure, I don't think that it would cause any issue


Affirm position no longer exists.

Affirm just opened new position for new grad. Updated. Thanks for letting me know.


After upgrade to faiss-1.7.4, Milvus ut-cpp coredump

https://github.com/milvus-io/milvus/pull/28325
https://github.com/milvus-io/milvus/pull/28356
https://github.com/milvus-io/milvus/pull/28402
Above 3 PRs report same ut-cpp coredump

/assign @alexanderguzhva

@cydrain I cannot reproduce it locally either, even with valgrind, just like you. I'll try fully replicating builder.sh from milvus

use following steps, but cannot reproduce this coredump:
under Milvus root dir, run "make build-cpp-with-unittest"
run ./cmake_build/bin/all_tests --gtest_filter="BinaryVecIndex"
 ~/vec/milvus/ [master*] ./cmake_build/bin/all_tests --gtest_filter="BinaryVecIndex"
Note: Google Test filter = BinaryVecIndex
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from BinaryVecIndex
[ RUN      ] BinaryVecIndex.All
WARNING: Logging before InitGoogleLogging() is written to STDERR
I20231114 15:16:27.458635 3248961 factory.cc:20] [KNOWHERE][Create][all_tests] create knowhere index BIN_IVF_FLAT with version 1
W20231114 15:16:27.458894 3248961 thread_pool.h:153] [KNOWHERE][GetGlobalSearchThreadPool][all_tests] Global Search ThreadPool has not been initialized yet, init it with threads num: 12
W20231114 15:16:27.459206 3248961 ivf.cc:209] [KNOWHERE][MatchNlist][all_tests] nlist(16) is too large, adjust to a proper value
W20231114 15:16:27.459316 3248961 ivf.cc:211] [KNOWHERE][MatchNlist][all_tests] Row num 10 match nlist 1
WARNING clustering 10 points to 1 centroids: please provide at least 39 training points
I20231114 15:16:27.519685 3248961 time_recorder.cc:49] [KNOWHERE][PrintTimeRecord][all_tests] Build index: done (60.495163 ms)
I20231114 15:16:27.519801 3248961 factory.cc:20] [KNOWHERE][Create][all_tests] create knowhere index BIN_IVF_FLAT with version 1
[       OK ] BinaryVecIndex.All (63 ms)
[----------] 1 test from BinaryVecIndex (63 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (63 ms total)
[  PASSED  ] 1 test.

https://github.com/milvus-io/milvus/actions/runs/6892564731/job/18750145333?pr=28473 passed


chore(package): update angular-library-builder to version 1.5.11

Closes #61

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   96.55%   96.55%
=======================================
  Files           6        6
  Lines          58       58
  Branches        3        3
=======================================
  Hits           56       56
  Partials        2        2

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 52359e7...74bc4ae. Read the comment docs.


Versioning and releases

Eventually we want robosat to be a library with a thin user-interface (command line tools) on top of it: https://github.com/mapbox/robosat/issues/9. One step in that direction is making proper releases and versioning the code.
We should

use semantic versioning for releases
look into PyPI for publishing releases

Semantic versioning is especially required to communicate with users when we change our mode architectures and when loading old .pth weight files will no longer work. We should avoid that in the first place but if we have to make changes we should be explicit about it.
The releasing playbook could look like the following

Check Travis is green
Tag the release git tag vx.y.z -a on master, git push origin vx.y.z
Make Github Release for tag
Push release onto PyPI

We should follow https://packaging.python.org/tutorials/packaging-projects/
and look into publishing packages to PyPI directly through Travis for every release we do.

With https://github.com/mapbox/robosat/pull/87 we are building Docker images now for latest master and for all tags:
https://hub.docker.com/r/mapbox/robosat/tags/
I just tagged v0.0.1.
From now on we need to respect semantic versioning, e.g. every time we break the .pth format because we change something in our model Python classes we need to bump the major version number.
The process is as follows:

make pull request
wait on Travis
merge into master
git tag vx.y.z -a

we then will automatically get Docker images for this tag (and latest for all commits in master).
cc @karenzshea


RegexSet match first [feature request]

The common case (for me at least) is to expect only one of the regular expressions in the set to match. I assume the is_match function works by finding the first match. It'd be nice to be able to know which regular expression that was. The (somewhat contrived) benchmark below shows that performing the full match and then getting the first match is slower than is_match. Could adding a function to return the index of the first match have the same performance as is_match? Let me know if I'm off in my assumptions or if there's something else causing the slowdown.
#[cfg(test)]
mod tests {
  use regex::RegexSet;
  use test::Bencher;

  #[bench]
  fn is_match_regex_set(b: &mut Bencher) {
    let regexset = RegexSet::new(&[
      r"^0+",
      r"^1+",
      r"^2+",
      r"^3+",
      r"^4+",
      r"^a+",
      r"^5+",
      r"^6+",
      r"^7+",
      r"^8+",
      r"^9+",
    ]).unwrap();
    b.iter(|| regexset.is_match("aaaaa"));
  }

  #[bench]
  fn match_first_regex_set(b: &mut Bencher) {
    let regexset = RegexSet::new(&[
      r"^0+",
      r"^1+",
      r"^2+",
      r"^3+",
      r"^4+",
      r"^a+",
      r"^5+",
      r"^6+",
      r"^7+",
      r"^8+",
      r"^9+",
    ]).unwrap();
    b.iter(|| regexset.matches("aaaaa").into_iter().next().unwrap());
  }
}

running 2 tests
test tests::is_match_regex_set    ... bench:          13 ns/iter (+/- 1)
test tests::match_first_regex_set ... bench:         320 ns/iter (+/- 8)

test result: ok. 0 passed; 0 failed; 0 ignored; 2 measured

This sounds interesting and potentially plausible, but I have to think about whether it's possible or not. The RegexSet additions to the DFA can be pretty tricky to reason about. In theory, it seems quite possible, since is_match does indeed have to observe a Match state, and if it observes a Match state, then it must also observe the identifier stored in that Match state (which corresponds to the regex in the set that matched).
I am also a little surprised at the speed difference in your benchmark, particularly given that only one regex will match and all of them are anchored (so that both regexes will short circuit). The only real extra work involved is looping over all of the NFA states in the final DFA state to find the precise Match states (which contain the ids of the regexes that matched). This same exact step would be required for your use case too, which suggests one wouldn't realize any performance gains.
With that said, it's probably worth an experiment to make sure I've got the performance analysis correct. Unfortunately, that experiment requires a bit of plumbing through both the DFA and the execution planner, so it's probably not very accessible.

(I don't know good ways to get profiling data and I'm out of my depth in this code and rust really, but here's a try.) Playing with https://github.com/pegasos1/cargo-profiler I was able to run this code for testing:
  let regexset = RegexSet::new(&[
    r"^0+",
    r"^1+",
    r"^2+",
    r"^3+",
    r"^4+",
    r"^a+",
    r"^5+",
    r"^6+",
    r"^7+",
    r"^8+",
    r"^9+",
  ]).unwrap();
  for _ in 0..10000 {
    regexset.is_match("aaaaa");
    //regexset.matches("aaaaa");
  }

I put this in a main file and swapped out which method it ran against. Then I used cargo profiler callgrind -n 20 --bin ./target/release/regex_set to get the callgrind data. Here's with `is_match:
Profiling regex_set with callgrind...

Total Instructions...2,353,786

770,108 (32.7%) dfa.rs:regex::re_set::unicode::RegexSet::is_match
-----------------------------------------------------------------------
300,002 (12.7%) re_set.rs:regex::re_set::unicode::RegexSet::is_match
-----------------------------------------------------------------------
170,004 (7.2%) exec.rs:regex::re_set::unicode::RegexSet::is_match
-----------------------------------------------------------------------
75,295 (3.2%) dl-lookup.c:do_lookup_x
-----------------------------------------------------------------------
63,269 (2.7%) ???:je_arena_tcache_fill_small
-----------------------------------------------------------------------
51,459 (2.2%) ???:je_arena_boot
-----------------------------------------------------------------------
50,979 (2.2%) vfscanf.c:_IO_vfscanf
-----------------------------------------------------------------------
50,000 (2.1%) cell.rs:regex::re_set::unicode::RegexSet::is_match
-----------------------------------------------------------------------
40,504 (1.7%) dl-lookup.c:_dl_lookup_symbol_x
-----------------------------------------------------------------------
34,041 (1.4%) ???:sdallocx
-----------------------------------------------------------------------
33,799 (1.4%) ???:mallocx
-----------------------------------------------------------------------
31,518 (1.3%) dl-machine.h:_dl_relocate_object
-----------------------------------------------------------------------
31,179 (1.3%) strcmp.S:strcmp'2
-----------------------------------------------------------------------
30,077 (1.3%) vec.rs:regex_set::main
-----------------------------------------------------------------------
30,003 (1.3%) cmp.rs:regex_set::main
-----------------------------------------------------------------------
30,003 (1.3%) lib.rs:regex::re_set::unicode::RegexSet::is_match
-----------------------------------------------------------------------
25,087 (1.1%) strtol_l.c:____strtoul_l_internal
-----------------------------------------------------------------------
20,079 (0.9%) dfa.rs:regex::compile::Compiler::new
-----------------------------------------------------------------------
20,008 (0.9%) main.rs:regex_set::main
-----------------------------------------------------------------------
20,006 (0.8%) pthread_self.c:pthread_self
-----------------------------------------------------------------------

Here's with matches:
Profiling regex_set with callgrind...

Total Instructions...27,953,521

4,410,053 (15.8%) dfa.rs:regex::exec::ExecNoSync::many_matches_at
-----------------------------------------------------------------------
3,247,128 (11.6%) ???:rallocx
-----------------------------------------------------------------------
2,584,953 (9.2%) ???:je_arena_ralloc_no_move
-----------------------------------------------------------------------
2,271,827 (8.1%) dfa.rs:regex::dfa::Fsm::next_state
-----------------------------------------------------------------------
1,634,549 (5.8%) ???:sdallocx
-----------------------------------------------------------------------
1,614,489 (5.8%) ???:mallocx
-----------------------------------------------------------------------
1,448,503 (5.2%) ???:je_arena_ralloc
-----------------------------------------------------------------------
1,191,288 (4.3%) dfa.rs:regex::dfa::Fsm::cached_state
-----------------------------------------------------------------------
910,872 (3.3%) sip.rs:_..std..collections..hash..map..DefaultHasher..as..core..hash..Hasher..::write
-----------------------------------------------------------------------
720,003 (2.6%) exec.rs:regex::exec::ExecNoSync::many_matches_at
-----------------------------------------------------------------------
709,978 (2.5%) slice.rs:regex::exec::ExecNoSync::many_matches_at
-----------------------------------------------------------------------
500,400 (1.8%) sip.rs:_..std..collections..hash..map..DefaultHasher..as..core..hash..Hasher..::finish
-----------------------------------------------------------------------
400,118 (1.4%) map.rs:_..std..collections..hash..map..HashMap..K$C$..V$C$..S....::get
-----------------------------------------------------------------------
380,317 (1.4%) raw_vec.rs:_..alloc..raw_vec..RawVec..T....::double
-----------------------------------------------------------------------
360,001 (1.3%) re_set.rs:regex::re_set::unicode::RegexSet::matches
-----------------------------------------------------------------------
290,222 (1.0%) sparse.rs:regex::dfa::Fsm::next_state
-----------------------------------------------------------------------
260,126 (0.9%) vec.rs:regex::dfa::Fsm::next_state
-----------------------------------------------------------------------
239,994 (0.9%) vec.rs:regex::exec::ExecNoSync::many_matches_at
-----------------------------------------------------------------------
210,215 (0.8%) slice.rs:regex::dfa::Fsm::next_state
-----------------------------------------------------------------------
200,042 (0.7%) table.rs:_..std..collections..hash..map..HashMap..K$C$..V$C$..S....::get
-----------------------------------------------------------------------

I'm not sure how helpful this really is. I can play with it a bit more if so.

That is certainly interesting. The fact that matches has entries for things like next_state and cached_state but is_match doesn't is surprising. I'll have to investigate when I get the chance.

This would be useful for me as well. I can imagine a number of use cases that require taking only the first match. For example URL routers in web frameworks often use regexes for flexibility and they want to provide a sequential match semantics. In this case only the first match is required. (Although you can imagine a warning in development if a URL would have had multiple matches).

While this may not be available on RegexSet in the immediate future, you will be able to do this with regex-automata 0.3 once it's out. It will expose complete support for multi-regex APIs, including even asking for capture groups (in the non-overlapping case anyway).


Debug info is missing

Hi everyone,
I am writing a perf_event probe and I am using aya-bpf and the bpf-linker. I've used the blueprint build "scripts" from Aya. When I was trying to debug some BPF verifier errors, I tried to get more info on the generated byte code using:
llvm-objdump-16 -S --no-show-raw-insn --section=perf_event target/bpfel-unknown-none/debug/my-bin
However, this does not print the source code, just the byte code:
$ llvm-objdump-16 --source --no-show-raw-insn --section=perf_event target/bpfel-unknown-none/debug/my-bin

target/bpfel-unknown-none/debug/my-bin:	file format elf64-bpf

Disassembly of section perf_event:

0000000000000000 <decode_frame>:
       0:	r6 = r1
       1:	r1 = 0x0
       2:	*(u32 *)(r10 - 0x4) = r1
       3:	r2 = r10
       4:	r2 += -0x4
       5:	r1 = 0x0 ll

I then checked if the binary has debug info at all. Since it didn't, I then changed the Cargo.toml for the ebpf code to have debug = true, but this didn't help as well.
file target/bpfel-unknown-none/debug/my-bin
target/bpfel-unknown-none/debug/my-bin: ELF 64-bit LSB relocatable, eBPF, version 1 (SYSV), not stripped

This is happens on:
$ uname -a
Linux xdp-test 6.2.0-37-generic #38~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Nov  2 22:07:48 UTC 2 aarch64 aarch64 aarch64 GNU/Linux

(this is a VM running on a macOS on arm).
Cargo and rustc are on 1.73, Aya is pulled directly from github.
Any idea on what might be wrong?

I guess this is why? https://github.com/aya-rs/aya/issues/351
Can someone confirm so we can close this?


created initial buzz.d.ts file

for the library: http://buzz.jaysalvat.com/

close the PR because this is too old.


EC2 Discovery dropped all EC2 targets when AWS EC2 API respond 503.

What did you do?
When AWS EC2 API has issues, Prometheus server loses EC2 targets.
What did you expect to see?
Prometheus should keep stale EC2 targets, when EC2 API call fails.
What did you see instead? Under which circumstances?
Prometheus dropped all EC2 targets when EC2 API respond 503
Environment

System information:

$ uname -srm
Linux 3.10.0-327.4.4.el7.x86_64 x86_64

Prometheus version:

$ prometheus -version
prometheus, version 1.4.1 (branch: master, revision: 2a89e8733f240d3cd57a6520b52c36ac4744ce12)
  build user:       root@e685d23d8809
  build date:       20161128-09:59:22
  go version:       go1.7.3

Prometheus configuration file:

global:
  scrape_interval: 600s

scrape_configs:
  - job_name: 'prod-exporter'
    scrape_interval: 30s
    scrape_timeout: 30s
    scheme: 'https'
    ec2_sd_configs:
      - region: us-east-1
        port: 8100
        refresh_interval: 600s
    relabel_configs:
      - source_labels: [__meta_ec2_public_ip]
        regex: .+
        action: keep
      - source_labels: [__meta_ec2_tag_Name]
        regex: prod-.*
        action: keep
      - source_labels: [__address__]
        modulus: 2
        target_label: __tmp_hash
        action: hashmod
      - source_labels: [__tmp_hash]
        regex: ^1$
        action: keep
      - source_labels: [__meta_ec2_private_ip]
        regex: (.*)
        replacement: ${1}:8100
        action: replace
        target_label: __address__
      - source_labels: [__meta_ec2_tag_Name]
        action: replace
        target_label: hostname

Logs:

prometheus[15492]: time="2017-01-10T20:41:48Z" level=error msg="could not describe instances: Unavailable: Tags could not be retrieved.\n\tstatus code: 503, request id: REQUEST_ID" source="ec2.go:116"
prometheus[15492]: time="2017-01-10T20:41:49Z" level=error msg="could not describe instances: Unavailable: Tags could not be retrieved.\n\tstatus code: 503, request id: REQUEST_ID" source="ec2.go:116"

This needs confirmation.

@WeiBanjo is still the case even with the latest version?
I did a big refactoring of the SD  if you want to give it try and let me know if it still experiences the same bug.
Here is a link to download an executable for Linux 64bit
https://github.com/krasi-georgiev/prometheus/releases/download/v2.0.0-beta.x/prometheus

@krasi-georgiev Thanks. I will test that version in next couple days.

We need some proper testing before the merge so I am on IRC if encounter anymore bugs.

@krasi-georgiev Your build fails on start up. See following logs. My configuration works on Prometheus v2 binary.
Is there any EC2 discover related breaking changes I should aware ?
Dec 02 18:39:18 prometheus[2540]: level=error ts=2017-12-02T18:39:18.720043373Z caller=main.go:503 err="Error loading config one or more errors occurred while applying the new configuration (--config.file=/opt/prometheus/prometheus.yaml)"

And my configuration passes config check
$ /opt/go/bin/promtool --version
promtool, version 2.0.0 (branch: HEAD, revision: 0a74f98628a0463dddc90528220c94de5032d1a0)
  build user:       root@615b82cb36b6
  build date:       20171108-07:11:59
  go version:       go1.9.2

$ /opt/go/bin/promtool check config /opt/prometheus/prometheus.yaml
Checking /opt/prometheus/prometheus.yaml
  SUCCESS: 1 rule files found

Checking /opt/prometheus/eng-alerts/prometheus/recording_rules/collector_v2/fleet/common.rules.yml
  SUCCESS: 4 rules found

Some system infos
$ uname -a
Linux ip-172-31-28-9.ec2.internal 3.10.0-693.2.2.el7.x86_64 #1 SMP Tue Sep 12 22:26:13 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

$ cat /etc/redhat-release
CentOS Linux release 7.4.1708 (Core)

@WeiBanjo  I tested using the config from your first comment and it loads fine on my side. Are you using the same config?
@Gouthamve mentioned that this might be caused by  https://github.com/prometheus/prometheus/pull/3524 so I rebased and uploaded the new version.
Can you please download again form the same link and try again.
If you current config is different please post it here so I can find what is the bug.

@krasi-georgiev New build works.
I tested EC2 API failure scenario. Prometheus keeps using cached targets after failure.
Thanks for the fix.

@WeiBanjo glad to hear than , let me know if you find any other bugs as I want to resolve before the merge.

@brancz , @brian-brazil you can close this one.


feat: implement GET /person/{id}/registered_devices/

Description
Implement GET /person/{person_id}/registered_devices endpoint
Related Tickets & Documents
Closes #23
Steps to Verify
Untested, insufficient data

@coderabbitai review


how to go to new url such as '/' => '/chat'

i need a condition to check whether change url
{callBack && ( callBack.code==0 || callBack.code==2) ? <Redirect to='/chat'/> : tip }

while callBack is same of condition, i need to change url to '/chat'
i use Redirect  to change url which i found the only useful way to change url, but now browser throw me an warning
Warning: You tried to redirect to the same route you're currently on: "/chat"

i really know the mean of the warn,but can't fix it.
that's the only place i use Router to '/chat'
export default class Root extends Component{
  render(){
    return(
      <Provider store={store}>
        <Router>

            <Route exact path="/" component={Login} />
            <Route path="/register" component={Register} />
            <Route path="/chat" component={AsyncApp} />

        </Router>
      </Provider>
    )
  }
}

i swear i never do the double things to change url to '/chat'
so i beg yours to help me fix the problem ,
or i will outlaw the react-router-dom,
i create a new react-router-dom with only one feature that  as  my given condition to change url

i really found there is no use with other way ,
but know i found the way to fix the problem
import { Route, Redirect } from 'react-router'

<Route exact path="/" render={() => (
  loggedIn ? (
    <Redirect to="/dashboard"/>
  ) : (
    <PublicHomePage/>
  )
)}/>

are you (author) change the document of https://reacttraining.com/react-router/web/api/Redirect/push-bool????????
i never saw that before


How to add custom data pulled from database

How can one add data such as name and set value to id ?

You have to create the array manually and pass it to the list prop


Parameter is not defined

I followed the instructions according to readme.md but I am getting an error in SwitchSettingsTile. I can't add children. Actually I want to add a dark mode option for this switch but I have no idea how to do. Please help me. I am new to flutter. Thanks in advanced.

Probably children takes an array of widgets but the IDE will show detailed error message.

Ok, I will check my code again. Thanks a lot.


Can't Find In Maven

https://mvnrepository.com/artifact/com.github.slack-scala-client/slack-scala-client Where is 0.2.6?
      "com.github.slack-scala-client" %% "slack-scala-client" % "0.2.6", in my build.sbt gives me
[warn] 	module not found: com.github.slack-scala-client#slack-scala-client_2.11;0.2.6

Thanks!

Hi -- looks like this JAR isn't published yet, I only see 2.4 and 2.5. Can we get the 2.6 version published to maven?
https://mvnrepository.com/artifact/com.github.slack-scala-client/slack-scala-client

Same here.
@mccartney  can you please help?

Thanks for tagging. I still haven't managed to set up notifications of new issues properly.
Let me look around.

Ah. My mistake. The release process hasn't been fully completed. The new version got stuck in the final step in Sonatype. To my defense the UI there is not very intuitive.
Fixed that. It should be working now. Thanks for reporting.

Hi @mccartney, it looks like this still isn't published https://mvnrepository.com/artifact/com.github.slack-scala-client/slack-scala-client
Would you mind taking another look? Maven reports only 0.2.4 and 0.2.5 under the new namespace

It does show up right now. So I believe, for transparency:

Initial there was a mistake on my end and it wasn't published in February 27.
I believe the fix on Mar 21 worked.
mvnrepository has been seen to be slow in refreshing the actual content; while the JARs are available in practice for Maven clients, WWW at mvnrepository is not showing it for a few hours.


chrome-extension issue

tried loading it through chrome extension like below but not working
content.js
var helperScripts = document.createElement('script');
helperScripts.setAttribute('disable-devtool-auto','');
helperScripts.setAttribute("md5", "71f6ac3385ce284152a64208521c592b");
helperScripts.src = chrome.runtime.getURL('helperRoutines.js');
document.head.insertBefore(helperScripts, document.head.firstElementChild);
helperRoutines.js -> this has the minifed code for the blocking as per your code
manifest is as follows
"content_scripts": [{
"exclude_globs": [
"chrome://extensions/"
],
"matches": [
"://.incometax.gov.in/"
],
"all_frames": true,
"js": ["jquery.min.js","content.js"]
}],
"web_accessible_resources": [{
"resources": ["inject.js","helperRoutines.js"],
"matches": [ "://.incometax.gov.in/" ]
}],
The script is getting injected but does not have any effect
after injection it looks like following

But I am able to open the dev tools, please let me know what I am doing wrong
tried below code as well
helperScripts.addEventListener("load", scriptLoaded, false);
function scriptLoaded() {
console.log("helperScripts is ready to rock and roll!");
disableDevtool(); //DisableDevTool() also tried but this is giving error always
}
Same issue with disableDevtool line giving error saying undefined function
Any help would be highly appreciated

Thanks for your attention, I will check it in next version
---Original---
From: "SplendorNet Technologies Pvt. @.>
Date: Wed, Feb 1, 2023 22:44 PM
To: @.>;
Cc: @.***>;
Subject: [theajack/disable-devtool] chrome-extension issue (Issue #52)
tried loading it through chrome extension like below but not working
content.js
var helperScripts = document.createElement('script');
helperScripts.setAttribute('disable-devtool-auto','');
helperScripts.setAttribute("md5", "71f6ac3385ce284152a64208521c592b");
helperScripts.src = chrome.runtime.getURL('helperRoutines.js');
document.head.insertBefore(helperScripts, document.head.firstElementChild);
//(document.head||document.documentElement).appendChild(helperScripts);
helperScripts.addEventListener("load", scriptLoaded, false);
function scriptLoaded() {
console.log("helperScripts is ready to rock and roll!");
disableDevtool();
}
helperRoutines.js -> this has the minifed code for the blocking as per your code
manifest is as follows
"content_scripts": [{
"exclude_globs": [
"chrome://extensions/"
],
"matches": [
"://.incometax.gov.in/"
],
"all_frames": true,
"js": ["jquery.min.js","content.js"]
}],
"web_accessible_resources": [{
"resources": ["inject.js","helperRoutines.js"],
"matches": [ "://.incometax.gov.in/" ]
}],
The script is getting injected but does not have any effect
after injection it looks like following
 <script disable-devtool-auto md5="71f6ac3385ce284152a64208521c592b" src="chrome-extension://gnpngafhhnadjdebljpoknfkcaffjfki/helperRoutines.js"></script>
But I am able to open the dev tools, please let me know what I am doing wrong

tried below code as well
 helperScripts.addEventListener("load", scriptLoaded, false);

function scriptLoaded() {
 console.log("helperScripts is ready to rock and roll!");
 disableDevtool(); //DisableDevTool() also tried but this is giving error always
 }

Same issue with disableDevtool line giving error saying undefined function

Any help would be highly appreciated

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>
I tried diable-devtool in chrome-extension, and it works fine.
So maybe this is not problem with diable-devtool, I think you can try putting the code directly in the Google plugin where you can run js，and see if it works
Start by copying all the code here into your js, then add a line of code below
DisableDevtool();

Ok will try it again, thanks for replying back, really appreciate the help. -- Thanks and RegardsAditya AchlerkarCEO, CTO,ExpandX Solutions Pvt. Ltd.www.expandx.inDirectorSplendorNet Technologies Pvt. Ltd., Pune+91<PHONE_NUMBER> | +1<PHONE_NUMBER> | Skype: aditya_a_a31www.SplendorNet.com    16.02.2023, 21:44, "Tack Chen" @.>: I tried diable-devtool in chrome-extension, and it works fine.So maybe this is not problem with diable-devtool, I think you can try putting the code directly in the Google plugin where you can run js，and see if it worksStart by copying all the code here into your js, then add a line of code belowDisableDevtool();—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.>


[FEATURE] ADD private mode for poll results.

Description
Current issue:
when someone access the discovery page they can view the polls and can also vote but along with that they can also access the poll results ... there might be some polls whose results the poll owner might not want to show to public
Solution:
Add private poll results option ... so the owner can hide the poll result form other users and only he can access the results form his account once logged in ... for normal polls which don't choose public should still be showing the results to public

$ open to all gssoc 23 contributors

I want to work on this issue under gssoc'23

@vk20july sure go ahead


Updating ARC to reflect transfer to actions organization

Regarding https://github.com/actions/actions-runner-controller/discussions/2072, this PR updates references to the old organization to being adopted by GitHub Action team.

thanks @andyfeller


File structure

So I'm thinking about making the switch from CRA to next.js. One question I have is what is folder structure like for a larger app. I have a modules folder now for my pages. So for example I have a login module that has the main login component, a folder for tests, a folder for any components that login needs and a folder for the mobx store. Is this possible with next, or do I need to go about it differently?

next.js doesn't require any file structures except pages dir and static dir. You can have any folders in your app.


Fallback locale if current locale is not supported

Environment
Tested on Android Emulator
Package name and version:
form_builder_validators 8.3.0
Description
If you set the locale to something that is not supported (like Norwegian, NO) you'll get when saving with validation:
======== Exception caught by widgets ===============================================================
The following message was thrown:
Warning: This application's locale, no, is not supported by all of its localization delegates.

• A FormBuilderLocalizations delegate that supports the no locale was not found.

The declared supported locales for this app are: no, da, de, en, et, fi, fr, lt, lv, nl, pl, sv

See https://flutter.dev/tutorials/internationalization/ for more information about configuring an app's locale, supportedLocales, and localizationsDelegates parameters.
====================================================================================================

What you'd like to happen:
A fallback locale option would be nice.
If a locale is not supported by FormBuilderLocalizations, it should fallback to for instance English.
Alternatives you've considered:
Ignore validate and just save() instead of saveAndValidate()

Hi!
Can share your main App code?
Something like this:
    return MaterialApp(
      title: 'Form Builder Validators Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const HomePage(),
      supportedLocales: [...FormBuilderLocalizations.delegate.supportedLocales],
      localizationsDelegates: const [
        ...GlobalMaterialLocalizations.delegates,
        FormBuilderLocalizations.delegate,
      ],
    );

Thanks.
PS: This problem should be a bug and not a enhancement

@deandreamatias I'm getting the same error and also believe that English must be default in such case.
Don't know how App code could help, my code looks like this:
return MaterialApp.router(
        routerDelegate: App.router.delegate(navigatorObservers: () => [appRouteObserver]),
        routeInformationParser: App.router.defaultRouteParser(),
        title: App.config.appTitle,
        theme: appTheme.main,
        debugShowCheckedModeBanner: false,
        showSemanticsDebugger: false, // Accessibility preview mode
        localizationsDelegates: const [
          customDelegate, //S.delegate,
          formBuilderLocalizFixDelegate, //FormBuilderLocalizations.delegate,
          ...GlobalMaterialLocalizations.delegates,
        ],
        supportedLocales: S.delegate.supportedLocales,
        locale: createLocale(settingsStore.locale),
        builder: (BuildContext context, Widget? child) {

I have fixed FormBuilderLocalizations in my project, because there is another nasty bug in it (let me know if separate issue should be opened).
class FormBuilderLocalizations {

  static Future<FormBuilderLocalizations> load(Locale locale) {
    final name = (locale.countryCode?.isEmpty ?? false)
        ? locale.languageCode
        : locale.toString();
    final localeName = Intl.canonicalizedLocale(name);
    return initializeMessages(localeName).then((_) {
      Intl.defaultLocale = localeName; << !!! This is WRONG and must not be called, because breaks other localizations loading
      final instance = FormBuilderLocalizations();
      FormBuilderLocalizations._current = instance;

      return instance;
    });
  }

So my fixed FormBuilderLocalizations looks like this:
const formBuilderLocalizFixDelegate = _FormBuilderLocalizFix();

class _FormBuilderLocalizFix extends AppLocalizationDelegate {
  const _FormBuilderLocalizFix();

  @override
  Future<FormBuilderLocalizations> load(Locale locale) async {
    //XXX: without this delay, switching languages in the app is getting completely broken
    await Future.delayed(
        Duration(milliseconds: 600)); // 200 was not enough, so increased 3 times to be safe

    var supp = super.isSupported(locale);
    if (!supp) {
      final l = Locale('en', locale.countryCode);
      final savedDefaultLocale = Intl.defaultLocale;
      final rslt = await super.load(l);
      Intl.defaultLocale = savedDefaultLocale; // Fix for intrusive code of super.load()
      return rslt;
    } else {
      return super.load(locale);
    }
  }

  @override
  bool isSupported(Locale locale) {
    var supp = super.isSupported(locale);
    if (!supp) {
      var l = Locale('en', locale.countryCode);
      supp = super.isSupported(l);
    }
    return supp;
  }
}

This packages has a some issues with i18n. Maybe we need remove this feature to focus only on validators, that are the really importante

@deandreamatias  I think initially it will be just enough to implement two simple fixes:

English as default if current language is not supported,
Remove Intl.defaultLocale = localeName;

any fix?

With #22 I think that this is solved


Initial version of MS acquisition geometry storage

The PR enables sm-engine to store a json object in the DB, which describes the geometry of mass spec acquisition in terms of pixel layout and pixel sizes.
Some ideas are maybe not implemented in the best way. I'll highlight them in the diff shortly.
The acquisition geometry can be stored, but nothing is implemented to retrieve it, e.g. for the web app. I'm planning to add necessary endpoints later, likely within this PR, after the current code is reviewed. I also haven't yet updated any tests that were affected by the changes.

@iprotsyuk Please make sure the code passes existing tests. It would be nice to have tests for the new classes as well. Once it's done, we'll do merge

@iprotsyuk By the way, I didn't see a parser_factory for LC data

@intsco, yes, I'll add new tests.
As for the LC data parser factory, would you suggest to keep it in the master branch or in a separate "LC branch" instead?

@iprotsyuk all the code should eventually be merged into master. Otherwise, it will be quite difficult to handle cross updates of both branches.

@intsco, alright. I wasn't sure about this, so didn't add the LC implementation. Will add it to this PR as well

@intsco, I think all the discussed points are addressed in my latest commits

@iprotsyuk the next would be pulling all the changes to master and testing it components together first locally. Have you finished with changes of the graphql and web app?

@intsco, not yet. I thought these components could be merged independently of each other provided no existing tests fail and new tests added where appropriate.
But if for you it's easier to merge everything at once, we can wait till other layers are ready.


Moved intermediate factor out of sum in VE

The optimal variable elimination sum would move the intermediate factors outside of sums over variables not contained in the factors' scope.
Here this means moving p(\bar x_6 | x_2, x_3), which is the intermediate factor resulting from summing over x_5, out of the \sum{x_4}.
I also added a line showing how the sums were pushed into the product, before summing out x_5, whereas before you (or maybe I) did this both in a single line.

Right on, thank you!


alarm state

If I get the state inside the alarm, it is not the current state, it is just the initialisation data
how can you get the current state and change it?

If you initialize the DO with { persist: true }, this should work without additional steps I believe.  I use this for the auto-expirations behind itty.sh :)

Otherwise, when the DO sleeps, memory/state is lost.

I think you're right. Maybe either I was using an older version. (now upgraded to the latest version). Or maybe I did something wrong. Thanks


can't configure search to be on bottom

Describe the bug
https://github.com/folke/noice.nvim/wiki/Configuration-Recipes#use-a-classic-bottom-cmdline-for-search
this doesn't work
Which version of Neovim are you using?
NVIM v0.9.0-dev-77-gf175ca9f7
To Reproduce
Steps to reproduce the behavior:
add what's written in wiki to config
/
Expected Behavior
search on bottom
Config
local noice = {}

noice.plugins = {
    ["noice.nvim"] = {
        "folke/noice.nvim",
    },
    ["nvim-notify"] = {
        "rcarriga/nvim-notify",
        module = "notify",
    },
    ["nui.nvim"] = {
        "MunifTanjim/nui.nvim",
        module = { "nui" },
    },
}

noice.configs = {
    ["noice.nvim"] = function()
        vim.o.lazyredraw = false
        require("noice").setup({
            cmdline = {
                enabled = true,
            },
            popupmenu = {
                enabled = true,
            },
            notiy = {
                enabled = true,
            },
            messages = {
                enabled = true,
            },
            lsp_progress = {
                enabled = false,
            },
            routes = {
                {
                    filter = {
                        event = "cmdline",
                        find = "^%s*[/?]",
                    },
                    view = "cmdline",
                },
            },
            views = {
                cmdline_popup = {
                    position = {
                        row = 5,
                        col = "50%",
                    },
                    size = {
                        width = 60,
                        height = "auto",
                    },
                },
                popupmenu = {
                    relative = "editor",
                    position = {
                        row = 8,
                        col = "50%",
                    },
                    size = {
                        width = 60,
                        height = 10,
                    },
                    border = {
                        style = "rounded",
                        padding = { 0, 1 },
                    },
                    win_options = {
                        winhighlight = { Normal = "Normal", FloatBorder = "DiagnosticInfo" },
                    },
                },
            },
        })
    end,
}

return noice

The grammar changed slightly in the last commits and it isn't reflected yet in the wiki: you can configure search at the bottom by passing a specific view into the format table in the setup, namely:
		format = {
			cmdline = { pattern = "^:", icon = "" },
			search_down = {
				kind = "Search",
				pattern = "^/",
				ft = "regex",
				view = "cmdline",  <--- this here
			},

Fixed the wiki https://github.com/folke/noice.nvim/wiki/Configuration-Recipes#use-a-classic-bottom-cmdline-for-search


Does updating the key version for a customer-managed key have any effect on decrypting the content/doc/blob?

The document explains that "Updating the key version for a customer-managed key does not trigger re-encryption of data in the storage account. There is no further action required from the user.". From the document though, it is not very clear, what effect the key update has, if there is any, during the decryption of the content/document, stored in Azure Storage.
Can we please clarify so customer has full context of the effect, both encryption and decryption, using the customer managed keys, when the keys are getting updated.

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

ID: 7f527b03-ace9-5b5c-0def-d4c6fe3451f3
Version Independent ID: aa34244f-414d-70ec-16e9-bc52c5cde909
Content: Customer-managed keys for account encryption - Azure Storage
Content Source: articles/storage/common/customer-managed-keys-overview.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@msdpalam Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@msdpalam - I think the docs is pretty clear - updating the key version has no effect on encryption/decryption of the blob. It's simply a different key version that is subsequently used for encryption/decryption. If one of those actions is in progress, it will not be affected by the new key version; the system will handle this and use the new key version next time there is a read or write to the blob.
#please-close

@tamram the documentation states that after 24 hours the original key can be disabled. What has happened with the existing encrypted data in the meanwhile? has it been re-encrypted with the new key? Can I see this re-encryption progress somewhere? How is it possible to guarantee this time window of 24 hours even with large volumes of storage?

@cveld - the CMK wraps a root key, so there's no need for the data to be decrypted or re-encrypted. Azure just changes the wrapping key, and that's an instantaneous change. That's why we state in the docs that changing the key doesn't trigger re-encryption.
The capacity used by the account makes no difference.


server: do not update background cpu limit when auto-tune is off

What is changed and how it works?

Issue Number: Close #13055
What's Changed:

ThreadTime is collected when the cpu limit in the QuotaLimiter
is not infinity.

#12679 updated the background cpu limit even if auto-tune is off,
which is unnecessary. So, that causes additional cost of collecting
thread CPU time in some critical paths.

This commit sets cpu_time_limit of the QuotaLimiter only if auto-tune
is enabled, so we don't waste effort to collect CPU time when
auto-tune is not enabled.

Check List
Tests

Manual test (add detailed scripts or steps below)

ThreadTime does not appear on the flamegraph now.
Release note
None

cc @BornChanger

seems quota limiter won't work if auto-tune is disabled?

If quota is set explicitly (through foreground_xxx_limit or background_xxx_limit), it will work.

I got your point now.  The redundant setting's side effect hurts performance. LGTM~~

Please update the issue description, Rest LGTM~

OK. I've clarified in the issue that the regression is because additional thread time collecting happening in get or process_write.

/merge

/test


image is broken for LLaVa models

The bug
I am attempting to run LLaVa through LLamacpp and I am getting incorrect responses
To Reproduce
# Imports
import guidance
from guidance import image
from guidance import user, assistant, system
from guidance import gen, select
from guidance import capture, Tool, regex

# Paths
path_miqu = "/llava-v1.6-34b.Q6_K.gguf"

# Models
llama2 = guidance.models.LlamaCpp(path_miqu, n_gpu_layers=-1, n_ctx=5000)

lm = llama2
lm += "USER: What is this an image of?" + image("/forza.png") + "\nASSISTANT:" + gen(stop="</s>")

Output

Alternatively using a different image

System info (please complete the following information):

OS Ubuntu 22.04
Guidance Version (1.10)

The model works perfectly in Ollama and generates the proper responses

I attempted to run the same image using llamacpp and it worked successfully
import base64

def image_to_base64_data_uri(file_path):
    with open(file_path, "rb") as img_file:
        base64_data = base64.b64encode(img_file.read()).decode('utf-8')
        return f"data:image/png;base64,{base64_data}"

# Replace 'file_path.png' with the actual path to your PNG file
file_path = '/riverwood.jpg'
data_uri = image_to_base64_data_uri(file_path)

from llama_cpp import Llama
from llama_cpp.llama_chat_format import Llava15ChatHandler
chat_handler = Llava15ChatHandler(clip_model_path="/mmproj-model-f16.gguf")
llm = Llama(
  model_path="/llava-v1.6-34b.Q6_K.gguf",
  chat_handler=chat_handler,
  n_ctx=2048, # n_ctx should be increased to accomodate the image embedding
  logits_all=True,# needed to make llava work
  n_gpu_layers=-1
)
llm.create_chat_completion(
    messages = [
        {"role": "system", "content": "You are an assistant who perfectly describes images."},
        {
            "role": "user",
            "content": [
                {"type": "image_url", "image_url": {"url": data_uri }},
                {"type" : "text", "text": "Describe this image in detail please."}
            ]
        }
    ]
)

Output
The image depicts a scene from what appears to be a video game, given the graphical style and the characters' attire. In the foreground, there is a character wearing armor with a helmet that has horns attached.

I attempted to modify the Guidance LLamaCPP object by passing in the chat handler
# Imports
import guidance
from guidance import image
from guidance import user, assistant, system
from guidance import gen, select
from guidance import capture, Tool, regex

# Paths
path_miqu = "/llava-v1.6-34b.Q6_K.gguf"

# Models
from llama_cpp.llama_chat_format import Llava15ChatHandler
chat_handler = Llava15ChatHandler(clip_model_path="/CloserModels/mmproj-model-f16.gguf")
llama2 = guidance.models.LlamaCpp(path_miqu, n_gpu_layers=-1, n_ctx=5000, chat_handler = chat_handler) #n_ctx=8192

However this made no difference and the model still failed.

I am facing this same problem. Any update on this issue?

Any updates?

I don't think supporting images for opensource multi-modal models is pursued actively, https://github.com/guidance-ai/guidance/issues/554#issuecomment-1878163908
This support was introduced only for Gemini models.

I have been digging into this a bit. You can pass a Llama model directly into guidance's models.LlamaCpp and use it. However: the image object that guidance uses seems to be made to support VertexAI's image capabilities; I can't figure out how to pass {"type": "image_url", "image_url": {"url": data_uri }} to the LlamaCpp model and that's where I'm stuck.

I have been digging into this a bit. You can pass a Llama model directly into guidance's models.LlamaCpp and use it. However: the image object that guidance uses seems to be made to support VertexAI's image capabilities; I can't figure out how to pass {"type": "image_url", "image_url": {"url": data_uri }} to the LlamaCpp model and that's where I'm stuck.

Agreed. I have also been stuck exactly there. If I can figure out how the {"type": "text", "text": "some prompt"}} is converted in the guidance model, I think I can figure out how the image_url can be passed in a similar fashion.

@slundberg Could you please provide us with high-level pointers to start our dev efforts? Thx!


DBZ-1863 Centralizing coordinator shutdown

https://issues.redhat.com/browse/DBZ-1863
Small follow-up based on the changes to BaseSourceTask to handle the coordinator lifecycle in one place.

Kinda liking how we're moving to a truly unified model :)

@gunnarmorling Do we need an incubator follow-up issue?

Do we need an incubator follow-up issue?

Absolutely: https://github.com/debezium/debezium-incubator/pull/148 :)


BuildStatusReporter is no longer defined in GithubCoprBuildHandler

Fixes #349

Ah, the code has been moved to packit-service.