Parser should honor literals

Describe the bug

01 Num2     PIC 9 values zeros.

The Language parser failing for numeral literals
To Reproduce
https://gist.github.com/birojnayak/4bd0d4332d98170f74803b3e217ca4f8
Expected behavior
Parser should honor it.
Screenshots
If applicable, add screenshots to help explain the issue. This could be terminal screenshots that can help us find the source of the issue.
Platform Information (please complete the following information):

OS: [e.g. Linux]
OS Version [e.g. Ubuntu 22.04]
CPU Architecture [e.g. x64]
Libotterkit Version [e.g. 1.0]

Additional context
Add any other context about the problem here.

Hey @birojnayak sorry for the late reply, was refactoring a few things in the runtime library.
I'll start working on a fix for this right now.
The parsing method for the value clause currently doesn't check for figurative literals, but the lexer should be generating the tokens for them. So it shouldn't take too long to fix.

Oh also, the value clause in those variables should be value instead of values.
The standard syntax rules for single data items are: VALUE [IS] literal.
VALUES can only be used for variables with multiple possible values, such as tables and 88-level conditions.
We should add appropriate error messages for this.

@birojnayak, this is now fixed in the latest commit in the main branch.
The following working-storage will now successfully parse:
       working-storage section.
           01 UserName PIC X(30)  value "you".
           01 Num1     PIC 9 value zeros.
           01 Num2     PIC 9 value zeros.
           01 Total    PIC 99 value 0.
           01 SsnNum.
               05  SSArea  PIC 999.
               05  SSGroup PIC 99.
               05  SSSerial PIC 9999.

Note: Type checking for the literals is still missing, working on name resolution for the whole parser right now, which will include type checking for this as well.

@KTSnowy closing this.. I am going to test a bit more and create few more issues. Also as a general guidelines you need to create some sample COBOL projects for users to try and help everyone where it stands now.

@birojnayak, We need to get more parts of the codegen working to be able to provide runnable samples for users.
I was planning to make a sample collection with algorithm examples written in Standard COBOL, but we'll have to get classes and objects working first. Would be awesome if we could provide examples of generic collections in COBOL as well!

sure I will do some sample COBOL programs and submit a PR over weekend... we should include them as a part of tests...


Introduce "Dead Letter Handlers" allowing interception of failures to handle/ignore them

ES-Hadoop is a project that often embraces the fail-fast mantra, that exceptions and errors are often exceptional circumstances that need direct user interaction to fix and should not be ignored. This works great for batch jobs that are idempotent and easy to re-run, but the Hadoop ecosystem has since moved on to streaming applications being just as common as batch with patterns like lambda architecture and near-realtime processing.
As such, users are finding it difficult to use the connector as every time the connector experiences a non-recoverable exception such as in serializing malformed data, or data that is rejected by Elasticsearch under consistency guarantees.
We want to implement something close to Logstash's Dead Letter Queues, but persisting bad data can be a challenge when working on a distributed system like Hadoop. Logstash's Dead Letter Queues takes advantage of Logstash's internal binary formats and the fact that they are installed on a system with a (often) non-ephemeral filesystem. ES-Hadoop has no such internal binary format, opting to use each integration's data abstraction, and supports a plethora of integrations where HDFS or some other persistent storage is not always guaranteed to be available.
We are planning instead to implement something like a "Dead Letter Handler" instead of queue. Users would be able to implement interfaces for certain classes of errors or certain critical sections within the connector. These handlers would allow users to provide implementations that decide the connector's actions when potentially recoverable error states pertaining to data are encountered. Users would register the handlers with the connector by implementation name, and the connector will instantiate and configure the handlers at start up by means of reflection or SPI.
The project will also provide out of the box implementations of these handlers to cover the most common use cases, such as Ignore errors and log them, Ignore all errors, and the existing behavior raise all exceptions.
Users that wish to gain something similar to the Logstash Dead Letter Queue, would be able to implement their own handlers to persist the troubled data, such as sending it to HDFS, storing it in a queuing system like Kafka, or storing it on the local filesystem for example.

This is a good idea, but looking at all referenced pull requests they are all requesting a specific, rather simple, capability of ignoring specific HTTP response codes. Is it not possible to implement this feature first as a short-term solution until the proposed error handlers are implemented?

@dmarkhas Thanks for your input here. We're apprehensive toward introducing a short term solution that would need to be very quickly deprecated and replaced when this solution arrives.
While the above linked PR's are on their own centered around ignoring HTTP responses, there have been other discussions in the community around other problems like serialization issues killing streaming processes, and the desire to increase control over handling data related exceptions.

@jbaiera That makes sense.
The proposed solution seems comprehensive enough to cover all of these cases.
Some questions:

Would it be possible to register more than one handler?
When would these handlers be invoked? Your description mentions "when potentially recoverable error states pertaining to data are encountered", but that seems to leave the definition of what is a recoverable state to the connector author.

Thanks for the response :-)

@dmarkhas To give some answers:

Would it be possible to register more than one handler?

While the structure of this isn't too hard (just one handler that runs a chained set of handlers), the configuration for this could be fairly verbose/clunky. I think the simplest way to do this would be to set a single configuration property with a comma separated list of the handlers in the order that they should run.

When would these handlers be invoked? Your description mentions "when potentially recoverable error states pertaining to data are encountered", but that seems to leave the definition of what is a recoverable state to the connector author.

We'll have to start off with defining the error states that we want to handle. We could define a universal error handler/listener interface, but my concern here is that it would either be a confusingly broad interface, or hard to extend for an error handling situation that doesn't conform to it.
Obviously, we want to catch/handle an exception as close to the location that it occurred so that the response to the error is valuable. These locations are diverse, and may be dealing with different objects from different domains. Each handler may have to be structured differently from each other. Ultimately, the failure scenarios are going to dictate the interfaces that handle them.
My thoughts are that there will be two handler interfaces to start off with: A handler interface for serialization errors going from Tuple/Row/Writable/etc to JSON, and a handler interface for failed bulk entries on write operations. The sections of code enclosed and managed by these handlers would most likely be within the RestRepository at the writeToIndex method, and within the flush method.


Added support for DR-HTF009S

Added support for DR-HTF009S, have tested in HomeAssistant and it works as expected.

Looks like a duplicate of #96
Since it went first, I'm closing this one out.


c.addi should be nzimm (non zero imm)

Hi
c.addi should be nzimm (non zero imm), but when i use objdump to diassemble, i saw -16.
b:     file format elf32-littleriscv
Disassembly of section .text:
00010074 :
10074:       1141                    addi    sp,sp,-16
Why?
Thanks
Peter

From the spec:

C.ADDI adds the non-zero sign-extended 6-bit immediate to the value in register rd then writes the result to rd.

The only requirement is that the immediate is non-zero, not that it is positive.
If the immediate field is all zeroes, then this encodes C.NOP or hint instructions (depending on rd).

my fault, i thought nzimm mean unsign, thanks Lenary. This group is really super helpful. Thank you to all heros here.


throw er; // Unhandled 'error' event

C:\Users\Servr>frontail C:\log\log.1520503819
events.js:160
throw er; // Unhandled 'error' event
^
Error: spawn tail ENOENT
at exports._errnoException (util.js:1022:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:359:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:606:11)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3

Hey @hazimaqlan,
looks like you are running frontail on Windows where there is no tail command. Right now that's the limitation.

I was able to run frontail on Windows 10 using the Bash shell on Ubuntu provided by the "Windows Subsystem for Linux (WSL)".
For instructions on how to install WSL you can refer to this article.
You also need to make a separate install of Node.js in this Bash shell which can be done as follows (see also here):
curl -sL https://deb.nodesource.com/setup_12.x | sudo -E bash -
sudo apt-get install -y nodejs


#PickhardtPayments: add support for non-zero base fee channels

Is your feature request related to a problem? Please describe.
When computing routes, edges with a non-zero base fee are ignored. Especially edges with low base fees should be considered to make payments more reliable.
Describe the solution you'd like
Edges with a non-zeor base fee are considered in the computation, possibly with an over-approximation that makes them (far) less likely to be used for payments.

Worst case scenario is that we send several paths via the same channel, but diverging before/after. In this scenario we have to pay the base fee for each attempt. The routing (fee) cost of each attempt is determined by the size of the payment and the fee rate, where the fee rate is known and the size is a multiple of the configured quantization. As such, we could change a channel's fee rate so that the minimum amount that we could send through the channel (which is the configured quantization) covers the base fee.
For a quantization amount of 100k sat, a base fee of 2sat, and a regular fee rate of 123ppm:

true cost for 100k: 100,000 * 123 / 1,000,000 + 2 = 14.3
effective fee rate: (100,000 * 123 / 1,000,000 + 2) * 1,000,000 / 100,000 = 143


User ID Error

The auto-incrementing big serial ID field value is still incremented even though the previous INSERT SQL statement was found to be invalid, leading to abnormal ID value assignment of new users.

Commit: b5a034f979a5cd8b1a49e5c0455621b7fe921697
Addition of a function resetUserIDSequence() that decrements the SERIAL id sequence value by 1 to prevent id value assigning mismatch. This error has been known to occur after a DELETE SQL statement, where the id sequence is still incremented. Therefore, this function is added right after the execution of the DELETE PreparedStatement within the removeUserInfo() function to prevent the id assigning error.

Code fault found: The committed changes do not work whenever the deleted user is not the last user. Fix the code to alllow users prior to the last user to be deleted safely.

Commit: 33dfc62f525bfe6fe5b5abd4ae03ffc6c0ebc101
Addition of function that determines whether or not to decrement the userID sequence.


request parameter --seq has no effect

Hi,
Thanks for this great tool, I compiled everything successfully from commit 8f12bb1 and tested the request command on a small dataset with good results, using your pre-built GTDB-based dataset as reference.
Since the dataset was a multifasta containing multiple MAGs and I wanted to use each contig as a separate query, I tried to use the --seq option, but it seems to have no effect on the output. Could you provide more details about the effects of that parameter?
Thank you very much!

In fact --seq to have needs a database built with this option. Then your request is matched against
split blocks in database. With a database in one block, it has effectively no effect.

Thanks for the clarification! I am closing the issue but I suggest adding this info to the help message and/or the readme


Add instance properties extractor in MAL

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

Then this is ready to review

Seems the instance properties are not updated incrementally but overridden instead. That might need more modifications..

