fix: Allow custom headers and other arguments supported by FlightCall…

…Options

Allow passing FlightCallOptions as keyword args in query().
Merges default args with custom args set by the user, and then passes them into the FlightCallOptions constructor.
Only supports headers for now.
Successfully tested with iox-debug header.
TODO: verify that the server is receiving the specified trace-id header.
TODO: test InfluxDB support for other FlightCallOptions (e.g. timeout).

@Jayclifford345 As noted in the description, this needs a bit more testing. I haven't verified the trace-id yet.  I can probably use btasker's callback example to test that. I really need to setup a good grpc sniffer.... Oh, nvm...
@reidkaufmann to the rescue.

Added the check for user-provided headers before merging them.
Factored out a _merge_options() function.
Got a successful test sending an influx-debug-id header and getting a trace-id in the response:

...
TracingClientMiddlewareFactory
...
class TracingClientMiddleware(flight.ClientMiddleware):
    def __init__(self, method, callback_obj):
        self._method = method
        self.callback = callback_obj
    def call_completed(self, exception):
      print('callback: call_completed')
      if(exception):
        print(f"  ...with exception: {exception}")

    def sending_headers(self):
      print('callback: sending_headers')

    def received_headers(self, headers):
        print('callback: received_headers')
        print(headers)

def test_custom_headers():
    print('test_custom_headers')
    res = TracingClientMiddleWareFactory()
    client = InfluxDBClient3(token = DATABASE_TOKEN,
                        host = HOSTNAME,
                        org = '',
                        database=DATABASE_NAME,
                        flight_client_options={"middleware": (res,)})
    client.query('select * from home',
                        headers=[(b"influx-debug-id", b"JasTraceId")],
                        timeout=60)
    print("client attrs: ", vars(client))
    reader = client.query('''SELECT query_type, completed_duration, trace_id
                          FROM system.queries
                          WHERE issue_time >= now() - INTERVAL '1 day'
                          AND trace_id != 'None'
                          AND query_text LIKE '%select * from home%'
                        ''',
                        language='sql',
                        headers=[(b"iox-debug", b"true")],
                        mode="reader")
    print(reader.read_all().to_pandas())

test_custom_headers()

test_custom_headers
{'headers': [(b'authorization', b'Bearer <Token>'), (b'influx-debug-id', b'JasTraceId')], 'timeout': 60}
callback: sending_headers
callback: received_headers
{'date': ['Wed, 27 Sep 2023 18:53:51 GMT'], 'server': ['envoy'], 'trace-id': ['1b00b90c801c27e35dbadb212f207430'], 'x-envoy-upstream-service-time': ['3']}
callback: call_completed
client attrs:  {'_org': '', '_database': 'get-started', '_token': '<Token>', '_write_client_options': {'write_options': <influxdb_client_3.write_client.client.write_api.WriteOptions object at 0x112a76910>}, '_client': <influxdb_client_3.write_client.client.influxdb_client.InfluxDBClient object at 0x17fd87b10>, '_write_api': <influxdb_client_3.write_client.client.write_api.WriteApi object at 0x17dc6e350>, '_flight_client_options': {'middleware': (<__main__.TracingClientMiddleWareFactory object at 0x17fd878d0>,)}, '_flight_client': <pyarrow._flight.FlightClient object at 0x112a8a280>}
{'headers': [(b'authorization', b'Bearer <Token>'), (b'iox-debug', b'true')], 'timeout': 300}
callback: sending_headers
callback: received_headers
{'date': ['Wed, 27 Sep 2023 18:53:51 GMT'], 'server': ['envoy'], 'x-envoy-upstream-service-time': ['2']}
callback: call_completed
  query_type        completed_duration                          trace_id
0        sql 0 days 00:00:00.002831083  a37492c7ecbd3dda9be26d0799e2cb07
1        sql 0 days 00:00:00.003215176  ac88e5e0ae81d31ae29673e623036f71
2        sql 0 days 00:00:00.002598998  1993b5ffc8e11a2cbbca5a7cff227ddc
3        sql 0 days 00:00:00.002932482  a5e4ea567ff95e901a354dbc2a9d3835
4        sql 0 days 00:00:00.005362857  c2c81a63d781fc8b7389b537bc02daac
5        sql 0 days 00:00:00.002949846  faa4fbfa52100885c80057860d595c05

Added the check for user-provided headers before merging them.
Factored out a _merge_options() function.
Got a successful test sending an influx-debug-id header and getting a trace-id in the response:

...
TracingClientMiddlewareFactory
...
class TracingClientMiddleware(flight.ClientMiddleware):
    def __init__(self, method, callback_obj):
        self._method = method
        self.callback = callback_obj
    def call_completed(self, exception):
      print('callback: call_completed')
      if(exception):
        print(f"  ...with exception: {exception}")

    def sending_headers(self):
      print('callback: sending_headers')

    def received_headers(self, headers):
        print('callback: received_headers')
        print(headers)

def test_custom_headers():
    print('test_custom_headers')
    res = TracingClientMiddleWareFactory()
    client = InfluxDBClient3(token = DATABASE_TOKEN,
                        host = HOSTNAME,
                        org = '',
                        database=DATABASE_NAME,
                        flight_client_options={"middleware": (res,)})
    client.query('select * from home',
                        headers=[(b"influx-debug-id", b"JasTraceId")],
                        timeout=60)
    print("client attrs: ", vars(client))
    reader = client.query('''SELECT query_type, completed_duration, trace_id
                          FROM system.queries
                          WHERE issue_time >= now() - INTERVAL '1 day'
                          AND trace_id != 'None'
                          AND query_text LIKE '%select * from home%'
                        ''',
                        language='sql',
                        headers=[(b"iox-debug", b"true")],
                        mode="reader")
    print(reader.read_all().to_pandas())

test_custom_headers()

test_custom_headers
{'headers': [(b'authorization', b'Bearer <Token>'), (b'influx-debug-id', b'JasTraceId')], 'timeout': 60}
callback: sending_headers
callback: received_headers
{'date': ['Wed, 27 Sep 2023 18:53:51 GMT'], 'server': ['envoy'], 'trace-id': ['1b00b90c801c27e35dbadb212f207430'], 'x-envoy-upstream-service-time': ['3']}
callback: call_completed
client attrs:  {'_org': '', '_database': 'get-started', '_token': '<Token>', '_write_client_options': {'write_options': <influxdb_client_3.write_client.client.write_api.WriteOptions object at 0x112a76910>}, '_client': <influxdb_client_3.write_client.client.influxdb_client.InfluxDBClient object at 0x17fd87b10>, '_write_api': <influxdb_client_3.write_client.client.write_api.WriteApi object at 0x17dc6e350>, '_flight_client_options': {'middleware': (<__main__.TracingClientMiddleWareFactory object at 0x17fd878d0>,)}, '_flight_client': <pyarrow._flight.FlightClient object at 0x112a8a280>}
{'headers': [(b'authorization', b'Bearer <Token>'), (b'iox-debug', b'true')], 'timeout': 300}
callback: sending_headers
callback: received_headers
{'date': ['Wed, 27 Sep 2023 18:53:51 GMT'], 'server': ['envoy'], 'x-envoy-upstream-service-time': ['2']}
callback: call_completed
  query_type        completed_duration                          trace_id
0        sql 0 days 00:00:00.002831083  a37492c7ecbd3dda9be26d0799e2cb07
1        sql 0 days 00:00:00.003215176  ac88e5e0ae81d31ae29673e623036f71
2        sql 0 days 00:00:00.002598998  1993b5ffc8e11a2cbbca5a7cff227ddc
3        sql 0 days 00:00:00.002932482  a5e4ea567ff95e901a354dbc2a9d3835
4        sql 0 days 00:00:00.005362857  c2c81a63d781fc8b7389b537bc02daac
5        sql 0 days 00:00:00.002949846  faa4fbfa52100885c80057860d595c05

@Jayclifford345 can you review this again?


Package maintenance

[x] tests and linter show no problems (npm t)
[ ] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Landed in https://github.com/metarhia/metacalc/commit/df930368827244366554b93f052928d328d4ad3e and https://github.com/metarhia/metacalc/commit/21a152cbe0c83fcc346acd6964380d249a37f453


Add Brandon Gioggia to Contributors list

@bgioggia, would you like to reopen this pull request so that I can merge it to this pull request?


Nuget for CacheManager.SystemRuntimeCaching not targeting netstandard2.0

I was looking at a build warning I got when building against .NET Core claiming CacheManager.SystemRuntimeCaching was restored using .NET Framework.
It seems as the csproj for CacheManager.SystemRuntimeCaching builds a netstandard2.0 version, but it does not appear to have been published to the NuGet feed? Or am I missing something?

It seems the last publish of these packages into NuGet was in late 2018.
@MichaCo Does this mean the library is no longer being actively maintained?
@pergardebrink for .net core why not use CacheManager.Microsoft.Extensions.Caching.Memory?

The beta packe supports netstandard 2.0.
If you really have to use runtime caching, just use that for now, or use any other memory cache.
Regarding releases, there hasn't been any release lately because I did not have any time to add new features, hence no need to release anything.

Can this be moved to a non-beta release at least? We're facing the same issue as above.

Yeah when I find some time this year, there will be a release. I got some vacation this month, lets see ;)

Any update on this please?


Support for setting the owner after closing the new file

What changes are proposed in this pull request?
We have found that when writing files to Alluxio via FUSE with write type set to CACHE_THROUGH or THROUGH, it can lead to situations where the owner and group of the Alluxio and UFS files differ.
See alluxio.fuse.AlluxioFuseUtils#createFile:

When setting the owner, the file is not yet created in the UFS, so this code snippet only applies to the file on Alluxio. The file on UFS will be created later, and the change in owner will not take effect.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs NO
addition or removal of property keys YES
webui NO

Meanwhile, @humengyu2012 could you fix the failed tests?

Meanwhile, @humengyu2012 could you fix the failed tests?

It seems that the test failed while downloading the JAR from Maven, which is unrelated to the code I submitted. The success of the test is not important. This PR may not be good enough because I just wanted to discuss whether we need to set the file's owner and group again after the file is written to ensure consistency between UFS and Alluxio permissions.

note this is going to master-2.x instead of main


Add a save-artifiacts script to the s2i build

This would allow developers to use the --incremental flag when building their application image. The result would be that the ./node_modules directory is preserved between builds so that that phase of the assembly doesn't have to get repeated for every build if not needed.

