zero/one to many relationship generating incorrect inner join

I have two tables:
Job, integer ID, string column Container (nullable)
HaulageJob, integer ID, nullable column DeliverJobID with FK relationship to ID on Job
I have classes defined for each of them
public class Itinerary {
        public int ID { get; set; }
        public int? DeliverJobID { get; set; }
        public virtual Booking DeliveryBooking { get; set; }
}

public class Booking {
        public int ID { get; set; }
        public string Container { get; set; }
}

I configure them as follows:
            modelBuilder.Entity<Itinerary>(e => {
                    e.ToTable("HaulageJob");
                    e.HasOne(d => d.DeliveryBooking).WithMany().HasForeignKey(d => d.DeliverJobID ).IsRequired(false);
            });

            modelBuilder.Entity<Booking>(e => {
                    e.ToTable("Job");
                });

I then try to do a query something like this:
_context.Itineraries.where(i=>i.DeliveryBooking==null || i.DeliveryBooking.ContainerNumber==container)
But the generated SQL contains an inner join to the Job, so the DeliveryBooking==null is effectively ignored:
SELECT [...]

To avoid confusion with any simplifications I made to produce the above, I've reproduced it with two self contained tables. Also, never mentioned it above, but this is running on SQL Server 2014.
So here is the script for the tables:
CREATE TABLE [dbo].[ATestC](
	[ID] [int] IDENTITY(1,1) NOT NULL,
	[StringField] [varchar](50) NOT NULL,
 CONSTRAINT [PK_ATestC] PRIMARY KEY CLUSTERED ([ID] ASC )
)

CREATE TABLE [dbo].[ATestP](
	[ID] [INT] IDENTITY(1,1) NOT NULL,
	[ChildID] [INT] NULL,
	[OtherField] [INT] NOT NULL,
 CONSTRAINT [PK_ATestP] PRIMARY KEY CLUSTERED (	[ID] ASC )
)

ALTER TABLE [dbo].[ATestP]  WITH CHECK ADD  CONSTRAINT [FK_ATestP_ATestC] FOREIGN KEY([ChildID])
REFERENCES [dbo].[ATestC] ([ID])

Here are the classes:
    public class ATestC {
        public int ID { get; set; }
        public string StringField { get; set; }
    }
    public class ATestP {
        public int ID { get; set; }
        public int OtherField { get; set; }
        public int? ChildID { get; set; }
        public virtual ATestC Child { get; set; }
    }

Here is the configuration:
    public class SaturnContext : DbContext {

        protected override void OnModelCreating(ModelBuilder modelBuilder) {
            base.OnModelCreating(modelBuilder);

            modelBuilder.Entity<ATestP>(e => {
                    e.HasOne(d => d.Child).WithMany().HasForeignKey(d => d.ChildID).IsRequired(false);
                });
        }

        public virtual DbSet<ATestP> Parents { get; set; }
        public virtual DbSet<ATestC> Children { get; set; }
}

and here is the query:
                var test = _context.Parents.Where(i => i.Child.StringField == "test" || i.OtherField == 5).ToList();
This produces the following SQL:
SELECT [i].[ID], [i].[ChildID], [i].[OtherField]
FROM [ATestP] AS [i]
INNER JOIN [ATestC] AS [i.Child] ON [i].[ChildID] = [i.Child].[ID]
WHERE ([i.Child].[StringField] = 'test') OR ([i].[OtherField] = 5)

If I have  a ATestP record with ChildID: null and OtherField: 5, the Linq query should find it, but the inner join prevents this

This is a known issue that has been fixed after RC1 has been shipped. Essentially every navigation (optional and required) would get translated into INNER JOIN. This is fixed in the current code base, producing LEFT OUTER JOIN in case if optional navigations.

Thank you maumar.

You can use the nightly builds (different nuget source), you don't need to clone the EF repository for that.
Bear in mind that there many many things have changed since RC 1.
And you won't be able to use VS until they finish wrapping the dotnet CLI tooling (Scott said RC2 Will be released in 4-6 weeks)

Thanks Doron, I was thinking much the same - I hadn't realised how much things had diverged from the DNX model until I looked into this.


Boilerplated all necessary public interfaces

Created the structure of the library overall. Mainly based upon the existing JS library, but with some C# changes.
I believe I have captured the most important parts, and can start to actually implement the methods.
Testing
Going to work on this, see how we pull the library into an app.

Good shout @amrc-benmorrow , didn't consider that. Can we use the UniTask nuget package and still have the library be crossplatform or would it require separate dotnet vs Unity builds?

