Has Anyone Ran Into db query error: pq: syntax error at or near ")" Errors?

Hi!
I do not consider myself knowledgeable when it comes to DB's, and a google search for this exact error did not yield anything useful.
All of my Grafana dashboards show the same error which is:
db query error: pq: syntax error at or near ")"
Looking at the logs from within obmp-sql, I see more detail:
023-04-07 19:17:08 2023-04-07 23:17:08.869 UTC [176] ERROR:  syntax error at or near ")" at character 90
2023-04-07 19:17:08 2023-04-07 23:17:08.869 UTC [176] STATEMENT:  select peername as __text, peer_hash_id as __value from v_peers where router_hash_id in ()
2023-04-07 19:17:09 2023-04-07 23:17:09.122 UTC [176] ERROR:  syntax error at or near ")" at character 302
2023-04-07 19:17:09 2023-04-07 23:17:09.122 UTC [176] STATEMENT:  SELECT
2023-04-07 19:17:09         time_bucket('60.000s',interval_time) as time,
2023-04-07 19:17:09        sum(updates) as updates,sum(withdraws) as withdraws
2023-04-07 19:17:09     FROM stats_chg_bypeer s
2023-04-07 19:17:09       JOIN v_peers p ON (s.peer_hash_id = p.peer_hash_id)
2023-04-07 19:17:09     WHERE interval_time BETWEEN '2023-04-06T23:17:09.536Z' AND '2023-04-07T23:17:09.536Z'
2023-04-07 19:17:09       AND s.peer_hash_id in ()
2023-04-07 19:17:09     GROUP BY time
2023-04-07 19:17:09     ORDER BY time ASC
2023-04-07 19:17:09
2023-04-07 19:17:09
2023-04-07 19:17:09 2023-04-07 23:17:09.136 UTC [176] ERROR:  syntax error at or near ")" at character 302
2023-04-07 19:17:09 2023-04-07 23:17:09.136 UTC [176] STATEMENT:  SELECT
2023-04-07 19:17:09         time_bucket('120.000s',interval_time) as time,
2023-04-07 19:17:09         sum(withdraws),
2023-04-07 19:17:09         p.routername as metric
2023-04-07 19:17:09       FROM stats_chg_bypeer l
2023-04-07 19:17:09       JOIN v_peers p ON (p.peer_hash_id = l.peer_hash_id)
2023-04-07 19:17:09       WHERE interval_time BETWEEN '2023-04-06T23:17:09.572Z' AND '2023-04-07T23:17:09.572Z'
2023-04-07 19:17:09          AND l.peer_hash_id in ()
2023-04-07 19:17:09       group by routername,time
2023-04-07 19:17:09       order by time
2023-04-07 19:17:09 2023-04-07 23:17:09.181 UTC [176] ERROR:  syntax error at or near ")" at character 298
2023-04-07 19:17:09 2023-04-07 23:17:09.181 UTC [176] STATEMENT:  SELECT
2023-04-07 19:17:09         time_bucket('900.000s',interval_time) as time,
2023-04-07 19:17:09         sum(updates),
2023-04-07 19:17:09         p.PeerName as metric
2023-04-07 19:17:09       FROM stats_chg_bypeer l
2023-04-07 19:17:09       JOIN v_peers p ON (p.peer_hash_id = l.peer_hash_id)
2023-04-07 19:17:09       WHERE interval_time BETWEEN '2023-04-06T23:17:09.584Z' AND '2023-04-07T23:17:09.584Z'
2023-04-07 19:17:09          AND l.peer_hash_id in ()
2023-04-07 19:17:09       GROUP BY peername,time
2023-04-07 19:17:09       order by time asc
2023-04-07 19:17:09
2023-04-07 19:17:09 2023-04-07 23:17:09.208 UTC [176] ERROR:  syntax error at or near ")" at character 251
2023-04-07 19:17:09 2023-04-07 23:17:09.208 UTC [176] STATEMENT:  SELECT as_name as "AS Name",d.* from (
2023-04-07 19:17:09       SELECT
2023-04-07 19:17:09           origin_as as asn,
2023-04-07 19:17:09           sum(updates) as "Updates"
2023-04-07 19:17:09         FROM stats_chg_byasn l
2023-04-07 19:17:09         WHERE interval_time BETWEEN '2023-04-06T23:17:09.603Z' AND '2023-04-07T23:17:09.603Z'
2023-04-07 19:17:09             AND peer_hash_id in ()
2023-04-07 19:17:09         GROUP BY asn
2023-04-07 19:17:09         order by "Updates" desc
2023-04-07 19:17:09       LIMIT 20
2023-04-07 19:17:09     ) d
2023-04-07 19:17:09     LEFT JOIN info_asn i ON (i.asn = d.asn)
2023-04-07 19:17:09
2023-04-07 19:17:09 2023-04-07 23:17:09.228 UTC [521] ERROR:  syntax error at or near ")" at character 300
2023-04-07 19:17:09 2023-04-07 23:17:09.228 UTC [521] STATEMENT:  SELECT
2023-04-07 19:17:09         time_bucket('900.000s',interval_time) as time,
2023-04-07 19:17:09         sum(updates),
2023-04-07 19:17:09         p.routername as metric
2023-04-07 19:17:09       FROM stats_chg_bypeer l
2023-04-07 19:17:09       JOIN v_peers p ON (p.peer_hash_id = l.peer_hash_id)
2023-04-07 19:17:09       WHERE interval_time BETWEEN '2023-04-06T23:17:09.556Z' AND '2023-04-07T23:17:09.556Z'
2023-04-07 19:17:09          AND l.peer_hash_id in ()
2023-04-07 19:17:09       GROUP BY routername,time
2023-04-07 19:17:09       order by time asc
2023-04-07 19:17:09
2023-04-07 19:17:09 2023-04-07 23:17:09.275 UTC [176] ERROR:  syntax error at or near ")" at character 240
2023-04-07 19:17:09 2023-04-07 23:17:09.275 UTC [176] STATEMENT:  SELECT as_name,d.* from (
2023-04-07 19:17:09       SELECT
2023-04-07 19:17:09           origin_as as asn,
2023-04-07 19:17:09           sum(withdraws) as "Withdraws"
2023-04-07 19:17:09         FROM stats_chg_byasn l
2023-04-07 19:17:09         WHERE interval_time BETWEEN '2023-04-06T23:17:09.61Z' AND '2023-04-07T23:17:09.61Z'
2023-04-07 19:17:09             AND peer_hash_id in ()
2023-04-07 19:17:09         GROUP BY asn
2023-04-07 19:17:09         order by "Withdraws" desc
2023-04-07 19:17:09       LIMIT 20
2023-04-07 19:17:09     ) d
2023-04-07 19:17:09     LEFT JOIN info_asn i ON (i.asn = d.asn)
2023-04-07 19:17:09
2023-04-07 19:17:09 2023-04-07 23:17:09.297 UTC [522] ERROR:  syntax error at or near ")" at character 300
2023-04-07 19:17:09 2023-04-07 23:17:09.297 UTC [522] STATEMENT:  SELECT
2023-04-07 19:17:09         time_bucket('120.000s',interval_time) as time,
2023-04-07 19:17:09         sum(withdraws),
2023-04-07 19:17:09         p.PeerName as metric
2023-04-07 19:17:09       FROM stats_chg_bypeer l
2023-04-07 19:17:09       JOIN v_peers p ON (p.peer_hash_id = l.peer_hash_id)
2023-04-07 19:17:09       WHERE interval_time BETWEEN '2023-04-06T23:17:09.593Z' AND '2023-04-07T23:17:09.593Z'
2023-04-07 19:17:09          AND l.peer_hash_id in ()
2023-04-07 19:17:09       group by peername,time
2023-04-07 19:17:09       order by time

Has anyone ran into a similar issue before?

Can you confirm if you have connected any BMP sources feeding your DB? I am not sure if this is the specific issue you are facing but I see that AND s.peer_hash_id in () shows an empty array/list. I think this would stem from the dashboard variables being empty.
You could look at the dashboard settings and see what queries grafana is using to build the variables


royalty dependency fix

fixed the dependency problem, it was the scrambler implant

Fixes #90  and #91


[EPIC] Authorization service for SAVE cloud

Need to create a service that will unify all authorisation logic: for backend, sandbox and other future tools.

@kgevorkyan as I can see - we already do it with api-gateway. Do you remember the problem?

The main logic was implemented long time ago, also we extract all our auth logic into the separate module auth-service,
However, now it's not really the service. Someday we planned to make the real service, to which all of our components will address to, when they need the auth.
It's kind of big task, and my investigations gave no valuable results, the are several approaches how we can do it, e.g. with help of dex, but there is too little amount of documentation, so it was postponed
Actually, I don't know, will we do it at all one day or not
It also could be connected with https://github.com/saveourtool/save-cloud/pull/1247 and https://github.com/saveourtool/save-cloud/pull/1238

Will be partially implemented by https://github.com/saveourtool/save-cloud/issues/2336


Sort history by "Date created": latest copied should be at the top

Hi,
In version 1.2.8, when the sort order is set to "Date created", we get the latest copied at the bottom. It would make more sense to have it at the top, as recently copied items are more likely to be pasted than older ones.
Could you either

reverse the sort,
or add a 3rd sort choice: "Reversed Date Created",
or add a distinct new option to chose the sort direction?

Thanks

Any comment? agree? disagree?

sort by Date created is ascending or descending?

I'm not sure how to understand ascending or descending in this context, but currently, with Date created, the most recent items get displayed at the bottom of the list. I'd be happy to have them instead at the top of the list.

I'm not sure how to understand ascending or descending in this context, but currently, with Date created, the most recent items get displayed at the bottom of the list. I'd be happy to have them instead at the top of the list.
Somehow related issue: if I set