BTW, could you add e2e verification to do properties check on the mesh case? (We don't have k8s monitoring e2e, I think)
Is current check engine working well for this list check?


Added developer Plugin Panel, to allow adjustment of Environment properties in real time

Changed the Environment Enum to a Class, so that it can be used a bit more flexibly.
Specifically in the context of allowing the creation of Environments on the fly and overriding Environment values as well.

Created a side Panel for the HD plugin. It is currently populated with the Environment.Properties. You can edit
the properties in the panel and apply it to the current Environment to have it update in real-time.

Going to close this since it's quite outdated and I think we have most of the functionality available now. Feel free to re-open this with a rebase or chat with us in the Discord if you want. :smile:


isGroupTransactionPart failed because of missing value brq_transactions

When Buckaroo push the data after a successful checkout there is a error in the Magento logs;
[2020-07-30 07:06:36] main.CRITICAL: Report ID: webapi-5f22717c3684d; Message: Notice: Undefined index: brq_transactions in /home/xxx/releases/20200729072923/vendor/buckaroo/magento2/Model/Push.php on line 1193 {"exception":"[object] (Exception(code: 0): Report ID: webapi-5f22717c3684d; Message: Notice: Undefined index: brq_transactions in /home/xxx/releases/20200729072923/vendor/buckaroo/magento2/Model/Push.php on line 1193 at /home/xxx/releases/20200729072923/vendor/magento/framework/Webapi/ErrorProcessor.php:208, Exception(code: 0): Notice: Undefined index: brq_transactions in /home/xxx/releases/20200729072923/vendor/buckaroo/magento2/Model/Push.php on line 1193 at /home/xxx/releases/20200729072923/vendor/magento/framework/App/ErrorHandler.php:61)"} []
There is a check if a transactions is a grouptransactionPart in this code
    private function isGroupTransactionPart()
    {
        return $this->groupTransaction->getGroupTransactionByTrxId($this->originalPostData['brq_transactions']);
    }

But in our case Buckaroo push doesn't contain brq_transactions and it will generate a PHP error and the response is not processed.
This is the response from buckaroo
[30-Jul-2020 07:12:20 UTC] Array
(
    [ADD_initiated_by_magento] => 1
    [ADD_service_action_from_magento] => reserve
    [brq_datarequest] => *****************
    [brq_datarequest_] => *****************
    [brq_invoicenumber] => *****************
    [brq_primary_service] => KlarnaKp
    [brq_SERVICE_klarnakp_ReservationNumber] => *****************
    [brq_statuscode] => 190
    [brq_statuscode_detail] => S990
    [brq_statusmessage] => The request was successful.
    [brq_test] => true
    [brq_timestamp] => 2020-07-30 09:12:20
    [brq_websitekey] => *****************
    [brq_signature] => *****************
)

Hi @JoostWan !
Thanks for you report !
This error has been fixed ~2 weeks ago, but as for know it's available in master branch only
We expect to release new public version soon which will include that fix as well.

When there will be a composer release with this issue fixed?

we expect RC to be published today or tomorrow

Issue was fixed in 1.25.0-RC1


returnPartialData: true no longer works in latest react-apollo

I've tried to copy what they do here http://dev.apollodata.com/react/subscriptions.html#subscribe-to-more, but with no luck. updateQuery is returning correct data but the collection gets emptied on an insert from explorer.

Fixed this! I have pushed the updated version.


Increase range of recommended per-node volume size to 2 TB

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c9fc9f95bb9b1edf4abff2913236cbb1adc0e98c/
Edited pages:

v20.1/recommended-production-settings.md


ATH V2.0.0 Overview and Display data issues

@0xCUB3
Hi.
On screenshots, your release V2.0.0 is on the right, and my beta on the left
Hard : Laptop Dell Latitude 3510 (i5-10210U, 16 giga DDR4, Intel UHD 620...) Bootloader : Clover r5160
On Overview window :

Succinct data on Memory and Monitor
wrong bootloader data

and tooltip needs some line feeds (on V2.0.0)

On Display window :

wrong model icon

And new Settings window

and two ways to show tooltips when detail content is truncated
Cursor on detail content

Cursor on head line content

Regards

Hello matxpa, thank you for reaching out. The reason I haven't implemented the features in your beta is because it increases the clutter in the app. The settings are nice, but a bit overkill for such a simple app, and I want most of the fine details (like clover version) to be only in the tooltip.
As for the display issue, I added code to make it so that "unknown" displays would just have the default LCD panel show up, as some users had issues with it showing laptop screens even if they were using external monitors because macOS would mistakenly call them "built-in".
To be honest, I haven't looked into the fine details of your beta too much, but I will take a look and see what I might want to include.
P.S. looks like the "moniteurs" text isn't centered in French. Something to fix for next release!

@0xCUB3
I forget to tell you that this beta uses NO more data files.

it's not about data files - it's about a cluttered view. The app goes for minimalism, which your beta starts to discard.

ha, ok succinct vs cluttered

yep exactly! I love some of the things you put into your beta, but I still haven't decided what/how to implement it :)

but a bootloader displaying "Apple UEFI" instead of "Clover ..." remains an issue

I'm not sure why that's happening. I don't have a clover mac, so you'd have to debug that one.

It worked fine before V2.0.0, someone introduces a bug, not me.

I rewrote a lot of the code to be swift native rather than relying on shell commands. Everything else is working fine, but since I don't have a Clover machine to test, I never could confirm if clover still works...
to be fair, you are probably one of maybe 3 people who still use clover 🤣

And if I should be the last one i'll be the only one

@matxpa
Good night, can I try your beta version?
@0xCUB3
I can confirm boot loader wrong info (Apple UEFI having OpenCore). Maybe last statement
      else {
            return "Apple UEFI"
        }

always is run at the end of these blocks.
Apart from this, the new version 2 is a great job. Very "swifted".

@0xCUB3 @matxpa
c578f89 is the latest build showing right boot loader info.
After that, "Minimize use of shell commands in all sections" f049cc2 changes shell commands to swift and HCBootloader.swift shows wrong boot loader info.

Issue has been fixed in the latest commit.

@0xCUB3 @matxpa
b50635a fixes the wrong bootlader info. Tried on OpenCore.
It's a step back but, until the swift code works well on this part, at least the issue has been fixed.


Resubmit pr from Azure/azure-rest-api-specs-pr to Azure/azure-rest-ap…

This Pull Request has been proved by ARM
https://github.com/Azure/azure-rest-api-specs-pr/pull/1098

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

@yzhu1Azdata
Seems you've fix all required CIs. Let me know when you are good to merge (comment or email).

@yzhu1Azdata
Seems you've fix all required CIs. Let me know when you are good to merge (comment or email).

@weidongxu-microsoft  Thank you so much for your help and timely response! we are good to merge now :)


WIP: Add list content blocks

While working with @isakelaris today we discovered that we could add some endpoints that would be handy for her as she works with the email template side of things. We paired on this approach and I wanted to push it up so that she could see where things stood and possibly do this same operation for the email version:
/templates/email/list
This is still WIP until I come back and deal with the params.

hi @jonallured. I'm interested in this feature and I would gladly collaborate. Would you mind if I cherry-pick your commit to a new PR?


Missing values from tuple when writing to Elasticsearch

Issue description
After grouping and using BagToTuple to convert the bag to a tuple, writing to Elasticsearch loses all but the first value in the tuple.
Steps to reproduce
Code:
> cat test.csv
docId,name
a,All
a,Ball
a,Call
b,Bat
b,Cat
b,Fat
c,Cool
c,Fool

DEFINE CSVExcelStorage org.apache.pig.piggybank.storage.CSVExcelStorage(',', 'NO_MULTILINE', 'UNIX', 'SKIP_INPUT_HEADER');
dd_info = LOAD '/path/to/test.csv' USING CSVExcelStorage() as (docId:chararray, name:chararray);

grouped = GROUP dd_info BY docId;
final_info = FOREACH grouped {
    names = DISTINCT dd_info.name;
    names = BagToTuple(names);
    GENERATE group as docId, COUNT(dd_info) as count, names as names;
}

DESCRIBE final_info;
DUMP final_info;

STORE final_info INTO 'dd_info_test/final_info' USING org.elasticsearch.hadoop.pig.EsStorage('es.resource=dd_info_test/final_info', 'es.write.operation=index', 'es.mapping.id=docId');

Output:
final_info: {docId: chararray,count: long,names: (name: chararray)}

(a,3,(All,Ball,Call))
(b,3,(Bat,Cat,Fat))
(c,2,(Cool,Fool))

But on Elasticsearch:
GET /dd_info_test/_search

{
  "took": 4,
  "timed_out": false,
  "_shards": {
    "total": 2,
    "successful": 2,
    "failed": 0
  },
  "hits": {
    "total": 3,
    "max_score": 1,
    "hits": [
      {
        "_index": "dd_info_test",
        "_type": "final_info",
        "_id": "a",
        "_score": 1,
        "_source": {
          "docId": "a",
          "count": 3,
          "names": [
            "All"
          ]
        }
      },
      {
        "_index": "dd_info_test",
        "_type": "final_info",
        "_id": "b",
        "_score": 1,
        "_source": {
          "docId": "b",
          "count": 3,
          "names": [
            "Bat"
          ]
        }
      },
      {
        "_index": "dd_info_test",
        "_type": "final_info",
        "_id": "c",
        "_score": 1,
        "_source": {
          "docId": "c",
          "count": 2,
          "names": [
            "Cool"
          ]
        }
      }
    ]
  }
}

The array "names" has only one field. Is this a bug or am I doing something wrong? Thank you.
Version Info
OS:         :
JVM         :  1.8
Hadoop/Spark:
ES-Hadoop   :  5.1.1
ES          :  2.3

Since we have removed support for Apache Pig in the latest versions of ES-Hadoop, I will go ahead and close this issue.


Enable covgen --help

Which should just show the README.

LMK if you want me to pursue this; I can add a PR.

Please do!


Add Google Analytics and Google Ads actions

The time has come! 🎉
This branch includes both the Google Analytics Data Import action and the Google Ads Customer Match action. (They share common code and it was easier to handle them together...)
The Google Analytics Data Import action feels production ready and has already been beta tested by customers. It includes what should be a complete suite of automated integration tests. There is a basic README which will be converted into a Community post that we can later link to instead.
The Google Ads Customer Match action is feature complete, but still in the alpha phase. The test suite and documentation for it still need more work.
There are two new top-level dependencies:

