Azure Policy Regulatory Compliance controls for Azure Data Lake Storage Gen2 - not working

Hi, i am CSS dev support engineer working for a cx case.
Cx is following the below documentation - https://docs.microsoft.com/en-us/azure/storage/blobs/security-controls-policy#hipaa-hitrust-92 for setting diagnostic logs audit policy on ADLS gen2 storage account.
while using the below template , it is working for gen1 account only for the Cx, but the public documentation states that it is for Gen2 -  "Azure Policy Regulatory Compliance controls for Azure Data Lake Storage Gen2"
https://github.com/Azure/azure-policy/blob/master/built-in-policies/policyDefinitions/Data Lake/DataLakeStore_AuditDiagnosticLog_Audit.json
Cx - " When we applied the built in policy mentioned in the json ,the policy is only showing compliance checkup for Gen1 resource.
The json file contains the policy rule as below.  Here resource showing as "Microsoft.DataLakeStore/accounts" but for gen2 it is a storage account. Please clarify."
"policyRule": {
   "if": {
   "field": "type",
  "equals": "Microsoft.DataLakeStore/accounts"
   },

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

ID: 871f0a7e-d3e0-3787-9482-91662e2b4e91
Version Independent ID: 3b0deb46-b3c8-b7e8-378d-2b575a1c7dd4
Content: Azure Policy Regulatory Compliance controls for Azure Data Lake Storage Gen2
Content Source: articles/storage/blobs/security-controls-policy.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@vsriram-binary  Thanks for the question! We are investigating and will update you shortly.

@DCtheGeek can you help with this? I'm unfamiliar with these controls.

@normesta The list is everything that maps to Azure Policy Regulatory Compliance for the Microsoft.DataLakeStore.  If that RP covers both "Gen1 and Gen2" or "only Gen1" we should probably update both the title of the page itself to properly reflect that as well as enhance the description of the definition itself.  I'll start an internal email and add you to it.

Ok thank you @DCtheGeek! I'll await final confirmation. @vsriram-binary - we will keep you updated with what we learn. Thank you for raising this to our attention!

@vsriram-binary We've confirmed that Microsoft.DataLakeStore RP is for Gen1, so this is both in the wrong TOC and has the wrong title.  I've submitted a PR to fix both issues.  Once that PR is merged, this issue will automatically close.  Publish to live will occur later.  Thank you for pointing this out!
@MicrosoftDocs/azure-cxp-triage Please remove label 'product-issue' and add label 'doc-bug'.  #in-progress

Thank you @DCtheGeek for the update. So for Customer's ask on diagnostic logs audit template for ADLS gen2 storage accounts - is it available?

@v-albemi Hi, can i know why this request is closed ? i was expecting a response from @DCtheGeek . Please advise.

@vsriram-binary
It appears that I merged the pull request, but it’s actually the commit that was merged in a different pull request. My understanding is that the commit exists in at 2 least PRs, and it was merged in one of the PRs that was signed off. (In  MicrosoftDocs/azure-docs-pr#134207)
This is a GitHub technical issue and not something that PR reviewers can manually watch for or prevent, unfortunately. And I don't have a complete grasp of the situation. The solution might be to be careful to work on an article in only one working branch, one level down from master. If you need more technical help with this, you can send email to azcontechat@microsoft or look for Docs support on Teams.
Thanks

@vsriram-binary Sorry! I didn't see your follow-up question and the PR I created was set to automatically close this GitHub Issue.  @v-albemi appears to have closed it, but that's because he merged my commit. :)
This issue should be closed as the incorrect page was updated, moved, and clarified to be specific to Gen1. If a built-in policy definition for Azure Storage does exist, it'll be documented in one of these places:

All Azure Policy built-in definitions
Azure Policy built-in definitions for Azure Storage

If the built-in doesn't exist, the recommendation is to build a custom definition. This tutorial is the best place to start.  Thanks!


rails db:migrate failing

Hi,
I am using
Rails 5.1.6
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-linux]
has_friendship-1.1.1

While trying to run migration,  it is giving bellow error
rails aborted!

StandardError: An error has occurred, this and all later migrations canceled:

Directly inheriting from ActiveRecord::Migration is not supported. Please specify the Rails release the migration was written for:

  class CreateFriendships < ActiveRecord::Migration[4.2]

Though it can be cleared by overriding all the has_friendship migration files by inserting appropriate rails  version like bellow
class CreateFriendships < ActiveRecord::Migration[5.1]
#
end

Was it missed from the gem or did I missed anything while installing gem?

Hi! I want to resolve this issue.

@khatanark, this is a duplicate of https://github.com/sungwoncho/has_friendship/issues/53


Hangs using OS 10.13.2

Speech begins and quickly hangs.

I'm experiencing the same issue :(

I have this issue on 10.13.3
Work around:

Open "Activity Monitor".
Search for com.apple.speech.speechsynthesisd and force quit.
Resend the text from your document to Dictate (no need to quit Dictate).

Interestingly, when following this work around, 90% of the time, you can resend the same text that caused it to hang in the first place and it will play just fine.
For the other 10% of the time it will hang again. But if you send a smaller amount of text it will work.
It's not clear what the limit on the amount of words you can send before it hangs. It almost seems to be random.

I guess my stuff is all in the 10%. Thanks anyway.
On Fri, Feb 23, 2018 at 11:41 AM, bigsmile01<EMAIL_ADDRESS>wrote:

I have this issue on 10.13.3
Work around:

Open "Activity Monitor".
Search for com.apple.speech.speechsynthesisd and force quit.
Resend the text from your document to Dictate (no need to quit
Dictate).

Interestingly, when following this work around, 90% of the time, you can
resend the same text that caused it to hang in the first place and it will
play just fine.
For the other 10% of the time it will hang again. But if you send a
smaller amount of text it will work.
It's not clear what the limit on the amount of words you can send before
it hangs. It almost seems to be random.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nosrac/Dictater/issues/19#issuecomment-368064156, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AiNdt4_bUxHb-TY8c8B4vvkn083LmZOnks5tXupMgaJpZM4RvVq8
.

The issue seems to be with the voice Alex. I deleted all voices from my Mac and then used Daniel (UK) and it seems to work now. If I reinstall Alex and try it, then it hangs again.

Sorry for the stupid-long delay in my reply.  I've updated the app for Catalina, let me know if it works for you now (assuming you still care!)

@Nosrac
It's like Christmas, I'm so thrilled Dictator has been updated. I am still on Mojave. But the new Dictator seems to work! Thanks for your efforts.


One-lane roads with straight and allowed turn-left traffic movements

I was wondering if CityFlow support the case that there are single lane in a three/four-way intersections in which turn-left is allowed. In other words, does it support the case that one lane can have both straight and turn-left traffic movements? If so, would you mind let me know how can I create the roadnet.json file?
I attached two drawing examples of the corresponding intersections:
four_roads.pdf
three_roads.pdf
Thanks in advance,
Afshin

Yes,  one lane can have multi movements in CItyflow.  You just need to define it as the "roadlink" features of "intersection", in roadnet.json;
For example, check this Atlanta roadnet https://github.com/traffic-signal-control/sample-code/blob/master/data/atlanta_1x5/roadnet_atlanta.json
Search for "startRoad": "9246225#4,  you can see this road has only one lane but can both go straight and turn left.

Thanks for clarification!.


CSV not found error?

Hi Champs:
I'm in the last miles to get it done.
May you guide me how to solve the CSV not found error?
Thanks.
BR Rio
rio@ubuntu:/opt/tensorflow/tensorflow/models/research$ python3 AIAlpha/run.py
Using TensorFlow backend.
Traceback (most recent call last):
File "AIAlpha/run.py", line 8, in
dataset, average, std = nnmodel(500, 0.01, 0.01)
File "/opt/tensorflow/tensorflow/models/research/AIAlpha/model_20_encoded.py", line 13, in nnmodel
train_data = np.array(pd.read_csv("60_return_forex/encoded_return_train_data.csv", index_col=0))
File "/usr/local/lib/python3.6/dist-packages/pandas/io/parsers.py", line 702, in parser_f
return _read(filepath_or_buffer, kwds)
File "/usr/local/lib/python3.6/dist-packages/pandas/io/parsers.py", line 429, in _read
parser = TextFileReader(filepath_or_buffer, **kwds)
File "/usr/local/lib/python3.6/dist-packages/pandas/io/parsers.py", line 895, in init
self._make_engine(self.engine)
File "/usr/local/lib/python3.6/dist-packages/pandas/io/parsers.py", line 1122, in _make_engine
self._engine = CParserWrapper(self.f, **self.options)
File "/usr/local/lib/python3.6/dist-packages/pandas/io/parsers.py", line 1853, in init
self._reader = parsers.TextReader(src, **kwds)
File "pandas/_libs/parsers.pyx", line 387, in pandas._libs.parsers.TextReader.cinit
File "pandas/_libs/parsers.pyx", line 705, in pandas._libs.parsers.TextReader._setup_parser_source
FileNotFoundError: [Errno 2] File b'60_return_forex/encoded_return_train_data.csv' does not exist: b'60_return_forex/encoded_return_train_data.csv'
rio@ubuntu:/opt/tensorflow/tensorflow/models/research$ find . -name "*data.csv"
./AIAlpha/60_return_forex/encoded_return_train_data.csv
./AIAlpha/60_return_forex/return_train_data.csv
./AIAlpha/60_return_forex/return_test_data.csv
./AIAlpha/60_return_forex/encoded_return_test_data.csv
./AIAlpha/features/autoencoded_corrected_data.csv
./AIAlpha/features/autoencoded_data.csv
./AIAlpha/features/autoencoded_train_data.csv
./AIAlpha/features/autoencoded_test_data.csv
./AIAlpha/preprocessing/test_data.csv
./AIAlpha/data_folder/train_data.csv
./AIAlpha/data_folder/training_data.csv
rio@ubuntu:/opt/tensorflow/tensorflow/models/research$

I believe there is code missing as there is no script to generate the files found in the forex_y and 60_return_forex.

@VivekPa
Some of the code appears to be is missing...

