Fix isConstraintsBatch loop not applying

Track a flag (hasCreatedTable) for every schema that has been created successfully and only then it will be applied of its constraints to avoid errors of constraints already existing and for a clean table creation.

Pull Request Test Coverage Report for Build 20

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

Totals

Change from base Build 18:
0.0%

Covered Lines:
557

Relevant Lines:
557

💛  - Coveralls


Any support for bach queries  ?

Hey bro
I found cassandra batch query is used for atomic transactions.
https://docs.datastax.com/en/cql/3.1/cql/cql_reference/batch_r.html
and in django for running multi queries in one batch we can use something like this code
with transaction.commit_manually:
    for row in rows:
        row.modify()
    transaction.commit()

btw how can I achieve to something like this ?

I think I will probably need to add support. I would love to add this
feature. I will see if I can find time to figure out exactly what work is
involved in adding support for this.
[image: photo]
Seth Denner
Chief Architecht / VP of Engineering
t:<PHONE_NUMBER> | e<EMAIL_ADDRESS>On Sat, Oct 21, 2017 at 2:04 AM, Baltazar<EMAIL_ADDRESS>wrote:

Hey bro
I found cassandra batch query is used for atomic transactions.
https://docs.datastax.com/en/cql/3.1/cql/cql_reference/batch_r.html
and in django for running multi queries in one batch we can use something
like this code
with transaction.commit_manually:
for row in rows:
row.modify()
transaction.commit()
btw how can I achieve to something like this ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Knotis/djangocassandra/issues/105, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABpt-w4JrPpGV24fii-ihe34lM41OnHYks5subOQgaJpZM4QBfsm
.


Refactored file system to use routers and upgraded filenames

I am going to leave this branch with the broken tabs so that this branch just specifically refactors the routers. Then I am going to make a new branch with the customs tabs and working routers.

I am going to leave this branch with the broken tabs so that this branch just specifically refactors the routers. Then I am going to make a new branch with the customs tabs and working routers.

Sounds good


Navbar menu not visible on Mobile

So when I was working on #133, I noticed some bugs:

Navbar menu  is not visible on mobile web version.

and what is the purpose of this blank space below the footer?

I can work on this, assign me this issue I'll fix it.

Hi @dimlaksana. Actually, the design for mobile is not quite complete and needs a lot of improvements. I can use some help!
Will be more than happy to have this designed properly. The theme picker was left out on purpose, assuming that it will have a design that fitted in the mobile view. You can suggest and add the view. Thanks! :)


Inkgd docs: Add Sphinx initial files.
These files are necessary to setup a document in Sphinx.
This will be later used in ReadTheDocs: https://inkgd.readthedocs.io/

#41

Isn't that file windows-only?

This was automatically generated by Sphinx, but my understanding is that the command make html should work out of the box on any OS. make.bat will run on Windows systems and makefile will run on Linux and macOS.

make.bat will run on Windows systems and makefile will run on Linux and macOS.

Makes total sense, I hadn't realised makefile and make.bat served the same purpose. 🤦
Thanks for the explanation, merging now!


Sparc 32 defaulting to returning undefined8 for void

Describe the bug
Upgraded an image.  Noticed uncommitted void return functions want to now return undefined8.
Using unknown logic for the low 32 or really high, but wants to use param_2 as the high 32-bits.  Inserting at least a param_1 and param_2 into the prototype even when it's zero arguments.
Previous ghidra versions were also broken though sometimes, converting void into undefined4, returning param_1 (and adding param_1 as an argument even when not present)
Environment (please complete the following information):

OS: win10
Java Version: 21.0.4
Ghidra Version: 11.2
Ghidra Origin: official GitHub distro

No, nvmd.  It's def breaking lots of functions, not just void return.  It doesn't pick up the return 4-byte value and uses the previous o0 value

Are these assigned return and param types, or is the decompiler recovering the types on the fly?
I'm assuming you mean the decompiler is computing them on the fly and you haven't  committed the param/return from the decompiler.
64 or 32 bit sparc?
Do you have any example functions you could share?  Either bytes or copy/paste of listing?

sorry @emteere  missed the reply.  pcodetest is showing it looks like in 11.2.1, built with sparc-suse-linux-gcc (14). i just did the basic analysis and main (and others) were showing return CONCAT(param_1,param_2) for undefined8
SPARC_GCC_O0_pcodtest.txt


gf callback execution breaks unpack

As noted in the readme, the gf scripts require for the duration of their execution iskeyword to include the period character. However, it seems that the behaviour of the Unpack command depends on iskeyword not including a period character. So there's a race condition - if you call Unpack while the gf callback array is running, the result is nonsensical; for example:
const { Route.extend } = Ember.Route.extend;

export default Route.extend({});

Obviously the usual behaviour of w and b is altered during this time too, which is irritating.
I'm not sure what the underlying issue is - I have not configured any extra gf callbacks, so it's just the defaults. They're either (sometimes) taking a very long time or silently failing without setting iskeyword back to its pre-existing value. By the same token, I'm not sure how to reproduce the issue (although the specific problem with Unpack can be simulated just by set iskeyword+=.)

This is interesting. I wouldn't really expect that gf would take a lot of time, and even if it did, I'm not sure that it would allow you to do something while it's working. It's supposed to be synchronous. It might be that, as you say, gf is silently failing or something. I'll check the iskeyword setting and unsetting, but could you give me an example of a particular gf that takes a lot of time for you?


Issue in this demo example while select .xlsx file from browser

Hi,
I am not able to get any data in res variable if I select .xlsx file from chrome browser. It works with txt file. Can someone please help that how it works for xlsx file.
Thanks in advance.
Prem

Are you using it locally on your machine?

Hi Agershun,
Thank you very much and now my issue has been resolved by calling XLSX
method rather than FILE.
thanks,
Prem
On Nov 24, 2015 3:58 AM, "Mathias Rangel Wulff"<EMAIL_ADDRESS>wrote:

Are you using it locally on your machine?
—
Reply to this email directly or view it on GitHub
https://github.com/agershun/alasql/issues/464#issuecomment-159088014.


Set Fields Indicating Anonymization

After running set "Patient Identity Removed" (0012,0062) = 'YES'
and "De-identification Method" (0012,0063) to something like "dicom-anon"
See: http://dicom.nema.org/medical/dicom/current/output/chtml/part03/sect_C.7.html

Good idea- will do.

This is done, but the values method has been put into 0012,0064, De-identification Method Code Sequence, describing the profile, Clean or Clean Descriptors.


Auto Plot New Opens All DB Runs

I noticed this is the same as an old issue which was closed: #49 but the issue persists for me with plottr version 0.9.1:
Opening a .db file with plottr-inspectr after having ticked 'auto-plot new' causes all measurement runs from that database file to be opened. I believe this is undesirable behavior since .db files can easily exceed 100 measurements.

I am able to reproduce this but only if both refresh interval is set to none zero and autoplot new is toggled

This happens if the refresh interval is set low. I think what happens is that while the first load correctly excludes plotting of everything the first refresh is triggered before this is done. That then triggers a replot of every thing


Diagnostics

Would it be possible to add a set of functions that would get some information about  the current state of
RecyclableMemoryStreamManager? This would be useful for debugging and unit testing. For example, a unit test may want to ensure that one stream with a given guid or tag is now closed. Or a debug function at the end of a program may want to ensure that ALL streams have been disposed, and if some are not,  list their guids and/or tags (For tags, that could maybe just be collapsed down to a tag plus count).
I realize you have ETW events, but those are a pain to deal with, and I don't see how you can even use them in unit tests.

I'm looking at the unit tests project now, I think I'll close this and see what I can do with the existing code.


Got crash while compression multiple file in loop, more than 4 media file

** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '*** -[AVAssetReader startReading] cannot be called again after reading has already started'
*** First throw call stack:
(0x19c817298 0x1b0571480 0x1a3c92638 0x100ac49d0 0x100ac4e48 0x10060cf04 0x1a3cb3834 0x1026b7ae8 0x1026b932c 0x1026c038c 0x1026c1010 0x1026cc820 0x1e4d3d5bc 0x1e4d4086c)
libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '*** -[AVAssetReader startReading] cannot be called again after reading has already started'
terminating with uncaught exception of type NSException

crash on
audioReader?.startReading()

I got the same crash issue. At audioReader?.startReading()
If i check in the console the audioReader status is already reading.
so i changed the code like this.