I'm doing this and is working:
save-artifacts:
#!/bin/bash

tar -C /opt/app-root/src -cf - node_modules

assemble:
if [ "$(ls /tmp/artifacts 2>/dev/null)" ]; then
  echo "---> Restore things..."
  mv /tmp/artifacts/* ./
fi

https://github.com/bucharest-gold/origin-s2i-nodejs/commit/0c500a517bfbef3fbe2c4bbc8baa48a3d043b330


Added Panthera

Q
A

Bug fix?
No

New feature?
Yes

Needs wipe?
No

Fixed issues
N/A

Description:
Adds a Liberation mission.sqm for the map Panthera
Content:

[x] Panthera mission.sqm

Successfully tested on:

[x] Local MP
[x] Dedicated MP

Thanks for your contribution.
I'll have a look at the mission.sqm during the weekend, as I've not enough time to do that in enough detail tomorrow.
In general I would, after possible needed adjustments, add it as a basefile, but I wouldn't publish it on the workshop by myself, as that would need me to maintain it additionally to all the other maps we've published. So the general idea would be that we merge it (if it gets a positive review) and with release of 0.96.5, I'll hand you over a pbo with the mission, which you can publish and maintain on the workshop yourself.
At least that's my current thought. We could also have some talk about this during the weekend in the contributors channel on Discord.

So the general idea would be that we merge it (if it gets a positive review) and with release of 0.96.5, I'll hand you over a pbo with the mission, which you can publish and maintain on the workshop yourself.

That should be fine, though updates to the new framework may be slow/late from time to time.

Had a brief look at the mission.sqm

Remove the Liberty and use the Freedom like in the other legacy framework maps
Remove all unnecessary "immersive" clutter stuff at the placed bases/locations for performance reasons. Have a look at the other official maps with placed bases. It's basically buildings/walls and nothing else. No crates, garbage, furniture, etc.
Make sure it has the same attributes like all the other mission files concerning strings, eden settings, etc.

Remove all unnecessary "immersive" clutter stuff at the placed bases/locations for performance reasons. Have a look at the other official maps with placed bases. It's basically buildings/walls and nothing else. No crates, garbage, furniture, etc.

Damn, alright will do when I can. The rest should be fairly straight forward to fix as well.

@Wyqer Actually, would it be ok to switch them to simple objects for some of them instead of outright removing all of them? I would like to keep some of them in like the helipad areas and what not so it isn't just an open square.

It's our convention to keep it as simple as possible, so removal would be the way to go. Best you orientate on Altis or Chernarus for a feeling of what could stay and how "empty" is should be.
I can understand that you want to keep it "nice and immersive", I've done the same at the beginning.
But the players wouldn't really notice it at an enemy base. Especially they wouldn't notice "missing immersive clutter stuff" in the end, when fighting against tanks and infantry in the area.

There are still many grass cutters / hide terrain modules and I don't know if they're really necessary.

Most of those are from one specific sector on the North West island. There is an abandoned factory sitting out in the middle of a field. I turned it into a proper factory complex but had to remove the grass and wheat/trees/rocks. The grass cutters are pretty self explanatory but the wheat, trees, and rocks had to be removed by hide terrain modules.
TL;DR Yes they're necessary unless you want me to remove an entire factory sector which please don't it took me like a solid hour to get that complex into a way that looked nice

Medic buildings not declared as ACE medical facility.
I assume that's as simple as dropping a line of code in the init of the building on the Freedom?

For context, here is some pictures of the complex I made:
https://i.imgur.com/YFR2msS.jpg
https://i.imgur.com/KHHuEkK.jpg
rather proud of it tbh, it looks quite nice imo. AI even respects the fencing around it and goes through the gate which I wasn't expecting when I built it if I'm honest.

Yeah the factory looks good but there's quite much stuff around it.
I can't assess if it'll affect the performance.

Yeah the factory looks good but there's quite much stuff around it.
I can't assess if it'll affect the performance.

It really shouldn't. If someones already having bad performance in that area removing those objects won't do a whole lot. The only performance issues people might have is number of AI but I've already addressed a lot of that from the original version by removing sectors in places that were spawning too many AI before the limiter could kick in.
I know it's not meaning as much from me cause I built it so of course it runs fine for me but it's 9 buildings (not counting ones there by default), some lights, fencing, and runway texture objects. Other than that everything in there is hide terrain modules and grass cutters which if anything help performance rather than hinder it.

Looks fine so far.
You just missing the ACE medical facility attributes for the medical buildings in your placed base Medical Complex layer.
this setVariable ["ace_medical_isMedicalFacility", true, true];

needs to be in their init field.
 Or asked in another way: Can you really ensure you've catched anything which should be ignored?
After that it should be fine and would get my approval. I guess I can also publish and maintain it directly on the workshop to avoid mission file sending to you, so that you update, etc.

You just missing the ACE medical facility attributes for the medical buildings in your placed base Medical Complex layer.
this setVariable ["ace_medical_isMedicalFacility", true, true];

needs to be in their init field.

Can do, will make another PR tonight with that fixed.

 Or asked in another way: Can you really ensure you've catched anything which should be ignored?

Nope. When I initially made this for my group I was unaware of how the ignore lists worked and didn't make one, for this I copied the Sahrani one because they have similar building and vegetation usage for the most part at the suggestion of @veteran29
It should catch a lot of them as Panthera does use CUP buildings, but there may be some missing.

It should catch a lot of them as Panthera does use CUP buildings, but there may be some missing.

Please recheck it once and make sure it's covering everything basically. 👍

Please recheck it once and make sure it's covering everything basically.

Everything should be caught now as of KPLP5


dependencies

It takes ages to install the Harmonization package via pip. This is probably due to the lack of version specifications of the modules in the requirements.txt file. Pip needs to resolve everything by itself. This can at times be quite time-consuming. Unfortunately, I had to remove the Harmonization package from thingsvision because it takes too long to install including that dependency. I'll include the package again until both issues are resolved.

Update: This is only the case on macos. On ubuntu it seems to work fine!


Support to  constructor using ProvidesKey

uberfire
(master) http://github.com/uberfire/uberfire/commit/86586bc53

Thanks @cristianonicolai


Update roadmap page

amended the 'last updated' date
removed releases 4.1.0 and 4.2.0 from 'Recently shipped' as over 6 months ago
removed Confirm a phone number from 'Recently shipped' as over 6 months ago
added "hosted Design System Day 2022" to 'Recently shipped'
added "publish an accessibility strategy for the GOV.UK Design System" to 'Working on now'

Squashed down to 1 commit (and updated the date to today, again)


FCS_CKM.1/SK - Not CC Compliant

Language should be changed to:
The TSF shall generate symmetric cryptographic keys in accordance with a
specified cryptographic key generation algorithm [assignment:
cryptographic key generation algorithm] and specified cryptographic key
sizes [assignment: cryptographic key sizes] that meet the following:
[assignment: list of standards].
Column headers of table to be adjusted appropriately.

Done


Feature Parity

Create

[x] #2
[ ] #3
[x] #30
[ ] #4
[ ] #5
[ ] #6
[ ] #7
[ ] #8

Read

[x] #31
[ ] #9
[x] Replies (Partial but viewable)
[x] Images (Implemented but disabled)
[x] Profile Pictures (Implemented but disabled)
[x] Embeds
[ ] #10
[ ] #11
[x] Channels
[ ] #12
[ ] #13
[ ] #14
[x] DMs
[ ] #15
[ ] #16
[x] Categories

Update

[x] Read Indicators TODO: Test
[ ] #17
[x] Reactions
[ ] #18
[ ] #19
[ ] #20

Delete

[ ] #21
[x] Reactions

Live Update

[x] Websocket thread/task
[ ] #22
[ ] #23
[ ] #24

UI/Polish

[x] Proper Image Storage
[ ] #25
[ ] #26
[x] Secrets Saving/management
[x] Settings
[ ] #27
[ ] #28
[ ] #29
[x] Graceful Error Handling
[ ] Graceful Panic Handling
[x] App Name
[ ] App Icon
[x] Window Config

prontus


Logger is adding empty object

Logger is sometime adding messages in the form:
<date>[app] message: {}
when they should be of the form:
<date>[app] message

Thanks for reporting this issue Robin. We really appreciate your help.


matrix buildx by tool.

Now testing with poc branch, updating shortly.

poc run is here: https://github.com/anarkiwi/network-tools/actions/runs/1115245417

Updated poc run is here: https://github.com/anarkiwi/network-tools/runs/3296888193?check_suite_focus=true


Deploy to production failed!

I got an error after adding public_activity to Gemfile and deploy to production:
ENOENT: no such file or directory, stat '/Users/h/layer/ruby/gems/2.5.0/gems/public_activity-1.6.4/test/migrations/001_create_activities.rb
Everything is ok if removed public_activity. So I've checked migration file and found the file is a link instead of actual file.
Any idea how to fix this issue? Thanks

What version are you running?

v1.6.4

same here. (v1.6.4)
https://github.com/chaps-io/public_activity/blob/1-6-stable/test/migrations/001_create_activities.rb
../../lib/generators/public_activity/migration/templates/migration.rb

Is this symbolic link file broken?
In my local environment:
$ readlink /usr/local/rbenv/versions/2.6.2/lib/ruby/gems/2.6.0/gems/public_activity-1.6.4/test/migrations/001_create_activities.rb
test/migrations/../../lib/generators/public_activity/migration/templates/migration.rb
$ cat /usr/local/rbenv/versions/2.6.2/lib/ruby/gems/2.6.0/gems/public_activity-1.6.4/test/migrations/001_create_activities.rb
cat: /usr/local/rbenv/versions/2.6.2/lib/ruby/gems/2.6.0/gems/public_activity-1.6.4/test/migrations/001_create_activities.rb: No such file or directory
$

Mark as resolved, s. https://github.com/public-activity/public_activity/issues/346


Still feel like it needs some work but I think the changes in there make some things flow a little better.
Also updated the tests to account for tags having the correct parents so people don't do something crazy like:

<title>Jon</title>

Hello
Hello, Pluralsight!</hi>

Hi

Gonna merge this with reckless abandon.


Add uppload token to Github Actions to make sure codecov uploads are successful

https://docs.codecov.com/docs/codecov-uploader#upload-token

example on how to do this:
https://github.com/ztcjoe93/tatwritescode/commit/3ac9e4b500e99a3a742ecaac99b10b7bdc68e1ff


Feat(compiler_base):add codecov for compiler_base in CI.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

issue #115
2. What is the scope of this PR (e.g. component or file name):
compiler_base
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

add codecov for compiler_base in CI.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
749 unchanged lines in 7 files lost coverage.
Overall coverage increased (+1.8%) to 60.809%

Files with Coverage Reduction
New Missed Lines
%

kclvm/parser/src/lexer/indent.rs
4
78.31%

kclvm/sema/src/resolver/scope.rs
5
61.67%

kclvm/sema/src/resolver/node.rs
9
69.25%

kclvm/sema/src/resolver/schema.rs
27
80.99%

kclvm/parser/src/parser/stmt.rs
128
88.78%

kclvm/parser/src/parser/expr.rs
282
81.77%

kclvm/sema/src/resolver/global.rs
294
61.42%

Totals

Change from base Build<PHONE_NUMBER>:
1.8%

Covered Lines:
22692

Relevant Lines:
37317

💛  - Coveralls

Consider a new single github workflow to fully test compiler_base with OS matrixes including ubuntu, macos, windows, etc.


allow setting install prefix for jdks, create junction point / symlink for multiple LTS versions

Is your feature request related to a problem? Please describe.
Whenever a jdk is updated, it breaks the path for those apps that need a particular LTS version. If you have general apps JAVA_HOME set for jdk 11 LTS which seems to work for most things. But then when you want more modern LTS 17 for things ( even very old things that use JDK 8) that want to support a particular LTS the path breaks.
Also, the updater always defaults to the msi installer's notion of a default location, usually somewhere on C: which if you have a PC or laptop with C: SSD smaller disk and D: larger spinning disk, it's usually a battle to keep C: less full, as Windows OS and Programs insist on using C: ( Windows Docker Desktop ) to install.
Also OCD people might like everything in D:\Java\ or some such convention.
I'm sure that it's not a 'known-not-a-bug'
[X] Yes
Describe the solution you'd like
It'd be nice to specify a filesystem prefix where new jdks are installed by update watcher, when the msi's are run you could specify the prefix for the installer, and default to prefix baked into the .msi installer if there was no default prefix.
Also it'd be nice if it could create a junction point / symbolic link for each major/LTS version. This used to be a feature in the old oracle installer ( but it ignored LTS's only did one version ) to make a junction point, and handle the modern multi LTS world and not just one latest version. There is another LTS coming soon, at the end of the year. Already I can see use case for java only supporting 8 or 11, while the most recent version is 17. The release LTS 21 might make some things only supporting 17 ?
If javas installed in default prefix D:\Java, say there is D:\Java\jdk-<IP_ADDRESS>-hotspot a windows junction point could be created ( D:\Java\jdk-17 -> D:\Java\jdk-<IP_ADDRESS>-hotspot ) and programs than wanted LTS 17 specifically, rather than the default version in JAVA_HOME it could point at D:\Java\jdk-17, then not break when the java updates ( the update watcher would create/update the junction point when installing a new version )
Describe alternatives you've considered
A simple option might be to set a custom java home, eg LEGACY_JAVA_HOME for jdk versions 8 (instead of JAVA_HOME for eg. the jdk8* versions)for compiling older code or test cases, and the usual JAVA_HOME for most things.
But that would be less ideal, as there is less opportunity to use custom java home, it would require developer or shell script etc., prior to app running and most people who are app users may not be able to do this, and JAVA_HOME is universally supported for most apps.
Additional context
Add any other context or screenshots about the feature request here.

Hello, thank you for your suggestions.
As I see it, you're asking about two features:

The first one is a consistency in installation paths. Currently there are two types of Java installations here: auto-discovered and manually-managed ones. I can add an option to keep the installation directory unchanged for manually-managed ones. So, if you have Javas in D:\Java\JRE11 and D:\Java\JDK17 you would simply convert them to manually-managed ones, and set a flag to keep the install dir.

I will implement it in one of the future releases.

As for automatic management of symlinks+, it's an extremely messy-and-hard-to-implement solution - because there are no existing standards.

If you find this app useful, stars are appreciated :) GitHub stars
how can i give you a star?


Cannot copy quotation in mail

When copy text that contains nested quotation I cannot copy those information. This problem is related to #3207 and #3131. Alternatively, you can solve this issue, if you allow the user to display the ">", ">>" as text.
Expected behavior
When copy all text in a plain text email, options paste should return the full email text including the ">" , ">>" characters.
Actual behavior
"Paste as text" does not contain any ">" characters. This can be seen as a feature.  Paste however show only a single bar where all the conversation should be.
Steps to reproduce

select all mail or part containing quotations (better with second level: ">" and ">>")
copy
paste in any editor, e.g., droidEdit or in new k9 mail.

Environment
K-9 Mail version: 5.403
Android version: 8.0.0
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

That's not something we can fix easily. We're using Android's WebView component to display the message and for that convert text/plain parts to HTML if necessary.
You could use the "Share" functionality together with "Copy to clipboard" from the Google Drive app (or some other app that provides this functionality). That will copy the text/plain part including the quote characters (>) to the clipboard.

Hi cketti,
thanks for feedback. I can only choose share or copy when I select all text in the messgae view (WebView ?) But than share with Google drive or copy to clipboard does not include the ">" characters. Tried share with google disk, droidedit. I cpoief with system app, "clipboard actions”, "clip stack" and "native clipboard". There is only the text in the clip. Maybe different behaviour in android 8? Thanks and best regards, Adam

Use the "Share" menu option in K-9 Mail's "Send…" sub menu (two arrows pointing left, if displayed as icon).

OK, thx. So this will lead to empty text for content type application/pgp-encrypted messages as reported in #3131. So there is no real workaround but wait for #3131 ;). Thanks and best regards, Adam


Mixing shell/docker executor jobs, doesn't play well with new docker artifacts handling.

./ts-node --cwd examples/docker-compose-nodejs/

@bcouetil Working on a solution.

---
# @Description Install npm packages
npm-install:
  stage: .pre
  image: node:alpine
  dependencies: []
  artifacts:
    paths:
      - node_modules/
  script:
    - npm install --no-audit

# @Description Find security vulnerabilities in node_modules
npm-audit:
  stage: test
  image: node
  allow_failure: true
  needs: []
  script: npm audit

# @Description Find outdated packages in node_modules
npm-outdated:
  stage: test
  image: node
  allow_failure: true
  needs: [npm-install]
  script:
    - npx -s ncu -e2 --deprecated --packageFile package.json

# @Description Up docker-compose services
docker-compose-up:
  stage: deploy
  environment:
    url: http://localhost:8891
    name: local
  script:
    - docker-compose up -d

How does it sound to solve this by introducting a # @CopyArtifactsToHost decorator? @bcouetil
# @CopyArtifactsToHost
# @Description Install npm packages
npm-install:
  stage: .pre
  image: node:alpine
  dependencies: []
  artifacts:
    paths:
      - node_modules/
  script:
    - npm install --no-audit

You use artifacts as a cache mecanism, node modules should be cached instead (per branch if you want). Not the problem here, just saying.
You set dependencies to [ ] and expect to get artifacts ? Set your dependencies to some job, and it will work right away, no ?

Sorry no, I understand now, let me think

I would add "dependencies" in deploy job.
And "dependencies" on a shell job should make the tool search artifacts in .gitlab-ci-local/artifacts/my-pid/ if any.

So just default to copy to "host", if a shell job ever needs artifacts from the .gitlab-ci-local/artifacts/ folder.
That could work.

Ah yes forgot the default behavior, so your job is correct.
Then yes, if dependencies is not [ ] on a shell jobs, get all artifacts if any !

Will take a little work. I'm considering master broken until I've fixed this.
I expect quite alot of people already "taking" advantage of having been able to rely on "host/project dir root" artifacts.

I know quite a lot of CICD Engineers and clients, and never saw shell jobs in gitlab pipeline. Docker jobs are so helpfull to have a reliable pipeline.

https://github.com/firecow/gitlab-ci-local/commit/efa2d0b338f2d5f74aaa416ce15d0791587a4b42 Its been "fixed" anyways.


Create database insert statement

After some discussion and research... Insert statements are not efficient for the amount of data we are working with. They are incredibly slow to execute taking an hour for one year of data, and this site will be dealing with decades of data. Due to this we did find a better solution to insert our data. We are using LOAD DATA INFILE command to insert our data into our mysql database. This takes a CSV file, which we are given for the data, and imports the data into our sql database. This eliminates a step to generate insert statements, and just takes the CSV and imports the data directly. Instead of possible hours, we are seeing possible minutes. We will still use a small sample of insert statements for testing purposes.


MPIJob worker still running  when NotEnoughResources with enable-gang-scheduling==true?

请问下，MPIJOB刚性调度在资源不足的情况下，显示Running状态，是Bug吗？
== MPIJOB
NAME            AGE   STATE
hvd-tf1-mnist   16m   Running
===  POD
NAME                                     READY   STATUS    RESTARTS   AGE     IP            NODE                NOMINATED NODE   READINESS GATES
hvd-tf1-mnist-launcher                   0/1     Pending   0          14m
hvd-tf1-mnist-worker-0                   1/1     Running   0          14m     <IP_ADDRESS>   openpai-212
hvd-tf1-mnist-worker-1                   0/1     Pending   0          14m
==== PodGroup Status
status:
conditions:
- lastTransitionTime: "2022-06-17T10:21:33Z"
message: '2/1 tasks in gang unschedulable: pod group is not ready, 1 Running,
3 minAvailable'
reason: NotEnoughResources
status: "True"
transitionID: cd024380-e518-43f0-9c44-3664ebb10429
type: Unschedulable
phase: Unknown
running: 1
==== MPIJOB Status
status:
conditions:
- lastTransitionTime: "2022-06-17T10:06:10Z"
lastUpdateTime: "2022-06-17T10:06:10Z"
message: MPIJob aios/hvd-tf1-mnist is created.
reason: MPIJobCreated
status: "True"
type: Created
- lastTransitionTime: "2022-06-17T10:06:11Z"
lastUpdateTime: "2022-06-17T10:06:11Z"
message: MPIJob hvd-tf1-mnist is running.
reason: JobRunning
status: "True"
type: Running
replicaStatuses:
Launcher: {}
Worker:
active: 1

training-operator version （v1.4.0）
public.ecr.aws/j1r0q0g6/training/training-operator:174e8813666951ded505daf334a37f60fd50c18d

A bit translation. MPIJob shows Status as 'Running' while not every related Pod is running.
It seems two issues come at once:

Even with enable-gang-scheduling=true, some Worker Pods get scheduled. (This seems a gang-scheduler issue instead of the training-operator one.)
With partial Worker Pods in Pending status, the MPIJob appeared in Running status. (If reproduced, this is an operator issue.)

I shall try to reproduce the issue and fix it.

@zw0610 是的，确实存在2个问题

补充一些日志：
MPIJob启动8个worker，每个worker一个GPU，空闲4个GPU
== MPIJOB
NAME                              AGE    STATE
tjob-horovod-test-demo-12-1-0-9   4m4s   Running
=== POD
tjob-horovod-test-demo-12-1-0-9-launcher            0/1     Init:0/1    0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-0            1/1     Running     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-1            1/1     Running     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-2            1/1     Running     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-3            1/1     Running     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-4            0/1     Pending     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-5            0/1     Pending     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-6            0/1     Pending     0          4m28s
tjob-horovod-test-demo-12-1-0-9-worker-7            0/1     Pending     0          4m28s
== MPIJOB status
status:
conditions:
- lastTransitionTime: "2022-06-22T03:16:24Z"
lastUpdateTime: "2022-06-22T03:16:24Z"
message: MPIJob aios/tjob-horovod-test-demo-12-1-0-9 is created.
reason: MPIJobCreated
status: "True"
type: Created
- lastTransitionTime: "2022-06-22T03:16:26Z"
lastUpdateTime: "2022-06-22T03:16:26Z"
message: MPIJob tjob-horovod-test-demo-12-1-0-9 is running.
reason: JobRunning
status: "True"
type: Running
replicaStatuses:
Launcher: {}
Worker:
active: 4
=== PodGroup status
status:
conditions:
- lastTransitionTime: "2022-06-22T03:21:14Z"
message: '4/5 tasks in gang unschedulable: pod group is not ready, 1 Bound,
4 Running, 9 minAvailable'
reason: NotEnoughResources
status: "True"
transitionID: d3456190-5eee-4254-86a4-94f477fb5fdc
type: Unschedulable
phase: Unknown
running: 4

/assign @hackerboy01
/assign @Garrybest


Can't Run on Debian 9

Hi, since updating to Debian Stretch from Jessie, I have been unable to get Nadeko to run. Initially it seemed to be related to a dotnet issue, but restoring the libcurl3 version from Jessie's repository seemed to fix that. Unfortunately I'm now absolutely stumped on this error. I have no idea how to alter ImageSharp, or even what it is. Unfortunately the window has taken to crashing whenever I highlight the text, so this screenshot will have to suffice.

Any help is greatly appreciated.

Is that 1.3 version Nadeko?

Yea, you just have to update.

Does the launcher not update it to the latest version anymore? I had it download the latest stable version multiple times. And if not, how can I transfer to this newer version without losing my settings, custom commands, etc.? Thanks in advance.

Okay I can confirm that re-downloading it works fine. I'll just have to re-create all my custom reactions and what not. Thanks!

just copy the data/NadekoBot.db over

Great, thanks!


Uncaught ReferenceError: Deno is not defined

when I bundle some files it includes  :
new Deno.errors.AlreadyExists("dest already exists.");

and browser does not support deno

Hi! Thank you for filing the issue.
As you have experienced, deno_emit is meant to be used to bundle code that will run with Deno. If you want to bundle code for the browser, I suggest you use something like esbuild,  vite, turbopack, parcel (or others; there are countless options).
You can close the issue.


ide-helper:models -I --ignore is not working

Generating the docblocks for the models does not work in my case. It's trying to generate the docblock for a model where I define a specific __construct, so I tryied adding the model to the --ignore list, but it always tries to include that model. This is how I'm calling the command:
 php artisan ide-helper:models -D app/ -N  --ignore=app/Library/Reports/CRWordReport.php

I think it should allow not only to ignore a specific model but also to ignore a full dir, but that's another issue.

You need to provide the class name not the filename
@barryvdh issue can be closed :}

Wow, yea, it took a while I ended changing IDE, latest Laravel does not use facades, nor magic classes.  But thank you :)


feat: create components ttoss

Task linked: TASK-1358 Criar os components no @ttoss/forms

Task linked: TASK-1358 Criar os components no @ttoss/forms

Task linked: TASK-1358 Criar os components no @ttoss/forms

Task linked: TASK-1358 Criar os components no @ttoss/forms


bug(Select): the position for select list is not correctly calculated if set a constant width for .cdk-overlay-pane css class

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
This bug didn't appear in version 14.
The main problem is that the width is calculated by the width of the mat-form-field and not by the content that is presented in the select list.
an example of how it looks in the project.

Reproduction
Steps to reproduce:

set some default width for  .cdk-overlay-pane CSS class and place mat-select on right side of screen

Expected Behavior
Would be cool be fixed it up with an additional API or dynamically by JS.
Actual Behavior
It cuts off mat-options with additional HTML elements and it looks buggy.
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

I think that this is the same root cause as https://github.com/angular/components/issues/26000 where we have the dropdown matching the width of the form field.


Silent Authorization Code Flow instead of Refresh

We have an interesting requirement in our company for extending the Login time.
In our case it is forbidden to get new access_tokens by refresh token.
Instead we have to use a full authorization code flow every time nearly the expiry of the token.
It is garanteed, that the auth code flow is complete transparent.
So we have to do the flow in "silent mode" using an iframe.
That's why we are asking/searching for a solution in your lib.
Alternativly we will do a fork and extend the lib with our implementation.
kind regards
Holger

Hi,
I have a similar issue. For security issue, our IdP just removed the refresh token generation for the authorization code flow with PKCE.
We now have to generate a new token trough a full authorization code flow.
@faehne have you reached to add some custom code to do the flow in "silent mode" using an iframe?
Thanks

Im not sure... but it should be possible to use the setupSilentRefreshEventListener (via silentRefresh) and also setup to code flow. Im not able to test this right now. I will write comment if i figured out ;)
An alternative would be the popup feature:
https://github.com/manfredsteyer/angular-oauth2-oidc/issues/1382
  /**
   * This method exists for backwards compatibility.
   * {@link OAuthService#initLoginFlowInPopup} handles both code
   * and implicit flows.
   */
  public initImplicitFlowInPopup...


