no exported member 'SecureStorage'.

Hello there!
I am using cordova secure storage on my ionic 2 app,
I got this error:
TypeScript error: /Users/endriazizi/Documents/Project/app/app/pages/oauth/oauth.
service.ts(7,10): Error TS2305: Module '"/Users/endriazizi/Documents/Project/app
/node_modules/ionic-native/dist/index"' has no exported member 'SecureStorage'.
TypeScript error: app/app.ts(12,10): Error TS2305: Module '"/Users/endriazizi/Documents/Project/
app/node_modules/ionic-native/dist/index"' has no exported member 'SecureStorage

in my package.json:
"cordova-plugin-facebook4",
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>I don't know what to do
Can someone help me?

This seems like an issue with your config. I am afraid there's not much I can do to help you here.


Telegram Bot with Scheduled Tasks

The Telegram Bot with Scheduled Tasks is designed to schedule the tasks for example for 10am you should read
Pls assign this issue to me @Shubhanshu156

issue assigned to @samanth2012

Hi
I would like to work on this. Please can you assign it to me?

please assign this to me.

I have already implemented such project so maybe I can help resolve issue and most importantly I consider that teamwork can provide more optimization as well boost the implementation process.

Hi
I would like to work on this. Please can you assign it to me?

Hi
I would like to work on this. Please can you assign it to me?

@Shubhanshu156 I want to contribute.Please assign me this issue


[Feature][PyDolphinScheduler] Support Kubernetes task in pyds

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
part of  https://github.com/apache/dolphinscheduler/issues/11906
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@jieguangzhou hi,guys i want to try it

Thanks for the help!

close due to https://github.com/apache/dolphinscheduler-sdk-python/pull/15


Error on addon not working any more

I know you have depreciated the addon, but I am writing maybe you can recover with a small tweak.
The ngrok client addon log gives following error:
s6-overlay-suexec: fatal: can only run as pid 1
and not working anymore
I will be grateful if you can handle it.

https://developers.home-assistant.io/blog/2022/05/12/s6-overlay-base-images/
according to direction above, github files permissions should be updated???

I just pushed 2.1.8 which should resolve that issue.

works well I appreciate your effort and prompt reply, thanks

I can't update to version 2.1.8 [ngrok client 2.1.6 installed on Hassio which works fine] I have this error
Error updating ngrok Client: Can't install 1ded40ef/amd64-addon-ngrok:2.1.8: 404 Client Error for http+docker://localhost/v1.41/images/create?tag=2.1.8&fromImage=1ded40ef%2Famd64-addon-ngrok&platform=linux%2Famd64: Not Found ("pull access denied for 1ded40ef/amd64-addon-ngrok, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
TNX

Nothing works anymore .... Home Assistant has disabled everything ... Even restoring the old configurations does not allow any functionality ... Was it bothering?!?!


ASIM Parser for AuditEventMicrosoftWindowsEvents Issue with 1102

Describe the bug
The ASIM Parser AuditEventMicrosoftWindowsEvents incorrectly categorizes some 1102 events as a Delete Logs operation.
As 1102 events are not exclusive to event log clears, this creates a false positive detection if you are working off the ASIM parser
AD FS servers will log an 1102 to the security log that are not log clears
Another example of such an 1102 can be found here which is in the Application log: https://social.technet.microsoft.com/Forums/en-US/7061c9d8-a03c-4195-9e20-e8d94eaee453/microsoftfilteringfipfs which is also not a log clear
To Reproduce
Generate any 1102 event that is not a security log clear and the parser will indicate that logs were deleted.
Expected behavior
Only actual security audit/log clear 1102 should trigger a log clear event with the parser

Hi @briandelmsft, thanks for flagging this, we will get back to you once we receive an update from the internal team.

Hi @briandelmsft, we have reached out to concerned team for this issue, we will get back to you once we receive an update.

Hi @briandelmsft, we have reached out to concerned team for this issue, we will get back to you once we receive an update.

Hi @briandelmsft, we have reached out to concerned team for this issue, we will get back to you once we receive an update.

Hi @briandelmsft, once we receive a response from concerned team, we will let you know

Hi @briandelmsft, once we receive a response from concerned team, we will let you know

Hi @briandelmsft, once we receive a response from concerned team, we will let you know

Hi @briandelmsft, once we receive a response from concerned team, we will let you know

Hi @briandelmsft, once we receive a response from concerned team, we will let you know

@briandelmsft Thanks for raising this issue. I'll be looking into this request. Are you still facing this issue or have applied any workaround for this? If possible, please help sharing the ADFS or 'other' log sample that causing false positive.

@vakohl Yes I am still facing the issue.  I have not applied a workaround though a simple KQL tweak to the parsers should do it.
Really we need a more granular filter on the 1102 event to only capture log clear, and not exclude the 'other' events as there could be many 1102 from other products.  As I originally indicated, I've already found 2 cases.
Keep in mind, this issues applies to the SecurityEvent, Event and WindowsEvent tables but my examples below only show the SecurityEvent data
Here is an event that triggers a lot clear... but it isn't.  I know the activity says log clear, but that seems to be something log analytics/Sentinel is doing on ingestion as it is not and the source event does not contain any reference to log clear:

here's the source event before ingestion copied from event viewer:
Log Name:      Security
Source:        AD FS Auditing
Date:          9/28/2023 12:55:14 PM
Event ID:      1102
Task Category: (3)
Level:         Information
Keywords:      Classic,Audit Success
User:          XXXXX\xxxxxxxxx$
Computer:      ADFS01.xxxx.xx
Description:
The Federation Service authorized a request to one of the REST endpoints.

Additional Data

Activity ID: 163ff081-41d9-4cdc-2b19-0080010000f0

Request Details:
    Client IP: 192.168.xx.xx
    Requested URI: https://xxx.xxxxxxxx.xxx:443/adfs/Proxy/GetConfiguration?api-version=2

Additional details:
The client was authenticated using the client certificate with thumbprint '4BC8E9A4623BF4A60AD4329763D07ABA3CAA2368' and subject 'CN=ADFS ProxyTrust - XXXXXXXXX'.
Event Xml:
<Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
  <System>
    <Provider Name="AD FS Auditing" />
    <EventID Qualifiers="0">1102</EventID>
    <Level>0</Level>
    <Task>3</Task>
    <Keywords>0x80a0000000000000</Keywords>
    <TimeCreated SystemTime="2023-09-28T16:55:14.556141400Z" />
    <EventRecordID>86381748</EventRecordID>
    <Channel>Security</Channel>
    <Computer>ADFS01.xxxxxx.xxx</Computer>
    <Security UserID="S-1-5-21-0000000-100000000000-24000001-1107" />
  </System>
  <EventData>
    <Data>163ff081-41d9-4cdc-2b19-0080010000f0</Data>
    <Data>192.168.xx.xx</Data>
    <Data>https://xx.xxxxxxxx.xxx:443/adfs/Proxy/GetConfiguration?api-version=2</Data>
    <Data>The client was authenticated using the client certificate with thumbprint '4BC8E9A4623BF4A60AD4329763D07ABA3CAA2368' and subject 'CN=ADFS ProxyTrust - XXXXXXXX'.</Data>
  </EventData>
