IRIs of SHACL shapes

Currently, IRIs of SHACL shapes seem to be constructed from the human readable label and IRI of a class, which is somehow converted to Snake_case and Shape is added, e.g. https://slovník.gov.cz/generický/čas/pojem/Časový_intervalShape based on Časový interval. There should be a discussion of considered alternatives and edge cases (possible naming conflicts), especially in cases where we have two data structures with different specifications for the same class, e.g. a less detailed data structure of Tourist destination and a more detailed structure of Tourist destination. In this case, both shapes would get the same IRI, which is not desirable.
Also related to #328
This is especially weird for Shapes based on classes from web vocabularies, e.g. http://www.w3.org/ns/Legal_entityShape where the IRI namespace is hijacked.

For now I am implementing the unique iri with the help of example.org, md5, technical label if present and a suffix whether it is NodeShape or PropertyShape. There is still some work to be done as md5 is now generated from the iri of the object. But that means that there is no distinction between different objects with the same iri.
Work to be done to resolve this issue of same iri but different structure.

Also, consider that there is already a setting for Base URL of the specification:

As of now, there is added support for baseURL usage in the IRI naming convention.
The issue of different classes and the naming remains.

If there is no baseURL specified, the IRIs for the shapes are relative now.

The IRI of the generated shacl shapes are now tied to the psm of the given entity. That should help maintain the difference between different entities with the same cim but also should be the same every time for the same structure.


Gateway support functions with same selector but different parameters

Issue is happening on the CCIP-Read-Server implementation where the Ethers v5 is unable to identify different functions signatures given that it only checks the function selector, see issue and Ethers v5 implementation.
[
  'function setAddr(bytes32 node, address addr) returns (address)',
  'function setAddr(bytes32 node, uint coinType, bytes calldata addr) returns (string)'
]

solved on #37


Fix sealed vtable calls on Wasm

Fix calls on sealed vtables by disabling relative pointers for WebAssembly.
Fixes #6330.

What would the assert be on? Usage of RelocType.IMAGE_REL_BASED_RELPTR32? I think some nodes use that unconditionally.

What would the assert be on? Usage of RelocType.IMAGE_REL_BASED_RELPTR32?

Yeah. Something like what CppCodegen does
https://github.com/dotnet/corert/blob/267410d0ab65fabc61d6cd3e0e1c2ebed43ea661/src/ILCompiler.CppCodeGen/src/CppCodeGen/CppWriter.cs#L782

I think some nodes use that unconditionally.

Those are probably not emitted by WASM yet (I know e.g. reflection uses the 32bit relocs unconditionally for external references table). They'll need to be converted to the SupportsRelativePointers plan at the point they become relevant for WASM. I kind of prefer to have these things assert at compile time instead of having to troubleshoot random runtime failures. Tracing random runtime failures to a compiler component responsible requires more steps than an assert.

Some of the queues are down, so I did manual clean builds for debug and release (and this passed tests before adding the assert as well), so I'll go ahead and merge.


Out of memory error when trying to compress a really huge hostfile

Just like the title says. I am trying to compress a hostfile ~400mb and it fails with this error:
runtime: out of memory: cannot allocate 281018368-byte block<PHONE_NUMBER> in use)
fatal error: out of memory
goroutine 1 [running]:
runtime.throw({0x2343d4, 0xd})
C:/Program Files/Go/src/runtime/panic.go:1047 +0x4d fp=0x984dc08 sp=0x984dbf4 pc=0x1c2d6d
runtime.(*mcache).allocLarge(0x10e0088, 0x109c6000, 0x1)
C:/Program Files/Go/src/runtime/mcache.go:236 +0x1be fp=0x984dc30 sp=0x984dc08 pc=0x1a08fe
runtime.mallocgc(0x109c6000, 0x0, 0x0)
C:/Program Files/Go/src/runtime/malloc.go:1053 +0x3eb fp=0x984dc68 sp=0x984dc30 pc=0x19a69b
runtime.rawbyteslice(0x109c4800)
C:/Program Files/Go/src/runtime/string.go:274 +0xdb fp=0x984dc84 sp=0x984dc68 pc=0x1d97db
runtime.stringtoslicebyte(0x0, {0x5cbc4000, 0x109c4800})
C:/Program Files/Go/src/runtime/string.go:172 +0x4c fp=0x984dca4 sp=0x984dc84 pc=0x1d950c
main.main()
C:/Users/CJWally/Desktop/projects/GitHub/Hosts-BL/hosts-bl.go:201 +0xd8b fp=0x984dfc4 sp=0x984dca4 pc=0x21e3eb
runtime.main()
C:/Program Files/Go/src/runtime/proc.go:250 +0x22e fp=0x984dff0 sp=0x984dfc4 pc=0x1c552e
runtime.goexit()
C:/Program Files/Go/src/runtime/asm_386.s:1326 +0x1 fp=0x984dff4 sp=0x984dff0 pc=0x1eb321
goroutine 2 [force gc (idle)]:
runtime.gopark(0x23c288, 0x2c6010, 0x11, 0x14, 0x1)
C:/Program Files/Go/src/runtime/proc.go:381 +0xff fp=0x9847fdc sp=0x9847fc8 pc=0x1c595f
runtime.goparkunlock(...)
C:/Program Files/Go/src/runtime/proc.go:387
runtime.forcegchelper()
C:/Program Files/Go/src/runtime/proc.go:305 +0xcf fp=0x9847ff0 sp=0x9847fdc pc=0x1c578f
runtime.goexit()
C:/Program Files/Go/src/runtime/asm_386.s:1326 +0x1 fp=0x9847ff4 sp=0x9847ff0 pc=0x1eb321
created by runtime.init.5
C:/Program Files/Go/src/runtime/proc.go:293 +0x23
...

Try running with the -dupe argument and see if that works.

Tried that argument and got the same error. I guess the file is too big