GRPC server not being traced.

I have the trace integration defined for two NodeJS servers, written in Typescript. One is an Express server, one is a GRPC server.
Both have the following setup:

Running inside GKE in a cluster that has the trace API enabled.
Node 8.9.4
env variable GCLOUD_TRACE_NEW_CONTEXT=1
"@google-cloud/trace-agent": "2.8.1",
"google-protobuf": "3.5.0",

The GRPC server uses:

"grpc": "1.10.1",

I'm using the statically compiled implementation for my protos.
The Express server creates traces successfully for received requests, the GRPC one doesn't.
GRPC server code:
import * as _tracing from '@google-cloud/trace-agent'

_tracing.start({
  serviceContext: {
    service: 'content-service',
  },
})

import { ContentService, IContentServer } from 'lib/generated/content_service_grpc_pb'
import grpc, { Metadata, ServerUnaryCall, sendUnaryData } from 'grpc'

class Service implements IContentServer {
  // (Definition of service handlers)
}

const server = new grpc.Server()
server.addService(ContentService, new Service())
server.bind(`<IP_ADDRESS>:${port}`, grpc.ServerCredentials.createInsecure())
server.start()

Testing a simple gRPC server I'm able to see traces in the Cloud Platform UI. Could you try starting with warning-level logs and immediate flush (via bufferSize = 1)? E.g.
_tracing.start({
  serviceContext: {
    service: 'content-service',
  },
  logLevel: 1, // warnings
  bufferSize: 1
})