</Event>

here is an actual log clear:

thanks for sharing @briandelmsft , I'll be looking into this.

@vakohl Any update on this? This makes this parser unusable as a number of systems, particularly AD FS log 1102 events that are not event log clears

Hi @briandelmsft , apologies for delay on this, I'll try to prioritize this case.

@briandelmsft
The information you shared is useful. Even I see multiple EventType for 1102. Also, your observation regarding Sentinel tagging those events as Log cleared is correct. Also, I noticed that, since actual 'audit log clear' has different Eventdata field format, hence fields like ActorUsername, ActorUserId, ActorSessionId won't be getting parsed and would be coming as null currently with 1102, can you please confirm if these are coming as null in your case to? We might need to fix this parsing issue as well.
I'll keep you posted for further progress on this issue.

@briandelmsft can you please confirm, with EventID:1102 logs the fields ActorUsername, ActorUserId, ActorSessionId are not getting parsed.

@vakohl they are getting parsed for me when it is a legitimate event log clear.  That is true for both when found in SecurityEvent and WindowsEvent tables.  In looking for this I've located another bug, the imAuditEvent unifying parser doesn't call the right stub parsers and therefore drops all of the invoked parameters, for example imAuditEvent(starttime=ago(14d)) does not return the last 14 days of data.  I will open another issue for this.

just logged the additional bug as #9474

Hi @briandelmsft, Noted. Thanks!

@vakohl they are getting parsed for me when it is a legitimate event log clear. That is true for both when found in SecurityEvent and WindowsEvent tables. In looking for this I've located another bug, the imAuditEvent unifying parser doesn't call the right stub parsers and therefore drops all of the invoked parameters, for example imAuditEvent(starttime=ago(14d)) does not return the last 14 days of data. I will open another issue for this.

@briandelmsft I doubt that  ActorUsername, ActorUserId, ActorSessionId will get parsed with 1102 with SecurityEvent table, they should be coming null. Can you please help sharing the 'EventData' field value for legitimate event log clear.

just logged the additional bug as #9474

@briandelmsft
thanks for reporting this issue. This require union parser to be updated, will get this fixed in separate PR. For now, please use the built-in AuditEvent functions instead of workspace deployed ones. e.g.
_Im_AuditEvent(starttime=ago(14d))
| summarize max(TimeGenerated), min(TimeGenerated)

@briandelmsft I doubt that ActorUsername, ActorUserId, ActorSessionId will get parsed with 1102 with SecurityEvent table, they should be coming null. Can you please help sharing the 'EventData' field value for legitimate event log clear?

here's EventData from a legitimate clear from the SecurityEvent table:
<UserData xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
  <LogFileCleared xmlns="http://manifests.microsoft.com/win/2004/08/windows/eventlog">
    <SubjectUserSid>S-1-5-21-111111111-1111111111-1111111111-11111</SubjectUserSid>
    <SubjectUserName>username</SubjectUserName>
    <SubjectDomainName>DOMAIN</SubjectDomainName>
    <SubjectLogonId>0x1fc296c</SubjectLogonId>
    <ClientProcessId>8412</ClientProcessId>
    <ClientProcessStartKey>82753643152935667</ClientProcessStartKey>
  </LogFileCleared>
</UserData>

The specific event does parse out those 3 fields: ActorUsername, ActorUserId, ActorSessionId with the correct data


Add Losses

This PR adds losses to framework.
All the loss sub-classes inherit from Loss.
The Losses class, has methods that can be called directly to get raw loss values. These are utilized by the Loss subclasses before applying a Reduction to the loss. The Losses class will also  be used by some of the Metric classes when that feature is submitted.
The impl package has some helper methods and classes utilized by the loss classes, and are not expected to be exposed outside the framework module,  when we do modules.

Hi @JimClarke5 , what would be the best order for reviewing your PRs? You have this one, #123 and #106 that are still opened.

I would do activations #123 first.

@karllessard I have started working on Metrics which depends on Loss, and thusly this PR. I plan to do Metrics in two PRs, the first PR will focus on Metrics that depend on Losses functions. The second PR will focus on Metrics that don't depend on Losses functions, so these are more complicated to implement. My goal is to set up the scaffolding in the first Metric PR, then focus on the functionality required in the second PR.  As to which existing PR to work on after activations #123 , may I suggest that this PR will need to be done before I create the first Metric PR.

@Craigacp we could set a range check, but it would have to be a control dependency e.g. using tf.assertThat

I tested the range check, but found out that Eager mode does not support Control Dependencies.  I have modified my function to check if its Graph or Eager. Graph mode produces a Control Dependency, while Eager mode throws IllegalArgumentException. The new method is in LossesImpl.rangeCheck. I have to fix the rest of the test cases and see what other Classes besides xxxCrossentropy need this feature.

I have added 2 methods to LossesImpl, rangeCheck and valueCheck.
rangeCheck checks that all the values are within the provided min and max inclusive, eg [0. - 1.].
valueCheck will check that all the values are contained in a set of values (e.g. [-1, 0, 1]).
FIrst, control dependancies do not work in Eager mode. To handle this, I throw an IllegalArgumentException from the call method when in Eager mode if the check fails. In Graph mode, I set up a control dependency for the check, and it it will throw TFInvalidArgumentException if the check fails  when the loss Operand is run.
One question, should these utilities be stored in a common location like framework.utils.  I suspect that metrics will have the same checks, but metrics could easily still call the LossesImpl methods.

@Craigacp , @deansher , can you please go through all your previous comments/suggestions/questions and mark as resolved those that are either resolved by a fix from @JimClarke5 or are minor and can wait? The PR looks mature enough to be merged soon.