lodash - "A modern JavaScript utility library delivering modularity, performance & extras"
gaxios - "An HTTP request client that provides an axios like interface over top of node-fetch." [This is a Google-owned packaged already used by the googleapis package under the hood.]

Note that the full test suite does not run clean for me:
  1) QueueTestAction unit tests for ProcessQueue
       action
         ExecuteProcessQueue returns success on successful response:
     AssertionError: expected promise to be fulfilled but it was rejected with 'AssertionError: expected false to equal true'

  2) QueueTestAction unit tests for ProcessQueue
       action
         ExtendedProcessQueue returns success on successful response:
     AssertionError: expected promise to be fulfilled but it was rejected with 'AssertionError: expected false to equal true'

Although I don't think my code is contributing to this. Also, several of the Marketo action tests are exceeding the default timeout for async tests.
I'm looking forward to any and all feedback!

@phillipperalez I've left this as a draft PR. We probably don't want to merge directly to master, but I don't believe I have the ability to create new remote branches in this repo. If you create a new branch (or give me the permissions) we can update the merge target.

After discussing with Aryeh, I am closing this PR in favor of a new one that splits up the two actions.


Some Videos are available only in English

Hi,
We are using Learning Pathways at our global company and we have enabled multilingual support for the Learning Pathways. The availability of these translated versions of the learning materials is very important to us. We have discovered that some of the video materials embedded in the articles are only available in English with no subtitles, audio or other options to translate them. In the first screenshot (Example_english version only) you can see an example of an article with a video embedded; even though the user is displaying all the contents, including the article, in German, (following his language settings) the video is only available in English, without any translation possibility: no subtitles, no audio translation and no visuals in german available.
In the second screenshot (List of Videos available only in English) we have noted down a list of assets whose videos are available only in English. Are there plans to convert these videos to ensure they have at least one translation option available for each?
Thanks for your help!

@uxwcs23 unfortunately, the content on those pages is pulled into Learning Pathways so we don't have any control over contents of the pages. My suggestion would be to visit the pages in question (e.g. https://support.office.com/de-de/client/2eb1d51f-a897-4d73-ba76-c550f005e5ef) and at the bottom of the page there is a feedback link. Fill in the feedback link that there should be German subtitles or transcript, etc. and let the content authors know.

Thank you for you quick response and support 😊
We will contact Microsoft as suggested.


Align to validation webhook pattern

The pattern is documented in https://github.com/openstack-k8s-operators/docs/pull/47

/test ironic-operator-build-deploy-tempest
The failed tempest test case seem unrelated to this change:
ironic_tempest_plugin.tests.api.admin.test_volume_connector.TestVolumeConnector.test_delete_volume_connector[id-5795f816-0789-42e6-bb9c-91b4876ad13f]
-----------------------------------------------------------------------------------------------------------------------------------------------------

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/tests/api/admin/test_volume_connector.py", line 65, in test_delete_volume_connector
    self.delete_volume_connector(self.volume_connector['uuid'])

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/tests/api/base.py", line 428, in delete_volume_connector
    resp, body = cls.client.delete_volume_connector(volume_connector_id)

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/services/baremetal/base.py", line 49, in wrapper
    return f(*args, **kwargs)

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/services/baremetal/v1/json/baremetal_client.py", line 467, in delete_volume_connector
    return self._delete_request('volume/connectors',

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/services/baremetal/base.py", line 236, in _delete_request
    resp, body = self.delete(uri)

      File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 339, in delete
    return self.request('DELETE', url, extra_headers, headers, body)

      File "/usr/lib/python3.9/site-packages/ironic_tempest_plugin/services/baremetal/base.py", line 70, in request
    resp, resp_body = super(BaremetalClient, self).request(*args, **kwargs)

      File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 742, in request
    self._error_checker(resp, resp_body)

      File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 867, in _error_checker
    raise exceptions.Conflict(resp_body, resp=resp)

    tempest.lib.exceptions.Conflict: Conflict with state of target resource
Details: {'error_message': '{"faultcode": "Client", "faultstring": "Node 439340ce-78d0-4ca8-868d-af6ed2eb39e2 is locked by host ironic-conductor-0, please retry after the current operation is completed.", "debuginfo": null}'}


feat(prank): startPrank/stopPrank

would be dope to have a start and stop prank cheatcode that would allow impersonating an account over multiple txs so users won't have to repeat themselves before each

The only difficult part here is making it such that internal calls of the first call don't inherit the wrong msg.sender as well. Should the test contract basically always be that new address? i.e. Test::test -> other::something -> Test::otherFunc -> other::something2
should the Test::otherFunc -> other::something2 call also be pranked?

I'd only go for the first call's level, and not for every subsequent internal call


[NSE-130] fix overflow and precision loss

What changes were proposed in this pull request?
This pr fixes overflow and precision loss in projection.
https://github.com/oap-project/native-sql-engine/issues/130
How was this patch tested?
under test

@zhouyuan @xuechendi precision loss for q58/61 is fixed in this pr.

@rui-mo also verified on my local env


Upgrade featured model(s) for text-generation to something more state of the art

internal slack convo

Fixed in #853 I think

thanks will close.
thanks @merveenoyan


[Feature Request] - Timeouts for active data connection acceptance

Description
When active data connections are configured to be used, currently FtpStream waits for an incoming connection indefinitely, while it may never happen, leading to a program hanging. Would be nice if it was possible to set a timeout.
Changes
The following changes to the application are expected

A data connection acceptance timeout setting and a handling of that setting.
Perhaps a similar one for passive connections: currently there is the connect_timeout function for control channel connection, but data channel in passive mode always uses plain connect.

Implementation
Once a setting is introduced, select or something along those lines can be called with a timeout, only calling accept when there is something to accept. For passive mode, TcpStream::connect_timeout can be used, as for the control channel.

Thanks!


Clarify values used in the evaluation in the forecast example

In the evaluation method data from the second year is transformed by the model.
Previewing the predictions I see 7 values for ForecastedRentals in the results. I assumed that is because of the horizon = 7.

What do the values represent? Is that one week after the training data, i.e forecast starts at 2012-01-01?
Or is it forecasting after the second year has been added by transforming the testdata, i.e forecast starts at 2013-01-01?
In either case it looks like the example code only looks at the first day of each prediction (ForecastedRentals[0]).
Can that really be compared with actual values from the testdata, which has period 1 day rather than 7 days?

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

ID: 32d03435-cc6c-1123-a86c-d5c3eacc24bd
Version Independent ID: 460afd2b-383d-7b90-6804-4b900c3bc9b2
Content: Tutorial: Forecast bike rental demand - time series - ML.NET
Content Source: docs/machine-learning/tutorials/time-series-demand-forecasting.md
Product: dotnet-ml
GitHub Login: @luisquintanilla
Microsoft Alias: luquinta

I modified the Evaluate method to do a prediction instead, similar to Forecast method. This seems more reasonable to me in order to compare with the actual results from year 2
Modified code: Program.cs
I don't understand why the predictions are different in this case, unless the Transform used in the example code actually moved the prediction point to 2013-01-01.

However, with the modified code the results from the Forecast method looks more similar to the graph in the tutorial.
Except that the lower estimates are different.
This makes me wonder if this was the way the graph was generated in the first place?

Related to #27469


Invite you to try FslexFsyacc

Lex/Yacc programming enthusiasts, I am very pleased to invite you to try a new series of lex/yacc tools.
Fslex is a code generator that uses regular expression syntax as a rule to generate a function, which divides the input token sequence into groups at a higher level. The Fslex is often used to remove redundant delimiters, add omitted delimiters or other syntax components, and so on. The Fslex is also used to determine context somewhere in the stream.
Fsyacc is a code generator that use BNF productions and precedences as a rule to generate a function, which resolves the input token sequence to an abstract syntax tree.
Why use this package?

You can use your existing handwriting tokenizer.

This package uses standard lex/yacc syntax to minimize your learning costs.

fslex/fsyacc generates respectively an independent, side-effect-free function that can be called flexibly.

The method of generating code is simple, without command lines and without the need to configure projects.

The result code is data-driven and highly readable.

Flexiblely compose of tokenize, regular expressions, BNF technology.

You can check it by following the link: https://github.com/xp44mm/FslexFsyacc

Would need a CSharpMath-specific analysis of benefits. Closing.


Enable send streaming

Change DataSource.send_stream type to IO

Codecov Report

Merging #5 (3a5d072) into main (4b6d856) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   78.07%   78.07%
=======================================
  Files           1        1
  Lines         406      406
=======================================
  Hits          317      317
  Misses         89       89

Impacted Files
Coverage Δ

src/Nghttp2.jl
78.07% <100.00%> (ø)

Continue to review full report at Codecov.

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


fix(docs, sb): handle tooltip what trigger with click

Closes #569
Closes #538

📝 Description
Handle docs Tooltip what trigger with click to showing it's contents.
⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

https://nextui.org/docs/components/tooltip#trigger
https://nextui.org/docs/components/tooltip#custom-content
these Tooltip does not work.
also storyboard too.
🚀 New behavior

Please describe the behavior or changes this PR adds

replace Button what Tooltip's children to Link
https://user-images.githubusercontent.com/26461307/175788626-182a9369-1d6f-4d23-a3b5-51046da242bf.mov
https://user-images.githubusercontent.com/26461307/175788640-29d2de74-0472-47cc-a703-c7a593d26b59.mov
💣 Is this a breaking change (Yes/No): No

📝 Additional Information
I think it is good to change all the buttons on the page to Link for uniformity.
If you think the same way as me, please leave a comment. I can handle it.

Oh, I didn't see this pr #537
sorry to bothering you
I think merge only this file apps/docs/content/components/tooltip/customContent.ts

Hey @hyesungoh, really appreciate your time and effort, however, since another PR created before this one, I will close it, sorry 🙏🏻


default config fix for MidiUartTransport

Hey this is my very first pull request ever :)
I've been adding midi in port to daisy and somehow it wasn't working, so I've noticed in the midi.h that default config of MidiUartTransport wasn't referencing correct pin numbers. When I changed those numbers according to the daisy datasheet my midi input started working.

Hi @drum-computer thanks for the PR!
Excited to see that you're interested in contributing back to the Daisy libraries!
To clarify, the MIDIUartTransport::Config has the default pins set up for the pins exposed on the Daisy Seed and Patch SM pin-banks.
These can be overridden when creating the config struct by manually setting:
MidiUartTransport:::Config midi_cfg;
midi_cfg.rx = Pin(kMyRxPin);
midi_cfg.tx = Pin(kMyTxPin);
// and then initialize
my_midi_object.Init(midi_cfg);