If that doesn't work, are there any warnings (or errors) that get printed?

Since I added those lines, traces started appearing 🤔I know that change didn't cause that to happen, so I'm going to put this down to PEBKAC for now. Thanks for checking it out.


Jason's mask sprite

Added Jason's hokey mask from movie Friday 13th. #182

Wow. Jaws dropped.
Are you doing pixel art professionally by any chance? :smile:

Well this is embarrassing, I'm actually a computer engineer. I do know my way
a bit around photoshop and I've read a lot on tutorials on pixel art, but these are the first drawings I post anywhere 😅
I saw your project and just thought it would be simpler to learn the ropes of git by dealing with images instead of code, and that it'd be a great way of practising my pixel art skills all in one shot.
I'm very glad you like my sprites so much that you'd suggest that! 😀

I totally had the impression that you had at least a few thousand hours worth of practice. :smile: In that case, you are very talented. ^^
I have also been reading a few pixel art tutorials. Do you have any that you would strongly recommend?

Maybe you know it already, but I have learned a lot from the animated tutorials by Pedro Medeiros that can be found on his Patreon (all tutorials are free for everyone to see despite being on there).
The coolest part is that they are all animated gifs, which makes it easier to understand and visualize some concepts. He also has great examples on how to make sprite animations!

Oh boy, when I first stumbled upon that series it was already like 1 AM. I hated myself the next day at work. But now that you have reminded me, I guess I will go through them again. :smiley:


Add BlockDataContainer geometry and pnorm fixes

Changes

Adds geometry property to BlockDataContainer.
Adds __eq__ to BlockGeometry
Merges BlockGeometry.py and BlockDataContainer.py into BlockFramwork.py to avoid circular dependency

Testing you performed

Adds unit tests for

pnorm(1),
pnorm(2),
allocate,
__eq__
geometry property with BlockDataContainer containing also a simple DataContainer, i.e. the property should return None.

Related issues/links

closes #474
closes #473
closes #1445
closes #1525
closes #1798

Checklist

[x] I have performed a self-review of my code
[x] I have added docstrings in line with the guidance in the developer guide
[x] I have updated the relevant documentation
[x] I have implemented unit tests that cover any new or modified functionality
[x] CHANGELOG.md has been updated with any functionality change
[x] Request review from all relevant developers
[x] Change pull request label to 'Waiting for review'

Contribution Notes
Please read and adhere to the developer guide and local patterns and conventions.

