offline_fs_kbc: doesn't reload the content of aa-offline_fs_kbc-keys.json

Today I was using attestation-agent (AA) to check an image I had encrypted correctly. I noticed two behaviors that I am not sure they are bug or implemented by-design:

After the first request to AA, the aa-offline_fs_kbc-keys.json is never reloaded so changes on the file are ignore
If you run skopeo a couple of times at some point it stops to report the key was not found even though the decryption process fails

Here goes the steps I did:

Built AA with make KBC=offline_fs_kbc

Created the keys file

$ cat /etc/aa-offline_fs_kbc-keys.json
{
  "key_id": "qQ7Lm/5sPqPyTP+1FF0YVkpQ0Q+bgbW9YgJJiHNili0="
}

Created the ocicrypt config file

$ cat ocicrypt_config.json
{
  "key-providers": {
	"attestation-agent": {
  	"grpc": "<IP_ADDRESS>:50000"
	}
  }
}

Started AA:

$ RUST_LOG=attestation_agent ./attestation-agent --keyprovider_sock <IP_ADDRESS>:50000 --getresource_sock <IP_ADDRESS>:50001 &
[2] 2146032
[wmoschet@virtlab1012 attestation-agent]$ [2022-12-15T18:26:58Z DEBUG attestation_agent] KeyProvider gRPC service listening on: <IP_ADDRESS>:50000
[2022-12-15T18:26:58Z DEBUG attestation_agent] GetResource gRPC service listening on: <IP_ADDRESS>:50001

Built skopeo v1.10.0 from source

Trying to decrypt a image:

$ ./skopeo/bin/skopeo copy docker://docker.io/wainersm/coco-fedora_sshd-encrypted:latest oci:coco-fedora_sshd-dencrypted --insecure-policy --decryption-key "provider:attestation-agent:offline_fs_kbc::null"
Getting image source signatures
Copying blob 9786b4129b50 [--------------------------------------] 0.0b / 677.0b
Copying blob 95ffc60be344 [--------------------------------------] 0.0b / 68.1MiB
Copying blob f3ad3fffa30c [--------------------------------------] 0.0b / 190.3MiB
[2022-12-15T18:28:17Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-12-15T18:28:17Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-12-15T18:28:17Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Received unknown key ID: key_id1
[2022-12-15T18:28:17Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-12-15T18:28:17Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-12-15T18:28:17Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Received unknown key ID: key_id1
Copying blob 9786b4129b50 [--------------------------------------] 0.0b / 677.0b
Copying blob 95ffc60be344 [--------------------------------------] 0.0b / 68.1MiB
Copying blob f3ad3fffa30c [--------------------------------------] 0.0b / 190.3MiB
FATA[0000] decrypting layer sha256:95ffc60be344202ceb5647c530097040ed8182b7b842990ad46fbfc7ed769097: missing private key needed for decryption

Edited aa-offline_fs_kbc-keys.json to change "key_id" to "key_id1" and ran again:

$ vi /etc/aa-offline_fs_kbc-keys.json
$ ./skopeo/bin/skopeo copy docker://docker.io/wainersm/coco-fedora_sshd-encrypted:latest oci:coco-fedora_sshd-dencrypted --insecure-policy --decryption-key "provider:attestation-agent:offline_fs_kbc::null"
Getting image source signatures
[2022-12-15T18:35:34Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Received unknown key ID: key_id1
[2022-12-15T18:35:34Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Received unknown key ID: key_id1
[2022-12-15T18:35:34Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Received unknown key ID: key_id1
Copying blob 9786b4129b50 [--------------------------------------] 0.0b / 677.0b
Copying blob f3ad3fffa30c [--------------------------------------] 0.0b / 190.3MiB
Copying blob 95ffc60be344 [--------------------------------------] 0.0b / 68.1MiB
FATA[0000] decrypting layer sha256:95ffc60be344202ceb5647c530097040ed8182b7b842990ad46fbfc7ed769097: missing private key needed for decryption

I ran a couple of times again until it doesn't show the ERROR attestation_agent::grpc::keyprovider (but decryption keeps failing):

$ ./skopeo/bin/skopeo copy docker://docker.io/wainersm/coco-fedora_sshd-encrypted:latest oci:coco-fedora_sshd-dencrypted --insecure-policy --decryption-key "provider:attestation-agent:offline_fs_kbc::null"
Getting image source signatures
Copying blob 9786b4129b50 [--------------------------------------] 0.0b / 677.0b
Copying blob 95ffc60be344 [--------------------------------------] 0.0b / 68.1MiB
Copying blob f3ad3fffa30c [--------------------------------------] 0.0b / 190.3MiB
Copying blob 9786b4129b50 [--------------------------------------] 0.0b / 677.0b
Copying blob 95ffc60be344 [--------------------------------------] 0.0b / 68.1MiB
Copying blob f3ad3fffa30c [--------------------------------------] 0.0b / 190.3MiB
FATA[0000] decrypting layer sha256:95ffc60be344202ceb5647c530097040ed8182b7b842990ad46fbfc7ed769097: missing private key needed for decryption

I restarted the AA process:

$ fg
./attestation-agent --keyprovider_sock <IP_ADDRESS>:50000 --getresource_sock <IP_ADDRESS>:50001	(wd: ~/attestation-agent)
^C
$ cd attestation-agent/
$ ./attestation-agent --keyprovider_sock <IP_ADDRESS>:50000 --getresource_sock <IP_ADDRESS>:50001 &
[2] 2148109

Ran skopeo again. This time it unencrypted the image:

$ ./skopeo/bin/skopeo copy docker://docker.io/wainersm/coco-fedora_sshd-encrypted:latest oci:coco-fedora_sshd-dencrypted --insecure-policy --decryption-key "provider:attestation-agent:offline_fs_kbc::null"
Getting image source signatures
Copying blob 9786b4129b50 done
Copying blob 95ffc60be344 done
Copying blob f3ad3fffa30c done
Copying config 94e20b2b96 done
Writing manifest to image destination
Storing signatures

The first issue is a little weird, but it's probably not a huge problem in context. The offline_fs_kbc is going to be loaded once inside a podvm. The keyfile will be baked into the initrd and isn't going to change at runtime. So it might be a bit unintuitive when using the AA standalone, but I don't think it matters much in production. It would be fairly simple to parse the keyfile at each request if we wanted to fix this.
I'm not really sure about the second issue. It seems like a problem with skopeo. I have definitely run into some oddities with how skopeo does error reporting. It might be handy if we built a front-end for image-rs so that we could ditch skopeo entirely.


Browser freeze when loading values above 200k and works fine below 4k

Hello , I am loading actually 12 series af values each with a min yaxis value of 121 and a max of 788888 and it freeze my browser and go then into angular js error about $digest already in progress..
To be more precise a load a resource through a call to a modal which returns a promis to the highcharts controller... (I am not sure if it has any incidence on my problem)... I wanted to know if this kind of problem occured once and what could be the kind of solutions to avoid that...
Thanks a lot

I solved the problem. It was related to the minorTickInterval in the yaxis.


make thumbnail toggle a linkable value

won't fix because user might receive a link to manuscripts with thumbnails in their phone and it will be a 200mb unwanted download


Build Jerryscript for LLVM

Hi,
I want to build Jerryscript for LLVM in Contiki/Galileo board. Can you please point me to what Makefile/compiler options I should change? Previously I built Jerryscript for Contiki OS/Galileo board. Basically, I  made a directory jerryscript/targets/contiki-x86, wrote a Makefile jerryscript/targets/contiki-x86/Makefile.contikix86 for Contiki/Galileo board and set the appropriate toolchain.
To build in LLVM, I've so far tried the following changes but haven't been successful to build JerryScript.

Updated PATH variable with llvm/build/bin path and Modified compiler options in  Makefile.contikix86 as

jerry:
... ...
-DCMAKE_TOOLCHAIN_FILE=build/configs/toolchain_external_clang.cmake
-DEXTERNAL_LIBC_INTERFACE=$(CONTIKI)/platform/galileo/bsp/libc/i586-elf/include
-DEXTERNAL_CMAKE_SYSTEM_PROCESSOR=x86
-DEXTERNAL_CMAKE_C_COMPILER=clang
-DEXTERNAL_CMAKE_CXX_COMPILER=clang++
-DEXTERNAL_COMPILE_FLAGS="$(EXT_CFLAGS) $(CFLAGS)"

2. Wrote `build/configs/toolchain_external_clang.cmake` as

      ```
      ... ...
      CMAKE_FORCE_C_COMPILER(${EXTERNAL_CMAKE_C_COMPILER} Clang)
      CMAKE_FORCE_CXX_COMPILER(${EXTERNAL_CMAKE_CXX_COMPILER} Clang)

Changed Imported Library Options in  JerryScript/CMakeLists.txt as

elseif("${CMAKE_C_COMPILER_ID}" STREQUAL "Clang")
  add_library(${PREFIX_IMPORTED_LIB}libclang STATIC IMPORTED)
  execute_process(COMMAND ${CMAKE_C_COMPILER} ${FLAGS_COMMON_ARCH} -print-file-name=
                  OUTPUT_VARIABLE IMPORTED_LIBCLANG_RT_LOCATION
                  OUTPUT_STRIP_TRAILING_WHITESPACE)
  set(IMPORTED_LIBCLANG_RT_LOCATION "/home/llvm-git/build/lib/libclang.so")
  set_property(TARGET ${PREFIX_IMPORTED_LIB}libclang
               PROPERTY IMPORTED_LOCATION ${IMPORTED_LIBCLANG_RT_LOCATION})

@seanshpark

@ssultana , Hello,
I've not much experience with LLVM Clang, but it seems that some compile options are different. With your comment above I don't see any problem. If possible, can you post compile or link errors here?
May be you already have tried like this (http://stackoverflow.com/questions/7031126/switching-between-gcc-and-clang-llvm-using-cmake) page...

Please have a look at #1192, with the the new build system it is very easy to build with Clang.

Thanks @tilmannOSG . I'll check that.


Support for DataMapper pattern

In FactoryMuffin v2.x I could define my own saver in setCustomSaver method and have it working with DataMapper pattern. In 3.x I see there is no such option left. I need to create new class RepositoryStore or some kind of.
Is it possible to include such class by default in 3.0 or bring setCustomServer method back?

In 3.x, the recommended practice is to extend the https://github.com/thephpleague/factory-muffin/blob/master/src/ModelStore.php class.

Then, just provide an instance as the first param to https://github.com/thephpleague/factory-muffin/blob/master/src/FactoryMuffin.php#L63.

This felt like a much cleaner way of doing it to me.

Why not to include RepositoryStore into the FactoryMuffin then? DataMapper is pretty popular thing, you know

I can send PR for it

I'm not really against it if you have the time to make a PR. I don't have the time to add it myself atm though.

Ok. I will try to get it implemented today.

I have implementation for v2 so I don't see problems to make it work in v3

Hey, @GrahamCampbell I've found one inconsitency which makes hard to implement proper wrapper and tests for Doctrine:
factory|User returns and Id of a user, while Doctrine does not operate with Ids and uses the instances instead. So if you are used to have
'user_id' => 'factory|User'

for Doctrine we need
'user' => 'factory|User'

and yes, here goes an issue, in second case we still have id value. Can this be changed?
My proposal is for the first case to change syntax to factory_id|User which seems more logical, as we dont put the User into a value but it's Id. And this should be noted somehow.
While factory|User will return a created user.
I know that makes BC break, but before 3.0 is released I think it is possible to have one.
I thought of the idea of overriding GeneratorFactory and FactoryGenerator for this case, but that seemed too much for me. And yes, it's not clear why factory| returns an Id. Maybe it could be a more general version where you can specify which field to return after is an instance is created.
P.S. Please reopen this issue. I'm working on it

I know that makes BC break, but before 3.0 is released I think it is possible to have one.

I wanted to avoid BC breaks after having an RC release. I'll make a decision on what to do later.

As an alternative, we can have different word:
'user_id' => 'factory|User'
'user' => 'entity|User'

@GrahamCampbell  ping?

I've run into the same problems trying with https://github.com/cakephp/orm.
+1 for BC before release - otherwise, it looses some of its framework agnostic*-ity*, at least IMO.

@GrahamCampbell so no response for a while. Should I send PR with BC break?

Is it possible without the break?

Sure, just need to choose proper naming. OK if bc is priority I will send
PR today without break
30 січ. 2016 01:19, користувач "Graham Campbell"<EMAIL_ADDRESS>написав:

Is it possible without the break?
—
Reply to this email directly or view it on GitHub
https://github.com/thephpleague/factory-muffin/issues/408#issuecomment-177013987
.

Thank you. :)

Done in #415

In 3.x, the recommended practice is to extend the https://github.com/thephpleague/factory-muffin/blob/master/src/ModelStore.php class.

@GrahamCampbell any chance we can see this covered in the docs soon?

There is some docs here:

The specific implementation for each is not included. If I were to write that, then I might as well actually write a doctrine adapter. Unfortunately, I don't have time to do this, or to test it.

Thanks!  Any chance of adding a getStore method to the FactoryMuffin class?  I sometimes might want to modify the behavior of the custom ModelStore dynamically - for example, using Laravel's setConnection() on the model instance when saving/updating.

I'd rather not allow mutation of the object after construction. I'd rather the object be constructed with the store set on it. You can extend the store or pass your own class, provided it implements the needed interface.

You can extend the store or pass your own class

Yes of course, but the problem is that if I want to control the behavior of the store after constructing the FactoryMuffin object, I need some way to get at the store.


GC: Tombstone for blobs

Tombstone for GC Blobs
Done when

Mark as tombstone
Prevents handles/requests from getting tombstoned blobs
can untombstone

Closing in favor of https://github.com/microsoft/FluidFramework/pull/13266


Add cucumber config iocuak

Added
Added cucumber config at iocuak-e2e.
Changed

Updated iocuak-e2e directory structure.

Removed
Removed root cucumber config.

Codecov Report

Merging #246 (3514977) into master (0d43131) will decrease coverage by 0.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #246      +/-   ##
==========================================
- Coverage   95.77%   94.99%   -0.79%
==========================================
  Files          86       75      -11
  Lines        1066      839     -227
  Branches      144      123      -21
==========================================
- Hits         1021      797     -224
+ Misses         43       40       -3
  Partials        2        2

Flag
Coverage Δ

iocuak
94.45% <ø> (ø)

porygon
98.30% <ø> (ø)

porygon-typeorm
?

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

Impacted Files
Coverage Δ

...utils/typeorm/findManyOptionsToFindOptionsWhere.ts

...m/src/crud/adapter/typeorm/UpdateTypeOrmAdapter.ts

...orm/src/crud/adapter/typeorm/FindTypeOrmAdapter.ts

...m/src/crud/adapter/typeorm/InsertTypeOrmAdapter.ts

.../crud/modules/iocuak/TypeOrmCrudContainerModule.ts

...rm/src/crud/models/domain/CrudTypeOrmModuleType.ts

...rud/modules/iocuak/TypeOrmDeleteContainerModule.ts

...m/src/crud/adapter/typeorm/DeleteTypeOrmAdapter.ts

.../crud/modules/iocuak/TypeOrmReadContainerModule.ts

...rud/modules/iocuak/TypeOrmUpdateContainerModule.ts

... and 1 more

Continue to review full report at Codecov.

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


how to install druid database?

i see the docs
i can't understand
please give better install docs?

@zengzhihai110 This is a very vague question. What exactly are you confused about? Have you tried the tutorials?

您能听懂中文吗？我英文不大好。那个官方文档，我按照上面的走，都不能正常的按照那个的结果显示出来，你能给下更好地方式让我明白安装druid的 过程吗？

@zengzhihai110
我可以讲中文，但是我们希望用英文来作为主要语言，以方便Druid社区查询问题。在线有一些教程可以帮助你理解Druid setup：http://druid.io/docs/0.8.0/tutorials/index.html。Druid是一个分布式系统，所以我很难用短篇的中文来解释setup的过程，最好的方法还是学习在线教程。
I do speak Chinese but we prefer to keep the language here in English so the community can easily search for issues. There are a few tutorials related to setting up Druid for the first time: http://druid.io/docs/0.8.0/tutorials/index.html. Druid is a distributed system so it is very difficult for me to write a short summary of how you can set it up without you spending some time to learn about the system.

ok  thank you

You can also ask questions and get help through our user group: https://groups.google.com/forum/#!forum/druid-user

thank you very much


Remove direct access to console commands.

Remove any 'console()' commands, such as "console.log()" and "console.warn()".

Closing this PR in favour of PR #939


Download

https://iscanner.com/sharing/8dc5914f

Thank you for opening this issue, we will look into it.

[ ]

[]


Log uncaught exceptions at the top of threads (RIPD-1166)

This is my shot at making it more feasible to track down uncaught exceptions.  The code's not too complicated, but the logging only happens where I've actively added the logging.  If I missed any threads then they won't be reported.
Logging is disabled in the case where a good stack dump is created.  As far as I know good stack dumps are only created when the APPLE macro is defined.
My questions are:

Is there a better way to do this?
Is there a better name than reportUncaughtException for the function?
Did I miss any threads?

Reviewers: @nbougalis (for threads and wisdom) and @HowardHinnant (for template finesse).

[Current coverage][cc-pull] is 69.17%

Merging [#1731][cc-pull] into [develop][cc-base-branch] will increase coverage by <.01%

5 files (not in diff) in src/ripple were modified. more

Misses +8
Hits -8

File ...main/LoadManager.cpp was modified. more

Misses +1
Partials 0
Hits -1

@@            develop      #1731   diff @@
==========================================
  Files           858        860     +2
  Lines         62048      62149   +101
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42899      42990    +91
- Misses        19149      19159    +10
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [44e3312...f8382e8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/ripple/rippled/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/ripple/rippled/compare/44e33121c7c798daf3edfcaddbe8648a34f8a735...f8382e8923f4a97ef32d45b896fdc49550129ed8
[cc-pull]: https://codecov.io/gh/ripple/rippled/pull/1731?src=pr

👍

Incorporated improvement from @HowardHinnant to remove temp variable.

@scottschurr Have you considered writing your own version of std::thread? This would eliminate the possibility of mistakes (e.g. not hooking at the highest level) and also make it easy to convert new code. Possible signature:
template<class InnerThread = std::thread>
class tracked_thread
{
    InnerThread t_;
    ...
};

I've already done something like this for unit tests:
https://github.com/ripple/rippled/blob/develop/src/beast/extras/beast/unit_test/thread.hpp#L19

@vinniefalco I actually had not considered making a new thread type.  I was looking for something I could do that would be minimally invasive.  And I found this approach pretty easy to inject into place.  As you point out, the approach I took has the disadvantage that each thread top must be individually identified and modified.
We're currently using two different kinds of threads: std::thread and beast::Thread.  So if I made a template wrapper type I'd need it to interoperate with both kinds of thread.  But that can probably be done.  The current approach doesn't need to be aware of the distinction.
If you'd like, I could take a shot at making a template wrapper for threads.  Shall I close this pull request and make a run at that?

Your pull request provides immediate relief so we should not wait for a better way.
beast::Thread is deprecated, we should strive to eliminate its use
With the removal of beast::Thread, the std::thread wrapper becomes more practical

@vinniefalco Sounds like a plan to me.  I'll assign a JIRA to myself to remove uses of beast::Thread (I think there are two uses, but I'm not positive).  Once that's done we can revisit.

Merged as fdd1f2ec367539340f79e5ff4eb9b705d936e20b.


Feature Request: Add functionality to check S3 bucket logging

In my use case, I'll need to check whether a logging is enabled / disabled for an S3 bucket.  I'll add these functions to support this case:
GetS3BucketLogging(t *testing.T) *s3.LoggingEnabled
GetS3BucketLoggingE(t *testing.T, region string, bucket string) (*s3.LoggingEnabled, error)
AssertS3LoggingEnabled(t *testing.T, region string, bucket string)

Thanks for the suggestion! While we do have a lot of functions that wrap the SDK into a better interface for other services, we realized that if we wrapped every function of the SDK, it would be redundant and not very maintainable. So lately we have been taking a step back and trying to focus on exposing functions or routines that we find are especially hard or unintuitive to do directly with the SDK.
Looking at the implementation referenced, I can see that these three functions are fairly thin wrappers around the SDK, which suggest that it might be better to directly use the SDK for these particular checks.
With that said, if you still find these wrappers useful, it would be helpful to know what makes the SDK calls unintuitive to you, and the advantages of these terratest wrappers that has helped you overcome those challenges.

Noted. It's indeed a very thin wrapper. I can just add those as helpers (extension of terratest) inside my terraform code for testing instead of bloating the terratest itself. Thanks :D


Third-party "Hide your user agent" breaks Safari

e.g.  set up Hide your user agent with theese params: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.113 Safari/537.36 or Mozilla/5.0 (Windows NT 6.1; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0 and try to open google.com or yandex.ru in Safari

Moved to https://github.com/AdguardTeam/AdguardAssistant/issues/279


Adds MustChangePassword to user create/edit API

This adds the MustChangePassword field to user creation/editing via the admin API endpoints.
Defaults to true, same as CLI.

Codecov Report

Merging #6193 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #6193      +/-   ##
==========================================
- Coverage   38.88%   38.87%   -0.02%
==========================================
  Files         354      354
  Lines       50192    50198       +6
==========================================
- Hits        19516    19512       -4
- Misses      27850    27861      +11
+ Partials     2826     2825       -1

Impacted Files
Coverage Δ

routers/api/v1/admin/user.go
21.97% <0%> (-0.5%)
:arrow_down:

models/unit.go
0% <0%> (-14.29%)
:arrow_down:

models/repo_list.go
66.84% <0%> (-1.06%)
:arrow_down:

Continue to review full report at Codecov.

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


fix bug if call->receiving_buffer is nullptr

We've never seen this bug. Have you seen this bug? Can you provide a stack trace?

Just to make it clear: I don't believe this bug exists since I can see in the code that any time we have a recv_message we are clearly assigning to receiving_buffer. If there is a crash, it is caused by API misuse of not passing in a valid receiving buffer for the recv_message operation, and we can't protect against all the varieties of API misuse that can exist (even if we protect against nullptr, we wouldn't be protecting against other arbitrary pointers). So, I'm going to close the PR but restart one if you have a repro case and a stack trace showing the crash.


[Admin] FHL scrub

Fix dbl space bold**

Docs Build status updates of commit f338db5:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/VBA-Docs, working directory: W:\sloq-s, target branch: HEAD, target branch commit: 118de7dad6a8d1cafd812cdd46903794a8116756, source commit: f338db5a0c307467397e0bcbb8291c36775c5bfc.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d2fa56c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch lindalu-FHL-scrub-day5 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/lindalu-FHL-scrub-day5

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Seq2seq multiple input features

Is there a way to pass extra feature along with the existing word tokens as input and feed it to the encoder RNN?
Lets consider the NMT problem , say I have 2 more feature columns for the corresponding source vocabulary( Feature1 here ). For example, consider this below:
Feature1 Feature2 Feature3
word1 x a
word2 y b
word3 y c
.
.
Moreover, I believe this is glossed over in the seq2seq tutorial(https://github.com/spro/practical-pytorch/blob/master/seq2seq-translation/seq2seq-translation.ipynb) as quoted below:
“When using a single RNN, there is a one-to-one relationship between inputs and outputs. We would quickly run into problems with different sequence orders and lengths that are common during translation…….With the seq2seq model, by encoding many inputs into one vector, and decoding from one vector into many outputs, we are freed from the constraints of sequence order and length. The encoded sequence is represented by a single vector, a single point in some N dimensional space of sequences. In an ideal case, this point can be considered the "meaning" of the sequence.”
Would be great of anyone tells how to practically implement/get this done in pytorch. Thanks in advance.

The simplest way is to concatenate features into a single input vector. However this only works if your RNN takes vector input, not discrete inputs (LongTensor) through an embedding layer. In that case you would want to concatenate your extra features after the input is embedded. This can most easily be achieved by using an extra input argument in your encoder and using torch.cat (see the category input in https://github.com/spro/practical-pytorch/blob/master/conditional-char-rnn/conditional-char-rnn.ipynb for a simple example, though that does not use an embedding layer).
If your features are also discrete you would want multiple embedding layers, one for each, and concatenate all the results.
The quoted section is more about input and target sequence length, not feature size.

@spro  Thank you very much for the reply. Furthermore, I'm new to both pytorch and seq2seq. I wrapped my head around in tensorflow and wasted more than a week on this and got nowhere. Would be of great help if you could translate your suggested approach into code in reference to your seq2seq tutorial(https://github.com/spro/practical-pytorch/blob/master/seq2seq-translation/seq2seq-translation-batched.ipynb) as I got limited time to get this done.
Moreover, You have no idea how useful this repository is to me and many more ppl, please keep continuing your stupendous work. Thanks again.

For the initializer you would need to add arguments for your feature sizes, and create a new Embedding layer for each discrete feature. In the forward() method you would add arguments for the extra features, forward them through the relevant embedding layers, and concatenate them all into one vector for the RNN. The RNN also would have an input size of 3 * hidden_size because of the three embeddings added together (or you could create different embedding sizes for each feature).
Overall it should look something like this:
class EncoderRNN(nn.Module):
    def __init__(..., word_size, feature2_size, feature3_size, hidden_size, ...):

        ...

        self.word_embedding = nn.Embedding(word_size, hidden_size)
        self.feature2_embedding = nn.Embedding(feature2_size, hidden_size)
        self.feature3_embedding = nn.Embedding(feature3_size, hidden_size)
        # Note: * 3 because the above 3 embeddings will be concatenated
        self.gru = nn.GRU(hidden_size * 3, hidden_size, n_layers, dropout=self.dropout, bidirectional=True)

    def forward(self, word_seqs, feature2_seqs, feature3_seqs, input_lengths, hidden=None):
        # Note: we run this all at once (over multiple batches of multiple sequences)
        word_embedded = self.word_embedding(word_seqs)
        feature2_embedded = self.feature2_embedding(feature2_seqs)
        feature3_embedded = self.feature3_embedding(feature3_seqs)
        combined = torch.cat((word_embedded, feature2_embedded, feature3_embedded), 2)
        packed = torch.nn.utils.rnn.pack_padded_sequence(combined, input_lengths)
        ...


Add a logger warning of #LANGUAGE tag is missing.

I think this should be logged on the debug level, like other messages of the same kind.


quick fix low throughout capacity

Fix issue discussed in #417

added some tests + fixes in #419 (on top of your commit here)


Re-assign variants

context: https://centrepopgen.slack.com/archives/C046XUS8DGT/p1691967601787409
Option 1
Edit individual curators. This will take you to the upload page specific to a single curator.
Step 1: Click Edit for a single curator

Step 2: Edit via upload for a single curator (upload page not longer renders curator input field)

Option 2
Edit all curators. This will take you to the upload page which accepts a file that can contain assignments for all curators.
Step 1: Click Edit down the bottom

Step 2: Edit via upload for all curators (upload page as currently implemented will be re-purposed)

@SamBryen What's your preference between these two option? Also feel free to suggest new options!

Additional notes:

Assignments will be reconfigured to match the uploaded file. Assignments no longer present will be deleted, but an existing curation result that is not empty will be kept and marked as orphaned. These are visible and editable to project owners only.

@daniaki Option 2 is preferable as option 1 sounds more time consuming if I have multiple edits to make across multiple curators. The bulk upload in option 2 makes more sense to me.
There is currently a 'download template' option that gives you a list of all the variants currently available for assignment. Could there be a similar option to download a list of current assignments in the format that it needs to be to uploaded in? Would that be the 'download curators' option in your screenshot?
I'll also mention that you can't currently (or I failed to) assign variants to multiple curators at once. If I wanted to assign 3 people the same 10 variants, I need to create 3 different csv files and do 3 separate uploads. I'm not sure if that's because it doesn't like the repeated variant or the multiple curators listed in the same file. Thought I'd mention that incase that impacts how this new feature may work.

It's because the CSV validator doesn't like multiple variants in the same file regardless of who they are assigned to. I can change that! I'll also add the download option for all current assignments.


cargo test reports errors on the master branch.

Describe the bug
A clear and concise description of what the bug is.
test parquet::filter_pushdown::single_file_small_data_pages ... ok
[2022-11-16T07:11:47Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'container_min' is declared as non-nullable but contains null values
[2022-11-16T07:11:47Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'container_min' is declared as non-nullable but contains null values
[2022-11-16T07:11:48Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'container_min' is declared as non-nullable but contains null values
[2022-11-16T07:11:48Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'container_min' is declared as non-nullable but contains null values
[2022-11-16T07:11:49Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'container_min' is declared as non-nullable but contains null values
[2022-11-16T07:11:49Z ERROR datafusion::physical_plan::file_format::parquet::page_filter] Error evaluating page index predicate values Error during planning: Can not create statistics record batch: Invalid argument error: Column 'pod_min' is declared as non-nullable but contains null values
...

To Reproduce
Steps to reproduce the behavior:
Run
cargo test --all

on the master branch
Or just run the test single_file_small_data_pages
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

A weird thing is that the cargo test doesn't fail.

@alamb, could you please take a look?

@HaoYang670  Thanks , this expected now.
because these cols are string , not supported yet , after https://github.com/apache/arrow-datafusion/pull/4132 done,this will be fixed.
Although it reports errors, will have the fall back in https://github.com/apache/arrow-datafusion/blob/509c82c6d624bb63531f67531195b562a241c854/datafusion/core/src/physical_plan/file_format/parquet/page_filter.rs#L350-L351
--------------------------------------------------------------------------------
service:             OPTIONAL BINARY L:STRING R:0 D:1
host:                REQUIRED BINARY L:STRING R:0 D:0
pod:                 REQUIRED BINARY L:STRING R:0 D:0
container:           REQUIRED BINARY L:STRING R:0 D:0
image:               REQUIRED BINARY L:STRING R:0 D:0
time:                REQUIRED INT64 L:TIMESTAMP(NANOS,false) R:0 D:0
client_addr:         OPTIONAL BINARY L:STRING R:0 D:1
request_duration_ns: REQUIRED INT32 R:0 D:0
request_user_agent:  OPTIONAL BINARY L:STRING R:0 D:1
request_method:      OPTIONAL BINARY L:STRING R:0 D:1
request_host:        OPTIONAL BINARY L:STRING R:0 D:1
request_bytes:       OPTIONAL INT32 R:0 D:1
response_bytes:      OPTIONAL INT32 R:0 D:1
response_status:     REQUIRED INT32 L:INTEGER(16,false) R:0 D:0
decimal_price:       REQUIRED FIXED_LEN_BYTE_ARRAY L:DECIMAL(38,0) R:0 D:0

row group 1:         RC:53819 TS:6441769 OFFSET:4
``

The strange thing is that the errors appear on stdout while running cargo test
i wonder if some test is accidentally enabling logging always 🤔

Here is a proposed fix: https://github.com/apache/arrow-datafusion/pull/4358


Cross-resonance integration tests fail in my environment

(env1) hrlquantum:System$ stestr run -n test/test_cross_resonance_hamiltonian.py
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_circuit_generation [0.041296s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_experiment_config [0.006548s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_1__1000000_0__2000000_0__1000_0___3000000_0___2000000_0__10000_0_ [604.556753s] ... FAILED

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

      File "/home/hrlquantum/miniconda3/envs/env1/lib/python3.9/site-packages/ddt.py", line 182, in wrapper
    return func(self, *args, **kwargs)

      File "/home/hrlquantum/work/experiments/System/test/test_cross_resonance_hamiltonian.py", line 250, in test_integration
    self.assertExperimentDone(exp_data, timeout=600)

      File "/home/hrlquantum/work/experiments/System/test/base.py", line 56, in assertExperimentDone
    msg="All threads are executed but status is not DONE. " + experiment_data.errors(),

      File "/home/hrlquantum/work/experiments/System/qiskit_experiments/database_service/db_experiment_data.py", line 1400, in errors
    return self.job_errors() + self.analysis_errors()

      File "/home/hrlquantum/work/experiments/System/qiskit_experiments/database_service/db_experiment_data.py", line 1382, in analysis_errors
    ex = fut.exception()

    AttributeError: 'NoneType' object has no attribute 'exception'

{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_2___1000000_0___2000000_0__1000_0__3000000_0__2000000_0__10000_0_ [504.584740s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_3__10000_0__20000_0__1000_0__5000000_0__1000000_0__2000_0_ [257.341428s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_4__10000_0___1000_0__1000_0__500000_0__1000_0___1000_0_ [188.185892s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_5___100000_0__120000_0__1000_0__150000_0___110000_0___1000_0_ [96.004021s] ... ok
{0} test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_roundtrip_serializable [0.006110s] ... ok

==============================
Failed 1 tests - output below:
==============================

test.test_cross_resonance_hamiltonian.TestCrossResonanceHamiltonian.test_integration_1__1000000_0__2000000_0__1000_0___3000000_0___2000000_0__10000_0_
------------------------------------------------------------------------------------------------------------------------------------------------------

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

      File "/home/hrlquantum/miniconda3/envs/env1/lib/python3.9/site-packages/ddt.py", line 182, in wrapper
    return func(self, *args, **kwargs)

      File "/home/hrlquantum/work/experiments/System/test/test_cross_resonance_hamiltonian.py", line 250, in test_integration
    self.assertExperimentDone(exp_data, timeout=600)

      File "/home/hrlquantum/work/experiments/System/test/base.py", line 56, in assertExperimentDone
    msg="All threads are executed but status is not DONE. " + experiment_data.errors(),

      File "/home/hrlquantum/work/experiments/System/qiskit_experiments/database_service/db_experiment_data.py", line 1400, in errors
    return self.job_errors() + self.analysis_errors()

      File "/home/hrlquantum/work/experiments/System/qiskit_experiments/database_service/db_experiment_data.py", line 1382, in analysis_errors
    ex = fut.exception()

    AttributeError: 'NoneType' object has no attribute 'exception'

======
Totals
======
Ran: 8 tests in 1650.7602 sec.
 - Passed: 7
 - Skipped: 0
 - Expected Fail: 0
 - Unexpected Success: 0
 - Failed: 1
Sum of execute time for each test: 1650.7268 sec.

==============
Worker Balance
==============
 - Worker 0 (8 tests) => 0:27:30.760168

Comments:

This happens consistently.
Note the long run times also in the tests that pass.
More cross-resonance integration tests fail when I run the entire test suite (stestr run) .
I guess that I run into timeout, and that there's a bug in the exception raised by the timeout.

Happens also in the CI:  https://github.com/Qiskit/qiskit-experiments/runs/5217453192?check_suite_focus=true

Yeah probably I underestimated the execution time. This is much shorter in my laptop. I know the bottleneck of the current implementation. Probably I'll open the PR to fix this but meanwhile you can just increase alleviate this limitation
https://github.com/Qiskit/qiskit-experiments/blob/317b964b8cbde0febcda8696fd4bd5cce3490e0a/test/test_cross_resonance_hamiltonian.py#L250

@nkanazawa1989 Can we close this issue?

No, still we need group analysis to fix the performance issue. I'm working on it.


Update node_exporter.service.j2

Without [install] section you can not enable service.

Dupplicate #61

Updated since https://github.com/William-Yeh/ansible-prometheus/commit/89e8a511212708d27326d0532a22f67e5e0d3679


Update Makefile

Removed static linking as it's less likely on modern systems that you'll have the archive files instead of the dynamic libraries
Added -pipe to RFLAGS, will improve compile time, uses more RAM for compilation
Added RFLAGS into LDFLAGS, I believe this is needed for the optimise options to be used correctly with -flto
Added install, symbolic links to /usr/local/bin

You could also add -lopenblas under LDFLAGS to use the faster openBLAS library instead of CBLAS
LDFLAGS= $(RFLAGS) -L. -lfftw3f_omp -lfftw3f -lm -lgsl -lopenblas -fopenmp

@kahlos Well done! Which distro are you using?

I'm on Arch linux, once I got rid of the -static flag it was a breeze :) Installed the dependencies via:
pacaur -S fftw gsl openblas openmp

@kahlos Could you please rebase against master and add openblas to the Makefile?

I'm not very experienced with git, but I think I've done as you were asking.


Error when running initial model build

Last time I was using Robyn was a couple months ago so I was running on an older version. I went to try building a new model with additional data and tried updating what version I was using as well. I've never had an issues with the model building before but I am now receiving an error anytime I run the following code:

OutputCollect <- robyn_run(InputCollect = InputCollect
                           , plot_folder = robyn_object
                           , pareto_fronts = 1
                           , plot_pareto = T)

Error:
>>> Collecting results...

Error in quantile.default(resultHypParam$mape, probs = calibration_constraint): missing values and NaN's not allowed if 'na.rm' is FALSE
Traceback:

1. robyn_run(InputCollect = InputCollect, plot_folder = robyn_object,
 .     pareto_fronts = 1, plot_pareto = T)
2. quantile(resultHypParam$mape, probs = calibration_constraint)
3. quantile.default(resultHypParam$mape, probs = calibration_constraint)
4. stop("missing values and NaN's not allowed if 'na.rm' is FALSE")

I have no nulls or missing values in my input dataframe. I'm running this in an R Jupyter Notebook on GCP. My Robyn package is version 3.4.6
Any guidance would be appreciated! Thank you!

Thanks for reporting this issue. With your example I can't replicate the issue but should be fixed by now. Please, update with remotes::install_github('facebookexperimental/Robyn/R'), reset your R session and retry. Let us know how it goes and feel free to close this issue in case it's fixed.

hm, I'm now getting this:
>>> Collecting results...

Error in RNGseq(n, seed, ..., version = if (checkRNGversion("1.4") >= : NMF::createStream - invalid value for 'n' [positive value expected]
Traceback:

1. robyn_run(InputCollect = InputCollect, plot_folder = robyn_object,
 .     pareto_fronts = 1, plot_pareto = T)
2. foreach(respN = seq_along(decompSpendDistPar$rn), .combine = rbind) %dorng%
 .     {
 .         get_resp <- robyn_response(paid_media_var = decompSpendDistPar$rn[respN],
 .             select_model = decompSpendDistPar[respN, solID],
 .             spend = decompSpendDistPar[respN, mean_spend], dt_hyppar = resultHypParamPar,
 .             dt_coef = xDecompAggPar, InputCollect = InputCollect)
 .         dt_resp <- data.table(mean_response = get_resp, rn = decompSpendDistPar$rn[respN],
 .             solID = decompSpendDistPar$solID[respN])
 .         return(dt_resp)
 .     }
3. do.call("doRNGseq", c(list(n = N_elem, verbose = obj$verbose),
 .     rngSeed))
4. doRNGseq(n = 0L, verbose = FALSE)
5. RNGseq(n, seed, ..., version = if (checkRNGversion("1.4") >=
 .     0) 2 else 1, simplify = FALSE)
6. stop("NMF::createStream - invalid value for 'n' [positive value expected]")

Hi, reading the error message, it seems like N_elem = 0 is causing the problem. Running doRNG:::doRNGseq(0) returns the same error. How many iterations are you using? Are you calibrating the model?

@gufengzhou I'm running 10 trials, 2000 iterations. I am calibrating the model with 21 calibrations. My data is daily and running the model for Jan 1, 2020 until july 1, 2021.
Let me know if I should get you an anonymized version. I'm doing this on the side of some other work so getting an anonymized version might take me a bit.

Yes, your dataset and your demo.R script are needed if I should debug. The easiest way is just rename the vars as var1, var2 etc. But of course you should do whatever is necessary to protect sensitive information. feel free to send it over to<EMAIL_ADDRESS>

Ok, I'll try to get it over to you sometime tomorrow! Thanks!


IE Rendering Problem

I have problems to run the examples with Internet Explorer.
Firefox and Chrome works, but IE loads very slow and shows a weird page as result.
Screenshot: http://imageshack.us/photo/my-images/72/foundationieproblem.png/
OS: Win7Pro 64bit
IE: 9 (9.0.8112.16421)

Hey there - thanks for reaching out. We tested the page you're showing on our Win7 / IE9 machine and we can't seem to recreate the error. Are you getting any errors about loading stylesheets or javascript? Some degree of slowness is to be expected (IE doesn't have the fastest JS parser) but certainly not what you're seeing.
FTW
Jonathan

Jonathan Smiley
ZURB | Design Lead
<EMAIL_ADDRESS>408.540.8667
Build for Every Device
http://foundation.zurb.com
On Oct 31, 2011, at 5:44 AM, newgench wrote:

I have problems to run the examples with Internet Explorer.
Firefox and Chrome works, but IE loads very slow and shows a weird page as result.
Screenshot: http://imageshack.us/photo/my-images/72/foundationieproblem.png/
OS: Win7Pro 64bit
IE: 9 (9.0.8112.16421)
--
Reply to this email directly or view it on GitHub:
https://github.com/zurb/foundation/issues/49

Ah sorry!
Yes I get a 404:

GET http://foundation.zurb.com/src/stylesheets/ie.css
404 Not Found (text/html)

You might check and see if you have that file or if perhaps the permissions are off - failing that you can download the Foundation pack from foundation.zurb.com and get the file if it's missing. Let me know if that corrects what you're seeing. Thanks!
FTW
Jonathan

Jonathan Smiley
ZURB | Design Lead
<EMAIL_ADDRESS>408.540.8667
Build for Every Device
http://foundation.zurb.com
On Nov 1, 2011, at 9:10 AM, newgench wrote:

Ah sorry!
Yes I get a 404:

GET http://foundation.zurb.com/src/stylesheets/ie.css
404 Not Found (text/html)

--
Reply to this email directly or view it on GitHub:
https://github.com/zurb/foundation/issues/49#issuecomment-2592231

Local it works.
But I have Problem at foundation.zurb.com/grid-example2.php and f.exp. www.rcskids.org too

Is it the same problem you saw before, a 404 for ie.css?

The 404 on http://www.rcskids.org/ has been fixed.


[FEAT]: Support for BSDs systems

Contact Details
No response
What is your suggestion?
BSDs are a nice operating system.
It is not a kernel, is the complete operating system!
So, maybe a syscall that works on FREEBSD, might not work in NETBSD. Which, is different from Linux that is just a kernel, and, then it works for different linux variants.

FREEBSD Resources

Assembly and System Calls


Fix versioning & tagging

lerna publish --canary (which is called by yarn run release) is attempting to version packages using a format similar to null-alpha.49+7b9118b which in turn causes a tagging error similar to Error: Invalid tag name "null-alpha.49+7b9118b": Tags may not have any characters that encodeURIComponent encodes.
Interestingly this is not an issue when run locally, perhaps Travis has a different version of npm?

This was fixed by https://github.com/brandingbrand/flagship/pull/616


CB-10798, CB-10384: Fixing permissions for Marshmallow

The present code used to ask for permissions on Marshmallow even while trying to read/write from the application's own locations. It should only ask for permission while trying to write to external locations. Also, the getDirectory method currently didn't include permission check which caused it to break while trying to create a directory in an external location. We can get the list of allowed read/write directories and make sure the native url in the picture lies among them.

@riknoll , @dblotsky , @jasongin : Addressed feedback and added case for {create: true} flag.

You can also refactor needPermission into needWritePermission and needReadPermission, and similarly you can add getPermissionWriteFile, getPermissionReadFile and their directory-related friends in order to have a cleaner API (i.e. no flag args).

@rakatyal, do you happen to know when this will be released, or who I could ask? Just to see if we should wait for it on our own plugin: https://github.com/Microsoft/cordova-plugin-code-push/issues/57

@macdonst: Hey Simon! I know you were planning to do a plugin release. Any update on when it's scheduled?

@rakatyal no one really responded with a date and I'm going to hand it over to Steve to get the release out.

@Macdonst, should I follow up with this person to get a rough guess on the timeline (in which case I might need his details), or will he respond in this thread? Thanks heaps for any help you can give!

@silhouettes I don't think @stevengill will reply on this thread but maybe now that he's been @ messaged. I'm sure there will be a plugins release for Cordova soon but don't know of a timeline.


Fix refresh to revoke the old token

Addresses the bug reported in https://github.com/ueberauth/guardian/pull/121

Looks good @hassox :+1:


Doesn't work with fields which are not primary or foreign keys

Using TypeID with regular fields (not primary keys or assocs) doesn't work.
Using something like field :external_id, TypeID, prefix: "venue", type: :uuid doesn't work. After a bit of investigation, the reason was the following lines in lib/type_id/ecto.ex:
      if primary_key do
        %{primary_key: primary_key, schema: schema, field: field, prefix: prefix, type: type}
      else
        %{schema: schema, field: field, type: type}
      end
    end

    defp find_prefix(%{prefix: prefix}), do: prefix

    defp find_prefix(%{schema: schema, field: field}) do
      %{related: schema, related_key: field} = schema.__schema__(:association, field)

      prefix =
        case schema.__schema__(:type, field) do
          {:parameterized, {TypeID, %{prefix: prefix}}} -> prefix
          {:parameterized, TypeID, %{prefix: prefix}} -> prefix
          _ -> nil
        end

      prefix
    end

It seems to assume all fields are either primary keys, for which we read the prefix from the map directly, or assocs, for which we lookup the prefix in the assoc's module.
I made a PR with the minimal changes https://github.com/lurodrigo/typeid-elixir/pull/1/files to make it work for me (2 lines). But I suspect a better approach would be to, instead of using a boolean primary_key field, use a key_type :: :primary_key | :foreign_key | :none field and pattern match on that. I suspect such an approach would need to also take the problems described in #36 into consideration. Would such a PR be welcome?

I would prefer a PR that just adds support manually specifying the prefix on non-primary key fields. I don't want to change the existing primary_key boolean option.

I just ran in to this issue as well.

I would prefer a PR that just adds support manually specifying the prefix on non-primary key fields. I don't want to change the existing primary_key boolean option.

Isn't this what @lurodrigo have implemented in his PR?

fixed by #39


Put full colors in CSS variables

I'm not sure if there is a reasoning behind the current way, but why do we have:
module.exports = {
  theme: {
    extend: {
      colors: {
        border: "hsl(var(--primary))",
      }
    }
  }
}

And then:
@layer base {
  :root {
    --primary: 222.2 47.4% 11.2%;
  }
}

This way VSCode doesn't give you a color preview in the CSS file (probably other editors as well).
Wouldn't it be better to do:
module.exports = {
  theme: {
    extend: {
      colors: {
        border: "var(--primary)",
      }
    }
  }
}

And then:
@layer base {
  :root {
    --primary: hsl(222.2 47.4% 11.2%);
  }
}

That way we get a preview of the color and a color picker in VSCode like this beside --primary:

@multiwebinc This is how Tailwind does CSS variables. See https://tailwindcss.com/docs/customizing-colors#using-css-variables
See also my comment here: https://github.com/shadcn/ui/issues/199#issuecomment-1513596747

Ok, I read the docs more in detail and I understand why they do it like that. It's because of the <alpha-value> that allows tailwindcss opacity modifiers to work. So maybe this library should add  / <alpha-value> to the end of all colors in tailwind.config.js in case anyone wants to use opacity modifiers.

I understand the design decision given the options, but currently this makes shadcn unusable for me.  I need to see the colors in VsCode more than the time saved with darkmode handling.  Is there a good workaround?

Same issue in Webstorm. Having to deal with hsl values is a real pain 😕.

Agree with @multiwebinc on adding the / <alpha-value> to all colors in tailwind.config.js. Anything opinion against this, @shadcn? Could also PR it in.

Just stumbled onto this pain point and wanted to +1 the notion of working towards a more robust solution

I also agree with building a more robust solution which provides better developer experience. Not being able to visualize colors while applying them is a very big pain point.
Maybe build a vs code plugin to rectify this.

any update on this ?

Hi everyone,
I faced the same problem today and I created a vscode extension for it. The extension adds color decorators to css variables. You can preview and edit the color with color decorators. It only works for hsl and hsla.
 But when I install it, I see no difference in my CSS file. The plugin is enabled in VSCode. Is there some activation or configuration that I need to make? How can I check if I'm hitting a bug?
My index.css file looks like this:
@tailwind base;
@tailwind components;
@tailwind utilities;

@layer base {
  :root {
    --background: 0 0% 100%;
    --foreground: 20 14.3% 4.1%;

    --card: 0 0% 100%;
    --card-foreground: 20 14.3% 4.1%;

    --popover: 0 0% 100%;
    --popover-foreground: 20 14.3% 4.1%;

Same here, nothing happens in my CSS file

Same here, nothing happens in my CSS file

@benyamynbrkyc  i add this in my .vscode/settings.json  "cssVarColorDecorator.supportedLanguages": ["css", "scss", "tailwindcss", "html"], and it works

Hi everyone, I faced the same problem today and I created a vscode extension for it. The extension adds color decorators to css variables. You can preview and edit the color with color decorators. It only works for hsl and hsla.
This is the first time I write a vscode extension, so there may be some bugs.
https://marketplace.visualstudio.com/items?itemName=meouwu.css-var-color-decorator

I love you bro

Same here, nothing happens in my CSS file

@benyamynbrkyc i add this in my .vscode/settings.json "cssVarColorDecorator.supportedLanguages": ["css", "scss", "tailwindcss", "html"], and it works

Tried it, still nothing :(

Not working for me either, looks like a fantastic solution though.

@benyamynbrkyc @tauren I found what was causing it not to work for me... The PostCSS Language Support extension (csstools.postcss) was blocking it, if you have that installed try disabling it.

Any extension like for this Webstorm?

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


New release on npm/crates

Hi!
Would it be possible to have a new release on NPM (https://www.npmjs.com/package/tree-sitter-solidity) and on Crates.io (https://crates.io/crates/tree-sitter-solidity)? It seems the last version was uploaded 2 years ago.
Thanks a lot for the project!

Done!

@JoranHonig Thanks a lot! Sorry I forgot to mention https://pypi.org/project/tree-sitter-solidity/ as well. If you could update it too it would be great ;)


E2E Tests PermGen fix

:+1:

With @just-boris @eroshenkoam we decided to write integration tests in separate repository


A way to get hero names via "hero_id"

When getting data on heroes, it would be nice to get data on a single hero in order to retrieve the name of the hero.
This comes from a need to get hero names when gathering data on a user's data via "/players/{account_id}/heroes" and not having the heroes' names included in the returned data. Thus, I turn to fetching data on heroes via "/heroes" as it includes the hero names, but it returns all the heroes, and it doesn't give the option to retrieve data on a single hero/set of heroes.
Or maybe there is actually an easy way to doing what I am trying to do and I just don't know it, so feel free to let me know  (I am new to the API).

There are only ~100 heroes--you should probably just cache that list somewhere and do a lookup against it?

Yeah, that's what I have been doing now. I thought it would be a nice feature, but it isn't much of an issue tbh.


Previous State Data Shown

I am using angular ui router and Controller As syntax instead of the scope. I am facing a very strange issue which I do not find when I used to have $scope.
The issue comes when I navigate to a new state by filling up the form and clicking on a button. When I go back to the previous state using $state.go() then I can see the form filled up and it does not clear or reload the previous state.

I have even tried using
$state.go('statename', {}, { reload: true });

but I still see the previous state form input fields filled with data I had entered.

can you post a plnkr showing this please?

^^ @zeeshanjan82

Closing due to inactivity. If further information is provided that shows this as a bug, I'll reopen

I apologize  for the late response. I can confirm the issue was not related to the ui-router but the way I was using services in angularjs


general settings

i have change the account activation type for members
to
self activation via email
and i get this errors
error the hex color value is invalid has to be
•Error: The HEX color value is invalid. Has to be "#XXXXXX" format.??
can you help me
thanks
greatings

Please post a screenshot so we can see  in a better context.

Hi @tendoryu
Probably your browser doesn't support the HTML5 input color type.
pH7Builder uses the [https://www.w3schools.com/tags/att_input_type_color.asp](HTML5 input type="color"), however some browser versions such as IE, Edge or Safari don't support that field, and gives wrong value to pH7Builder, and that's why you get this error message.
Can you please use the latest versions of Chrome, Brave or Firefox? You shouldn't have this problem with them :smiley: Let me know

This might also help you :) https://caniuse.com/#search=input type

Or Safari (Mac)

Oops, sorry, there is apparently a problem with the latest link I gave.
This one is more relevant :) https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/color#Browser_compatibility

@polynamaude Yes, Safari is now the NEW IE that everybody hates... :D and gives some hard time to devs...


Replaced TextInput with Input in StreamBox.

(Replaced with input from common)
Android

iOS

Input is not theme-aware and not i18n aware.
BTW I am working on a larger rework for the compose.

ok cool.
if required we can do this in rework :+1:


Ability to override response_body_formatter per test?

I have two separate specs. One spec has complex batch updating logic with a hundred fields that I don't need in my examples, so I am using this config to iterate over the response_body and call a class method to slice out some unnecessary fields.
However, all the other spec files now have this option set which causes them to return nothing at all for the generated JSON for the docs. Setting it back to
RspecApiDocumentation.configure do |config|     config.response_body_formatter = :json   end
has no effect. It uses only the last one that was set regardless of when it was changed.
Is there a proper way to define these options per test or a better way to handle formatting the response_body whenever and however I need to do it? Thanks in advance.

Nevermind. It was my problem. I had a
before each: :customers_query which is incorrect syntax.
Changing it to before :each, :customers_query fixed it.
Thanks for the awesome gem.

so this config is right?
RspecApiDocumentation.configure do |config|
   // ...
  config.response_body_formatter = :json
end

I see the doc is response body  is [binary data]

<EMAIL_ADDRESS>


Hetzner Load Balancer

As talked by email, Is it possible to add option to use hetzner load balancers?
->  to add load-balancer.hetzner.cloud/disable-private-ingress: "true"
Thaks
Joao

As I wrote in the mail, have you red the following docs?
https://github.com/hetznercloud/hcloud-cloud-controller-manager/blob/main/docs/load_balancers.md
I think you have to add the annotation.
apiVersion: v1
kind: Service
metadata:
  name: example-service
  annotations:
    load-balancer.hetzner.cloud/location: hel1
spec:
  selector:
    app: example
  ports:
    - port: 80
      targetPort: 8080
  type: LoadBalancer

How does your service config look?

Hi Marcel,
Thanks a lot for your help. I've managed to make it work... as I'm not a kubernetes experience professional I struggled a lot... but finally made it.
Take a look how I've done it:
apiVersion: v1
kind: Service
metadata:
  name: myapp
  annotations:
    load-balancer.hetzner.cloud/location: fsn1
    load-balancer.hetzner.cloud/use-private-ip: "true"
    load-balancer.hetzner.cloud/disable-private-ingress: "true"
spec:
  selector:
    app: myapp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 5000
  type: LoadBalancer

without the use-private-ip the Loadbalancer couldn't get into the container... .and without disable-private-ingress the load balance isn't created.
Thanks,
Joao

You are welcome. Kubernetes has a steep learning curve. Always try to understand what settings have what effects. Keep having fun.


Integration Testing topic+sample update for 3.0

Fixes #13353
Internal Review Topic

Are there new integration testing concepts/API for 3.0? I focus on the following items and don't add new content outside of 3.0 project updates. If there's nothing new on the framework side and if the following items sound ok, this PR doesn't require engineering review (unless you want to look).
Whole-topic versioning on this one.
A few minor language/description nits.
New 3.0 sample confirmed working.
Made two enhancements to the test for Post_DeleteMessageHandler_ReturnsRedirectToRoot. I make these updates to both the 2.x and 3.x sample apps:

Delete the records in the dB and reseed the dB for the test. We end up with one, non-duplicated set of seeded records this way regardless of other tests running and changing the dB.
I also select the first button of the panel-body div instead of grabbing a named button from the form. QuerySelector in AngleSharp uses takes a "first or default" grab for the element requested. Since the id values of the buttons are arbitrary, the updates here are robust for the test going forward ... it just grabs the first button of the div and doesn't try to find a named button of the whole form, where there's also a "Delete All" button (that we don't want to accidentally grab for this test).

@guardrex Can you make sure we cover this comment in the update? https://github.com/aspnet/AspNetCore/issues/13918#issuecomment-532162945

The doc update looks good. It might be a good idea for someone in the EF team to provide guidance on how to setup and initialize the database for integration tests. @divega or @ajcvickers can probably help here.
In the past a common approach was to use the in memory Db, but as an alternative I believe in memory SQL Lite could be used.
No matter what the approach is, ultimately its a matter of replacing services.

The doc update looks good. It might be a good idea for someone in the EF team to provide guidance on how to setup and initialize the database for integration tests. @divega or @ajcvickers can probably help here.

I volunteer @bricelam. I hope he can take a look or find someone else in the team.

@javiercn Can I open a new issue for the additional updates and we go ahead and merge this?
I'm 🏃😵😅 for the next two weeks with 3.0 priority issues. I'd like to put the additional work (changing the dB and changing the app's environment) on the October sprint. I'll try to work them together (one issue).
Cross-references (just so I don't lose them) ...

Testing with SQLite - https://docs.microsoft.com/en-us/ef/core/miscellaneous/testing/sqlite
Testing with InMemory - https://docs.microsoft.com/en-us/ef/core/miscellaneous/testing/in-memory

@guardrex Sounds good to me.

@Rick-Anderson I'm going to open a new issue for the new work shortly. We can go ahead with our review+merge here now.

I would like to say thanks guys @guardrex @jodavis @divega. I really appreciate it.

This fixed the error in the test, but it doesn't solve the fact that the test is running on the db in Startup instead the db setup in ConfigureWebHost...
From some other issue linked in this whole WebApplicationFactory mess, I found this snippet
        var descriptor = services.SingleOrDefault(d => d.ServiceType == typeof(DbContextOptions<ApplicationDbContext>));                 if (descriptor != null)                 {                     services.Remove(descriptor);                 }
that seems to fix it (at least in the old guardrex/int-test-30 branch), placed just before adding the new AddDbContext with the InMemoryDatabase. But is that correct? Does it have some collateral am I not seeing?
Thanks!

@pedropablo The docs have been updated to reflect the proper way to do it. This is how we mock the database on the Identity UI tests and we haven't had a problem with it.

That's not the proper way to do it. Check the latest comments on #9956 you are using whatever is configured in the real project not what you setup in the WebApplicationFactory.


ipfs:// not supported in XMLHttpRequest

Seems companion does not process ipfs XMLHttpRequest requests.
Created a simple test:
http://node2.e-mesh.net/ipfstest.html

Yes, you are correct. This is because there is no native handler for ipfs:// (yet).
For now an address of HTTP Gateway can be used, it will be opportunistically redirected by the extension if possible.
(I am closing this issue, as we are tracking this effort already in https://github.com/ipfs-shipyard/ipfs-companion/issues/164, feel free to subscribe to that one for updates)


fixed bug caused by previous merged commit

Description
What does this PR: do, how, why?
Limitations & risks
Are there any trade-off or new vulnarbility surface based on theses changes?
Check-list

[ ] Tests are covering the new feature
[ ] Code is natspec'd
[ ] Code is linted and formatted
[ ] I have run the test locally (and they pass)
[ ] I have rebased to the latest main commit (and tests still pass)

Interactions
These changes will impact the following contracts:

Directly:

Indirectly:

https://github.com/Bananapus/nana-swap-terminal/pull/20/files#r1561806410


Fix coverage tracking

Turns out after_success doesn't fail the build on Travis CI so we were running without updating coveralls.io for a while now: https://github.com/travis-ci/travis-ci/issues/758
We'll want to run the coveralls command as part of the regular build phase so that it breaks the build if we stop sending coverage data:
if [ $TRAVIS_TEST_RESULT -eq 0 ]; coveralls; fi

Changes Unknown when pulling 53333117a3f1a58fa4d3c9d00a270dc1d4407f73 on fix-coverage into ** on master**.

Changes Unknown when pulling 53333117a3f1a58fa4d3c9d00a270dc1d4407f73 on fix-coverage into ** on master**.

Changes Unknown when pulling 9d463c0e705cb13cfde35428a882a26ca1d080ab on fix-coverage into ** on master**.

Changes Unknown when pulling 9d463c0e705cb13cfde35428a882a26ca1d080ab on fix-coverage into ** on master**.

Changes Unknown when pulling 9d463c0e705cb13cfde35428a882a26ca1d080ab on fix-coverage into ** on master**.

Changes Unknown when pulling 9f365f04df5bde0197e9a2a0e24152889927da91 on fix-coverage into ** on master**.


Replace threshold with topOffset and bottomOffset

This is a WIP to resolve https://github.com/brigade/react-waypoint/issues/77
It has two notable changes:

threshold has been split into offsetTop and offsetBottom
The new options accept strings and numbers. Strings can be of the form "100", "100px" or "20%". Actual numbers, like 20, are assumed to be pixel values

I haven't added tests 'cause I figure there might be feedback on the implementation. Once the implementation is :+1:'d I'll update the tests.

This looks good! You may go ahead and fix tests. 👍

This looks good! You may go ahead and fix tests

It might take a few days, but it's on my todo list!

It's been some time, but I've rebased this off master and made the existing tests pass (with a small tweak to the current test for threshold). I still need to write the rest of the offset tests (offset top + pixel values for offset), but I figured I'd post this update :v:

Cool, this is coming together nicely.

@trotzig I updated the JSDoc comments and added tests for all of the different ways to specify the bottom offset. I'd love to get any feedback you have on the JSDoc comments.
Would you also like to see analogous tests for the top offset prop?

Sorry for the delay. Let me have a look at this tomorrow when my eyes are less tired. 🌃

Sure thing. Thanks!

All good. If it's not a big deal, we should probably have a simple test for topOffset as well. Don't flesh it out like you did for bottomOffset though. We're probably fine with a single test that checks for onEnter being called.

I'll put that together tomorrow.

Woops – took a few days longer than expected, but this has been updated. The "blah blah"s are gone, and there's a new test for topOffset. I also squashed the commits.
@trotzing, the current setup of the block of the tests that the bottomOffset tests made it easier to test for onLeave than onEnter, so I went ahead and did that.
Lmk if there's anything else I should do/change here :)

Thanks! I'm releasing this as 3.0.0 (going from 2.0.3, since this is a breaking change). Stay tuned.

Rad. Thanks 🙂


Add upsert support

Renames the current insert to upsert, and introduces a new insert that throws an error for duplicates (instead of performing a replace).
Resolves #10.

@lukehorvat Thank you! 👍


[Nodepool_Sync] Sync_201904300100_modified_by_theopenlab-ci

fixed : https://github.com/theopenlab/labkeeper/commit/321626bd5af391649bfd9f243f9ae332e036cecd


PropTypes should be imported from "prop-types"

In react 16, we cannot import PropTypes from 'react', It shows an error in propTypes. We should import it like below
import PropTypes from "prop-types";
Please update your library index.js file.
Thanks

Hi, @husny-ahamed.
PRs are welcome 😄

this is done already https://github.com/lwansbrough/react-native-camera/blob/master/index.js#L2


reduce the network traffic for single-namespace tests

This changes the behavior of CRD store to watch only the specified
namespace if namespace restriction is set. This is a feature
to be used by tests only, and this reduces the network traffic
significantly, when multiple tests are running in a single cluster.
This does not affect production use cases.

/assign @mandarjog

can you validate it still works for cluster-wide by installing istio in istio-system and bookinfo in default namespace? (Unfortunately there is no automated test).

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
56f67399fb28053950613bf39c29a97cca760a52
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@andraxylia make e2e_mixer with --cluster_wide succeeds on my environment.

/test istio-pilot-e2e

PTAL @mandarjog @andraxylia

@mandarjog please review this


[QUESTION] Regarding DNS lookup when using custom Socket Factory

Question
Hi,
I'm facing an issue when I'm trying to connect to SQL Server.  My SQL server in a private network accessible by a proxy server.
When I am trying to create the connection using my db host name it throws exception as the driver try to do dns look up locally for the host name before creating the custom socket. Since, the db is in private network it will not be resolve by dns and hence if address is unresolved then it is throwing exception here.
How can i delay this lookup to be done by the proxy then we are passing using custom socket.
Similar to the ticket https://github.com/microsoft/mssql-jdbc/issues/1339

Thanks for your question ravi-Mode, we are investigating the issue.
Thx,
Barry

Hi ravi-Mode,
From your description, it sounds like you are trying to connect from one private network (i.e. hosting the JDBC driver) and the SQL Server DB hosted on a second private network with a proxy server providing traffic routing between the 2 private networks. There are different flavors of proxy server but the simplest is an anonymous proxy. This means that the client/driver never knows the server or in this case SQL Server DB and the server never actually knows the client. But both the client and server can see the proxy server (i.e. it bridges the 2 private networks).  The point is neither the client or the server ever directly talk to each other. They only talk to the proxy server. That is why the DNS address of the SQL Server DB on the private network cannot be used by the driver on the other network to connect to the DB.
In this case you must configure the driver to talk to the proxy server and not the SQL Server DB. The second thing that needs to happen is that the proxy server needs to be configured to route the traffic intended for the SQL Server DB through the proxy server to the DB. If it is not properly configured, the SQL Server is essentially behind a firewall and is not visible to anyone outside if the DB private network.
Depending on the proxy server configuration, it should accept traffic on the SQL Server default port 1433 and redirect all traffic from the port on the proxy server to the SQL Server on the private network. The poxy server can use DNS to determine the IP of the SQL Server on the private network.
If you don't have access to the proxy server to perform that configuration you should discuss this issue with the owner of the proxy server, presumable your IT department. They should be able to help you properly configure your JDBC connection string to talk to the correct IP/port on the proxy server to access the SQL Server DB on the private network.
Please let us know if this helps.
Thx,
Barry

Hi @barryw-mssql ,
Thank you for the response. Here proxy server is configure and able to forward the request to SQL server DB. But the problem arises when SQL server is configured to have AD authentication. In this case when proxy send the request to SQL server then it redirect. In this case, JDBC uses the proxy host name HTTP header. Because of this authentication failed as the Authentication server is trying to check for Proxy host name instead of DB host name.
Due to this reason we can not just use proxy hostname in JDBC connection string.

Hi ravi-Mode,
It sounds like we are no longer discussing a proxy issue but a domain authentication issue across separate networks. From your description, if you were using SQL Server Authentication (i.e. DB user/password) everything would work because as you stated, the proxy server is properly configured to route the traffic from the driver to the SQL Server DB. Therefore the proxy server is likely not a factor in the problem. Generally proxy servers are not used in the authentication process except to pass the authentication request with associated credentials or Kerberos tokens. The validation of the provided Kerberos tokens are performed by the DC (i.e. not the SQL Server DB).
To completely rule out the proxy server as a factor in this problem could you please try configuring the SQL Sever to use SQL Server authentication. If that does work we will know the proxy server is not the issue.
It sounds like the issue is that you are trying to pass a Kerberos token (i.e. Active Directory authentication is Kerberos based) across separate networks with different domain controllers (DC). The JDBC driver is hosted on one network with it's own DC and the SQL Server is on a 2nd network with it's own separate DC. Since you are using Active Directory authentication (i.e. domain based authentication), the token from your system hosting the driver cannot be authenticated by the SQL Server on a different domain. Is that correct?
If the above is not correct can you please provide more details about your network configuration and specifically information about any domain controllers on the 2 networks.
Thx,
Barry

Hi @ravi-Mode,
Please let us know if you can follow up on @barryw-mssql's above post. If not, we will be closing this issue for inactivity.

Closing for inactivity.


how to resume training Process.

i have train pix-2-pix algorithms for 200 epoch. i want to train it for 100 more epochs. can you tell me how to resume training process.   @christopherhesse  @leberknecht

--checkpoint folder_train

When you want to continue training add the "--checkpoint path/to/output_dir" with the corresponding folder that was the output_dir the first time you ran it.

--checkpoint folder_train

When you want to continue training add the "--checkpoint path/to/output_dir" with the corresponding folder that was the output_dir the first time you ran it.

Do you mean I can continue training the network just by adding the checkpoint parameter in following code:???
python pix2pix.py
--mode train
--output_dir facades_train
--max_epochs 200
--input_dir facades/train
--which_direction BtoA
【add here：--checkpoint ....】

Yes, that is correct. The folder that you specify needs to include the checkpoint that was previously saved.


Add the ability to auto update bots

Given this is a public bot people use, the bot should feature the ability to automatically update itself from the master branch of this repository using GitPython

Outside the scope of this


fix build error: duplicate symbol 'poselib_info()'

when I intergrade PoseLib to my own slam code, clang has give me this error:
[build] duplicate symbol 'poselib_info()' in:
[build]     CMakeFiles/re0slam.dir/Debug/run.cc.o
[build]     CMakeFiles/re0slam.dir/Debug/slam.cc.o
[build] duplicate symbol 'poselib_info()' in:
[build]     CMakeFiles/re0slam.dir/Debug/run.cc.o
[build]     CMakeFiles/re0slam.dir/Debug/frame.cc.o
[build] ld: 2 duplicate symbols for architecture arm64
[build] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[build] ninja: build stopped: subcommand failed.

this caused by 'poselib_info' defined in a header file; function definitions in header files can lead to ODR violations (fix available), add inline to avoid it.

Thanks for finding and fixing this.


how to 'proxy'?

I built it and can run it.
Pardon, but how do I proxy a site, lets say twitter.com or anything?

Hi @vicApakau, while running the included Launcher class you have to set the clients proxy settings adress to localhost and port to 9090. Of course, this depends on the client. With Mozilla, for example, please have a look at this: https://support.mozilla.org/en-US/kb/advanced-panel-settings-in-firefox#w_connection

Click the Connection > Settings... button
Check Manual proxy configuration
Enter HTTP-Proxy and Port
Check Use this proxy server for all protocols

For now, if you get an URL with your browser, it's delivered via the proxy. Try to shut down the proxy to validate this.


ci/bazel: Add CI reporting tool

a step towards fixing #36326

main@2eebf0d

ext_proc: skip timeout timer on trailer in async mode. (#36524)

Commit Message: Like header and body, timer/timeout should be skipped on
trailer as well in async. It is because there is no response/ or
response is ignored.
Risk Level: Low
Testing: Integration test
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: tyxia<EMAIL_ADDRESS>

push@2024-10-10T15:55:22:

Envoy/Publish & verify (failure)
Envoy/Prechecks (failure)

main@601292a

proto util: change the input string ref to string view (#36525)

Commit Message: proto util: change the input string ref to string view
Additional Description:
absl::string_view is better thanconst string& in most cases to avoid
unnecessary copy.
Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

Signed-off-by: wangbaiping<EMAIL_ADDRESS>

push@2024-10-10T13:06:35:

Mobile/iOS tests (failure)

main@06ea5be

deps: Fix (renamed) ipp-crypto repo hash (#36540)

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>

push@2024-10-10T17:42:44:

Mobile/iOS tests (failure)

main@965647e

mobile: Update the Apple proxy settings polling to 10s (#36538)

This is the same value that Cronet uses:
https://source.chromium.org/chromium/chromium/src/+/main:net/proxy_resolution/proxy_config_service_ios.cc;l=23;drc=0ddf755aab1e6d0f9c21fa09cd098c4ff6d1cac0.
Signed-off-by: Ali Beyad<EMAIL_ADDRESS>

push@2024-10-10T19:13:22:

Envoy/Checks (failure)
Mobile/iOS tests (failure)

main@74f71bb

coverage: ratcheting (#36518)

Signed-off-by: Alyssa Wilk<EMAIL_ADDRESS>

push@2024-10-10T19:14:26:

Mobile/iOS tests (failure)

main@bfa0e07

ci: Remove redundant AZP gc (#36541)

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>

push@2024-10-10T20:50:40:

Mobile/iOS tests (failure)


Remove link to altinn-din from app-template

Description
In the next release of app-frontend, the font is switched to use Inter, and this font is added directly through app-frontend's css bundle, so a link in app-template is not needed https://github.com/Altinn/app-frontend-react/pull/1173. Therefore, the link to Altinn-DIN should be removed since this will no longer be used.
Related Issue(s)
Fixes #181
Verification

[ ] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

FYI: the switch to Inter is not in prod yet

Setting @bjosttveit as reviewer, he can approve this when the necessary changes in app-frontend is in prod

Setting @bjosttveit as reviewer, he can approve this when the necessary changes in app-frontend is in prod

Missed the comment and approved, but still valid as I'm just a poor backender :-D


Fix Ban Appeals on Fly.io

Tweaks request headers to make ban appeals work again on Fly.io

Goofy Redgurt


Change templates to use new version

@rylev can you rebase this on master?  I implemented a stable vs nightly Rust CI leg and will need the rebase to pass the checks.  Thanks!

@all-contributors please add @rylev for code, ideas, maintenance, review, and infra.


bosco run now has --cdn command

This will run bosco cdn as a command supervised by pm2, which allows it to be
stop and started as well being able to view its logs via pm2.
Its not an elegant solution by a long shot, however it works.
Example:
$ bosco run --cdn
$ bosco stop #This will stop cdn aswell

:+1: for the idea

:+1: thanks Tom

:+1: thanks @tomgco


perf: reduce-testing-time

Description
As the title. Reducing testing time.
Issue linked
Close #3281
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.

I'm going to reduce also test_only_one_instance by just checking passing arguments.


Remove flagsReady hook

Remove Unleash's flagsReady hook.
This does not appear to be doing anything as it's always true. And don't want to block the UI from rendering, considering the UI will update when Redux dispatches flag events.
https://issues.redhat.com/browse/COST-3089

Codecov Report

Merging #2555 (037c0f4) into main (84b2a3e) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 037c0f4 differs from pull request most recent head 7da33c9. Consider uploading reports for the commit 7da33c9 to get more accurate results

@@           Coverage Diff            @@
##             main    #2555    +/-   ##
========================================
  Coverage   65.86%   65.86%
========================================
  Files         452      452
  Lines        8979     8979
  Branches     2081     1938   -143
========================================
  Hits         5914     5914
  Misses       3062     3062
  Partials        3        3

Continue to review full report at Codecov.

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


Why the Wallpapers?

The README instructions say:

Copy the whole repository to EFI partition to the path mentioned above. (You have to create an empty EFI directory, and put the whole repo in it).

The Wallpapers directory's size (478,085,111 bytes (478.2 MB on disk) for 35 items) exceeds the size of the EFI partition (200MB).
May I suggest either deleting the Wallpapers directory (and purging it from git history), or updating the README?

Problem fixed: 'Remove the "Wallpapers"'
Case closed.


[TextField] New filled variant override example

We have a customization example for the standard and outlined variant, but none for the filled.

I wish we had a size=small implementation, the demo would be even better.

Title says Button but addition is a TextField?

Title says Button but addition is a TextField?

You're right. Does the changelog use commit messages or PR titles?

It normally uses the commit title which is the old PR title at first glance unless you click into it

🙈. The changelog uses the pull request title.


Update package.json (fixes #5)

Uses HTTPS protocol instead of SSH (fixes #5)

Oops, sorry, the issue is not coming from this package but https://github.com/yWorks/jsPDF, see yWorks/jsPDF#10.
In the other hand, this change could still be interesting.


Fix timeseries reporting for Usecase 5

This fixes missing timeseries in UC 5:

RE Generation - not reported
Slack: heat-shortage, electricity-shortage - not reported
Heat Generation - reported, issues in plot script?
Gas Turbine Generation - reported, issues in plot script?

TODOs:

[x] preparation: re-structure postprocessed_path.yaml
[x] preparation: re-structure sequences_by_tech
[x] calculation of re generation timeseries
[x] report slack timeseries: heat-shortage, electricity-shortage
[x] issues with plot script: Heat Generation, Gas Turbine Generation
[ ] Avoid SettingWithCopyWarning → own issue
[x] Sort output
[ ] get_re_generation() can be dropped: get_summed_sequences() could process re_generation sequences if 'energysystem' as another technology can be safely added to prep_elements. Other functions using prep_elements need be checked for this. → more thinking needed, later
[ ] check if the workaround for subnodes in get_sequences_by_tech() can be dropped now

As a first step, I restructured postprocessed_paths.yaml because currently it will not be able to hold data for RE Generation.
https://github.com/modex-flexmex/oemo-flex/blob/663ed694668e42fb0150fb4ef8ad7dfb10e87fb6/oemoflex/postprocessed_paths.yaml#L43-L46
RE Generation has to go into this block, but this has electricity-curtialment as the source component. So a timeseries from another component cannot be appended to this block. Alternatively, there has to be a second block starting with "RE". Since these keys have to be unique, this isn't possible either.
Another, clearer structure has another set of advantages:

direction of processing is reflected (<from>: <to>)
component-oriented like other maps
complete path name instead of splitted (RE/Curtailment instead of RE: ... Curtailment: in the example)

Step 2 reformatted sequences_by_tech's DataFrame-in-dict structure to a plain DataFrame with one more column level. This allows for simpler timeseries aggregation (for each time step and across the different techs solar-pv, wind-offshore, wind-onshore) needed for calculating re generation timeseries.

Heat Generation, Gas Turbine Generation where not plotted because the output directories were "misnamed". The scripts is case-sensitive and the spelling is camelCased.

I tried to test this, but I get a key error for 'region' here

The sorting in oemoflex_scalars.csv is now different from former runs, but better reproducible. Sorting in Scalars.csv is not affected - can still be used for integrity checks.

The workaround for components with sub-nodes (ReservoirWithPump, Bev) at
https://github.com/modex-flexmex/oemo-flex/blob/5f34c7b290deb8198d55b6068ce0e6af68c7bbf7/oemoflex/postprocessing.py#L327-L336 is not needed anymore. Naming the sequences is done different now (using component.carrier and component.tech, i.e. without looking at the subnodes' names in component.label) but with the same result (<carrier>_<tech>). String operations can be limited to this line:
https://github.com/modex-flexmex/oemo-flex/blob/5f34c7b290deb8198d55b6068ce0e6af68c7bbf7/oemoflex/postprocessing.py#L325
This is not true for the region. Sub-nodes don't have a region attribute. So, we cannot get the region from component.region or bus.region for every sub-node (sometimes bus is a sub-node too!). That is why we still need string operations here:
https://github.com/modex-flexmex/oemo-flex/blob/5f34c7b290deb8198d55b6068ce0e6af68c7bbf7/oemoflex/postprocessing.py#L338

I tried to test this, but I get a key error for 'region' here.

To be on the safe side, I tried a whole run through every step in UC 5 without any problems. Could you give more information?

To be on the safe side, I tried a whole run through every step in UC 5 - without any problems. Could you give more information?

Still fails here. The KeyError occurs in aggregate_re_generation_timeseries:
https://github.com/modex-flexmex/oemo-flex/blob/d4bfe9c787fe1802dc619d5f839a2680422afdcf/oemoflex/postprocessing.py#L1030

When running all usecases with this branch, I do not get results for 2d and 10. Scalars.csv remains empty.

Tested against dev (with some adaptations there to have the same sorting) with no deviations in Scalars.csv and oemoflex_scalars.csv.


Response code range not supported

Thanks for adding check of response code!
Response codes may contain a range of code in the format XXX
https://swagger.io/specification/#responsesObject
My output
        if errors:
>           raise AssertionError(*errors)
E           AssertionError: Received a response with a status code, which is not defined in the schema: 404
E
E           Declared status codes: 4XX, 200

You're welcome :)
Wow, cool - I didn't know about this feature - will implement it soon

Released in 0.20.1 - https://pypi.org/project/schemathesis/0.20.1/

Still not working
The common problem in your code:
In my tests, I use a fastapi client based on an older version requests.  In this version response.status_code may have a textual representation of the error. For example:
response.status_code == HTTPStatus.BAD_REQUEST

You should use int status code, not str
def status_code_conformance(response: GenericResponse, case: "Case") -> None:
    responses = case.endpoint.definition.get("responses", {})
    # "default" can be used as the default response object for all HTTP codes that are not covered individually
    if "default" in responses:
        return
    allowed_response_statuses = list(map(int, _expand_responses(responses)))
    if int(response.status_code) not in allowed_response_statuses:
        message = (
            f"Received a response with a status code, which is not defined in the schema: "
            f"{response.status_code}\n\nDeclared status codes: {', '.join(allowed_response_statuses)}"
        )
        raise AssertionError(message)

In the current implementation they both casted to string - https://github.com/kiwicom/schemathesis/blob/master/src/schemathesis/checks.py#L28
Since they should be strings on the open api spec side anyway

However let me check that client you mentioned

Do you still get the same error with the new version of schemathesis?

It would be nice to know what version of fastapi / requests you're using and some minimal code to reproduce the error - could you, please, add it? :)
I didn't find any version that has status_code as text representation - maybe I am looking in a wrong direction

Btw, IntEnum is comparable with ints, that's why the comparison is true in your example (I assume it is)
>>> 400 == HTTPStatus.BAD_REQUEST
True

So, it is not a textual representation as far as I can see

Still, the error is in your code. You convert a response.status_code to a string, but you need convert to int
>>> 400 == str(HTTPStatus.BAD_REQUEST)

False

Aha, now I understand the issue. Would you mind providing requests / fastapi versions that have HTTPStatus instances in status_code attribute?

This one should fix it - #334
However, it will be nice to have a test case that will cover your case - with package versions I will try to add them

Pls, check if release 0.20.3 works for you

@MissiaL
Feel free to reopen the issue if the problem will occur again


Typo in "VS Code Extension" section in the docs

Under the subheading "Debugging Tasks":
"Note that when debugging a task, the Insepct extension will automatically limit the eval to a single sample (--limit 1 on the command line). "
Correction:
"Insepct" should be spelled as "Inspect"

Thanks! This is fixed in this PR which will be merged soon: https://github.com/UKGovernmentBEIS/inspect_ai/pull/262


Created examples build file

Basic github actions file that simply builds MLOps.NET.Examples.sln
Resolves
Fixes #110
Description
Created a basic github actions file to build the MLOps.NET.Examples.sln

@aslotte not really my strength (Github actions). However, I have created the above and tested it in my forked repository. MLOps.NET.Examples.sln builds as expected. I suspect there are some improvements that can be made but its a foundation.
Ready for review.

This is a great contribution @Brett-Parker! If nothing else, it ensures that our examples always builds. We can continue to build on this later, e.g. doing dotnet run to ensure that we can also train the models.


Fix code scanning alert no. 1: SQL query built from user-controlled sources

Fixes https://github.com/fba223/demo-python/security/code-scanning/1
To fix the SQL injection vulnerability, we should use parameterized queries. This approach ensures that user input is properly escaped and treated as data rather than executable code.
In the provided code, we need to modify the SQL queries on lines 16 and 22 to use parameterized queries. This can be done by using placeholders (%s) in the SQL string and passing the user-provided values as parameters to the cursor.execute method.
Suggested fixes powered by Copilot Autofix. Review carefully before merging.

yets