Max clipboard history size to 30 items
Number of items place inline to 30 items too (because I don't want any items in a folder)
Then, after the first 30 copied items in the list, no further copied items is added to the list, whereas they should rotate, I suppose (oldest items should be discarded to allow new items to be added to the list).

The history is already rotated.

@ian4hu , thanks for the fix of #35 through #36. However this issue (#30) is not fixed: This issue is asking for a way to reverse the sort order when using sort by "Date Created". Quoting my initial request:

Could you either
* reverse the sort,
* or add a 3rd sort choice: "Reversed Date Created",
* or add a distinct new option to chose the sort direction?

Thanks

IMO, Sort by Date Created is a total different to Last Used, and it will make history sort by create time with natural order (the last created will on the bottom). So if there is a sort order like Reversed Date Created, what the different compare with last used?

Could you explain your situation about the Reversed sort? @afeblot

Ah, you're right. I think your change in #36 also fixed the behavior of the "Last used" ordering, which is now exactly as what I expect, whereas it really was not initially.
Thanks and sorry. 👍


fs.info() crash when fs is version-aware

When using a version-aware filesystem, fs.info() fails on every file (directories are fine).
For instance, the following test fails with a FileNotFoundError:
def test_info_version_aware(storage):
    fs = AzureBlobFileSystem(
        account_name=storage.account_name, connection_string=CONN_STR,
        version_aware=True, skip_instance_cache=True
    )
    file_info = fs.info("data/root/a/file.txt")

@rlamy Could you post the traceback, please?


Add option to specify output format

Is your feature request related to a problem? Please describe.
No, not related to a problem.
Describe the solution you'd like
The default output of the conversion is parquet.  While parquet is efficient, having the option to output CSV would be beneficial.  It would support quick checks of the conversion, as well as further analysis/manipulation without having to deal with a parquet file.
What I would like:  I would like to be able to specify the output format, e.g. -output-format csv to override the default format.
Describe alternatives you've considered
Converting parquet to CSV manually is certainly workable, but the tool should provide this option directly.

Hi @varunmittal91 - that is a consideration, especially when dealing with multi-GB CSV files.
Ideally, users would be presented with options on how they want the output generated, something like:
Select your CSV output option:

1 - A single CSV with all converted rows
2 - Multiple CSVs with up to 1,048,575 rows each (the current row limit for Microsoft Excel)
3 - Multiple CSVs with a user-specified number of rows per file

If the user selects option 3, they would be prompted to enter the number of rows they want per CSV.  These could also be passed as arguments, e.g. --csv-rows all or --csv-rows 1000.
There are tools that could be adapted for use to reduce coding efforts?
xsv: https://github.com/BurntSushi/xsv
csvkit: https://github.com/wireservice/csvkit

sorry I added #348 before I read this... multiple output options would be beneficial

Hey Folks, as a workaround I'm looking at using https://github.com/clemensv/avrotize as a second conversion step to get to AVRO format. fingers crossed...


What is the best way to merge 2 hnsw indices?

Hi   @Neoanarika ,
I have 2 indices hnsw1(with 100 vectors), hnsw2(with 100 vectors). Currently i take the union of vectors from both the indices(200 vectors) and create a new index hnsw3.
Not sure if this is better approach. I am looking some thing like, Instead of building hnsw3 from scratch, is there a way i can merge hnsw1 and hnsw2 in place?
Also what do you think will be better in terms of time complexity?
Building from scratch by taking union and creating new index OR merging approach.
Thanks in advance

Hi @vamshin currently there's no way to do this, but there may be a way in the future. It's on the internal feature list already.


Certificate missing on ubuntu

Hi, thanks for the great work but i have a problem, when i run coffee example.logTraffic.coffee without error and i close the terminal, i don't have any location named .http-mitm-proxy/certs/ca.pem
Where cai i found it?
Thanks

press CTRL+H to unhide the folder, and it'll be in the pokemon-go-mitm folder

thanks :smile:
have a good day


Canonical example - Visual Studio 2017 SQLite EF6 example without using App.Config

It would be good to have a canonical example that shows very simple implementation.  The idea behind EF6 is very simple and the examples should be presented that way. You don't teach poetry to 1st graders, so show the bone simple way to do things - no fancy constructors, subclassing, don't even use Using( ) constucts. etc etc. Just stone axe least amount of code possible simple. Step 1: Set the EF references + specific ones for your database. Step 2: Create some classes (whose properties have appropriate datatypes)  Step 3: Create a dbContext class that has the dbSets and that decides the CodeFirst behaviors. Step 3.5 the part that's elusive to me - create the connection string stuff.    Step 4 - run it :) ... This is not fully accurate and way to feature rich... but its at was somewhat close. https://www.codeproject.com/Articles/1158937/SQLite-with-Csharp-Net-and-Entity-Framework

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

ID: 7f431e6c-6c5f-9056-1486-99ba6b02c954
Version Independent ID: 37943f43-c279-f190-b134-48f54ead1e38
Content: Connection strings and models - EF6
Content Source: entity-framework/ef6/fundamentals/configuring/connection-strings.md
Product: entity-framework
GitHub Login: @divega
Microsoft Alias: divega

The SQLite provider does not support Code First migrations

Hi Erik,
Thank you for the fast reply!  For my particular needs: I have base set of tables /classes that I need to clone  from a template + I want some degree of “scriptability” afterwards.  I can accomplish what EF6 /codefirst was going to give me , I can handle with a bespoke SQL generator + .NET’s tricks for “auto-understanding” classes + Roslyn for the scripting.
Speaking for just for me, that’s the problem with the “crowdsourced” approach (e.g.StackOverflow etc) – I’d read some articles saying that you can’t do CodeFirst with SQLite – but I can’t count how many times I’ve gotten “not supported” stuff to work. There’s a ton of “noise/misunderstanding out there” – hence my request for the stone simple canonical implementation.
To my (self taught programmer’s mind), EF6 appears a simple concept that fit my needs perfectly, so I figured if its supported by one database – then anything preventing another is just a matter of something simple that I can bumble through – not something fundamentally wrong with the universe beyond my control.
I’m going to take your advice! Call it that I’ve learned enough for one week and am moving along with the approaches I know.
Again, thank you for the reply – I’ve been impressed about how rapidly folks at Microsoft are replying to these comments. I know that’s a thankless job.
Regards,
Thom
Thom C. Blackwell,  VP Technical Services
Boston Software Systems, Inc.
Phone:<PHONE_NUMBER> x807
Mobile:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
www.bostonsoftwaresystems.comhttp://www.bostonsoftwaresystems.com/
The information contained in this message may be confidential and legally protected under applicable law. The message is intended solely for the addressee(s). If you are not the intended recipient, you are hereby notified that any use, forwarding, dissemination, or reproduction of this message is strictly prohibited and may be unlawful. If you are not the intended recipient, please contact the sender by return e-mail and destroy all copies of the original message.
From: Erik Ejlskov Jensen<EMAIL_ADDRESS>Sent: Wednesday, July 11, 2018 5:11 PM
To: aspnet/EntityFramework.Docs<EMAIL_ADDRESS>Cc: Thom Blackwell<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [aspnet/EntityFramework.Docs] Canonical example - Visual Studio 2017 SQLite EF6 example without using App.Config (#834)
The SQLite provider does not support Code First migrations
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/aspnet/EntityFramework.Docs/issues/834#issuecomment-404310456, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGV4z4LuEEmhtPcWlwB18DpWWuiqe4yfks5uFmnegaJpZM4VL2V8.

Triage: we don't intend to update the documentation to include SQLite for EF6. However, we will, as part of other work, attempt to make some of the walkthroughs easier to follow.


“Ambiguous use of 'cornerRadius'” in Xcode 11, Swift 4.2

Ambiguous use of 'cornerRadius'

The requirements state Swift 5.0+ on the README

hi  mac-gallagher , this issue. (“Ambiguous use of 'cornerRadius'”) in swift 5.0 also
see that : https://github.com/AssistoLab/DropDown/issues/192
the same issue , can u update your pod ?

@AMM001  Please create a new issue using the issue template and provide an example code snippet and I will look into it


[Feature] Add a timeout parameter to the subscribe attribute command in YAML

Feature description
I was tried to add a timeout to the subscribeAttribute command in tests yaml, but it doesn't work.
What we tried to achieve is

SMCO.S.A0001(SmokeState)
After about 30 seconds, TH reads COState attribute from DUT
Verify that SmokeState attribute value

- label: "TH subscribes to SmokeState attribute from DUT"
  PICS: SMCO.S.A0001
  endpoint: 1
  command: "subscribeAttribute"
  attribute: "SmokeState"
  minInterval: 3
  maxInterval: 60
  timeout: 300 # Not working
  response:
      value: 0
      constraints:
          type: enum8

And there is also no timeout parameter in the function.
CHIP_ERROR SubscribeEvent(const char * identity, chip::EndpointId endpointId, chip::ClusterId clusterId, chip::EventId eventId,
                          uint16_t minInterval, uint16_t maxInterval, bool fabricFiltered,
                          const chip::Optional<chip::EventNumber> & eventNumber, const chip::Optional<bool> & keepSubscriptions,
                          const chip::Optional<bool> & autoResubscribe);

It seems that a timeout parameter needs to be added to set the subscription time?
Platform
all
Platform Version(s)
No response
Anything else?
No response

What does "subscription time" mean?  A "subscribeAttribute" command completes when a subscription is established.  If you then need to wait for some reason, there's a "WaitForMs" to do that.
@ericzijian1994 Please reopen if you have a clear description of what this timeout parameter would do.


dispose() method not working properly

Hi Guys,
in Videos-vast-vpaid plugin and after the video is finishing to play and the player.dispose() method is executed, i always got this error-message
Error:
Uncaught TypeError: Cannot read property 'vdataXXXXX of null in Chrome
TypeError: el is null in Firefox
I also saw in this Issue that there is a "solution" with  the setTimeout() function and it is working, but is there any Other Solution without the setTimeout ?
Best
versions
Videojs-vast-vpaid v.1.0.0
Videojs 5.0.0 rc2
browsers
Chrome/48.0.2564.95
FF 44

I have the same issue. Has anyone an idea what is causing this?

@derdeka @xcuben can you share some url for debugging?

hi @Fetz
Here is a test Gist:
https://gist.github.com/xcuben/cbf7f971a9fe0ea0a745e6ef28d6dc8c#file-index-html
You might need a black.mp4 and VPAIDFlash.swf .
I unfortunately couldn't upload them
Here is also a screenshot of the Console Error :
Thanks.

+1


How can we execute parallel test in current CI/CD pipeline(Jenkins) on server?

We need to execute our test in 5 browsers parallely.
1.Win+Chrome
2.Win+Mozilla
3.Win10+Edge
4.Mac+Safali
5.Mac+Chrome
P.s : We have done opentest jenkins  integration on local machine for one browser.

If your version of Jenkins supports it, you should be able to use parallel stages for this. You can run a separate test session in each parallel stage, one per each OS/browser combination.


Access token issuance diagram suggested correction

Referring to https://learn.microsoft.com/en-gb/azure/active-directory/conditional-access/plan-conditional-access?source=docs#access-token-issuance, the flow chart down arrow for 3rd box labelled "access control BLOCK conditions satisfied" should be modified to "YES" and the right arrow should be modified to "NO"

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

ID: 1e4eb266-d556-29dd-8c22-1b0221a048a7
Version Independent ID: 3b4c4a89-85a8-5948-3ad2-efc7459945c2
Content: Plan an Azure Active Directory Conditional Access deployment - Microsoft Entra
Content Source: articles/active-directory/conditional-access/plan-conditional-access.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @Gargi-Sinha
Microsoft Alias: gasinh

@jamesmkfoo23
Thanks for your feedback! We will investigate and update as appropriate.

@jamesmkfoo23
Thanks for your feedback! We have assigned the issue to author who will provide further updates.

Hi @YashikaTyagii - Please close this request the diagram is updated.

@Gargi-Sinha
Thanks for the update. I have closed the issue.


[Cherry pick] SDK Adapter components: Consume passed in IConfiguration instead of the one registered in DI (#5582)

Cherry pick of: https://github.com/microsoft/botbuilder-dotnet/pull/5582

Pull Request Test Coverage Report for Build 245255

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.009%) to 78.658%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Facebook/FacebookAdapterBotComponent.cs
3
0%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Slack/SlackAdapterBotComponent.cs
3
0%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Twilio/TwilioAdapterBotComponent.cs
3
0%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Webex/WebexAdapterBotComponent.cs
3
0%

Totals

Change from base Build 243587:
0.009%

Covered Lines:
18067

Relevant Lines:
22969

💛  - Coveralls


function(prepHumanModelForftINIT)

Hello, sorry to ask a running problem, when I run this function, will always stay here, stay for 16 h without the next step, the tutorial is written on 2~3h, when I look at the cpu occupancy, found that it is not like computing, how to solve this problem? It will stop at the detection of this reaction
PASS: [ER] Ubiquinol-to-ATP
PASS: [SU] Beta oxidation of saturated FA
PASS: [SU] Beta oxidation of long-chain FA
......

I suspect it may be the solver's problem, so I suggest using Gurobi. Some people have encountered similar problems, you can refer to #826.


Allow maxTime to be set greater than 24 hours

I was attempting to set minTime to 7 and maxTime to 27, for a radio schedule that starts at 7am and ends at 3am. There were issues with negative array indices, and I believe this is a reasonable fix.

hey,
good idea. but the events are not displayed correctly and the "selection" does not work past the 24h limit...

Yeah, it turns out it only works under specific circumstances. Darn. I'll be back!


There is an exception if the admin is trying to add transactions for any invoice.

Bagisto Version: Master
Steps To reproduce--
1. Log in as an admin.
2. Click on transaction inside the sales tab.
3.  Add a transaction for any invoice.
Expected result: There should not be an exception if the admin is trying to add transactions for any invoice.
Actual result: There is an exception if the admin is trying to add transactions for any invoice. Please find the screen recording  Here
Please find the screenshot here.

@jitendra-webkul
Now, this issue is fixed And I have attached a screenshot Here, for future reference.


Can't get it to work

I am new to using nodes. I'm currently working with homebridge-Xbox-one. Xbox-on is the only dependency. I've added Xbox one to my accessories in the config file and I have installed the Xbox-on  using npm install -g Xbox. Not sure if I'm missing any config files or if I need to do additional steps. However whenever I request to have my Xbox turn on, I see the request through terminal but Xbox never turns on. Any help would be greatly appreciated. Thank you!!!

Assuming that you have the correct IP address and Live ID, it must be mentioned that Xbox WoL reliability is... just plain awful.  Even their own smartglass app doesn't really work, so I'm assuming this is probably for the same reasons that nothing else made by Microsoft seems to work correctly.
Only thing I can suggest is to add "tries" to your accessory config and set it to something higher than 5 (the default).  It will just blindly send more dgrams to the Xbox, and hopefully one of them will work.  The other thing you can try is to make sure that your Xbox is using a wired ethernet connection, if possible.  Final thing to confirm is that you're using the "instant on" setting on the Xbox, as it won't be listening to the dgrams if the xbox is fully off.
I'll confirm mine still works and re-open if it doesn't, thanks for the information :-)

Thank you so much for getting back to me. Now pardon my ignorance (again I'm new to this), but where exactly do I add "tries" or change the default. Thank you in advance.
Sent from my iPhone.

On Jun 1, 2016, at 10:22 PM, Adam Robertson<EMAIL_ADDRESS>wrote:
Assuming that you have the correct IP address and Live ID, it must be mentioned that Xbox WoL reliability is... just plain awful. Even their own smartglass app doesn't really work, so I'm assuming this is probably for the same reasons that nothing else made by Microsoft seems to work correctly.
Only thing I can suggest is to add "tries" to your accessory config and set it to something higher than 5 (the default). It will just blindly send more dgrams to the Xbox, and hopefully one of them will work. The other thing you can try is to make sure that your Xbox is using a wired ethernet connection, if possible. Final thing to confirm is that you're using the "instant on" setting on the Xbox, as it won't be listening to the dgrams if the xbox is fully off.
I'll confirm mine still works and re-open if it doesn't, thanks for the information :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

In your homebridge config.json file:
...
"accessories": [
    {
      "accessory": "Xbox",
      "name": "Xbox",
      "ipAddress": "<your XBox IP address here>",
      "liveId": "<your live ID here>",
      "tries": 10
    }
]
...

I've confirmed it still working on my system, but I had to hard reset it first by holding the power button, then boot it up and down normally.

Thank you so much!! You've been a great help. I'll try that now. Tomorrow is the Nest -- wish me luck :)
Sent from my iPhone.

On Jun 1, 2016, at 10:39 PM, Adam Robertson<EMAIL_ADDRESS>wrote:
In your homebridge config.json file:
...
"accessories": [
{
"accessory": "Xbox",
"name": "Xbox",
"ipAddress": "",
"liveId": "",
"tries": 10
}
]
...
I've confirmed it still working on my system, but I had to hard reset it first by holding the power button, then boot it up and down normally.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Good luck, Nest should be easier :-)


Use stdlib csv module to write csv

I noticed while using pandas that data with commas in the fields wasn't getting escaped properly when written to CSV (data in biology often has bits of text as well as numbers). This causes the data to spread out over extra columns. This commit uses the csv module from the standard library to write out CSV 'properly'.
I've also added a simple test for this case.

Cool. I was actually made aware of this a while ago. I'm sure performance is better, too.

I wouldn't expect a big performance difference, but it may be a bit faster, since I think csv.writer is implemented in C.


Update CODE_OF_CONDUCT.md

Thanks!  Our code-of-conduct is a standard one: https://www.contributor-covenant.org/version/1/4/code-of-conduct/.  I'm afraid PRs to change it will need to include a substantial description.
Could you please re-open a new PR and provide such a description, to provide the reasoning behind this change?  Alternatively, you could make a more substantial impact by changing that the contributor covenant code-of-conduct upstream -- we and many others will follow a change that has passed many eyeballs.


Column "Calendar Month" is sorted by "Calendar QuarterNumber" rather than "Calendar MonthNumber"

Fixed in 1.11 just released.


Either disable clicking on date or override it in month view

Is there a way to either disable clicking on date or override it in month view? Right now it navigates to Date view when I click the date within a month view.
I have removed date from views to prevent navigation, but still the clicking on date in a month is still enabled. I want to do some other operations on clicking a date in a month view, like an alerts or sending the events list of that date to a different component.
By using "onSelectSlot", I am able to do a similar activity for the selected dates. But "onSelectSlot" is not called when I click on the date in the month view, it is only called when I click at any other place within the cell of a day.

FullCalendar supports date click: https://fullcalendar.io/docs/dateClick. I am looking for this functionality in react-big-calendar.

Hi avrpaduru,
One way I've disabled this feature in month view is by setting the components prop for month dateHeader to a custom date header:
      <Calendar
           components={{
                month: {
                    dateHeader: this.myCustomMonthDateHeader,
                     },
           }}
       />

This component will pass in {label, date} to this.myCustomMonthDateHeader where label is the day number and date is the date object, based on your localizer. This function will be called for each day "box" in month view.
Let me know if this isn't what you are looking for.

Can you elaborate more on this ?
Can you share the snippet of the custom component? I am not able to create a custom component which can help me resolve the above mentioned problem.


Release 0.3.2

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I'm marginally concerned about old builds of QCPortal being pulled down (https://github.com/openforcefield/openff-qcsubmit/pull/206#issuecomment-1307510452) but until users report issues I don't want to add a constraint into the recipe. I think most users of QCSubmit will probably be responsible enough with their environments and/or have other packages with constraints that prevent ~2019 packages from sneaking in.


✨ add getBaladyiatsForDaira

I added getBaladyiatsForDaira feature
Fixes #42
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Updated the dataset

Checklist:

[x] I checked that there's no dataset update (can be done by running npm run update-dataset)
[x] npm test passes on my machine
[x] npm run lint passes on my machine

Codecov Report

Merging #48 into develop will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop       #48   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        16    +1
  Lines          111       130   +19
  Branches        22        31    +9
=========================================
+ Hits           111       130   +19

Impacted Files
Coverage Δ

src/api/getBaladyiatsForDaira.js
100.00% <100.00%> (ø)

src/index.js
100.00% <100.00%> (ø)

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 a03cea2...5cb2ed1. Read the comment docs.


[Diagnostics] Improve diagnostics when passing async to a sync parameter

If async effect has been inferred from the body of the closure,
let's find out the first occurrence of async node and point it out
to make it clear why closure is async.
Resolves: rdar://70610141

@swift-ci please smoke test

@swift-ci please smoke test Windows platform

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform


Sweetalert closes immediately when Enter key is used

I have an event that triggers when the Enter key is pressed, that triggers the sweet alert to come up. But this closes immediately after opening. But when the same function runs on a mouse click event, then Sweetalert pop up remains open. I think what's happening is, on the Enter key event it immediately detects that the Enter key's been pressed and closes the popup. How can I stop this?
Thanks.

Probably you want to take a look on the allowEnterKey option.
Otherwise, for bug reports, please include a jsfiddle in order for us to be able to reproduce the issue.

JSFiddle template: https://jsfiddle.net/limon/ad3quksn/

See Contributing.md


Corrected a incorrect parameter

Changed to Username from the First Name. As the placeholder should contain the Username.

#245

+1


Update dependabot config

This is a meaningless change to the dependabot configuration to try to force dependabot updates to happen.

It worked :)