[x] The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CIL (the Work) under the terms and conditions of the Apache-2.0 License
[x] I confirm that the contribution does not violate any intellectual property rights of third parties

@casperdcl since I renamed and merged 2 files, do you think this is backward compatible?
Looking at the __init__.py I redefined the lines
from .BlockGeometry import BlockGeometry
# as
from .BlockFramework import BlockGeometry

Is this sufficient?


Include Statement

Just found this through Googling something and started using it. For some reason, when I installed through composter, the dir structure was ../vendor/easygit/easygit/EasyGit.php rather than what's in the README file. Might be just how my system handled it...not sure. Thought I'd mention it ;)

Merged your PR :) Can we close the issue?

Done :)


A11y_MySQLextensionforAzureDataStudio_New deployment Azure Database for MySQL_Name Role Value:"Learn more" link is not descriptive in Deployment Pre-Requisites page.

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #GH-MySQLextensionforAzureDataStudio-Win32-Mar23; #MySQL extension for Azure Data Studio; #Win32; #WCAG4.1.2; #Name Role Value; #Screen Reader;  #Benchmark; #AILimited;  #NVDA; #JAWS;
"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver."
Environment and OS details:
Azure Data Studio Version: 1.42.0 (user setup)
Windows 11 Enterprise Version	22H2
Installed on	28-12-2022
OS build	22621.1413
Experience	Windows Feature Experience Pack 1000.22639.1000.0
Pre-Requisite:

Install Azure data studio From here https://learn.microsoft.com/en-us/sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16&tabs=redhat-install%2Credhat-uninstall
Once installation done , Install MySQL Extension in Azure data studio
Tab Till  new connection and press enter
Enter a valid server name username and password and press connect

Repro Steps:

Launch Azure Data Studio.
Select the Connections icon (if not already selected) from left bar.
Click on the ellipses '...' and select "New Deployment"
In the Deployment options, select "Azure Database For MySQL" card.
Tab till Microsoft azure terms of use and Microsoft Privacy Statement
Verify that Learn more Microsoft azure terms of use and Microsoft Privacy Statement  defined as link or not

Actual Result:
"Learn more" link is not descriptive in Deployment Pre-Requisites page.
Notes:
Similar issue repro on dashboard page also
Expected Result:
The "Learn more" link should be informative and relevant to the page that is opening.
Attachment:

Notes :

This issue is external to MySQL extension as it is a common feature of Azure Data Studio.

This bug has two issue
Title issue :
"Learn more" link is not descriptive in Deployment Pre-Requisites page. This by Design as per  this bughttps://github.com/microsoft/azuredatastudio/issues/15807
Note issue :
Learn more link in manage dashboard page is not descriptive. for note issue we have created New External bug
So i have changed the title
Tracking Bug ID :https://github.com/microsoft/azuredatastudio/issues/22656
Could You please Reopen the bug since We have follow External process for this bug We will act on this bug once external bug is fixed and closed. Please don't close the bug

#Regressed:05-15-23; #Closed
Verified issue in below environment :
Issue is fixed
Version: 1.45.0-insider (user setup)
Commit: 4329d04da3b86689556d4dceab7d9b3d5cde3f58
Date: 2023-05-15T05:30:26.887Z
VS Code: 1.70.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621


add link testing to scripts

this is low priority but, @mikedotexe, can you check this out on Windows please?
specifically

does docs/website/test-links.sh work on Windows?
do the instructions for adding docs/.git/hooks/pre-push work on Windows?

this is low priority but, @mikedotexe, can you check this out on Windows please?
specifically

does docs/website/test-links.sh work on Windows?
do the instructions for adding docs/.git/hooks/pre-push work on Windows?

Will do right now


Uniform Buffer Object for per-frame logic

Everything the main render loops communicates via uniforms right now need to go into a single UBO we can pass to shaders across all the Drawables.
This is stuff like the map scale, the matrix, etc.  But not things specific to the Layers.  They have their own UBOs.

In the end I don't think you ended up doing this, right?  It turned into a per layer UBO?

The UBOs are owned only by drawables in this moment. From the point of view how often are created or updated, there 3 types:

UBOs created each frame and different for each drawable (like matrix).
UBOs created per layer each frame and shared to all drawables of the same layer group.
UBOs created/updated per layer only when zoom is changed or other triggers and shared to all drawables in the same layer group.


Update the docs for the latest Keycloak version

Summary

Keycloak doesn't maintain docs for the older versions, so I've updated the Mattermost docs to reflect changes from the latest Keycloak version at the time v22.0.5.
Ticket Link

thank you @vish9812 ,  the changes looks good to me.


Start to use pool="dynamic_single" to ensure FIFO event processing on interruptions

Currently, on_interruption_start and on_interruption_end handlers are both running on the default app pool, they have a lock protection, but still it's vulnerable to out of order events, depending on the frequency this can happen.
These two handlers could be refactored in a single using a dynamic_single pool that will be shipped on this PR https://github.com/kytos-ng/kytos/pull/399, and then branch from here processing each start and end accordingly, these handlers don't deal wit blocking IO, it can utilize the dynamic_single:
    @listen_to('topology.interruption.*', pool="dynamic_single")
    def on_interruption(self, event: KytosEvent):
        ...

Ideally, this needs to be done before liveness start to use interruptions cc'ing @Ktmi for his information

Yeah, we can go this route. In fact it would probably help simplify the logic in a few places. I just have one question; are each of the listeners a separate thread, or does one thread from the pool execute each listener?


⚠ ClamAV has degraded performance

In 06c7dfa, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 208 ms

Resolved: ClamAV performance has improved in 9d8f1d5.


ARROW-2612: [Plasma] Fix deprecated PLASMA_DEFAULT_RELEASE_DELAY

AppVeyor passed here: https://ci.appveyor.com/project/pcmoritz/arrow/build/1.0.745


How to add Bcc to the current README.md example?

This fails:
<?php require("path/to/sendgrid-php/sendgrid-php.php");

$from = new SendGrid\Email(null, "test@example.com");
$subject = "Hello World from the SendGrid PHP Library!";
$to = new SendGrid\Email(null, "test@example.com");
$content = new SendGrid\Content("text/plain", "Hello, Email!");
$mail = new SendGrid\Mail($from, $subject, $to, $content);

$mail->addBcc('foo@bar.com');   // !! FAILURE HERE !!

$apiKey = getenv('SENDGRID_API_KEY');
$sg = new \SendGrid($apiKey);

$response = $sg->client->mail()->send()->post($mail);
echo $response->statusCode();
echo $response->headers();
echo $response->body();
?>

How to do it properly?

Hello @josephernest,
Because you are using the Mail constructor, we create an initial personalizations object for you behind the scenes. So to add the BCC, you need to do something like: https://github.com/sendgrid/sendgrid-php/blob/master/examples/helpers/mail/example.php#L16
Thanks!

@thinkingserious I don't understand: why would this $mail->personalization[0]->addTo($to); be equivalent to a regular BCC in a email header ?

Hi @josephernest,
We are trying to build an object similar to this. Since we already create that personalization object with the constructor, we need to add the BCC to it.
When you do $mail->addBcc, you are adding it to the mail object, when it really needs to go inside of the personalization object.
Here is what your code would look like:
<?php require("path/to/sendgrid-php/sendgrid-php.php");

$from = new SendGrid\Email(null, "test@example.com");
$subject = "Hello World from the SendGrid PHP Library!";
$to = new SendGrid\Email(null, "test@example.com");
$content = new SendGrid\Content("text/plain", "Hello, Email!");
$mail = new SendGrid\Mail($from, $subject, $to, $content);
$mail->personalization[0]->addBcc('foo@bar.com');

$apiKey = getenv('SENDGRID_API_KEY');
$sg = new \SendGrid($apiKey);

$response = $sg->client->mail()->send()->post($mail);
echo $response->statusCode();
echo $response->headers();
echo $response->body();
?>

But you are correct, it makes sense that you should be able to simply do $mail->addBcc('foo@bar.com');. We are going to figure out the best way to do that in the next iteration of the Mail Helper. I hope that when we create the issue to do that, you will chime in with your feedback :)
Thanks!

Thanks a lot for this! I have to confess that, after discovering this
library, and spending 2 hours digging, without finding what I needed
(simply sending an email with a bcc), I moved to another solution that is
now running on production... But maybe one day I'll come back again! I wish
you the best for your library.
On Sat, Sep 10, 2016 at 1:20 AM, Elmer Thomas<EMAIL_ADDRESS>wrote:

Hi @josephernest https://github.com/josephernest,
We are trying to build an object similar to this
https://sendgrid.com/docs/Classroom/Send/v3_Mail_Send/personalizations.html#-Sending-a-Single-Email-to-a-Single-Recipient-With-a-CC-and-a-BCC.
Since we already create that personalization object with the constructor,
we need to add the BCC to it.
When you do $mail->addBcc, you are adding it to the mail object, when it
really needs to go inside of the personalization object.
Here is what your code would look like:
personalization[0]->addBcc('foo@bar.com');$apiKey = getenv('SENDGRID_API_KEY');$sg = new \SendGrid($apiKey);$response = $sg->client->mail()->send()->post($mail);echo $response->statusCode();echo $response->headers();echo $response->body();?>
But you are correct, it makes sense that you should be able to simply do
$mail->addBcc('foo@bar.com');. We are going to figure out the best way to
do that in the next iteration of the Mail Helper. I hope that when we
create the issue to do that, you will chime in with your feedback :)
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/288#issuecomment-246066833,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF4eE20h7Is9SwdM6t0wUUU9mw4R36YZks5qoenSgaJpZM4J2N3b
.

Hi @josephernest,
Thanks for the follow up! Though I am sad that you have left us, I'm greatly appreciative for your feedback, which will help others avoid this issue in the future.
Best of luck!
With Best Regards,
Elmer

Hi @thinkingserious,
I tried your last code with:
$mail->personalization[0]->addBcc('test@example.com');