Hosts-BL is designed and optimized to manipulate smaller hosts files extremely quickly, specifically the Steven Black hosts files. I'll leave this issue open though and add some additional options over time to allow for less memory-intensive operations.
For now, there are a couple of other things you can try. You could trying using the 64-bit Linux version of Hosts-BL in a Google Colab and see if that works. You could also check out the Compressed.cmd script in my Hosts-Conversions repo (https://github.com/ScriptTiger/Hosts-Conversions) and see if that works. This script will run considerably slower, but it's a trade-off for being far less memory-intensive by reading from and writing to your hard drive for everything rather than just slurping everything into memory, as Hosts-BL does.

@Tendodeku Try the latest release and see if that fixes the issue.

Sounds good. Thanks for the update I will try it when I get home

Tried the new version and the destination file it creates is empty, 0 bytes.

Have you confirmed you have the appropriate blackhole address configured? The default blackhole address it looks for is <IP_ADDRESS>, but some hosts files use <IP_ADDRESS>, or possibly even something else. Use the -from_blackhole argument to set the blackhole which is appropriate for your hosts file.
I've also just published a new release that doesn't sound like it's related to your issue, but you may want to use the latest release just in case to make sure you're working with the latest one.

You were right. The source file was using <IP_ADDRESS>. The new version worked wonderfully, thank you so much

And thank you, too, for your feedback!

@Tendodeku I just added a new feature to automatically detect what black hole address is being used. However, using the -from_blackhole argument would still be slightly faster since it can skip that automatic detection process.
I'm also curious if you've had a chance to try out the new -hash configurations with your larger files? Collisions may or may not be a realistic concern, depending on the size of your hosts file. But just curious if you've tried different hash sizes and noticed any difference in the results, other than it just running more slowly?


Loading model problems

When I'm trying to load a saved model, I get the following error. ! "A Dot layer should be called on a list of 2 inputs".

I've met the same problem.
Traceback (most recent call last):   File "/home/cedar/kzheng/MAP569_AMF_Challenge/resume_training.py", line 97, in <module>     model = keras.models.load_model(MODEL_PATH)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/saving/save.py", line 182, in load_model     return hdf5_format.load_model_from_hdf5(filepath, custom_objects, compile)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/saving/hdf5_format.py", line 177, in load_model_from_hdf5     model = model_config_lib.model_from_config(model_config,   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/saving/model_config.py", line 55, in model_from_config     return deserialize(config, custom_objects=custom_objects)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/layers/serialization.py", line 171, in deserialize     return generic_utils.deserialize_keras_object(   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/utils/generic_utils.py", line 354, in deserialize_keras_object     return cls.from_config(   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/engine/sequential.py", line 488, in from_config     model.add(layer)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/training/tracking/base.py", line 457, in _method_wrapper     result = method(self, *args, **kwargs)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/engine/sequential.py", line 221, in add     output_tensor = layer(self.outputs[0])   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 925, in __call__     return self._functional_construction_call(inputs, args, kwargs,   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 1098, in _functional_construction_call     self._maybe_build(inputs)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/engine/base_layer.py", line 2643, in _maybe_build     self.build(input_shapes)  # pylint:disable=not-callable   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/utils/tf_utils.py", line 323, in wrapper     output_shape = fn(instance, input_shape)   File "/home/cedar/kzheng/anaconda3/envs/gt/lib/python3.8/site-packages/tensorflow/python/keras/layers/merge.py", line 663, in build     raise ValueError('A Dot layer should be called '

I'll close this issue for now. Thank you guys for your help!


fix: do not encode file names of dynamic routes

Description 📖
This fixes #167.
Background 📜
#167
The Fix 🔨
Decode the resolved path in the file name, but keep %2F (/) encoded.

Thanks @ouuan! Would you rebase so that the CI runs all tests?

Excellent, thanks!

May I request a release for this? Now I'm having broken links in production.

Hi @ouuan, this was released in 0.8.2.


Theme Button Animation

Theme Button Animation
📝 Description
Made the theme button more interactive with simple animation.
⛳ Current behavior (updates)
Stiff and not fluid with just a sudden Icon change
🚀 New behavior
A fluid animation that changes the Icon slowly
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
An example recording (The recording is laggy not the animation, was recorded with chrome extension)

Delete this pr for now, Seems like I did something wrong + I forgot to create a different branch,  Sorry


Can't generate layout files

The command that I run:
./bin/rails g administrate_tailwind_theme:views:layout

The result:
Could not find "../../../app/views/layouts/admin/application.html.erb" in any of your source paths. Your current source paths are:
/usr/share/rvm/gems/ruby-3.2.2/gems/administrate_tailwind_theme-0.0.5/app/views/administrate/application

Issue seems to be in layout_generator.rb:
def copy_template
  copy_file(
    '../../app/views/layouts/admin/application.html.erb',
    'app/views/layouts/admin/application.html.erb'
  )
end

Indeed, sorry for the inconvenience. I have just corrected, it will be available in the version 0.0.6

No problem! Thank you for maintaining the gem :)


Copyrights for licenses

Please have a look at the individual commit messages for the details.

This change is

I see Open Source notices in this PR are added to the Reporter which is not inline  with README.md should be part of a Documenter component. Reporter should only summarize the output from Analyzer, Scanner and Evaluator in an interactive UI whereas the Documenter generates the outcome of the review, e.g. Open Source notices and annotated SPDX files that can be included a SW release

@tsteenbe, I believe we should rather fix the README.md then in this regard. To me it makes more sense to have NOTICE generation as part of the reported, as it's not adding more information to what we already have, and in particular no information is opinionated (like Legal conclusions would be). I believe that's the line where we should split from the "Documenter".

Merging without explicit approval as this has been collaborative work by myself and @mnonnenmacher.


Pinot SPI Refactor

In the first phase of refactoring, we introduced the plugin based architecture for file-system, ingestion, metrics, etc. This is a continuation of the work. In this scope we would like to refactor the segment and query functionality as spi as well.
A detailed design doc can be found here: https://docs.google.com/document/d/1urROfQZuTE8JJmW3IMCeB2i3FYoEyG1TCyPsxvSaNuw/edit?usp=sharing

Segment SPI refactor PR: #6721

Segment-local module PR: #6761


feat: 3180 fix fps in referer

PR copies across the new false positive tests so that they also test against rule 932200, which is the rule that was originally causing the false positive problem.

Superseded by https://github.com/coreruleset/coreruleset/pull/3309


[SYS LIB] Run Until keyword fails

Robot provides only "Wait Until Keyword Succeeds" but in this use case we want something "Run Until Keyword Fails" .. so need to add a python code to loop and expire timer till a failure is detected.

coding in progress


Encryption on newest versions doesn't work.

I have tried to use encryption on newest version (3.3.5) but got the problem.

.Net Framework version  - 4.7.2
On version 3.3.0 all is working fine.

Thanks by feedback @PTerentev, will verify this issue asap.

i've found that problem appears only by the dll from nuget package, i changed dependences from the project to dll from nuget.

I've tested here using the same environment configuration then yours (.NET Framework 4.6.2), everything works fine. Did you ensured your  parameter "message" is not null?

Have you tested on version from nuget?

Yes, both versions 3.3.0 and 3.3.5, everything working fine.

Hmm.. Im getting this one on 3.3.5 (tested on 2 PCs)

Maybe .NET Framework (aka Full-Framework) 4.6.2 suffers from the same issues than 4.6.1, but I will investigate this situation in depth. For now, if you can, try update your version and please verify if the error persists...

I`ve found the cause. It's all because of this line. Its throwing an error because my PC doesn't have this directory. The line only exists in nuget dll, not in the git hub code.

Oh my God, it looks like this line that was just a test was shipped in build-release on version 3.3.5. Sorry for that, I'm going to inspect all the source and draft a new Nuget release 3.3.6 okay. Thanks for finding this terrible thing.

NuGet package update -> https://www.nuget.org/packages/CryptHash.Net/3.3.6; still some minutes to get listed.


Question About The project

Must say this is an awesome project, I would like to know if by any chance this can be archieved on low end computers?

Hey nonunknown, thanks! It depends on how low spec you're talking about, and what sacrifices you're willing to make in terms of quality. All of the lighting is calculated on the GPU so it is entirely dependent on that hardware. You should be able to run this project on quite old hardware, though I think it would need GLES3 support (I haven't tested GLES2 though, it might work).
However when you scale it up to an actual game, things become harder. A game made with this sort of lighting will definitely have higher hardware requirements than other 2D games.
What are you thinking of doing?

Well I've already tested your project some weeks ago, I have a intel HD Graphics 3000 , intel i3 2.1Ghz 2 Gen, 8 GB Mem. So I'm working on a top-down shooter, and would like to have a more realistic lighting, and run on most hardware as possible. Even if this means some sacrifices on graphics quality in terms of your shader...

I wouldn't recommend using an integrated graphics chip with this lighting technique, there's no way you would get acceptable performance unfortunately. I would stick to sprite based lighting for your project, sorry!

Hmm, okay thanks for your time :+1:


hasMany or hasOne doesnt work without getIdAttribute

This issue has been around, can you guys provide a documentation how to properly make sure things work in hasMany and hasOne
for now using this to make sure hasMany and hasOne work
  public function getIdAttribute($value = null)
    {
        return $value;
    }

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

I found the workaround because of this issue https://github.com/mongodb/laravel-mongodb/issues/1902#issuecomment-882694504
I believe the cause of the issue since I am using an existing db that is saved base on mongoose or prisma, foreign keys also get converted into snake_case instead of camelCase.
This are the gist of my collection
rulesauditlogs
{
  "_id": {
    "$oid": "65e74b683d61600fe666d04f"
  },
  "usersId": {
    "$oid": "5be9ce7b3de6dd77db832950"
  },
  "violationsId": {
    "$oid": "65e74addda6dfa40dd5ff39f"
  },
  "createdAt": {
    "$date": "2024-03-05T16:42:16.962Z"
  },
  "updatedAt": {
    "$date": "2024-03-05T16:42:16.962Z"
  },
  "processed": true,
  "isReviewed": true,
  "reviewedAt": {
    "$date": "2024-03-07T10:52:36.027Z"
  },
  "reviewedTimeSpentSeconds": 0
}

rules
{
  "_id": {
    "$oid": "65e74addda6dfa40dd5ff39f"
  },

  "ruleType": "Adherence",
}

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

I found the workaround because of this issue https://github.com/mongodb/laravel-mongodb/issues/1902#issuecomment-882694504
I believe the cause of the issue since I am using an existing db that is saved base on mongoose or prisma, foreign keys also get converted into snake_case instead of camelCase.
This are the gist of my collection
rulesauditlogs
{
  "_id": {
    "$oid": "65e74b683d61600fe666d04f"
  },
  "usersId": {
    "$oid": "5be9ce7b3de6dd77db832950"
  },
  "rulesId": {
    "$oid": "65e74addda6dfa40dd5ff39f"
  },
  "createdAt": {
    "$date": "2024-03-05T16:42:16.962Z"
  },
  "updatedAt": {
    "$date": "2024-03-05T16:42:16.962Z"
  },
  "processed": true,
  "isReviewed": true,
  "reviewedAt": {
    "$date": "2024-03-07T10:52:36.027Z"
  },
  "reviewedTimeSpentSeconds": 0
}

rules
{
  "_id": {
    "$oid": "65e74addda6dfa40dd5ff39f"
  },

  "ruleType": "Adherence",
}

@masterbater They're the same as the Eloquent's hasMany or hasOne. You can pass your custom keys as parameters.
For hasOne:
https://github.com/mongodb/laravel-mongodb/blob/963d01f1af6b1d54082399972523bde4abdf0788/src/Eloquent/HybridRelations.php#L46
For hasMany:
https://github.com/mongodb/laravel-mongodb/blob/963d01f1af6b1d54082399972523bde4abdf0788/src/Eloquent/HybridRelations.php#L102
The issue that you referred to, is solved by now and no longer exists. I think you are using an older version of this package. Please specify the version you are using.

@masterbater They're the same as the Eloquent's hasMany or hasOne. You can pass your custom keys as parameters.
For hasOne:
https://github.com/mongodb/laravel-mongodb/blob/963d01f1af6b1d54082399972523bde4abdf0788/src/Eloquent/HybridRelations.php#L46
For hasMany:
https://github.com/mongodb/laravel-mongodb/blob/963d01f1af6b1d54082399972523bde4abdf0788/src/Eloquent/HybridRelations.php#L102
The issue that you referred to, is solved by now and no longer exists. I think you are using an older version of this package. Please specify the version you are using.

version 4.1.3,  I did it work but needs to add getIdAttribute to each models. I pass in custom keys since it convert camelCase keys into snake_keys

 "name": "laravel/laravel",
    "type": "project",
    "description": "The skeleton application for the Laravel framework.",
    "keywords": [
        "laravel",
        "framework"
    ],
    "license": "MIT",
    "require": {
        "php": "^8.1",
        "guzzlehttp/guzzle": "^7.2",
        "laravel/framework": "^10.10",
        "laravel/octane": "^2.3",
        "laravel/sanctum": "^3.3",
        "laravel/tinker": "^2.8",
        "mongodb/laravel-mongodb": "^4.1",
        "mongodb/mongodb": "^1.17"
    },

No sweat buddy❤

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

I got the biggest issue with your test. You dont test this. For data consistency we always save the real objectid not a string representation on the collection.
   $user = new User;
    $user->name = 'Test';
    $user->email = 'test@gmail';
    $user->save();
    $readings = new Reading;
    $readings->patientId = new ObjectId($user->id);
    $readings->value = 120;
    $readings->unit = 'mmHg';
    $readings->save();
    return $user->readings;

I think relationship with ObjectId native type should be supported

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

I think you dont have a test for this. For data consistency we always save the real objectid not a string representation on the collection.
This Fails hasMany

   $user = new User;
    $user->name = 'Test';
    $user->email = 'test@gmail';
    $user->save();
    $readings = new Reading;
    $readings->patientId = new ObjectId($user->id);
    $readings->value = 120;
    $readings->unit = 'mmHg';
    $readings->save();
    return $user->readings;

This make the relationship hasMany work

   $user = new User;
    $user->name = 'Test';
    $user->email = 'test@gmail';
    $user->save();
    $readings = new Reading;
    $readings->patientId = $user->id;
    $readings->value = 120;
    $readings->unit = 'mmHg';
    $readings->save();
    return $user->readings;

User.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Notifications\Notifiable;
use MongoDB\Laravel\Eloquent\Model;

class User extends Model
{
    use HasFactory, Notifiable;
    protected $connection = 'mongodb';
    protected $collection = 'users';
    const CREATED_AT = 'createdAt';
    const UPDATED_AT = 'updatedAt';
    public static $snakeAttributes = false;

    public function readings()
    {
        return $this->hasMany(Reading::class, 'patientId');
    }
}

Hopefully it could be fix, this I think should be important especially for some people having different codebases.

I don't understand why there is a default attribute mutator on the id key. 👨💻

Sorry for the delay. I will work on this whenever I find some free time.

Hi @masterbater, we have a test suite that cover relationships, including hasMany and hasOne. Could you be more precise about what is the issue: steps to reproduce, expected result and observed result.

I think you dont have a test for this. For data consistency we always save the real objectid not a string representation on the collection.
This Fails hasMany
   $user = new User;
    $user->name = 'Test';
    $user->email = 'test@gmail';
    $user->save();
    $readings = new Reading;
    $readings->patientId = new ObjectId($user->id);
    $readings->value = 120;
    $readings->unit = 'mmHg';
    $readings->save();
    return $user->readings;

This make the relationship hasMany work
   $user = new User;
    $user->name = 'Test';
    $user->email = 'test@gmail';
    $user->save();
    $readings = new Reading;
    $readings->patientId = $user->id;
    $readings->value = 120;
    $readings->unit = 'mmHg';
    $readings->save();
    return $user->readings;

User.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Notifications\Notifiable;
use MongoDB\Laravel\Eloquent\Model;

class User extends Model
{
    use HasFactory, Notifiable;
    protected $connection = 'mongodb';
    protected $collection = 'users';
    const CREATED_AT = 'createdAt';
    const UPDATED_AT = 'updatedAt';
    public static $snakeAttributes = false;

    public function readings()
    {
        return $this->hasMany(Reading::class, 'patientId');
    }
}

Hopefully it could be fix, this I think should be important especially for some people having different codebases.

@GromNaN Sorry for pinging you directly is there any chance you can support relationship that is natively an objectid? Was this already been resolved?

I agree that using ObjectIds is a good practice for foreign keys.
I tried to implement it, but found issues with how the parent Eloquent classes work with foreign keys. They assume every primary/foreign key can be represented as a string. This need in-deep investigation to find how we can implement it in a way that we can maintain in a long-term, without breaking existing application that rely on string foreign keys.

Hello, do we have any news on this subject?
Overriding getIdAttribute works most of the time, but breaks belongToMany relationships.

There is no plan for now.


imap_open(): Couldn't open stream {outlook.office365.com:993/imap/ssl}

ERROR: imap_open(): Couldn't open stream {outlook.office365.com:993/imap/ssl}
Retrying PLAIN authentication after AUTHENTICATE failed.; Retrying PLAIN authentication after AUTHENTICATE failed.; Can not authenticate to IMAP server: AUTHENTICATE failed. {"exception":"[object] (Webklex\IMAP\Exceptions\ConnectionFailedException(code: 0): imap_open(): Couldn't open stream {outlook.office365.com:993/imap/ssl}. Retrying PLAIN authentication after AUTHENTICATE failed.; Retrying PLAIN authentication after AUTHENTICATE failed.; Can not authenticate to IMAP server: AUTHENTICATE failed. at /var/www/html/ats1/api-pms/vendor/webklex/laravel-imap/src/IMAP/Client.php:221

show me your code i will help you

Any chance you could post the solution?

Hi @rohit-2018 @poseidonphp ,
please take a look at these issues: #153#100 #78.
So far it could be caused by to many things. Please try to provide some more information (code samples, call stacks, etc)  in order to figure things out :)
Best regards

how can i fix: imap_open(): Couldn't open stream {outlook.office365.com:993/imap/ssl}

So far, IF you have a shared mailbox, the parameters for the user name should be something like<EMAIL_ADDRESS>like<EMAIL_ADDRESS>But imap_open failed to authenticate.
The only solution I've found was to patch the getAddress() function in Client.php
        $slash = strpos($this->username,'\\');         if ($slash > 0) {             $address .= '/authuser=' . substr($this->username,0,$slash);             $address .= '/user=' . substr($this->username,$slash+1);         }         $address .= '}';
==> address becomes<EMAIL_ADDRESS>


chromium: 106.0.5249.61 -> 106.0.5249.91

Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] 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

[ ] Fits CONTRIBUTING.md.

@kalekseev (6e0a2c1c5422706447a34f71c48f6c37f0a6542d): Do you have time to look into the chromedriver mac64_m1 issue?

@primeos m2 released so they switched from m1 to arm64, use mac_arm64

https://chromedriver.storage.googleapis.com/index.html?path=106.0.5249.61/

Ok, thanks a lot for your quick response! Would you mind sending a PR? I'll merge this as-is for now as I'm unfortunately sick atm (so I lack time/energy to take care of it) and the security release already got delayed a bit... :o


login

Add login feature:
login for users, so they can resume progress when come back.
login for admin, to use certain functions such as pull data and flush database.

Add funcionalty for when crash mid sec 1


[SPARK-29448][SQL] Support the INTERVAL type by Parquet datasource

What changes were proposed in this pull request?
Catalyst's CalendarIntervalType is supported in the Parquet datasource. Interval values are saved as parquet INTERVAL logical type according to the format specification - https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#interval .
Parquet format allows to store intervals in millisecond precision. Because of this restriction, values of Spark's INTERVAL type have to be truncated to milliseconds before storing to parquet files.
Why are the changes needed?

Spark users will be able to load interval columns stored to parquet files in other systems
Datasets with interval columns can be stored to parquet files for future processing

Does this PR introduce any user-facing change?
No
How was this patch tested?

Add tests to ParquetSchemaSuite and ParquetIOSuite
by end-to-end test in ParquetQuerySuite which writes intervals and read them back

Test build #111969 has started for PR 26102 at commit 4563a4e.

Test build #111969 has finished for PR 26102 at commit 4563a4e.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

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-K8s/16984/
Test PASSed.

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-K8s/16986/
Test PASSed.

Test build #111972 has started for PR 26102 at commit 83b4bf2.

Test build #111972 has finished for PR 26102 at commit 83b4bf2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #111973 has started for PR 26102 at commit 81ef7be.

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-K8s/16987/
Test PASSed.

Test build #111973 has finished for PR 26102 at commit 81ef7be.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #111975 has started for PR 26102 at commit de0872e.

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-K8s/16989/
Test PASSed.

Test build #111975 has finished for PR 26102 at commit de0872e.

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/111975/
Test PASSed.

@cloud-fan @HyukjinKwon @dongjoon-hyun @srowen Could you take a look at the PR, please.

My only concern is loss of precision, but what can you do, I suppose, besides not support it?

Just in case, when spark.sql.parquet.outputTimestampType is set to TIMESTAMP_MILLIS, Spark looses precision: https://github.com/apache/spark/blob/ba4d413fc95f65f8b3f28a166ca397612a956e47/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala#L482-L483 . This is not default but anyway.

TODO: Need to exclude the check https://github.com/apache/spark/blob/053dd858d38e6107bc71e0aa3a4954291b74f8c8/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala#L501-L503 for Parquet as well.

Test build #112007 has started for PR 26102 at commit c260622.

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-K8s/17018/
Test PASSed.

Test build #112007 has finished for PR 26102 at commit c260622.

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/112007/
Test PASSed.

In general, are you ok with the proposed changes?

let's wait for https://github.com/apache/spark/pull/26134 and revisit this later.

Test build #112674 has started for PR 26102 at commit 8fd037b.

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

Test build #112674 has finished for PR 26102 at commit 8fd037b.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #112681 has started for PR 26102 at commit 032a2ea.

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-K8s/17637/
Test PASSed.

Test build #112681 has finished for PR 26102 at commit 032a2ea.

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/112681/
Test PASSed.

Test build #113093 has started for PR 26102 at commit cf8023a.

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-K8s/18010/
Test PASSed.

Test build #113093 has finished for PR 26102 at commit cf8023a.

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/113093/
Test PASSed.

let's wait for #26134 and revisit this later.

@cloud-fan @srowen Could you take a look at the PR one more time since it has been unblocked by #26134 recently.

So there should no longer be a difference in reading/writing "1 day" as an interval after the previous change?

It should be no difference since days are written and read back from a separate field.

Test build #115128 has started for PR 26102 at commit 9041110.

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-K8s/19941/
Test PASSed.

Test build #115128 has finished for PR 26102 at commit 9041110.

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/115128/
Test PASSed.

@cloud-fan Should I close this PR?


hey i just scanned neuralnote in virustotal.com and there was something malliscous "Win.MxResIcn.Heur.Gen" false positive or not?

Hi, thanks for pointing this out.
Honestly, I'm not sure what "Win.MxResIcn.Heur.Gen" means exactly, but it looks like one of those heuristic detections that can sometimes flag harmless stuff as malware. It's probably a false positive.


Split error from failure in xml result files.

Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage decreased (-0.0006%) to 99.217% when pulling 54444061c0f654492f78a0e8a7cc51da8b7c6404 on xml_failure into 5a29febebacf61426eac516ece4c157e518268d9 on main.


Initial WIP on documenting options for AWS clusters

Very incomplete, but a start.

Not sure what to do with a WIP. Do you want to have this left floating for comments? Or do you want it merged?

I wanted to push it now so we could discuss if this was the right way to document these options.  @a-robinson suggested putting an option that would otherwise be invisible into config-default.sh, this is an alternative that allows more/better docs.
I guess the first question is whether I should do this...
(1) is this a good idea at all? will other clouds (e.g. gce) document their options also?
(2) is doing it in markdown the right way to do this? I'd say if we keep the markdown simple we could parse or convert this for whatever purposes we had in future, but e.g. do want to put this inline into util.sh and extract it with some magic doctool that Google has?


Build failures (on Linux)

Hey there, this looks so cool
Following the simple instructions in the README I wasn't able to successfully build.
❯ zig build install -Drelease
zig build-exe habu ReleaseSafe native: error: the following command failed with 1 compilation errors:
/home/hiljusti/.local/share/rtx/installs/zig/master/zig build-exe /home/hiljusti/.local/share/czbi/schmee/habu/src/main.zig -lc -OReleaseSafe --cache-dir /home/hiljusti/.local/share/czbi/schmee/habu/zig-cache --global-cache-dir /home/hiljusti/.cache/zig --name habu --listen=-
Build Summary: 0/3 steps succeeded; 1 failed (disable with --summary none)
install transitive failure
└─ install habu transitive failure
   └─ zig build-exe habu ReleaseSafe native 1 errors
src/main.zig:1005:48: error: expected type 'json.stringify.StringifyOptions', found 'comptime_int'
            var jw = std.json.writeStream(sow, 20);
                                               ^~
/home/hiljusti/.local/share/rtx/installs/zig/master/lib/std/json/stringify.zig:10:30: note: struct declared here
pub const StringifyOptions = struct {
                             ^~~~~~
referenced by:
    callMain: /home/hiljusti/.local/share/rtx/installs/zig/master/lib/std/start.zig:588:32
    initEventLoopAndCallMain: /home/hiljusti/.local/share/rtx/installs/zig/master/lib/std/start.zig:522:34
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

I suspect we're both on "master" versions of Zig, but slightly different versions.
Some details on my Zig version and my system:
❯ zig version
0.11.0-dev.4404+4f6013bf5

❯ uname -a
Linux blackbox 6.3.13_1 #1 SMP PREEMPT_DYNAMIC Tue Jul 25 00:19:40 UTC 2023 x86_64 GNU/Linux

Living on the edge!


native pod install error (IOS)

When i try to 'pod install' i am getting this error.
Installing RNSegmentedPicker (2.0.2)
[!] Error installing RNSegmentedPicker
[!] /usr/local/bin/git clone git+https://github.com/adammcarth/react-native-segmented-picker.git /var/folders/ct/v1xh12996pj13djk0jbh5fn80000gn/T/d20210121-97138-3di43f --template= --single-branch --depth 1 --branch v2.0.2
Cloning into '/var/folders/ct/v1xh12996pj13djk0jbh5fn80000gn/T/d20210121-97138-3di43f'...
fatal: unable to find remote helper for 'git+https'

Hey @greenglobaltr :)
As far as I can tell, this sounds like a local issue. A quick google suggested this might fix it for you? But that's definitely weird... I've never seen that error in any context before.
Out of interest, what's are your versions for?:

OSX
Git
Cocoapods

Hey @greenglobaltr :)
As far as I can tell, this sounds like a local issue. A quick google suggested this might fix it for you? But that's definitely weird... I've never seen that error in any context before.
Out of interest, what's are your versions for?:

OSX
Git
Cocoapods


Stable release?

Would you be willing to make a stable release tag for this package? At the moment packagist only has dev-master, which is never recommended on a production site, as there is no control over what changes get pulled in.
Any semver version number would be great, then we can lock usage to major or minor versions as necessary.
Thanks.

I've just created a 1.0 release.

Perfect, thank you :thumbsup:


chore: run Batteries linter on Lean

This PR makes fixes suggested by the Batteries environment linters, particularly simpNF, and unusedHavesSuffices.

Pinging @tobiasgrosser and @luisacicolini, as this suggested changes in BitVec/Lemmas.lean that touched code from your PRs. We're hoping to just have this linter running in Lean CI at some point (soon?), but for now these are just things to watch out for.


Clarify Backfill instruction text

Critical Changes
Changes
Issues Closed

@chrismcmahon Do you mind reviewing the changes I made to the text based on your feedback?

looks much better to me, I like it

@chrismcmahon glad this is better. Once Liane has a chance to review then we can merge. Thanks!

@chrismcmahon Liane was able to review the changes, and I applied them. If all looks good, do you mind merging?


DETACHABLE TERMINAL

Issue Type: Feature Request
while working with internal files one can easily open files in code with inbuilt terminal but while working with remote connection code command will not be working and always having my terminal open is anoying to work like i can see only half screen insted a detachable terminal screen would be a good option where one can just concentrate on the code while working on linux servers using ssh
VS Code version: Code 1.45.0 (d69a79b73808559a91206d73d7717ff5f798f23c, 2020-05-07T16:10:24.511Z)
OS version: Linux x64 5.3.0-51-generic
Remote OS version: Linux x64 4.9.0-8-amd64

/duplicate https://github.com/microsoft/vscode/issues/10121 https://github.com/microsoft/vscode/issues/34442


Extract Text into i18n Files

@johnpatrickmorgan is picking i18n string extraction, please co-ordinate with him if you want to lend and help.


Replaying non deterministic workflows that use local activities can give false negatives

Expected Behavior
Removing/adding a local activity should fail with a nondeterminism error
Actual Behavior
Workflow succeeds
Steps to Reproduce the Problem
If you save a history for the following workflow, add or remove calls to helloworldActivity, and replay the replay does not error
func LocalActivityWorkflow(ctx workflow.Context, name string) error {
	ao := workflow.LocalActivityOptions{
		ScheduleToCloseTimeout: time.Minute,
	}
	ctx = workflow.WithLocalActivityOptions(ctx, ao)
	var helloworldResult string
         workflow.ExecuteLocalActivity(ctx, helloworldActivity, name).Get(ctx, &helloworldResult)
         workflow.ExecuteLocalActivity(ctx, helloworldActivity, name).Get(ctx, &helloworldResult)
	return workflow.ExecuteLocalActivity(ctx, helloworldActivity, name).Get(ctx, &helloworldResult)
}

Note: assuming https://github.com/temporalio/sdk-go/issues/876 is already resolved
This seems to be due to https://github.com/temporalio/sdk-go/blob/38b2b69f18bdc3bcaf1b4e48f940d1c60b5f9253/internal/internal_task_handlers.go#L919 when the workflow returns we skip adding to the replay commands https://github.com/temporalio/sdk-go/blob/38b2b69f18bdc3bcaf1b4e48f940d1c60b5f9253/internal/internal_task_handlers.go#L928
This issue probably effects more commands than just local activities.

I tried just removing these breaks and it seems to cause false positives with other commands like UpsertMemo

Ug, was introduced at the same as the other workflow completed ignoring stuff. Does it seem like too much of a burden to change w.isWorkflowCompleted && !w.isInReplayer { on these and get the matcher working? We can discuss half-impl alternatives that at least solves matching completed workflows if not very well.

Looking at the error more closely the actual issue is the workflow execution commands are not in the replay history, but are in the real history causing a mismatch. hmm
--- FAIL: TestReplayTestSuite (0.00s)
    --- FAIL: TestReplayTestSuite/TestContinueAsNewWorkflow (0.00s)
        /Users/quinnklassen/Documents/Code/sdk-go/test/replaytests/replay_test.go:172:
            	Error Trace:	/Users/quinnklassen/Documents/Code/sdk-go/test/replaytests/replay_test.go:172
            	Error:      	Received unexpected error:
            	            	nondeterministic workflow: missing replay command for WorkflowExecutionContinuedAsNew: (EventId:5, EventTime:(), EventType:WorkflowExecutionContinuedAsNew, Version:0, TaskId:7343443, WorkerMayIgnore:false, Attributes:&HistoryEvent_WorkflowExecutionContinuedAsNewEventAttributes{WorkflowExecutionContinuedAsNewEventAttributes:&WorkflowExecutionContinuedAsNewEventAttributes{NewExecutionRunId:74f38af0-7c7d-4aae-bc10-6c34ba946693,WorkflowType:&v1.WorkflowType{Name:ContinueAsNewWorkflow,},TaskQueue:&v11.TaskQueue{Name:replay-test,Kind:Normal,},Input:&v1.Payloads{Payloads:[]*Payload{&Payload{Metadata:map[string][]byte{encoding: [106 115 111 110 47 112 108 97 105 110],},Data:[102 97 108 115 101],},},},WorkflowRunTimeout:0s,WorkflowTaskTimeout:10s,WorkflowTaskCompletedEventId:4,BackoffStartInterval:<nil>,Initiator:Unspecified,Failure:nil,LastCompletionResult:nil,Header:&v1.Header{Fields:map[string]*Payload{},},Memo:nil,SearchAttributes:nil,},})
            	Test:       	TestReplayTestSuite/TestContinueAsNewWorkflow

possibly we could add all the the workflow execution commands to skipDeterministicCheckForEvent as the execution result mismatch would get caught by the replayer at a higher level, thoughts @cretz ?

Not following exactly as the original description was saying "the replay does not error" was the issue but this is showing an error.
Regardless, can you replicate this error in replay today? Even without the replayer (i.e. unfinished workflow on another worker or a zero-cache setup)? What exactly is the problem here? (sorry for not understanding) I want to be very cautious about doing anything but fixing something that may already be broken and even then be very cautious there.
The goal was to get replayer working with no effect to existing workflows, but if existing workflow replay checks are broken giving false mismatches, we should probably fix that. I'd be hesitant to add more mismatch check possibilities to non-replayer workflow checks today.

The error I posted is the error we get if we remove the breaks in other words making the change you suggested w.isWorkflowCompleted && !w.isInReplayer { .

Regardless, can you replicate this error in replay today? Even without the replayer (i.e. unfinished workflow on another worker or a zero-cache setup)?

No it requires making the above code changes.

I want to be very cautious about doing anything but fixing something that may already be broken and even then be very cautious there.

completely agree, the original issue is as I described in this issue. That issue could be solved by removing the break statements on workflow complete. Removing those breaks causes the WorkflowExecution... events to end up in the his respondEvents but not the replayEvents

Now I definitely admin ignorance wrt respond vs replay, etc. I'm on board w/ any solution that lets most non-determinisms get caught by the replayer for a complete workflow and, most importantly, has no impact on non-replayer code today from a user POV. If that means fixing this or ignoring it, your call.


Invalid notation used in user guide

There is a use of notation [TIP] in the user guide. Somehow, asciidoctor reports that this tag is "invalid for sidestyle block". I have included an excerpt of the paragraph at which the error occurs.
[TIP]
A person can have any number of tags (including 0).
A person can add the symbol '-' if a second phone number is not required for the contact.
Person names should only contain alphanumeric characters and spaces, and it should not be blank.
Person emails should be 2 alphanumeric/period strings separated by '@'.
Phone numbers can only contain numbers, and should be at least 3 digits long.
Tags names should be alphanumeric
Specifically, it occurs under the section "Editing a person".
I don't know whose part is this so I assigned everyone.

This part is input by me last nite. I didn't have a time to change that yet. My plan is to change it before Tuesday.


Restoring mysql backups from s3

This adds the Puppet and tools to enable restoring MySQL backups from an S3 bucket.
Please see commits for detail.

Thank you, rebased and repushed.

Other than a couple of comments the code looks good.

We should wait until this story is complete and then it can be correctly refactored

This is now ready for final review.

One small comment. Let's get this in and testing in integration.
Merge away

Addressed small comment. Merging!


Nighly builds CI failed? :) No new builds since first of March

Nighly builds CI failed? :) No new builds since first of March

Nighly builds CI failed?

Waiting for Appveyor to update their build images with the latest Visual Studio updates.


chore(pgmoon) bump pgmoon from 1.8.0 to 1.9.0

Summary
Bump pgmoon from 1.8.0 to 1.9.0.

@bungle Mind including a link to changelog?

https://github.com/leafo/pgmoon/compare/v1.8.0...v1.9.0

Moving to master and renaming commit to chore(deps)

Done!


nuxt-telemetry make nuxt start blocked when using docker

Versions

nuxt: v2.13.2

Steps to reproduce
I deploy nuxt with docker.
A few days ago, I upgrade nuxt.js. Everything is ok as usual. The container is upped successfully, but I can't visit the site, only got 502. At the moment, I'm very confused.  I found the nuxt isn't started.
What I see in the command line is just like this:
yarn run v1.22.4
$ nuxt start

It's in that state all the time.
By comparison in local without docker, I realized that reason is the nuxt-telemetry:
ℹ NuxtJS collects completely anonymous data about usage.                                                                                09:10:29
  This will help us improving Nuxt developer experience over the time.
  Read more on https://git.io/nuxt-telemetry

? Are you interested in participation?

I can't respond to it in docker. So it Has been waiting.
This way is really too should not. it is strange and weird.
What is Expected?
I found can jump over it by config telemetry: false. But I hope the operation like it should be skipped in production.
What is actually happening?
nuxt-telemetry make nuxt start blocked

Thank you.

Seem here but not using docker
const nuxt = new Nuxt(config);
const builder = new Builder(nuxt);

v1.2.3


Simple queue question

Do you think I should use
https://github.com/antirez/disque
vs Kue?
Do you know about anything? I need a queue and I am deciding Disque vs Kue.
I am thinking Kue is enough I use Redis, I guess I would need another server...
What do you know about?
Thanks so much!

Start with Kue, as it is more abstract and easier to start. Switch to Disque if you faced issues with Kue.


consistent type for genericBuild phases

Issue description
A issue that hit me quite a few times is that most docs refer to the build phases in a nix-shell via
patchPhase/configurePhase etc. Sometimes I got surprised that configurePhase would do nothing. Only recently did I realize (as a sideeffect of someone telling me about set|less) that sometimes  $configurePhase was fine while sometimes it should be $eval "$configurePhase".
Isn't there a way to always be able to use the function configurePhase ? This would reduce the cognitive load and prevent rookie misunderstandings.

Also it would allow to plug in third party tools such as neomake so that it would just have to run "buildPhase" (relevant discussion https://github.com/neomake/neomake/issues/1594)
I commented on the mailing list too: https://groups.google.com/forum/#!topic/nix-devel/FdrsuYeQk6Q

Yeah, we should collect all issues when build behavior isn't matched with building inside nix-shell
I know of two other things:

unpackPhase in builder makes cd .. into unpacked folder, while in nix-shell it doesn't
nix-shell doesn't set all build hooks. So configurePhase or even eval "$configurePhase" doesn't work for some build systems (CMake, I think)

On a related cool rfc https://github.com/NixOS/rfcs/pull/32

very much interested still

I was going nuts trying to figure this out, until someone pointed out this bug for me, and pointed out https://github.com/NixOS/nix/pull/758 which I've now also left a comment on.


TL-DR example is not working

Is this a bug report?
Yes it is.
TL DR example contains app.use('/api', proxy({target: 'http://www.example.org', changeOrigin: true})); which is not working as expected.
Correct syntax is app.use(proxy('/api', {target: 'http://www.example.org', changeOrigin: true})); for this example.
It's small issue, but it takes some time to fix it for me,  and it's not good to have bug in first TLDR example.

it should work both way actually https://github.com/chimurai/http-proxy-middleware#appusepath-proxy


Accessing non-existent property: npm run build

I haven't seen this in a while of my react development and want to know how can I work around it?
I'm trying to do a npm run build and Its giving me the following logs
><EMAIL_ADDRESS>build
> react-scripts build

Creating an optimized production build...
(node:8780) Warning: Accessing non-existent property 'cat' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
(node:8780) Warning: Accessing non-existent property 'cd' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'chmod' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'cp' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'dirs' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'pushd' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'popd' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'echo' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'tempdir' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'pwd' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'exec' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'ls' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'find' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'grep' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'head' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'ln' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'mkdir' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'rm' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'mv' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'sed' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'set' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'sort' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'tail' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'test' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'to' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'toEnd' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'touch' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'uniq' of module exports inside circular dependency
(node:8780) Warning: Accessing non-existent property 'which' of module exports inside circular dependency
Failed to compile.

Unknown browser query `dead`

and it doesn't even npm start node v17 so I down graded to v16+ lts. Any solution on what should I do?

i ran into exactly the same circular dependency error, the following resolved the error though:
upgrading my node version to the latest release with nvm install latest
and updating my dependencies with yarn upgrade-interactive --latest
cheers.

I tried latest version of node v17 on windows but it doesn't even npm start

I got the same warnings in a project freshly created with create-react-app. It turned out that for some reason I had a fairly old version of react-scripts (0.9.5). I upgraded to 5.0.1 and the warnings disappeared.


When JavaScript syntax isn't valid error is misleading

If I enter some some code of:
for(

Then when it runs in the browser JavaScript playground I get:
SyntaxError: Unexpected token '}'

which is confusing because there isn't a } in the code that I've entered (I'm not using a template). If I eval the same code:
eval("for(")

I get:
VM41831:1 Uncaught SyntaxError: Unexpected end of input
    at <anonymous>:1:1

which is what I was expecting to see.

This is how AsyncFunction works. I don't see any way to do a better syntax check. eval messes with global scope and is not suitable for async, so I don't think it's the way to go.


Funds stuck in my ethereum wallet

Hi guys, somehow I managed to cut and paste in a UTC file(to do what , i dont know, i think it was just to get the public address). Anyway I have eth stuck in the account and when I try and withdraw it to put in my current account the send shows a completely different address in my etherwallet or ethereum wallet. I have the password to my original account so if there is anything you can do with the public address and password i would give 10% of what I have in the account. My original mined account number was 0x0095DdBF6B001065cA6C84D9b57DE1C51d5523A6, all mined at dwarfpool. I have sent 20 emails and they have no respect to even respond. If you can do something , please let me know and I will give the password. If i can create a JSON or UTC file from my public address and password,that would be the fix. Is there such a way to do this?
Dumb new miner of 4 months. Not a rich man so this money would really help us out, we have been down on our luck for a while now. Please help.Windows 10 64bit mining on GTX1070 cards.
Bob-wizzsleeve

I'm gonna close this. Don't hesitate to reopen if needed.


Changes related to the IO.read() method

added docstring
fixed incorrect example in docs
fixed inconsistency in Brainware IOs - read() now returns a list of Blocks

Fixes #762

When a particular IO do not have muti block read ability I would totally remove the def read(...) and would stick on the baseio.read(...).
In other words I would remove the def read.
What do you think ?

OK for me.
tell me if I can merge.


fix DChunks rules conflict with zcl-packet

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


Bump Go version to v1.16

Updates the version of Go used in Circle to 1.16 so that we can use goreleaser to test building for the darwin_arm64 target.
Discovered via #147.

No longer seems necessary. I suspect goreleaser fixed at their end.


Not logged in. Log in to deploy to a site

- Do you want to request a feature or report a bug?

Bug

- If the current behavior is a bug, please provide the steps to reproduce.
I tried deploying my site using netlify-cli commands.
Here is my command in batch file I'm using :
npm run build
set NETLIFY_SITE_ID=<my site id>
set NETLIFY_ACCESS_TOKEN=<my access token>
netlify deploy --prod --dir=dist

An error is thrown to me:
Not logged in. Log in to deploy to a site
- What is the expected behavior?
->> Netlify should allow user to pass parameters to deploy to hosting.
- Please mention your node.js, and operating system version.

Node 9.8.0

Apologies, that env var you are using is incorrect and was documented incorrectly at one point.  Updating the docs as we speak.
Use these instead:

NETLIFY_AUTH_TOKEN
NETLIFY_SITE_ID

Additionally, we have improvements around prompt behavior when run from a CI environment in the pipeline.

Let me know if there are any other issues and we can reopen.

Thank you,
After having change NETLIFY_ACCESS_TOKEN to NETLIFY_AUTH_TOKEN, the deployment was successful.
Please update your document, because I've searched for every post about netlify cli deployment and found nothing about NETLIFY_AUTH_TOKEN before asking this question.

We should have improvements out by next week.  Thanks for the feedback!


GAM Backup Code Generation does not put user in grace period for SK enforcement

Please confirm the following:

I have upgraded to the latest GAM release from https://git.io/gamreleases and I still have this issue.
Yes
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/gam/wiki
Yes

Full steps to reproduce the issue:

Have users in G-Suite Enterprise have security key only enforcement enabled
Use the command "gam user  update backupcodes" to create a list of backup codes
User attempts to sign in, but is not given the option to use the backup codes when choosing "Having trouble logging in".  (Codes generated in Admin Console will add user to grace period and enable option to sign in with a backup code)

Expected outcome (what are you trying to do?):
After the codes are generated from GAM, the user should be placed into the grace period for Security Key enforcement so they can use the backup codes.
Actual outcome (what errors or bad behavior do you see instead?):
User does not get added to grace period so they can only use the security key still.

The command: gam user <EmailAddress> update backupcodes
generates a call to this API: https://developers.google.com/admin-sdk/directory/v1/reference/verificationCodes/generate
There is no API support for adding a user to a grace period.
These articles talk about setting up exception groups so that users can use their backup codes until they enroll in 2-Step verification
https://support.google.com/a/answer/2548882?hl=en
https://support.google.com/a/answer/2370108
Maybe when you generate the backupcodes you also add the user to an exception group.
gam user <EmailAddress> update backupcodes
gam update group <2StepExceptions> add member user <EmailAddress>

That's disappointing they don't have an API call for doing that, since the Grace Period has the benefit of not allowing users to add other 2FA methods, and also auto expire the codes after a set time.

I submitted a feature request for an API call to do this.
https://www.cloudconnectcommunity.com/ccc/ls/community/g-suite-feature-ideas/post/4910620885647360


Wip 11605

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1453/
Tests passed for this pull request.

If I take existing master and pull down test_suite.py, test_disable_extension() doesn't fail. :-/

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1454/
Tests passed for this pull request.

Looks good! Thanks for the tests and splitting commits.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1455/
Tests passed for this pull request.


console doesn't support xterm-256color's "Esc E" for newline

When I try to use the control sequence \EE ("Esc E") on a recent WSL,
ganesh@DESKTOP-VCELNMA:~$ echo -e 'foo\EEbar'
foobar

Whereas in native Linux terminals I've tried, and PuTTy, I get:
[ganesh@paxton:~]$ echo -e 'foo\EEbar'
foo
bar

This has recently become a problem for me because it's listed as the sequence for a newline (nel) for xterm-256color in recent versions of the ncurses terminfo database (follow the link to xterm-new and then xterm-p370). It's also documented as an Xterm control sequence here.
From browsing the changelog I think this was added in early 2021 and has recently made its way into Ubuntu - it's not there in Ubuntu 20.04 and is there in Ubuntu 22.04. One way to see if it's listed on a particular machine is to run infocmp -x -1 xterm-256color | grep nel.
It goes wrong for me in practice with tools that use the Haskell terminfo library, which has this code to look up how to produce a newline. If nel isn't listed, then the fallback option works fine, but if it is listed then it tries to use \EE as instructed. This means that tools like ghci (the Haskell REPL) do this:
ghci> 5+510

instead of the expected.
ghci> 5+5
10

It can be reproduced without running a Linux shell by this .cmd file (rename to remove the .txt after downloading). You can see other control sequences do work by changing the E on the final echo line, for example changing it to an M invokes ri (scroll backwards). Of course without the WSL angle there's no particular reason to expect a random xterm control sequence to work in console.
Apologies if I've got completely the wrong end of the stick here about the problem, I didn't know anything much about how terminals actually communicate until I started digging into this problem. Now that I understand the issue better there should be various workarounds available (like setting TERM to something different).

In Powershell works
PS > "foo`eEbar"    # pwsh
foo
bar

PS > "foo$([char]27)Ebar"    # powershell
foo
bar

@hsenag We added support for NEL (ESC E) way back in 2020 (see PR #3271), but it's possible you have a version of Windows that doesn't include that patch - you didn't say what version you're using. The notes on PR #3271 say it was released in build 19603, so if you have a version earlier than that you'll need to upgrade. Alternatively you can use Windows Terminal, which will always get the most recent features and fixes, without having to upgrade the whole operating system.

Thanks! And sorry for forgetting about the version info. Having had this on an older install of Windows, I recently reproduced this on a fresh install with what I thought was all the updates, version 19045.2364, but I now notice my system has been failing to download an update and I hadn't realised. I'll sort that out and try again.


New problem in backpack/crud/src/app/Console/Commands/Version.php file :(.

Bug report
What I did
New problem in backpack/crud/src/app/Console/Commands/Version.php file :(.
Please change to the following, well, or similar at your discretion:
$this->comment('### LARAVEL VERSION:');
// $this->runConsoleCommand('composer show | grep "laravel/framework"'); // -
$this->runConsoleCommand('php artisan -V'); // +

or

$this->line("Package version: " . explode('@', \PackageVersions\Versions::getVersion('laravel/framework'))[0]); // +

$this->comment('### BACKPACK VERSION:');
// $this->runConsoleCommand('composer show | grep "backpack"'); // -
$this->line("Package version " . explode('@', \PackageVersions\Versions::getVersion('backpack/crud'))[0]); // +
}

/**
* Run a shell command in a separate process.
*
* @param string $command Text to be executed.
*
* @return void
*/
private function runConsoleCommand($command)
{
// convert string to array
$command = \is_string($command) ? explode(' ', $command) : $command; // +
$process = new Process($command, null, null, null, 60, null);

Thanks :).

Thank you @sensor-dream ! I've just merged the fix here https://github.com/Laravel-Backpack/CRUD/pull/2520
I'll be tagging it shortly.


[Benchmark] remove proxy

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.xiangyihua seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


react-navigation collapsible minHeaderHeight

Hi, i use this plugin with react-navigation. I want the collapsible header to have minHeaderHeight.
Please how to implement min height.
I add minHeaderHeight to collapsibleOptions but it didn't work

were you able to achieve this?


feature_added

#54 wtr to this PR

Hi @sratslla please have a look on the failing tests.

found a bug, when selecting an item on the files site the selection jumps to an other file

This is rather odd. I will investigate the matter and resolve it. Additionally, I am under the process of updating the tests using your pycharm debugging approch.I anticipate completing this task by the upcoming weekend. I apologize for the delay; this is my first time updating such extensive tests.

please add a NOTICE file on top level dir,
including the icon license
similiar to https://github.com/Open-MSS/MSS/blob/develop/NOTICE


Need to restart app for changes in settings to take effect

Changing the location, source, or time doesn't alter the theme. Only works after quitting and re-opening the app.
Ex. Changing to a location that has daylight from a location that is night-time
Ex. At 2.25pm, setting custom sunset time to 2.30pm.

Thank you for this feedback! I realized the Save button doesn't properly reset the waiting times and WaitUntilNextTimeOfDayAsync() needs to handle the TaskCanceledException that gets generated when saving new times. While fixing this, I came across another bug where I incorrectly assumed that sunrise < sunset < midnight. I'm going to fix these together and publish another release soon.


[docs][setup] use aws profiles instead of exporting creds directly

to: @austinbyers
cc: @airbnb/streamalert-maintainers
size: small
Background
Provide instructions on how to use named AWS profiles for account credential setup.
Changes

Default to using AWS profiles

Testing
Ran the test_the_docs.sh script locally

Coverage remained the same at 95.815% when pulling 1239e255df4c9c345567cb0b89d861abe84da5c8 on jacknaglieri-doc-creds into b0c016d24ca08827cbf2e378a13648bcf3f196a0 on master.

Coverage remained the same at 95.815% when pulling c9c6c498b27c6ce61e5e0cc33d263a9f3141fd91 on jacknaglieri-doc-creds into b0c016d24ca08827cbf2e378a13648bcf3f196a0 on master.

Coverage remained the same at 95.815% when pulling cd627dfddd47aacdb850483e72f22ce3f5ade055 on jacknaglieri-doc-creds into 10cee12eacfda7faf54f1b3e1f8a2c9550bf1594 on master.

Coverage remained the same at 95.815% when pulling cd627dfddd47aacdb850483e72f22ce3f5ade055 on jacknaglieri-doc-creds into 10cee12eacfda7faf54f1b3e1f8a2c9550bf1594 on master.


Object reference not set to an instance of an object.

Hello Rambalac,
First of all, congratulations on this initiative - it is a very useful project. I have been successfully using an older version of the ACDDokanNet software on a Windows 7 machine so far and it runs great.
However, I also wanted to use it on an Amazon EC2 instance running Windows Server 2012 R2, and after installing the latest Dokany <IP_ADDRESS>0 and ACDDokanNet 1.6.2, and trying to mount the Amazon Drive, after the browser logon step I get the following error:
"Object reference not set to an instance of an object."
Here's a detailed description from the Event Viewer log:
MountButton_Click: System.NullReferenceException: Object reference not set to an instance of an object.
   at Azi.Cloud.DokanNet.Gui.CloudMount.<>c__DisplayClass54_0.<Mount>b__1()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Azi.Cloud.DokanNet.Gui.CloudMount.<Mount>d__54.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Azi.Cloud.DokanNet.Gui.CloudMount.<MountAsync>d__46.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Azi.Cloud.DokanNet.Gui.CloudItemControl.<MountButton_Click>d__4.MoveNext()

H:\Projects\C#\ACDDokanNet\ACD.DokanNet.Gui\CloudItemControl.xaml.cs: 33

Also, the OneDrive mount button gives another error, referring to Microsoft.OneDrive.Sdk. You can see it in the attached screenshots.

DETAILED MACHINE INFO:
OS Name	Microsoft Windows Server 2012 R2 Standard
Version	6.3.9600 Build 9600
Other OS Description 	Not Available
OS Manufacturer	Microsoft Corporation
System Name	WIN-UUIUKNDLJ9T
System Manufacturer	Xen
System Model	HVM domU
System Type	x64-based PC
System SKU
Processor	Intel(R) Xeon(R) CPU E5-2670 v2 @ 2.50GHz, 2500 Mhz, 1 Core(s), 1 Logical Processor(s)
BIOS Version/Date	Xen 4.2.amazon, 11/11/2016
SMBIOS Version	2.4
Embedded Controller Version	255.255
BIOS Mode	Legacy
Platform Role	Desktop
Secure Boot State	Unsupported
PCR7 Configuration	Not Available
Windows Directory	C:\Windows
System Directory	C:\Windows\system32
Boot Device	\Device\HarddiskVolume1
Locale	Japan
Hardware Abstraction Layer	Version = "6.3.9600.17196"
User Name	Not Available
Time Zone	GTB Standard Time
Installed Physical Memory (RAM)	1.00 GB
Total Physical Memory	1.00 GB
Available Physical Memory	119 MB
Total Virtual Memory	3.05 GB
Available Virtual Memory	1.56 GB
Page File Space	2.05 GB
Page File	C:\pagefile.sys
A hypervisor has been detected. Features required for Hyper-V will not be displayed.

I wish you all the best and please continue to improve the code :)
Andrei

Hello again,
I managed to fix the "Object reference not set to an instance of an object." error accidentally by installing the Dokan Library 0.8.0 aside the <IP_ADDRESS>0, because in your instructions you said it's needed for ACDDokanNet versions before 1.6.0. I wanted to install version 1.5.5 so I uninstalled the 1.6.2 but then after downloading the 1.5.5 I decided to retry with the 1.6.2, and it worked :D
Still, the OneDrive error remains. Also, when trying to delete it, the Settings program crashes.
Have a good day,
Andrei

I uploaded v1.6.3. Could you try it?

Yes, will do. I'll return with new feedback soon.
Thanks,
Andrei

Oh, tried it and it's really improved the upload and download speeds! Wow! Now it almost really works like an actual drive. This is when copying a setup folder for Photoshop with lots of files of different sizes:

After that it sped up around 3-4x so ~12-15 MB/s. Not bad!
The OneDrive mount still didn't work for me, now it just says Authentication failed (doesn't open a browser tab like the Amazon drive does). But the deletion crash does not happen, that one's gone :)

Have a good day,
Andrei

Hi,
i have this error too with  rel 1.6.3 whenn i try to mount ACD or OneDrive is the same.

The only reason of Object reference not set to an instance of an object during mount I found is wrong Dokan library. I will add version check and warning message.

Hi,
i have installed ACDDokanNet 1.6.4 to solve "Object reference not set to an instance of an object", this version (1.6.4) ask to me "Please install Dokan 1.0.1 to use this application" but i already have this installed on my PC.
i tried to uninstall and reinstall Dokan <IP_ADDRESS> without result.
How can i solve this?


ContentFields missing if part is welded on via ActivatingFilter

Hey guys!
I was wondering: I have a part with two fields attached to it.
Right now I attach this part to my ContentType inside the handler via ActivatingFilter.
My problem is: the fields only show up when i attach the part via migrations and I just can't find out what exactly is different.
Has anyone some experience with this problem?
PS: My issue seems to be related to this StackOverflow question (which sadly has no answers):
http://stackoverflow.com/questions/23829385/welding-a-contentpart-having-a-contentfield

I agree, in its Weld() method, the ContentItemBuilder try to retrieve the part definition through the content type definition on which we want to add the part. So, a new part is created but with an empty field collection...
For more details and a quick workaround for testing, see the above StackOverflow question
Best

I'm glad you had the time to investigate this further!
I tried your workaround in a quick test and it seems to solve the issue. Nonetheless it feels like a bug that should be adressed.

Thank you, I ran into the same problem and this workaround works. But it should be fixed in the source.


collect all antrea-agent pod logs before redeploying antrea - e2e tests

Fixes #1226
Add log collection for antrea-agent pods when antrea is redeployed. This PR would lead to following behavioral change -
If the user provides a log directory which was used for a previous run, existing contents (subdirectories for each test case) will be overridden.
To give overview of dir tree, running tests for TestIPSecTunnelConnectivity & TestPodConnectivityDifferentNodes (e2e) with --logs-export-on-success would result in the following dir tree -
── TestIPSecTunnelConnectivity
│   ├── beforeDisableIPSec.Sep22-09-31-52
│   │   ├── k8s-node-master-antrea-agent-vnmwb-describe
│   │   ├── k8s-node-master-antrea-agent-vnmwb-logs
│   │   ├── k8s-node-worker-1-antrea-agent-kr9v4-describe
│   │   ├── k8s-node-worker-1-antrea-agent-kr9v4-logs
│   │   ├── k8s-node-worker-1-antrea-controller-6fbdccbd67-m8vht-describe
│   │   └── k8s-node-worker-1-antrea-controller-6fbdccbd67-m8vht-logs
│   ├── beforeEnableIPSec.Sep22-09-30-38
│   │   ├── k8s-node-master-antrea-agent-zslgr-describe
│   │   ├── k8s-node-master-antrea-agent-zslgr-logs
│   │   ├── k8s-node-worker-1-antrea-agent-wtbps-describe
│   │   ├── k8s-node-worker-1-antrea-agent-wtbps-logs
│   │   ├── k8s-node-worker-1-antrea-controller-6df878958d-dj67t-describe
│   │   └── k8s-node-worker-1-antrea-controller-6df878958d-dj67t-logs
│   └── beforeTeardown.Sep22-09-32-48
│       ├── k8s-node-master-antrea-agent-9z6sx-describe
│       ├── k8s-node-master-antrea-agent-9z6sx-logs
│       ├── k8s-node-master-kubelet
│       ├── k8s-node-worker-1-antrea-agent-qm2d6-describe
│       ├── k8s-node-worker-1-antrea-agent-qm2d6-logs
│       ├── k8s-node-worker-1-antrea-controller-6df878958d-542ss-describe
│       ├── k8s-node-worker-1-antrea-controller-6df878958d-542ss-logs
│       └── k8s-node-worker-1-kubelet
└── TestPodConnectivityDifferentNodes
    └── beforeTeardown.Sep22-09-54-28
        ├── k8s-node-master-antrea-agent-9z6sx-describe
        ├── k8s-node-master-antrea-agent-9z6sx-logs
        ├── k8s-node-master-kubelet
        ├── k8s-node-worker-1-antrea-agent-qm2d6-describe
        ├── k8s-node-worker-1-antrea-agent-qm2d6-logs
        ├── k8s-node-worker-1-antrea-controller-6df878958d-542ss-describe
        ├── k8s-node-worker-1-antrea-controller-6df878958d-542ss-logs
        └── k8s-node-worker-1-kubelet

Codecov Report

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

@@             Coverage Diff             @@
##           master    #1276       +/-   ##
===========================================
- Coverage   54.98%   41.80%   -13.18%
===========================================
  Files         110       77       -33
  Lines       10573     9639      -934
===========================================
- Hits         5814     4030     -1784
- Misses       4186     5249     +1063
+ Partials      573      360      -213

Flag
Coverage Δ

#integration-tests
?

#unit-tests
41.80% <ø> (+0.03%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/ovs/openflow/ofctrl_group.go
0.00% <0.00%> (-78.13%)
:arrow_down:

pkg/ovs/openflow/ofctrl_bridge.go
1.97% <0.00%> (-67.59%)
:arrow_down:

pkg/ovs/openflow/ofctrl_action.go
17.76% <0.00%> (-65.99%)
:arrow_down:

pkg/ovs/ovsconfig/ovs_client.go
0.00% <0.00%> (-63.55%)
:arrow_down:

pkg/ovs/openflow/ofctrl_builder.go
23.84% <0.00%> (-50.21%)
:arrow_down:

pkg/ovs/openflow/ofctrl_packetout.go
0.00% <0.00%> (-41.43%)
:arrow_down:

pkg/agent/openflow/pipeline.go
22.97% <0.00%> (-34.23%)
:arrow_down:

... and 49 more

Question - what you guys think about mount a separate host dir to /var/log/antrea and /var/log/openvswitch and collect logs there? @antoninbas @siddhant94

@siddhant94 your PR is not rebased properly, as a result it shows a huge diff

Question - what you guys think about mount a separate host dir to /var/log/antrea and /var/log/openvswitch and collect logs there? @antoninbas @siddhant94

I have no strong preference. We don't need to mount anything though, we could just scp the contents from the nodes, since we already have ssh access. I would say it's better than the current approach if it gives us a way to cleanly separate the logs across test cases, which may require some work. The antrea-ovs container logs sometimes have useful information that we do not write to the host's filesystem.

@siddhant94 your PR is not rebased properly, as a result it shows a huge diff

Fixed it and squashed all the commits

Question - what you guys think about mount a separate host dir to /var/log/antrea and /var/log/openvswitch and collect logs there? @antoninbas @siddhant94

I literally have very little insight to provide a preference here. :)
Would definitely go through the logs (ovs) part to gain some meaningful insight as I get the chance.
Do let me know if the separation (antrea/ovs) needs to be implemented in this PR, would dig into further if that's the case.

Would definitely go through the logs (ovs) part to gain some meaningful insight as I get the chance.
Do let me know if the separation (antrea/ovs) needs to be implemented in this PR, would dig into further if that's the case.

I am fine to merge your current implementation first, as it is complete for collecting controller and agent logs.
We can have a separate PR to cover OVS and strongSwan logs. And at that time we can re-evaluate whether to change to collecting logs from the host directory.

Would definitely go through the logs (ovs) part to gain some meaningful insight as I get the chance.
Do let me know if the separation (antrea/ovs) needs to be implemented in this PR, would dig into further if that's the case.

I am fine to merge your current implementation first, as it is complete for collecting controller and agent logs.
We can have a separate PR to cover OVS and strongSwan logs. And at that time we can re-evaluate whether to change to collecting logs from the host directory.

@antoninbas @jianjuns I fixed the merged conflict as https://github.com/vmware-tanzu/antrea/pull/1160 got merged, but for some reason the Kind tests are failing with FAIL: TestNetworkPolicyStats, but the same tests run successfully when running as part of vagrant set-up locally. What is it that I am missing, any pointers would be very much helpful.

@siddhant94 I have observed this test case failing as well. It was recently added. I have opened an issue for this: https://github.com/vmware-tanzu/antrea/issues/1309. Hoping @tnqn can take a look.

@siddhant94 I have observed this test case failing as well. It was recently added. I have opened an issue for this: #1309. Hoping @tnqn can take a look.

The test became flaky because a following PR changed the behavior of termination of antrea pods, created #1312 to fix it.

@siddhant94 this PR is still tagged as a draft. Once Quan's PR (#1312) is merged, please rebase your PR and mark it as "Ready for review".

@siddhant94 this PR is still tagged as a draft. Once Quan's PR (#1312) is merged, please rebase your PR and mark it as "Ready for review".

Done @antoninbas , rebased with master and removed draft tag. Thanks for review and corrections, it helped a  ton.

/test-all

/test-all


Umlauts not working

I installed the bash on the anniversary update. I have a German Windows and a German keyboard. I can enter special characters that require Alt Gr (such as [ or }, see #11) but I can't enter any umlauts (äöü).
I commented on #11 about the problem but probably nobody will look at it as the issue has been closed.

I just tried Alt+0228 on my US keyboard and got ä while in a terminal session connecting to WSL via ssh (PuTTY). I tried the same keystroke in the console one gets with bash.exe, and nothing happened. Maybe PuTTY and ssh can be a workaround until the console is fixed?

@lanthaler I have a German keyboard and German Windows too, but I can type and see all the umlauts properly. I don't need putty or anything.
I'm on build 14393.10, Win 10 Pro

I tried to install console-common, didn't work.
Btw. not only entering umlauts doesn't work, also the output fails. Instead of a benötigt for instance, it renders something like ben├Âtigt. I highlighted renders because when I copy and paste it somewhere else it actually turns into an ö

This is still UTF-8. We're working on it.

Thanks for the discussion.
Closing this issue since:

We are actively working to improve Console's ability to support UNICODE and UTF-8. Bear with us while we complete this work
This is the WSL issues repo, but this is an issue in Console which has its own Console GitHub Repo
GitHub doesn't allow issues to be moved between repos, preserving posters' identity :(

If you have further asks/issues, please file new issues on our Console GitHub Repo.


addEventListener: add capture phase of DOM event dispatch

Fixes #1085.
Supersedes #1897.

Excellent. Thank you!


[Edge] Login button redirects to regular tab, does not authenticate for popup

What I Did

Click Dissenter extension button next to address bar
Click "Log In" button in Dissenter popup window
Log in in new tab opened up
Click Dissenter extension button again to re-open popup

What I Expected

Dissenter page for current tab shown in popup

What I Got

Dissenter extension login page

Browser, OS, and Extension Info

Microsoft Edge 44.17763.1.0 (Microsoft EdgeHTML 18.17763)
Windows 10 version 1809 (OS Build 17763.316)
Dissenter extension v0.1.1

@coldacid thanks for submitting an issue. This is expected behavior. The reason for this is when attempting to log into Gab using the Gab.com API via Dissenter.com through the extension, it is not secure and it cannot perform the OAuth redirect as the extension is simply an iFrame of https://dissenter.com.
Does the extension properly work after logging in via the newly created tab as you described?

@mgabdev no, that's what I'm saying. After logging in, I still just get the login page in the popup, and there's no sign of Dissenter itself on any of my other tabs.

@coldacid Are you still experiencing this issue with the latest release of r9? https://github.com/gab-ai-inc/gab-dissenter-extension/releases/tag/r9

@mgabdev let me set it up and I'll tell you.

Yep, still experiencing it unfortunately.

Still seeing this in the r12 release.  I was logged into Gab, but clicking the Dissenter extension results in a window with a spinner.
The DevTools Console shows the following error:
Uncaught ReferenceError: browser is not defined
    at Popup.e.init (popup.js:4)
    at HTMLDocument.<anonymous> (popup.js:4)

@mgabdev I just tested the change in my PR and with the spinner issue fixed in Edge (Chromium) users can now navigate to https://dissenter.com/ and log-in, then the Dissenter plugin works in Edge.

It's a workaround, but a very low-impact one that makes the plugin quite usable on Edge now.


OPF constraints

Hi, a good work! I have a question regarding the constraint formulation in LLM_as_optimizer_gpt.py. I see that the load constraint is set as 'generation >= the total demand'. However, usually power system requires a equality constraint so that the system can be balanced. May I ask if there are some issues in imposing a strict equality constraint?
def satisfy_load_constraints(thermal_output):     if np.sum(thermal_output)>=400: return True     else: return False
Thank you!

Hi, thanks for your question. Based on our observations, equality constraints are more difficult to satisfy compare to inequality constraints for LLM, therefore in this version we haven't consider them. You're right though, generation and demand balance is important in OPF, please stay tuned for our next updates.


[5.4] no eval for RedisStore::add() [requires Redis version >= 2.6.12]

We already need Redis >= 2.6.0 for RedisStore::add() to work. For the sake of performance, I think it is OK not to support Redis < 2.6.12. Those are old versions and migration should be easy.
The decision is yours.

Sorry sounds like there is difference between signature of \Redis::set() and \Predis\ClientInterface::set().
Please revert this @taylorotwell
@tillkruss Is there any way to call set with 'EX' and 'NX' arguments that work with both predis and phpredis?

Yes, you can overwrite the behavior for PhpRedis in PhpRedisConnection.php.

Can we inherit a new class from \Redis class itself and match the signature with Predis?

No need. Just add PhpRedisConnection::set() to bend PhpRedis to Redis' syntax.

Predis and PhpRedis are using different signatures for a bunch of methods. I'll work on this the next couple of days. After that you can re-submit your PR.

Good. Thanks a lot.


then() doesn't take on promise returned by called handler

In the docs its stated that The call to .then also returns a promise. If the handler that is called returns a promise, the promise returned by .then takes on the state of that returned promise.
So .then(my_func()) should take on the promise returned by my_func()
Take a look at this example:
def testfunc():
    return Promise(lambda resolve, reject: resolve("second"))

Promise(lambda resolve, reject: resolve("first")).then(testfunc()).then(lambda res: print(res))

The first .then should take on the state of the promise returned by testfunc(), so in theory the second .then should print out "second". But this is not what happens, instead it prints out "first" from the first promise.

Nevermind. Apparently resolve() absolutely needs an argument and the function absolutely needs a lambda.


Single field validation as boolean

Hi! I'm looking at ways to validate form inputs without having to rewrite all validation logic in the view itself (such as using ViewModel validation).
The validate method can be used on a single field with class.validate('fieldName'), but it doesn't return a simple true/false value that I can use to either display or hide validation error message. It instead throws a validation error that I need to catch in a callback, which doesn't seem to work well with reactive message display.
Is there a way that I'm not aware of to validate some fields and getting a simple boolean back, without error handling? Thanks in advance!

@davidsavoie1 validation in Astronomy works the same way as Meteor methods so that when you do not pass callback function it will perform client simulation and return immediately without waiting for the server response. So what you can do is:
var u = new User({ firstName: 234 });
try {
  u.validate("firstName");
} catch(err) {
  // Do something with the validation error.
}

So it's quite easy to write helper function:
import { ValidationError } from 'meteor/jagi:astronomy';

const isValid = (doc, fieldName) {
  try {
    doc.validate(fieldName);
    return true;
  } catch(err) {
    if (ValidationError.is(err)) {
      return false;
    }
    // If an error thrown from validation is not instance of ValidationError then we throw this error again.
    throw err;
  }
}

USAGE
var u = new User({ firstName: 234 });
if (!isValid(u, "firstName")) {
  // Do something with the validation error.
}

Actually, it would be probably a good idea to have such a helper built in, so if you would like to contribute it would be great :)

Aaaah! Makes sense! OK, great, I'll try that out!
Thanks a lot for your time and effort, I really appreciate that!

This kind of helper seems to work, but for the record, an actual helper function would be more like so:
const isValid = (doc, fields) => {
  try {
    doc.validate({ fields: [fields] });
    return true;
  } catch (err) {
    if (ValidationError.is(err)) {
      return false;
    }
    // If an error thrown from validation is not instance of ValidationError then we throw this error again.
    throw err;
  }
};

The syntax with a single field in the doc.validate method seems to validate other fields too!

Yep, sorry it should be validate({fields}).
If it goes about validation error messages, take a look at how I've implemented it in this example repository https://github.com/jagi/meteor-astronomy-examples/blob/v2/client/templates.js#L101
Actually, I only care about the first error message and most users too. If you fix this problem, the next time validation happens it will throw another error and so on, until you fix all validation errors. Flooding user with error messages is not a good idea from the UX perspective.


Player Name Input Box

Is your feature request related to a problem? Please describe.
Till this version GOD should ask IDs of each player getting new roles.
What if we can make a process in which player can send his/her real name to the server so we can have a map from player names to roles.
Describe the solution you'd like

Add a textbox beneath id section which user will write down his/her name.
Add a POST request to mafia.py which post string containing player name.
Add a dictionary for mapping names to roles

OR

Just printing the player_name : role in server CLI so GOD will know it]

[optional]:

Save the player name and it's mapping to ip so that frequent users don't bother themselves typing their names each time of entry

this commit closes #5 .
Using authentication page is rather different approach to solve this issue, now each player should enter his/her name in username field and be don't care about password part, server will map this name to it's role and inform GOD with these information.


pb2nc_wrapper-provide support for prepBUFR files with no offset time

In her comparison tests against VSDB, Mallory Row has uncovered yet another type of prepBUFR file naming convention in the production directories.  Some files do not contain offset times and look like:
/gpfs/hps/nco/ops/com/gfs/prod/gdas.2018006/gdas.t00z.prepbufr
Input files are found, but at generate_output_nc_filename, the output filename cannot be generated.  This may require making modifications to string_template_substition code to support this new prepBUFR filename.

Confirmed as working from Mallory.


Premerge for the distributed version

Seems new files are defined.
partitionDirectFileVersions, partitionMaxFileVersions.
Is there some design docs?

Seems new files are defined.
partitionDirectFileVersions, partitionMaxFileVersions.
Is there some design docs?

These are not persisted because they can be calculated by TsFileResources and there are no new files.


Clarification on US #427 Missing error message information in the "Name" field

Dear Mr. Oleh,
I am writing to you in connection with US #427
We noticed that there is no information in the user story about the types of error messages when entering invalid values in the "Name" field
May we suggest adding the following information to the user story?
If entered data in "Name" field is invalid:

enter more than 30 characters - Error message: ‘Name is too long (maximum is 30 characters)’
enter less than 3 characters - Error message: ‘Name is too short (minimum is 3 characters)’
doesn’t enter any data in field - Error message: ‘Please fill out this field’

It would be very much appreciated if you could reply at your earliest convenience.
Best regards,
Oksana Lysak
QC engineer

At the meeting on 12/30/2023 with BA Oleh Nakonechniy and Ihor Kohut confirmed making these changes. They are added to the US.