I believe the nuget will work if you pull the DLL into Unity (I think that's the point) but honestly I've never tried it.

@amrc-benmorrow I also meant just using it in some other C# app like a .Net desktop app or ASPNET server? We could rename the library to unity-serviceclient if it's only unity-compatible though. Although our (very simple) sample test app wasn't going to be Unity.


Unstyled component teaser

Please refers to this https://jira.economist.com/browse/TEC-154 about the specs for this implementation and consider styles are out of scope for this ticket.
This component can be used for blog post collections or general article list, this is why I moved custom microdata at instance level.
Inner microdata are the same for Blog posts or articles.

Done all the required improvement except for the tests.
If the tests are not blocking due to the fact that anyway they do the job, I will consider @keithamus suggestion to be applied to the next component due to time constrain I will move to style the component.
Date:
In a separate discussion the team agreed to create a separated component or mixing to provide custom date format especially considering that the format Sep 19th 2015, 9:49 used on the website seems to be out-of-standard at the next needing of a dateFormat, so consider the creation of the component out-of-scope for this implementation.
Please @fampinheiro and @keithamus evaluate commits and give me a feedback.


Fix the issue with "AtlassianSecurityContext" being nil after migration to oauth

Summary
This PR is a continuation of https://github.com/mattermost/mattermost-plugin-jira/pull/999. Tests are added for all permutations of instance type installations, instance migration, and user connection status.
Ticket Link
Fixes https://github.com/mattermost/mattermost-plugin-jira/issues/1001

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (06a02e8) 29.48% compared to head (de010c3) 33.01%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

server/mock_kv_store.go
68.00%
6 Missing and 2 partials :warning:

server/kv.go
40.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1002      +/-   ##
==========================================
+ Coverage   29.48%   33.01%   +3.53%
==========================================
  Files          52       53       +1
  Lines        7848     7890      +42
==========================================
+ Hits         2314     2605     +291
+ Misses       5337     5058     -279
- Partials      197      227      +30

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


use bang to switch behaviour between lcd and cd

Sometimes I want to lcd, sometimes I want to cd. The use of the bang is a nice convenience for swapping the behaviours. Using the bang causes :rooter to do the opposite of whatever it's set to by default. Right now I'm just doing this:
function! s:rooter(lcd)
  if a:lcd
    let g:rooter_use_lcd = 1
  else
    let g:rooter_use_lcd = 0
  endif
  Rooter
endfunction
nnoremap <silent> <unique> <Leader>gl :call <SID>rooter(1)<CR>
nnoremap <silent> <unique> <Leader>gc :call <SID>rooter(0)<CR>

It's not obvious to me that Rooter! would toggle cd and lcd.  I'm not inclined to add functionality just to toggle a config option: it feels like the extra code isn't worth it.


Envelope-From setting

There seems to be no way to set a value for Envelope-Sender / Envelope-From to use with an SMTP server, which seems to be required by some servers.
Would it be feasible to add a setting for this?

Those are not standard headers and not something that is added by a mail client. SMTP servers often add these.

Maybe I did not get the correct name of the field, and it's just the way other clients name it in the settings. But last time I discussed this in a k9 mail forum, I was told this is from RFC 821, so that would make it a standard header.

From your description it's not clear to me what the actual problem is. If you have something concrete to point to, please do.

Some servers I use won't accept arbitrary addresses in the message From: field unless the Envelope-From is set to an address that belongs to the same server.

I don't believe a mail client for end users should use a sender address in the SMTP session that's different from the one in the actual message. I haven't seen this supported by any other client either.

I have been able to do this with different clients on a desktop computer. Still, regardless of whether other clients support it or not, it would be convenient for me if k9 mail had this feature. Otherwise I have to get a different mail server just to use k9 mail as a main mail client.


Questions about synthetic data scene name

Hi,
Is the blurwine scene the same as Trolley scene in the paper?
Thank you.

Yes, they are the same


Backport #368 - json value and builder annotations support for enums

Added changes so that json builder and creator could support null values.


Please expand

Great reference material, those of us in HIM would benefit greatly from more information, such as potential use cases, alternative services, implementation considerations, or pricing guidance.

Thank you for bringing this to our attention.  To better assist you, could you please identify which document this is in reference to?

Hi @wmstjean
Sorry for the delay and thanks for the suggestions.
This current article is a  “Solution idea” which its goals is to keep it simple, here there is more information about healthcare which would cover your concerns
https://learn.microsoft.com/en-us/industry/healthcare/
https://learn.microsoft.com/en-us/azure/architecture/industries/healthcare
I will close the ticket, but feel free to re-open if additional clarification is needed.

#please-close

@ChrisCynefin -- Content signal to expand a solution idea into an example workload.


Add support for multiple network interfaces

Currently, we only support one network interface per network namespace.
We should add support for multiple network interfaces in the same network namespace.

Since we do not have any specific use case for this yet, we can close it and reopen it, when we have a use case


Review request/response objects and make sure we're happy with them

This might be an opportunity to rename the statuses and ensure they're consistent ("success" vs "failed").
Do we want the input type to have a second input level so we can have top-level request options?
Migrated from a todo in https://github.com/replicate/cog/pull/378.

Closing. This is being covered in #413


[SPARK-15315][SQL] Adding error check to  the CSV datasource writer for unsupported complex data types.

What changes were proposed in this pull request?
Adds error handling to the CSV writer  for unsupported complex data types.  Currently garbage gets written to the output csv files if the data frame schema has complex data types.
How was this patch tested?
Added new unit test case.

Can one of the admins verify this patch?

Just for a note, JSON data source is doing this in  via JacsonGenerator.apply() which is equivalent with CSVRelation.rowToString(). However, it seems JSON is doing this because it needs to traverse the schema as a tree to verify if each type is supported or not. So, it seems this is doing this while writing each row.
For CSV, it does not need to traverse a tree but just look the root types because it does not support nested types. So, schema can be checked faster.
Therefore, it seems reasonable to me.

CC @rxin

cc @cloud-fan / @liancheng

ok to test

Test build #59078 has started for PR 13105 at commit 87c6f27.

Test build #59078 has finished for PR 13105 at commit 87c6f27.

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

Merged build finished. Test PASSed.

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

thanks, merging to master and 2.0!

Thank you for merging the PR, Wenchen.


Add support for NanoRange

This PR (hopefully?) adds support for the NanoRange library, an independent implementation of the C++20 ranges proposals.
Image PR: https://github.com/mattgodbolt/compiler-explorer-image/pull/261

Thanks for the PR


Improve load time of location endpoint

Short description
Reduced the load time of this endpoint, by reducing db calls and removing deep copy
Proposed changes
I used cProfiler to get insights to the loading times and snakeviz to visualize them, getting this output for the location endpoint:

We can see that from 1.70sec total time of this request, 1.078s were spent in deepcopy. Also we see that only 583 calls of the 394670 calls are non-recursive, indication a infinite loop. After removing the deepcopy, we get this result:

The total time already decreased to 0.717s and the most time is spent in 102 database calls. These were made due to getting the default translations of objects. Getting the default translations at once, we can reduce the number of database calls to 5, and the total time to 0.075s.

So at the end with 2 small changes the load time decreased from 1.7s to 0.08s.
The test-scenario contains about 120 locations.
Side effects
Resolved issues

Fixes: #1989

Pull Request Review Guidelines

In addition to my comments, I noticed that the poi categories also cause a lot of duplicated queries, I would suggest to prefetch them (including their translations) as well.

I also saw this yesterday and I am struggling with this. I try to pretech POICategoryTranslation of all Pois of a region, but POICategory inherites from AbstractBaseModel and not from AbstractContentModel, therefore I cannot use functions like "prefetch_public_translations" to prefetch them.
So I thought if might make sense to let POICategoryTranslation inherit from AbstractContentModel, but then it would need to have a "region" and this does not make sense here.
Other thought I have is to add a "prefetch_public_translations" method to either POICategory or AbstractBaseModel. To AbstractBaseModel does not really make sense, as not every class inheriting from AbstractBaseModel needs or has translations. But to move it to POICategory seems to create a lot of overhead and also won't work, as POICategroyTranslations is missing all the translation-related-fields like "status, ...".
So I decided to add (copy) the prefetch methods to poi_category.py, but the result is not saved, "prefetched_translations" is not available, so always a db call is made. I do not really know how i should call these methods in locations.py. Can you help please?

Thanks for reviewing. I adjusted the prefetching of category translations.
I started to test with test_poi data, but there are some problems:

loading the pois works fine, with the difference, that "organization" and "barrier_free" options are not passed, and on develop they are passed but null.
When i want to change the category of a poi in the ui, i get an error

Exception Type: ValueError
Exception Value: not enough values to unpack (expected 3, got 1)
Exception Location: /home/steffi/Dokumente/integreat-cms/integreat_cms/cms/utils/translation_utils.py, line 40, in translate_link

As this is not ready and needs fixes, no need to review now. I will continue to work on this next week.

Awesome work and great investigation!


Is it possible to change the default audio and video settings?

Hi,
I have been using your project with great success so far. The only I can't do yet is change the default settings or use any commands for the picam. Is it possible, with this project, to control the picam directly or at least via settings?
Thanks,
Mathew

Hi,
May be you can read this issue. It contains an example of changing the picam settings.
[https://github.com/iizukanao/picam-streamer/issues/3]
In line 44 you can add or change your settings of picam.
I changed the image size in this way.
Succes,
Er


Possibly incorrect cell selection to match Nature Energy paper

👋 Hi!
The discards cells listed here: https://github.com/TRI-AMDD/beep/blob/cca82e75a241f4545cc4f0fa86c606f1d057690a/scripts/batch_generate_structure_feature.py#L31 look like they were designed to match the cells listed here: https://github.com/rdbraatz/data-driven-prediction-of-battery-cycle-life-before-capacity-degradation/blob/master/Load Data.ipynb. However, I don't think the cell numbers in the second link refer to channel numbers, as their use in the first link implies. The notes for each of the batches here https://data.matr.io/1/projects/5c48dd2bc625d700019f3204 describe a different set of cells being left out of the Nature Energy paper due to various issues. Table 9 in the supplementary info of the paper has barcodes consistent with the cells referenced in the third link, and is inconsistent with the cells "discarded" in the first link.
Please do let me know if I am misunderstanding something. I'm trying to get the "cleanest" dataset possible from the data collected in the Nature Energy paper. My current understanding is that this requires removing the cells listed in the third link above.

@jkoeller The cells listed on the data.matr.io/1 page should be regarded as the most canonical. As always, discarding outliers is more of an art than a science and different people have made different choices over time 🤷

Understood. Just wanted to point out this discrepancy to prevent future confusion!

Yeah, thanks for pointing that out. We can definitely change the beep discards to match the Nature Energy paper!


feat!: add support for BatchCheck API

BREAKING CHANGE: Usage of the existing batch_check should now use client_batch_check instead,
additionally the existing BatchCheckResponse has been renamed to ClientBatchCheckClientResponse.
Description
This PR introduces a method for using the BatchCheck API released in OpenFGA v1.8.0, in order to facilitate this it includes a breaking change where the existing batch_check method (and associated type) has been renamed to client_batch_check. This is so that our naming conventions remain consistent.
Migration
"I want to continue using client side batch check"
Given that this necessitates an upgrade of OpenFGA, it might not be feasible for everyone to switch to the new server based method, if you wish to delay the migration, rename the existing usage to the new method name as shown below.
- client.batch_check()
+ client.client_batch_check()

"I want to migrate to the new server based batch check"
If you wish to migrate to the new method, whilst the method name remains the same. You will need to alter the way you construct the checks passed.

Previously a list of ClientCheckRequest was constructed and passed directly to batch_check, now you should construct a list of ClientBatchCheckItem and pass a ClientBatchCheckRequest to batch_check with that list as the checks property

The correlation_id on a ClientBatchCheckItem is set for you if you do not provide it.

The result now contains a correlation_id property in addition to the error and request types and has removed the response property. Additionally, the error property is now of a CheckError type, rather than an Exception type.

checks = [
-  ClientCheckRequest(
+  ClientBatchCheckItem(
        user="user:1",
        relation="owner",
        object="document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a",
        contextual_tuples=[
            ClientTuple(
                user="user:1",
                relation="owner",
                object="document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a",
            )
        ]
    ),
-  ClientCheckRequest(
+  ClientBatchCheckItem(
        user="use:2",
        relation="owner",
        object="document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a"
    )
]

- result = fga_client.client_batch_check(checks)
+ result = fga_client.batch_check(ClientBatchCheckRequest(checks=checks))

# response.result = [{
#   allowed: true,
#   correlation_id: "de3630c2-f9be-4ee5-9441-cb1fbd82ce75", # generated by the SDK
#   tuple: {
#     user: "user:1",
#     relation: "viewer",
#     object: "document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a",
#     contextual_tuples: [{
#       user: "user:1",
#       relation: "editor",
#       object: "document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a"
#     }]
#   }
# }, {
#   allowed: false,
#   correlation_id: "6d7c7129-9607-480e-bfd0-17c16e46b9ec",
#   tuple: {
#     user="user:2",
#     relation="own",
#     object="document:0192ab2a-d83f-756d-9397-c5ed9f3cb69a"
#   },
#   error: {
#     input_error: "validation_error",
#     message: "type 'doc' not found"
#   }
# }]

References
Review Checklist

[x] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

BREAKING CHANGE: Usage of the existing batch_check should now use client_batch_check instead,
additionally the existing BatchCheckResponse has been renamed to ClientBatchCheckClientResponse.

Changes LGTM 👍


Remove windows download from release list page

This change is

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

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

releases/v19.2.0-alpha.20190701.md


Element giving some standard behaviour

Why it is needed?
To make the library easier to use in an application, a wrapping element would really help.
Possible implementation
Covering various situations should be possible but maybe a minimum of handling permissions that are required before a children can be displayed.
Code sample
Usage
<PermissionsRequired android={[PERMISSIONS.ANDROID.ACCESS_COARSE_LOCATION, PERMISSIONS.ANDROID.ACCESS_FINE_LOCATION]}
                     missing={<Text>This cannot be displayed without the correct permissions</Text>}>
  <Text>Permissions have been granted</Text>
</PermissionsRequired>

Could result in showing an ActivityIndicator whilst it calls the existing checkMultiple then requestMultiple methods then either "This cannot be displayed without the correct permissions" if the permissions checks failed or "Permissions have been granted" if they succeeded.

Something like this:
import React, {useEffect, useState} from 'react';
import {checkMultiple, requestMultiple, RESULTS} from 'react-native-permissions';
import type {Permission} from 'react-native-permissions/src/types.ts';
import {ActivityIndicator, Platform} from 'react-native';

interface Props extends React.PropsWithChildren {
  missing: React.JSX.Element;
  androidPermissions?: Permission[];
  iosPermissions?: Permission[];
}

export default function PermissionsRequired(props: Props): React.JSX.Element {
  const [busy, setBusy] = useState(true);
  const [missingPermissions, setMissingPermissions] = useState(false);

  const permissions = Platform.OS === 'android' ? props.androidPermissions : props.iosPermissions;

  useEffect(() => {
    if (permissions === undefined || permissions.length === 0) {
      setBusy(false);
      return;
    }

    checkMultiple(permissions)
      .then(r => {
        const failed: Permission[] = [];
        for (let perm of permissions) {
          if (r[perm] !== RESULTS.GRANTED) {
            failed.push(perm);
          }
        }
        if (failed.length > 0) {
          requestMultiple(failed).then(rec => {
            let anyMissing = false;
            for (let perm of permissions) {
              if (rec[perm] !== RESULTS.GRANTED) {
                anyMissing = true;
              }
            }
            setMissingPermissions(anyMissing);
          });
        }
      })
      .catch(e => {
        setBusy(false);
        setMissingPermissions(true); //Should we show a different error here?
      });
  }, []);

  return busy ? <ActivityIndicator /> : missingPermissions ? props.missing : <>{props.children}</>;
}

A hook is planned for v5: https://x.com/zoontek/status/1798829383559602444/photo/1
As it requires synchronous check, it couldn't be added to v4 (it's breaking)


"Error: fetch failed" on node 18 behind corporate proxy

I upgraded my pipeline to use node 18.17.1. With that upgrade I get an error "Error: fetch failed". (see screenshot below).
Downgrading to node 16 resolves that error.
My pipeline runs behind a corporate proxy server.
My wild guess for what might be happening is this:
Node 18 introduced a global fetch api.
I'm guessing that the http client falls back to using that native fetch api when it's available, without honoring the proxy settings.

Lucky coincidence, this should be fixing that: https://github.com/nrwl/nx-set-shas/pull/106
Can you double-check, please? The version 3.3 updated repo to Node v18 and bumps the version of octokit dependencies (which require v18) so hopefully that should solve it.

@meeroslav  works like a charm! Thank you very much!

@meeroslav should that version bump requirement have been a major version bump rather than a minor version? It has recently caused some failures in our CI process.

This in particular was tested on Node 14 and works the same way as it does on v18. Initially we wanted to make it a major version (v4) but after testing we bumped it down to just minor.

@meeroslav interesting ... we are on node 14 and it does not work. we had to pin everything to 3.2.2 and the problem was resolved.

@dekkofilms can you create a new issue and report what error you get?
I tested with two of our repos and didn't encounter any issues.


docs(jsdoc): change jsdoc of .basePath

The author should do the following, if applicable

[ ] Add tests
[ ] Run tests
[x] bun run format:fix && bun run lint:fix to format the code

@EdamAme-x
Please don't create the small PRs separately.

@EdamAme-x
小さな PR を個別に作成しないでください。

I'm sorry. Should I put them together file by file?


suggested fix for issue #384

The compiled rules keys now contains the name of each RuleParameter, sorted alphabetically.
Without this, the attached test fails.

Looks good to me. I've pushed that suggested change.
On Sun, 4 Sept 2022 at 07:13, Abbas Cyclewala @.***>
wrote:

@.**** commented on this pull request.
In src/RulesEngine/RulesEngine.cs
https://github.com/microsoft/RulesEngine/pull/386#discussion_r962256595:

@@ -343,7 +345,7 @@ private List ExecuteAllRuleByWorkflow(string workflowName, RuleP

     private string GetCompiledRulesKey(string workflowName, RuleParameter[] ruleParams)
     {

       var key = $"{workflowName}-" + string.Join("-", ruleParams.Select(c => c.Type.Name));

       var key = $"{workflowName}-" + string.Join("-", ruleParams.Select(c => $"{c.Name}_{c.Type.Name}"));

This should fix the mentioned issue but will make the key longer.
Maybe we can hash the ruleParameters part of the key to ensure length of
string is always constrained?
Something on the lines of
var ruleParamsHash = string.Join("-", ruleParams.Select(c => $"{c.Name}_{c.Type.Name}")).GetHashCode();var key = $"{workflowName}-" + rulrParamsHash;
We need workflow name in key for clearing compiled rules when a workflow
is updated/deleted, as for the rest, a hash should be sufficient
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/RulesEngine/pull/386#pullrequestreview-1095617758,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSA3OX63ENQHO352YLQT7TV4Q4XLANCNFSM6AAAAAAQDCEDIY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for this PR!


[NO-JIRA] - add service account to enable pulling private docker images in ocp st…

…age cluster
Signed-off-by: Mauricio Magnani<EMAIL_ADDRESS>Insert JIRA Issue number and update link
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

did you have the chance to try with an upstream branch instead of main for app-interface?
looks like it's possible, I'll test that on Monday.


Collective enhancement of FAQ (#1 before beta release)

There is a list of enhancement of FAQ that comes from user testing summary. (not completed yet)

Add cancellation time in the FAQ.
Email notification wait-time instructions, including all situations of senders and receivers.
Emphasis users that they can use any emails to get notifications but will need a google account to deposit the money.

Better organization of information. Put the most important information at the beginning. make users easier to read through.

what's the status if this issue?

Completed the following sections


Is there a way to reuse already installed packages?

Hello, whenever I run Xcodegen, it redownloads the packages all over again, is there a way to avoid this?

Having the same issue. Every time I try to move through my repo with checkout or change branch it needs to reload packages, even if the dependencies stay the same. Reloading Package Graph is a bottleneck in my workflow right now.
Any way to prevent this?


added quantization and rotation

this adds the configuration for the quantisation on the realsense, it is based on https://github.com/robotology/yarp-device-realsense2/pull/30, the rotation is based on https://github.com/robotology/yarp-device-realsense2/pull/27 that has already been merged

@randaz81 @elandini84 @datspace fyi


build: make the static build work well

This is needed to make the shared+static build for build-script work.  The
primary issue is the dependency on the module-maps doesn't work due to the lack
of add_custom_command, so the target dependency cannot be hooked up.
Additionally, undo the hack for the AST_NAME as we need that to be proper for
the copy to succeed.  Finally, make sure that the we set the location for the
archive output correctly.

@swift-ci please test

CC: @ktopley-apple @MadCoder @kevints


Sensor ventilator Help

Hi guys
It's for a ventilator (open source ) project
So I want to modify some stuff
& Kinda upgrading the app
So my probleme are in sensors ( pressure sensor , airflow sensor)
For the pressure sensor I decide to take this one:(1st picture)
The range of pressure needed(0- 5 pka) is fine (0-40 pka)
But the problem is the airflow sensor.. (0- 20/25L/MN)
I thinker about water flow sensor (0-30L/MN) {YF-S201}
That's will be work??? If no please give me some advice or , kinda of sensor that I can use for measuring the airflow
The porpus of all that it's to make an graphic , like monitoring of the ventilator(3rd picture)
Best regards

This is not what github is for, so no surprise that you don't have answers.
Post your question on forum.arduino.cc (if you're using an Arduino).


Testconfig mode (-tc / -testconfig) doesn't work

Gollum has a mode to "test" a configuration file (for what? syntax errors or also plugin startup?), but all of the invocations below just produce the "usage" message and exit.
./gollum -tc -c config/http_auth_receive.conf
./gollum -testconfig config/http_auth_receive.conf
./gollum -ll 3 -tc -c config/http_auth_receive.conf
./gollum -tc config/http_auth_receive.conf
./gollum -testconfig -c config/http_auth_receive.conf
./gollum -ll 3 -tc config/http_auth_receive.conf

The "tc" should do two things
a) check if the yaml is parsable (which it does)
b) check if all plugins can be instantiated correctly without really activiating them (which it does not)
So this should be about properly implementing b)

c) Print an unambigious result instead of the usage instructions


[BUG] Warnings on func.exe startup and elsewhere on non-success Azure Blob Storage responses.

Library name and version
Microsoft.Azure.WebJobs.Extensions.EventHubs (5.0.0)
Describe the bug
When creating a vanilla Azure Function application with a reference to Microsoft.Azure.WebJobs.Extensions.EventHubs.5.0.0, three warnings about ContainerAlreadyExists/BlobNotFound are reported on startup.
It seems a warning logging has been added somewhere when Azure Blob Storage return a non-success error code,
Please note that although I am using Function startup as a minimal reproducer, this is now also an issue everywhere in application code when using Azure Blobs Storage. Even if we handle the expected exceptions e.g. ContainerAlreadyExists, BlobNotFound, we now get warnings in the logs. This was not the case with the previous 4.3.1 version of Microsoft.Azure.WebJobs.Extensions.EventHubs.
Expected behavior
No warnings when getting a non-success response from Azure Blob storage.
Actual behavior
Azure Functions Core Tools
Core Tools Version:       4.0.3928 Commit hash: 7d1d4a32700695bbf290a871c7492fde84bb767e  (64-bit)
Function Runtime Version: <IP_ADDRESS>15

[2022-01-13T09:46:05.372Z] Found C:\*****\FunctionApp1\FunctionApp1\FunctionApp1.csproj. Using for user secrets file configuration.

Functions:

        Function1: eventHubTrigger

For detailed output, run func with --verbose flag.
[2022-01-13T09:46:07.714Z] Error response [24a653b5-abe5-4c25-970f-2f0639039571] 409 The specified container already exists. (00.1s)
[2022-01-13T09:46:07.715Z] Server:Azurite-Blob/3.14.3
[2022-01-13T09:46:07.715Z] x-ms-error-code:ContainerAlreadyExists
[2022-01-13T09:46:07.716Z] x-ms-request-id:991c90d9-7323-49ed-b6b5-f286021ca30a
[2022-01-13T09:46:07.717Z] Date:Thu, 13 Jan 2022 09:46:07 GMT
[2022-01-13T09:46:07.718Z] Connection:keep-alive
[2022-01-13T09:46:07.718Z] Keep-Alive:REDACTED
[2022-01-13T09:46:07.719Z] Transfer-Encoding:chunked
[2022-01-13T09:46:07.720Z] Content-Type:application/xml
[2022-01-13T09:46:07.721Z]
[2022-01-13T09:46:07.821Z] Error response [2fb6eea3-e89b-4e6a-8fbe-fd1feb66f458] 404 The specified blob does not exist. (00.0s)
[2022-01-13T09:46:07.822Z] Server:Azurite-Blob/3.14.3
[2022-01-13T09:46:07.823Z] x-ms-error-code:BlobNotFound
[2022-01-13T09:46:07.824Z] x-ms-request-id:5b22999a-9123-4e74-aab4-8502802de705
[2022-01-13T09:46:07.825Z] Date:Thu, 13 Jan 2022 09:46:07 GMT
[2022-01-13T09:46:07.826Z] Connection:keep-alive
[2022-01-13T09:46:07.827Z] Keep-Alive:REDACTED
[2022-01-13T09:46:07.827Z]
[2022-01-13T09:46:07.846Z] Error response [3f9092d4-fe47-475a-8727-a9432cb30f84] 404 The specified blob does not exist. (00.0s)
[2022-01-13T09:46:07.848Z] Server:Azurite-Blob/3.14.3
[2022-01-13T09:46:07.848Z] x-ms-error-code:BlobNotFound
[2022-01-13T09:46:07.849Z] x-ms-request-id:ef9e96fc-82ca-48a9-a71b-6927d5937201
[2022-01-13T09:46:07.850Z] Date:Thu, 13 Jan 2022 09:46:07 GMT
[2022-01-13T09:46:07.851Z] Connection:keep-alive
[2022-01-13T09:46:07.852Z] Keep-Alive:REDACTED
[2022-01-13T09:46:07.853Z] Transfer-Encoding:chunked
[2022-01-13T09:46:07.857Z] Content-Type:application/xml
[2022-01-13T09:46:07.858Z]
[2022-01-13T09:46:12.360Z] Host lock lease acquired by instance ID '000000000000000000000000F42E305D'.

Reproduction Steps
Create an Azure Function app using the standard template in Visual Studio 2022. Add a reference to Microsoft.Azure.WebJobs.Extensions.EventHubs.5.0.0 and have the function implement an eventhub-trigger:
    public class Function1
    {
        [FunctionName("Function1")]
        public Task Run([EventHubTrigger("%EventHubName%", Connection = "EventHubConnection")] EventData[] events, ILogger log)
        {
            // Nothing here
            return Task.CompletedTask;
        }
    }

Environment
Visual Studio 2022 (17.0.4)
TargetFramework: net6.0
AzureFunctionsVersion: v4
Azure Function Core Tools Version:   4.0.3928

Hi @andhallberg.  Thank you for reaching out and we regret that you're experiencing difficulties.  The warnings that you're seeing are, unfortunately, expected.  Because the service responses for those operations are using HTTP status codes in the 4xx range, they're treated as errors by the Azure SDK diagnostics, despite being expected and treated as normal code paths by the client library.   This is something that we are tracking in #9934 and have begun working to correct, tracked by #25626.
I'm going to mark this as addressed, but please feel free to unresolve if you'd like to continue the conversation.


[WIP] Integration test phase 1: all APIs with all arguments combinations

Closed due to new PR #488


[Question/Proposal] protected_response_semaphore

Hi,
today I noticed that fetch_result() defined in the frameworks src_target_dependent/td_vvc_framework_common_methods_pkg.vhd uses a globally unique semaphore protected_response_semaphore defined in the utils global_signals_and_shared_variables_pkg.vhd to protect access to the shared variables shared_vvc_cmd and shared_vvc_response.
The aforementioned shared variables are defined in the scope of each VVC.
Why is a truly global semaphore used to protect access relevant to individual VVCs and its instances?
A tighter approach using only the smallest scopes necessary would be to have a protected_response_semaphore declared in each VVC.
Besides the extra work for adding a shared variable to each VVC, is there any particular reason for not doing this on VVC level? I might have missed something here...
kind regards
Thiemo

Hi,
This semaphore also protects other ordinary (old) shared variables (e.g. shared_vvc_cmd.cmd_idx inside send_cmd_to_vvc (used globally), which is called from inside fetch_result(). Thus it needs to be global.
We believe this will no longer be an issue when moving to UVVM version 3, which will only use protected shared variables only.
Regards
The UVVM team


Adds command: m365 pp solution publisher remove, closes #3950

This PR adds command m365 pp solution publisher remove, which will remove a publisher from a given environment.
Closes #3950
Please not that in this PR I use the uncommited solution m365 pp solution publisher get for retrieving the publisher by name

Thank you @nicodecleyre
We'll review it ASAP!

Hi @plamber ,
Thank you for your review!
As written above, this pr relies on the uncommited pr m365 pp solution publisher get, more specifically the function Cli.executeCommandWithOutput(PpSolutionPublisherGetCommand as Command,....
If I delete the files regarding m365 pp solution publisher get, the pr would result in errors. So I'll wait until m365 pp solution publisher get is merged before removing these files in this pr.
Kind regards,
Nico

Hi @nicodecleyre ,
you can remove the files now. Since I am merging the other commands I can add the delta threre.
Cheers

Hi @nicodecleyre , you can remove the files now. Since I am merging the other commands I can add the delta threre.
Cheers

Done 😄
Kind regards


Bump bot to v4.1.0

https://github.com/vipyrsec/bot/releases/tag/v4.1.0

bot image is pinned using the commit SHA on staging as well as prod, so this isn't needed any more


Update to Mutation Assessor v4

The new Mutation Assessor v4 data has 208.7m variants.
Example data:

uniprotId
SV
hgvspShort
F_score
F_impact
MSA
MAV

P00519
4
p.L25A
3.479676644421987
low
P00519.4_full_b0.6_rg0.3
4

P00519
4
p.L25C
4.799205309229616
low
P00519.4_full_b0.6_rg0.3
4

P00519
4
p.L25E
0.015792660298301824
neutral
P00519.4_full_b0.6_rg0.3
4

Tasks:
Database

[x] Genome Nexus importer: import new Mutation Assessor v4 data

In genome-nexus-importer, replace V3 mutation assessor data source
Rename field name to fit new data
Previous field name:

    sed -i -e 's/"Mutation","RefGenome variant","Gene","Uniprot","Info","Uniprot variant","Func. Impact","FI score"/_id,rgaa,gene,uprot,info,var,F_impact,F_score/g' MA_scores_rel3_hg19_full/MA_scores_rel3_hg19_chr*

New field name:
uniprotId	SV	hgvspShort	F_score	F_impact	MSA	MAV

Validate new V4 data in database is correct
When ready, release new database (but do not use in production env yet)

Genome Nexus backend

[x] Genome Nexus backend: new data need to be fetched by uniprot Id + hgvsp short

Create new model to fit with new V4 data in database
Previously we use genomic location as the index for mapping, the new V4 data requires uniprotId and HGVSp as the index. We need to modify the current logic to use uniprotId and HGVSp instead
Testing to ensure performance

Genome Nexus frontend

[x] Genome Nexus frontend: change frontend to fit with new data fields on variant page

Update frontend components in Genome Nexus

cBioPortal frontend

[x] cBioPortal frontend: change frontend to fit with new data fields, this should apply to all packages as well (react-mutation-mapper etc...)

Genome Nexus annotation pipeline

[x] Genome Nexus annotation pipeline: change to fit with new data fields

All tasks are done, close issue


BAEL-8318 Code and Unit Tests for Years Start on Sunday

Implements the solution using Date and Calendar, and the new approach using java.time API;
Implements an optimized example using Spliterator;
Provides unit tests for each of the added classes;

Articles: BAEL-8318

According to reviewer's comments, the code needs to be moved in a different module.


GCD bug fix

The standard-library documentation for trailingZeroBitCount states:

If the value is zero, then trailingZeroBitCount is equal to bitWidth.

Thus, if T is not a fixed-width integer type, and x == 0, and y has more trailing zeros than the representation of x, then the gcd calculation will be incorrect.
In particular, the left-shift by min(xtz, ytz) == xtz will not “undo” the earlier right-shift by ytz,  so the final result will be incorrectly right-shifted by ytz - xtz.
To fix this I have added an early exit when x == 0. I am not sure if this is the optimal solution, and I’m open to alternatives.

This change ensures that x != 0 at the top of the loop, so I have also converted it from a while to a repeat-while loop, in order to eliminate a redundant extra comparison on the first iteration.
I don’t know if this actually affects performance, and I’m happy to change it back if that’s preferred.

@swift-ci test


Adding j-isley file

Added in a file to do the thing.

New file updated, changes confirmed. Go for merge.


M1 should install on /usr/local

Provide a detailed description of the proposed feature
Install brew on /usr/local also on M1 platform.
/usr/local/bin is in the default path on Big Sur (minimum OS requirement for M1 devices) and there would be no need to tweak with any shell environments or override $PATH on other tools.
What is the motivation for the feature?
Having brew installing on /opt/hombrew on M1 is reducing a lot the seamless user experience that brew used to have.
/usr/local/bin is in the default path on Big Sur without needing to tweak any shell environment and will be available to tools that don't use a shell.
Just as an example on JetBrains IDEs in order to access brew installation on build and debug features requires overriding the path in every single IDE.
If I remember well back then homebrew was advertising the fact to install in /usr/local as a better feature over other alternatives.
How will the feature be relevant to at least 90% of Homebrew users?
This will impact 100% of Hombrew users on M1.
What alternatives to the feature have been considered?
The alternatives require a lot of custom setup for every different tools (as explained in the JetBrains example).

Sorry, but this isn't going to happen. You could maybe patch brew to let you do it, but you'll be on your own with that.
See this issue for a discussion of Homebrew's prefix on ARM: https://github.com/Homebrew/brew/issues/9177

Having brew installing on /opt/hombrew on M1 is reducing a lot the seamless user experience that brew used to have.

Sorry but there’s no way around it for the time being.
Intel bottles are pinned to the /usr/local prefix. We need to provide both architectures to users, which means ARM Homebrew has to live somewhere else.

Just as an example on JetBrains IDEs in order to access brew installation on build and debug features requires overriding the path in every single IDE.

I figured it should be possible to set it for all apps.
Have you tried launchctl setenv?

I read https://github.com/Homebrew/brew/issues/9177
Honestly I wouldn't expect people complaining about stuff in /usr/local, for me having the contrary is rather annoying, not only for path but also for pkgconfig etc, as mentioned in the discussion.
Indeed for me this mean there's no more reason to favour Homebrew over other package managers, sadly.

Sorry to hear that! I hope you're able to find a package manager that works for you.

@carlocab thanks
Indeed I see that the main point of the discussion was to provide both platform binaries at the same time, but that should be feasible building universal ones.
I don't see this mentioned in the issue, was it ever considered?

@aspacca We’ve had long discussions about that internally and in #7857 but ultimately decided against it.

Homebrew tried to support building universal binaries before but they never worked properly and were a nightmare to maintain.

Just as an example on JetBrains IDEs in order to access brew installation on build and debug features requires overriding the path in every single IDE.

I suggest making a feature request to JetBrains.

what would the feature that JetBrains has to implement?
Just anothe example:
LIBRARY_PATH=/opt/homebrew/lib/ make
this is now requested for building from sources in a lot of cases

what would the feature that JetBrains has to implement?

I don't use it so I don't really know. Presumably you didn't have to add /usr/local/bin to the PATH there so perhaps /opt/homebrew/bin should be in the default PATH.

shoukd I open a feature request to make maintainers? ;)

Projects shouldn't assume Homebrew is installed in a certain PATH and, where they have made this assumption, that should be fixed.

@MikeMcQuaid I assume you are familiar with what's stated on own Homebrew's FAQ:
https://docs.brew.sh/FAQ#why-does-homebrew-prefer-i-install-to-usrlocal
This was historically a selling point for Homebrew to push itself over other package managers.
No projects in the examples I've posted have made any assumption about Homebrew, they don't even know about it or care for it (sorry if this could sound unfair).
Having /opt/homebrew as Hombrew prefix on M1 is against your own FAQ, it breaks the seamless experience Homebrew user used to have and that the project itself has always pushed for.

@aspacca -- I don't think this is your intention, but you make it sound like the choice to use /opt/homebrew instead of /usr/local wasn't carefully considered. What is the point of bringing up the FAQ here? What do you hope to achieve?
We're well aware of the benefits of using /usr/local, but decided that the benefits of using /opt/homebrew outweighed those. That may make using Homebrew inconvenient for you. We find that regrettable, but not regrettable enough to change anything, because using /opt/homebrew is still what works best for most of our users -- despite the inconvenience it imposes on some.

@carlocab I just answered to the Homebrew project leader that suggested to ask for a feature request to JetBrains so that they could add /opt/homebrew/bin to the default path, while he stated that external tools should not make any assumption on Homebrew installed path.
The issue was already closed, I was not the one to bring it up again ;)

That answer is either totally disingenuous or complete nonsense. In either case, there's no point in continuing this discussion.

No projects in the examples I've posted have made any assumption about Homebrew, they don't even know about it or care for it (sorry if this could sound unfair).

@aspacca macOS developer tools do tend to care about Homebrew because millions of macOS developers use Homebrew.

Having /opt/homebrew as Homebrew prefix on M1 is against your own FAQ, it breaks the seamless experience Homebrew user used to have and that the project itself has always pushed for.

It was not possible to have Rosetta 2 support for M1 (still required for tools as widely used as ElasticSearch) with Intel/Rosetta 2 binary packages and use /usr/local for ARM.
Everything in software is a trade-off. /usr/local has some benefits and would have been better in some ways than /opt/homebrew, I agree. That doesn't make it the right decision.


App Store Binary Rejected - Guideline 1.2 - Safety - User Generated Content

Hi
After submitting your original 2.6 release on App Store it get rejected here what they wrote to me :
Binary Rejected
Guideline 1.2 - Safety - User Generated Content
Your app enables the display of user-generated content but does not have the proper precautions in place.
Next Steps
To resolve this issue, please revise your app to implement all of the following precautions:

Require that users agree to terms (EULA) and these terms must make it clear that there is no tolerance for objectionable content or abusive users
A method for filtering objectionable content
A mechanism for users to flag objectionable content
A mechanism for users to block abusive users
The developer must act on objectionable content reports within 24 hours by removing the content and ejecting the user who provided the offending content

any help please Thanks

I got a new respond from App Store Review says :

Thank you for your response. To clarify, while our App Store Review Guidelines is a living document and subject to change, all social networking apps are subject to meet compliance with guideline 1.2.0.

In order for your app to be reconsidered for the App Store, it would be appropriate to bring your app into compliance.
Best regards,
App Store Review
So are you goanna implement these features in your next update ?
Thanks for your support
Regards

Hi @gbr33
It seems like you need to add some kind of "report abuse" button. Add it to a profile, so any user can report another user for some kind of misbehaviour. Make that button to send a user-input text to any of you email or system user, for example.
We haven't met such requirements in Q-municate yet, but will surely add it in a future if needed.
For now you have to implement that feature by yourself.


Run ngspice in batch mode

Currently CACE starts ngspice in interactive mode. This means if there is no quit at the end of the control script, CACE gets stuck.
Therefore, pass --batch to ngspice to automatically quit at the end of simulation.

Fixed by #75


Add consume to SList

A range that iterates then removes elements for which the predicate is
true. An efficient combination of filter!(pred) and remove.

This is a good idea, but we could achieve the desired behaviour by using std.algorithm.iteration.each
One could do something like this:
	auto a = SList!int(-1, 1, -2, 1, -3, 4);
	SList!int acc;
	each!((int e) => (e > 0) ? acc.insert(e) : 0)(a[]);

I believe that consume should return a SList rather than a different type.
@andralex

That works but creates a new list (with allocations and copying of data), rather than mutate the old (reference semantics). The whole point is to amortise theO(n) iteration and provide an efficient way to remove based on a predicate.
Perhaps it would be more aptly named removeYield.

One could do something like this:

If you already create a new List, you can simply use filter, e.g.:
auto a = SList!int(-1, 1, -2, 1, -3, 4);
auto acc = a[].filter!(x => x > 0).SList!int;
acc.array.writeln;

Perhaps it would be more aptly named removeYield.

Yes, but the word yield isn't very common in D as everything is a range. removePred or remove might be better.

The whole point is to amortise theO(n) iteration and provide an efficient way to remove based on a predicate.

I get your point, but I'm not fully convinced that this is a popular operation, especially considering that std.container will hopefully get replaced / deprecated soonish. What do other people think?
CC @ZombineDev @JackStouffer

This seems like a lot of code for a very specific problem. What's the use case?

This is a difficult and subtle API, though I see its necessity.
A tricky problem with SList is that removal primitives need to access the node just before the element being removed. I think the right approach is along the lines of linearRemoveElement. Currently the function takes a value and returns bool. A more composable version would return a range positioned just before the removed element. Then successive calls to linearRemoveElement would remove as many as needed, one at a time.
Then a predicated version of the function would remove by predicate, not a specific value.

I like that idea much better. Will work on it tomorrow.

friendly ping @thewilsonator. Any updates on this?

Heh, completely forgot about it. It looks like removeLinearElement was added, but takes a value only not a predicate.

How about adding an overload that takes a predicate? I think that's what @andralex was suggesting

PR closed as stalled. If you wish to resurrect it you are welcome to do so.


SyntaxError: Cannot use 'import.meta' outside a module

What version of Remix are you using?
1.14.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Using a project based on the cloudflare-workers template:

Upgrade from Remix 1.13.0 to 1.14.0
Launch the app in local development mode (npm run dev)

Expected Behavior
The app boots in Miniflare as normal
Actual Behavior
On startup, Miniflare throws the following fatal errors:
[mf:err] /home/jim/src/ksoc/web-app/build/index.js:7882
    import.meta && import.meta.hot && import.meta.hot.accept("remix:manifest", async (newManifest) => {
           ^^^^

SyntaxError: Cannot use 'import.meta' outside a module
    at new Script (node:vm:100:7)
    at VMScriptRunner.runAsScript (/home/jim/src/ksoc/web-app/node_modules/@miniflare/runner-vm/src/index.ts:23:20)
    at VMScriptRunner.run (/home/jim/src/ksoc/web-app/node_modules/@miniflare/runner-vm/src/index.ts:88:12)
    at Miniflare.#reload (/home/jim/src/ksoc/web-app/node_modules/@miniflare/core/src/index.ts:775:42)
    at Miniflare.getPlugins (/home/jim/src/ksoc/web-app/node_modules/@miniflare/core/src/index.ts:1018:5)
    at createServer (/home/jim/src/ksoc/web-app/node_modules/@miniflare/http-server/src/index.ts:378:19)
    at startServer (/home/jim/src/ksoc/web-app/node_modules/@miniflare/http-server/src/index.ts:485:18)
    at main (/home/jim/src/ksoc/web-app/node_modules/miniflare/src/cli.ts:111:7)
[mf:err] Unhandled Promise Rejection: /home/jim/src/ksoc/web-app/build/index.js:7882
    import.meta && import.meta.hot && import.meta.hot.accept("remix:manifest", async (newManifest) => {
           ^^^^

SyntaxError: Cannot use 'import.meta' outside a module
    at new Script (node:vm:100:7)
    at VMScriptRunner.runAsScript (/home/jim/src/ksoc/web-app/node_modules/@miniflare/runner-vm/src/index.ts:23:20)
    at VMScriptRunner.run (/home/jim/src/ksoc/web-app/node_modules/@miniflare/runner-vm/src/index.ts:88:12)
    at Miniflare.#reload (/home/jim/src/ksoc/web-app/node_modules/@miniflare/core/src/index.ts:775:42)
    at Miniflare.getPlugins (/home/jim/src/ksoc/web-app/node_modules/@miniflare/core/src/index.ts:1018:5)
    at createServer (/home/jim/src/ksoc/web-app/node_modules/@miniflare/http-server/src/index.ts:378:19)
    at startServer (/home/jim/src/ksoc/web-app/node_modules/@miniflare/http-server/src/index.ts:485:18)
    at main (/home/jim/src/ksoc/web-app/node_modules/miniflare/src/cli.ts:111:7)

This issue persists in 1.14.1

Happens for me to but only when using
import { useDehydratedState } from 'use-dehydrated-state'
When I use the same code from a local file e.g. ./use-dehydrated-state, it magically works. 🤔

I encountered the same issue when trying to use @clerk/remix package in official cloudflare template from create-remix.
This happened when I tried to follow the official documents at https://clerk.com/docs/quickstarts/get-started-with-remix
And the error occurs at 2nd step as follows:

And also this looks like a known issue https://github.com/remix-run/remix/pull/5259#discussion_r1131700486

This seems to be fixed as of @remix-run/* v1.15.0


Upgrade to Electron v 5

https://electronjs.org/blog/electron-5-0

Fixed here: https://github.com/crilleengvall/electron-tutorial-app/pull/32


Fixes for SNAP-3253:

What changes were proposed in this pull request?

Call to add structured streaming UI tab is shifted to SnappySession class.
Renamed method updateUI to updateUIWithStructuredStreamingTab and
also changed access modifier from private to protected.

How was this patch tested?

Tested manually and precheckin is run. No relevant failures.

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Merging.


Boss kill time does not register correctly if you teleport at the exact same time as the kill message is sent

Boss kill time does not register correctly if you teleport at the exact same time as the kill message is sent, fix for this will be includes in the next update

fixed by #022d555. Closed


Add recursive = True to glob

The documentation states:

The file_or_dir specifier can include wildcards e.g. c:\temp*\devskim.sarif (i.e. a "glob").

However it does not currently enable the use of ** when using glob. This PR adds recursive = True to enable it.

Thank you for your pull request to improve the flexibility of this tool, and apologies for the long delay before we spotted it.  Now meged.


Quick Open should default-select previous file

Issue Type: Feature Request
When using "Go to File..." (cmd-p on Mac), it shows a list of the recently opened files. The latest file is at the top and it is selected. I believe this would be more useful if the previous file was selected by default. It would allow for much faster back-forwards navigation. All I'd need to do it cmd-p+enter to go to the previous file.
Btw my proposed change is the default behaviour in TextMate.
VS Code version: Code 1.21.0 (9a199d77c82fcb82f39c68bb33c614af01c111ba, 2018-03-07T10:54:22.798Z)
OS version: Darwin x64 17.4.0

If you use Cmd+P while the file picker is open we keep moving in the file list and also open the editor without you to have to press Enter. So, Cmd+p Cmd+p gives you the previous file. Same number of key strokes you propose.

OMG - thank you @kieferrm ! Yes, this is as good. Guess I can close this already.


Move CNO into management cluster / add OVN to types

Just a rebase of https://github.com/openshift/hypershift/pull/1101 while @kyrtapz is on vacation.

/test ?

e2e-aws passed!

Found a small issue with leader election. Let's see if that fixes it.

@enxebre @alvaroaleman can you take a look at this? Assuming this goes green, this should be good to go

One comment but generally I am good with this and would prefer merging it sooner rather than later

@alvaroaleman all green :)

Thanks for your work!

/hold

We are currently not testing that we can actually create a cluster with this. It will likely work, but lets be safe. https://github.com/openshift/hypershift/pull/1260 adds that test. Lets retest once that merged and if that passes, we are good.

@alvaroaleman I made a small change requesting a few more RBAC permissions for CNO -- could you re-lgtm please?

(not that it matters, we'll have to rebase before merging anyways)

/lgtm
/hold cancel


change logo adress / rep closed #13427

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
Hi, i have entered assets adress by mistake instead of main in the logo url could you please update it for the correct address ?

https://raw.githubusercontent.com/astefanis/HappyX/main/HApexsmall.png
Thank you for your understanding and sorry for the mess

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

I am adding a link to my fork : https://github.com/astefanis/token-list/tree/happyx-coin-info
Thnx again

please submit a PR and link it here

https://github.com/solana-labs/token-list/pull/13518

im not sure if you ment that i changed the url my self then submit the pr so i submited withouth changes.

astefanis:happyx-coin-info
this is with the changed url. Dont know if i need to do smth else ? ! Sorry, is this okay ?

Im closing this issue and starting a new one with the new PR. Thank you !


Fix deep linking to reports after signing in

Details
Fixes an issue where we attempt to load a report before the app is done loading, causing the OpenReport response to be dropped.
Fixed Issues
$ https://github.com/Expensify/App/issues/42753
Tests

Log into NewDot as user A
Verify the app and the initial report load without issue
Open a DM to user B
Submit an expense in the DM
Open the report and copy the link
Sign out
Open the link and sign in as user B
Verify

the app loads
the expense report loads
the header includes a link to the parent report

Offline tests
None
QA Steps
Same as Tests
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/6833644/e8237324-036d-465c-bc5d-3f831d35dc67

Android: mWeb Chrome

iOS: Native
https://github.com/Expensify/App/assets/6833644/302cfafb-fd01-4670-b591-0e14fa3e872e

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/6833644/388ed05a-a3c4-4d7a-883e-4887ed7dd404

MacOS: Desktop

I'm only able to test this on web atm. Attempting to deep link on mobile gave me this error
 WARN  The navigation state parsed from the URL contains routes not present in the root navigator. This usually means that the linking configuration doesn't match the navigation structure. See https://reactnavigation.org/docs/configuring-links for more details on how to specify a linking configuration.

It's working well on web where this was reported so posting this for review anyway

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Not Found page appears briefly before transaction report appears on native and mWeb platforms. Videos here.

 WARN  The navigation state parsed from the URL contains routes not present in the root navigator. This usually means that the linking configuration doesn't match the navigation structure. See https://reactnavigation.org/docs/configuring-links for more details on how to specify a linking configuration.

This warning has always appeared whenever I opened a deeplink on iOS and Android with uri-scheme. I think this is not an issue.

Not Found page appears briefly before transaction report appears on native and mWeb platforms. Videos here.
This warning has always appeared whenever I opened a deeplink on iOS and Android with uri-scheme. I think this is not an issue.

Thanks for the tip, let me see if I can fix the Not Found issue

Alright that issue should be fixed now. Ready for another review @c3024

🚀 Deployed to staging by https://github.com/roryabraham in version: 9.0.2-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 9.0.3-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 9.0.5-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Automated build of Sphinx docs fails

Automated build of Sphinx docs fails because sphinx_rtd_theme is not a hard dependency as of Sphinx version 1.4.0. As a result, the Github Pages generation also fails.
Full error logs follow:
sphinx-build -b html -d build/doctrees   source build/html
Running Sphinx v1.4.1
making output directory...
loading pickled environment... not yet created

Theme error:
sphinx_rtd_theme is no longer a hard dependency since version 1.4.0. Please install it manually.
make: *** [html] Error 1
Makefile:55: recipe for target 'html' failed

Verified that PR #63 fixes the issue by manually running the Jenkins job for building the docs using the updated Dockerfile. The documentation for the latest and greatest B2HANDLE version 1.0.1 is finally available from the projects's GitHub pages!


Create new tag

There are a couple of commits in master that are not released/tagged yet. Could you create a new tag?

How about a new tag @sagikazarmark? Why shouldn't we just release it?

Yeah, good idea. I'm going to release 3.2 and postpone the rest of the things in the tube.

I'd also love to have a 3.1.4 or 3.2.0, can I help somehow?

Done in: https://github.com/moneyphp/money/releases/tag/v3.2.0


The 'auto restore'  not work after set auto_session_root_dir

The session auto restore not works when set  auto_session_root_dir to current working dir:
let g:auto_session_root_dir = getcwd()

I think you figured it out since you closed this but for future reference this is just not compatible with how the plugin works since it needs only one root dir to keep the session files.


Add project install instructions to project

I should have waited a month for the Hacktoberfest... but I couldn't. So here it is!
I could not get the project to install following the instructions in the readme, so I "reverse-engineered" them as I don't really code in Java.
Have fun!
Note: I could not get mvn package -Pwasm to work this way:

I may make a few small adjustments and move the installation instructions lower in the README, but thanks very much for writing them!


docker-compose build error

`Traceback (most recent call last):
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/connectionpool.py", line 670, in urlopen
httplib_response = self._make_request(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/connectionpool.py", line 392, in _make_request
conn.request(method, url, **httplib_request_kw)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1255, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1301, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1250, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1010, in _send_output
self.send(msg)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 950, in send
self.connect()
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/transport/unixconn.py", line 43, in connect
sock.connect(self.unix_socket)
FileNotFoundError: [Errno 2] No such file or directory
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/requests/adapters.py", line 439, in send
resp = conn.urlopen(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/connectionpool.py", line 726, in urlopen
retries = retries.increment(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/util/retry.py", line 410, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/packages/six.py", line 734, in reraise
raise value.with_traceback(tb)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/connectionpool.py", line 670, in urlopen
httplib_response = self._make_request(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/urllib3/connectionpool.py", line 392, in _make_request
conn.request(method, url, **httplib_request_kw)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1255, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1301, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1250, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 1010, in _send_output
self.send(msg)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/http/client.py", line 950, in send
self.connect()
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/transport/unixconn.py", line 43, in connect
sock.connect(self.unix_socket)
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/api/client.py", line 205, in _retrieve_server_version
return self.version(api_version=False)["ApiVersion"]
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/api/daemon.py", line 181, in version
return self._result(self._get(url), json=True)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/utils/decorators.py", line 46, in inner
return f(self, *args, **kwargs)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/api/client.py", line 228, in _get
return self.get(url, **self._set_request_timeout(kwargs))
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/requests/sessions.py", line 543, in get
return self.request('GET', url, **kwargs)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/requests/sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/requests/sessions.py", line 643, in send
r = adapter.send(request, **kwargs)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/requests/adapters.py", line 498, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/saumya/anaconda3/envs/env/bin/docker-compose", line 11, in
sys.exit(main())
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/main.py", line 67, in main
command()
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/main.py", line 123, in perform_command
project = project_from_options('.', options)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/command.py", line 60, in project_from_options
return get_project(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/command.py", line 131, in get_project
client = get_client(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/docker_client.py", line 41, in get_client
client = docker_client(
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/compose/cli/docker_client.py", line 170, in docker_client
client = APIClient(**kwargs)
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/api/client.py", line 188, in init
self._version = self._retrieve_server_version()
File "/home/saumya/anaconda3/envs/env/lib/python3.8/site-packages/docker/api/client.py", line 212, in _retrieve_server_version
raise DockerException(
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))`

Could you tell me what command did you use?
I seem we don't use anaconda in Docker. But there is an anaconda directory in your error code.


Cannot login, whatever the service

I paired the FreeOTP app on my phone to my professional AWS account and many personal accounts.
I cannot log in to any of these services.
I didn't list them all here as that would be a security issue for these, and am quite distressed as most of my accounts use this MFA feature.

Does your phone have the correct time and date set?

Good shot.
It didn't this morning, can't figure out why, I had to set it by hand, but I only set the hour.
I tried after having set the correct time, to no avail.
Then after reading your answer, the minutes were off by 1 , apparently that was enough to loose the system.
Thanks for your answer, now I have to find why my phone's clock got out of track :)

One minute is enough, because most tokens change every 30 seconds. (the actual time is part of the QR-Code that you scan, but 30 seconds is the default).

Just for the sake of others : I checked the option "internet time" off , which, of course, ment my phone didn't syncrhonise over NTP...
Sorry for the trouble, and again thank you for your help :)


Session cannot be saved

Latest version
Session cannot be saved
session()->set('test', 123);
// The result is empty
echo session()->get('test');

`$session->set('some_name', 'some_value');
Your calling it wrong...

Nothing should have changed with the sessions recently. You should know it’s impossible for me to debug without more information up front.

What environment are you running under?
what session driver are you using?
what does your configuration look like?

And last but most definitely not least - what have you done to debug it yourself and show that it’s an actual bug and not just a misconfiguration!

Lonnie,
Can we call it like that with the () or is that the helper way?

Yes, it can be called that way. It's in the Common.php file, I believe where we provide a few helper methods for things like sesison, request, and response.

Thank you for clarifying that for me.

windows and centos7.0
php7.0 and php7.1
mysql
There's been a mistake.

I'm on Windows 10 Pro I'll check it out today.

@InsiteFX thanks for testing.
@jinmarcus Can you verify your configuration works with, say the cache library using a Redis driver? At this point I can't confirm whether or not it's your system configuration (which is what I'm leaning toward at the moment) or an actual bug. and I know that Redis can be somewhat tricky to configure properly. Please check the docs, maybe head over to the phpredis page that's linked in the docs and do some experiments with the caching to see if you can narrow things down.

@lonnieezell A few weeks ago I was using Redis and it was working fine on Windows 10 Pro and 2 Instances of Server 2012.

I use DatabaseHandler and every time get 'new data'. Data in DB write and read correct. But something get wrong..
		session()->start(); 		if (session('logged_in') == 'true') { 			print_r(session()->get()); 		} else { 			session()->set(array( 				'logged_in' => 'true', 			)); 			print_r('new data'); 		}


Investigate random infrequent HTTPS read time out errors via the Trakt API

When exception occurs, Github Actions builds exit with code 1 (indicates that the command being executed in the Github action has failed).
Exceptions occur in ~1/15 runs.
See:

https://github.com/sadmanca/trakt-randomizer/actions/runs/5192872583
https://github.com/sadmanca/trakt-randomizer/actions/runs/5180646791

Example error output:
INFO:root:users/sadmanca/lists/hindi
INFO:root:ORIGINAL:
INFO:root:    <Movie 'Baby' (2015)>
INFO:root:    <Movie 'Khatta Meetha' (2010)>
INFO:root:    ...
INFO:root:    <Movie 'Dil Hai Tumhaara' (2002)>
INFO:root:    <Movie 'Ek Tha Tiger' (2012)>
INFO:root:SHUFFLED:
INFO:root:    <Movie 'Badhaai Ho Badhaai' (2002)>
INFO:root:    <Movie 'Tezz' (2012)>
INFO:root:    ...
INFO:root:    <Movie 'Krrish' (2006)>
INFO:root:    <Movie 'R... Rajkumar' (2013)>
INFO:root:users/sadmanca/lists/best-of-the-2000s
INFO:root:ORIGINAL:
INFO:root:    <Movie 'Where the Wild Things Are' (2009)>
INFO:root:    <Movie 'Cast Away' (2000)>
INFO:root:    ...
INFO:root:    <Movie 'The Village' (2004)>
INFO:root:    <Movie 'In the Bedroom' (2001)>
INFO:root:users/hdlists/lists/best-of-the-00s
INFO:root:NEW, SHUFFLED:
INFO:root:    <Movie 'Shadow of the Vampire' (2000)>
INFO:root:    <Movie 'Inland Empire' (2006)>
INFO:root:    ...
INFO:root:    <Movie 'Cast Away' (2000)>
INFO:root:    <Movie 'Intolerable Cruelty' (2003)>
INFO:root:users/sadmanca/lists/best-of-the-2010s
INFO:root:ORIGINAL:
INFO:root:    <Movie 'Blackfish' (2013)>
INFO:root:    <Movie 'The Descendants' (2011)>
INFO:root:    ...
INFO:root:    <Movie 'X-Men: Days of Future Past' (2014)>
INFO:root:    <Movie 'Side Effects' (2013)>
INFO:root:users/hdlists/lists/best-of-the-2010s
INFO:root:NEW, SHUFFLED:
INFO:root:    <Movie 'Fruitvale Station' (2013)>
INFO:root:    <Movie 'Spider-Man: Homecoming' (2017)>
INFO:root:    ...
INFO:root:    <Movie 'The Perks of Being a Wallflower' (2012)>
INFO:root:    <Movie 'The Beatles: Eight Days a Week - The Touring Years' (2016)>
INFO:root:users/sadmanca/lists/action-movies-top-rated-from-1980-to-today
INFO:root:ORIGINAL:
INFO:root:    <Movie 'The Abyss' (1989)>
INFO:root:    <Movie "Pete's Dragon" (2016)>
INFO:root:    ...
INFO:root:    <Movie 'Clash of the Titans' (1981)>
INFO:root:    <Movie 'Dragonslayer' (1981)>
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connectionpool.py", line 536, in _make_request
    response = conn.getresponse()
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connection.py", line 454, in getresponse
    httplib_response = super().getresponse()
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/http/client.py", line 1377, in getresponse
    response.begin()
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/http/client.py", line 320, in begin
    version, status, reason = self._read_status()
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/http/client.py", line 281, in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/socket.py", line 704, in readinto
    return self._sock.recv_into(b)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/ssl.py", line 1242, in recv_into
    return self.read(nbytes, buffer)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/ssl.py", line 1100, in read
    return self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out

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

Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/requests/adapters.py", line 486, in send
    resp = conn.urlopen(
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connectionpool.py", line 844, in urlopen
    retries = retries.increment(
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/util/retry.py", line 470, in increment
    raise reraise(type(error), error, _stacktrace)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/util/util.py", line 39, in reraise
    raise value
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connectionpool.py", line 790, in urlopen
    response = self._make_request(
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connectionpool.py", line 538, in _make_request
    self._raise_timeout(err=e, url=url, timeout_value=read_timeout)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/urllib3/connectionpool.py", line 370, in _raise_timeout
    raise ReadTimeoutError(
urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='api.trakt.tv', port=443): Read timed out. (read timeout=24)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/runner/work/trakt-randomizer/trakt-randomizer/main.py", line 181, in <module>
    main()
  File "/home/runner/work/trakt-randomizer/trakt-randomizer/main.py", line 177, in main
    multi_run(url_list)
  File "/home/runner/work/trakt-randomizer/trakt-randomizer/main.py", line 167, in multi_run
    copy_run(urlparse(key).path[1:], urlparse(value).path[1:], randomize=True)
  File "/home/runner/work/trakt-randomizer/trakt-randomizer/main.py", line 126, in copy_run
    Trakt[self_list_slug].remove(data)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/interfaces/base/__init__.py", line 126, in wrap
    return value(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/interfaces/base/__init__.py", line 20, in wrap
    return func(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/interfaces/users/lists/list_.py", line 150, in remove
    response = self.http.post(
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/core/http.py", line 188, in post
    return self.request('POST', path, params, data, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/core/http.py", line 121, in request
    return self.send(prepared)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/trakt/core/http.py", line 142, in send
    response = self.session.send(request, timeout=timeout)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/requests/adapters.py", line 532, in send
    raise ReadTimeout(e, request=request)
requests.exceptions.ReadTimeout: HTTPSConnectionPool(host='api.trakt.tv', port=443): Read timed out. (read timeout=24)
Error: Process completed with exit code 1.

Current temporary solution: increase time between Github Actions runs to decrease probability of read time out occurring.


upx: apply patch for CVE-2021-20285

Description of changes
Did not bump to 4.0.0 yet because the 4.0.0 release is affected by CVE-2021-30500 and CVE-2021-30501.
The patch for CVE-2021-30500 does not apply cleanly on top of 4.0.0.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

So this should be closed because https://github.com/NixOS/nixpkgs/pull/202254 ?
Or retargeted to 22.11?

#202254 is stale due to an upstream bug at the moment. I think the patch is the way to go for now.


rpc: Do not return ContractAddress for Deploy and DeployAccount transactions

This pr closes #1118.
From What I understood the target was only for transactions related stuff, Receipts still should include the address.

I think we can do this more cleanly.
One way to do this is to remove ContractAddress from rpc.Transaction entirely.
and use h.bcReader.TransactionByHash(&hash) in TransactionReceiptByHash instead of  h.TransactionByHash. Since it needs the ContractAddress information from core.Transaction

I think we can do this more cleanly.

One way to do this is to remove ContractAddress from rpc.Transaction entirely. and use h.bcReader.TransactionByHash(&hash) in TransactionReceiptByHash instead of h.TransactionByHash. Since it needs the ContractAddress information from core.Transaction

Removing the contract address entirely would affect INVOKE_TXN_V0 and L1_HANDLER_TXN . They include FUNCTION_CALL which requires contract_address

Ah, I see. Then just dont copy the ContractAddress to rpc.Transaction for Deploy and DeployAccount transactions


Update font-noto-sans-myanmar to latest

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Merged as 55911e709d3b364ee7b8437cf3e540fc0856cfeb.