Hi

Where could i see the prediction of credit risk in your project ?
thanks

Please follow the instructions.


chore(v2): bump clean-css to 5.1.1

Motivation
Should fix #4319
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
See #4319
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

great! thanks


ISPN-14974 OpenTelemetry Tracing runtime kinds span filtering

https://issues.redhat.com/browse/ISPN-14974

I asked several reviews since the contributions affect different areas of the projects (clustering, x-site, persistence, security, documentation, ...)

Any better implementation ideas are more than welcome!

@kittylyst I don't know if want to have a look...

@fax4ever I don't understand why we have CacheTelemetry, CoreTelemetry,  GlobalTelemetry and TelemetryService.
Wouldn't be simpler just having 2 interfaces?
interface InfinispanTelemetryService {
  InfinispanSpan requestStart(/*args*/);
}

and
interface InfinispanSpan {
  AutoCloseable makeCurrent();
  void complete();
  void completeExceptionally(Throwable);
}

@fax4ever I don't understand why we have CacheTelemetry, CoreTelemetry, GlobalTelemetry and TelemetryService.
Wouldn't be simpler just having 2 interfaces?
interface InfinispanTelemetryService {
  InfinispanSpan requestStart(/*args*/);
}

and
interface InfinispanSpan {
  AutoCloseable makeCurrent();
  void complete();
  void completeExceptionally(Throwable);
}