@karllessard  Build failing after push.
Error:  Failed to execute goal on project tensorflow-core-api: Could not resolve dependencies for project org.tensorflow:tensorflow-core-api:jar:0.3.0-SNAPSHOT: Could not find artifact org.tensorflow:tensorflow-core-api:jar:linux-x86_64:0.3.0-SNAPSHOT in ossrh-snapshots (https://oss.sonatype.org/content/repositories/snapshots) -> [Help 1]

@karllessard Build failing after push.

That's just a transient network error:
2020-11-12T15:19:41.5511149Z [WARNING] Could not transfer metadata org.tensorflow:tensorflow-core-api:0.3.0-SNAPSHOT/maven-metadata.xml from/to ossrh-snapshots (https://oss.sonatype.org/content/repositories/snapshots): Transfer failed for https://oss.sonatype.org/content/repositories/snapshots/org/tensorflow/tensorflow-core-api/0.3.0-SNAPSHOT/maven-metadata.xml 502 Bad Gateway

@saudet , @JimClarke5 , it might also be related that we are having trouble these days to build our artifacts as you can see here, the linux GPU build runs out of space and that prevent the last step to occur (i.e. the bulk deploy that normalize all snapshots timestamp which might explain why a few artifacts disappeared lately: #142).
I’ve retried many times but without success. Samuel, any idea how to solve this again?

I’ve retried many times but without success. Samuel, any idea how to solve this again?

It just looks like GitHub Actions is below its guaranteed 14 GB of disk space:
2020-11-12T04:02:15.8038646Z Filesystem      Size  Used Avail Use% Mounted on
2020-11-12T04:02:15.8039270Z overlay          84G   75G  8.9G  90% /
2020-11-12T04:02:15.8039572Z tmpfs            64M     0   64M   0% /dev
2020-11-12T04:02:15.8039893Z tmpfs           3.4G     0  3.4G   0% /sys/fs/cgroup
2020-11-12T04:02:15.8040222Z shm              64M     0   64M   0% /dev/shm
2020-11-12T04:02:15.8040517Z /dev/sdb1        84G   75G  8.9G  90% /__w
2020-11-12T04:02:15.8040859Z tmpfs           696M  768K  695M   1% /run/docker.sock
2020-11-12T04:02:15.8041228Z tmpfs           3.4G     0  3.4G   0% /proc/acpi
2020-11-12T04:02:15.8041550Z tmpfs           3.4G     0  3.4G   0% /proc/scsi
2020-11-12T04:02:15.8041886Z tmpfs           3.4G     0  3.4G   0% /sys/firmware

We'll have to wait until they fix that, again, I guess?

@Craigacp I have modified the JavaDoc for Losses.cosineSimilarity to insert this section in front to clarify the differences between the mathematical definition of cosine similarity and this loss function.
   *
Note that it is a number between -1 and 1, which is different from the mathematical definition of cosine
   * similarity where 1 represents similar vectors, and 0 represents dissimilar vectors.
   * In this function, the numbers are inverted in a range of -1 to 1.
....

@Craigacp , it looks like @JimClarke5 pushed all commits for the last changes you've requested, I let you validate and dismiss your review if that's the case, thanks
@JimClarke5 , some of your unit test files don't have a valid source header, can you please add one? Also, I've noticed that now you are authoring your work to Oracle, that is ok but I just want to validate with you if it was intentional, thanks

I have fixed the copyright issues. The attribution to Oracle was a mistake, as I copied the copyright from Optimizers to the Loss classes. I have replaced them with the TensorFlow Authors copyright.
@Craigacp did you intend this for Optimizers?

@JimClarke5 yes, I'm required to put the Oracle copyright header on substantive external open source contributions that are part of my job. There's an internal review process for all the things that I write that are longer than a line or two.
@karllessard I've resolved my two comments, I think this is good to be merged now. I can approve it if you want.

All right, this one is merged now, thanks for great contribution again @JimClarke5 !

::party::
On Tue, Nov 17, 2020 at 7:09 AM Karl Lessard<EMAIL_ADDRESS>wrote:

All right, this one is merged now, thanks for great contribution again
@JimClarke5 https://github.com/JimClarke5 !
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/java/pull/129#issuecomment-728885532, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABZ7X44NRW56MHAI4DLBCDSQJRZBANCNFSM4SJLM76A
.


Break on Atom 1.12.0 update

I was happily working away inside Atom with terminal-plus open, got the prompt to update Atom and on restart I got a native extension (??) error msg (sorry, cannot find it now, not sure how to trigger it again in Atom either bc Atom prompted to rebuild the package and apparently it did with no issues).
Interestingly, my global node-gyp vanished too, but reinstalling it did not fix the problem for terminal-plus, which loads the pane and a blinking cursor shows, but no input etc.
Here's the console output from devtools:
Cannot find module '../build/Release/pty.node'
task.coffee:86 Module version mismatch. Expected 49, got 46. Error: Module version mismatch. Expected 49, got 46.
    at Error (native)
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:168:20)
    at Object.Module._extensions..node (module.js:568:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:168:20)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/pborbely/.atom/packages/terminal-plus/node_modules/pty.js/lib/pty.js:17:9)
    at Module._compile (module.js:541:32)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:201:21)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/pborbely/.atom/packages/terminal-plus/node_modules/pty.js/index.js:2:18)
    at Module._compile (module.js:541:32)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:201:21)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/pborbely/.atom/packages/terminal-plus/lib/process.coffee:1:7)
    at Object.<anonymous> (/Users/pborbely/.atom/packages/terminal-plus/lib/process.coffee:1:1)
    at Module._compile (module.js:541:32)
    at Object.value [as .coffee] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:201:21)
    at Module.load (module.js:458:32)(anonymous function) @ task.coffee:86(anonymous function) @ task.coffee:146module.exports.Emitter.simpleDispatch @ emitter.js:25module.exports.Emitter.emit @ emitter.js:129(anonymous function) @ task.coffee:98emitTwo @ events.js:106emit @ events.js:191process.nextTick @ internal/child_process.js:719_combinedTickCallback @ internal/process/next_tick.js:67_tickCallback @ internal/process/next_tick.js:98
task.coffee:86 Cannot read property 'bind' of null TypeError: Cannot read property 'bind' of null
    at process.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/task-bootstrap.js:82:23)
    at emitTwo (events.js:106:13)
    at process.emit (events.js:191:7)
    at process.nextTick (internal/child_process.js:719:12)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)(anonymous function) @ task.coffee:86(anonymous function) @ task.coffee:146module.exports.Emitter.simpleDispatch @ emitter.js:25module.exports.Emitter.emit @ emitter.js:129(anonymous function) @ task.coffee:98emitTwo @ events.js:106emit @ events.js:191process.nextTick @ internal/child_process.js:719_combinedTickCallback @ internal/process/next_tick.js:67_tickCallback @ internal/process/next_tick.js:98

Also, I am on:

npm 3.10.3
node 6.4.0 (system via nvm)

I hope this helps and great work btw - loving the plugin

Sorry, it must have just happened to a bunch of peeps w/ 1.12.0, feel free to mark duplicate and close...


Fix resource queue corruption

This commit fixes a situation, when a process waiting for resource group slot receives SIGTERM. Some callbacks - RemoveTempRelationsCallback for example - try to open new transactions on proc exit. It caused an attempt to add the process to the waiting queue for the second time (and queue corruption). Now we don't add process to the queue if it is already waiting for the slot.
This PR fixes https://github.com/greenplum-db/gpdb/issues/11369 and should be back ported to 6X and 5X.

@Stolb27 sorry for that inaccuracy in the tests - it seems I have made a mistake when shipped/applied the patch from linux server (where was running tests) to my mac.

@darthunix you have branched out from adb/resgroup_queue. What is this branch? What GPDB version it corresponds?

@maksm90 it was Greenplum's actual master at the moment I have finished this PR (https://github.com/greenplum-db/gpdb/commit/ddb78b43bc35c53005406e001da46b621d31c082).

@maksm90 it was Greenplum's actual master at the moment I have finished this PR (greenplum-db@ddb78b4).

OK, thanks, approved. But it's better to explicitly point failed backtrace with violated Assert(!procIsWaiting(proc)), not that is in https://github.com/greenplum-db/gpdb/issues/11369

@darthunix one thing that confuses me is that termination of process occurs after resource slot releasing but in case without temp table it occurs immediately

@darthunix one thing that confuses me is that termination of process occurs after resource slot releasing but in case without temp table it occurs immediately