but I get this response:
400Array ( [0] => HTTP/1.1 400 Bad Request [1] => Server: nginx [2] => Date: Fri, 08 Sep 2017 17:41:50 GMT [3] => Content-Type: application/json [4] => Content-Length: 213 [5] => Connection: keep-alive [6] => X-Frame-Options: DENY [7] => Access-Control-Allow-Origin: https://sendgrid.api-docs.io [8] => Access-Control-Allow-Methods: POST [9] => Access-Control-Allow-Headers: Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [10] => Access-Control-Max-Age: 600 [11] => X-No-CORS-Reason: https://sendgrid.com/docs/Classroom/Basics/API/cors.html [12] => [13] => )

In particular this error:
 {"errors":[{"message":"Invalid type. Expected: object, given: string.","field":"personalizations.0.bcc.0","help":"http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.personalizations.bcc"}]}

What's the problem?
Thanks in advance.

There is a little mistake in your previous code @thinkingserious. Please replace:
$mail->personalization[0]->addBcc('foo@bar.com');

by
$bcc = new SendGrid\Email(null, "foo@bar.com");
$mail->personalization[0]->addBcc($bcc);

Thanks @josephernest!
Have you had a chance to look at the new SDK design? Your feedback would be greatly appreciated.

Hmmm, the implementation that actually just worked for me was:
$bcc = new SendGrid\Email("Example Name", "email@example.com");
$mail->personalization[0]->addBcc($bcc);

Hi @chrisarusso,
Thanks for the confirmation!
Do you have any thoughts on the new interface?
Thanks!
With Best Regards,
Elmer

Hello @thinkingserious
when I use the code to send to BCC addresses as you suggested ($mail->personalization[0]->addBcc($bcc);)  , I see in the email header the BCC: name added under the Subject. Also, Outlook displays the BCC in the header of the email

. This is the first time when I see the BCC added to the email header.
How can I remove this before sending the email? Please see below a snippet from the email header.
Thank you,
Flaviu
To: Email Name email@addresss
Message-ID<EMAIL_ADDRESS>Cc: Email Name email@addresss
Subject: Subject de email - trimit bcc la gmail 11:26
Bcc: Email Name email@addresss

Hello @FlaviuRadulescu,
Are you saying that you can see the bcc field when receiving the email in Outlook?
With Best Regards,
Elmer

Yes, I have attached a picture. Can you see it?

Hello @FlaviuRadulescu,
I'm not sure what the issue is here. Could you please open up a new issue so that I may add this to my backlog for further review and also possibly get some insight from our PHP community?
Thanks!
Elmer


feat:popover menu item

Describe pull-request
Added a component for the popover-menu-item.
BREAKING
This removes all native element styling previously available in the popover-menu.
Solving issue
Fixes:  -
How to test

Go to Popover Menu
Check the design of the Popover Menu and its items.

Nice one really! I do prefer this one over the styling of native elements. The only thing to add is the disabled cursor on the disabled item.

Sweet! It's an issue using cursor on element that have pointer-events: none (which is how we hinder clicks on the disabled items.) So I think we can leave this for now and fix it post 1.0?


fix the "console" option

fix the "console" option so it doesn't screw up the indexes of ispn on shutdown.
This now puts the server in background just like start_alerting, but it also immediately tails
the log file for you so it shows in the console, plus it adds a trap so when you control-c
the stop_alerting is called and the server is shutdown normally.

LGTM, merging as travis doesn't test this one


[WIP] Factor out primitives and types to GeometryOpsCore.jl

This PR creates a new GeometryOpsCore.jl package that other packages can depend on so we have a common core of types and frameworks throughout the ecosystem.
cc @rafaqz
This is still VERY WIP, we may choose to put more functionality in (I do want table support for flatten etc) or take functionality out.
Ideally, this goes out as a first draft in the next few days so it can be registered in the General registry.
This will enable the cellarea(Spherical()) idea for https://github.com/rafaqz/Rasters.jl/pull/768.

┌ GeometryOps
│  WARNING: could not import Base.nthreads into GeometryOpsCore
│  WARNING: could not import Base.@threads into GeometryOpsCore
│  WARNING: could not import Base.@spawn into GeometryOpsCore
└

is that just because of the double / triple import?

Will merge if docs build correctly now.


Bootstrap menu-item not clickable!!!

Laravel Version: 5.4
Voyager Version: 0.11.9
PHP Version: 5.6
Database Driver & Version: MySQL 5.7

Description:
When you create dropdown menu-item  in admin menu builder then after it paste another menu-item(not dropdown) its not clickable in front-end.

Fix PR #1107


npm install fails on [[ ]] conditionals on Ubuntu

This is because on Ubuntu 18, /bin/sh is Dash shell. Dash does not support the fancy [[ ]] style conditionals in the scripts section of package.json.
Description of changes:
I changed these to [ ] style conditionals, which work just as well for any shell here, including Dash.
The rest of the workflow works fine on Ubuntu 18.04. Thanks for the nice code.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Ok sure! Thanks for the PR 👍


docs: Need clearer description about triple slash command

Clear and concise description of the problem
In the vitest documentation, i think we should clearer description about triple slash command,
👇 This is the description of aaa in the documentation
To configure vitest itself, add test property in your Vite config. You'll also need to add a reference to Vitest types using a triple slash command at the top of your config file, if you are importing defineConfig from vite itself.
import { defineConfig } from 'vitest/config'

export default defineConfig({
  test: {
    // ...
  },
})

Suggested solution
should we update the doc to this ? 👇
To configure vitest itself, add test property in your Vite config. You'll also need to add a reference to Vitest types using a triple slash command at the top of your config file, if you are importing defineConfig from vite itself.
using defineConfig from vite you should follow this:
/// <reference types="vitest" />
import { defineConfig } from 'vite'

export default defineConfig({
  test: {
    // ...
  },
})

using defineConfig from vitest/config you should follow this:
import { defineConfig } from 'vitest/config'

export default defineConfig({
  test: {
    // ...
  },
})

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I couldn't get this to work following the instructions as documented. The thing that was missing from my understanding was that 'vitest' needed to be imported for the reference to work. Could we add that into the docs as well?
ie:
import 'vitest'
/// <reference types="vitest" />
import { defineConfig } from 'vite'

@meshantz I had the exact same issue. I don't know what the problem was exactly, but I nuked my lock file and reinstalled and that did the trick.
rm yarn.lock
yarn install

Note: you only either need the triple reference in your vite.config.ts (/// <reference types="vitest" />), or you can just add "vitest/globals" to your types in your tsconfig.json.
{
  "compilerOptions": {
    "types": ["vitest/globals"]
  }
}

It might have been because I had multiple versions of vite installed unintentionally (yarn why vite said I had 4.1.2 and 4.0.4 installed), but I'm not certain.


Add docs, immutable values and find_closest

@rultor merge

@rultor merge

@abdealijk OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@abdealijk Done! FYI, the full log is here (took me 7min)


How maintain scroll position?

How can I still scroll position when I load new messages?
If disabled automaticallyScrollsToMostRecentMessage scroll to up, if enabled scroll to down.
Thank you

@dim0v works perfectly even on iOS 9 ;)


Clean up PYTHONPATH and packager handling

We previously hard-coded the path to the packager, which included a Python version. This change will try to find the installation and set PYTHONPATH appropriately.

This 'else' case only occurs if we don't pass in a packager/opae installation via environment or path. It falls back to trying to use the packager and opae built by build-bsp.sh.


How to deal with NAN

Hey, thanks amazing lib.
I try to use it to denoising my signal.
But there some np.nan in my signal data, like [1, 2, 3, np.nan, 4, 5....]
coeffs = pywt.wavedec(data, "db4")
coeffs[i] = pywt.threshold(coeffs[i], threshold_value)
meta = pywt.waverec(coeffs, "db4")

The output will contain more np.nan.
Is there any methods to reduce this impact, i.e.  add no np.nan into the signal?
(E.g. methods to process the raw signal or the reconstructed signal)

There is no built-in functionality in PyWavelets.  In general this is a hard problem, so the choice and implementation of solution will depend a bit on your use case.

There is no built-in functionality in PyWavelets.  In general this is a hard problem, so the choice and implementation of solution will depend a bit on your use case.

Thanks, and,
I just wonder which part (operation or formula) cause the NAN in the reconsturcted signal?
I will decide to filter out nan  value or pad it  depend on the source reason.

I just wonder which part (operation or formula) cause the NAN in the reconsturcted signal?

It's hard to point at a single line of code or formula. Going from the time to the frequency domain is in general ill-specified in the presence of nan's.

I just wonder which part (operation or formula) cause the NAN in the reconsturcted signal?

It's hard to point at a single line of code or formula. Going from the time to the frequency domain is in general ill-specified in the presence of nan's.

Okay, i see , thank you very much.

I just wonder which part (operation or formula) cause the NAN in the reconsturcted signal?

The basic cause is that the forward and inverse transforms involve convolutions with some sets of wavelet filters. For the db4 case, the filters have 8 coefficients and at each level any Nan value would spread out by an amount equal to the filter footprint! This will occur again during reconstruction so you would expect to see many more NaN in the reconstructed signal than in your source. This is why @rgommers suggestion to somehow replace NaN values prior to applying the transforms seems like the best bet


changed the image user

Now you can use an uri or a local image for the user.

looks good. thanks


fix Fix wrong instance number in query profile

Proposed changes
Issue Number: close #xxx
when PipelineEngine = true
before:
          -  Total  Instances  Num:  2
          -  Instances  Num  Per  BE:  xxx:2
          -  Parallel  Fragment  Exec  Instance  Num:  48

after:
          -  Total  Instances  Num:  49
          -  Instances  Num  Per  BE:  xxx:49
          -  Parallel  Fragment  Exec  Instance  Num:  48

(the more 1 is for Fragment0-exchange)
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.7 seconds
stream load tsv:          503 seconds loaded<PHONE_NUMBER>9 Bytes, about 141 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230714114958_clickbench_pr_178319.html

run build all

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 55.64 seconds
stream load tsv:          508 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.5 seconds inserted 10000000 Rows, about 338K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.88 seconds
stream load tsv:          507 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          30.4 seconds inserted 10000000 Rows, about 328K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 55.24 seconds
stream load tsv:          506 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          36.3 seconds inserted 10000000 Rows, about 275K ops/s
storage size:<PHONE_NUMBER>2 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 50.16 seconds
stream load tsv:          504 seconds loaded<PHONE_NUMBER>9 Bytes, about 141 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>7 Bytes


[WIP] Symbol Random api

Description
Introduce Symbol.random module in scala API
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

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

Changes

Symbol.random api generated by macros

Comments

Is there any JIRA mentioning this feature ?
Only implemented for Symbol. Adding the same for NDArray seems quite straightforward. Tell me if I should to it in the same PR or in a separate
I moved functions from Symbol.api to Symbol.random. Is it OK ?
The produced API is not strictly similar to the one in python:

in python, each function, eg random.normal, has 2 overrides, one scalar, one symbolic.
the generated scala api has different namings: Symbol.random.sample_normal for symbol inputs, and Symbol.random.random_normal for scalars. The arguments naming is also not fully consistent (mu & sigma, vs loc & scale).
=> What are your recommendations on this ?

The unit-test is poor. What should be tested for generated code like this ?

@mdespriee Firstly for unit test, there are tests covers in Python on these operators. In that case we decide not to test them on Scala end.
the generated scala api has different namings: Symbol.random.sample_normal for symbol inputs, and Symbol.random.random_normal for scalars. The arguments naming is also not fully consistent (mu & sigma, vs loc & scale).

For this question, let me consult with people working on Python.
Please also consider adding NDArray with these operators (in NDArrayAPI)

Hi @mdespriee , I have created a JIRA ticket for you: https://issues.apache.org/jira/browse/MXNET-918. You can put it on the title so people know what issue it is related.

Thanks for your contribution @mdespriee
@mxnet-label-bot[pr-work-in-progress]

@lanking520. Underscore removed. We now have this kind of signatures:
def random_normal (loc : Option[org.apache.mxnet.Base.MXFloat] = None, scale : Option[org.apache.mxnet.Base.MXFloat] = None, shape : Option[org.apache.mxnet.Shape] = None, ctx : Option[String] = None, dtype : Option[String] = None, name : String = null, attr : Map[String, String] = null) : org.apache.mxnet.Symbol

and
def sample_normal (mu : Option[org.apache.mxnet.Symbol] = None, shape : Option[org.apache.mxnet.Shape] = None, dtype : Option[String] = None, sigma : Option[org.apache.mxnet.Symbol] = None, name : String = null, attr : Map[String, String] = null) : org.apache.mxnet.Symbol

I don't think we can go further. If remove both "random" and "sample" prefixes, we'll have collisions that the scala compiler can't resolve since all parameters have default values.
(multiple overloaded alternatives of method uniform define default arguments)

@mdespriee Firstly I think we have a collide operators. _sample and _random looks like the same with random and sample. So it looks like we shouldn't need to generate a new set of API in order to get this work. Instead, we should try to use Macros to do something similar to python. Create a helper function that can choose between sample and random. Change the implementation a little bit more for Random Module:

Set the args type as Any for Random API in order to take in NDArray/Float or Symbol/Float
Do a instance check such as asInstanceOf[NDArray] to see if it is an object or numeric value
Do random_ or sample_ in the function

Thanks for your contribution @mdespriee. Please have a look at @lanking520's feedback.

@mdespriee Have you had a chance to look at @lanking520 's feedback? Requesting an update from you on this.

Hi,
I've been busy the 2 past weeks (spark summit...) I noticed @lanking520
about that.
I'll probably be able to work on it this week. This PR is not dead  :-)
Mathieu
Le mar. 9 oct. 2018 à 09:01, Rakesh Vasudevan<EMAIL_ADDRESS>a
écrit :

@mdespriee https://github.com/mdespriee Have you had a chance to look
at @lanking520 https://github.com/lanking520 's feedback? Requesting an
update from you on this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-mxnet/pull/12489#issuecomment-428082344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhFNMShE8rC0j-fVtDh3yxRUmtfguy9ks5ujEmsgaJpZM4WgCDH
.

@mdespriee Good to see you back!

@mdespriee Firstly I think we have a collide operators. _sample and _random looks like the same with random and sample. So it looks like we shouldn't need to generate a new set of API in order to get this work. Instead, we should try to use Macros to do something similar to python. Create a helper function that can choose between sample and random. Change the implementation a little bit more for Random Module:

Set the args type as Any for Random API in order to take in NDArray/Float or Symbol/Float
Do a instance check such as asInstanceOf[NDArray] to see if it is an object or numeric value
Do random_ or sample_ in the function

I've noticed the following inconsistencies between sample and random:

multinomial only exists in sample-> This is catchable at runtime in case of mismatch
normal has different params in both worlds : loc&scale vs mu&sigma. -> How do we manage this ?

@mdespriee Firstly thanks for coming back to maintain this PR!
I see your point, is there a mapping can be applied to these operators? If yes as they are just the difference like that, we can apply something in Macros end to make it change.
Apart from that, I think you can extend https://github.com/apache/incubator-mxnet/blob/master/scala-package/core/src/main/scala/org/apache/mxnet/Random.scala this class to support Random with your Macros design

Can we add a gist of the generated code?
Also, is there a reason that we're adding a new NDArray.random as an api instead of extending NDArray.api (same for Symbol)?

@andrewfayres here is what we are trying to do: https://mxnet.incubator.apache.org/api/python/ndarray/random.html. In NDArray API we already have some random generation function, but a Random namespace would be great since it is compatible with both NDArray/Float.

@mdespriee thanks for your effort to refactor the code. The issue shown above is the mismatch on the parameter. It can be caused by function name do not have this parameter, or you put a param in the wrong type. I would recommend you to split the refactoring portion and Random generation portion into two PRs. The good thing:

you can focus on each of them and make the refactoring and Random part even better.

Reduce the change on your code, it's easier for you to find where the problem is and easy for us to help you find where you can make a mistake.

I know this may introduce more work to you but I think this could be the best solution.

Following @lanking520 comment, I split the work in 2 PRs:

https://github.com/apache/incubator-mxnet/pull/13038 (refactoring of code gen)
https://github.com/apache/incubator-mxnet/pull/13039 (Random module itself)


devcontainer template publish does not support gcp artifact registry

SyntaxError: Unexpected token N in JSON at position 0
at JSON.parse ()
at jH (/opt/homebrew/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:602:773)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async qH (/opt/homebrew/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:600:654)
at async JC (/opt/homebrew/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:604:1304)
at async HeA (/opt/homebrew/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:604:5465)
at async /opt/homebrew/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:604:4569

Hi 👋
Unfortunately, there is no current support for Google Cloud Platform's Artifact Registry. However, the CLI supports GitHub Container Registry and Azure Container Registry.
We'd be happy to add support for GCP given the community's interested. Feel free to contribute a PR if you'd be interested.
Thanks!


Cleanup and Most Recent Ordering

Most of the changes are whitespacing with a few changes to make comparisons to None, strings, and booleans more pythonic.
A "--recent" or "-r" (logically also "reverse") flag was added to log to order the output by most recent transactions.

Thanks for the contribution! Good idea with the recent time stamp ordering.


Cucumber only showing Capybara errors

I keep having an annoying issue with Cucumber. If there are any errors in the Controller, it only shows the errors raised by capybara. I'm sure that's not the way it used to work?
For example, if an exception was raised, the only output I see is: expected to find css "h1" with text "blah"
In order to actually see the error, I have to puts page.body in order to see the Controller error
Is this expected behaviour?

Any traction on this?

@segantii-alex-falkowski Are you seeing the same issue? I've recently moved away from cucumber as nobody outside of engineering was writing gherkin, but I've got some legacy features I could check for this.

Yep I am seeing this issue. We just tail the logs. Though I thought I would raise just so you guys know :)