The pins that you configured to the defaults are valid pins for the USART1 peripheral, but all of the example hardware platforms with MIDI use the pins already set as defaults.
If you're using the Patch SM, what I've described above would work for your projects, but does not require an update to the library itself.
You can also use PB8/PB9 via UART4 on the PatchSM.
Feel free to let me know if you have any questions!


Add new sidebar links

Test plan

[x] automated tests that sidebar only shows links based on AuthStatus matching the criteria below
[x] Check usage link shows up when cody_pro_dec_ga flag is enabled
[x] Check usage link does not show up when cody_pro_dec_ga flag is not enabled
[x] Check Upgrade link shows up when cody_pro_dec_ga is enabled and codyProEnabled=false in GraphQL response when using DotCom
[x] Check Upgrade link does not show up when cody_pro_dec_ga is enabled and codyProEnabled=true in GraphQL response when using DotCom
[x] Check Upgrade link does not show up when cody_pro_dec_ga is disabled and codyProEnabled=false in GraphQL response when using DotCom
[x] Check Upgrade link does not show up when cody_pro_dec_ga is enabled and codyProEnabled=false in GraphQL response when not using DotCom

@toolmantim I presume these are both just opening external pages? I've added placeholder URIs for now. The upgrade link needs to be conditional, but I don't know what the data/structure for accounts looks like yet (or whether it's part of an existing auth response or new APIs).

@toolmantim @kalanchan I pushed some changes to add conditions to the buttons. It needs tests, but I wanted to check if this is what you had in mind first.
There are some TODOs with questions - mainly around whether it's safe to always query for codyProEnabled (so this can be done along with an existing query for verified email). And ofc, the URLs for the buttons are just placeholders.

Looks good @DanTup!

I presume these are both just opening external pages? I've added placeholder URIs for now. The upgrade link needs to be conditional

I believe so!

I wanted to check if this is what you had in mind first

From a design POV, this looks good to me.

@philipp-spiess @abeatrix I have some questions in TODOs in the code that I'm not sure who is best to answer (mostly around graphql to check for Pro (reusing an existing verified email query and whether it's safe to assume the field exists).

@abeatrix in this PR there are a bunch of places where code had to be made async to check the GA feature flag. I have the same in my next PR (adding the upgrade notification to rate limit errors for completion). This feels awkward and can be removed once the GA flag goes away.
I wonder if we should change how it works, fetching the feature flag during auth and caching it on AuthStatus so it could be checked synchronously in these places? (I don't know if there's a danger of it becoming stale then though - do feature flags usually change and update without re-authenticating?)

I wonder if we should change how it works, fetching the feature flag during auth and caching it on AuthStatus so it could be checked synchronously in these places? (I don't know if there's a danger of it becoming stale then though - do feature flags usually change and update without re-authenticating?)

Good question 🤔 I've never looked into the implementation before, but I am guessing they can change without re-authenticating so that when a feature flag is turned on/off in the backend, it will have immediate effect?
I think Taras will be the right person to ask since he implemented the provider :D  cc @taras-yemets

I wonder if we should change how it works, fetching the feature flag during auth and caching it on AuthStatus so it could be checked synchronously in these places? (I don't know if there's a danger of it becoming stale then though - do feature flags usually change and update without re-authenticating?)

Once fetched, we store the value in the cache for 1 hour. As far as I remember, the interval was selected arbitrarily. The answer depends on how often the auth happens. If it's frequent enough to evaluate the potentially changed feature flag value, then I'd say we're good to cache it and check synchronously.
https://github.com/sourcegraph/cody/blob/ec008a8814f824313c02037b4a4d93456ca3987b/lib/shared/src/experimentation/FeatureFlagProvider.ts#L56
cc: @abeatrix, @philipp-spiess, @DanTup