I have investigated this behavior and this doesn't seem wrong. As you mentioned above the process tries to open another txn to remove temp objects at the time of termination and to do this it tries to acquire free resource slot and gets stuck once more. Backtrace of new waiting state is following:
#0  waitOnGroup (group=0x7f7d89809928, isMoveQuery=false) at resgroup.c:2908
#1  0x00005561d20428c2 in groupAcquireSlot (pGroupInfo=0x7ffe673f9cd0, isMoveQuery=false) at resgroup.c:1950
#2  0x00005561d2043f2f in AssignResGroupOnMaster () at resgroup.c:2649
#3  0x00005561d18f478d in StartTransaction () at xact.c:2701
#4  0x00005561d18f5b53 in StartTransactionCommand () at xact.c:3697
#5  0x00005561d1988f35 in RemoveTempRelationsCallback (code=1, arg=0) at namespace.c:4404
#6  0x00005561d1de1195 in shmem_exit (code=1) at ipc.c:266
#7  0x00005561d1de104b in proc_exit_prepare (code=1) at ipc.c:221
#8  0x00005561d1de0f07 in proc_exit (code=1) at ipc.c:109
#9  0x00005561d1fe338e in errfinish (filename=0x5561d26d8df6 "postgres.c", lineno=3842, funcname=0x5561d26db230 <__func__.31259> "ProcessInterrupts") at elog.c:689
#10  0x00005561d1e20613 in ProcessInterrupts (filename=0x5561d27439fb "resgroup.c", lineno=2910) at postgres.c:3842
#11  0x00005561d2044b04 in waitOnGroup (group=0x7f7d89809928, isMoveQuery=false) at resgroup.c:2910
#12  0x00005561d20428c2 in groupAcquireSlot (pGroupInfo=0x7ffe673fa120, isMoveQuery=false) at resgroup.c:1950
#13  0x00005561d2043f2f in AssignResGroupOnMaster () at resgroup.c:2649
#14  0x00005561d18f478d in StartTransaction () at xact.c:2701
#15  0x00005561d18f5b53 in StartTransactionCommand () at xact.c:3697
#16  0x00005561d1e1fb07 in start_xact_command () at postgres.c:3307

But in this new state the process is kind of terminating - the global variable proc_exit_inprogress is set to true therefore new TERM signals will be ignored.

I suppose we could think about some NOTICE message illuminating why we cannot immediately terminate backend after receiving SIGTERM. This could be achieved by checking proc_exit_inprogress flag.

@maksm90 I think that pg_stat_activity information is enough for this case. Backend with temp tables, that hangs on acquiring resource group slot after SIGTERM, still has ResourceGroup in wait_event_type column. May be it is a good case for our CC (but it has some problems with detecting DML commands - i.e. temp table rollback - but we can think about it).

I plan to close this PR and reopen it to upstream Greenplum. In that PR I'll ask about a proper way to inform users about a process, that handles SIGTERM and is waiting for a new slot. Is pg_stat_activity is enough for that or not?


blue-green stopped working in us-east

I have a strange issue: blue-green deploy does not work after moving to us-east-1 region.
After playing around I found that ElasticBeanstalk adds region to the EB URLs like my-app.us-east-1.elasticbeanstalk.com. That was not the case in eu-west-1.
I worked around by hacking https://github.com/ThoughtWorksStudios/eb_deployer/blob/master/lib/eb_deployer/deployment_strategy/blue_green.rb#L62
  def active_ebenv?(ebenv)
    str = ebenv.cname_prefix.nil? ? nil : ebenv.cname_prefix.gsub(/\..*/,'')
    str == @component.cname_prefix
  end

This solves the issue for me - but it's not the way it should be implemented I guess ;-)

Blue/green is working for me in us-east using 0.6.6 .  The dns naming change was announced back in January and presumably has rolled out to all regions by now.

I have a similar issue. I recently renamed our EB application (in ap-southeast-2 region) and re-ran the deployer where the only config change was a new application name. This created a new environment correctly, but subsequent deploys always seem to do an inplace update. The output shows eb_deployer trying to do a describe on a second env which doesn't exist but then proceeds to deploy to the original instead of creating the new environment.
I tried cloning the original environment and naming it after the search string from the output, but the behaviour remained the same.

This issue should already been fixed 0.6.4 (released back to Jan 2016)
@darkoz and @hoshy, do you still have this problem? Which version of eb_deployer are you guys using?

@wpc I use eb_deployer (0.6.6) - yes problem still exists.

@hoshy:
Can you take a look at your environment url, is that exactly match ${application-name}-${environment-name}.us-east-1.elasticbeanstalk.com?