I would suggest raising a new issue with what you're seeing and closing this one since it was raised Aug 23, 2013.

Including an example Rails app demonstrating the problem will help your issue get solved much more quickly


Create a new github repo dedicated to the UxPerf WA workload dependency files

Any of us can set this up and then transfer ownership back to Michael or myself.

Repo currently at https://github.com/jimboatarm/wa-assets
Will be migrated to upstream when ready


Add support for setting ping timeout

Add support for setting ping timeout.
In certain network conditions, the default 2 second timeout for the ping test can be too short resulting in false failures. This feature adds support to setting the timeout to something custom but keeps the default at 2 seconds.
It might be worth changing the default value from 2 seconds to 5 seconds as 5 seconds appears to be sufficient when on a network with high latency. This has been tested using the "Network Link Conditioner" developer tool.

Nice feature. I was wondering if that will be sufficient in some cases. Now you test that using the "Network Link Conditioner" developer tool, that's great! Thank you!


Call to bugsnag sending Referer header with SAMLRequest payload

My Setup

Operating System: Windows 10
App Version: 3.1.1
Installation type: EXE

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description

Initiate connection to a RocketChat server with the open developer tools window
Check HTTP headers on the call to bugsnag
Observe SAMLRequest payload

Current Behavior

When using desktop client SAMLRequest being sent in Referer header when HTTP call to bugsnag happens
Expected Behavior

Call to bugsnag must have no SAMLRequest data in it

There should be also install option to avoid sending any data to Bugsnag. I think this feature causes data leakage. I don't get why the desktop client even sends any data outside.  If privacy is important, the Bugsnag feature should be completely removed.

Please test if the problem persists with our latest release. I will be closing the issue but if there is any problem yet, we will open it again.


merge: dev to stage

merge: dev to stage

:tada: This PR is included in version 1.11.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Login password to SSI.sh

Hi,
thanks for your service. I'm mistaking something.
If I try with ssh -R jacopo:80:localhost:80 ssi.sh I get a response with login and password from ssi.sh.
I don't know ssi.sh login, of course. What I'm doing wrong?
J.

Please refer here:
https://github.com/antoniomika/sish/issues/201#issuecomment-988937007


LanguageModelData not defined

hello, i was following the tutorial. and in this code give me an error
trn_dl = LanguageModelLoader(pd.DataFrame(trn_lm,columns=['x']), bs, bptt)
val_dl = LanguageModelLoader(pd.DataFrame(trn_lm,columns=['x']), bs, bptt)
md = LanguageModelData(PATH_MODELS, 1, vs, trn_dl, val_dl, bs=bs, bptt=bptt)

NameError: name 'LanguageModelData' is not defined

i was check on this file  and there are no class 'LanguageModelData' .  and on this doc at 'Language Model Data' at the bottom say 'This is all done internally when we use TextLMDataBunch, by creating DataLoader using the following class:'
its that mean the class LanguageModelData was refactored to TextLMDataBunch ?
i will used fastai for production so i used the new version of fastai.
so how to fix that problem with the new version of fastai?
thanks

So, what's the equivalent of LanguageModelData in V.1?

forget about it, just read the documentation v.1
if you  read v.06 code and used v1 you will be confused
better if you forget all about v.06 and learn from the beginning with v.1

@alibaltschun Ok thanks. Any link to a tutorial of V.1?