after that I got crash at  audioWriterInput.requestMediaDataWhenReady(on: processingQueue, using: {() -> Void in stating you cannot requestMediaData again

yes, I did by doing 3 compression at a time. that fixed my crash

I'd appriciate anyone's contribution to build multiple video compression, i don't have access to a Mac at the moment. I have added the feature to Android .. you can use it as a referenece.
https://github.com/AbedElazizShe/LightCompressor/commit/8d8a35d9733b78724cd4466608075ca01742a78e

yes, I did by doing 3 compression at a time. that fixed my crash

No, I also tried compressing three videos at a time. But the issue still exists.
Can someone suggest any alternate library for video compression with multiple files at a time in swift ios

if audioReader?.status == .unknown{
}
Please add this line and check
I hope issue will be fix


Make constants between JS and bgfx more consistent

Some constants are defined in nativeEngine.ts, some are converted from bjs to bgfx in cpp.
Make that mecanism more consistent: stick with one convention.

Being addressed by #416.

Unless there are any I missed, this should be resolved as of #416.


White logo on White background login page

Bug description
When i choose a custom logo it appears on the left side menu and in the login land page.
but if the logo is white (to match the blue left side menu) I can barely see it in the login screen because the background is white as well.
Expected behavior
I expect to see my logo clearly in both places. on the blue background menu and on the white background login page.
change the background in the login screen.
or give an option to upload 2 different logos.
Steps to reproduce the issue:
Steps to reproduce the behavior:

use a custom white only logo.
inspect the menu and the login page.

Technical details:

Portainer version: 1.19.1
Docker version (managed by Portainer): 18.06.0 ce
Platform (windows/linux): linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): docker run -p 9000:9000 portainer/portainer
Browser: Chrome Version 68.0.3440.106

Additional context
Add any other context about the problem here.

Re-opening as this should not have been closed.

Hello,
I'd prefer uploading two different logos, one for the left menu and another one for the login page.
Thanks!

Hi - Was this actually completed? Just seeing this ticket now as I'm having the same issue, I have a logo with alpha and a white background.
I do have a coloured text version of my logo but that doesn't look great once you've logged in and it's on the side bar


Add a switch to disable splash screen on startup

Description
Make showing of splash screen optional
Motivation and Context
Some users don't like the splash screen always on top of everything.
This seems to be especially annoying, when JMeter takes a long time
to initialize.
Bugzilla Id: 64658
How Has This Been Tested?
Tested by hand (starting from CLI)
Screenshots (if appropriate):
Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

@FSchumacher , the original issue was that there's setAlwaysOnTop property which basically blocks all other applications.
What do you think if we avoid setAlwaysOnTop?

I just tried to remove setAlwaysOnTop(true);, and it produces reasonable behavior at least for macOS.
It does show the splash screen, and it enables to switch between the applications.
So I think we should remote setAlwaysOnTop(true);, and that is it.
@FSchumacher ,  WDYT?

I haven't tried, what the splash screen does, when it is not instructed to be always on top. I fear, that it can come up behind the main window and irritate people, as JMeter will seem to stop being responsive until the (possibly hidden?) splash screen is removed.
But I am really not sure, what the correct thing is.
On the other hand, I think the real problems here are not the splash screen. If it would show for two seconds and then go away (as it usually does), nobody would really complain. This issue (the bugzilla one) has been opened as we have a lot of problems working together.

The splash screen stays always on top (or is displayed at all)
Every added element to the tree leads to an initialization of the complete tree with all already added and initialized elements
The construction of test elements via reflection seems to be slow (especially when a lot of elements are created serially)
(and this is the real killer) The usage of undo feature is used while constructing the tree on load and leads to a real bad performance

So, all in all, I am pretty sure, that this PR is more of a cosmetic change and we should really focus on the other problems in that list.

I fear, that it can come up behind the main window and irritate people, as JMeter will seem to stop being responsive until the (possibly hidden?) splash screen is removed.

That is valid.
It looks like JDialog + ModalityType.APPLICATION_MODAL + undecorated nails it.
The result looks like "splash screen is always above JMeter window. The main window is unclickable as long as the splash is there, and the splash does not block other apps.
--- a/src/core/src/main/java/org/apache/jmeter/SplashScreen.java
+++ b/src/core/src/main/java/org/apache/jmeter/SplashScreen.java
@@ -18,15 +18,16 @@
 package org.apache.jmeter;

 import java.awt.BorderLayout;
+import java.awt.Frame;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.URL;

 import javax.swing.Icon;
 import javax.swing.JComponent;
+import javax.swing.JDialog;
 import javax.swing.JLabel;
 import javax.swing.JProgressBar;
-import javax.swing.JWindow;
 import javax.swing.SwingUtilities;
 import javax.swing.border.EmptyBorder;

@@ -40,7 +41,7 @@ import com.github.weisj.darklaf.icons.ThemedSVGIcon;
  * Splash Screen
  * @since 3.2
  */
-public class SplashScreen extends JWindow {
+public class SplashScreen extends JDialog {
     private static final Logger log = LoggerFactory.getLogger(SplashScreen.class);

     private static final long serialVersionUID = 1L;
@@ -53,6 +54,9 @@ public class SplashScreen extends JWindow {
         setLayout(new BorderLayout());
         add(loadLogo(), BorderLayout.CENTER);
         add(progressBar, BorderLayout.SOUTH);
+        setModalityType(ModalityType.APPLICATION_MODAL);
+        setAutoRequestFocus(true);
+        setUndecorated(true);
         pack();
         setLocationRelativeTo(null);
     }
@@ -91,7 +95,6 @@ public class SplashScreen extends JWindow {
     public void showScreen() {
         SwingUtilities.invokeLater(() -> {
             setVisible(true);
-            setAlwaysOnTop(true);
         });

So, all in all, I am pretty sure, that this PR is more of a cosmetic change and we should really focus on the other problems in that list.

I agree the thing is cosmetic, yet I still think it is weird to create a configuration property for that.
I'm inclined to merge https://github.com/apache/jmeter/pull/614 if you don't mind.

Merged in https://github.com/apache/jmeter/pull/614


Reconsider Langchain dependency

We rely on langchain for text splitting.
This is quite a large dependency for the very small part of it we use. Consider whether it would be worth vendorising this text splitter, implementing our own, or finding a less heavyweight text splitting package.

Related issue on wagtail-ai https://github.com/wagtail/wagtail-ai/issues/17

Fixed by #24


feat(options): Home button url, custom css/js injection, allowing local requests

Added:

Option in UIOptions to supply a custom home button url so that users which operate with different sub-domains can fix the home button (in my case the web api is detached from the frontend server and they're hosted on different domains, so the home button never worked)
Option in AuthorizationOptions to disable the feature to always exempt local requests from authentication. This way I can test my application locally.
Option in UIOptions  to inject custom CSS / JS in the head of the index.html, just like swagger can. This is the main feature and allows for great customizations, and for me particularly should help with #22.

Changed:

Added samples of these options in the sample web app, not sure if you'd want to leave those there but it provides a quick way to inspect the results of this feature branch so i left them in.
Changed the target to net6 for the WebApi since that's the only framework version i've got installed.

Remarks:

Right now i'm using a dummy meta tag in the head for the custom head content since the dev server with parcel js does not replace the template values
Feel free to edit and so on - i just need to inject some custom JS if possible.

@mo-esmp @followynne for review.

Thanks for the PR, I will review it soon.

Hey @sommmen,
Could you please update the readme file and add descriptions on injecting custom css/js injection, home button, ...?

Hey @sommmen, Could you please update the readme file and add descriptions on injecting custom css/js injection, home button, ...?

Will do tomorrow

Hey @sommmen, Could you please update the readme file and add descriptions on injecting custom css/js injection, home button, ...?

Done:

@sommmen new version is released and please upgrade to the version 2.2.0.


analog output

@ethierc  please correct me if I'm wrong about this
We need an analogue output to signal the TDT device, indicating the trial's state. This will allow the TDT to synchronize the fibre photometry data with the behavioural data.

@ethierc please correct me if I'm wrong about this
We need an analogue output to signal the TDT device, indicating the trial's state. This will allow the TDT to synchronize the fibre photometry data with the behavioural data.

We might not need to have an output for TDT, if Pynapse works fine. However, it could be good to have that possibility anyways. Not sure it needs to be analog though. we could use digital pulses.
But we will eventually need analog outputs to control 1 speakers and 1 or 2 LEDs. so at least 3, more is better if it's easy.

I guess for the LEDs and speakers, it could also be digital out 🤔

@ethierc  we can have as much as we need for LEDs and a speaker or buzzer.
there is no limitation for this but to know the usage we can have a better implementation.

I did not think of that before, but I guess for speaker we could just use square (DigOut) waves

to record the rotating angle with the fiber photometry system is possible using the two A-B digital signals directly. So I don't think we need analog out if it's complicated to do


Azure App Gateway is including Domain as part of Session Affinity Cookie - why? How to remove?

We have Azure Appgateway running test and prod with session based cookie authentication
For some reason in prod the cookie is being set as:
Set-Cookie: Affinity=abcdefg;Path=/;Domain=mywebsite.domain.com.au
The Domain component is causing issues with an application, in test we get:
Set-Cookie: Affinity=abcdefg;Path=/
Which works fine.
What determines whether the Domain parameter is included in the cookie?

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

ID: f4b2b8dc-b612-882d-3496-063cbe7772e9
Version Independent ID: b2919efa-17da-e3ad-631d-72d8e4b328e0
Content: Troubleshoot session affinity issues - Azure Application Gateway
Content Source: articles/application-gateway/how-to-troubleshoot-application-gateway-session-affinity-issues.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

Thanks for bringing this to our attention. We are actively investigating and will get back to you soon.

@catmanjan Do you have the option Host Name Override selected in your HTTP setting with the above domain? That's probably why you have that domain parameter included in the cookie for your prod environment.

@catmanjan Any update?

@SaiKishor-MSFT we are not using the Host Name Override setting in either test or production

@catmanjan Can you confirm the versions of App Gateway for Test and Production?

@SaiKishor-MSFT they are both Standard V2 if thats what you mean?

@catmanjan Thanks for confirming, yes, that is what I wanted to know. Can you confirm if this cookie is coming from the backend by verifying the same in the Access logs? https://docs.microsoft.com/en-us/azure/application-gateway/application-gateway-diagnostics#diagnostic-logging

@catmanjan Any update?

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Cannot PATCH '/api/my-model/id' in IE/Edge

I have a jsonapi.org route set up in config.js like this:
this.patch('/model/:id', (db, request) => {
  let params = JSON.parse(request.requestBody);
  let model = params.data.attributes;
  db.model.update(model);
  return params;
});

My tests work fine in all browsers except Edge and Internet Explorer where it fails with an adapter operation error. It looks like Mirage isn't catching the XHR request (the example code above is never ran) so it makes it onto the network where it of course 404s.

strange...sounds like a bug in pretender. I bet it would be super helpful if you ran pretender's test suite in IE, as a start!

So, I cloned pretender and wrote a test like this one except using type: PATCH and sending data and it completes fine on Internet Explorer.
I'm still digging but the best I can tell the request doesn't get into pretender code. I simplified my mirage config down to this.patch('/model/:id') with the same behavior. If I remove it completely then Mirage properly throws the error that "there was no route defined to handle this request" but with it defined it acts like it worked. It has all appearances of a passthrough request but a debugger in pretender's passthrough code is never hit

I'm out of time to work on this more at the moment so for now I overwrote updateRecord to force PUT as the verb. I've been unable to determine where the issue lies; it fails on multiple versions of IE including Edge which makes it seem like it's more of a Windows thing rather than "old IE".
The closest thing in jQuery (thinking it could be feature detection) that I found was https://github.com/jquery/jquery/issues/2068 but I'm doing neither cross domain nor using 1.11.1 (I'm on 1.11.3). Pretender seems like it works since a simple PATCH test passes.

Closing due to inactivity. Let me know if you need more help.


401 Client Error: Unauthorized for url

I'm trying to run the example code provided in the readme and I'm getting this error:
401 Client Error: Unauthorized for url: https://api.elevenlabs.io/v1/voices
  File "C:\Users\jarod\Desktop\Code\Vivy\Test.py", line 11, in <module>
    voice = user.get_voices_by_name("Rachel")[0]  # This is a list because multiple voices can have the same name
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://api.elevenlabs.io/v1/voices

The code is simply:
import io
import json
import os
import elevenlabslib.helpers
from elevenlabslib import *

user = ElevenLabsUser("[my api key]")
voice = user.get_voices_by_name("Rachel")[0]  # This is a list because multiple voices can have the same name

voice.play_preview(playInBackground=False)
for sample in voice.get_samples():
    sample.play_audio(playInBackground=False)

voice.generate_and_play_audio("Test.", playInBackground=False)

for historyItem in user.get_history_items():
    if historyItem.text == "Test.":
        # The first items are the newest, so we can stop as soon as we find one.
        historyItem.delete()
        break

I don't have much experience working with Python and API's so I'm not sure why this error is occuring, I've tried to use a new API key, clear browser cookies, and log back in/out.

Yeah it's my bad, I shouldn't have put those goddamn brackets in the example, it's misleading. Just don't put the square brackets and you'll be fine.

Ah, solved.  Thanks man!


Mongoose connects to MongoLab DB but cannot read data (Auth Error)

I set up my config like this:
var config = {
  "scheme": "mongodb",
  "hosts": [
    {
      "host": "ds1111-a1.mongolab.com",
      "port": 1234
    }
  ],
  "username": "user:n@me",
  "password": "p@ssword",
  "options": {
    "authSource": "@dmin"
  },
  "database": "d@tabase"
}

Then:
var uriUtil = require('mongodb-uri');
var mongodbUri = uriUtil.format(config);
var mongooseUri = uriUtil.formatMongoose(mongodbUri);
mongoose.connect(mongooseUri);

The app connects to the db successfully, but when I make a query, I get this error: 'MongoError: not authorized for query on 'dbname.coll'
Since it's pointing to auth, I tried with the username and password in the options, but I still get same error.
Any ideas on what's causing this?

Since it looks like you're using a MongoLab deployment, you should start from the URI given in the MongoLab UI. You can find it using these instructions:
http://docs.mongolab.com/connecting/#connect-string
Be sure to fill in the <dbuser> and <dbpassword> placeholders with their actual values. Then feed it into formatMongoose():
var uriUtil = require('mongodb-uri');
var standardUri = "mongodb://copied:and@pasted/from/mongolab/ui";
var mongooseUri = uriUtil.formatMongoose(standardUri);
mongoose.connect(mongooseUri);

Finally, some modern versions of Mongoose do accept the standard URI so you might check the docs and try that directly if supported.
PS -<EMAIL_ADDRESS>is a more expeditious method of getting support for your MongoLab-hosted deployment.


Enable STG Pixie Video Test code

Enable the STG 1861 Pixie Video test routines in the STG ROM code and add the Test Pixie option to the Help Menu.

Updated and assembled boots.asm to provide Pixie Video Test. Available with Release 2 of the STG + Video Rom code.


Remove unavailable theme

Removed material-lite theme from repository.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @rpidanny,
thank you for notifying us about this issue and for sending this pull request.
For the record: I've searched for this theme using different search queries on Github as well as on Gitlab in the hope, that the repository name has been changed, the ownership has been transferred or that the theme simply moved to a different hosting service. Unfortunately, I wasn't able to find it.


Wikimedia DAG iterates over thousands of pages without processing any data

Description

The Wikimedia provider DAG failed for 2022-07-26 after timing out, but seemingly not for the reasons in #725. Instead, it looked like from the logs that the script was iterating over thousands of pages without encountering new media or a giacontinue token.
Here's the log for the run (with Got 250 pages log lines replaced with a * in order to grasp the scale a bit easier):
wikimedia-2022-07-26.log
Reproduction

Set the delay attribute of the WikimediaCommonsDataIngester class to 0.1

diff --git a/openverse_catalog/dags/providers/provider_api_scripts/wikimedia_commons.py b/openverse_catalog/dags/providers/provider_api_scripts/wikimedia_commons.py
index d73c9f2..c28b128 100644
--- a/openverse_catalog/dags/providers/provider_api_scripts/wikimedia_commons.py
+++ b/openverse_catalog/dags/providers/provider_api_scripts/wikimedia_commons.py
@@ -47,6 +47,7 @@ class WikimediaCommonsDataIngester(ProviderDataIngester):
         "audio": prov.WIKIMEDIA_AUDIO_PROVIDER,
     }
     endpoint = f"https://{HOST}/w/api.php"
+    delay = 0.1
     headers = {"User-Agent": prov.UA_STRING}

     # The batch_limit applies to the number of pages received by the API, rather

just recreate
just shell
airflow dags backfill -s 2022-07-26 -e 2022-07-26 -v wikimedia_commons_workflow

Resolution

[ ] 🙋 I would be interested in resolving this bug.

After running this locally, I noticed that the log messages looked quite different from the above posted logs. Turns out we merged #614 after this particular backfill run had started, so it was still using the old code! This is actually a duplicate of #725, we just have better logging now so it's easier to tell what's going on 😄 Closing this!


Embedded Ansible doesn't clear authentications on destroy

(created because of https://github.com/ManageIQ/guides/pull/276#discussion_r156130517 )
Running ManageIQ::Providers::EmbeddedAnsible::Provider.first.destroy (while having exactly 1 embedded ansible provider) does not delete all it's authentications.
This needed to be done manually after:
db = MiqDatabase.first
db.authentication_type('ansible_secret_key').delete    # db.ansible_secret_key.delete
db.ansible_rabbitmq_authentication.delete
db.ansible_admin_authentication.delete
db.ansible_database_authentication.delete

Cc @jrafanie

@jameswnl could this be related to the orchestrated destroy?

Very likely.
Do we support this use case? To delete the embedded?

So this is interesting because it is something we should never do in production.
The authentication records stored against the MiqDatabase row are not really dependents of the provider. They are there so that we can recreate the provider if it was removed.
I don't really think this is a thing we should add.

So the other issue that @jrafanie mentioned is that external Ansible Tower (as well as Foreman) provider destroy is always rolling back and the fix is here.
Now reading a bit closer, i am not sure if this issue is the same - @himdel when you do the provider.destroy, does the provider remain or not?

Now reading a bit closer, i am not sure if this issue is the same - @himdel when you do the provider.destroy, does the provider remain or not?

This is not the same, when I did provider.destroy, it worked and the provider (and the manager) went away.

How I hit this was: I had my dev setup set to use an external Ansible Tower as an embedded ansible.
Then that stopped working because of some "invalid pk" issues, so I started looking at how to make embedded ansible work again on dev setups - @carbonin provided a solution (the one documented in the guides PR), but it didn't work until I removed the old keys.
(I also have no opinion whatsoever on whether this should happen by default or not, it was a special case.)

I'm fine with leaving this as something to document as developer specific workflow.  I was concerned there was something else causing authentications to stick around.

@miq-bot remove_label stale
@miq-bot add_label pinned

@himdel since we changed the implementation of Embedded Ansible in the Ivanchuk release, would you please re-test once again.

db.ansible_rabbitmq_authentication
db.ansible_admin_authentication
db.ansible_database_authentication

None of those methods exist anymore.
But, I'm currently seeing these authentication types:
> db.send(:available_authentications).pluck(:name, :authtype)
=> [["Ansible Rabbitmq Authentication", "ansible_rabbitmq_auth"],
    ["MiqDatabase vmdb_production", "registration"],
    ["Ansible Admin Authentication", "ansible_admin_password"],
    ["Ansible Database Authentication", "ansible_database_password"],
    ["Ansible Secret Key", "ansible_secret_key"]]

None of ansible_rabbitmq_auth, ansible_admin_password, ansible_database_password, ansible_secret_key appear anywhere in the code.
So, I would guess that this is irrelevant to the new implementation but we still need a migration to remove the old authentications.
@NickLaMuro WDYT?^

This is no longer an issue with the new ansible-runner based embedded ansible implementation.


Database errors running "python3 manage.py runstator"

I'm trying to setup a test installation of Takahe on my local machine.
When I run docker run --env-file takahe.env --net=host jointakahe/takahe python3 manage.py runstator I get the following error.
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
psycopg2.errors.UndefinedTable: relation "core_config" does not exist
LINE 1: ... "core_config"."json", "core_config"."image" FROM "core_conf...
                                                             ^

takahe.env:
TAKAHE_DATABASE_SERVER=postgres://takahe@<IP_ADDRESS>:5432/takahe
TAKAHE_MEDIA_BACKEND=local://local
TAKAHE_MAIN_DOMAIN=<IP_ADDRESS>
TAKAHE_EMAIL_SERVER=console://console
TAKAHE_EMAIL_FROM=<secret email>
TAKAHE_AUTO_ADMIN_EMAIL=<secret email>
TAKAHE_USE_PROXY_HEADERS=false
TAKAHE_ERROR_EMAILS='["<secret email>"]'
TAKAHE_SECRET_KEY=<secret>

I already have an empty user and database for Takahe on my machine.

The error "psycopg2.errors.UndefinedTable: relation "core_config" does not exist" indicates your database migrations have not yet been applied.
We recommend running the docker compose setup, which runs the migration automatically. However if you just need to setup the db the below command will run that command:
docker run --env-file takahe.env --net=host jointakahe/takahe python3 manage.py migrate

Yes, this means you've not run the migrations - you can read more about that here: https://docs.jointakahe.org/en/latest/installation/#database


Partial update, or load current state into cfg files (feature req)

I have a number of existing repos for which I need to update a couple of fields.
This extension looks perfect, but I'm wondering about a couple of things.

Can this be used to populate the .github/config settings from the existing repo?
ie: rather than using gh repo-config init can one populate these settings from the current repo, maybe with a gh repo-config load ?

If only some of the .github/config files exist, will it only update those settings?
eg:
If I only provide a .github/config/branch-protection/main.json and nothing else, will the only action taken be to update the main branch protection rules?

In the absence of # 1, the ability to do # 2 would be great.
Thanks in advance.

So, after actually testing this, I can do # 2 above.
Still think # 1 would be a useful feature though, as that lets you take an existing repo dump it's entire config, then copy those files to other repos and use as a source to setup "same as existing".
This saves one from needing to review all options and figure out the mapping between how it's set in the UI, vs the json representation.

I'll second the request for pulling the current configuration down and initializing. Then I know if I make a change, to one setting, only that change will be applied.


GPT-4o

Hi @ekondur!
GPT-4o is here!

Hello @gluks, have you experienced this? I was wondering what the difference was. :)

Hello @gluks, have you experienced this? I was wondering what the difference was. :)

Yes. I don't really know if it is "smarter" or not, but at least it works noticeably faster than gpt4 or gpt4-turbo. And also OpenAI claims that it costs 4 times cheaper.

thank you @gluks, it is released with v1.3.2


feat(impl):[TRI-905] helper analyzer for counting expected results

Helper analyzer for counting expected results

Works fine for me, thanks for the presentation Krzysztof! Only creation of json for submodels is missing but this will follow as discussed.


Mitogen issues in AWX when used verbosity 3 and higher

Mitogen is configured to be used in AWX. Version of mitogen: 0.2.2, AWX: 2.0.0
When used verbosity 0 (Normal), 1 (Verbose) or 2 (More verbose) everything works as expected.
When used verbosity 3 (Debug), or 4 (Connection Debug) template runs "forever" with output
[pid 1498] 13:56:51.875998 D mitogen: unix.connect(path='/tmp/mitogen_unix_ahHeVg')
Exception in thread mitogen-broker:
Traceback (most recent call last):
  File "/usr/lib64/python2.7/threading.py", line 812, in __bootstrap_inner  self.run()
  File "/usr/lib64/python2.7/threading.py", line 765, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/var/lib/awx/projects/_8__git_onecloud/strategy_plugins/mitogen/core.py", line 340, in <lambda>
    _profile_hook = lambda name, func, *args: func(*args)
  File "/var/lib/awx/projects/_8__git_onecloud/strategy_plugins/mitogen/core.py", line 1813, in _broker_main
    LOG.exception('_broker_main() crashed')
  File "/usr/lib64/python2.7/logging/__init__.py", line 1182, in exception
    self.error(msg, *args, **kwargs)
  File "/usr/lib64/python2.7/logging/__init__.py", line 1175, in error
    self._log(ERROR, msg, args, **kwargs)
  File "/usr/lib64/python2.7/logging/__init__.py", line 1268, in _log
    self.handle(record)
  File "/usr/lib64/python2.7/logging/__init__.py", line 1278, in handle
    self.callHandlers(record)
  File "/usr/lib64/python2.7/logging/__init__.py", line 1318, in callHandlers
    hdlr.handle(record)
  File "/usr/lib64/python2.7/logging/__init__.py", line 749, in handle
    self.emit(record)
  File "/var/lib/awx/projects/_8__git_onecloud/strategy_plugins/ansible_mitogen/logging.py", line 70, in emit
    display.error(s, wrap_text=False)
  File "/usr/lib/python2.7/site-packages/awx/lib/awx_display_callback/display.py", line 40, in wrapper
    with event_context.set_local(**context):
  File "/usr/lib64/python2.7/contextlib.py", line 17, in __enter__
    return self.gen.next()
  File "/usr/lib/python2.7/site-packages/awx/lib/awx_display_callback/events.py", line 96, in set_local
    self.remove_local(**kwargs)
  File "/usr/lib/python2.7/site-packages/awx/lib/awx_display_callback/events.py", line 88, in remove_local
    self._local._ctx.pop(key, None)
AttributeError: 'thread._local' object has no attribute '_ctx'

Steps to reproduce:

Install AWX, mitogen, put mitogen in strategy_plugins (everything standard installation)
Create some template in AWX
Run the template in AWX with desired verbosity

It looks like 2 issues here:

broker_main() crashes, which is probably a Mitogen bug, but
it can't log the cause of the crash because of the events.py issue, which looks like an AWX bug. I'm not entirely sure what the relevant AWX code is doing, but it's possible Mitogen's use of the Ansible display framework from a non-main thread is confusing it

Please leave this with me and I'll try to setup a reproduction shortly.

Re: your offer to supply a public AWX instance, that would be immensely useful! Sorry, just digging through old chat history and noticed you message. :)

I've spotted the bug in AWX at least. Going to open a PR for that to begin with.

The pull request is here: https://github.com/ansible/awx/pull/2565
Allow me a few hours to think about a workaround I can include in Mitogen, but it looks like potentially it would have to be a monkeypatch.

Hi there,
I have checked in a workaround for the issue on dmw branch of the Git repository. Would it be possible for you to test it? I don't know much about AWX configuration :)
I left the worker you supplied unpatched, so dropping Mitogen into its config should be enough to prove the workaround I've added works correctly.

Thanks a ton for reporting this! The workaround on master solves the problem, and hopefully future versions of AWX will include the PR I sent.
This is now on the master branch and will make it into the next release. To be updated when a new release is made, subscribe to https://www.freelists.org/list/mitogen-announce
Thanks for reporting this!


About odb::dbDatabase::getChips()

Hi, I noticed that in odb/include/odb/db.h, getChips() is marked as “DEPRECATED”. Can a dbDatabase contain multiple dbChip still?

The database supports it but the rest of OR assumes a single chip.  What is motivating your question?

Thank you for replying.
Here is my motivation: A large chip always contains several parts, and each part can be implemented seperately. Since dbBlock supports only two-levels of hierarchy, if there exists one dbChip in dbDatabase, the parts has to be child of the top block and the child of the top dbBlock must be “flatten”, i.e. without submodules in it. This is not so friendly for each part implementation. So I’m thinking that make each part a single chip may be a good choice.

Thanks, I've got it.

question solved.


gRPC service failed with status: TIMEOUT

Description
In 50 out of 3000 transactions more or less , we are getting this error:
GrpcServiceError: gRPC service failed with status: TIMEOUT
    at Timeout._onTimeout (...\@hashgraph\sdk\lib\channel\NodeChannel.cjs:111:20)
    at listOnTimeout (node:internal/timers:564:17)
    at process.processTimers (node:internal/timers:507:7) {
  status: GrpcStatus { _code: 17 }
}

It happens either sending NFTs, either sending Hbar or inclusive creating a new Account. We are executing it in BackEnd in Node.js
This is a big problem for us and we're having big troubles with our Clients because of this.
This is happening in v2.18.5 , v2.19.1 and v2.19.2 . In these versions which are the ones I tested in all of them happens.
Steps to reproduce
1-  .addNftTransfer()
2- .addHbarTransfer()
3-  Generating a new Account in Hedera
Additional context
No response
Hedera network
mainnet
Version
2.19.1
Operating system
None

Hello @AdriMsM91, There is a default value of the maxExecutionTime of the Node Client in the SDK. The default time is set to 10 seconds. When the transaction is too big, you might hit the TIMEOUT error from the GRPC. You can use the setter: setMaxExecutionTime to increase this time and try again. You can also send us some code in order to test and replicate your issue.

Hello @ochikov the Transaction isn't too big is just an NFT Transfer. I know I can retry... but I shouldn't be worried everytime I'm using the SDK to add Works Arounds... As you can see there are more people with this same problem in Hedera Dicord #javascript
 let response = new TransferTransaction()
  .setMaxTransactionFee(new Hbar(maxFee))
  .addNftTransfer(tokenId, serialNumber, originAccount, receiverAccount)
  .setTransactionMemo(memo);
const txSubmit = await response.execute(client);
const receipt = await txSubmit.getReceipt(client);

const newAccount = await new AccountCreateTransaction()
 .setKey(newAccountPublicKey)
 .execute(client);
const getReceipt = await newAccount.getReceipt(client);

This is a behaviour I am seeing as of today (post the upgrade yesterday as never happened before):
Using<EMAIL_ADDRESS>GrpcServiceError: gRPC service failed with status: TIMEOUT
at Timeout._onTimeout (...@hashgraph\sdk\lib\channel\NodeChannel.cjs:111:20)
at listOnTimeout (node:internal/timers:564:17)
at process.processTimers (node:internal/timers:507:7) {
status: GrpcStatus { _code: 17 }
}
calling smart contract methods (any even lightweight getters).

We are going to investigate it. Keep in mind that the issue might be in some of the Nodes, and nodes not being able to respond. Can you confirm that this is through Node client or Web client? I need to know which type of GRPCs are accessed?

node.js for me
If i retry it will work - worst I have had to do 2 retries to get it through.

We are going to investigate it. Keep in mind that the issue might be in some of the Nodes, and nodes not being able to respond. Can you confirm that this is through Node client or Web client? I need to know which type of GRPCs are accessed.

Node.js for me also.

Can you give me information about the node that you are trying to process through when the TIMEOUT is received?

I was not pinning it down to a single node. just instantiated via Client.forTestnet()

I was not pinning it down to a single node. just instantiated via Client.forTestnet()

Same but in my case  Client.forNetwork('mainnet');

There appears to have been some sort of brief GCP disruption, though they haven't reported any interruptions. But we did observe proxy checks on GCP proxies failing briefly around ~1100UTC, which lines up with when you posted. There doesn't appear to have been any impact on any actual nodes running in GCP.
This is what I've received from the Network team.

We have with this problems for almost 2 weeks, but we have detected is happening way more with v2.19.2

Good morning,
Yesterday after we launched 3 more Launchpads (One at 20 UTC) and 2 more at (23 UTC) , in 150 NFTs sent, we got 7 times TIMEOUT.

Version 2.19.2 did not change anything related to the transaction workflow and timeouts. I am investigating it at the moment. From Yesterday, 1000 NFT transfers and 1000 HBAR transfers are successfully processed.

I think that I've managed to isolate the issue. It is happening only with Node 10 with IP: <IP_ADDRESS>

@AdriMsM91 I've located the issue. Will work to be fixed. Thank you in advance for reporting it. 🙏

@ochikov Glad to hear that! Thanks you! <3

Just to let you know how we are going to fix it:
Let's say that you are hitting Node 10 which contains 3 different IPs. If one of the IPs is not responsible and we are not getting any error (timeout), we will wait some default time and then try to process the transaction through another Node. You now see the Timeout error because we are not marking this node as unhealthy. Now we will mark it, your execution will be awaiting to switch for a new node and try to process the transaction, BUT this transaction will not be discarded, but trying to process it. I will give you a beta release version when we are ready. Roughly estimated time for this fix will be tonight/tomorrow morning.

v2.20.0-beta.3 Should fix the issue.

Just wanted to note that the maxExecutionTime passed to setMaxExecutionTime should be specified in ms not seconds.

@mshakeg Yes, it is in milliseconds. All the timings in the SDKs are in milliseconds.


How do i use this

i do not know how to use this, and also i don't know what they mean by "run 'py -3 -m pip install requests'", i don't see the "py -3 -m pip install requests: file, and also i didn't use a webhook, so how do i install the module requests? i really need to know.

i do not know how to use this, and also i don't know what they mean by "run 'py -3 -m pip install requests'", i don't see the "py -3 -m pip install requests: file, and also i didn't use a webhook, so how do i install the module requests? i really need to know.

they mean open cmd and write this command run 'py -3 -m pip install requests' then it will ask yout to write another command then it will open

RTFM


Aggregate new samples to previous analysis

Hi, thanks for making FRASER open-source.
I ran FRASER on 250 samples and want to add new samples to the analysis.
How do you guys recommend running a new BAM file and aggregating the counts to the previous counts from the first project?
Sorry to ask in the issues session, but I couldn't find any e-mail for that.
Thank you!

Hi @geocarvalho,
When you add a new sample the following steps happen if recount=True:

FRASER reads the split_reads for already counted samples from cached data
FRASER counts the split_reads for newly added samples
FRASER merges the split_reads and creates an updated spliceSites object
FRASER counts the non_split_reads for all samples using the new spliceSites

Since adding new sample updates the spliceSites, We have to recount the non_split_reads for all samples.
We are thinking of optimizing this part and making it faster but unfortunately right now there is no other way and you have to recount the non_split_reads for all samples.
Best,
Ata

Hi @AtaJadidAhari, I have a question about whether I need to keep the BAM files for all the samples when I need to rerun the analysis with a new sample. In the documentation, there is a topic titled "Creating a FraserDataSet from existing count matrices", but I'm unsure how to create the junctionCts and spliceSiteCts, and whether it's necessary to have all the BAM files available.

Hi @geocarvalho ,
Unfortunately you need the bam files to count the non-split reads on the new junctions that will probably be added by aggregating new samples.
For future references, running snakemake  exportCounts --rerun-triggers mtime will create the required external files.

Thanks, @AtaJadidAhari. That is a lot of space in my case.
I'm unable to run DROP on my environment, so I was trying to use the FRASER script.


Return database status in /tenantinfo call based on HC max level status

https://github.com/ydb-platform/ydb/pull/5543


Logs files are filled with this error

I have got API Manager and API Manager Analytics installed with oracle database successfully running. As API's are used, i keep getting following error and entire log file gets filled with it. Not able to make much out of this. After day or two of running, server freezes and have to kill the application.
Any insights on this error?
ERROR {org.wso2.carbon.metrics.jdbc.reporter.JDBCReporter} -  Error when reporting timers {org.wso2.carbon.metrics.jdbc.reporter.JDBCReporter}
java.sql.BatchUpdateException: Internal Error: Underflow Exception trying to bind 1.3934662705180957E-131
at oracle.jdbc.driver.OraclePreparedStatement.executeBatch(OraclePreparedStatement.java:10070)
at oracle.jdbc.driver.OracleStatementWrapper.executeBatch(OracleStatementWrapper.java:213)
at org.wso2.carbon.metrics.jdbc.reporter.JDBCReporter.reportTimers(JDBCReporter.java:389)
at org.wso2.carbon.metrics.jdbc.reporter.JDBCReporter.report(JDBCReporter.java:200)
at com.codahale.metrics.ScheduledReporter.report(ScheduledReporter.java:162)
at com.codahale.metrics.ScheduledReporter$1.run(ScheduledReporter.java:117)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

One possible reason for this is that there might be a configuration issue in the metrics-datasources.xml file. Please double-check this file. There is a commented out section for oracle specific configuration. please check whether your configuration is similar to the commented out section

Metrics in APIM has disabled  and not in use from the 3.x release .


Inconsistent styling

Describe the bug
There seems to be inconsistent styling between the text field and prefix country selector (font/size/position) also there is no indication of the selector being clickable if we have set the selectorType to PhoneInputSelectorType.DIALOG, or am I missing something

Package version
^0.7.0+2
Flutter version
2.0.6

@Siqlain-Hanif sorry for the off-top, but tell please how have you surrounded all the field with border? Wrapped in another widget or it's built-in feature of the field?

And you can change style of the selector text with "selectorTextStyle" attribute.

I have also the same issue, is there any fix or workaround?

There is also an issue when you want to have different style (like border color) on input focus.


Add streaming conformer rnnt for Chinese

This PR is to add streaming conformer rnnt for Chinese ASR. I test it in my server.

Please create a copy of
https://github.com/k2-fsa/sherpa/blob/master/.github/workflows/run-streaming-conformer-test.yaml
say, run-streaming-conformer-wenetspeech-test.yaml
and change the following lines accordingly:
https://github.com/k2-fsa/sherpa/blob/12915bf094cff0e91a622319ec41cea007e70099/.github/workflows/run-streaming-conformer-test.yaml#L113
https://github.com/k2-fsa/sherpa/blob/12915bf094cff0e91a622319ec41cea007e70099/.github/workflows/run-streaming-conformer-test.yaml#L126
https://github.com/k2-fsa/sherpa/blob/12915bf094cff0e91a622319ec41cea007e70099/.github/workflows/run-streaming-conformer-test.yaml#L128

It seems there is a failure when go on checks:
Run ./sherpa/bin/streaming_conformer_rnnt/streaming_client.py \
Traceback (most recent call last):
  File "./sherpa/bin/streaming_conformer_rnnt/streaming_client.py", line [1](https://github.com/k2-fsa/sherpa/runs/7447149774?check_suite_focus=true#step:12:1)43, in <module>
    asyncio.run(main())
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/asyncio/runners.py", line 43, in run
    return loop.run_until_complete(main)
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/asyncio/base_events.py", line 587, in run_until_complete
    return future.result()
  File "./sherpa/bin/streaming_conformer_rnnt/streaming_client.py", line [12](https://github.com/k2-fsa/sherpa/runs/7447149774?check_suite_focus=true#step:12:13)4, in main
    await run(server_addr, server_port, test_wav)
  File "./sherpa/bin/streaming_conformer_rnnt/streaming_client.py", line 84, in run
    f"ws://{server_addr}:{server_port}"
  File "/opt/hostedtoolcache/Python/3.7.[13](https://github.com/k2-fsa/sherpa/runs/7447149774?check_suite_focus=true#step:12:14)/x64/lib/python3.7/site-packages/websockets/legacy/client.py", line 633, in __aenter__
    return await self
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/site-packages/websockets/legacy/client.py", line 650, in __await_impl_timeout__
    return await asyncio.wait_for(self.__await_impl__(), self.open_timeout)
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/site-packages/websockets/legacy/client.py", line 654, in __await_impl__
    transport, protocol = await self._create_connection()
  File "/opt/hostedtoolcache/Python/3.7.13/x64/lib/python3.7/asyncio/base_events.py", line 971, in create_connection
    ', '.join(str(exc) for exc in exceptions)))
OSError: Multiple exceptions: [Errno 111] Connect call failed ('::1', 6006, 0, 0), [Errno 111] Connect call failed ('1[27](https://github.com/k2-fsa/sherpa/runs/7447149774?check_suite_focus=true#step:12:28).0.0.1', 6006)
Error: Process completed with exit code 1.

Thanks!


Many buttons are overflowing on mobile in Portrait mode

Many buttons are overflowing.

On smartphone, Sugarizer should be used via the dedicated Android/iOS app.


Introduce a queue such that a Node thread is available for fs tasks

Should fix #100 and #119 but this needs testing.

This seems to fix #100 for me.

I've tested it in the context of #100 and it works for me. :+1:

I've also switched to this branch in the context of #100 and this seems to resolve the problem. :shipit:

Nice! :+1:
Thx @jorrit

Great! Solved. :+1:


[Documentation Improvement] Convert Cost Management data to FOCUS

⚠ Problem
Today, the Convert Cost Management data to FOCUS documentation for performing transformation is not 100% clear for  cost related columns.
BilledCost and EffectiveCost reference this footnote:

BilledCost should copy cost from all rows except commitment usage that has a PricingModel of “Reservation” or “SavingsPlan” which should be 0. EffectiveCost should copy cost from all amortized dataset rows; commitment purchases and refunds from the actual cost dataset should be 0.

I understand this as the following for BilledCost, but not sure if its correct.
IF (PricingModel = 'Reseration` OR PricingModel = 'SavingsPlan') {
  0
} else {
 CostInBillingCurrency
}

However, EffectiveCost does not give actual values that should be checked for proper calculation.  What is the criteria for knowing a row is amortized and if the row is "commitment purchase or refund"?
ContractedCost has the column as UnitPrice * Quantity and a description of a lookup required against the PricingUnits reference data.

Map UnitOfMeasure using Pricing units data file and divide Quantity by the PricingBlockSize

Is the proper equation UnitPrice * (Quantity / PricingBlockSize)?
🛠 Solution
Please clearly document the conversion of CostInBillingCurrency for BilledCost, EffectiveCost and ContractedCost so there is no guessing in how this is done.
🙋♀ Ask for the community

We could use your help:

Please vote this issue up (👍) to prioritize it.
Leave comments to help us solidify the vision.

We can clarify this. Your logic is correct for BilledCost.
As a starting point, you should copy all rows from the amortized cost dataset and only the commitment purchases and refunds from the actual cost dataset. To identify commitment discount purchases and refunds, look for ChargeType == "Purchase" and PricingModel == "Reservation" or "SavingsPlan".
For ContractedCost, your formula is also correct.

@brianwyka Would you mind taking a look at PR #1153 to see if that helps?

Thanks @flanakin .  I'll take a look at it today!
If I understand correctly, I need both the Cost and usage details (actual) and Cost and usage details (amortized) to correctly calculate those cost fields.
Is there any alternative with just the actual cost data?  If we only have the actual, can we only safely calculate ContractedCost and LiatCost?

If I understand correctly, I need both the Cost and usage details (actual) and Cost and usage details (amortized) to correctly calculate BilledCost and EffectiveCost fields.

Correct.

Is there any alternative with just the actual cost data? If we only have the actual, can we only safely calculate ContractedCost and LiatCost?

If you don't have any commitment discounts, then using actual cost alone is fine. If you do have commitment discounts, then you won't be able to show an accurate EffectiveCost value.

Btw, thanks for the review! I love your comments and depth in reviewing this! So awesome to see from the community! 😊
I responded to your comments with some additional suggestions. Take a look and let me know if it'd be better to walk thru this in a call. Happy to find some time.


Use ServiceToken and Kubernetes URL rather than kubectl proxy

It might be worth waiting for this -- https://github.com/kubernetes/client-go

I heard it was ok to use on 1.3 clusters.

Let's do it!

I can have a crack at it. What's your appetite?

If you want to see what you can come up with feel free, if not I can give it a try as well.


Authentication Not Recognized for 2 Factor Nintendo Users

First off thanks for this awesome add on! Enjoying it.
Im a developer by trade and just now getting into open source powershell and c# development, so I did some detailed analysis and might have time to take a stab at a fix myself if you’d like.
Describe the bug
Nintendo has issues honoring a full path post_login_redirect_uri url parameter when the user has two factor authentication enabled. It only reads the path and I guess hardcodes the domain, I ended up getting redirected to accounts.nintendo.com/my/ instead of the intended API domain ec.nintendo.com/my/.
This means that the token json file doesn’t get created since the login is recognized as successful based on the web view loading the redirect url.
A simple fix might be to just consider accounts.nintendo.com/my as the user being logged in in the web view. Or more thorough check the cookies on each web view change until some Nintendo auth ones are there.
To Reproduce
Enable 2 factor authentication on your Nintendo account. Reinstall the addon or delete the token file, try to authenticate with that account. Personally I tried authenticator and passkey based methods, I’m assuming this issue would be the same for an SMS based method too. Note page not found error and the addon not recognizing you are authenticated.
Workaround
As a workaround if I create an empty token.json file in the addon data directory and then restart playnite, everything WAI afterwards.
Diagnostics ID
Please generate diagnostics package from "About Playnite" menu and attach identifier you received. If the issue is a crash bug, you can generate diag. package directly from crash dialog.
f5a5149c-3a35-45ee-b848-f8838bfb51a0

Thanks for the detailed report, I'll try to implement at least the simple fix shortly and release it.

I've tried both workarounds, and they don't seem to work for me. The blank json doesn't seem to change anything, and I'm not able to go back with right clicking on the webpage when it asks to log in.

just to comment on this one. the workaround worked fine. just login with your 2FA or Passkey and then right click and click back a couple of times until you get to the ec.nintendo.com/my/ page and it will automatically close the window and log you in.
but sadly, even if it states its authenticated and logged in. it fails in the sync itself

I am also having issues getting this to work no matter what I do... any other workarounds?

I've released a new version that should work: https://github.com/XenorPLxx/playnite-library-nintendo/releases/tag/1.7
If you'll still have issues, you might need to go to plugins/installed/libraries/nintendo, click data folder and remove token.json, go to Playnite/settings/advanced, click clear web cache. Playnite will then restart and you should be able to log in successfully. During testing, I had to do that on my main Playnite, but for dev instances it wasn't necessary.

I've released a new version that should work: https://github.com/XenorPLxx/playnite-library-nintendo/releases/tag/1.7 If you'll still have issues, you might need to go to plugins/installed/libraries/nintendo, click data folder and remove token.json, go to Playnite/settings/advanced, click clear web cache. Playnite will then restart and you should be able to log in successfully. During testing, I had to do that on my main Playnite, but for dev instances it wasn't necessary.

Thank you! I can report that this procedure worked for me 😃

Same problem here and I'm using version 1.7. It worked for one connection after installation and now I can't authenticate anymore. It asks for password, 2FA and all, but won't change to "authenticated". I'm using a German Nintendo account if it matters. Workaround with empty token.json works though.

go to plugins/installed/libraries/nintendo, click data folder and remove token.json, go to Playnite/settings/advanced, click clear web cache. Playnite will then restart and you should be able to log in successfully.

Did not work for me. After using 2FA code the window closes, after a few seconds the plugin returns to saying Requires authentication

Yea seems like Nintendo broke something again, ill investigate


PolygonDef SetAsBox(....) Angle maybe radian?

test code
var b = world.CreateBody(new BodyDef { Position = new Vec2(0, 0), MassData = new MassData { Mass = 0 } });
var s = new PolygonDef();
s.SetAsBox(10, 10, Vec2.Zero, 30 * 0.01745f);
b.CreateFixture(s);

need the convert angle to radian, current i test meet expectation, i am correct?
not found need radian in SetAsBox params comment
thank

Hi, yes you need to convert radian to angle, please take a look on this answer [1] and see [2].
[1] https://stackoverflow.com/a/135930
[2] https://www.rapidtables.com/convert/number/radians-to-degrees.html

Hi, yes you need to convert radian to angle, please take a look on this answer [1] and see [2].
[1] https://stackoverflow.com/a/135930
[2] https://www.rapidtables.com/convert/number/radians-to-degrees.html

thank, need unite change angle comment to radian?
may misunderstand angle/radian param comment

Hi, yes you need to convert radian to angle, please take a look on this answer [1] and see [2].
[1] https://stackoverflow.com/a/135930
[2] https://www.rapidtables.com/convert/number/radians-to-degrees.html

thank, need unite change angle comment to radian?
may misunderstand angle/radian param comment

I don't think we need to change the parameter from angle to radian.
Thank you! :smiley:


Vscode 1.53.2(system setup) Javascript intellisense is not working on my windows 10

We have written the needed data into your clipboard This issue always occur whenever i want to use a javascript object or array propertise and methods. For instance:
const number = {1, 2, 3};
number. (after the dot, the propertise and methods of the object suppose to show but unfortunately is not showing so please help to fix this issue. Thanks!
About my pc:
model: Dell Inspiron 3558
Processor: Intel(R) Core(TM) i3-55015U CPU @ 2.10GHz
Installed Ram: 12.0 GB
system type: 64-bit operating system, x64-based processor
windows specifications:
Edition: Windows 10 Pro
Version: 1903
Installed on: 9/30/2019
OS build: 18362.10024
ard because it was too large to send. Please paste.

take a look on https://devhints.io/jsdoc
jsdoc is required when you work in js, also take a look in vscode guide.
https://code.visualstudio.com/docs/languages/jsconfig
You need a jsconfig or tsconfig (both work for js project)
for you case try .
/** @type {Object.<number, string>} */
const a = {1,2};

but const a = {1,2} it not a valide syntaxe in js
you need define key value in literal  const a = {1:void 0,2:void 0}


Run sar on on remote VM

Description
In this PR, I added an example of how to train SAR on remote compute resources with python script. It's relatively straightforward from existing notebook, just re-arrange python code. Two helper function were created to movielens.py util class.
This example shows how much and what kind of change is required in order to run a local training process as script on Azure Machine Learning.
notebooks that I referenced to:
sar_movielens.ipynb
img-classification-part1-training.ipynb

Related Issues
#424
Checklist:

[ ] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

Based on our discussions, i don't think we will want to have an AzureML folder. I think it might just be good to have a few examples of azureml sprinkled in since for most cases we will be able to submit the notebooks directly to AML when the papermill integration is done. For these two, we could name them sar_movielens_on_remote_vm and sar_movielens_on_amlcompute.  We should make sure to update it in the readme too that there are quickstart examples with AzureML as well.

Also, I don't think we should reference the outside configuration.ipynb to create the compute and then use this notebook for attaching compute.  That will not be a good experience for the user to go to a completely diffferent repo and have to walk through steps to createcompute. We should have those steps in this notebook.

hey @jingyanwangms, you don't have your github user configured correctly. Please take a look at https://github.com/Microsoft/Recommenders/wiki/How-to-add-your-name-as-a-contributor-to-the-repo#make-sure-that-your-github-user-is-setup-correctly

Left a couple comments above, but otherwise this is looking really good. Once those changes are made I think it is good to go! Great work :)

hey @jingyanwangms I still don't see the metrics in the notebook, can you see them?


Installing protractor hangs every one in three times.

I use
nvm use 4.2.3
npm install protractor -dd
Sometimes installations hangs. I found there is 3 cases. Success, hang on installOne, and hang on unlock.

Success:
the installation completed smoothly and under 1 minute.
Hang on unlock:
npm verb unlock done using /home/jchn/.npm/_locks/selenium-webdriver-552a20708a2c24b7.lock for /home/jchn/333/node_modules/protractor/node_modules/selenium-webdriver
It hanged for 15 mins! Installing selenium-webdriver alone doesn't has this problem.
Hang on installOne:
npm verb installOne of protractor to /home/jchn/333 not in flight; installing
In this case it hanged for about 30 secs and went smoothly and completed.

At first I thought the cases appears randomly, but found that the cases repeated in this order. It repeats with a period of 3. I means I install for 15 times, and It succeed, hang, hang, succeed, hang, hang... etc.
I am new to the node. The box is Ubuntu 14.04. Node version is 4.2.3.

I strongly suspect this is an issue with your particular computer. Angular uses npm to install Protractor on continuous integration several times a day and it's never an issue.
There's a monster issue which looks similar to this problem at https://github.com/npm/npm/issues/7862. Can you check that out and see if any solutions there work for you?
Closing because the problem is not with Protractor specifically.


Can't compile wgsl shaders that use the "modf" function

Bevy version
0.15.0
[Optional] Relevant system information
AdapterInfo { name: "AMD Radeon RX 6600 (RADV NAVI23)", vendor: 4098, device: 29695, device_type: DiscreteGpu, driver: "radv", driver_info: "Mesa 24.2.8", backend: Vulkan }
What you did
I tried to use the following shader:
#import bevy_pbr::forward_io::VertexOutput

@fragment
fn fragment(
    mesh: VertexOutput
) -> @location(0) vec4<f32> {
    let uv = mesh.uv * 40;
    let u_fract = modf(uv.x).fract;
    let v_fract = modf(uv.y).fract;
    let u = step(0.99, u_fract);
    let v = step(0.99, v_fract);
    return vec4(0.0, 0.0, 0.0, max(u, v));
}

What went wrong
2024-12-29T07:41:28.067671Z ERROR bevy_render::render_resource::pipeline_cache: failed to process shader:
error: failed to build a valid final module: Entry point fragment at Fragment is invalid
  ┌─ grid_material.wgsl:9:19
  │
9 │     let v_fract = modf(uv.y).fract;
  │                   ^^^^ naga::Expression [7]
  │
  = Expression [7] is invalid
  = Type resolution failed
  = Special type is not registered within the module

Issue with naga oil - it's not handling special types between modules. Same as what happens with ray query types.


Bug in nested tabs. [4.0.0-beta]

In nested tabs when switch parent tabs teveral times non-active tabs are showed
https://plnkr.co/edit/UNQUVV?p=preview

Fixed by : https://github.com/twbs/bootstrap/pull/23087
and will be available in our next release thanks 👍


We cannot fold on top of this native query. Please modify the native query or remove the 'EnableFolding' option

This happens when loading data in contoso sales.pbit on all queries except "Owner" table.
Changing the query to [EnableFolding=false] gives error Microsoft SQL: Unexpected end of file occurred. Line:47, Position:9
on query "Opportunity"

This happens when loading data in contoso sales.pbit on all queries except "Owner" table. Changing the query to [EnableFolding=false] gives error Microsoft SQL: Unexpected end of file occurred. Line:47, Position:9 on query "Opportunity"

Changed the parameter "monthsofhistory" from negative (-6) to positive (6).

I'll take a look this morning. Thank you for the info.

'Months of history' should always be a positive number. - I'll look at adding "Number.Abs()" around the value to force it to be positive if this becomes a common question.


Fix warnings on comparing value types to null

RT
looks like incorrect porting from Java to C# / from Hashtable to Dictionary.

LGTM


NPE when disable mockserver

quarkus.mockserver.devservices.enabled=false leads to a NPE because of the following line
https://github.com/quarkiverse/quarkus-mockserver/blob/6838e18b22138348e94cd8bfc3ab49426f9ecf0d/deployment/src/main/java/io/quarkiverse/mockserver/devservices/DevServicesMockServerProcessor.java#L126
The reason is:
https://github.com/quarkiverse/quarkus-mockserver/blob/6838e18b22138348e94cd8bfc3ab49426f9ecf0d/deployment/src/main/java/io/quarkiverse/mockserver/devservices/DevServicesMockServerProcessor.java#L132-L136

@darkeagle1337 thank you for reporting the NPE issue. It will be fixed by #51


Feature: Harden and release ntools version 1.6.0

Acceptance Criteria:

Review deployed documentation
Respond to issues found in getting-started repo.

This repo demonstrates the use of setting up a project and use ntools

Release version 1.6.0

A few notes regarding the docs:

In the setup page, the command below must be corrected (-tag):

ng -c settag tag 0.0.1

The section right below the command above has some redundancy in explaining folder structure
It would be better if for the section Add a new task, we improve the example by adding another task to existing example (right now the example is used for Add a new target so it might not be very easy to tell the difference between target and task)

A few notes regarding DevSetup:

DevSetup.ps1  is outdated, the location of install.psm1 has changed
DevSetup folder section doesn't mention apps-Ntools.json which is needed.
Add a new tool section mentions that you can install the apps using the CLI but that is already covered in the script above.
It would be helpful to suggest to modify the sample DevSetup.ps1 according to your project's needs

resolved in #59


[SPARK-22430][R][Docs] Unknown tag warnings when building R docs with Roxygen 6.0.1

What changes were proposed in this pull request?
Removed @export tag to get rid of unknown tag warnings
How was this patch tested?
Existing tests

Test build #87040 has finished for PR 20501 at commit 7db58db.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87040/
Test PASSed.

I'm/We're aware of this and to keep diff small and manageable for backporting fixes, it is preferable to avoid broad changes like this.
We can revisit when new releases are stable down, perhaps.

I see this too and Roxygen 6.0.1 seems common now. Is there any value to the tags? I don't think these removals can cause much merge conflict. It can go in master right?

It was not used in the current way we have the export and a change in 6.x started warning about this.
Jenkins is not running Roxygen 6 though. There are pending tasks to upgrade. IMO it's better to update our toolset and validate the doc output is correct officially and then remove these tags for the warnings. The warnings are harmless.
Historically a lot of changes get backported around a new releases (or RCs of releases) and it was a pain to hand merge every single commit.

Ack. thanks for the update @felixcheung @srowen Closing this.

@rekhajoshm feel free to follow up after we are through with 2.3.0, thanks!

@rekhajoshm @felixcheung since 2.3.0 was just finished, OK to merge?

can you reopen this PR and add [R] in the title?

ping @rekhajoshm ?

done @felixcheung thanks

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1267/
Test PASSed.

Test build #87949 has started for PR 20501 at commit 7db58db.

Test build #87949 has finished for PR 20501 at commit 7db58db.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87949/
Test PASSed.

merged to master


cannot open abd.log:permission denied

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows 10
scrcpy version: 2.1.1
installation method: windows release
device model: Redmi Note 11
Android version: 13

Describe the bug

i try to execute the program with the mobile connected but it gives this error

You don't have permissions to write to this location. This is not even scrcpy, but adb. Try to execute adb devices you should have the same message.
You must give access to that location (I don't know how your system prevents it).


There's a serious bug. I can't see the code

Perhaps you have heard the story of "The Emperor's New Clothes"?hahaha

have you tried doing the following
rm -rf node_modules, and then re-installing.
Failing that you should try restarting the internet.


Fix inconsistent styling of type links / input field descriptions

There are a few minor inconsistencies with how we're rendering type links / descriptions for connections and input fields.  These inconsistencies result in styles not being applied properly.
Connections / Input Fields:

In fields.html, we wrap the link in a <code> tag, whereas it's the other way around for connections and input fields.  This isn't apparent in the sample docs page @ https://www.gjtorikian.com/graphql-docs/, but the styles in my project are affected by it.

Input Fields:

In fields.html, we markdownify the field description which results in a <p> tag surrounding basic text.  For input fields, we use the raw description which causes the text to be squished vertically.  You can see it if you compare https://www.gjtorikian.com/graphql-docs/object/addcommentpayload/ with https://www.gjtorikian.com/graphql-docs/input_object/addcommentinput/

Eep! Thank you! Looks good and I'll bump a bugfix version out now.

:+1: Thanks for the quick turnaround on the merge!


Use dynamic_cast<> for pointers?

C-style cast on pointers always makes me nervous, for example the following code snippet.
https://github.com/NCAR/VAPOR/blob/040ecf6fd59eb883d4889ebb9a25e10359870d54/lib/render/WireFrameRenderer.cpp#L61
Would it make sense to introduce to our code standard to use dynamic_cast<> for such occurrences?

Probably.

On Jul 6, 2018, at 11:39 AM, Samuel Li<EMAIL_ADDRESS>wrote:
C-style cast on pointers always makes me nervous, for example the following code snippet.
https://github.com/NCAR/VAPOR/blob/040ecf6fd59eb883d4889ebb9a25e10359870d54/lib/render/WireFrameRenderer.cpp#L61
Would it make sense to introduce to our code standard to use dynamic_cast<> for such occurrences?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

John Clyne
National Center for Atmospheric Research
303.497.1236 (w),<PHONE_NUMBER> (c)
<EMAIL_ADDRESS>

We decided to add this rule to the code convention, as we can see https://github.com/NCAR/VAPOR/wiki/Vapor-Coding-Convention


feat(api): mask data from traces logged to langfuse

Supports https://github.com/langfuse/langfuse/issues/3518
Developer Experience
from langfuse.decorators import observe, langfuse_context

langfuse_context.configure(
  # example lambda to replace "home_address" from input and output
  mask=lambda data: {**data, "home_address": "masked"} if isinstance(data, dict) else data
)

@observe()
def process_pii(home_address: str):
    print("Hello, from the process_pii function!")

process_pii("777 this road")

langfuse_context.flush()

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks a lot for your work and for raising this PR @shawnzhu !
In the current state, this PR is not yet handling masking IO from StatefulClients. Rather than applying the mask on the Langfuse Client level, I'd suggest passing the mask down to the task_manager and apply the mask in the add_task method. You can check the event body there whether there are any to-be-masked dict keys and apply the mask there.

I'd suggest passing the mask down to the task_manager and apply the mask in the add_task method

Great idea. Let me try

@hassiebp may I ask for a review again?

Thanks a lot for your contribution, @shawnzhu! Will take it from here in #977 👍🏾


homematic component servicemessages count wrong

Sometimes the "Sensor" for CCU servicemessages are not reflecting the CCU variable state correctly, see attached screenshots. HA says 1 message, the CCU variable is 0. In addition I recognice that the sensor most of the time shows doubled values, for example the CCU has 3 servicemessages, the homematic component in HA shows 6. I am using the HA version 0.63.2
Is this a bug?

Sometimes the "Sensor" for CCU servicemessages are not reflecting the CCU variable state correctly, see attached screenshots.

Can you show us the sensor configuration? I have Homematic configured using Homegear and some time in the past I saw this sensor, but now it's gone.

@m0wlheld this is not configured as a standalone sensor, its an attribute of the Home Assisstant Homematic component integration.

Sorry to bother you again, but what is the entity_id of that sensor? As I said: I had it in the past but cannot find it with recent HA versions (0.63 and above).

In my case the entity_id is "homematic.ccu2" as seen in the attached screenshot. As you can see the problem still exists with my current HA version 0.64.3, because the attribute of the entitiy shows "3" messages, my CCU shows "1".

Thanks. I have no homematic.XXX sensor in my entity list, maybe because I use Homegear, not an Homematic appliance like CCU2. Not subject to this issue.

any news to this problem? @danielperna84 perhaps you could have a look? ;)

@m0wlheld
To get the CCU entity you have to add a host-block to your configuration. Something like this:
homematic:
  interfaces:
    rf:
      host: <IP_ADDRESS>
  hosts:
    homegear:
      host: <IP_ADDRESS>

Looking at this though there might be a problem though. It's not possible to customize the port, which might be necessary in some cases. It's always using 2001. But besides that you should get an entity where the state is the sum of service messages, and the variables should be accessible via the attributes of the entity.
@nicx
I have just tested it again. I detached a door-sensor, which causes a sabotage-servicemessage. This did show up instantly on the CCU, and on HA it took a couple minutes (the messages are polled every 5 minutes). I also detached a second one to see how it behaves with multiple messages. In every case the correct number of messages was displayed.
Does the value change back to the correct value if you restart HA? Additionally you could manually execute the following in a Python3 shell to do what HA is doing to get the number of service messages:
from xmlrpc.client import ServerProxy
p = ServerProxy("http://<IP_ADDRESS>:2001")
len(p.getServiceMessages)

Of course replace the IP in the example with the IP of your CCU.

@danielperna84 in the meantime I switched from my CCU2 to Homegear. With this everything is working as expected ;)
One question: Is it possible not only to count the service messages but to get the message text itself?

Is it possible not only to count the service messages but to get the message text itself?

In general this would be possible. Back then we created the implementation we couldn't find a proper way to implement this though. We can't create entities for each message, and the attributes of the hub-entity are already used for the variables (which are much more useful than the content of the service messages). So if you have a suggestion how they could be implemented we are open for that. Persistent notifications could perhaps be a solution. But some service messages automatically disappear, so this would need to be taken care of and programmatically remove such persistent notifications.
Another thing: the messages we get are cryptic. I don't have a real example right now, but a device with sabotage detection when the case is opened would have a message something like FOO_SABOTAGE_ERR and the device ID. It gives you a hint at what the error means, but it's not as nice as the full description you get from the CCU.

ok, thanks for clarification! great work btw ;)


Fix hash calculation

Resolves #59
The diff looks confusing because I've moved some things around, so I'll try to explain a bit.
There was a (pe_bits.size() <= cert_table_offset + 8) check near the end of the function which I don't think made sense there, so I moved it to before cert_table_offset is used as an index into pe_bits.
The main change is hashing everything past the header, which I'm doing without allocating another intermediate buffer, just feeding the appropriate data directly into EVP_DigestUpdate. That function cleanly returns an error if it's given 0 bytes, so no additional checks are necessary when hashing any leftovers at the end of the file.
I had to change the expected hashes in NoAuthTest because they were wrong. This can be verified by signing pegoat.exe with a test certificate, then checking it with signtool.
> signtool sign /fd sha256 pegoat.exe
Done Adding Additional Store
Successfully signed: pegoat.exe

> signtool verify /v pegoat.exe
Verifying: pegoat.exe

Signature Index: 0 (Primary Signature)
Hash of file (sha256): 6B7FA3E8298F33BC47F4ABB9C845930B1EACC0DAD96503CFA52D4EA18DDC89F0

Changes LGTM, CI is unhappy with some of the formatting:
diff --git a/src/uthenticode.cpp b/src/uthenticode.cpp
index 5a264d9..8a0a1aa [10](https://github.com/trailofbits/uthenticode/runs/6885050812?check_suite_focus=true#step:4:11)0644
--- a/src/uthenticode.cpp
+++ b/src/uthenticode.cpp
@@ -520,8 +520,7 @@ std::string calculate_checksum(peparse::parsed_pe *pe, checksum_kind kind) {
   EVP_DigestInit(md_ctx, md);
   EVP_DigestUpdate(md_ctx, pe_bits.data(), pe_bits.size());
-  if (security_dir.VirtualAddress > 0)
-  {
+  if (security_dir.VirtualAddress > 0) {
     /* If a certificate table exists, hash everything before and after it.
      */
     EVP_DigestUpdate(md_ctx,
@@ -534,14 +533,[11](https://github.com/trailofbits/uthenticode/runs/6885050812?check_suite_focus=true#step:4:12) @@ std::string calculate_checksum(peparse::parsed_pe *pe, checksum_kind kind) {
     EVP_DigestUpdate(md_ctx,
                      pe->fileBuffer->buf + security_dir.VirtualAddress + security_dir.Size,
                      pe->fileBuffer->bufLen - (security_dir.VirtualAddress + security_dir.Size));
-  }
-  else
-  {
+  } else {
     /* If there's no certificate table, just hash the rest of the file.
      */
-    EVP_DigestUpdate(md_ctx,
-                     pe->fileBuffer->buf + size_of_headers,
-                     pe->fileBuffer->bufLen - size_of_headers);
+    EVP_DigestUpdate(
+        md_ctx, pe->fileBuffer->buf + size_of_headers, pe->fileBuffer->bufLen - size_of_headers);
   }

Thought my editor would automatically find the .clang-format as usual, but it was silently failing. Oops.

Sorry for the delay here! This looks great, merging now. Thanks for all the work.


[bug]:  No platform for Windows & Linux shown in CLI

Is there an existing issue for this?

[X] I have searched the existing issues.

CLI Version
0.2.4
Firebase Tools version
11.4.2
Flutter Doctor Output
[√] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.22000.795], locale en-MY)
    • Flutter version 3.0.5 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (4 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

Checking Android licenses is taking an unexpectedly long time...[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\Iqmal\AppData\Local\Android\Sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = C:\Users\Iqmal\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

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

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.17)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.32630.194
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[√] VS Code, 64-bit edition (version 1.70.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.46.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.795]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 103.0.5060.134
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 104.0.1293.47

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Description
The flutterfire configure didn't show the Windows platform. However, the folder windows is available in my project.
Steps to reproduce

Create new Flutter project
Run flutterfire configure
Select an existing Firebase project

Expected behavior
The platform selection should show Windows & Linux
Screenshots

Additional context and comments
No response

Oops sorry, it is not supported yet is it?


CanAxe for tiledefs needs removing

This should be using flags or the likes rather than cluttering tiledefs with another field, see https://github.com/space-wizards/space-station-14/pull/18087

Got removed in #24432


Documentation & Cleanup

Coverage decreased (-0.09%) to 95.745% when pulling e67805e21d63959d2f461cf43ee5203faab72baf on feature/documentation_cleanup into 12bbadaa3431cdac0fff467186e4496ceb98bd71 on master.


Add SocketHandle::inner

Since Hash and Display are already implemented for smoltcp::socket::set::Handle, the numeric value of the handle is obtainable through these indirect means. I think it makes sense to expose a method to get this value explicitly.

I would specifically like to avoid exposing the internals of SocketHandle because this keeps the possibility of efficiency improvements in the future, e.g. tracking whether the socket was written to without O(n) iteration in poll().

Makes sense! How would you recommend passing it through the FFI boundary? Should it be treated as *const SocketHandle?


Invalidating an Input component?

I was wondering if there is recommended a way to show feeback to the user manually if the data provided in an input component is invalid ?
Show a red outline around the input and some error message for example.

There are no official recommendations as Shadcn is based on Radix.  You can use native HTML methods for validation such as onChange or pattern attribute

There should be a clear way if doing this. Not everyone uses React Hook Form...


Trigger Events/Observers Like saved, created, or updated

This is a feature request to fire off events for observers methods such as saved, created, or updated. As it is one must manually do the work of the observers or explicitly trigger the events each time.
Ideally this could also work with preemptive observer methods like saving, creating, updating, etc.  And it could also add a non-static upserting method like saveUpsert(array $update).

Please #2 why that's not possible.


[Youdao Translate] ...Error code 222

Extension
https://www.raycast.com/tiancheng92/youdao-translate
Description

Steps To Reproduce
type any text, return errors
Current Behaviour
No response
Expected Behaviour
No response

API Key error


Integration test for retry flow

Currently, retry is not tested, yet it has the potential to cause tricky issues.
For example - if external API is not available for a longer period, the exponential backoff strategy from sidekiq can make the next attempt to be scheduled in X hours/days. If we schedule a job with the same args what will happen?
Scenario to be tested:
Given we have EnqueueConflictResolutionStrategy defined as drop_self or replace and job in retry queue scheduled to be executed in X hours, what will happen when we try to schedule a new job with the same args.

Not sure if that's feasible to test but seems like it's not a problem - once job is executed at least once it's no longer subject of uniqueness checks - hence everything that ends up in the retry queue will not block freshly scheduled jobs


After a crash during unit testing, the app cannot start up #3669

Closes #3669

@oa-s bit delicate. Could you describe the cause of the problem and how the PR fixes it?

Does this create a separate Realm folder for unit tests?

@oa-s bit delicate. Could you describe the cause of the problem and how the PR fixes it?

The issue apears by next cause: after app launch we don't have created wallet, and we run tests, realm files created during tests stay in the document folder, and don't get deleted. On the next launch we can pass througn check
MigrationInitializer.hasRealmDatabasesForWallet && !keystore.hasWallets && !isRunningTests()
as we got db files and dont have addreses in keychain or user default.
including the next one
https://github.com/AlphaWallet/alpha-wallet-ios/issues/3669#issuecomment-1014208789

The issue apears by next cause: after app launch we don't have created wallet, and we run tests, realm files created during tests stay in the document folder, and don't get deleted.

Does it mean that if the user does this (before this PR):

Install app
Launch app
Kill app
Launch app

App will not start up?

@hboon  a bit different:

Install app
Launch app for tests (tests've passed or app has crashed)
Launch app
App stucks on launch screen

@oa-s so it could only happen if the test suite is run? It will not happen for normal users (app store)

@oa-s so it could only happen if the test suite is run? It will not happen for normal users (app store)

right


hot update 1.19

In 1.19 the structure has changed.

Adjusting the file structure for the new version

I already fixed this


Build error

I'm getting this build error:
./powerdns_exporter.go:260: e.counterVecMetrics[def.id].WithLabelValues(label).Set undefined (type prometheus.Counter has no field or method Set)
make: *** [powerdns_exporter] Error 2
Makefile.COMMON:97: recipe for target 'powerdns_exporter' failed

Google says that prometheus.Counter has been deprecated. Thoughts?

Specifically the Set() method for counter is deprecated. It seems we need to use Gauges now.


ropee deployment on kubernetes

Hi, thank you for your work on ropee!
We have ropee running in a kubernetes cluster, I do not know if you tried to do that before.
kubernetes requires a readiness / liveness probe, which needs to return 200 (or ropee will be killed by k8s)
We tried to use /metrics as probe, but that caused some issues, maybe because prometheus is using the same endpoint to collect metrics.
Have you ever tried ropee on k8s?
Do you expect /metrics to work as liveness probe call?
Do you have ideas about better healthchecks than just returning 200?
I can help with an upstream PR if you are interested.

@benoittoulme Hi, I have ever deployed ropee on Kubernetes, but no liveness probe added.
It is a good idea to add a /healthz endpoint for liveness or readyness probe.
Look forward your PR. ; )


Added Freepic icons resource.

Updated json file for frontend/icons category.

Review and merge @Anmol-Baranwal @CBID2

Did you raise this as an issue @Hemant-Lohar?

@CBID2 It's my bad I didn't follow all the guidelines provided in the contributions guidelines. I will correct it. Should I create a new pull request?

@CBID2 It's my bad I didn't follow all the guidelines provided in the contributions guidelines. I will correct it. Should I create a new pull request?

Not yet, just raise the issue and I'll review it. If it gets approved, I'll reopen your PR. If not, you can create a new one.

I had raised the issue it was #1993

I had raised the issue it was #1993 @CBID2

Great @Hemant-Lohar. Now just add it to the PR form. I highly recommend reading Writing Your First Pull Request: Tips, Best Practices, and AI-Powered Tools for Success to help you fill it out.

@Hemant-Lohar
Following the PR template is a must. It includes special words like 'fixes' or 'resolves' that connect your issue to the PR, as explained in the official docs.
If you don't link the issue correctly, even after the PR is merged, the issue might stay open, causing huge problems.
I'm doing it for you this time. Please ensure it next time.
Just a heads-up to help you link issue properly next time. Have a great day!

@Hemant-Lohar
Are you interested in contributing?
If not, please let me know so that I can close the issue.