So the data folder is completely missing now?

Hi, yes the data sets were too large to push to github, but the data used was free sample tick data from Tick Data LLC. I don’t know if they still have it on their website though. I can upload a small section of the data if that would help?

@VivekPa thanks for getting back to us. That would be great, yes! Even if you could upload the trimmed down versions of the files, it'd be a great help because then we can just run the quickstart without doing any more work.
We can also figure out the schema easily, should we decide to plug in our own data and then of course gradually dive into the code. Otherwise, we have to go straight to the code to figure things out...

Hi, I have uploaded a trimmed version of the data with 1 million rows to play around.

@VivekPa  first of all thanks a lot for uploading the data. I now tried again to run the Quickstart (based on your README) and still got issues because it's looking for a file that doesn't exist.
sample_data/processed_data/price_bars/dollar_bars.csv
I guess that the file you uploaded helps but not fully because we still have to sift through all the code to figure out how to connect the dots. Could you please also update the project or the documentation in a way that we could just run it and produce some results? Then we'll take it from there...

Project is indeed lacking documentation. Also getting FileNotFoundError: [Errno 2] No such file or directory: 'data/processed_data/sample_data/processed_data/autoencoder_data/train_x.csv'


Running on Android

Hello, I am trying to run my custom trained model on android.
Can anyone give me some help? I have converted the Keras model to tflite.
But I don't know how to run it on Android, is there any basic (easy to understand) example Apps?
I tried to use the Tensorflow Object Detection Example App, but without luck.

Hi,
I am also facing this problem. Did you figure out any solution?

Hi, in another issue I described how I converted the model to tflite. On my PC it throws Error that TFLITE doesn't support BatchNormalization but on my Notebook and the same code everything worked just fine.
For Running it on Android I customized the Tensorflow Object Detction Example App.
If you have any Questions feel free to ask


Simpler require statement

Was thinking about a way of having a simpler require statement. We can change this:
var carbon = require('carbon-io')
var o  = carbon.atom.o(module).main
var __ = carbon.fibers.__(module)

to something like this:
const {carbond, o, __} = require('carbon-io')(module);

If we change carbon-io to export like this:
var core = require('@carbon-io/carbon-core')

module.exports = (mod) => {
  return {
    atom: core.atom,
    bond: core.bond,
    fibers: core.fibers,
    carbond: require('@carbon-io/carbond'),
    ejson: core.ejson,
    leafnode: core.leafnode,
    testtube: core.testtube,
    HttpErrors: core.HttpErrors,

    // added lines:

    o: core.atom.o(mod).main,
    __: core.fibers.__(mod)
  }
}

Not a perfect way of doing it - this is just a POC. I think it will make documentation and examples a little easier. Just have to add one line to examples and it's all runnable rather than having these rather bulky require statements.
Thoughts?

So @willshulman and I were talking about this. Will made the point that we should extend the function we export with all of the attributes the previous export had. That would look something like this:
const core = require('@carbon-io/carbon-core')

// The old way of importing Carbon was with an object
let carbonObj = {
  atom: core.atom,
  bond: core.bond,
  fibers: core.fibers,
  carbond: require('@carbon-io/carbond'),
  ejson: core.ejson,
  leafnode: core.leafnode,
  testtube: core.testtube,
  HttpErrors: core.HttpErrors
}

// New way to import Carbon is from a function
// This allows for some nice destructuring assignment
let carbonFunc = (mod) => {
  return {
    atom: core.atom,
    bond: core.bond,
    fibers: core.fibers,
    carbond: require('@carbon-io/carbond'),
    ejson: core.ejson,
    leafnode: core.leafnode,
    testtube: core.testtube,
    HttpErrors: core.HttpErrors,
    o: core.atom.o(mod),
    _o:core.bond._o(mod),
    __: core.fibers.__(mod)
  }
}

// We add the object's properties to the function for backwards compatibility
// TODO: Remove at a later date
for (let attr in carbonObj) {
  carbonFunc[attr] = carbonObj[attr]
}

module.exports = carbonFunc

This would make this change backwards compatible. We'd be able to use the destructured assignment but it wouldn't break all of the code not using destructured assignment. This would give us some time to switch over all of the examples etc.
The one issue here is it would change the namespaces for the class types. E.g. if you require your carbon modules like this:
const {carbond, o, __} = require('carbon-io')(module);

As opposed to:
var carbon = require('carbon-io')
var o  = carbon.atom.o(module).main
var __ = carbon.fibers.__(module)

The carbon namespace is dropped. carbon.carbond.Service becomes carbond.Service. Do we want to say this will be the new way of referencing class types with the destructured assignments?
This would be a breaking change. If we wanted to make it non-breaking, we'd have to add something like carbon: carbonObj to the object returned by carbonFunc.


removed unused variables

to avoid warnings

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.959%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2224

Relevant Lines:
2367

💛  - Coveralls


Update submodule dmlc-core