@frannievas  hope you will find this useful:
https://github.com/fastai/fastai/blob/release-1.0.60/courses/00-DO-NOT-USE-WITH-FASTAI-1.0.x.txt

@zsquaredz Thanks!

@frannievas , did you resolve the issue?. I am stuck in finding alternative of LanguageModelData

@brij1823 In the end, I ended up using the newest version of FastAI and following the new courses/tutorials


Fix formatting of lat,long in traffic router access log

and log an error if a cache group with a location of 0.0,0.0 is selected during Geo lookup
(cherry picked from commit a06b20443078c19a4726104b88af3c38f4cfba1d)

already taken care of from previous PR's

Because RC0 is up for a vote, this is a nice to have for 1.4.x if we are planning to do an RC1.


[APIView] SwaggerAPIView failed with "Create ApiView failed because of authorization failure"

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2763385&view=logs&j=9e7477d6-80c4-54a8-ebef-1e18da128ba0&t=d53a2f62-bad1-573a-caa1-b07403b79c0d&l=294

@chidozieononiwu this was fixed in the latest deployment?

Happened again but this is due to an external resource.

Is there any work we still need to do or can we close the issue


[Feature Request]: Custom Fields in HomePage

Enhancement of issue
As per Homepage Version 0.8.6 (with PeaNUT Version 2.3.2) the only allowed fields are: ["battery_charge", "ups_load", "ups_status"] (https://gethomepage.dev/latest/widgets/services/peanut/)
It is possible to remove one or more of the allowed fields in the widget configuration (https://github.com/gethomepage/homepage/issues/2697), but you cannot add any other field (like battery runtime or something else) out of the key list in PeaNUT.
Solution
Possibility to add max. 4 custom fields in the PeaNUT widget (like described in #2697 listed above)
Alternatives

Additional Context

For this, I would suggest using the customapi widget and creating your own. Many of the fields in the api response are specific to your device, so others may not have them.
Here is an example with battery runtime:
        widget:
            type: customapi
            url: http://{localhost}:{port}/api/v1/devices/ups
            mappings:
            - field: battery.charge
              label: Battery Charge
              format: percent
            - field: battery.runtime
              label: Battery Runtime
              format: text
            - field: ups.load
              label: UPS Load
              format: percent
            - field: ups.status
              label: UPS Status
              format: text
              remap:
                - value: OL
                  to: Online
                - value: OB
                  to: On Battery
                - value: LB
                  to: Low Battery
                - any: true
                  to: Unknown

Some additional manual configuration in JS is needed to get the value "battery.runtime" which has to be scaled with 0.0166667 to get the runtime from seconds to minutes.
By now there is no rounding function available in the customapi widget in homepage.
Thank you @Brandawg93 :)

Found something a lot easier:
    - field: battery.runtime
      label: Runtime
      format: float
      scale: 1/60
      suffix: "Min"

Works as expected an no additional manual configuration in JS needed.
Mentioned by @ffeingol in https://github.com/Brandawg93/PeaNUT/issues/23


Unable to get Scenario step text at runtime

I am trying to retrieve the Scenario step text at run time. This used to be possible in the Java implementation via reflection. See answer here:
https://stackoverflow.com/questions/39664422/in-cucumber-is-it-possible-to-programmatically-get-the-current-step-being-execu
    Field f = scenario.getClass().getDeclaredField("testCase");
    f.setAccessible(true);
    TestCase r = (TestCase) f.get(scenario);

    //You need to filter out before/after hooks
    List<PickleStepTestStep> stepDefs = r.getTestSteps()
            .stream()
            .filter(x -> x instanceof PickleStepTestStep)
            .map(x -> (PickleStepTestStep) x)
            .collect(Collectors.toList());

However, with the more recent cucumber (from 4.8.0) this is no longer possible. I get the following error:
java.lang.NoSuchFieldException: testCase
Is there any other means to obtain the step text? I can do it with Cucumber.js via the Scenario.pickle object

What are you using the step text for?

What are you using the step text for?

Your favourite answer for this ;) Reporting/logging. I am aware that the plugin route is an option. However, we already have a fully working reporting mechanism working very well using the hooks. The only limitation we have is the fact we can't retrieve the step text.
I have done something similar in cucumber.js along with SpecFlow (for .NET) so just seems a shame it's not currently possibly with Java

Then you'll know that I don't see a good reason to support it. :smile:
Now I'm sure you can imagine I don't find the argument that time has already been invested in a solution quite compelling. Esp since that solution involves reflection. There should have been some awareness that this was not a maintainable solution.
Now I'm not sure what prompted the use of reflection in the first place, but if you can highlight a use case that can't be achieved using the plugin system that would be more compelling.

I have done something similar in cucumber.js along with SpecFlow (for .NET) so just seems a shame it's not currently possibly with Java

@aslakhellesoy do you have an opinion on this?

Using reflection to reach into the bellows to use a non-public API is a surefire way to make upgrades difficult, because there is no guarantee that non-public APIs will remain unchanged.
This sounds like an XY problem to me.
@Phonesis what kind of loggin/reporting are you doing that cannot be done with a plugin?


Add xcproj to the Project Setup section

Project URL
https://github.com/xcodeswift/xcproj
Description
Add xcproj to the project setup section.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Thx


v2023.12 Array API compliance

(Ported this issue over since there are a few outstanding tasks).
The v2023.12 iteration of the Array API standard was recently released (changelog).
The following changes to ndonnx will be required:

[ ] Add __array_namespace_info__
[x] Move clip from additional namespace to main one.
[x] Move cumsum from additional namespace to the main one and rename to cumulative_sum.
[x] Move searchsorted from additional namespace to main one.
[x] Add maximum.
[x] Add minimum.
[ ] Add signbit.
[ ] Add unstack.
[ ] Add repeat.
[ ] Add moveaxis.
[ ] Add tile.
[ ] Add hypot.

We are also missing linalg support.
It looks like ndonnx already satisfies the clarifications to the specification, including the most urgent ones motivated by the ONNX use case in the first place like raising in intrinsically eager functions like __bool__.

maximum and minimum at least are not yet implemented


⚠ Bofrost.be has degraded performance

In 4022fa1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1505 ms

Resolved: Bofrost.be performance has improved in 468833f.


Question

Why invoke a repo dispatch event? Can’t we instead open a check run from the issue_comment event directly?  Would something like this work:
If owner OR comment = “/ok-to-test”
...tests?
Thanks just learning I’m obviously missing something

Would something like this work:

If owner OR comment = “/ok-to-test”
.> ..tests?

Good question @hamelsmu – yes! I was envisioning more slash commands like deploy being parsed in a separate workflow (now called ok-to-test.yml). It's a tradeoff of complexity, which is now hidden behind the helper action(s) in ok-to-test.yml.
peter-evans/slash-command-dispatch takes care of a few other things that I'd want to see in a test-centric solution, like reacting back on the comment, ensuring the permission level of the commenter, restricting the issue type, and grabbing the pull request number that the test workflow can use.
Maybe we should make this into a stop/go action that takes an issue_comment, decides if it's a valid comment from a permissioned author, and proceeds. 🤔This part to update the PR check could also be a separate action: https://github.com/imjohnbo/ok-to-test/blob/02903c45e6a36ca82c7d68456571f9601b355879/.github/workflows/integration.yml#L40-L71.


2.0 fix for rebar/eunit problems with backend tests

Move test code from riak_kv_backend into its own module to avoid rebar/eunit complications. See #1127

:+1: 6d7507a

@borshop merge


[WIP] feat: Modal component

Summary
Related Issues or PRs

How To Test

Screenshots (optional)

Should this PR be targeting sr-1304-uswds-2.11 for now, given that we'll be merging that one first, so that the diff shows only the changes for this PR?


[DO NOT MERGE] Holding PR for Mina_base compiling to javascript

Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

Stale, closing.


Updating the index page + other small changes

Updated the index page with new links

And small HTML/CSS changes:

Hiding the search box (for now)
Adjusting the size of the blue box

👍

👍

@Davidslv Yep! Keeping it hacky and inefficient while still working out what padding works best.


Tech architecture/roadmap

Work-in-progress draft:
https://miro.com/app/board/o9J_l2aa1uc=/

Initial Q & A with Josh:
(This conversation initially happened in the #ds-general channel on Slack. Documenting it here for visibility.)
- how does the tech roadmap interact with enabling designers - it looks like what’s here is a dev solution and doesn’t address non-dev rapid prototyping or flows?
Hm.. it does not. This will constitute a conversation starter for designers as well. Also I think maybe I should specify that this is not a product roadmap, just a tech brainstorm of the parts that go into a design system. We will dig deeper into each part in conversations with design and research and everyone else.

Does the tech solution need to address the website that will house guidance or is that a separate product? If so, who will be building it?

I thought we can address that as part of the guidance piece. Not yet clear on where it will be hosted, who (besides content designers will actually be involved in it). But I was waiting for @David Pepin as well as @Amy M to return from holidays to hear from them how they’d like to approach this. Then we’ll arrange support as needed.

How do the tokens interact with components and their individual CSS?

That’s definitely a conversation to happen once we dig in deeper into each category in the diagram. One thing I’d mention that we don’t need to address them separately, the way they’re presented in the diagram.

If we’re supporting iOS and Android from the get go, what does this mean for the components and their individuality on those platforms? They have their own unique design languages and requirements.

That’s another great question that we need to consider and talk more in detail about. If we come to the conclusion that we’d rather leave Apps for later, that’s an option I’d like to leave on the table.

Does the tech solution need to think through ways to enable external devs to hook into certain core features/design tokens/etc. for their own custom components?

Eventually… yes. When, and where that fits on the product roadmap, I don’t know yet. There might be pieces that we’ll have to consider from the get-go and I think we need to stay in close connection to research around contribution models (that’s the governance piece). I’m noting this question down too, for sure.


Updating coverlet collector to use GetPropertyValue and if its missing fallback to GetProperties

To test https://github.com/tonerdo/coverlet/issues/748
Fallback to get the test source property when the GetPropertyValue method is not found.
If the GetProperties method is also not found, a missingMethodException will also be raised.
If the GetProperties method doesn't return the property, we rethrow the exception.
I've tested both methods return the TestSources property when run using .NET Core SDK 2.2.400.
This is a speculative fix to see if GetProperties method exists for .NET Core SDK 2.1.400.

/azp run