CacheTelemetry and GlobalTelemetry are not interfaces I think. The only interfaces we have I think are CoreTelemetry and  TelemetryService,  where the latter extends the former, so at the end there is only one interface that is TelemetryService.
The subinterface CoreTelemetry allows us to expose to the core Infinispan module only the operations that do make sense to be used from there, for instance we don't want recover the tracing context from an external process from the core module, but we want to do that only form modules that address client requests, such as the server HotRod or the server REST.
I suspect that simplifying it more would result in complicating something else.

@pruivo I was thinking that we can call TelemetryService the base interface instead of CoreTelemetry and ServerTelemetryService the one extending the first one used by the server modules (HotRod / Rest)

that way I see it, it simplifies everything since HotRodTelemetryService and company can be removed.

that way I see it, it simplifies everything since HotRodTelemetryService and company can be removed.

I'm doing the split of this contribution rearranging/rebasing the commits so that it will be easier to review and fix.
It is an hard work, since we have a lot of changes.
But when I finish I think that we can address better the revision.

@fax4ever did a small PoC with the API that I'm suggesting, in case I wasn't clear in the comment. Didn't test it
https://github.com/pruivo/infinispan/commit/dcd45ca828807d712d5d1b3477723e4ee3d1e395

@fax4ever did a small PoC with the API that I'm suggesting, in case I wasn't clear in the comment. Didn't test it pruivo@dcd45ca

I'm not sure with this change the propagation of the context from the client to the server still works. I don't see the code to address it.

pruivo@dcd45ca#diff-85a18c3339fab1033d034fd97b63c4100a864949f7d5580072c9f9276e99e14aR49

you're right... almost finish with the massive rebase... after that I'll check you idea. Thanks

Thanks @pruivo I applied all the changes / great suggestions here I think, with the only exception of decorating the  Request class as TracingRequest. I think that Request has too many methods. Moreover, we should also implement Callable<Void>, that will add more methods to implement.

Rebased :P

I pushed a first change that is the fact to add tracing to the PersistenceManagerImpl, extending it instead of changing its code. We could improve even further the modularizabilty of the solution implementing PersistenceManagerTraced as an extension of DelegatingPersistenceManager, like PassivationPersistenceManager, but I don't know if it does make sense.... @wburns I don't know what you think of it...

@jabolina @wburns thanks for the reviews, I'm addressing the other comments now

@jabolina @wburns thanks again for the reviews. I should have applied all the nice ideas from you comments here. Looking forward to new feedback of yours.

according the CI I still need to fix the native run:
17:18:50      MethodSource [className = 'org.infinispan.graalvm.EmbeddedTest', methodName = 'xmlConfigTest', methodParameterTypes = '']
[631](https://ci.infinispan.org/job/Infinispan/job/PR-11308/51/pipeline-console/?start-byte=31294095&selected-node=59#log-631)
17:18:50      => org.infinispan.commons.CacheConfigurationException: Unable to instantiate class 'org.infinispan.remoting.transport.jgroups.JGroupTracedTransport'
[632](https://ci.infinispan.org/job/Infinispan/job/PR-11308/51/pipeline-console/?start-byte=31294095&selected-node=59#log-632)
17:18:50         org.infinispan.commons.util.Util.getInstance(Util.java:270)

@wburns thank you again for the review. I pushed some changes. Still need to introduce the request factory, I'll do that in a different push.

Finally I think I have addressed all the nice ideas for the reviews. If there are no further improvements to be done, let's merge it. So that we can finally announce these new tracing features on the next release. Thank you again to all reviewers @pruivo @wburns and @jabolina.

Mutating "list" attributes at runtime also requires changes included in https://github.com/infinispan/infinispan/pull/11726

@tristantarrant with the last commit we should have addressed the configuration change to expose a list instead of a series of booleans.
I will address https://issues.redhat.com/browse/ISPN-15569 in a different PR if you agree, since the issue is not caused by the changes of this PR, but I think it still exists on the current main.
I really would like to merge it as soon as we can and address other further possible improvements in some follow ups. I hope you agree with me :P

I can be convinced if we want to have global tracing enabled by default, but I think we really need to also confirm the impact is negligible as well.

Thank @wburns for the review. I think that tracing is disabled by default. In order to enable tracing you need to explicit configure the collector-endpoint. Going to apply the changes. Thanks

Thanks @wburns and @jabolina for the review

If there are no more comments, it would be great to merge it soon, in order to have this nice feature for Infinispan 15.0.0.Final. I count on your collaborations ;)

With the last push I addressed the last comment and I updated also the documentation reflecting the change of using list of categories instead of booleans (as suggested by @tristantarrant)

@pruivo thanks for the review, I addressed the changes
and in general thanks to @pruivo, @wburns, @tristantarrant  and @jabolina for the great ideas provided here!
shall we merge it? :)

merged! thanks @fax4ever !

Thank you @pruivo


Enable TCP keepalive

By default on Linux, TCP connections in the ESTABLISHED state do not get reaped for 5 days:
$ sysctl net.netfilter.nf_conntrack_tcp_timeout_established
net.netfilter.nf_conntrack_tcp_timeout_established = 43200

This means that if a FIN or RST packet is not received, for example if a user closes their laptop instead of shutting down their browser cleanly, the server will keep blocking on the fd for a very long time, resulting in what appears to be a slow fd leak.
The solution is to enable TCP keepalive on the socket which will tell the kernel ping the client periodically and if no response is received, will move the connection into the CLOSE_WAIT state, freeing the fd and server resources that were serving it.
The creator of the socket has to opt-in to this. By default net/http does this when listening, but there are a variety of places where we don't rely on net/http to set up the socket. This patch wraps the listener in each of those cases so that keepalive is enabled.
Closes #2684

LGTM


Add ability to use VictoriaMetrics as long term remote storage

It would be great to be able to configure VictoriaMetrics as long-term remote storage for prometheus-boshrelease. VictoriaMetrics has the following features:

It is easy to configure and operate - it is just a single binary with small configs.
It provides better compression ratio for the data comparing to other remote storage solutions. This means more data may be stored on the limited disk space.
It requires less RAM and CPU comparing to competitors. This reduces hardware costs and improves query speed.
It supports PromQL, so it can be used as a drop-in replacement for Prometheus datasource in Grafana.
It can collect data from multiple Prometheus instances, so later the data may be queried at once (aka global querying view).

Read more info about VictoriaMetrics at this page.

Hi @valyala ! It's possible to configure external storage for prometheus-boshrelease. Both remote_read and remote_write properties could be configured for deployments.
https://github.com/bosh-prometheus/prometheus-boshrelease/blob/774725ecd80594602c4b50595ee9b07be3f5487e/jobs/prometheus2/spec#L50-L53

Great! The closing the issue.


Update to draft-04

Don't merge until draft update is complete in the datatracker.

Don't merge until draft update is complete in the datatracker.

Pull Request Test Coverage Report for Build 499328962

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.752%

Totals

Change from base Build 409130151:
0.0%

Covered Lines:
378

Relevant Lines:
499

💛  - Coveralls

Pull Request Test Coverage Report for Build 499328962

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.752%

Totals

Change from base Build 409130151:
0.0%

Covered Lines:
378

Relevant Lines:
499

💛  - Coveralls


Unable to connect via slack