Description
This PR advances the 3rdparty dmlc-core submodule by the following 2 commits:
ca9f932  2019-11-05  Dick Carter            Fix initial-release nvcc v10.1 issue with thread_local.h (#579)
05fb5ad  2019-10-24  Philip Hyunsu Cho      Make yarn.py and launcher.py compatible with Python 3 (#564)

Importantly, this pulls in dmlc-core PR https://github.com/dmlc/dmlc-core/pull/579, which is the work-around for MXNet issue https://github.com/apache/incubator-mxnet/issues/16612.  Once this is merged, an MXNet build against the initial release of cuDNN v10.1 will not fail.  The version of dmlc-core included as a submodule of 3rdparty/tvm will not change of course (it depends on the tvm submodule hash), but this is not causing a compile problem for MXNet.
@larroy
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[X ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

preferrably this should also go in 1.6 if we are still taking bug fixes.


DOMException [BUG]

Describe the bug
I am getting this error a lot on flutter web.
TypeError: Cannot read property 'setAttribute' of undefined     at handleMessage (about:srcdoc:64:68) DOMException: Failed to execute 'setStart' on 'Range': The offset 5 is larger than the node's length (1).     at t.value (http://localhost:/assets/packages/html_editor_enhanced/assets/summernote-lite.min.js:2:24950)     at t.value (http://localhost:/assets/packages/html_editor_enhanced/assets/summernote-lite.min.js:2:29853)     at t.value (http://localhost:/assets/packages/html_editor_enhanced/assets/summernote-lite.min.js:2:80888)     at t.value (http://localhost:/assets/packages/html_editor_enhanced/assets/summernote-lite.min.js:2:81569)     at HTMLDivElement.summernote.keyup (http://localhost:52648/assets/packages/html_editor_enhanced/assets/summernote-lite.min.js:2:80592)     at HTMLDivElement.dispatch (http://localhost:/assets/packages/html_editor_enhanced/assets/jquery.min.js:2:43090)     at HTMLDivElement.v.handle (http://localhost:/assets/packages/html_editor_enhanced/assets/jquery.min.js:2:41074)     at Object.trigger (http://localhost:/assets/packages/html_editor_enhanced/assets/jquery.min.js:2:71513)     at HTMLDivElement.<anonymous> (http://localhost:/assets/packages/html_editor_enhanced/assets/jquery.min.js:2:72108)     at Function.each (http://localhost:52648/assets/packages/html_editor_enhanced/assets/jquery.min.js:2:2976)
To Reproduce
Steps to reproduce the behavior:

Type something
Click on any button related to the font
click to type again
See error

Screenshots
getting an error when trying to type something like this:

Device:
chrome (web)
Additional context
Honestly, it seems like nothing working as expected. For example, I am also getting errors when trying to upload an image too. But that is for another issue.

What version of the package are you using? This is the first time I've seen those errors but it is working for everyone else on Web. The Type error shouldn't matter too much, don't worry about that one.

After wrapping the scaffold with GestureDetector (I saw this in your implementation) the errors gone. But I still can't upload an image. I guess I will report this later. Thanks though.

But I still can't upload an image.

Sure whenever you get a chance please let me know what errors you are getting.

I think it is related to the focus of the editor.

Quite possible, though I can't control it because it is from a library (summernote) that I use.


Enable gitlab login with gilab integration

What does this PR do?:
Switches the sign in page to gitlab provider, if gitlab integration is configured. Otherwise keeps github sign in.
Which issue(s) this PR fixes:

PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

[ ] Tested and Verified

[ ] Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:

Will we run into conflicts if both RHDH_GITHUB_INTEGRATION and RHDH_GITLAB_INTEGRATION are enabled? All of the other fields we update have the explicit .github or .gitlab naming but signInPage does not?

gitlab wins if both are present

Will we run into conflicts if both RHDH_GITHUB_INTEGRATION and RHDH_GITLAB_INTEGRATION are enabled? All of the other fields we update have the explicit .github or .gitlab naming but signInPage does not?

gitlab wins if both are present, but only because it gets configured last. for some reason it's one xor the other

Okay, can we add the required lines you need to set to the manual sections I linked above and add a snippet letting users know that if both integrations are enabled GitLab is going to win out? Seems like a Backstage/RHDH limitation that we should note?

Just one small comment I thought of when running through the workflow myself. Also since we will be supporting GitLab login with this can we also remove this notice?
https://github.com/redhat-ai-dev/ai-rhdh-installer/blob/main/README.md?plain=1#L4

@jrichter1 https://github.com/redhat-ai-dev/ai-rhdh-installer/pull/9 was also merged so we will need a rebase as well


Restart esp32

 64x64 4 panel. Poeer supply 5v 30A,  restart esp32, is not working properly. Connect wifi ntp and openweather, draws on matrix. Restart 3-5 times for day. Push restart esp32 module,  working ok.

Which panels are those please?
What does it do to require a restart? Totally stop running?

Can you capture the serial output?

May be a Power supply issue, consider 10-12A per panel
Best regards

Which panels are those please?
What does it do to require a restart? Totally stop running?

the controller restarts itself, and shows how the photo is pinned. you need to press restar to display OK

https://github.com/dmb-220/RGB-matrix

A couple of things to test:

Use of xTaskCreatePinnedToCore for TaskORAI and pin to core 1

use https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/freertos.html#_CPPv427uxTaskGetStackHighWaterMark12TaskHandle_t to find the right stack size for that task or just ramp it up a bit

use the latest version of Arduino ESP32

Does it restart if you do not have the LED matrix panels connected and turned on? You can still run the sketch and ESP32 without the LED matrix panels of course.... just keep a watch of the serial output.
If it still crashes without the LED panels then you a memory issue, else you have a power issue.
Either way, this isn't an issue caused by this library.


When a volunteer has a RedCall access, render his action structures

Currently, when someone has access to RedCall, a small RedCall logo appears.

We should be able to see which structures he is able to trigger by clicking on it (open a modal).

Done already in #265


Preprocessing

Add a preprocessing function like in Capse. The function should take as input an array with the cosmological parameters and the growth factor.

Closed by #37


[WIP] arjs-nft-init-data event

⚠ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠
What kind of change does this PR introduce?

Can it be referenced to an Issue? If so what is the issue # ?
This PR add a new CustomEvent arjs-nft-init-data to get basic nft data (id, dpi, width, height) for better repositioning the NFT marker
How can we test it?

test the nft example (threejs)
Summary

Try to solve the positioning of the model onto a NFT marker.
Does this PR introduce a breaking change?
Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser
tested on desktop (Ubuntu 18.04) for now
Other information
More to come

@kalwalt whats the steps needed to test this? It's been stuck as a PR for a while and I'm hopefully getting some free time which I can help to test this.
From the number of tickets still being raised by this not present, it would be a benefit to all to pull together and get this through and on main branch.

@JamesBotterill unfortunately i had no time to continue on it. I Hope to end this in a near future. 🙂

Is there any further updates on this?

Is there any further updates on this?

@JamesBotterill I should come back to this, I'm working a bit on the ES6 feaure, but this is also a must, although i think it will not complitely solve the issues.

@kalwalt no problem, at least I know I can reply with a little more detail and the multitude of dupes of the original issue.

@JamesBotterill worked a bit today, I think i will make some simpler examples without the gltf models. Testing with them take more time. More to come...

@kalwalt often with A-Frame, a common pattern is that the event is emitted by the entity that generates it.
e.g.
this.el.emit('eventname', data);

Closing this because the old code, i prefer starting from a fresh dev branch.


no read() data is received

"react-native": "0.76.5",
"react-native-bluetooth-classic": "^1.73.0-rc.12",
Device Arduino bluetooth HC-06
The command is successfully sent and executed but afterwards .available() returns zero and .read() returns nothing. It blocks my device because the device itself keeps the output in the buffer and doesn't accept new commands until is cleared. So I have to use another 3rd party program that reads the output buffer and then I can send commands with this plugin again. I tried both with .read() and .onDataReceived()
Here is a sample of my code
Reading method:
const performRead = async (device) => {
    try {
      console.log("Polling for available messages");
      let available = await device.available();
      console.log('There is data available [${available}], attempting read');

      if (available > 0) {
        for (let i = 0; i < available; i++) {
          console.log(`reading ${i}th time`);
          let data = await device.read();
          console.log(`Read data ${data}`);
          console.log(data);
        }
      } else {
        console.log("No read data available");
      }
    } catch (err) {
      console.log(err);
    }
  };

Main method:
const connectAndSendData = async () => {
    try {
      console.log("connecting");
      let device = await RNBluetoothClassic.connectToDevice(
        selectedDevice.address
      );
      if (await device.isConnected()) {
        console.log("device", device);
        const readInterval = setInterval(() => performRead(device), 5000);
        const write = await device.write(
          createCommand(1)
        );
        await new Promise((resolve) => setTimeout(resolve, 20000));
        console.log("disconecting", disconnect);
        clearInterval(readInterval);
        const disconnect = await device.disconnect();
        console.log("disconnected", disconnect);
      }
    } catch (err) {
      if (await device.isConnected()) {
        const disconnect = await device.disconnect();
        console.log("disconnected in catch", disconnect);
      }
      console.log("error", err.message);
      setError(err.message);
    }
  };

The majority of the time the issue with Arduino is the delimiter used for reading and writing.  Please ensure that you're matching the delimiter between the two.   You're connecting to the device without any parameters, which means the default delimiter is going to be \n make sure the writing to Arduino is not \r\n.
https://github.com/kenjdavidson/react-native-bluetooth-classic/blob/7e5c889670c56fa87954aee3baff9001e21f5568/android/src/main/java/kjd/reactnative/bluetooth/conn/StandardOption.java#L43
Other than that, you'll need to perform the debugging yourself.  My development on this project is limited and I've been relying on the community for assistance in maintaining since I no longer work with this library on a daily basis.

That helped in fact. The problem was in fact the delimiter.
Anyone having problem with no data in .read() with Arduino devices is because Arduino uses no line endings. You basically have to do
let device = await RNBluetoothClassic.connectToDevice(
        selectedDevice.address,
        {
          delimiter: "",
        }
      );

and voila, it works.


Update AI Pattern GPT-RAG alert page

Overview/Summary
Update GPT-RAG alert page
Tagging alerts in relevant services' alerts.yaml file for GPT-RAG
Update AI Landing page GPT-RAG section with link
This PR fixes/adds/changes/removes

Replace me
Replace me
Replace me

Breaking Changes

Replace me
Replace me

As part of this Pull Request I have

[ ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@Akiliu0512 - You need alt text on the following link:
/github/workspace/docs/content/patterns/artificial intelligence/rag/_index.md:14:1 MD045/no-alt-text Images should have alternate text (alt text)


chore: format Stepper style, update test snap

Coverage remained the same at 32.377% when pulling 0fa003cb153778cd94589ff9757ee89ac0cfef13 on lduoduo:stepper into e801d920937f7aaa767697c24d27e9feba950690 on ZhonganTechENG:dev.

Coverage remained the same at 32.377% when pulling 0fa003cb153778cd94589ff9757ee89ac0cfef13 on lduoduo:stepper into e801d920937f7aaa767697c24d27e9feba950690 on ZhonganTechENG:dev.


Make sure to turn reason into string when skipping test

A custom Test loader map pass the an Exception object for reason.
nosetests custom TestLoader does this.
You can see an example project where we see the bug: https://github.com/piyushg91/broken-teamcity-messages where it

Closing pull request cause I just realized my grammar is bad. Will create a new one


fix: correct dependabot update spam behaviour

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

This PR fixes the dependabot spam while keeping a small subset of features for security purposes.
Related Tickets & Documents

closes open-sauced/open-sauced#1388
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@bdougie we can still use security updates from the settings pane:

Don't have access to those settings so it's on you past this PR! 🍕


Using BM25 and word embeddings throws division by zero error

Hello @davidmezzetti - thanks for a cool package.
I am having some issues using BM25, please see the sample below:
from txtai.embeddings import Embeddings

embeddings = Embeddings({
    "path": "GoogleNews-vectors-negative300.magnitude",
    "scoring": "bm25",
})

embeddings.index(
    [(uid, text, None) for uid, text in enumerate(docs)])

Will create the following error:
RemoteTraceback                           Traceback (most recent call last)
RemoteTraceback:
"""
Traceback (most recent call last):
  File "/Users/dennisyurkevich/.pyenv/versions/3.7.8/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/Users/dennisyurkevich/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/vectors/words.py", line 57, in transform
    return (document[0], VECTORS.transform(document))
  File "/Users/dennisyurkevich/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/vectors/words.py", line 129, in transform
    weights = self.scoring.weights(document) if self.scoring else None
  File "/Users/dennisyurkevich/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/scoring/base.py", line 112, in weights
    weights.append(self.score(freq, idf, length))
  File "/Users/dennisyurkevich/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/scoring/bm25.py", line 29, in score
    k = self.k1 * ((1 - self.b) + self.b * length / self.avgdl)
ZeroDivisionError: float division by zero
"""

The above exception was the direct cause of the following exception:

ZeroDivisionError                         Traceback (most recent call last)
<ipython-input-29-b4da4f34aa72> in <module>
      7
      8 embeddings.index(
----> 9     [(uid, text, None) for uid, text in enumerate(docs)])
     10
     11

~/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/embeddings/base.py in index(self, documents, reindex)
     94         with tempfile.NamedTemporaryFile(mode="wb", suffix=".npy") as buffer:
     95             # Load documents into database and transform to vectors
---> 96             ids, dimensions, embeddings = transform(documents, buffer)
     97             if ids:
     98                 # Build LSA model (if enabled). Remove principal components from embeddings.

~/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/embeddings/transform.py in __call__(self, documents, buffer)
     55
     56         # Transform documents to vectors and load into database
---> 57         ids, dimensions, batches, stream = self.model.index(self.stream(documents))
     58
     59         # Check that embeddings are available and load as a memmap

~/.pyenv/versions/3.7.8/lib/python3.7/site-packages/txtai/vectors/words.py in index(self, documents)
    111             with tempfile.NamedTemporaryFile(mode="wb", suffix=".npy", delete=False) as output:
    112                 stream = output.name
--> 113                 for uid, embedding in pool.imap(transform, documents):
    114                     if not dimensions:
    115                         # Set number of dimensions for embeddings

~/.pyenv/versions/3.7.8/lib/python3.7/multiprocessing/pool.py in next(self, timeout)
    746         if success:
    747             return value
--> 748         raise value
    749
    750     __next__ = next                    # XXX

ZeroDivisionError: float division by zero

Removing bm25 from scoring will resolve the issue.

Hello, thanks for reaching out and the nice words on txtai.
I've updated the documentation to clarify this - word vector models require building the scoring index (in this case bm25) before the embeddings index.
For example:
from txtai.embeddings import Embeddings

embeddings = Embeddings({
    "path": "GoogleNews-vectors-negative300.magnitude",
    "scoring": "bm25",
})

# This line added to build the scoring index
embeddings.score(
    [(uid, text, None) for uid, text in enumerate(docs)])

embeddings.index(
    [(uid, text, None) for uid, text in enumerate(docs)])

Thanks @davidmezzetti all good now!


Vercel is not recognizing my LangchainRouter

I should be able to hit my End Point "/chat" on Vercel.
Scenario

I deployed my python application on Vercel

Actual result

When I try to hit my End Point on Vercel Project it returns as "404 Not Found". It looks like Vercel is not mapping this End Point.
Expected result

When I hit my End Point I should receive a stream of text.
Acceptance criteria

[ ]

@Gabrielmtvp just to confirm, are you able to access the endpoint locally?
I have not tried deploying on vercel (which can be a good adition to deployment options for the docs), can you please provide a step-by-step process of how you tried to deploy your app on vercel?

@ajndkr Yes, if I run locally the API works fine.
Vercel has an option where you can give permission to your repository and then they deploy automatically using your repository on Github. I used this option.
Other interest point is that I created another simple route using @app.get("/") method from FastAPI and that one Vercel mapped with succesfull.

@Gabrielmtvp can you please try again? I've recently released v0.8 which has some significant changes.
You can view the LangChainRouter documentation here: https://lanarky.ajndkr.com/learn/adapters/langchain/router/


Namespace mixup in string-range

The string-range pointers have XPaths that reference the tei namespace. This namespace is not defined - not on the Spine file and not in the chunk file (both have the default namespace as the TEI namespace, but a parser can't figure this out).
I am going to automatically remove all tei: references from string-range expressions for now, I think you should do the same in these files.
This, actually, is an interesting problem. If an xpath is written in one files but references another, which namespaces are used in the xpath? The ones in the file where the xpath is written, or the ones in the file the xpath references? I would expect it to be the namespaces in the referenced file.

On investigating this interesting question, we took a look at the way we're declaring namespaces in the spine files and in the target files from S-GA. They're related because in both, the TEI is the default namespace, lacking a prefix declaration. We think that means we can and should remove the tei: prefix and are proceeding to do so now...
Variorum Spine Namespace Declaration:
<TEI xmlns="http://www.tei-c.org/ns/1.0"
     xmlns:pitt="https://github.com/ebeshero/Pittsburgh_Frankenstein"
     xmlns:mith="http://mith.umd.edu/sc/ns1#"
     xmlns:th="http://www.blackmesatech.com/2017/nss/trojan-horse">

S-GA Files Namespace Declaration:
<surface xmlns="http://www.tei-c.org/ns/1.0"  xmlns:mith="http://mith.umd.edu/sc/ns1#"
  xml:id="ox-ms_abinger_c56-0045" ulx="0" uly="0" lrx="3847" lry="5342"
  mith:shelfmark="MS. Abinger c. 56" mith:folio="21r" partOf="#ox-frankenstein_volume_i">

@zmbq @raffazizzi

Addressed with freshly updated spine data to remove the tei: namespace prefix from string-pointer xpaths to SGA files:  https://github.com/PghFrankenstein/fv-data/commit/759e6d0a8fd27a133cc967f25de051c92634c84a
Let's hope this works.

I'm leaving this open until we're sure we've definitively resolved whether we're better off without the namespace prefix.

OK, this is a bigger problem. I couldn't resolve any XPath queries against the TEI. Turns out you can't match elements in the default namespace -you need to give the same namespace another prefix, and query with that prefix. This can be seen here https://docs.microsoft.com/en-us/dotnet/standard/data/xml/xpath-queries-and-namespaces (under the title The Default Namespace) and here http://www.edankert.com/defaultnamespaces.html.
So what we need to do is reintroduce the tei namespace to the XPath expressions, and also add it to the document as another namespace. So the TEI element should look like:
<TEI xmlns="http://www.tei-c.org/ns/1.0" xmlns:tei="http://www.tei-c.org/ns/1.0" xml:id="fMS-C09">

Everything else stays the same.

I think it's always Lupus.

Lupus or not...I'm redoing the namespaces now. First the spine, then I'm writing an identity transform to pop the namespace into the variorum-chunks.

I just need the xmlns:tei attribute on the root element, don't change anything else inside (or we may have issues with our working-with-no-styling-yet visualization code).

Understood!

Hello @zmbq! @ebeshero  and I have been talking about this and we'll go ahead and add the declaration as you requests. But if we want to make this tool as general as possible, in some (most ?) cases, pointers will refer to documents we have no control over, meaning that it wouldn't be possible to add the prefix declaration to the source.
Here's a suggestion on how to fix this. Since the XML document needs to be consumed in order to resolve the XPath, it's possible to add a namespace prefix declaration for the default namespace in case it's not there already. The prefix would match the one in the XPath, or would be set via a parameter on instantiation if we don't want to trust the XPath directly.

I don't think this is a bug in the code, I think it's a problem with the inputs. XPaths don't resolve default namespace elements. People should supply proper XPath experssions - so they shouldn't refer to default namespace elements, but rather add an explicit namespace (which can be in addition to the default namespace).
The documentation should clearly state, however, that the XPath expressions are resolved in the context of the target document - so the namespaces defined there are used. This can be confusing, because the s are in their own XML document with its own namespaces.

The spine files are updated with the tei: prefix back in place on the string-range pointers in fv-data now!
One of the readings you posted showed that a parsing tool could be reaching for namespaced elements with its own idiosyncratic prefix (say, to reach for TEI elements with a twinkie: prefix), while the source XML defined a different prefix entirely (tei:). I wonder how arbitrary these are--it seems most important to assert the namespaced condition of elements in the pointing/parsing tool. I wonder how necessary it is to set in the XML being interpreted...(Meanwhile, that's my next step...)

I'm not suggesting there's a bug, I'm offering a feauture request :)
But I still don't understand. You're saying that 1. the XPath expressions must use namespace prefixes because they can't rely on default namespaces, but you're also saying that 2. the target document must have the namespace prefix declared there as well in order to be resolved.

is easily enforced and we can make sure people supply proper XPath expressions. But we cannot enforce 2 because we won't always have control of the documents we target. Most XML documents will use default XML namespaces and use prefixes only for external vocabularies if at all needed.

What I'm suggesting is this:
input XPath: foo.xml#string-range(//tei:TEI, 0, 2) foo.xml: foo`
Before resolving the XPath, add declaration to the root (this is possible because we already have a copy in memory of foo.xml before applying the XPath)
DOM: <TEI xmlns="www.tei-c.org/ns/1.0" xmlns:tei="www.tei-c.org/ns/1.0">foo</TEI>
Now the XPath should resolve correctly.

In Real Life, of course, we do whatever we must for XPath to work!

@raffazizzi , what you suggest is problematic in two aspects - a conceptual one and a real-world one. The conceptual one is that you allow people to give you inconsistent data, and you try to fix it for them. People should just give us consistent data.
The real world one is more important in my opinion, and that is - we have no idea which namespaces to add to the target document. The XPath expressions can contain any number of namespaces. If it has three namespaces, how can we decide which one is the default one?  The XPath expressions just has the namespace names, not their URI. So we'll need to guess, which will let to very hard to figure out errors, when unsuspecting users suddenly realize we're matching against the wrong elements.

One of the readings you posted showed that a parsing tool could be reaching for namespaced elements with its own idiosyncratic prefix (say, to reach for TEI elements with a twinkie: prefix), while the source XML defined a different prefix entirely (tei:). I wonder how arbitrary these are--it seems most important to assert the namespaced condition of elements in the pointing/parsing tool, but the prefix you decide to use doesn't seem to matter. I wonder how necessary it is to set a prefixed namespace in the XML being interpreted...(Meanwhile, that's my next step...)

I'm initializing the XPath processor with the namespaces in the target document (the original chunk, not the spine), so these are the namespaces that should appear in the XPath expressions.
Before this tool is available for general use, we will need to build a utility that validates all the data, and tells the user what's wrong in case something is wrong. This utility will need to go over all the pointers and make sure they are valid references. Otherwise, nobody is every going to be able to use this.

@zmbq, the idea is to create a tool that is able to target any XML on the web -- are we going to go around telling people to adjust their data so we (not them) can use it? Or are we going to work with what's out there? Furthermore: it's not bad practice to not add an xml prefix when you have a default one! The XPath implementation you're using should be able to work with the default namespace instead.
I take your point about the real world: it's dangerous to add a namespace prefix based solely on the XPath string. But this can be easily parametrized via an option at instantiation or what have you.

Sorry for my delay on implementing the prefixed namespace in the root element: I'm trying to implement this in an XSLT at the last stage where we output the variorum-chunks, and I'm running into interesting XSLT problems to do with declaring a prefixed namespace in addition to the default namespace. To see what I'm talking about (maybe this isn't a problem, just an oddity), take a look at this example root element from my output for a variorum-chunk file:
<TEI xmlns="http://www.tei-c.org/ns/1.0" xmlns:pitt="https://github.com/ebeshero/Pittsburgh_Frankenstein" xmlns:mith="http://mith.umd.edu/sc/ns1#" xmlns:th="http://www.blackmesatech.com/2017/nss/trojan-horse" xmlns:ns0="http://www.tei-c.org/ns/1.0" ns0:tei="http://www.tei-c.org/ns/1.0">

@zmbq, the idea is to create a tool that is able to target any XML on the web -- are we going to go around telling people to adjust their data so we (not them) can use it? Or are we going to work with what's out there? Furthermore: it's not bad practice to not add an xml prefix when you have a default one! The XPath implementation you're using should be able to work with the default namespace instead.
I take your point about the real world: it's dangerous to add a namespace prefix based solely on the XPath string. But this can be easily parametrized via an option at instantiation or what have you.

I'm not sure I understand your use-case - someone is doing variations of someone else's documents? In that case, as I suggested a couple of weeks ago, they should make copies of those documents (because they will need to update their variations whenever there's a change), and they can add the correct namespace to that copy.
We can do as you suggest, but I'm quite certain it will cause a lot more problems than it will solve. Anyway, this is not for next week obviously.

Sorry for my delay on implementing the prefixed namespace in the root element: I'm trying to implement this in an XSLT at the last stage where we output the variorum-chunks, and I'm running into interesting XSLT problems to do with declaring a prefixed namespace in addition to the default namespace. To see what I'm talking about (maybe this isn't a problem, just an oddity), take a look at this example root element from my output for a variorum-chunk file:
<TEI xmlns="http://www.tei-c.org/ns/1.0"
xmlns:pitt="https://github.com/ebeshero/Pittsburgh_Frankenstein"
xmlns:mith="http://mith.umd.edu/sc/ns1#" xmlns:th="http://www.blackmesatech.com/2017/nss/trojan-horse
 xmlns:ns0="http://www.tei-c.org/ns/1.0" ns0:tei="http://www.tei-c.org/ns/1.0">

See those last two at the bottom? I don't know how those will work in real life (as in, I know that's not what you asked for...) so I'm trying something else that @raffazizzi suggested!

Now I have a mission for Utrecht! Convince you to stop using XSLT and use a proper programming language instead!  I know TEI people love XSLT, but you really shouldn't. XSLT was probably designed by Satan.

@zmbq Actually our very own project is a use-case, as our early goal was to process SGA's XML unmodified from its original source. There are all kinds of research possibilities for processing XML that wasn't home-cooked and pre-processed in your own space, and if we did research that way, we wouldn't be complaining so often about "silos" right?

(Trying Raff's strategy now for getting that blasted prefixed namespace in place...)

w00t! Okay we've got it working now--thanks @raffazizzi ! The root elements on variorum-chunk files will look like this now:
<TEI xmlns="http://www.tei-c.org/ns/1.0" xmlns:pitt="https://github.com/ebeshero/Pittsburgh_Frankenstein" xmlns:mith="http://mith.umd.edu/sc/ns1#" xmlns:th="http://www.blackmesatech.com/2017/nss/trojan-horse" xmlns:tei="http://www.tei-c.org/ns/1.0">

It'll be just a few minutes more until I've got the variorum-chunks updated for fv-data.

@zmbq Okay! Here are the variorum-chunk files with the new tei: prefixed namespace in addition to the default TEI namespace defined on the root element, as requested, in this commit:
https://github.com/PghFrankenstein/fv-data/commit/4df4b2a28f7f63a3cb518830d0905e7a29d8fd15

@ebeshero , you have updated all the files except the MS chunks that the Spine files point to.  Unfortunately I am now quite certain we will not be able to handle pointers to the MS edition by the end of this week. We are proceeding with the four other variants, and if we have time we'll get back to this. I suggest you plan your presentation as if MS is not there, at most - you'll be presently surprised.

The MS files generated as a result of my pipeline went into their old locations in reseq-MS-chunks. They are not S-GA files, and I am afraid I concentrated on getting the pointers correct. As it is now Trivially Easy to correct the SGA files like the others I ask that you give me a few minutes to correct the matter.
Elisa
Sent from my iPhone

On Jun 30, 2019, at 8:16 AM, Itay Zandbank<EMAIL_ADDRESS>wrote:
@ebeshero , you have updated all the files except the MS chunks that the Spine files point to. Unfortunately I am now quite certain we will not be able to handle pointers to the MS edition by the end of this week. We are proceeding with the four other variants, and if we have time we'll get back to this. I suggest you plan your presentation as if MS is not there, at most - you'll be presently surprised.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@zmbq I have now corrected the namespace line on the MS chunks that the Spine files point to, that is, those in variorum-chunks. I apologize for the oversight which came from me attempting to find a robust place to plant the prefix change in my postcollation processing pipeline, and ask that you please attempt to work with the SGA files now.


"CÂMPIA TURZII" is listed in the drop-down menu

Description:
"CÂMPIA TURZII" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "CÂMPIA TURZII" in the search bar.
Expected results
The location is listed in the drop-down menu as "Oraș CÂMPIA TURZII, CLUJ (CÂMPIA TURZII)".


[WFLY-11409] Fix base metrics descriptions

JIRA: https://issues.jboss.org/browse/WFLY-11409

superseded by #11884


1st draft of HIVE QC documentation for all Schemas

Steph will give Joe base documentation to work with in Jan

complete


Add splunk index name to grains

This is a pretty splunk-specific feature but would be very useful for the topfile features in #106. Team-specific targeting and the like.

https://jira.corp.adobe.com/browse/CSTTEAM-41387


Version and author(s) fields should be at the end of help text

When reading help text I'm usually more interested in how to use the command than in author/version/license information.
It would be great if author and version info was grouped together with license at the end.

Agreed :+1:

Though I agree, you can simply override this in your own app.. We've also updated it to also hide commands that have "cmd.HideHelp: true" set, but it should give you the right idea.
 cli.AppHelpTemplate = `NAME:
248    {{.Name}} - {{.Usage}}
249 USAGE:
250    {{.HelpName}} {{if .Flags}}[global options]{{end}}{{if .Commands}} command [command options]{{end}} {{if .ArgsUsage}}{{.ArgsUsage}}{{else}}[arguments...]{{end}}
251    {{if len .Authors}}
252 AUTHOR(S):
253    {{range .Authors}}{{ . }}{{end}}
254    {{end}}{{if .Commands}}
255 COMMANDS:
256 {{range .Commands}}{{if not .HideHelp}}   {{join .Names ", "}}{{ "\t" }}{{.Usage}}{{ "\n" }}{{end}}{{end}}{{end}}{{if .Flags}}
257 GLOBAL OPTIONS:
258    {{range .Flags}}{{.}}
259    {{end}}{{end}}{{if .Copyright }}
260 COPYRIGHT:
261    {{.Copyright}}
262    {{end}}{{if .Version}}
263 VERSION:
264    {{.Version}}
265    {{end}}
266 `

@frankcarey Thanks for the great example!  I'm adding it to the docs :smiley_cat: (https://github.com/codegangsta/cli/pull/373)


Backend class, run.py refactoring

Changes (in progress)

[X] Abstract base class for backends
[X] Logger implementation for local testing
[x] Selinon backend stub

Other changes:

[X] Refactor run.py into utils
[X] Mock testing for utils

[test]

[test]

[test]

[test]

[test]

Since CI is dead, this is the result from run-tests.sh:
Name                           Stmts   Miss  Cover   Missing
------------------------------------------------------------
ghmonitor/__init__.py              0      0   100%
ghmonitor/backend.py              30      6    80%   15, 21, 24, 33, 36-37
ghmonitor/gopkg/__init__.py        0      0   100%
ghmonitor/gopkg/translate.py      45      2    96%   30-31
ghmonitor/models.py               68      3    96%   71-72, 74
ghmonitor/monitor.py             120     15    88%   79-86, 97-108, 128
ghmonitor/utils.py                26      1    96%   20
tests/__init__.py                  0      0   100%
tests/test_monitor.py             20      3    85%   273, 275, 279
tests/test_utils.py               47      0   100%
------------------------------------------------------------
TOTAL                            356     30    92%

Required test coverage of 40% reached. Total coverage: 91.57%

[test]

[test]

@msehnout Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-5

@msehnout Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-5

@msrb I think the PR is ready for merge, I don't want it to grow even further. The selinon backend is far from ready, but there is a plenty of new tests. Pydoc and Pylint are failing because they are not set properly. See PR #7 for their fix.


Ticker info missing fiftyTwoWeekHigh and fiftyTwoWeekLow stats

Info about your system:

yfinance version - 0.1.87
operating system - Ubuntu 22.4

Simple code that reproduces your problem
`
import yfinance as yf
msft_ticker = yf.Ticker("MSFT")
print(msft_ticker.info)
`

The error message
above code used to give fiftyTwoWeekHigh and fiftyTwoWeekLow in JSON format with some other stats data till Dec 16, 2022. But since Dec 17, 2022, it suddenly stopped giving this information

seems to be related to the same issue reported in #1246


Can not disconnect slave, need help

async fn main() -> Result<()> {
{
let mut ctx1 = get_ctx(&get_default_sp_builder(), 2)?;
let rsp1 = ctx1.read_holding_registers(1, 2).await?;
println!("{rsp1:?}");
ctx1.disconnect().await?;
}
let mut ctx2 = get_ctx(&get_default_sp_builder(), 1)?;
let rsp2 = ctx2.read_holding_registers(1, 2).await?;
println!("{rsp2:?}");
ctx2.disconnect().await?;

Ok(())

}
above is my code, the rsp1 is right, but failed at the ctx2 row
What is the right method? thanks!

Context? Fails how?

Context? Fails how?

there are two contexts, the ctx1 works normal, then the ctx1.disconnect().await? is not take effect, so the ctx2 panic because access is denied.

get_ctx() and get_default_sp_builder() are not provided by tokio-modbus and I cannot comment on this. Everything might happen behind the scenes.
Please re-open if you could provide a runnable or at least comprehensible example.


Update Japanese translation and toc.yml

Changes

Fixed the wrong equations of Japanese translation of/algorithm-design/variational/variational.ipynb.
Updated toc.yaml to release two translation files of algorithm-design.

Implementation details

How to read this PR

Screenshots

@0sophy1 thank you!


Unmute on scroll

When the sink is muted (as opposed to unmuted with 0 volume), scrolling up doesn’t unmute. I’m using pipewire and wireplumber.

You can use on-scroll-up and on-scroll-down like so:
"wireplumber": {
        "on-scroll-up": "wpctl set-mute @DEFAULT_AUDIO_SINK@ 0; wpctl set-volume @DEFAULT_AUDIO_SINK@ 1%+",
        "on-scroll-down": "wpctl set-mute @DEFAULT_AUDIO_SINK@ 0; wpctl set-volume @DEFAULT_AUDIO_SINK@ 1%-"
}


Create AcdpParamsConverter and ACDP composition method

ACDP params conversion methods for both MPC and direct measurements are created
Rename class Charge to DpCharge. This also helps differentiate DpCharge from PrivacyBudgetLedger.charge
Move Noisers to package org.wfanet.measurement.eventdataprovider

Rally task link: TA662

This PR only covers 1 and 2. A lot of files are touched because of the renaming Charge to DpCharge


Controlling 3rd party devices

Can we use this API to access and control, through a google action, 3rd party devices which are added to Smart Home for a user , given account is linked. Or does it restrict me to only control devices which I have added/developed?
Detailed explanation:
I am building an action on Google Assistant which is basically a smart home assistant residing within google home.
Sample invocations and commands:

Nitto please turn on the Geyser(3rd party device)
-Nitto turn off the light in all rooms
I can do these with Ok google, turn on the Geyser
but the idea is to make it more personal and letting user use some of the regional dialects (powered by training on dialogflow).

I am stuck in controlling the device rest all flows around NLP part is already sorted. Please provide me some pointers on how to go about it.

The Google Assistant is already a smart home assistant and supports a variety of languages and locales already.
The smart home API is designed to let a service be accessed and controlled through the Google Assistant, and it does not give a service any indication of other services that the user has connected. Basically, Philips may have your smart lights, but it has no knowledge of your Roomba.
You can kind of use the Assistant SDK to act as a way to control all of a user's devices by sending commands to the Assistant on their behalf, although you aren't going to get the same level of detail and control (you won't know which devices they have or have direct control over them.)

Thanks for the reply. From what i found, Assistant SDK only allows to embed google assistant on custom built hardware. Correct me if I am missing anything.
Just to confirm, the use case is:
Controlling a Philips smart light with a custom command (e.g. Nitto light on karo) given to a custom agent(e.g. Nitto) built on Dialogflow.
Philips smart light is currently added in my Google home app and working with Google home - 'Ok google, turn on the light' command
How should I go about it?

If you want to control your Philips light from your custom Dialogflow agent, you'll need to do your own work of connecting to the Philips API.

Thanks for helping 👍


chore: improve loading behaviour add artists you collect

This PR resolves ONYX-185
Description
This PR improves the loading logic when a user is adding artists to their collection.

User dark modal spinner as we do for other surfaces
Show green toast of success because the query is so fast 😄 (good problem to have)

https://github.com/artsy/eigen/assets/11945712/184add1b-b10e-49e2-b1d3-1a32e523639b

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

improve loading behaviour add artists you collect - mounir

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (improve loading behaviour add artists you collect - mounir)

  Generated by :no_entry_sign: dangerJS against a570b6ddd1afe18d010eec1b12577a97cb339bec

🚀 🚀 🚀


cq:lastModified excluded from XF content path

PR for issue: https://github.com/Cognifide/gradle-aem-plugin/issues/452


Applications on custom datasets are expected

Now I'm not sure what data format I need to prepare to train this method

Hi @834799106,
Thank you for your interest in our work. Currently, we have only released the evaluation code along with the pre-trained models. We will be releasing the training instructions/code in a month or so.
Furthermore, as shown in Table 2 of the paper, MDef-DETR works well on many off-the-shelf datasets. You can try generating class-agnostic detections on your custom dataset and it's likely that you will get reasonable results.

Hi @mmaaz60 , is there any update on the training code status?

Hi @mmaaz60 , is there any update on the training code status?

Hi @nikky4D, we are planning to release the code by the end of this month. Thank you for your patience and stay tuned!

Any update on the training code status

Any update on the training code status

Hi @nikky4D, the code is now released. Please have a look at https://github.com/mmaaz60/mvits_for_class_agnostic_od/releases/tag/v1.0


update Babel to the latest version

Babel has been updated to the latest version to avoid issue when compiling. (#1261)

Thanks for the PR :-)
I'm not sure why, but this has the same issue as #1170, namely that in case there is no exports or define (such as a browser), the code that loads it all changes from
(function (global, factory) {
  global.parsley = factory(global.jQuery);
})(this, function ($) {

to
(function (global, factory) {
  global.parsley = factory(global.jQuery);
})(undefined, function ($) {

So instead of doing window.parsley = factory(window.jQuery) (since this at the top level of browser is window), the new code does undefined.parsley = factory(undefined.jQuery) which won't work...

help needed

Ok, I'm looking into this.

I've spent hours so far trying to update this. It is really painful.
Progress so far: #1267

Merged in master


[NPM] teste2e WKIT use recommended commandline to install chocolatey

What does this PR do?
Use recommended command line to install chocolatey, this may avoid CI to fail when installing chocolatey
Motivation
Avoid PR failing like here
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


spring.config.import configuration option doesn't support Spring Retry

Spring Cloud Consul Config always supported Spring Retry and there's a special consulRetryInterceptor bean in RetryConfiguration class (part of ConsulAutoConfiguration class).
Spring Boot 2.4 introduced new property - spring.config.import which we can use to specify Consul Config client settings:
spring:
  config:
    import: "consul:localhost:8500;app"

But it seems new approach doesn't support Spring Retry by two reasons:

consulRetryInterceptor bean loaded after spring.config.import configuration had been processed
There's no @Retryable methods in this new approach. ConsulPropertySourceLocator (which contains @Retryable methods) is not used

The first issue can be resolved by creating custom high-order EnvironmentPostProcessor. But the second issue seems unsolvable for now.
Please advice on how to use Spring Retry and spring.config.import together.

yes, this was left out do to timing. It was done in spring cloud config so it is possible.

yes, this was left out do to timing. It was done in spring cloud config so it is possible.

Thank you, @spencergibb
So the only way now to use Spring Retry in Spring Cloud Consul Config is to use legacy configuration with bootstrap.yml?

Thank you, @spencergibb
So the only way now to use Spring Retry in Spring Cloud Consul Config is to use legacy configuration with bootstrap.yml?

for now, using bootstrap will get you retry support.

for now, using bootstrap will get you retry support.

for now, using bootstrap will get you retry support.

Great. Can you please update documentation that Spring Retry is not supported with spring.config.import option now?
I was under impression it is supported and spent a lot of time on the investigation.
https://docs.spring.io/spring-cloud-consul/docs/current/reference/html/index.html#spring-cloud-consul-retry

for now, using bootstrap will get you retry support.

Great. Can you please update documentation that Spring Retry is not supported with spring.config.import option now?
I was under impression it is supported and spent a lot of time on the investigation.
https://docs.spring.io/spring-cloud-consul/docs/current/reference/html/index.html#spring-cloud-consul-retry

I can't update the docs since they are part of the release, but I will add something to the known issues section of the release notes

I can't update the docs since they are part of the release, but I will add something to the known issues section of the release notes

Note added here https://github.com/spring-cloud/spring-cloud-release/wiki/Spring-Cloud-2020.0-Release-Notes#known-issues

Note added here https://github.com/spring-cloud/spring-cloud-release/wiki/Spring-Cloud-2020.0-Release-Notes#known-issues

@spencergibb  Thank you for the fix.
Do you have estimation for 3.0.2 release date?

https://github.com/spring-cloud/spring-cloud-release/milestones

https://github.com/spring-cloud/spring-cloud-release/milestones

Hi, can i say that after the fix, this is resolved. I am currently 2020.0.3, but yet the retry still not working.

@hanct I'm not sure I understand. This issue was to support retry with spring.config.import with consul. Either it works or it doesn't.
If you'd like us to spend some time investigating, please take the time to open a new issue and provide a complete, minimal, verifiable sample (something that we can unzip attached to the new issue or git clone, build, and deploy) that reproduces the problem.


Update Windows/Mac usage of sync_package.py to Python 3

Currently used python binary causes this script to be run with Python 2 (default on Mac). This patch updates to use Python 3.
cc @areusch @driazati @Mousius for reviews

Makes sense to me 😸  thanks @leandron!


Strange behavior using IE browser

While being able to show graphs without any specific issue on Opera, Firefox, Chrome, Safari when the same code is executed on Internet Explorer Browser the graph has missing parts (CSS problem?) like in the example attached.
Any hint on where the issue could come from?

I looked into this and it seems to be a CSS problem, yes.  When I manually edit the  container of the charts and change it from (for example):
<svg class="nvd3-svg" width="100%" height="450">

to:
<svg class="nvd3-svg" style="width:100%; height:450px">

then it renders correctly.  I guess I would recommend using the style attribute rather than width/height attributes?  note that this is the same problem as #200 experiences

Not sure if it is the same but your proposal would be to change where the height is given ?
In the  ???

Yes, I was able to solve this by changing at angular-nvd3.js#L170 to
d3.select(element[0]).append('svg')
   .style({height: scope.options.chart.height+'px', width: scope.options.chart.width || '100%'})
   .datum(data)
   .transition().duration(scope.options.chart.transitionDuration)
   .call(scope.chart);

I'll submit a PR this evening when I'm not on a hobbled work pc.

Thanks guys for your investigation into this. I also added some fixes into your PR. The issue will be fixed in the new release (1.0.3)


Bug report - Invalid APK latest release

Describe the bug
The last released version fails with "Invalid" argument.
Tried with Obtanium, FR-Droid, NeoStore and direct download.
To reproduce

Expected behavior
Successful installation
Screenshots
See above.
Desktop (please complete the following information)
No response
Smartphone (please complete the following information)
Pixel 8 - GrapheneOS -<PHONE_NUMBER>
Additional context
No response

Yes, I did after the first failed update.

This is weird. I just tried to install LocalSend-1.15.3-android-arm64v8 and it worked on my phone

There was a build number change from 1.15.1 to 1.15.2 when using APK. Just uninstall the old version

I would do it if I would have the app installed, but what you see in the last screenshot with the app uninstalled.
It is somehow that the installer still detects the app installed when is obiously not.

Problem solved.
I had 2 different user accounts in my Pixel and the other account had Localsend as well. After uninstalling the application in both user accounts, I could install the latest version.


feat: Tool class

Related Issues

fixes https://github.com/deepset-ai/haystack/issues/8166

Proposed Changes:
Introduce a new Tool dataclass, that can store a tool specification and allow the invocation of the tool.
How did you test it?
CI, new tests.
Notes for the reviewer

minimal: the Language Model only needs the tool spec; invoke is added to handle tool invocation in our framework
in the future, we will introduce other more convenient/automated ways of creating a tool: starting from a function, etc.

On typing of parameters
I tried to introduce something like this
class Property(TypedDict):
  type: str
  description: str
  enum: NotRequired[List[str]]

class Parameters(TypedDict):
  type: str
  required: List[str]
  properties: Dict[str, Property]

Unfortunately, mypy complains if you do:
params = {"type": "object", "properties": {"city": {"type": "string"}}, "required": ["city"]}

tool = Tool(name="", description="", parameters=params, function=lambda x:x)

Argument "parameters" to "Tool" has incompatible type "dict[str, Collection[str]]"; expected "Parameters"

To fix this, you should explicitly do params: Parameters = {...}.
For this reason, in the current PR, I did not introduce typing for parameters, but we can also decide otherwise.
TBH, I want to provide flexibility, but I do not expect many users to manually input the JSON schema, especially if we provide convenient methods.
On validation

I'm only validating the JSON schema: if the schema is invalid, we will get an error from the LM API
several consistency checks could be done between the function and the schema
we could validate kwargs used to invoke the Tool against the schema

In general, when we will introduce more powerful constructors (e.g.: @from_function class method), I would expect most of the users to use them. So it can be ok to provide more freedom to people manually building this object from scratch.
Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
I documented my code
I ran pre-commit hooks and fixed any issue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

76 of 76 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 91.541%

Totals

Change from base Build<PHONE_NUMBER>7:
0.3%

Covered Lines:
2121

Relevant Lines:
2317

💛  - Coveralls


Always use require_once

The usage of require() and require_once() throughout the system is inconsistent, and "bugs" come up now and then when the order of "requires" is a bit different and some require() happens after the include file is already included/required.
It seems to me that there is no harm at all in always using require_once().

Thanks @phil-davis. Merged this to all 3 branches.
Where we use include I guess it's just a diff in that it won't fail if the included file is not found. Every use of it seems to be where it wouldn't matter much if the included file were missing. include_once should be fine. I don't see any uses of that where require_once would be preferable.


[FEAT]: Some very small changes in the UI.

What would you like to see?

The login image has very high resolution and some of the text has also been added at the bottom of the image in our case. when logged in the image looks odd on the top of the screen (in case of the mobile). Please add this small change to add another image that can be showed after logging in.

Additionally, if the image of the LLM bot in the chat can also be changed, it would be nice. although the current one also not so bad.

The Favicon.ico should also be allowed to change with the Text for the docker implementation.

The support email should be allowed to be changed as I don't want people from my organisation to fill up your in-boxes if they have forgotten their id-passwords 😆

each workspace can have one custom message so that people using that workspace for the first time can be told what to expect and how to chat.

more independence to make small changes in application .env for the people using docker desktop.

I would like to add the following small change in the interface. Please give the option in the system preferences to remove the "github", "docs" and "discord" icon. Or at least be able to change the links of those icons, in order to point the people of the organization to local information.

This has been live since https://github.com/Mintplex-Labs/anything-llm/pull/694 was merged

The login image has very high resolution and some of the text has also been added at the bottom of the image in our case. when logged in the image looks odd on the top of the screen (in case of the mobile). Please add this small change to add another image that can be showed after logging in.

This is something that can be revisited in the future. Right now it is a low priority but I can understand your concern there.

Additionally, if the image of the LLM bot in the chat can also be changed, it would be nice. although the current one also not so bad.

#706

The Favicon.ico should also be allowed to change with the Text for the docker implementation.
Since we compile the entire public folder into the server folder inside the docker container, this can be tricky for us to support since the favicon is not linked outside the container. Again, this falls to a lower priority then some of the other items outlined

The support email should be allowed to be changed as I don't want people from my organisation to fill up your in-boxes if they have forgotten their id-passwords 😆
#707

each workspace can have one custom message so that people using that workspace for the first time can be told what to expect and how to chat.
Merged with https://github.com/Mintplex-Labs/anything-llm/pull/664

more independence to make small changes in application .env for the people using docker desktop.
You can edit the .env in the bound docker storage already! Any changes made will take effect on container restart!


Provide --generate-hashes option to include hashes in requirements.txt output

As described in https://github.com/pypa/pipenv/issues/3057#issuecomment-589495763, emitting requirements.txt and then running pip-sync is much faster than running pipenv sync directly.
This speed increase comes from the fact that pip-sync is correctly checking for already installed packages, while pipenv sync doesn't yet do that.
However, the workaround has a problem, which is that if the pip-sync operation does need to install anything, then that installation won't check the artifact hashes, as those hashes aren't included in the requirements.txt output.
Supporting a pip-compile style --generate-hashes option to pipenv lock --requirements would allow the export to pip-sync (and other tools) to keep the expected hash information.

The documentation seems to suggest that by default pipenv lock --requirements would output hashes.  I would also like to see all the related hashes for all ranged and explicit version specifiers.

That's probably easy to resolve:

diff --git a/pipenv/core.py b/pipenv/core.py
index 361e21ee..ba265bc8 100644
--- a/pipenv/core.py
+++ b/pipenv/core.py
@@ -798,8 +798,10 @@ def do_install_dependencies(
             get_source_list(project, pypi_mirror=pypi_mirror)
         )
         index_args = " ".join(index_args).replace(" -", "\n-")
+        include_hashes=True
+        import re
         deps = [
-            req.as_line(sources=False, include_hashes=False) for req in deps_list
+            re.sub(" --hash", " \\\n --hash", req.as_line(sources=False, include_hashes=True)) for req in deps_list
         ]
         click.echo(index_args)
         click.echo("\n".join(sorted(deps)))

This is resolved when using the fairly new pipenv requirements command.


Move work history down the page

This removes job descriptions from work history (as in #268) and also moves the work history down the page to beneath the personal statement and referees sections, on the basis that candidates won't be asked for relevant skills and experience for each job, and instead a full work history is requested purely for safeguarding purposes.

I think users will need content to explain why descriptions aren't there for some applications.

@adamsilver something like this maybe? (date would be whichever day we make this change, and we'd only display this on application on or after this date)

Hey @frankieroberto I replied on Slack about this and how it crosses over with other work we're looking at.

Tweaked wording on the explanation for the missing skills and experience text (thanks @gregknight1!):


Add sonarcloud/ owasp Zap into our pipeline and use ACS to review/remediate image security issues

Our security team has made these requirements for going live with the new registry.

[ ] sonarcloud scan:

https://sonarcloud.io/project/overview?id=bcgov_platform-services-registry-web
https://sonarcloud.io/project/overview?id=bcgov_platform-services-registry-api

[ ] owasp zap scan: happens every monday

https://github.com/bcgov/platform-services-registry-web/actions/workflows/zap_scan.yaml
https://github.com/bcgov/platform-services-registry-api/actions/workflows/zap_scan.yaml

Will have a meeting with Nick about ACS image fix

Still haven't find a good solution yet, Nick asked me to try: https://github.com/flipkart-incubator/Astra, but it doesn't works out.

Since we have SonarCloud and OWASP ZAP in use I think this ticket should be closed.  If we want additional scanning for the api, then we can create a separate ticket.

Seems like the graphQL scan works: https://github.com/bcgov/platform-services-registry-api/actions/workflows/zap_scan.yaml


feat: integrate pushWalletClient.subscribe functionality + types

Description

Integrates<EMAIL_ADDRESS>which includes the pushWalletClient.subscribe() method and push_subscription event
Ensures AppSelector and Subscribe button reflects activeSubscriptions state
Removes monkey-patched type for pushWalletClient.subscribe()

Type of change
Please put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Draft PR (breaking/non-breaking change which needs more work for having a proper functionality [Mark this PR as ready to review only when completely ready])
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Fixes/Resolves (Optional)

Part of #105

Checklist:

[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

Additional Information (Optional)

Next step is adding update() support:

Implementation: https://github.com/WalletConnect/push-client-js/tree/feat/push-update
Depends on https://github.com/WalletConnect/walletconnect-docs/pull/660 being merged before moving ahead with release + web3inbox
Will be integrated into #118

I might follow up with a PR that shows a popup informing the user they need sign, using the  for continuity.

Yeah I'm actually not too sure why we're seeing that additional sign step if it should be pulling from the shared identityKeys class 🤔 Need to debug this more when I'm back on my laptop.


[nsec3] fix crash in nsec3 packing

Both NSEC and NSEC3 use the same logic to pack the bitmap.
CSYNC.pack also appear to make use of packDataNsec so I am giving it
the same treatment by moving the logic in a helper function and making
all those types len call use that function.

This is similar to #967 . The length computation logic is now refactored in a function that is used by all types that I could find that made use of packDataNsec

Codecov Report

Merging #973 into master will increase coverage by 0.07%.
The diff coverage is 64.7%.

@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
+ Coverage   55.42%   55.49%   +0.07%
==========================================
  Files          42       42
  Lines        9932     9923       -9
==========================================
+ Hits         5505     5507       +2
+ Misses       3398     3388      -10
+ Partials     1029     1028       -1

Impacted Files
Coverage Δ

msg_helpers.go
57.5% <64.28%> (+0.22%)
:arrow_up:

types.go
74.88% <66.66%> (+2.23%)
:arrow_up:

server.go
67.22% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #973 into master will increase coverage by 0.07%.
The diff coverage is 64.7%.

@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
+ Coverage   55.42%   55.49%   +0.07%
==========================================
  Files          42       42
  Lines        9932     9923       -9
==========================================
+ Hits         5505     5507       +2
+ Misses       3398     3388      -10
+ Partials     1029     1028       -1

Impacted Files
Coverage Δ

msg_helpers.go
57.5% <64.28%> (+0.22%)
:arrow_up:

types.go
74.88% <66.66%> (+2.23%)
:arrow_up:

server.go
67.22% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

updated helper name
updated TestCrashNSEC3 comment

address nits


[Object] Restore the StrMap behavior in JSON/SHash/SEqual

@tqchen Just addressed the comments and added a test case for JSON compatibility. Please take a look when you have time :-)

Thanks @junrushao1994 !


Bump npm to a version supporting package lockfiles

This would allow us to specify actual version ranges without potentially breaking builds

@peacememories  isn't this already done?

Yes, this is done


TPV removal of nonhomologous ends'

shouldn't be a child of
GO:0007534 gene conversion at mating-type locus
needs  to be
removal of nonhomologous ends involved in gene conversion at mating-type locus etc?
Also, I'm not  totally sure that mating type switching should be considered a 'reproductive process.. mating type switching can occur independently of mating...I would need to check that with @Antonialock and @mah11

I dont know if this is a sexually reproductive process or not: it happens during asexual reproduction but the purpose is (I guess) sexual reproductive fitness
eg
Extensive studies of fission yeast established the natural DNA strand chirality at the mat1 locus as the primary basis of asymmetric cell division. The asymmetry results from a unique site- and strand-specific epigenetic "imprint" at mat1 installed in one of the two chromatids during DNA replication. The imprint is inherited by one daughter cell, maintained for one cell cycle, and is then used for initiating recombination during mat1 replication in the following cell cycle.
PMID:26104357

I have a feeling we have discussed this before Maybe M will remember....if not will assume its OK and just fix
"removal of nonhomologous ends"

Hi Val, Antonia and Midori,
I have removed the part of relation. I tried to see if I could make it a pert of repair or recombination, but I think it is too generic. I am leaving this item open for the reproductive process bit. I think it is, but you guys are the experts.  Otherwise ping me and I will fix it.
-D

Hmm. I don't remember discussing this specific point before, but here's some fresh wittering.
It seems a bit counter-intuitive to have mating type switching as a reproductive process, but not outrageous. I guess it depends on how direct the connection to reproduction is meant to be — I don't think switching easily fits the "directly contributes to the process of producing new individuals by one or two organisms" part of the "reproductive process" definition; it seems not that direct to me. The immediate is_a link, though, is to "reproductive process in single-celled organism", which uses the woolier "involved in the reproductive function" wording in its def. It's a little easier to make that sort of apply to switching, since you don't get sexual reproduction if you don't have both mating types present (I do think it's ok to say "never mind" about asexual reproduction for this purpose - the question is whether switching is part of SOME reproduction, not whether it's part of ALL reproduction). So for me it comes down to how direct we want "involvement in making more lifeforms" to be.
Another thing - I see that mating type switching also has a path to "developmental process involved in reproduction". Are we confident that switching fits the description of "a progressive change"? (In other words, is the change that switching entails "progressive"?)

It's your call. What do you think your community would expect?

The formal definition of 'reproductive process' is simple a process that is
part_of (some)  reproduction  (axiom got deleted at some point.  I've added
it back in my branch, visisble soon).  So I think the important question
here is whether (all) mating type switching is part_of some reproduction.
If not, then the part_of should be deleted and the classifications up to
reproductive process will go away.  (If you decide to keep the link,
becuase it is intuitive to users even if not strictly always true, then
there should at least be a comment outlining reasons so future editors
don't need to puzzle over the choice again).
On Thu, Nov 12, 2015 at 2:25 PM, ukemi<EMAIL_ADDRESS>wrote:

It's your call. What do you think your community would expect?
—
Reply to this email directly or view it on GitHub
https://github.com/geneontology/go-ontology/issues/12161#issuecomment-156115274
.

I'm not the best judge of what the pombe community will find most intuitive, but I don't recall encountering active demand for an explicit connection linking mating type switching to reproduction.
Organisms that do mating type switching evolved the process to facilitate sexual reproduction, by increasing the probability that one cell of one mating type would encounter another cell of a compatible (as opposed to the same) mating type. But any individual instance of switching may take place in a cell that does not go on to find an opposite-type cell, mate, form zygotes, etc. So if the latter is the criterion for part_of, then the link to reproductive process should be removed.
Note, however, that the same reasoning applies to the paths via both of the asserted is_a parents, since both lead to reproductive process. Does that add a spanner, by calling the path via "sex determination" into question? If so, I'm not sure which link(s) pose any problem.
[Term]
id: GO:0007533
name: mating type switching
[snip]
is_a: GO:0007531 ! mating type determination
is_a: GO:0022413 ! reproductive process in single-celled organism
is_a: GO:0044710 {is_inferred="true"} ! single-organism metabolic process

I think the key here is to ask whether the switch happens for any other reason than reproduction. Is this analagous to something like gamete development? The purpose of that is for reproduction whether or not the animal ever mates, so by that standard it is still part of reproduction. Does that make sense?

Yep, it makes sense, and nope, there's no other point to mating type switching (as far as I know, anyway).

OK. I will close this then. If it comes up in the future, we can revisit. Thanks Midori.


DEP: Minimum supported Python version 3.10

Python 3.9 is no longer supported in the scientific python ecosystem: https://scientific-python.org/specs/spec-0000/
Related #1111

Ah so pyproj follows scipy instead of numpy which doesn't drop 3.9 until April:
https://numpy.org/neps/nep-0029-deprecation_policy.html

Ah sorry, numpy drops 3.8 in April if I'm reading that right.

Ah so pyproj follows scipy instead of numpy which doesn't drop 3.9 until April:
https://numpy.org/neps/nep-0029-deprecation_policy.html

The note at the top: "This NEP is superseded by the scientific python ecosystem coordination guideline SPEC 0 — Minimum Supported Versions."

Ubuntu LTS users (like me) will have only a few months to move to Ubuntu 24.04 (with python 3.12).
That applies as well for PROJ docker image.


IOException problem in NBug.Core.Util.Storage.Storer.GetFirstReportFile()

if any of generated report file("Exception_*.zip") is being used by other process, following line
AppDomain.CurrentDomain.UnhandledException += NBug.Handler.UnhandledException;

would raise exception as it reads in NBug.log(Note: the first Korean statement in Log reads   'c:\SmartADC\crash\Mirero.Server.SmartADC.AutoRuleValidate\Exception_130922186655821941.zip'  file cannot be accessed because it is being used by other process' or something)
NBugError: Cannot access the report file from the given custom path (it is probably locked, see the inner exception): c:\SmartADC\crash\Mirero.Server.SmartADC.AutoRuleValidate\Exception_130922186655821941.zip
Exception: System.IO.IOException: 'c:\SmartADC\crash\Mirero.Server.SmartADC.AutoRuleValidate\Exception_130922186655821941.zip' 파일은 다른 프로세스에서 사용 중이므로 프로세스에서 액세스할 수 없습니다.
   위치: System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   위치: System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   위치: System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy)
   위치: System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   위치: NBug.Core.Util.Storage.Storer.GetFirstReportFile() 파일 C:\Documents and Settings\Teoman Soygul\My Documents\ASFT\Dev\NBug\Source\NBug\Core\Util\Storage\Storer.cs:줄 287
NBugError: An exception occurred while dispatching bug report. Check the inner exception for details.
Exception: System.InvalidOperationException: 응용 프로그램이 UserInteractive 모드에서 실행되지 않을 때에는 모달 대화 상자나 폼을 표시할 수 없습니다. 서비스 응용 프로그램에서 알림을 표시하려면 ServiceNotification 또는 DefaultDesktopOnly 스타일을 지정하십시오.
   위치: System.Windows.Forms.Form.ShowDialog(IWin32Window owner)
   위치: NBug.Core.Util.Logging.Logger.Error(String message, Exception exception) 파일 C:\Documents and Settings\Teoman Soygul\My Documents\ASFT\Dev\NBug\Source\NBug\Core\Util\Logging\Logger.cs:줄 88
   위치: NBug.Core.Util.Storage.Storer.GetFirstReportFile() 파일 C:\Documents and Settings\Teoman Soygul\My Documents\ASFT\Dev\NBug\Source\NBug\Core\Util\Storage\Storer.cs:줄 294
   위치: NBug.Core.Submission.Dispatcher.Dispatch() 파일 C:\Documents and Settings\Teoman Soygul\My Documents\ASFT\Dev\NBug\Source\NBug\Core\Submission\Dispatcher.cs:줄 70
   위치: NBug.Core.Submission.Dispatcher..ctor(Boolean isAsynchronous) 파일 C:\Documents and Settings\Teoman Soygul\My Documents\ASFT\Dev\NBug\Source\NBug\Core\Submission\Dispatcher.cs:줄 41

BTW, I found there is no other process who opened that file because I was able to that file without any problem. I even tried to find the process accessing it  by Sysinternal 'handle.exe' utility but failed.
Is there any hint or solution for this?
FYI, my NBug configuration function follows.(my application is console program that is executed by windows service)
// in Main() function

                // https://github.com/soygul/NBug/issues/59 의 설명에 의거, 일단 WriteLogToDisk 옵션을 끔
                // 이렇게 하면 NBug.log는 생성되지 않음.
                NBug.Settings.WriteLogToDisk = false;
                NBug.Settings.UIMode = UIMode.None;
                NBug.Settings.StoragePath = NBug.Enums.StoragePath.Custom;
                NBug.Settings.StoragePath = (string)SystemPaths.CrashDirectory;
                NBug.Settings.MaxQueuedReports = int.MaxValue;

                AppDomain.CurrentDomain.UnhandledException += NBug.Handler.UnhandledException;
                TaskScheduler.UnobservedTaskException += NBug.Handler.UnobservedTaskException;

                //Application.ThreadException += NBug.Handler.ThreadException;

One more thing to mention.
My Application is executed many times in the same time. need consider concurrency or something?

True I've never considered concurrency as I've only used this library with big projects where there would always be one process.
On the other hand, that exception is raised only in non-release mode. If you put this on top of your NBug config:
NBug.Settings.ReleaseMode = true;

any locked filed should automatically skipped.

Thanks. helpful.


Should detect/model double parameter as such

Describe the bug
double value is transferred using two uint values.
To Reproduce
Steps to reproduce the behavior:

Decompile https://github.com/rfalke/decompiler-subjects/blob/master/from_holdec/i386_call_type_promotion/ia32_elf/subject.exe

Expected behavior
Output may look like
undefined4 print_double_as_hex(double param_1)

It currently is:
undefined4 print_double_as_hex(uint param_1,uint param_2)

Environment (please complete the following information):

OS: Linux
Java Version: openjdk version "11.0.5" 2019-10-15
Ghidra Version: 9.2_PUBLIC

Still an issue in version ghidra_10.3.2_PUBLIC.


perf: prevent changed senders clone

ref https://github.com/paradigmxyz/reth/pull/13187 #13193
cc @hai-rise
prevents a clone by changing the order, this is okay because all_transactions is entirely separate from sender_info

dup #13189