@kalanchan can you confirm the feature flag name? I added cody_pro_dec_ga in this PR (from https://github.com/sourcegraph/cody/issues/1576#issuecomment-1806466079) but I've also seen cody-pro in another doc.
Other than that, I think this is ready if @abeatrix is happy with the responses to her comments above.

@kalanchan can you confirm the feature flag name? I added cody_pro_dec_ga in this PR (from #1576 (comment)) but I've also seen cody-pro in another doc.
Other than that, I think this is ready if @abeatrix is happy with the responses to her comments above.

Sorry for the confusion! cody_pro_dec_ga was the original flag, but I've confirmed with Naman that its going to be cody-pro now. Let's reflect that in the PR

I pushed a change to update the flag and all the unit test bots are failing. I reverted the change and the same is happening - so I don't think it's related to that change.
It looks authentication related - and the error also seems to include the live dotcom endpoint (it also says "unknown test" as if the error is occurring outside of the tests).
<empty line>
Errored ➞ POST https://sourcegraph.com/.api/graphql?CurrentUser _PollyError: [Polly] [adapter:node-http] Recording for the following request is not found and `recordIfMissing` is `false`.
{
  "url": "https://sourcegraph.com/.api/graphql?CurrentUser",
  "method": "POST",
  "headers": {

I don't understand why this is happening with the exact same code that was green yesterday, and I can't repro it from my local machine. Any ideas? 🤔

oh nm, I realised the bots are merging latest main with the PR before running. After merging main I'm seeing the same failures locally and will investigate!

Ok, I think the failures might be related to the new agent tests from #1872. Based on @olafurpg's comments, I believe the tests are run locally and the HTTP requests and responses captured to be replayed during the CI runs.
This PR changes the graphql queries (to include the codyPro flag when fetching user information from dotCom) so there's no recorded responses.
@olafurpg I'm reading the instructions and it seems I need to set SRC_ACCESS_TOKEN to an access token to re-record these. However it's not clear to me what that token is (and whether it's something I will have access to?).

@DanTup you create an access token for your sourcegraph.com account by opening this page here https://sourcegraph.com/user/settings/tokens/new
You can verify that you are authenticated with the src cli tool (installable via npm package @sourcegraph/src)
❯ src login

❌ Problem: No access token is configured.

🛠  To fix: Create an access token at https://sourcegraph.com/user/settings/tokens, then set the following environment variables:

   SRC_ENDPOINT=https://sourcegraph.com
   SRC_ACCESS_TOKEN=(the access token you just created)

   To verify that it's working, run this command again.

BTW, I spent a while trying to see if I could automatically print out the instructions to update the HTTP recordings when the tests fails. I investigated custom vitest reporters but didn't make progress (because I think we're using an old vitest version 🤔 ). Either way, I'd love to see how we can make this process as smooth as possible.

@olafurpg thanks! I wasn't able to install the npm package (see https://github.com/sourcegraph/src-cli/issues/1050) but I set those env vars and commented out the call to src login when recording the tests and it seems to have worked, the bots are now green. Thanks! :)

I spoke too soon - it fixed most of the bots, but the Ubuntu one remained red. @olafurpg should there be differences across platforms in the HTTP requests?
Errored ➞ GET http://localhost:3080/__version _PollyError: [Polly] [adapter:node-http] Recording for the following request is not found and `recordIfMissing` is `false`.
{
  "url": "http://localhost:3080/__version",
  "method": "GET",
  "headers": {
    "accept": [
      "*/*"
    ],
    "user-agent": [
      "node-fetch/1.0 (+https://github.com/bitinn/node-fetch)"
    ],

@DanTup the agent tests are passing in ubuntu 16. The root failure seems unrelated and might be a flake so re-ran the failed tests
Failed to terminate worker while running /home/runner/work/cody/cody/lib/shared/src/common/markdown/markdown.test.ts.

Ah, thanks! I had re-run once and it failed the same way, but I guess it was just coincidence :)

I want to fix the tests so that we don't get failing HTTP requests when running the agent tests. There is some non-determinism in how we send HTTP requests causing the pollyjs errors even if they don't affect the behavior of the agent tests.

@abeatrix was just a flake - I re-ran that bot and it's green now.


feat: Record key value pairs

Support recording of slf4j 2.0 key value pairs in logback recorder

@slu-it any feedback on this?

Additionally: please change the commit message to "add support of SLF4j Key/Value data as properties for Logback"

Done

The variable properties: Map<String, String> should not be String to String. But I don't want to make that change in the codebase.

Hmm .. that is  a bit of a philosophical question/problem.
val properties1 = mutableMapOf<String, String>()
val a1 = properties1.get("foo") // -> String?
val a2 = properties1.getValue("foo") // -> String

val properties2 = mutableMapOf<String, String?>()
val b1 = properties2.get("foo") // -> String?
val b2 = properties2.getValue("foo") // -> String?

The properties were specified with case 1 in mind. So if there is a key, there should be a value.
Switching to case 2 would indeed possibly break some things, especially downward compatibility.
Maybe it would be best to remove key-value pairs where the value is null?

What I meant is that the values are not necessarily Strings but could be other types as well.
I don't mind dropping null values. But maybe someone wants to assert the key ist there?

I'll think about this for the weekend and get back to you beginning of next week. Right now I'm not sure what would be the best approach.
Switching from Map<String, String> to Map<String, Object> might not be too bad for compatibility. Depending on how people are using the properties.

@bomgar I updated the master with #55 :

new baseline versions for JUnit, Kotest, Kotlin, Logback etc.
switch to Map<String, Any> for properties

Could you please rebase your changes and also add the filtering of null-valued key-value pairs?

Done

@bomgar version is on its way to be released and should be on Maven central within 1 hour:
https://github.com/logrecorder/logrecorder/releases/tag/v2.10.0

Thank you!


Feature 1

Remake my branch

@Ielay merge was not correct!

@malinink I guess I did it now

@Ielay I suppose for correct merge ;) it's a real reason here!


updates space station 13's entry

space station 13 has never been turn based, so removes that keyword. also adds new keywords and adds more licensing information as well as other ss13 server branches

your right Space Station 13, often shortened to SS13, is a top-down tile-based action role-playing multiplayer video game running on the freeware BYOND game engine, originally released in 2003
source Wikipedia https://en.wikipedia.org/wiki/Space_Station_13#Gameplay

could i update it with top-down tile-based action role-playing?

@Kylerace how is your day doing

@Kylerace how is your day doing

alright i suppose, how are you

not bad

Spammers 👎 But also funny 👍 Okay I'll look at it.

Thanks very much for the commit.


Questions in termusic repo

These questions originated in the discussions part of another repo which uses rodio: https://github.com/tramhao/termusic/discussions/203 and where asked by @hasezoey
I have copied some of the rodio api related questions here so they (and the answers/discussion that might follow) can also benefit other rodio users.

send a message on EOS of a source (see this code in our implementation) (suggestion would be to get the handle to the sleep_until_end directly, or a function which accepts a callback and is similar to source::Done)

You might be able to do that using EmptyCallback. Its a source that will never play audio but will execute arbitrary code when it runs. You can append an EmptyCallback after each source. Let me know if that sounds good

possibly a way to add / replace other source modifiers after all the other processing in Sink::append

You already seem to be using PeriodicCallback. Adding effects is done by compiling the source with those effects applied but configured such that they have no effect. For example a speed of 1.0 does nothing to the underlying source. Some source effects like automatic_gain_control have an enable/disable toggle.

there is likely a possible de-sync between the track_position getting for events and the Sink::get_pos which is after speed / delay

That must be a bug in rodio. Please open an issue and provide a minimal example where the bug is easy to see so we can fix it.

You might be able to do that using EmptyCallback. Its a source that will never play audio but will execute arbitrary code when it runs. You can append an EmptyCallback after each source. Let me know if that sounds good

That does sound like a good idea. Alternatively i think its possible to custom-add something similar to source::Done which is not necessarily in rodio itself.

You already seem to be using PeriodicCallback. Adding effects is done by compiling the source with those effects applied but configured such that they have no effect. For example a speed of 1.0 does nothing to the underlying source. Some source effects like automatic_gain_control have an enable/disable toggle.

I meant it in the context of still being able to use Sink::controls (and by extension Sink::set_speed) without having to extra-wrap it for a case like replacing speed with our soundtouch source modifier.

there is likely a possible de-sync between the track_position getting for events and the Sink::get_pos which is after speed / delay

That must be a bug in rodio. Please open an issue and provide a minimal example where the bug is easy to see so we can fix it.

I did not test if there actually is a desync, but i noted it as possible as, if i understand correctly, track_position is affected by speedup / delay / other source modifiers (as per this comment and this doc-comment)

After having a look at those things again, with the current rodio structure it seems possible that we also could just make a custom Sink impl instead of using the rodio provided one, or would this be a problem?
My thinking is because everything rodio::Sink uses is also exported pub (like queue::SourcesQueueInput pub cpal re-export and all sources)

Alternatively i think its possible to custom-add something similar to source::Done which is not necessarily in rodio itself.

Yes anything that implements the Source trait will work. If you can think of something that specifically works better with termusic go for it!

I meant it in the context of still being able to use Sink::controls (and by extension Sink::set_speed) without having to extra-wrap it for a case like replacing speed with our soundtouch source modifier.

So you would like Sink to add controls for Sources defined outside of rodio?
I am also intrigued by the soundtouch source modifier what does it do?

I did not test if there actually is a desync, but i noted it as possible as, if i understand correctly, track_position is affected by speedup / delay / other source modifiers (as per this comment and this doc-comment)

Yes but only those applied after track_position. If you use the speed option from the Sink then track position will report the position in the underlying source and neglect the speedup.
From the Sink::get_pos docs:
Example: if you apply a speedup of *2* to an mp3 decoder source and
[`get_pos()`](Sink::get_pos) returns *5s* then the position in the mp3
recording is *10s* from its start.

wait... thats wrong.... Okay I am going to check this out and report back!

wait... thats wrong.... Okay I am going to check this out and report back!

Nope its correct. try_seek takes speedup's into account. We do that because otherwise mixing could become terribly confusing. That only works if the speedup is always the same though...... Thinking about it now that might have been a bad design choice. Tracked in new issue #638.

After having a look at those things again, with the current rodio structure it seems possible that we also could just make a custom Sink impl instead of using the rodio provided one, or would this be a problem?
My thinking is because everything rodio::Sink uses is also exported pub (like queue::SourcesQueueInput pub cpal re-export and all sources)

Yes, that is a very good idea. Originally that would have been the only possibility. I think Rodio's Sink was made because users found that hard to do and "just wanted to play some music". Its design however is questionable. We are going to rewrite it and make breaking changes to it, some discussion here: https://github.com/RustAudio/rodio/issues/624 and you have seen the user stories issue where we are gathering feedback so we can decide what the new sink like thing should be.
Making your own Sink like structure is quite doable. I have explained part of it here: https://github.com/RustAudio/rodio/issues/634 (second paragraph of my first response). Let me know if you need any help.

I am also intrigued by the soundtouch source modifier what does it do?

The soundtouch source modifier is currently just a wrapper for soundtouch which is a ffi for c++ soundtouch.
TL;DR: speed up/down with no pitch change (which rodio's current speed impl does)

So you would like Sink to add controls for Sources defined outside of rodio?

Not quite, for this specific example, i would like to just replace rodio's speed modifier (or at least disable it from being changed) and re-use Sink::controls's speed for the soundtouch speed modifier (see this disabling and this enabling) without some extra wrapper around Sink. This could likely be solved with the referenced #624 by exposing more controls or directly accessing source modifier's controls.
And otherwise i just meant to apply modifiers after the other changes which might depend on it, like after speed (which there currently, aside from periodic_access, we dont have) (a example could be to have a periodic_access which should run regardless of if it is paused or not). Which has also, as i have seen, been part of #624, though more broadly.


[Scheduled Maintenance] Automations down for Maintenance

Expected 5 to 10 minutos down time for software updates

Successfully updated.


test: improve html5lib test format handling

What problem is this PR intended to solve?
The parser for html5lib tests is conservative in what it accepts, and chooses to abort when formatting expectations aren't met. This PR updates that code to skip the test and emit a warning message.
Fixes #2555

Have you included adequate test coverage?
N/A

Does this change affect the behavior of either the C or the Java implementations?
N/A

This looks better all around! Thank you for cleaning that code up.

Also running this against upstream html5lib-tests at https://github.com/sparklemotion/nokogiri/runs/6598806979?check_suite_focus=true


Migration to sqlmodel

Codecov Report

Merging #4 (c436869) into master (732af63) will decrease coverage by 0.08%.
The diff coverage is 87.48%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   82.63%   82.54%   -0.09%
==========================================
  Files          55       58       +3
  Lines        1797     1891      +94
==========================================
+ Hits         1485     1561      +76
- Misses        312      330      +18

Impacted Files
Coverage Δ

app/api/api_v1/api.py
100.00% <ø> (ø)

app/sse/notifications.py
35.57% <0.00%> (-0.62%)
:arrow_down:

app/api/api_v1/endpoints/tasks/reactions.py
36.36% <37.50%> (+1.22%)
:arrow_up:

app/api/api_v1/endpoints/tasks/task_items.py
45.45% <58.33%> (-1.92%)
:arrow_down:

app/api/api_v1/endpoints/tasks/projects.py
54.83% <60.86%> (-10.55%)
:arrow_down:

app/schemas/tasks.py
82.85% <67.27%> (-17.15%)
:arrow_down:

app/db.py
70.00% <70.00%> (ø)

app/main.py
69.81% <71.42%> (+0.42%)
:arrow_up:

app/api/router.py
82.66% <72.41%> (-6.23%)
:arrow_down:

tests/tasks/conftest.py
77.19% <78.00%> (+19.29%)
:arrow_up:

... and 39 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


TypeError: The "string" argument must be of type string or an instance of Buffer or ArrayBuffer. Received an instance of Object

Hi! Thanks for all the work you've put into this project.
I'm following the quick start tutorial and have managed to set things up, after a some setbacks with simple_oauth library.
But, currently, I'm having an issue viewing my blog posts. I can see them at the home page and can also see that all their content is fetched in home page, as a collection. But once I visit a specific one (blog/first-post, as in blog/[node:title]), I get a 404 in the Network tab in Google Chrome.
And also this error.
 1 of 1 unhandled error

Unhandled Runtime Error
Error: Failed to load script: /_next/static/chunks/pages/404.js

Source
.next\static\chunks\main.js (83:51) @ HTMLScriptElement.script.onerror

  81 | //    executes when `src` is set.
  82 | script.onload = resolve;
> 83 | script.onerror = ()=>reject(markAssetError(new Error(`Failed to load script: ${src}`)))
     |                                           ^
  84 | ;
  85 | // 2. Configure the cross-origin attribute before setting `src` in case the
  86 | //    browser begins to fetch.

I guess the following error is due to not getting any data in the response.
error - TypeError [ERR_INVALID_ARG_TYPE]: The "string" argument must be of type string or an instance of Buffer or ArrayBuffer. Received an instance of Object
    at Function.byteLength (buffer.js:726:11)
    at ServerResponse.apiRes.end (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\api-utils.js:72:41)
    at C:\Users\trogl\Desktop\next-next-next\node_modules\next-drupal\dist\index.js:566:30
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Object.apiResolver (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\api-utils.js:101:9)
    at async DevServer.handleApiRequest (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\next-server.js:770:9)
    at async Object.fn (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\next-server.js:661:37)
    at async Router.execute (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\router.js:205:32)
    at async DevServer.run (C:\Users\trogl\Desktop\next-next-next\node_modules\next\dist\server\next-server.js:841:29) {
  code: 'ERR_INVALID_ARG_TYPE',
  page: '/api/preview'
}

Is this happening in the Drupal preview only? What happens if you visit the post directly?

You mean from Next.js or in Drupal?

On the Next site, homepage for the default-starter works great. Plus, I see all the data of the collection.

but if I click on one of the posts, I get this

@shadcn In my case, this is a local environment in Windows, with Xampp.
Other than that, just follow the exact instructions on Quick Start guide https://next-drupal.org/docs/quick-start.
That's about it I think, the repo is the same as the basic starter.

I am experiencing the same phenomenon.
TypeError [ERR_INVALID_ARG_TYPE]: The "string" argument must be of type string or an instance of Buffer or ArrayBuffer. Received an instance of Object
    at new NodeError (node:internal/errors:371:5)
    at Function.byteLength (node:buffer:733:11)
    at ServerResponse.apiRes.end (/opt/preview/.docker/preview/node_modules/next/dist/server/api-utils.js:72:41)
    at /opt/preview/.docker/preview/node_modules/next-drupal/dist/index.js:593:30
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.apiResolver (/opt/preview/.docker/preview/node_modules/next/dist/server/api-utils.js:101:9)
    at async Server.handleApiRequest (/opt/preview/.docker/preview/node_modules/next/dist/server/next-server.js:770:9)
    at async Object.fn (/opt/preview/.docker/preview/node_modules/next/dist/server/next-server.js:661:37)
    at async Router.execute (/opt/preview/.docker/preview/node_modules/next/dist/server/router.js:205:32)
    at async Server.run (/opt/preview/.docker/preview/node_modules/next/dist/server/next-server.js:841:29) {
  code: 'ERR_INVALID_ARG_TYPE'

The version I am using is as follows.

next-drupal: 0.16.2
next.js (Drupal Module): 1.0.0-beta5

I refer to this code,
On the Drupal side, the page loaded successfully by applying the following patch to the subrequests module.
        "patches": {
+	    "drupal/subrequests": {
+                "Subrequest failed validation": "https://www.drupal.org/files/issues/2019-05-27/3029570-array-not-object.patch",
+                "Get same results on different request": "https://www.drupal.org/files/issues/2019-07-18/change_request_type-63049395-09.patch"
+            }
        },

I faced the same issue. As per @taku3202 comment, applying Drupal Subrequests patches solved it :

Subrequest failed validation
Get same results on different request

@demo-Ghost Did you figure this out?

I'm getting the same error in preview. My Drupal site base URL is http://localhost:8080/blog/drupal/web

Also, applying the patches from https://github.com/chapter-three/next-drupal/issues/26#issuecomment-945509653 didn't help

Just for debugging I've replaced the RouterPathTranslatorSubscriber::cleanSubdirInPath() method (see https://github.com/chapter-three/next-drupal/issues/26#issuecomment-950711871) with:
  protected function cleanSubdirInPath($path, Request $request) {
    // Remove any possible leading subdir information in case Drupal is
    // installed under http://example.com/d8/index.php
    if (strpos($path, '/blog/drupal/web/') === 0) {
      $path = substr($path, strlen('/blog/drupal/web/'));
    }
    return $path;
  }

...and it fixes the issue.
Note that I've hardcoded my base path because it seems that I couldn't get it from request. What comes from $request->getBasePath() is an empty string. Same is stored in $GLOBALS['base_path'].
However, we need permanent fix. I see there is https://www.drupal.org/project/decoupled_router/issues/3133873 but the patch over there is not resolving the issue.

So this works but don't ask my why :) The Symfony request stored in $GLOBALS['request'] contains the correct base path information. But I have no idea why they are different...
  protected function cleanSubdirInPath($path, Request $request) {
    // Remove any possible leading subdir information in case Drupal is
    // installed under http://example.com/d8/index.php
    $regexp = preg_quote($GLOBALS['request']->getBasePath(), '/');
    return preg_replace(sprintf('/^%s/', $regexp), '', $path);
  }

Ok, so I've reworked https://www.drupal.org/project/decoupled_router/issues/3133873. @demo-Ghost could you try to apply also the https://git.drupalcode.org/project/decoupled_router/-/merge_requests/3.diff patch to decoupled_router module and see if fixes the issue? It does for me.
(However, still don't understand why, in cleanSubdirInPath(), $request and $GLOBALS['request'] are diverged)

I'll give it a shot @claudiu-cristea and I'll let you know.

@claudiu-cristea hi! I had the same issue described here, and your last patch fixed it!


Pin minimum version of Language Server

Ensure LS is pinned to a min version.

The PR broke language server downloads, e.g. https://pvsc.azureedge.net/python-language-server-stable.0.1.50.nupkg is not a valid URL.


Consider integrating another heatmap visualization

Consider implemeting the heatmap code from drazenz/heatmap to nominal.associations.

Requires an issue to be solved in that package: [https://github.com/drazenz/heatmap/issues/5](https://github.com/drazenz/heatmap/issues/5

Will require better API before integrating


HDDS-6961. [Snapshot] Bootstrapping slow followers/new followers.

What changes were proposed in this pull request?
This PR updates the om follower bootstrap mechanism to include the snapshot state.
It considers snapshot state to be all files under  metadataDir/db.snapshots.
That includes the om snapshot directories, as well as the snapshot diff compaction logs and backup sst files, (which have been moved to the "db.snapshots" dir by this PR.)
This PR adds the contents db.snapshots dir to the tarball sent to the follower.  To reduce the size of the tarball, it does not include multiple copies of any hard links found.  Instead, it includes a list of hard links to be generated by the follower.
Design doc here: https://docs.google.com/document/d/1cFZj-7NRxiHaZ56ndcf1Z1EqapPFy4fo4dDVIy_aCx4/edit
Recon
Recon also uses the same tarball to initialize its copy of the OM rocksdb.  Since it doesn't need the snapshot data, I've added the "includeSnapshotData" parameter to the http request.
Renamed OzoneManagerSnapshotProvider
The ratis code uses the term "snapshot" to mean something other than what we mean.  It uses "snapshot" to refer to the tarball as a whole, (which now includes all of the individual "OM snapshots".)
In particular, this class, in the "om/snapshot" directory is ambiguously named:
org/apache/hadoop/ozone/om/snapshot/OzoneManagerSnapshotProvider.java

To reduce potential confusion, I've renamed it to:
org/apache/hadoop/ozone/om/ratis_snapshot/OmRatisSnapshotProvider.java

Internal Consistency of Tarball
There are two areas of consistency I've thought about:
Snapshot Info Table Entries -> Snapshot Directories
There needs to be a directory for each snapshot info table entry.  These directories sometimes appear a short while after the snapshot info table entry is created.
This PR addresses that by ensuring the directories exist before creating the tarball, (pausing for a few seconds if needed.)
Compaction Logs -> SST Files
If the tarball is created during compaction, the snap diff compaction logs for the most recent compaction may not be included.  I'm not sure how bad a problem this is.  Please consider it in your review of this PR.
Incremental checkpointing
The addition of snapshot data will increase the size of the tarball, exacerbating the problem described here: https://issues.apache.org/jira/browse/HDDS-6510
We'll need to decide if incremental checkpointing needs to a part of the initial snapshot release.  If not, it may need to come soon afterwards, otherwise users could be stranded.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6961
How was this patch tested?
Unit/integration tests added

FYI: @hemantk-12 , I've made all your requested changes.  In addition, I've fixed a separate issue, so that the tarball creation skips any snapshot directories that are not in rocksdb.  Those changes are mostly in this commit:
https://github.com/apache/ozone/commit/e3a29217e37c7160c2bb3805e1b831fbc4c617d6

Since we decided not to integrate with IC yet, I didn't have to change this PR much, since the last time it appeared.
Mainly I've just merged the latest master and dealt with the conflicts.  There were two types of conflicts:

The http servlet in master was restructured somewhat since my original draft, (mainly to not zip the tarball.)  So I merged those changes with mine.

New snapshot code has been added using the old snapshot directory structure.  (As mentioned above, this PR changes the structure slightly so that the tarball creation code just pull most everything from a single directory.)  So I adjusted those directories as well.

TODO:
I've written a bunch of code to manage the interlocking between tarball creation and the background tasks that write snapshot state.  I decided to put that in a separate PR because this one is already too big.

@hemantk-12 @smengcl @prashantpogde I updated the bootstrapping PR to the latest master and fixed the merge conflicts. ^^
Please take a look again.

Can we write some test that can exercise these code paths end to end ? Perhaps mimic this with an integration test. We may have to tune down the RATIS log size to very small number of transaction so these code paths do get exercised.

Create a  OM HA mini ozone cluster
pause one of the OM nodes
continue creating multiple transactions and multiple snapshots.
after a while, bring up the paused OM node and make sure that it has identical snapshots + active OS data as the leader OM.
Step 3 should be configurable to create even thousands of snapshots. In our local build, we should be able to test this thoroughly.

Can we write some test that can exercise these code paths end to end ?

As mentioned above, this integration test does excercise it from end to end:
https://github.com/apache/ozone/blob/ed069f01142452dfc59b3f9703e486ef0434d7e4/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOMRatisSnapshots.java#L151
Most of it already existed already, (to test the existing bootstrapping code from end to end.) I just added a snapshot test to it here:  https://github.com/apache/ozone/blob/ed069f01142452dfc59b3f9703e486ef0434d7e4/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOMRatisSnapshots.java#L243
I believe it does all of your steps except a configurable number of snapshots.  That shouldn't be hard to add.  Is it ok if I just make the configuration a parameter in the test file itself?  Would that satisfy this request?

@smengcl you ask "Do we have a consensus here?"  in a couple of places above.
I've responded to one.  But I'm not sure what the second one is referring to.  Are they the same thing?

@smengcl you ask "Do we have a consensus here?" in a couple of places above.
I've responded to one. But I'm not sure what the second one is referring to. Are they the same thing?

Ah. Actually it is the same one. GitHub is showing this in a confusing way when I stage the review comments in one batch so it looks like I have poste twice.

@smengcl @prashantpogde
I've updated the code based on your requests.  In a few cases, I've pushed the work off to two other PRS:
Handle large tarballs in bootstrapping
https://issues.apache.org/jira/browse/HDDS-8208
Synchronize tarball creation with background processes
https://issues.apache.org/jira/browse/HDDS-8209
In one case I was unable to make the change Prashant requested.  I explain my reasoning here:  https://github.com/apache/ozone/pull/3980#discussion_r1144093843
Please take a look when you can.
Also, it looks like you need to start the CI workflow for me.

@hemantk-12 I've resolved all your suggestions

@GeorgeJahad There is one checkstyle warning:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/OmSnapshotUtils.java
 56: First sentence should end with a period.

@GeorgeJahad Please fix the CI failure and I can merge the PR.

@smengcl @prashantpogde  The merge conflict is resolved.

CI passed. I will merge this in a minute.
Thanks @GeorgeJahad a ton for working on the patch. Thanks @hemantk-12 @prashantpogde for reviewing this.


Train 167

Set ConditionalPathExists in correct area so docker-gc-enabled works. #1543
Disable opening the UI in iframe #1506
Remove dcos-launch #1587

@mesosphere-mergebot bump-ee

This needs debugging work in the downstream PR.

@mesosphere-mergebot bump-ee


data_type for odbc backend for numeric and decimal types

The data type for oracle type NUMBER(2) connecting to the oracle DB via ODBC was mapped to double. This is not as needed, also not as implemented in include/soci/column-info.h.
With this commit it is similarly handled.

Thank you for the patch! This looks correct to me, but ideal would be to have a test showing that binding a NUMERIC(n) column to an integer works now. We already have ul numeric(20) column in the table created by table_creator_1() (see tests/common-tests.h), so it shouldn't be difficult to add a test checking for this. Could you please do it? This would allow us to ensure that this also works with the other backends correctly and, perhaps even more importantly, that it keeps working in the future.
Thanks in advance!

I am testing this with odbc, when the underlying server is oracle. But on the server I test it I cannot install all other servers. How can I test them?

Could you please explain why did you add these YAML files? There is, of course, already ci.yml in SOCI...

Could you please explain why did you add these YAML files? There is, of course, already ci.yml in SOCI...

You mentioned GitHub CI, and I am trying to figure out how it works. Sorry, I didn't know it will immediately affect this pull request too.
Could you please give some advice how to continue with testing ODBC with Oracle backend? In this build: https://github.com/torbatamas/soci/runs/2388755328?check_suite_focus=true it did not detected ORACLE, so I assume GitHub CI needs some further configuration to be able to test with oracle too, but I didn't find what to do.

You mentioned GitHub CI, and I am trying to figure out how it works.

Just to be clear, SOCI already uses GitHub Actions for its CI, please the existing ci.yml file in master. I.e. there is no need to add new workflows, although it may be useful to modify the existing ones.

Could you please give some advice how to continue with testing ODBC with Oracle backend?

Do you have a working ODBC/Oracle configuration locally? If so, could you please confirm that the new test, that you've added, fails there and it passes with your changes?

In this build: https://github.com/torbatamas/soci/runs/2388755328?check_suite_focus=true it did not detected ORACLE, so I assume GitHub CI needs some further configuration to be able to test with oracle too, but I didn't find what to do.

There is already a job using Oracle and a job using ODBC, so it would be possible to combine them and run ODBC tests with Oracle too, but I'm not asking you to do this (which doesn't mean that it wouldn't be welcome if you did it, of course!) because it's bit complicated.


getThumbnailURL() Perfomance issues

I wanntet to use the thumbnails that concrete5 can create when uploading.
so i followed this.
so my code is
$type = \Concrete\Core\File\Image\Thumbnail\Type\Type::getByHandle('file_manager_listing');
$file = CustomDocumentsHelper::get_file_data($f);
$src = $file["image"]->getThumbnailURL($type->getBaseVersion());
<img src="<?=src  ?>" class="img-responsive" alt="<?= $file ['name']; ?>" />

all works fine if its just 1 img
but if i have lets say 20 the site takes arround 5 more seconds to load
with 50 images it takes ages to load

This is actually due to the middleware that generates the thumbnails on page load, getThumbnailURL() has basically no perf impact.
see: https://github.com/concrete5/concrete5/issues/5116 and https://github.com/concrete5/concrete5/issues/5049

We should limit the amount of files that are converted to 5 per page load, is that too high?

With #5105 it can be lowered to 1

IMHO this issue can be closed (we now have the possibility to create thumbnails in a synchronous or asynchronous way)


[RFE] tcpdump / wireshark cap/pcap output for tcp trace

Current situation
Output options for tcp trace only cover json, jsonpretty, yaml, columns.
$ kubectl gadget trace tcp --help | egrep '--output'
  -o, --output string          Output format (json, jsonpretty, yaml, columns).

Impact
I could not find a way to import the tcp trace output to wireshark for further analysis.
Wireshark native format for input data is pcapng, but it can import multiple formats, including pcap, from libpcap / tcpdump
Ideal future situation
New output option(s), that can be redirected to a file, that can be imported and usable under wireshark-like tools.

trace tcp only traces specific events for tcp connections (Trace TCP connect, accept and close). We have thought about implement a tcpdump like gadget, there is even a WIP PR in https://github.com/inspektor-gadget/inspektor-gadget/pull/1097, but we haven't finished it yet.


Media Library delete folder could delete media from neighbor folders.

https://github.com/OrchardCMS/Orchard/blob/89c562c50216498ecf3b0a87d7ca8832005b4cdf/src/Orchard.Web/Modules/Orchard.MediaLibrary/Services/MediaLibraryService.cs#L98
Example:

You have 2 folders: Folder1 and Folder11
Delete folder Folder1.
Files from Folder11 will also be removed because you cannot search sub-folders with StartsWith function.

This issue related to DeleteFolder and RenameFolder methods.

Nasty one, it needs to be fixed ASAP


403 forbidden

Hey there , i appreciate your work!
i linked the app to my basic django website and i added all the ebay keys , i deployed the website to heroku and suddenly i get 403 forbidden when i try to access 'https://www.example.com/ebay_accounts/'
do you know what causes this issue?
PS: im juste a newbie experementing
Thaanks!

Hi,
Thanks for trying this out :)
You might need to set up a user with the right permissions using the django admin.


ARM64 support

I notice ARM64 support for pocketbase - can I run pockethost and poscketbase on an ARM64 architecture? Cheers, Dave

@kubeworkz This is the discussion area for PocketHost. Your question is about PocketBase, please post in https://github.com/pocketbase/pocketbase/discussions.


refactor dataload use dataoperation reconcile

Ⅰ. Describe what this PR does
refactor dataload use dataoperation reconcile
Ⅱ. Does this pull request fix one issue?

fixes None
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Existing e2e test
Ⅴ. Special notes for reviews

/test fluid-e2e

/lgtm


Set country as well as lat lng when geocoding

Moved the code from the after party task to a rake task:
bundle exec rails school:set_countries


Add a way to shutdown the server gracefully

The decision of when and how should be left to the application.

Possible to add a test?

We should have proper e2e at some point.


Sentence similarity: Training failed when choosing File Validation Option.

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-22H2
ML.Net Model Builder 2022: <IP_ADDRESS>1602 (Main Build)
Microsoft Visual Studio Enterprise: 2022(17.8.7)
.Net Core: 6.0,7.0,8.0
Describe the bug

On which step of the process did you run into an issue:
Training filed when choosing File Validation Option.

TestMatrix
https://github.com/dotnet/machinelearning-modelbuilder/files/10534387/home-depot-sentence-similarity-small.csv
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio start window.
Choose the C# Console App (.NET Core) project template.
Add model builder by right click on the project.
Select "Sentence similarity" scenario.
On the Data page, choose the data source, then choosing File Validation Option on the "Advanced data options" dialog.

On the Train page, click "Start training" button, you will see that the training failed.

Expected behavior
Training successful.
Screenshot:

Additional context:
The bug is not repro when choosing Cross validation or Split validation option.

@v-Hailishi , this is related with data formatting option in advanced data options, if you make  allow quote set to yes it will work

@zewditu Sorry I am not clear the data source. I have updated the title and the repro steps. When training the SQL server data, it is not supporting the data formatting.

Got it, thanks,

Long term fix  work item https://github.com/dotnet/machinelearning-modelbuilder/issues/2861


Calculate molecular weight

We already have molecular structure in .mol format provided by Drugbank data.
from drug_named_entity_recognition.drugs_finder import find_drugs

drugs = find_drugs("i bought some Bivalirudin".split(" "), is_include_structure=True)

self.assertEqual(1, len(drugs))
self.assertEqual("Bivalirudin", drugs[0][0]['name'])
self.assertIn("0.0000 C", drugs[0][0]['structure_mol'])

Can we convert to SMILES on the fly? Can we calculate molecular weight on the fly?
Ideally can you do this without adding anything more to requirements.txt? There are some chemistry libraries but they can be quite heavy.

Hey @woodthom2, wanted to check if it is still open for contributions. If so, I would like to contribute.

Hi @abdullahwaqar yes this is still open! Can you see a way to add molecular weight, or a data source which will give us the molecular weight?
Here's an example for one single drug:
https://www.opnme.com/molecules/khk-inhibitor-bi-9787
we have Weight: 489.6 DA, or also properties such as tmax and Cmax. I don't know if such a database exists, it's possible Drugbank gives the data to us but we need to check licences.
Also, you can see in the link that I pasted there is a nice moving 3D image of the drug. The positions in the atoms are now returned by the library in a string format (you can see my example for paracetamol here: https://fastdatascience.com/ai-in-pharma/drug-named-entity-recognition-update-2/#molecular-structures ) - it would be nice to have a Jupyter notebook (or Colab notebook) or in-browser example of us rendering this molecular structure, either as a static image or a dynamic view of some kind. I definitely do not want to add any graphics libraries as dependencies to the project, but having this as an example would be great.
The molecular structure data that we have could also be a shortcut to getting the molecular weight of a drug..
Thanks!

https://pubchem.ncbi.nlm.nih.gov/docs/downloads#section=Individual-Record-Download <- this is a good source for molecular weight which appears to be allowed for our use. I cannot take the molecular weights from Drugbank because it is not allowed under the license.


bots: Add manual command to get bot output.

This eliminates the need to setup dev environment and to create a bot, setup zuliprc file, subscribe the bot to the stream in order to try out a bot.
Manual command get-bot-output gives the bots response content directly.

@derAnfaenger Thanks for the in-detail review. :)
I have tried to address all of your comments. It would be really great if you review once again.

@timabbott tested, LGTM! I think this is stable enough so we can merge this, and others can work on follow-ups. Thanks again to @abhijeetkaur for working through all those suggestions and writing this very helpful script!

Some follow-ups I can think of:

instead of extra-checking for equality in case of two specified botnames (one path and one name), we might just want to mutually exclude these arguments

there is a little extra space next to the output's quotation marks because of the list we pass to print,
should remove that.

instead of copying ExternalBotHandler's get_config_info method, we should use the original one by setting Mock's wrap argument.

Sounds good, I agree with all of those follow-ups.  I'd add:

It might make sense to have at least a symlink to the tool under tools/ so it's more obviously available as a developer tool.
We should document it.

I merged this, after renaming is_different to name_and_patch_match for clarity, and shortening the super-long comment about send_reply vs. send_message.
Thanks @abhijeetkaur and @derAnfaenger!


Bump golang.org/x/net to v0.7.0

To pull improvements from the library.

fyi this should get bumped by the rebease PR as well: https://github.com/openshift/route-controller-manager/pull/21

Closing in favor of https://github.com/openshift/route-controller-manager/pull/21


iTunes Connect > App Store Connect

In both the actual code, and readme of Harpy, the references to and bundle ID of iTunes Connect should be switched to App Store Connect, since iTunes Connect no longer exists.

thanks!


Error at docker-compose up --build

sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) FATAL:  password authentication failed for user "postgres"
My example .env file:
POSTGRES_SERVER=postgres
POSTGRES_PORT=5432
POSTGRES_USER=postgres
POSTGRES_PASSWORD=postgres
POSTGRES_DB=postgres
FIRST_SUPERUSER=postgresql
FIRST_SUPERUSER_PASSWORD=postgresql
USERS_OPEN_REGISTRATION=False

if you're using the postgres docker container than the POSTGRES_SERVER should be named as the db container name, which is db, or you can change the postgres container in the docker-compose file from db to postgres. That's the issue that I spotted in your .env file, but concerning the password authentication, the .env file is used by both of db and api containers, so it can't be different.

if you're using the postgres docker container than the POSTGRES_SERVER should be named as the db container name, which is db, or you can change the postgres container in the docker-compose file from db to postgres. That's the issue that I spotted in your .env file, but concerning the password authentication, the .env file is used by both of db and api containers, so it can't be different.

Ok thanks bro :)


use an app-specific password for 2 auth

What about use an app-specific password when signing in? https://support.apple.com/en-us/HT204397
In that case you won't ever have to reauthenticate...

Hi, I think I did try that earlier, but I wasn't able to get it to work. I believe we're using a private API for iCloud, so it didn't support app-specific passwords when I tested it last year. That would be awesome if you could please try that out and let me know if it works now!

Doesn’t work can be closed.


[BP-1.16][FLINK-31967][table-runner] Solving the Initial Value Problem of LagA…

What is the purpose of the change
Slove the Initial Value Problem of LagAggregation
Brief change log
dermine if the default value needs to be converted from non-null to null
Verifying this change
This change added tests and can be verified as follows:
org/apache/flink/table/table/planner/runtime/stream/sql/AggregateITCase#testLagAggFunction
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? ( no)

@flinkbot run azure


Update faucet links

Adds QuickNode's Sepolia faucet link to Bridging Assets page
Updates the faucet page to include QuickNode's Cyber Sepolia Faucet
Adds Faucets to the sidebar

Thanks for the PR! @zer0cache


Welcome #1

          Duplicate of #2

Originally posted by @ShomaVGujjar in https://github.com/skills/connect-the-dots/issues/32#issuecomment-1647551432

Duplicate of #2

Duplicate of #2

Duplicate of #2

"Duplicate of #2"

Duplicate of #2

Duplicate of #2 #42


Don't try to auto update if the data service can't be stopped

Right now, on automatic update, if this error occurs:
[2023-10-24T08:24:11.398Z] ERROR    ZitiUpdateService.UpdateService    Timout while trying to stop service!    System.InvalidOperationException: Cannot stop ziti service on computer '.'. ---> System.ComponentModel.Win32Exception: The service has not been started
   --- End of inner exception stack trace ---
   at System.ServiceProcess.ServiceController.Stop()
   at ZitiUpdateService.UpdateService.StopZiti()

The application will continuously try to update but never succeed because the data service is hung somehow. The user will need to restart Windows, or terminate the ziti-edge-tunnel service before updating can continue. This should be logged and probably surfaced as an error/message to the user to reboot/take action.

the monitor service will forcefully terminate the data service now


Incorrect deletion of messages

Why, when I send a message through my application, in UpdateNewMessage I get such<PHONE_NUMBER>) an message id, and when I delete the same message through the official telegram, then in UpdateDeleteMessages I get a different message id<PHONE_NUMBER>)

#511


Train on ScanRefer dataset

Dear Authors,
Thanks for your great work! When I execute the command "python train.py data=scanrefer experiment_name=scanrefer +detector_path=checkpoints/PointGroup_ScanNet.ckpt Model.net work. The detector. Use_gt_proposal = True ", appear the following error:
File "/data1/zyh/M3DRef-CLIP/m3drefclip/data/dataset/scanrefer.py", line 35, in _load_language_data
"eval_type": item["eval_type"],
KeyError: 'eval_type'
File "M3DRef - CLIP/dataset/scanrefer/metadata/ScanRefer_filtered_train json" does not contain 'eval_type'.
I noticed that when training on the nr3d dataset, its 'eval_type' value is added as an extra in the code.https://github.com/3dlg-hcvc/M3DRef-CLIP/blob/babfcc4b4f06bb8ce1471513095a1c80acc0edb5/m3drefclip/data/dataset/nr3d.py#L41-L50.
So I would like to ask whether it is necessary to add an extra 'eval_type' as when training on the nr3d data set?

Hi @yihang-xdu , I think you skipped step 2 for ScanRefer dataset preprocessing (please follow this instruction). The script will add the eval_type for you.


vNet Injection deployment documentation is ambiguous

During a recent deployment of Databricks, we came across a number of issues involving the network.
The customer is using UDR to NVA and also forced tunneling back to onprem data centers for internet connectivity.
The document lacks clarity on deploying UDR's for access to certain Azure Public services as listed here:
https://docs.microsoft.com/en-ca/azure/databricks/administration-guide/cloud-configurations/azure/udr#udr
We had a large number of failed deployments with little to no detail on the error/failure.
All UDR were in place. Deployments would get to a point of creating virtual machines in the resource group, but could not start the containers.
Eventually, we setup service endpoints on the public subnets and the deployment would work.
Then, we began to remove the UDR's and the service and deployments also worked.
Ask: Please update the documentation to reflect a use case scenario and when to use service endpoints over UDR.
Thanks,
[Enter feedback here]

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

ID: 93bd5c16-e0ec-5064-9bc0-1c527c328a7e
Version Independent ID: 5342d01e-c2da-f7a8-d22e-86995a17af79
Content: Deploy Azure Databricks in your Azure Virtual Network (VNet Injection) - Azure Databricks
Content Source: databricks/administration-guide/cloud-configurations/azure/vnet-inject.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@njsee Thanks for the feedback. We are currently investigating into the issue and will update you shortly.

@mamccrea Can you please take a look at this ?

Thanks for your feedback @njsee. We are currently reviewing your request.

Hi @njsee - thanks again for providing this feedback. We've created a work item to update this article. I will report back here once it has been updated. If you have any additional feedback or questions, please reopen this issue.
#please-close


Add ROADMAP.md to the project and cleanup existing items

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

cc @amylindburg @dmp42

The target date for Platform 1.8 is probably sooner -- convention is the first Tuesday in (every other) month -- so this would be August 4th. Let me get the project pages live ... I will send you the links...

Okay -- the wiki is live. Here are the links to the project pages:
https://github.com/docker/distribution/wiki/docker-1.6-Project-Page
https://github.com/docker/distribution/wiki/docker-1.7-Project-Page
https://github.com/docker/distribution/wiki/docker-1.8-Project-Page
There are "starter" goals in 1.7 and 1.8; we need to fill out the Release Goals (AKA Release Notes) for 1.6

I changed the links in the Wiki to point to this ROADMAP.md (when it is merged).

LGTM as a good first step. We can refine later.

LGTM - there's minor stuff - like a few lines that are way more than 80 chars.

@SvenDowideit The only lines over 80 characters are those with URLs. Is that an issue?

not imo :)

That is the only one I didn't check. Please submit a PR for formatting, if you are so inclined.


Web - New Room - Entered name cleared when changing the size of the window

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Go to any Expensifail account
Click on FAB button -> New room
Enter some symbols on the Room name input field
Change the window size

Expected Result:
Entered name NOT cleared
Actual Result:
Entered name cleared
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/aaeb5a5d-3a0b-433b-8a0b-27d3e4dce3d7
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Related to https://github.com/Expensify/App/issues/20404.

100% agree

closing, i think it's 1 in the same


Make isObjectShallowModified treat NaN equal to NaN

Sorry for the delay!
Fixes #363

Thanks for the PR.
Note that some of the tests where failing in this branch, because the isNaN check was not correct (cause isNaN({}) === true).
Next time make sure all tests pass :)
Merged now manually

I just added my VSCode debugging settings to the repo. That might make it
easier for next time. Beyond that I use mainly yarn jest --watch
Op do 1 feb. 2018 om 20:00 schreef Andrew Branch<EMAIL_ADDRESS>
@mweststrate https://github.com/mweststrate woops, thanks for fixing.
For what it's worth, I was unable to get the full test suite running
locally. The Electron shell would boot up, but then everything just hangs.
I could run individual tests with the Jest plugin for VS Code. I'm on Node
7.10.0. Maybe worth looking into?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/pull/402#issuecomment-362367177,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhIdcb5PmcldNaO8b89S4z_BSxQhKks5tQgm8gaJpZM4RvdXU
.


KAFKA-16156: beginningOrEndOffsets does not need to build an OffsetAndTimestamps object upon completion

A subtle difference in the behavior of the two API causes the failures with Invalid negative timestamp.
In this PR, the list offsets response will be processed differently based on the API.  For beginingOffsets/endOffsets - the offset response should be directly returned.
For offsetsForTimes - A OffsetAndTimestamp object is constructed for each requested TopicPartition before being returned.
The reason beginningOffsets and endOffsets - We are expecting a -1 timestamp from the response which subsequently causes the invalid timestamp exception because the original code tries to construct a OffsetAndTimestamp object upon returning.

Hey @lucasbru - Would it be possible to ask you to review this PR? Thanks!

Hey @lucasbru - Thanks for taking the time to review this PR.  Let me know if there's anything to add to the PR.

@lucasbru - Thanks again for reviewing the PR. Sorry about the misinterpretation on short circuting logic so here I updated the beginningOrEndOffsets API. It seems like the right thing to do here is to still send out the request but return it immediately for zero timeout (a bit strange because it does throw timeout when time runs out which seems inconsistent).

Yes, the behavior of the existing consumer is a bit curious, but it's not the only place where a zero duration is treated different from 0.000001s. Either way, we probably have to do it this way for compatibility. This part looks good to me now.

@philipnee Okay, thanks for creating the ticket. Not sure if it's blocker priority though. If it's a quick thing, you could address it in this PR.
Are you going to implement Lianets suggestion?

hi @lucasbru - Let me address Lianets comment in this PR and have a separated PR for the behavior inconsistency as it does require some changes to the unit test

@lucasbru - Thanks for taking time reviewing this PR.  This PR is ready for another pass.

hi @lianetm - Much appreciate for the reviews.  I think I've addressed your comments. LMK if there's anything more. cc @lucasbru

Hey @philipnee, thanks for the updates, just one minor comment left above.