@wpc yes the problem persists - I'm using version 0.6.6.
If the app name is something like xyz-website and my environment name (in the yml) is production this generated and env in EB called production-a-1234567 with the URL xyz-website-production.ap-southeast-2.elasticbeanstalk.com
When the deployer runs it searches for env production-b-7654321 doesn't find it and decides to deploy to env a.
Relevant logs:
[2016-07-14T06:04:17.529481 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 1.085341 0 retries] describe_environments(application_name:"xyz-website",environment_names:["production-a-1234567"])
[2016-07-14T06:04:17.646081 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 0.116127 0 retries] describe_environments(application_name:"xyz-website",environment_names:["production-b-7654321"])
[2016-07-14T06:04:17.867926 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 0.221493 0 retries] describe_environments(application_name:"xyz-website",environment_names:["production-a-1234567"])
[2016-07-14T06:04:17.929711 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 0.061392 0 retries] describe_events(max_records:1,application_name:"xyz-website",environment_name:"production-a-1234567")
[2016-07-14T06:04:19.260271 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 1.330172 0 retries] describe_environments(application_name:"xyz-website",environment_names:["production-a-1234567"])
[2016-07-14T06:04:22.659676 #15192]  INFO -- : [Aws::ElasticBeanstalk::Client 200 3.397742 0 retries] update_environment(environment_id:"${production-a-1234567-id}",version_label:"<IP_ADDRESS>",option_settings:[{...}],tier:{name:"WebServer",type:"Standard"})

@wpc the URL is ${application}-${environments.name}-${components.name}.us-east-1.elasticbeanstalk.com


[BUG] - Can not run successfully on m1 Mac using macOS Ventura 13.4's default terminal

Description
Can not run successfully on m1 Mac using macOS Ventura 13.4's default terminal
Steps to reproduce
download recent release
run termscp from system's term(will perform ok on iterm)
termscp ended
Steps to reproduce the bug you encountered
Expected behaviour
A clear and concise description of what you expected to happen.
Environment
m1 Mac using macOS Ventura 13.4 with system term app
Log
Additional information

Problem solved, when I change default terminal settings
Screen size's Number of rows to 25(before is 20)


Use annotate-snippets for rustc diagnostic output

Metadata

Owner(s)
@estebank, @Muscraft

Team(s)
compiler

Goal document
2024h2/annotate-snippets

Summary
Switch to annotate-snippets for rendering rustc's output, with no loss of functionality or visual regressions.
Tasks and status

[ ] Reach output parity of rustc/annotate-snippets

[ ] Port a subset of rustc's UI tests (@Muscraft)
[ ] Make list of current unnaddressed divergences (@Muscraft)
[ ] address divergences (@Muscraft)

[ ] Initial use of annotate-snippets

[ ] update annotate-snippets to latest version
[ ] teach cargo to pass annotate-snippets flag (@estebank)
[ ] add ui test mode comparing new output
[ ] switch default nightly rustc output

[ ] Production use of annotate-snippets

[ ] switch default rustc output
[ ] release notes
[ ] switch ui tests to only check new output
[ ] dedicated reviewer (compiler )

[ ] Standard reviews (compiler )

The last few weeks I have been trying to get suggestions working within annotate-snippets, and have been able to match rustc suggestion output in most cases. Most of my time was spent trying to understand the different rendering styles for suggestions, and how they should be adapted to fit within annotate-snippets.

I have been a bit busy over the past two months, so I haven't had as much time to work on annotate-snippets as I had hoped. That said, progress was still being made!
I continued working on suggestions, but as I got deeper into the implementation, I realized getting the corner cases correct would be very difficult with the annotate-snippets architecture. I started work on a more direct translation from rustc, and quickly ended up with something that closely matched it at the cost of significant differences from how annotations are rendered. These differences, and some problems I noticed with the API for suggestions, led to a discussion with @epage about how to resolve these problems. During the discussion, I realized it would be beneficial if the internals of annotate-snippets resembled rustc as it would make it easier for contributors to rustc to contribute to annotate-snippets. @epage was helpful enough to design a new API, which resolved my suggestion API concerns and should make it easier for annotate-snippets's internals to resemble rustc. Initial work on the new design has yielded positive results, so I am hopeful this was a good decision. During all of this, I started to work on a tool that should help convert some of rustc's UI tests to be used in annotate-snippet's testsuite.


ROX-24858: Replace "No results" text in summary cards

Description
Updates the text for severity and status summary cards to be less cryptic and to explicitly describe what type of results are hidden, or have a zero count.
e.g.
"Results hidden" => "Critical hidden"
"Results hidden" => "Fixable hidden"
"No results" => "0 Important"
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Results with values == 0 and values > 0:

Results with values hidden via filters:

ocp-4-12-e2e-tests failed to create cluster


Icon in Toast on Mobile is misaligned when the text is one line

Describe the bug
The Icon on the Toast on mobile is misaligned if the text is just one line but if it is two it aligns pretty well
To Reproduce
Steps to reproduce the behavior:

Go to 'JustD'
Click on '....'
Scroll down to 'Status.'
Click on 'Each Toast'
See the issue (on Mobile)

Thanks for let me know. Will fix in the next update.

This issue is been resolve on #198


Arch board comments

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Storage.Files.Shares


Update the tutorial to reflect the changes in the names and the style

Closes #512
(Wait for #511 to be reviewed)
docs preview

Codecov Report
Attention: Patch coverage is 82.14286% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 92.90%. Comparing base (d123e9c) to head (f34c320).

Files
Patch %
Lines

include/micm/solver/rosenbrock.inl
84.61%
2 Missing :warning:

include/micm/jit/jit_compiler.hpp
0.00%
1 Missing :warning:

include/micm/process/process_set.hpp
0.00%
1 Missing :warning:

include/micm/process/rate_constant.hpp
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #513   +/-   ##
=======================================
  Coverage   92.90%   92.90%
=======================================
  Files          42       42
  Lines        3270     3270
=======================================
  Hits         3038     3038
  Misses        232      232

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


Update example to use better initial value

The example above this in the documentation states
        // semantically, the monotonic timer is frozen at time "zero" during `init`
        // NOTE do *not* call `Instant::now` in this context; it will return a nonsense value
        let now = cx.start; // the start time of the system

It results in weird scheduling issues, but still eventually works.  cx.start is much more reliable.
Relates to https://github.com/rtfm-rs/cortex-m-rtfm/issues/196

Thanks for fixing!

bors r+


Adds Marathon support.

What does this PR do?
Adds Marathon support.
Motivation
To have the Marathon provider.
More

[x] Added/updated tests
~[ ] Added/updated documentation~

Additional Notes

Co-authored-by: Julien Salleyron<EMAIL_ADDRESS>

Hi all, I have tested the marathon provider and I keep having 504 error. It worked fine with docker provider but that's my requirement to go to production.
reverse-proxy_1  | time="2019-12-11T14:25:02Z" level=debug msg="'504 Gateway Timeout' caused by: dial tcp <IP_ADDRESS>:31968: i/o timeout"

forceTaskHostname: true did not help.
Issue reported here. I am still googling to get more tutorial with marathon but there's really nothing.
Thanks for sharing, helping and best!


[Consensus] Change the state synchronization flow

State Synchronization without a target
Synchronization to the specific target LedgerInfo has a number of disadvantages:

introduces non-trivial complexity for state synchronizer, which needs to support two different flows for full nodes and validators.
unnecessary slowdown when syncing to the "not latest" LI
opens consensus to the state sync byzantine attack (referenced in #1324).

We'd like to approach this problem in a different way.
StateSynchronizer will try to synchronize to some LedgerInfo given the potential peers:
/// State synchronizer returns when it successfully fully synchronizes to **one** of the given peers.
/// Full synchronization means that all the retrieved transactions in the local storage
/// correspond to the latest ledger info sent by the remote peer.
/// The returned value is a pair of the latest ledger info sent by the given remote peer and its
/// peer id.
/// Note that state synchronizer is not going to return unless it succeeds to fully synchronize to
/// some peer. It's going to retry synchronizing with different peers from the list until it
/// eventually succeeds. TODO: should we have timeout instead?
StateSynchronizer::sync_to(peers: Vec<PeerId>) -> (LedgerInfoWithSignatures, PeerId)

Thus the SyncManager will run the following retry loop upon state synchronization:
SyncManager::synchronize() method:
// This method terminates when it succeeds to synchronize to one of the peers that signed
// the given LedgerInfo
Retry until succeed:
 - invoke state synchronizer `sync_to()`
 - retrieve the 3-chain for the returned LedgerInfo from the given peer
 - rebuild the BlockTree

Note that SyncManager::synchronize() does not guarantee that it'll be able to synchronize to a given LedgerInfo. This is our way to make sure the system is not stuck by a state sync byzantine attack described in #1324.
SyncManager::synchronize() might make little progress / no progress at all in case a Byzantine remote validator provides an outdated ledger info. In today's implementation it'd be a responsibility of a StateSynchronizer to retry different peers until it reaches the target. In the new implementation we're going to log the fact that a given peer provided some old ledger info and just move on. It's absolutely ok to not be able to do state synchronization and to drop the message processing for any incoming message. At some point we'll be able to pick an honest peer and catch up.
Restartability logic
Today we're maintaining an invariant that the root of a block tree in ConsensusDB is either equal or higher than the highest ledger info in LibraDB. This invariant is not going to hold in the new solution because we're going to first retrieve the transactions and then retrieve the 3-chain for the block tree.
We don't need this invariant. We're not using it anywhere.
Everything will continue working as usual without it. The only addition we need to have is the list of peers to resume the state synchronization in case we crashed in the middle of state sync (the pending list of peers we're syncing with will be stored in ConsensusDB for the duration of the state synchronization).

Does the consensus system share it's HLI with the state synchronizer at all? At least the way I'm reading this, in this design state synchronizer has a completely different view of the universe in terms of what the latest commit is.
I guess my intuition was somewhat different here -- that we might want to think about how to unify the knowledge of the HLI. For example, if during state sync we learn about some HLI, it seems like we'd ideally want to inform consensus ASAP -- even if we haven't synced to that round we'd want to avoid any kinds of votes / processing on information in that round. I guess my intuition is that there's three basic pieces of info in the system -- highest synced LI (the HLI that is stored locally) and the Highest Known LI which is the highest commit we know about (but haven't synced to) and the Highest QC and that one component could track these pieces of info across the entire node.

In the new implementation we're going to log the fact that a given peer provided some old ledger info and just move on

Can you clarify this. How do you know it's old?

retrieve the 3-chain for the returned LedgerInfo from the given peer

I still don't quite understand the logic of this part of syncing (why it's important that the specific 3-chain for the commit be synced) but probably easier to discuss the next time we chat in person :-)

Consensus cannot operate without a valid block tree, which at the very least should have a root.

Is this a fundamental limitation to our on-chain structures, or a limitation of the blocktree today? i.e. is there a strategy one can use to propose a new block with the invariant that you never request to download a block that you know to be committed.
I'm wondering if the following approach works: a leader in round R must be able to assemble a block that has the HQC as it's parent. It seems one could build a block on top of the HQC by (1) synching to the highest committed block (2) syncing backwards from the HQC until you reach a block where the block's parent_id == the block_id in the HLI (thus avoiding downloading the committed block)

@bmaurer oh I think we are kind of talking about the same thing then. Do you think the differences boil down to whether we're building a block tree eagerly or lazily? (our current paradigm is to build a block tree in an eager manner, while you say we could try to build it lazily on demand)

I think some of the limitation comes from the pipelining, if we operate as 1-chain instead of 3-chain like block <- ledger_info <- block <- ledger_info, then the only thing we need to know is the highest ledger info, we had some discussion about the perf implications of this, it's not very clear what we get from the pipelining compared to make the block size 3x bigger.
For example, for reconfiguration block we gave up the pipelining and thus are able to only sync HLI without 3-chain to continue consensus(consensus would construct new genesis from the HLI), but it shows that the 3-chain requirements are there only for the two pending blocks.
The committed block is not required but rather optimization to maintain the tree property, we can retrieve 2-chain instead but make block store a forest structure.

Unfortunately we realized that the new flow might not work well with the way Consensus is planning to do epoch changes.
Consider the following scenario: a node at epoch n receives a proposal that is still at epoch n but with a higher version that triggers state synchronization. When a remote peer receives a ChunkRequest it might have already moved to epoch n+1. In the new flow it would mean that the state synchronizer switches to the next epoch within the same EventProcessor.
However, consensus is now trying to make sure that EventProcessor corresponds to a specific epoch: the epoch switches should happen at the level above EventProcessor only. The old flow allowed us to control the fact that we never switch epochs within the code of EventProcessor.


[HUDI-5296] Allow disable schema on read after enabling

Change Logs
If someone has enabled schema on read by mistake and never really renamed or dropped a column. it should be feasible to disable schema on read. This patch fixes that. essentially both on read and write path, if "hoodie.schema.on.read.enable" config is not set, it will fallback to regular code path. It might fail or users might miss data if any they have performed any irrevocable changes like renames. But for rest, this should work.
Impact
Users can now disable schema on read feature if need be.
Testing:
Manually tested that we could able to read a table which had schema on read enabled. after disabling, upserts succeed.
Risk level (write none, low medium or high below)
low.
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@alexeykudinkin : you can review the patch. for some reason, I could not update the older patch https://github.com/apache/hudi/pull/7333 and so have created a new one.

@alexeykudinkin : addressed all comments


On new installations, do not display warning (missing Settings.php)

On the first installation of ElkArte 1.1.0 beta1, despite following the documented installation steps precisely, the following warning message displays:

Warning!
It looks like Settings.php and/or Settings_bak.php are missing from the default directory of your forum, ElkArte will try to rename the sample files provided with the installation. If this operation fails, please rename Settings.sample.php and Settings_bak.sample.php respectively to Settings.php and Setting_bak.php before running this script.

While this warning suggests that an unexpected situation (may have) occurred, this is actually the expected situation on a new installation. As such, I suggest to modify the behaviour of the install script in a way which either

does not display this warning on a first install (if this situation can be reliably detected, such as by a shipped-by-default lock file which is later removed - or its content changed - as part of a successful installation process)
extends this warning to state that this situation is to be expected on a new installation
downgrades the warning to a hint / note, i.e. replace the 'warning' icon by 'hint' icon, replace the yellow background by white or grey or similar.

Ideally, during a successful standard installation which does not run into any unexpected situations, a user should not get to see any warnings at all.
Note: This is obviously just a very minor bug or feature request. I am unable to tag it as such, though.

Yeah, I know that one. :innocent:
TBH, I put the warning, mainly because I couldn't decide what was the best way to deal with it and then I forgot about it. lol
My first idea, was to suggest in the documentation to rename the files before uploading them, so in theory the warning was not shown to people that renamed the files (so it was legit to say "pay attention, you didn't what was suggested, I'll try to fix your behaviour, but I may not be able to"), then I started wondering if it was really worth (people usually tend to avoid reading documentation, I know very well because I'm one of them :angel: ) and never put any real warning anywhere, but at the same time I also forgot to ask for opinions on the matter, and nothing moved. :crying_cat_face:
Well, this is a good chance. :grinning:


oiiotool: work around static destruction order issue (#3295)

Description
In some cases (at least on Windows, with static linking) the oiiotool main returns zero, but the full process returns 3 error code due to abort() being called. And that is called because a global shared image cache is already destroyed, but the image references from another global (Oiiotool itself) try to use it, get a call into a "pure virtual function" stub that is put by MSVC C runtime into a destroyed object vtable, and that ends up in an abort().
Work around that by making sure various member variables holding ImageRecRef are cleared before main() finishes and global destructors are called.
Tests
N/A -- but it does make running OIIO test suite locally much easier for me. Previously python runners that were launching oiiotool were getting exit code 3 when "everything was fine", so I had to resort to running their final command lines manually. Which is quite cumbersome!
Checklist:

[x] I have read the contribution guidelines.
[ ] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[ ] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Hmmm, this also makes me think that it probably would have been better for ImageCache::create to have returned a ref-counted pointer. And perhaps for "ot" to be dynamically allocated rather than static, so we can explicitly destroy it before oiiotool exits.

Does this patch fully address #3295 ?

Does this patch fully address #3295 ?

It does for me, but I dunno if it does for the original bug reporter. My guess would be on "yes", from their posted callstack.


Deployment set up of additional components on d106d6-PROD  (Web, Zipper )

Title of ticket:
Description
The purpose of this task is to analyze, re-organize the resources, and set up deployment for  Document Reviewer Web, Zipper and/or any other components missing( technically all other components are already on PROD)
Tip :  Can consider #4585 as well with this task
Note : As discussed during the sprint planning on 11-OCT, @abin-aot 's personal suggestion is to have vanity URL for FOI Document Reviewer Web app after 9.0 release. since it  updates for SSL for the new vanity URL affects existing PROD, DEV, TEST set up , as showed on the meeting. We can plan/request it after 9.0 release , and aim for 9.1 release, till then Business will be seeing regular OCP provisioned URL , similar to what we have with TEST-Marshal URL on the browser address  bar
cc: @sumathi-thirumani-aot , @m-prodan @lmullane
Dependencies
Are there any dependencies?
DOD

[x] Analyze the resource usage(existing) on Dedupe, Conversion, API , DB on d106d5-prod - Update the team!
[ ] Regular DEV, TEST!!
[ ]  if any one of them has more than 60-70% unused we can release and reuse it on new or any existing components - Need to discuss. Don't think we need to request more resources! - but confirm!
[ ]  Set up yaml, ENV VAR, Vault integrations
[ ]  Can be do deployment for Web, Zipper Ahead of time ? I feel yes, but do an impact analysis, update team then we can action.
[ ] Update CORS setting on S3 Buckets with the new Web URL. Can consider #4585 as well with this task

Steps:

Resource Optimization

[ ] d1***-prod
[ ] d1***-test
[ ] d1***-dev

Private Zenhub Image

Updated all S3 Buckets with new Web URL for CORs Config


getDirectoryFiles should be clear and tested

https://github.com/snowleopard/shaking-up-ghc/issues/117#issuecomment-169290771 shows the confusion over what happens when which directories don't exist. Looking through, I didn't see enough tests, so a ticket to add haddock docs and tests.

An additional question about the lint error behaviour: if I run getDirectoryFiles "" [dir -/- "*.o"] and dir does not exist at this point (so the result is []), but it is created later in the build process, will lint also complain? Presumably it should complain about getDirectoryFiles returning a different result, not about doesDirectoryExist as in my case.

Yes, it will complain. So it ultimately leaves you with exactly the same error as before in your case.

Thanks. That's good. I think the error is also going to be more useful (it'll show the newly added files).

The test I used to answer these questions was:
    putNormal . show =<< getDirectoryFiles "C:/spacework/shake/docs" ["*.md"]
    putNormal . show =<< getDirectoryFiles "C:/spacework/shake" ["docs/*.md"]
    putNormal . show =<< getDirectoryFiles "C:/spacework/shake/doc" ["*.md"]

I am confused by this travis log: https://api.travis-ci.org/jobs/102204258/log.txt?deansi=true. The built ended with the following error:
* getDirectoryFiles  [.build/stage0/gmp/objs/*.o]
.build/stage0/gmp/objs: getDirectoryContents: does not exist (No such file or directory)

Here is the corresponding piece of code:
getDirectoryFiles "" [gmpObjects -/- "*.o"]

I thought that there shouldn't be any error if the directory does not exist with the above code. Am I mixing up things again? My current understanding is that the following should error instead:
getDirectoryFiles gmpObjects ["*.o"]

/cc #168

Your understanding matches mine. I will extensively test and audit this tomorrow morning and let you know if we were correct...

Hmm, I had some optimisations in Shake released version to do faster searches for concrete prefixes. I wonder if I screwed that up...

@snowleopard - confirmed, this is incorrect in the released Shake and got fixed when I rewrote the directory search optimiser, but I didn't realise it got fixed.

OK, I'll use a workaround with doesDirectoryExist while we are waiting for the next Shake release.

Now documented and tested.


Wind Tunnel scenario packaging

To deploy Wind Tunnel scenarios, a mechanism is needed to package them. The target environment is really any distributed compute environment but for now, we'll focus on NixOS machines and assume that they do not have the resources to build scenarios from scratch.
For context, each Wind Tunnel scenario is a binary which needs to be run on 1 or more machine. It may optionally have 1 or more happs that need to be built and provided with the scenario.
Either this needs to be bundled in a way that can be unpacked and run on a NixOS machine, or it needs to be possible to a partial build of the Wind Tunnel project that only builds the scenario and happs that are needed.

Best estimate for this issue is 3 days.


NuGet package 1.2.1 doesn't have the config transform

Hi, the NuGet package 1.2.1 doesn't have the configuration transform. So the clientresources give 404 and you need to manually add the "add name="geta-epicategories" to the protected modules.

I will fix this ASAP.

I recently uploaded version 1.2.3 which closes this issue. Should be available in the next few days.

Thanks by the way! :)

You are welcome :)


IPFS plugin

IPFS plugin could hook into a local IPFS daemon. Could offer features like dht findprovs and output if a CID is seen, and other helpful IPFS features which could be useful in a public chat setting.

This is a good idea, especially if people are always asking questions in the ipfs discord like "is my file even on the dht?"

This was resolved in 52833b1b5f54fdb76c2a623913cde946302c0b41, just forgot about this issue :)


Add assert_err_eq! macro

Introduces an assert_err_eq! macro, which asserts that a Result value is an Err and that its contained value is equal to the provided value. Also defines a debug_assert_err_eq! macro.
This is very similar to the assert_ok_eq! macro, and I think this nicely fills out the macros provided by this library. Similarly, the debug_assert_err_eq! macro is very similar to the debug_assert_ok_eq! macro.
This closes #7.

@svartalf friendly ping, would love to know your thoughts on this :)

The claims crate (which is a fork of this crate) has these macros merged. Given that this crate appears to be abandoned, I'm closing this. If anyone is looking for these macros, use claims instead.


Review error detection and correction unit plan and lessons

This involves checking the  error detection and correction unit plan and three lessons to check their:

Name
Learning outcomes
Text
Resources
Related programming challenges

Created after splitting #217.

Reviewed no changes.


How to prepare the data format?

I have read the Medium example and executed the example. I understand from the README that X_df must contain the columns ['unique_id', 'ds', 'x'] and y_df must contain the columns ['unique_id', 'ds', 'y'].
Now in my case I have:
     time              usage        requests  hits
5    1.589360e+12  1575.6520        5074.0  1588.0
12   1.589360e+12  1575.6520        5074.0  1588.0
19   1.589360e+12  1527.7666        5042.0  1580.0
26   1.589360e+12  1527.7666        5042.0  1580.0
33   1.589360e+12  1477.7880        5297.0  1584.0
...

and I want to predict usage. How should I map my data to the required input format? I can only think of time getting mapped to ds.
I have read many issues (such as this one), but I wasn't able to figure out how to map my data.

Here is Google Colab N-HiTS example.
You should rename your 'usage' -> 'y'.

Is it really as simple as:
X_df['ds'] = X_df['time'].values
y_df['ds'] = y_df['time'].values
X_df['x'] = X_df['bw'].values
y_df['y'] = y_df['bw'].values
X_df['unique_id']='dummy'
y_df['unique_id']='dummy'

# same for test data

@kdgutier?
Train compltes, but when I try to predict:
y_hat_df = model.predict(X_test_df)
I get:
ValueError: You are trying to merge on float64 and datetime64[ns] columns. If you wish to proceed you should use pd.concat

which I think happens because of some incompatibility between time and ds perhaps, which is another issue I guess.

I recommend to convert your 'time' column to a date stamp using the pd.to_datetime, a lot of methods in the Neuralforecast library rely on you sending datetime formatted 'ds'.

@kdgutier apologies for the late response because of the weekend. OK I think we are very close, but now predict crashes with batch size (=32).  Test size (1723) is greater than batch size (32).

The dataloader takes into consideration the number of 'unique_id', you have all of them as 'dummy' in your example.

Because the maximum batch_size that the dataloader can use here is 1, the ESRNN is a recurrent network.
It needs to sequentially visit all the observations of your series.

I suggest you to move to the NeuralForecast library, and try running the N-HiTS example.


S3 strategy inconsistent hash for gzipped files

Hi,
For some of the gzipped files local hash is different from one calculated by AWS S3 (ETag).
I worked it out that it is due to the following line (in strategies/base.py):
zf = gzip.GzipFile(mode="wb", compresslevel=6, fileobj=buffer, mtime=0.0)
There is a mismatch between compression levels (potentially a boto3 change), because currently the default one is 9. And the following code is being used in django-storages (1.8.0) to upload a compressed file into s3 bucket:
    def _compress_content(self, content):
        """Gzip a given string content."""
        content.seek(0)
        zbuf = io.BytesIO()
        #  The GZIP header has a modification time attribute (see http://www.zlib.org/rfc-gzip.html)
        #  This means each time a file is compressed it changes even if the other contents don't change
        #  For S3 this defeats detection of changes using MD5 sums on gzipped files
        #  Fixing the mtime at 0.0 at compression time avoids this problem
        zfile = GzipFile(mode='wb', fileobj=zbuf, mtime=0.0)
        try:
            zfile.write(force_bytes(content.read()))
        finally:
            zfile.close()
        zbuf.seek(0)
        # Boto 2 returned the InMemoryUploadedFile with the file pointer replaced,
        # but Boto 3 seems to have issues with that. No need for fp.name in Boto3
        # so just returning the BytesIO directly
        return zbuf

So if I remove compression level entirely, it works as expected:
zf = gzip.GzipFile(mode="wb", fileobj=buffer, mtime=0.0)
Would it be possible to adjust it?

Created a PR, however not 100% if it will affect other strategies (Google, Azure, etc). Please review.


[#noissue] Implement FormFieldErrorMessage component

Extract the displaying form field error message template into a shared component

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

Codecov Report

Merging #5256 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5256      +/-   ##
==========================================
+ Coverage   40.22%   40.24%   +0.02%
==========================================
  Files        2684     2684
  Lines       80989    80989
  Branches    10859    10859
==========================================
+ Hits        32581    32598      +17
+ Misses      45488    45469      -19
- Partials     2920     2922       +2

Impacted Files
Coverage Δ

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...pinpoint/profiler/instrument/AccessorAnalyzer.java
57.14% <0%> (-8.58%)
:arrow_down:

...vercorp/pinpoint/web/vo/ApplicationAgentsList.java
66.66% <0%> (-2.57%)
:arrow_down:

...corp/pinpoint/rpc/stream/StreamChannelManager.java
70.68% <0%> (-1.73%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.61% <0%> (+0.31%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.86% <0%> (+1.04%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
63.82% <0%> (+2.12%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0%> (+6.42%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
66.66% <0%> (+11.11%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Big Sur : screenshot issue

I want to capture the chrome browser screen
However, screen capture does not work properly on Mac OS Bic Sur.
If you take a screen capture while the Chrome browser is active, the background screen becomes a screen capture.

Maybe because macbook is working with different desktops? Or chrome is running in a dedicated window? You can check by swiping 3 fingers up on the touchpad

I had the same issue and finally resolved it.
You need to change Security & Privacy settings in MacOS to give Screen Recording permission to your application.
For more details, please refer to the below issue.
https://github.com/BoboTiG/python-mss/issues/134


Disable alt focusing menubar

This is a followup of #185739, where I never want to have a Alt press do a focus change out of the editor or what I'm doing at the moment.
I tried tweaking menuBarVisibility, and only hidden avoids Alt from changing the focus
BUT [❌] it also completely hides the menus and they can't be used with the mouse at all anymore.
I also considered having a keybind to cycle the menubar visibility as suggested in #185739, but I already have a lot of keybindings and I don't want to use a valuable key for that.
Also since I want to use the menus only with the mouse I would prefer to have the menus directly accessible.
I want to still be able to use the menus with the mouse, but have a Alt press never loose the current focus.
Maybe through a new setting window.altPressFocusesMenuBar or something similar?
Thank you for your consideration 🙏

Nevermind, I've found the following settings ❤
    "window.customMenuBarAltFocus": false,
    "window.enableMenuBarMnemonics": false,


feat(auth): expose linkWithPopup() & correctly parse credentials in exceptions

Description
Expose linkWithPopup() for web
Even though already implemented, the firebase_auth_web library did not expose the JS method authUser.linkWithPopup().
This PR exposes the method for use in the firebase_auth plugin.
If the documentation of the new method is not sufficient, please point me to the correct place to copy from, as I haven't found any JS doc.
Correctly parse credentials in exceptions for web
The method getFirebaseAuthException(Object exception) always used the convertWebAuthCredential function to convert the credentials from firebaseError.credential.
Converting the exception credentials like this isn't sufficient for OAuthClients, as the credentials do contain more information.
This information is now correctly returned, as the convert method was changed to convertWeb**O**AuthCredential for auth_interop.OAuthCredential
Related Issues
No issue was raised from my side, and I haven't found any fitting.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

I actually told him the same 😄
Anyways, I fixed it @russellwheatley - would be cool if you could approve and merge then 🚀
@Sese-Schneider is actually oof right now, so that would be a welcome surprise for him to see I bet 🙃

:wave:
Do you think you could add automated tests? Thanks 🙏

@rrousselGit I asked the same, but it seemed like that was not possible with any reasonable amount of effort.

As this isn't something we can create an e2e test for, the next best thing is to screenshot results as proof of concept. I went ahead and ran a test on the firebase_auth example app which demonstrates successful popup dialogue box:

and the resulting signed-in user:

@russellwheatley thank you very much 😍  - implementing e2e tests for that would have been really difficult. 😄
@creativecreatorormaybenot also thank you for updating the implementation while I was oof ❤


MGMT: Editing a service causes it to be duplicated.

Add a new service (doesn't matter what type) and save it.
Edit the service once added and change its name, description, service id, etc. (it seems to not matter at all if there are any changes. Clicking save would also just do)
Click save again. The service is duplicated.

It seems that objects passed through are always treated as strings no matter how you cast it.  This was an error in the duplicate logic.  The fix has been pushed into the mgmt-domain-regex branch

Thanks. Going to merge the branch and then cut Rc3.