Describe the bug
The application fails to get authorized by slack and throws the error 'Please specify client_id' even when the `clienID' key is provided
To Reproduce
Steps to reproduce the behavior:

install dependencies
fill the environment files as required
run the server and client separately
try to authorize slack and see the error

Expected behavior
I expected to move past the authorization step to the app home page
Screenshots

Desktop (please complete the following information):

OS: Ubuntu
Browser: Chrome
Version: 19.04

Additional context
slack requires the client_id key, passport-slack , the third party we're using requires clientID key. Don't know where the problem may lie.

did you create a slack user & app?
and if you did, did you make sure that env variables are loaded correctly?

did you create a slack user & app?
and if you did, did you make sure that env variables are loaded correctly?

I created a slack app ( along with a new workspace ) and used the env variables they provided, they load correctly. Not sure about the slack user

I created a slack app ( along with a new workspace ) and used the env variables they provided, they load correctly. Not sure about the slack user

I created a slack app ( along with a new workspace ) and used the env variables they provided, they load correctly. Not sure about the slack user

Try to debug config/passport.ts and see if passport indeed does not get the clientid env variable

I created a slack app ( along with a new workspace ) and used the env variables they provided, they load correctly. Not sure about the slack user

Try to debug config/passport.ts and see if passport indeed does not get the clientid env variable


Bug:  Track Metadata in music library does not get updated

Describe the Bug
Track Metadata does not actually get updated.
The Log claims that it was done, but it's not reflected on plex. Running PMM again has the exact same result, which means that PMM also gets the original values back when checking on the next run.
Relevant Config
metadata:
  "Linkin Park":
    country: "United States of America"
    album_sorting: newest
    albums:
      "Hybrid Theory":
        originally_available: "2000-10-24"
        tracks:
          1:
            rating: 5
            original_artist: "Test"
          "One Step Closer":
            rating: 5
            original_artist: "Test"
      "Meteora":
        originally_available: "2003-03-25"
        album_sorting: newest
        tracks:
          9:
            rating: 5
            original_artist: "Test"
          "Numb":
            rating: 5
            original_artist: "Test"

Plex Meta Manager Info
Version: 1.15.0
Link to logs (required)
https://gist.github.com/CherryKitten/2b518f3d268ad781e0087f7335ed8c1d
Those are logs for two subsequent runs with the same config, showing that the metadata hasn't been updated correctly and is being tried again.

also disc 2 tracks are now searchable using title


Change the analytics base data layer ahead of move from beta to live

Note: The term 'data layer' comes from how the analytics team would refer to the 'nhs:web' value

Closing and recreating to trigger rebuild on Travis


WIP: Migrate Workspace delegate warnings to diagnostics

@aciidb0mb3r I've done all but one warning. I've left it because I'm not 100% sure when and how it happens so can't find a good name for it. Can you elaborate on it?
https://github.com/apple/swift-package-manager/pull/1148/files#diff-16a17153d6f335742f5d4039edf8be5eL900
Thanks!

It is when we do dependency resolution and a package is removed but is still in edited mode. I am fine with having a FIXME there because we need to rework that part with the new proposal

This is great, thanks! We will need to update tests I think.

@aciidb0mb3r Yep, my next commit will fix the tests. But I think we may be missing a few tests actually. To be honest, I think we may even be missing a few tests for some errors too.

Oh? It will be nice to get tests for them (considering we won't lose them with the upcoming proposal)

@aciidb0mb3r Ready for review :)

LGTM, thanks!

@swift-ci please smoke test

@swift-ci please smoke test


add user authentication

_config.ts

export const AUTH_DISABLED = true; // default is false

export const AUTH_ROLE_FIELD = 'userRole'; // default is 'role'

export const AUTH_USER_MODEL = 'AdminUser'; // default is 'User'

// optional. adds additional encryption to jwt
export const AUTH_ENCRYPT = (data: string) => `${data}_`;

// optional. adds additional decryption to jwt
export const AUTH_DECRYPT = (data: string) => data.slice(0, -1);

handler example
// user roles that are authorized to use this api
export const ROLES = ['ADMIN'];

export const handler = async () => {}

I'm assuming auth will only be used on the gateway -> user service
So the gw will pass the authorization header to user service, user service will verify the token, gw will add user data to the context, context is shared between services.
On create user, user service will create the user - handles password hashing, adding client token, and basic verification.
User service has the user schema -> user schema implements password hashing.
On the handler level, this can be verified on the miniserver framework via context.user.role and export const ROLES.

user service needs jwt and bcryptjs.


Error in step1 finetuning decapoda-research/llama-7b-hf

Hi,
I can perform finetuning on facebook/opt-1b model. But when I change the model to decapoda-research/llama-7b-hf, it runs into this error:
Loading extension module utils...
Time to load utils op: 0.8032350540161133 seconds
***** Running training *****
***** Evaluating perplexity, Epoch 0/2 *****
ppl: 1034.3475341796875
Beginning of Epoch 1/2, Total Micro Batches 7360
Traceback (most recent call last):
File "/scratch/xy1234/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main_llama.py", line 348, in
Traceback (most recent call last):
File "/scratch/xy1234/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main_llama.py", line 348, in
main()
File "/scratch/xy1234/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main_llama.py", line 321, in main
model.backward(loss)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
main()
File "/scratch/xy1234/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main_llama.py", line 321, in main
model.backward(loss)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/utils/nvtx.py", line 15, in wrapped_fn
ret_val = func(*args, **kwargs)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 1807, in backward
ret_val = func(*args, **kwargs)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 1807, in backward
self.optimizer.backward(loss, retain_graph=retain_graph)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/runtime/fp16/fused_optimizer.py", line 353, in backward
self.optimizer.backward(loss, retain_graph=retain_graph)
File "/scratch/xy1234/miniconda3/envs/ds/lib/python3.8/site-packages/deepspeed/runtime/fp16/fused_optimizer.py", line 353, in backward
scaled_loss.backward(create_graph=create_graph, retain_graph=retain_graph)
File "/home/xy1234/.local/lib/python3.8/site-packages/torch/_tensor.py", line 363, in backward
scaled_loss.backward(create_graph=create_graph, retain_graph=retain_graph)
File "/home/xy1234/.local/lib/python3.8/site-packages/torch/_tensor.py", line 363, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/xy1234/.local/lib/python3.8/site-packages/torch/autograd/init.py", line 173, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/xy1234/.local/lib/python3.8/site-packages/torch/autograd/init.py", line 173, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn
It looks like the loss node does not have gradient function attached during backward().  However, the model has lora parameters and it shouldn't have such behavior. At least, I can finetune the opt-1b model with a similar script.
The command I used was (I modified main.py such that it loads the llama model correctly):
export CMD="
pwd/main_llama.py
--data_path Dahoas/rm-static Dahoas/full-hh-rlhf Dahoas/synthetic-instruct-gptj-pairwise yitingxie/rlhf-reward-datasets
--data_split 2,4,4
--model_name_or_path ./decapoda-research/llama-7b-hf-local
--per_device_train_batch_size 4 \
--per_device_eval_batch_size 4 \
--max_seq_len 512
--learning_rate 9.65e-6
--weight_decay 0.1
--num_train_epochs 2 \
--gradient_accumulation_steps 4 \
--lr_scheduler_type cosine
--num_warmup_steps 0 \
--seed 1234
--zero_stage 0 \
--lora_dim 128
--only_optimize_lora
--lora_module_name decoder.layers.
--deepspeed
--output_dir $OUTPUT
"
Any clue on that? Thank you.
YC

save error

same error

Since PandaLLM (https://github.com/dandelionsllm/pandallm) is using DeepSpeed to fine-tuning LLaMA, maybe you can refer to this project to see how it fix this issue.

I just switched to this repo: https://github.com/Facico/Chinese-Vicuna.

Hi @yctam,
Can you please try running with the huggyllama/llama-7b model?
We've been able to run it in our text-generation inference examples with this command:

deepspeed --num_gpus 1 inference-test.py --name huggyllama/llama-7b --use_kernel --ds_inference

Interested to see if it'll work for you during Step 1.
Thanks,
Lev


Notebook test clearance and fixing notebooks.

Hello everyone,
This PR includes notebook test conversion to pytest form, so that we could turn it on/off using pytest -k option. We have managed to merge unresolved conflicts in some notebooks, I fixed it too.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #604       +/-   ##
==========================================
- Coverage   98.37%   0.33%   -98.04%
==========================================
  Files          45      45
  Lines        3570    3570
==========================================
- Hits         3512      12     -3500
- Misses         58    3558     +3500

Impacted Files
Coverage Δ

gpflow/test_util.py
0% <0%> (-100%)
:arrow_down:

gpflow/mean_functions.py
0% <0%> (-100%)
:arrow_down:

gpflow/core/base.py
0% <0%> (-100%)
:arrow_down:

gpflow/models/sgpr.py
0% <0%> (-100%)
:arrow_down:

gpflow/models/gpr.py
0% <0%> (-100%)
:arrow_down:

gpflow/densities.py
0% <0%> (-100%)
:arrow_down:

gpflow/params/__init__.py
0% <0%> (-100%)
:arrow_down:

gpflow/features.py
0% <0%> (-100%)
:arrow_down:

gpflow/kullback_leiblers.py
0% <0%> (-100%)
:arrow_down:

gpflow/core/__init__.py
0% <0%> (-100%)
:arrow_down:

... and 34 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 140a53c...7ba6ece. Read the comment docs.


Enable CUDA Target For Virutal Machine Run with Dynamic Input

Currently, when we want to compile and execute a network with dynamic input, only using llvm target can work as expected.
If the target is as CUDA, the compilation part works fine, but the run part fails. The reason why this happens is that the shape functions dealing with dynamic inputs are normally built on target host like llvm, while the output tensors of shape functions need to locate on gpu target. Therefore, in the PR, I try to fix this bug by creating temporary output tensors on cpu for results of shape functions and coping them to the output tensors created by the vm located on gpu.

@zhiics

Let's not do this because it cannot solve the problem. Dynamic shape support on GPU needs heterogeneous execution. We have worked on it and will refine the code and upstream when we have some cycles.

@zhiics @kevinthesun I am not trying to support heterogeneous execution in this PR. What I want is to make a model with operators all on gpu and dynamic input can compile and execute as expected. Such as this network:
debug_graph = tf.Graph()
with debug_graph.as_default():
    input_1 = tf.placeholder(dtype=tf.int32, shape=[None], name='input_1')
    result = tf.nn.relu(input_1, name='result')
target = "cuda"
context = tvm.gpu()

After the compilation of the above network, the shape function for relu is located on cpu but the output tensor for the shape function is located on gpu, which causes the Argument arg1.device_type has an unsatisfied constraint error. By creating a temporary cpu tensor for outputs of shape function and then copying back to the tensor on gpu, the compilation and execution of the above network can currently work.
Besides, is it reasonable to deal with shape functions separately, like adding a Opcode::InvokeShapeFunc or something else? Since the shape function seems to always locate on cpu.
I am highly looking forward to the heterogeneous compilation and execution of the virtual machine, which will make my current work much easier. If there is anything I can help, just let me know. Thanks a lot.

The way you currently do is more like a hack to make the example work. We need to more systematically consider different instructions/ir nodes. ShapeFuncs are always on CPU and kernels are on GPU. But there are also more than that. For example, we will introduce some instructions in IR to simplify VM runtime, i.e. shapeof, etc. These instructions all need to be considered. We are currently working on some higher priority tasks. After that, I will work on upstreaming the code.
So Let's close this for now.


[Table] Project Setup 1

Thanks for contribution! Please go through following checklist before sending PR.
PR Branch Destination

For Azurite V3, please send PR to dev branch.
For legacy Azurite V2, please send PR to legacy-dev branch.

Always Add Test Cases
Make sure test cases are added to cover the code change.
Development Guideline
Please go to CONTRIBUTION.md for steps about setting up development environment and recommended Visual Studio Code extensions.

CodeFlow for GitHub does not download executable files. If you trust this file and would like to run it, please download it yourself through GitHub.

Remove this file.

Refers to: src/.DS_Store:1 in 860455e. [](commit_id = 860455ee2ec7e1ae74e6c598a08a3cfa3018206f, deletion_comment = False)


UICONSET-206: ECS | Eureka | Request to central tenant instead of member when compare users in "Consortium manager"

https://folio-org.atlassian.net/browse/UICONSET-206
Here we changed the logic of fetching roles for users from different members. Note: capabilities and capabilities set requests on video falling coz env was rebuilt without the required permissions for members
https://github.com/user-attachments/assets/cb2135f8-4812-4e0f-96e8-7aa8b2b31acd

Please add a CHANGELOG entry

@BogdanDenis  done!


Add an explanation of how 7-bit addresses should be aligned within a u8

As a note, this seems to match existing drivers and HALs that I spot-checked. We may want to also:

add a similar note for 10-bit addresses (I have never used them)
potentially go further and say that address values exceeding 0x7F should be treated as an error


Vscode 1.85.1 intellisense doesn't work

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

install vscode 1.85.1 and try setintellisense and ctrl+click doesn't work.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Issue with Mac's Homebrew python, and issue when not using legacy build meta

I found two problems.
The first one occurs with Mac. It seems like my package released with versioneer doesn't work well with mac. I used the new non-vendored mode with pyproject.toml.
https://github.com/kiyoon/jupynium.nvim/issues/2
So I tried to make it non-vendored. However, it also doesn't work with pip install . even on Linux. python setup.py install works.
I found that the solution is to go to the legacy mode.
Is there any solution to this?

Your second issue is covered by https://github.com/python-versioneer/python-versioneer/issues/249.
I have no idea why Homebrew Python would work differently from other Pythons. I'll leave this open in case anybody else can contribute.

Your second issue is covered by #249.
I have no idea why Homebrew Python would work differently from other Pythons. I'll leave this open in case anybody else can contribute.

I have a feeling it's a bug from Homebrew python, if it can't install extra dependencies. It is frustrating that I need to go to legacy just because of this.. Thanks for your comment and hopefully someone can find the solution!

I did some digging and found out.
The problem was that it wasn't able to build from build-time dependency because of homebrew's custom sitecustomize module not finding a proper path.
See https://sitetools.readthedocs.io/en/latest/path.html
You can find the error message if you run pip like SETUPTOOLS_SCM_DEBUG=1 PYTHONVERBOSE=1 python3 -m pip install -vvv .
assert not path in sys.path

This has been already fixed, although not included in the official pip yet.
https://github.com/pypa/pip/issues/11539


c-lightning announcing <IP_ADDRESS>:9736

@openoms is there a special reason why c-lightning is doing an annoucement of the local-network IP to the global peer network with announce-addr=<IP_ADDRESS>:${portprefix}9736 see:
https://github.com/rootzoll/raspiblitz/blob/aa962c105a589ee8b711e32718542787095a690f/home.admin/config.scripts/cl.install.sh#L186

That line was used initially to work around this (now fixed) issue: https://github.com/ElementsProject/lightning/issues/4597
Since https://github.com/ElementsProject/lightning/pull/4603 is merged it should be ok to remove.
Alternatively can be changed to use a public IP or IP2Tor address.
This is our use case: https://lightning.readthedocs.io/TOR.html#case-7-c-lightning-has-no-public-ip-address-a-fixed-tor-v3-service-address

the fix fix changes the config retroactively as well with the cl.check.sh on restart. Will run with it until the <IP_ADDRESS> announcement disappears from the gossip and will cherry-pick to v1.7
too

done, part of v1.7 and dev also

was part of the updated v1.7.1 image


Monkey patch http.send_typing to catch 403s

Sometimes discord turns off typing events by throwing 403's, so we should catch those

Thanks discord!

excellent pr


Lookup relations - Search external sources

This PR is dependent on #531 and expands on the relation lookup window added for submission.
Using the REST API's external sources endpoints, it adds a tab for each of the external sources.
These tabs display a list of external entries for the selected source. The user is able to select one or more of these entries to add them to the current selection.
All of the tabs also show the amount of results available for the current query (which is shared between all tabs), even when the relevant tabs are not active.
Currently, the user can only add external entries to the selection. Importing them into the repository as entities/authorities will be part of a separate PR.

I've created https://github.com/atmire/dspace-angular/pull/3 to clearly display what's new in this branch vs https://github.com/DSpace/dspace-angular/pull/531

Thank you for referring the internal diff - https://github.com/atmire/dspace-angular/pull/3/files - it helps the code reviewing process!
Despite the missing type docs, this PR is working as expected. But I'm finding some issues that I think are related with https://github.com/DSpace/dspace-angular/pull/531 and also the questionable search results of ORCID (but unrelated with this PR).
Overall this PR it's ok by me and I will give my thumbs up as soon as it passes the checks and the requested comments are added.

Just want to leave a message that is still working as expected:

I've updated this branch to ensure GIT will only display the commits which differ with master


Helm values - convert JSON schema to a standard format

Summary
Add functionality to convert JSON schema (values.schema.json) to a supported format which will be used by the GUI configuration view to render a form.
Motivation
Simplify fetching & conversion of values.schema.json to a standard supported format to be used for rendering the GUI form-based configuration view.
Proposal

Create a standard format to be used by UI (GUI view) to render a form
Fetch the values.schema.json
Parse it and retrieve the different properties with states like hidden, required, etc.
Covert retrieved data to a standard supported format.

Hey team! Please add your planning poker estimate with ZenHub @shivani170 @vivek-devtron


Crash when a context is destroyed, per test expectations

https://github.com/mozilla/cubeb/blob/master/src/cubeb_wasapi.cpp#L1808-L1809

https://treeherder.mozilla.org/logviewer?job_id=436376607&repo=try&lineNumber=3768 is how it errors on treeherder: this test is expected to crash.

https://github.com/mozilla/cubeb-pulse-rs/commit/c04c4d2c7f2291cb81a1c48f5a8c425748f18cd8 turns out this is incorrect heh.


Testing on home-screen

Has the home screen been tested? I noticed it was marked as closed on Taiga.

Framework and buttons are closed, too ambiguous tasks.  Home screen is set to ready for testing.


fix(example)Next JS _document error

Overview
If using nextjs =>12.1.6 , may get error: "MyDocument.getInitialProps()" should resolve to an object with a "html" prop set with a valid html string
Documentation

[ ] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

Would this break any existing functionality? I can see that the current version supports the logic that you recommended but I am just curious if it would break existing codebases that haven't updated to 12.x yet.

@TIMMLOPK Did you mean v12.2+? I don't think NextJS has a v12.6 out yet: https://www.npmjs.com/package/next
In either case, I'm not sure I'm able to reproduce this. Can you provide a CodeSandbox for reproduction?

@TIMMLOPK Did you mean v12.2+? I don't think NextJS has a v12.6 out yet: https://www.npmjs.com/package/next
In either case, I'm not sure I'm able to reproduce this. Can you provide a CodeSandbox for reproduction?

It is 12.1.6,I correct it.I would try to make reproduction

@brandongregoryscott seem can't make it in codesandbox because it crash.I made a repo for it :https://github.com/TIMMLOPK/error-reproduction

Would this break any existing functionality? I can see that the current version supports the logic that you recommended but I am just curious if it would break existing codebases that haven't updated to 12.x yet. I can't find older documentation, but here is some documentation from NEXT supporting @TIMMLOPK's change. https://nextjs.org/docs/advanced-features/custom-document#customizing-renderpage

No , I tried to use 11.x ,it doesn't matter


changed cstdlib rand to c++ random int generator

the  functions are much better at generating random numbers than the cstdlib rand() and srand() functions

Thank you :smile:


Update background color of selected Tree View item to meet contrast ratio requirement

Describe the bug

Selected TreeViewItems do not currently meet the 3:1 contrast ratio requirement relative to unselected items, as specified in MAS 1.4.11 - Non-text Contrast.
Light theme ratio is 1.5:1

Dark theme ratio is 1.3:1

Additional context

This issue is closely related to #2908 but requires different theme resources to be updated.
Proposed fix
Create new resources
<RevealBackgroundBrush x:Key="SystemControlHighlightAccent1RevealBackgroundBrush" TargetTheme="Dark" Color="{ThemeResource SystemAccentColorDark1}" FallbackColor="{ThemeResource SystemAccentColorDark1}" />
<RevealBackgroundBrush x:Key="SystemControlHighlightAccent1RevealBackgroundBrush" TargetTheme="Light" Color="{ThemeResource SystemAccentColorLight1}" FallbackColor="{ThemeResource SystemAccentColorLight1}" />

Update TreeViewItemBackgroundSelected to use the new SystemControlHighlightAccent1RevealBackgroundBrush.

Is there a reason the TreeView uses different brushes than the ListView here? Wouldn't it make sense here to use the same brushes for the selected ListViewItems and selected TreeViewItems?
Also: The selection brushes used for the TreeView rcurrently have an issue where they don't pick up in-app overrides to the SystemAccentColor resource. This works just fine with the brushes the ListView uses so if we can create a single set of brushes to use here for both TreeView and ListView - we should follow the example set by the ListView here.

@Felix-Dev  The TreeView would need it's own styles so they can be overridden, but those brushes could call upon the same core colour resource as the lists.
But that current colour may be changing, as there are concerns about the contrast ratios at present.

@mdtauk

The TreeView would need it's own styles so they can be overridden, but those brushes could call upon the same core colour resource as the lists.

This is exactly what I had in mind. We would still have the various ListViewItemBackroundSelected* and TreeViewItemBackroundSelected* resources (so we can individually style them), but they would both reference then same base brush. So, instead of
<StaticResource x:Key="ListViewItemBackgroundSelected" ResourceKey="SystemControlHighlightListAccentMediumLowBrush" />
<StaticResource x:Key="TreeViewItemBackgroundSelected" ResourceKey="SystemControlHighlightAccent1RevealBackgroundBrush" />

we would end up with
<StaticResource x:Key="ListViewItemBackgroundSelected" ResourceKey="SystemControlHighlightListAccentMediumLowBrush" />
<StaticResource x:Key="TreeViewItemBackgroundSelected" ResourceKey="SystemControlHighlightListAccentMediumLowBrush"/>

with the added bonus that the TreeViewItem selection brushes now would also pickup SystemAccentColor overrides without having to wait for WinUI 3.

@YuliKl In the related issue #2908 you wrote

Yes, that's the approach used by TreeView, so I've listed the new colors in #2926. I'm not seriously considering switching ListView's colors to these variants to minimize code churn.

Is there a chance we could switch the TreeView selected item brushes to the brushes usedby the ListView then? We would get some consistent coloring then and also solve the issue reported in #1189? Or should we just update them with your proposed SystemControlHighlightAccent1RevealBackgroundBrush theme resource as originally envisioned?

any chance the work for this issue can also encompass a fix for this? https://github.com/microsoft/microsoft-ui-xaml/issues/1189

I think that depends on how we fix this. If we just update the opacity values for the existing resources, this won't fix it. But maybe we will update the way the resources in this area work altogether, which could potentially be a breaking change. In that case, it could fix #1189.

Yep, I am asking @YuliKl for this very reason to consider switching the currently used TreeViewItem selected brushes to the one sused by the ListViewItem. If we won't do that, @dpaulino, you can always override those brushes yourself as shown in that issue (while waiting for a fix which would presumably have to wait until WinUI 3).

@YuliKl The ComboBox also uses the same brush for its ComboBoxItemRevealSelected brush as the TreeViewItem currently:
<StaticResource x:Key="ComboBoxItemRevealBackgroundSelected" ResourceKey="SystemControlHighlightAccent3RevealBackgroundBrush" />
<StaticResource x:Key="ComboBoxItemRevealBackgroundSelectedUnfocused" ResourceKey="SystemControlHighlightAccent3RevealBackgroundBrush" />
<StaticResource x:Key="ComboBoxItemRevealBackgroundSelectedPressed" ResourceKey="SystemControlHighlightAccent3RevealAccent2BackgroundBrush" />
<StaticResource x:Key="ComboBoxItemRevealBackgroundSelectedPointerOver" ResourceKey="SystemControlHighlightAccent2RevealBackgroundBrush" />

Light
Dark

We should thus update those brushes as well (perhaps tracking in a separate issue).

I'm very hesitant to start fundamentally changing brush values without input from other PM owners. @predavid-zz, @kikisaints, @chigy - any thoughts?
TL;DR: Selected items' background colors aren't meeting 3:1 contrast requirements compared to not selected items. ListViewItems are using partially transparent AccentColor as the selected background. TreeViewItems and ComboBoxItems are using RevealBackgroundBrushes with Color property set to accent color light/dark variants. For consistency between controls and as a workaround for #1189, there's a request to change TreeView and ComboBox to use the same brushes as ListView.
Would changing from using RevealBackgroundBrush to using SolidColorBrush result in visual regressions or other negative impact to these controls?

@YuliKl Do you have an update here about the brushes to use?

@Felix-Dev I think we have the green light to use the same brushes in TreeView and ComboBox as the updated brushes you added for ListView.

@YuliKl Unfortunately, I think I ran into an issue here when trying to bring over the updated ListView brushes to the TreeView.
As the TreeViewItem uses reveal highlight, we cannot simply re-use the ListView brushes here. Instead, the idea is to create a new set of RevealBrushes which are built the same as the ListView brushes. In other words, if we have a ListViewItem selected rest background brush like
<SolidColorBrush x:Key="SystemControlHighlightListAccentMediumLowBrush" Color="{ThemeResource SystemAccentColor}" Opacity="0.75" />

we will create an corresponding RevealBrush like this:
<contract5Present:RevealBackgroundBrush x:Key="SystemControlHighlightListAccentMediumLowRevealBackgroundBrush" TargetTheme="Dark" Color="{ThemeResource SystemAccentColor}" Opacity="0.75" FallbackColor="{ThemeResource SystemAccentColorDark1}"/>

I thus created a set of new RevealBrush equivalents to the used ListViewItem selection brushes (excerpt below shows the brushes defined for the Dark theme):
<contract5Present:RevealBackgroundBrush x:Key="SystemControlHighlightListAccentVeryHighRevealBackgroundBrush" TargetTheme="Dark" Color="{ThemeResource SystemAccentColor}" Opacity="0.9" FallbackColor="{ThemeResource SystemAccentColor}" />
<contract5Present:RevealBackgroundBrush x:Key="SystemControlHighlightListAccentMediumLowRevealBackgroundBrush" TargetTheme="Dark" Color="{ThemeResource SystemAccentColor}" Opacity="0.75" FallbackColor="{ThemeResource SystemAccentColorDark1}"/>
<contract5Present:RevealBackgroundBrush x:Key="SystemControlHighlightListAccentLowRevealBackgroundBrush" TargetTheme="Dark" Color="{ThemeResource SystemAccentColor}" Opacity="0.6" FallbackColor="{ThemeResource SystemAccentColorDark2}" />

These new resources added to WinUI will then be referenced by the relevant TreeView theme resources, like so:
<StaticResource x:Key="TreeViewItemBackgroundSelected" ResourceKey="SystemControlHighlightListAccentMediumLowRevealBackgroundBrush" />
<StaticResource x:Key="TreeViewItemBackgroundSelectedPointerOver" ResourceKey="SystemControlHighlightListAccentLowRevealBackgroundBrush" />
<StaticResource x:Key="TreeViewItemBackgroundSelectedPressed" ResourceKey="SystemControlHighlightListAccentVeryHighRevealBackgroundBrush" />

Unfortunately for us, while the selection background colors are indeed consistent now with the ListView item selection brushes, the reveal highlight effect is missing! See this gif:

As you can see, the reveal highlight effect is gone 😢
Note that even the original suggestion for this issue - introducing a new SystemControlHighlightAccent1RevealBackgroundBrush theme resource - would suffer from this issue as it is a RevealBrush as well.
I have also hit this issue in the past when using WinUI's reveal brushes. See this issue here.
I'm not sure right now if that is something we can fix in WinUI 2 or perhaps need WinUI 3 for that. Perhaps @ranjeshj and @StephenLPeters know more (mention is also meant as a FYI).
Your thoughts here, @YuliKl?

Closing this issue. TreeView styles and colors are being updated in the near future.


Revert "GraphQL error: Fragment on Bot can't be spread inside RequestedReviewer (#6461)"

This reverts commit b298cdbd9fa39385ce3c90e674034943228b50ad.

My fix for the GHPR recovery 1 revealed an issue with how I'm merging GraphQL schemas, which is how we handle different schema versions with older versions of GitHub Enterprise. The recovery 1 fix was for a really old version of GHE, so for recovery 2 we'll just revert the recovery 1 change. I'll work on a fix for the merging of schemas for the next regular release. The old GHE version will simply not work for now.


Passing connection option to event handler

Describe the bug
Not sure if this is a bug or not, but after 5 days, 10 hours a day i feel compelled to post this. The connect function to send to event handler does not seem to work at all with any examples. I constantly get a "Request server is denied as target server is invalid."
I've gone through the entire server sdk and I cant see where the handshake is failing.
In additional it never seems to pass CORS. Copy and pasting example does not work. Implementing CORS properly doesnt work.
To Reproduce
Copy and past chat code, dev code, all samples of handling connect option into a new pub sub.
We will close this issue if:

the repro project you share with us is complex. We can't investigate custom projects, so don't point us to such, please.
if we will not be able to repro the behavior you're reporting
--> https://github.com/Azure/azure-webpubsub/blob/main/samples/javascript/chatapp/sdk/server.js

Exceptions (if any)

Further technical details
-javascript, repo is above

Hi @Cytranics , have you configured the event handler settings? What tunnel tool are you using to expose your localhost to public?

Hi @Cytranics , have you configured the event handler settings? What tunnel tool are you using to expose your localhost to public?

Not for the connect. I have an azure web app so no tunnel, using node.js... I copy and paste your exact example and it does not authenticate. I have a feeling it wasnt passing the correct headers to authenticate via the Webhook Orgin. With that said, I really didnt need a connect handshake in the end, and ended up just using connected event.

I copy and paste your exact example and it does not authenticate.

How did you setup the event handlers in your portal? The sample does not need to configure connect so the event handler code does not handle the connect event (handleEvent), so if you configured connect event in your portal it actually fails because no upstream handles the event.

Facing the same issue. Event handler is not being called. When checked in Live Trace Tool, it always shows the Abuse protection and Connection Exception. The outbound traffic from Web PubSub service is restricted by default I guess. Not sure how to enable it, although enabled full network access.

Facing the same issue. Event handler is not being called. When checked in Live Trace Tool, it always shows the Abuse protection and Connection Exception. The outbound traffic from Web PubSub service is restricted by default I guess. Not sure how to enable it, although enabled full network access.

Have you configured any authenticate middleware in your app server? Is your app server accessible if you curl https://<your_web_app_host>/<your_eventhandler_path_as_configured> -X OPTIONS -H "WebHook-Request-Origin: *" -H "ce-awpsversion: 1.0" --ssl-no-revoke -i?

Facing the same issue. Event handler is not being called. When checked in Live Trace Tool, it always shows the Abuse protection and Connection Exception. The outbound traffic from Web PubSub service is restricted by default I guess. Not sure how to enable it, although enabled full network access.

Have you configured any authenticate middleware in your app server? Is your app server accessible if you curl https://<your_web_app_host>/<your_eventhandler_path_as_configured> -X OPTIONS -H "WebHook-Request-Origin: *" -H "ce-awpsversion: 1.0" --ssl-no-revoke -i?

We suspect it could be a firewall thing. We changed to a different publicly available URL and the events were being sent. It is something with our original URL we have to deal with .
Thanks for your insights @vicancy


build target subfolder for static app scripts

Is it anyhow possible do define a subfolder of the target where the application code should be located in?
This would make defining caching strategies and hosting much easier eg.:

/index.html (5min cached)
/static-app/ (cache for ever)

0.hash.js
1.hash.js
....

this would be useful. i'm looking for something similar - ability to include package.version in the chunk name. eg:
index.html (5min cached)
chunkA.1.0.0.js (cache for ever)
chunkB.1.0.0.js
...
chunkA.1.1.0.js
chunkB.1.1.0.js
...

The --public-url option for ng build does this I believe.

--public-url PR for reference


AtomSetOnce incomplete documentation

The documentation for AtomSetOnce stops mid-sentence with Meaning that AtomSetOnce is not usable as a... . I would love to know how this sentence ends. The suspense is killing me! :D Seriously, though, what is AtomSetOnce not usable as? That would be pretty useful information to have.

I would also point out that the documentation for AtomSetOnce::set_if_none is incorrect:

This will do a CAS setting the value only if it is NULL this will return OK(()) if the value was written, otherwise a Err(P) will be returned, where the value was the same value that you passed into this function.

In reality, the function returns an Option<P>, just like Atom::set_if_none. I could submit a PR with a fix to the library to actually mimic what is documented.


Problem: named jobs are not documented in README

Closed by merge of https://github.com/c-rack/quantum-elixir/pull/27.
Thanks!


feat(NavbarToggler): add aria-label to NavbarToggler

This adds the aria-label to the NavbarToggler as
seen in the bootstrap docs.
Fixes #1632

[ ] Bug fix
[x] New feature
[ ] Chore
[ ] Breaking change
[x] There is an open issue which this change addresses
[x] I have read the CONTRIBUTING document.
[x] My commits follow the Git Commit Guidelines
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Working on the breaking tests.

hi @TheSharpieOne any ideas when's this getting to release?


how do i customize the the latex defined by myself, and change it to mathml defined by myself

How can I convert my custom latex symbols to custom mathml

Cause: Now the server storage format is as follows:
<?xml version="1.0" encoding="utf-8"?> <math xmlns="http://www.w3.org/1998/Math/MathML">   <mrow>     <munderover>       <mo>⇌</mo>       <mi>A</mi>       <mi>B</mi>     </munderover>   </mrow> </math>
but use mathjax convert \overset A{\underset B\leftrightharpoons}will get the result:
<math xmlns="http://www.w3.org/1998/Math/MathML" display="block">   <mover>     <munder>       <mo stretchy="false">⇋</mo>       <mi>B</mi>     </munder>     <mi>A</mi>   </mover> </math>
the <munserover>will replaced by <mover> <munder>, I can't replace the store format, how can i do to solve this problem ?
in a word, i want to customize the  mathml from input latex

Also, the code example is for version 3.  If you are using version 2, this would have to be handled differently.


Infer type of method parameter from base class

Close #41303
Fixes #23911
Added some test cases.
Infer from interface does not support yet.

Seems bot does not know about issue and pr...

Any updates on this? I find weird that typescript won't let me break the type but yet shows it as any. This is really worrying me about further uses.

This PR needs some suggestions.

This experiment is pretty old, so I'm going to close it to reduce the number of open PRs.


net: why not check valid when parse CIDR

What version of Go are you using (go version)?
$ go version
go version go1.10.3 linux/amd64

Does this issue reproduce with the latest release?
yes
What did you do?
I setup an invalid CIDR address  by mistake in k8s CNI then all pods in different nodes can not communicate each other, then i find code never check a CIDR address is valid or not.
Based on example in https://golang.org/pkg/net/#ParseCIDR, test some invalid CIDR address like <IP_ADDRESS>/12, <IP_ADDRESS>/12, all can work. They are invalid based on my knowledge, aren't they?
What did you expect to see?
Check validation when parse.

#30825

They are invalid based on my knowledge, aren't they?

Please make sure what's valid and what's invalid from what point of view.

take <IP_ADDRESS>/12 as example:
CIDR bases on dotted decimal, 12 mean it uses 12 bits, the first num uses whole 8 bits and the second num should use first 4 bit of whole 8 bits, 97 is invalid since it is 01100001 while 96 is valid since it is 01100000.
there is a check way by bash: https://stackoverflow.com/questions/50084229/bash-check-if-cidr-address-is-valid

CIDR bases on dotted decimal

Oh, please don't forget Pv6 addresses.

12 mean it uses 12 bits

It's the length of the address prefix, no more no less.

<IP_ADDRESS>/12

We usually think that "okay, the attached address is <IP_ADDRESS>/32, the address prefix is <IP_ADDRESS>/12; presumably, the subnet address is <IP_ADDRESS> and the directed broadcast address is <IP_ADDRESS>; what a big subnet! which RIR allocates this for which organization?"

take k8s network as example:
kubeadm and CNI like calico use CIDR to setup cluster network, specify <IP_ADDRESS>/12 as pod ip range is common.  If specify <IP_ADDRESS>/12, network not works as expected.

CIDR format as ip/mask, now ParseCIDR() only check the validation of ip part not mask, so if use invalid CIDR,  in this range, must have many subnets can not communicate.

kubeadm and and CNI like ... network not works as expected.

Sounds like that is not an issue of ParseCIDR. If your issue is that ParseCIDR accepts a valid CIDR notation and the behavior is not what you want, changing the behavior of ParseCIDR for your application is not an ideal solution (moreover the change will break Kubernetes and Docker).
Instead, you may use ParseCIDR more appropriately like https://play.golang.org/p/z3cYDbR21ws Closing.


Retry failed connection

Hi, earlier today I posted issue #91 about trying to reconnect to a database multiple times in case of a failure on the first attempt. This afternoon, I took a crack at implementing this feature myself and think I am most of the way there. I didn't add any tests, but all of the current tests still pass after my changes. Admittedly, I'm not very familiar with Ruby, but I am more than happy to go back, do some more research, and make any necessary changes or modifications. Thanks!

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

@jeromefroe we'll definitely need tests. Lets us know if you need help writing tests. This blog may help you also: https://www.elastic.co/blog/logstash-functionality-through-testing

Thanks for helping make this plugin better! We'll review this shortly

@suyograo happy to help! I added two new tests to verify the plugin registers and stops without raising an exception when passed a value other than 1 (the default) for the new parameter maximum_connection_attempts. I wasn't sure what other tests to include, but I'd be glad to try and implement any suggestions.

@suyograo I made some changes incorporating your comments. Thanks!

I also rebased onto master to incorporate the latest commits. That's why the old commits show up as outdated.

Hi, I'm using the jdbc plug-in as well and ran into the same issue. does the retry connection make an attempt to reconnect at each interval, such as every hour in your case? this would help if the database server is down say for couple hours and up again in the third hour, logstash will try again in next interval and be able to reconnect again.

@dkolli every time the jdbc plugin is scheduled to run, it will try to connect connection_retry_attempts times in the event of any connection failures, and it will wait connection_retry_attempts_wait_time seconds between connection attempts. The use case I had in mind was if the database was up, but the connection failed for whatever reason, you could wait briefly and try to connect again. I imagined the waiting period to be on the order of seconds, not hours though. I think using the scheduler might be better for that task.

@jeromefroe can you rebase please?

@jeromefroe can you rebase please?

Hey @suyograo, just wanted to give you a heads up that I finished the rebase.

@jeromefroe I apologize again here, but there are still conflicts because this hasn't been merged yet, and a few changes were merged since then.  Could you please rebase again?  You may need to make a few other adjustments.

@untergeek, No need to apologize :) I rebased again, let me know if any further action is required.


Make arm-xlate.pl set use strict.

It was already nearly clean. Just one undeclared variable.

+1. @dot-asm? :)

+1

commit abeae4d, thanks.


Kodama join ECHO space timeout

If joining is not successful Kodama should not throw an error.
Currently, it throws an error after some time trying to connect.

Fixed with observer pattern


Remove visualize code

Closes #1473
Code Changes

[x] remove visualize routes
[x] remove visualize tests
[x] remove visualize core module

Steps to Confirm

[ ] automated tests pass
[ ] searching for visualize in the src/fides/ dir doesn't show anything

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Now that we have proper UI applications, we can remove this old stop-gap code.

When in doubt, cut it out ✂

I'm hitting it with the ctrl+x


Manage CodePipeline, etc. from within Visual Studio

It would be nice if I could monitor CodePipeline, CodeBuild, etc. from within Visual Studio. I assume in the AWS explorer, but it may be better integrated into the team explorer.

Hi @brianjbeach , thank you for reaching out with this request! What sorts of details would you be looking to obtain from your CodeBuild jobs and CodePipeline pipelines?

At a high level the same info that is in the console. The ability to create/start/stop/delete a pipeline/build, check the status, get logs, etc.

I'm interested in hearing more about how this would complement your current workflow. Are you interested in seeing a build pipeline progress after you push code to a repo (similar to CI), or did you have something else in mind like running a pipeline on demand as situations require it?


Support for converting ReferencableLists to/from protos

g3 side merged (with additional fixes): cl/316001326