H2O doesn't initalize on Yarn Managed Spark Cluster

I am trying to use rsparkling.
I am executing the following code after loading up my sc
h2o_context(sc, strict_version_check=FALSE)
Every time I try to connect to H2O, I get the following error in my spark log:
*** Your H2O version is too old! Please download the latest version <IP_ADDRESS> from http://h2o.ai/download/ ***IP address not found on this machine
I get the following error in the R Console:
Error in invoke_method.spark_shell_connection(sc, TRUE, class, method,  :
No status is returned. Spark R backend might have failed.
I have tried this with both H2O  <IP_ADDRESS> and the current version I am running <IP_ADDRESS>
Information about my session is below:
R version 3.3.0 (2016-05-03)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.2 LTS
locale:
[1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
[3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
[5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
[7] LC_PAPER=en_US.UTF-8       LC_NAME=C
[9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] caret_6.0-76       lattice_0.20-35    h2o_<IP_ADDRESS>       lazyeval_0.2.0
[5] stringr_1.2.0      RColorBrewer_1.1-2 data.table_1.10.4  rsparkling_0.2.0
[9] purrr_0.2.2        readr_1.1.0        tidyr_0.6.2        tibble_1.3.0
[13] ggplot2_2.2.1      tidyverse_1.1.1    dplyr_0.5.0        sparklyr_0.5.4
[17] Rook_1.1-1         RJSONIO_1.3-0      Cairo_1.5-6

Hi @atroiano
Do you have logs to share from this?
Cheers,
Navdeep

17/05/10 11:52:53 INFO cluster.YarnScheduler: Removed TaskSet 12.0, whose tasks have all completed, from pool
17/05/10 11:52:53 INFO scheduler.DAGScheduler: ResultStage 12 (foreach at InternalBackendUtils.scala:175) finished in 0.040 s
17/05/10 11:52:53 INFO scheduler.DAGScheduler: Job 10 finished: foreach at InternalBackendUtils.scala:175, took 0.047403 s
17/05/10 11:52:53 INFO internal.InternalH2OBackend: Starting H2O client on the Spark Driver (<IP_ADDRESS>): -name sparkling-water-cdsw_1521177490 -ga_opt_out -quiet -log_level WARN -log_dir /home/cdsw/h2ologs -baseport 54321 -client -ip <IP_ADDRESS> -flatfile /tmp/1494417173025-0/flatfile.txt
17/05/10 11:52:53 INFO reflections.Reflections: Reflections took 223 ms to scan 9 urls, producing 172 keys and 1190 values
17/05/10 11:52:53 INFO reflections.Reflections: Reflections took 86 ms to scan 5 urls, producing 132 keys and 661 values
*** Your H2O version is too old! Please download the latest version <IP_ADDRESS> from http://h2o.ai/download/ ***
IP address not found on this machine
17/05/10 11:52:53 INFO spark.SparkContext: Invoking stop() from shutdown hook
17/05/10 11:52:53 INFO server.ServerConnector: Stopped ServerConnector@2e4b345d{HTTP/1.1}{<IP_ADDRESS>:27025}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@2bacfa90{/stages/stage/kill,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@34cbbc24{/api,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@262c84f0{/,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@6d4e2820{/static,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@685fdba8{/executors/threadDump/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@7bc979dd{/executors/threadDump,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@777e294c{/executors/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@7fc75485{/executors,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@6a784c06{/environment/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@54e49324{/environment,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@52f121d6{/storage/rdd/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@555b2559{/storage/rdd,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@5e8693f2{/storage/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@370f8f43{/storage,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@cb8c6fb{/stages/pool/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@52678e71{/stages/pool,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@2b89889b{/stages/stage/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@1eea5940{/stages/stage,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@6aab7666{/stages/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@8e8c823{/stages,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@2103edac{/jobs/job/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@58c9af0e{/jobs/job,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@282cab3{/jobs/json,null,UNAVAILABLE}
17/05/10 11:52:53 INFO handler.ContextHandler: Stopped o.s.j.s.ServletContextHandler@6537120f{/jobs,null,UNAVAILABLE}
17/05/10 11:52:53 INFO ui.SparkUI: Stopped Spark web UI at http://<IP_ADDRESS>:27025
17/05/10 11:52:53 INFO cluster.YarnClientSchedulerBackend: Interrupting monitor thread
17/05/10 11:52:53 INFO cluster.YarnClientSchedulerBackend: Shutting down all executors
17/05/10 11:52:53 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Asking each executor to shut down
17/05/10 11:52:53 INFO cluster.SchedulerExtensionServices: Stopping SchedulerExtensionServices
(serviceOption=None,
services=List(),
started=false)
17/05/10 11:52:53 INFO cluster.YarnClientSchedulerBackend: Stopped
17/05/10 11:52:53 INFO spark.MapOutputTrackerMasterEndpoint: MapOutputTrackerMasterEndpoint stopped!
17/05/10 11:52:53 INFO memory.MemoryStore: MemoryStore cleared
17/05/10 11:52:53 INFO storage.BlockManager: BlockManager stopped
17/05/10 11:52:53 INFO storage.BlockManagerMaster: BlockManagerMaster stopped
17/05/10 11:52:53 INFO scheduler.OutputCommitCoordinator$OutputCommitCoordinatorEndpoint: OutputCommitCoordinator stopped!
17/05/10 11:52:53 INFO spark.SparkContext: Successfully stopped SparkContext
17/05/10 11:52:53 INFO util.ShutdownHookManager: Shutdown hook called
17/05/10 11:52:53 INFO util.ShutdownHookManager: Deleting directory /tmp/spark-2fcce6e9-85d7-4f92-9666-72718535c565

Hi @atroiano,
could you also share the full script you used to install RSparkling?

@jakubhava  I just used the standard install.packages("rsparkling")

Hi @atroiano
Can you share the complete script you used and also the version of h2o, sparklyr, and rsparkling?

my script is pretty basic because it's not working
config = list("spark.dynamicAllocation.enabled" = "false",
"sparklyr.shell.executor-memory" = "4G",
"sparklyr.shell.driver-memory"   = "16G",
"sparklyr.shell.executor-cores" = '32',
'sparklyr.shell.num-executors' = '5')
sc<-spark_connect(master="yarn-client"
, version = "2.0"
, spark_home="/spark2"
#, config = config
)
h2o_context(sc)
Versions - H20 - h2o_<IP_ADDRESS>, sparklyr - sparklyr_0.5.4, rsparkling - rsparkling_0.2.0

hi @atroiano
Sorry for the late update! How are things going now? Still the same issue?

Yeah,  I am diving back into it and I am getting a new issue now.  If I connect to spark without rsparkling loaded and do spark_version(sc), it says my spark version is 2.1.0 but I get the following message when I run spark_connect with rsparkling loaded.
Spark version 2.0 detected. Will call latest Sparkling Water version 2.0.8
Detected H2O version <IP_ADDRESS>. Please install H2O version <IP_ADDRESS>, which is compliant with the latest Sparkling Water version for Spark 2.0.* ->  Sparkling Water version 2.0.8
To update your h2o R package, copy/paste the following commands and then restart your R session:
detach("package:rsparkling", unload = TRUE)
if ("package:h2o" %in% search()) { detach("package:h2o", unload = TRUE) }
if (isNamespaceLoaded("h2o")){ unloadNamespace("h2o") }
remove.packages("h2o")
install.packages("h2o", type = "source", repos = "http://h2o-release.s3.amazonaws.com/h2o/rel-tverberg/2/R")
I installed the version of h2o that works with spark 2.1.0
when I try to run h2o_conext (just to test), the following error occurs:
Error: org.apache.spark.h2o.WrongSparkVersion: You are trying to use Sparkling Water built for Spark 2.0, but your $SPARK_HOME(=/opt/cloudera/parcels/SPARK2-2.0.0.cloudera2-1.cdh5.7.0.p0.118100/lib/spark2) property points to Spark of version 2.1.0.cloudera1. Please ensure correct Spark is provided and re-run Sparkling Water.
After I did this, I tried to set the sparkingwater version to work with 2.1 using options(rsparkling.sparklingwater.version = '2.1.5') and I get the following error:
Error in spark_dependencies(spark_version = spark_version, scala_version = scala_version)

some of the issues above are also related to this https://github.com/h2oai/rsparkling/issues/33

so I fixed that issue, @jakubhava  @navdeep-G  still having the same issue
Error in invoke_method.spark_shell_connection(sc, TRUE, class, method,  :
No status is returned. Spark R backend might have failed.

@atroiano Do you have logs to share?

@navdeep-G  I'll get these over by tomorrow, have been travelling the past week. Sorry for the delay!

@atroiano Thank you! Please post when you can.

Hi @atroiano - did you ever get the issue resolved? I am having similar issues as your original ones, with the IP address not found on this machine error seeming to be the reason my h2o context is being shut down.
Did you ever manage to resolve it?

@renegademonkey you can try to force client IP via --conf spark.ext.h2o.client.ip=<custom IP>

Thanks @mmalohlava - I've just tried that and get the same result. Even without that conf the InternalH2OBackend process has the correct driver IP, i.e.:
17/11/23 08:21:25 INFO InternalH2OBackend: Starting H2O client on the Spark Driver (XXX.XXX.248.73): -name sparkling-water-rafaelw_app-20171123082106-0040 -nthreads 5 -ga_opt_out -quiet -log_level WARN -log_dir /Users/user/appFolder/h2ologs/app-20171123082106-0040 -baseport 54321 -client -ip XXX.XXX.248.73 -flatfile /var/folders/gl/vgw262w9227cwqvzk595rbvjygdzh8/T/1511425285141-0/flatfile.txt
But still always exits with IP address not found on this machine. I get exactly the same error whether I use sparkling water via r, or integrated into a spark app.

I did not get it resolved.
On Thu, Nov 23, 2017 at 3:28 AM, renegademonkey<EMAIL_ADDRESS>wrote:

Thanks @mmalohlava https://github.com/mmalohlava - I've just tried that
and get the same result. Even without that conf the InternalH2OBackend
process has the correct driver IP, i.e.:
17/11/23 08:21:25 INFO InternalH2OBackend: Starting H2O client on the
Spark Driver (XXX.XXX.248.73): -name sparkling-water-rafaelw_app-20171123082106-0040
-nthreads 5 -ga_opt_out -quiet -log_level WARN -log_dir
/Users/user/appFolder/h2ologs/app-20171123082106-0040 -baseport 54321
-client -ip XXX.XXX.248.73 -flatfile /var/folders/gl/
vgw262w9227cwqvzk595rbvjygdzh8/T/1511425285141-0/flatfile.txt
But still always exits with IP address not found on this machine. I get
exactly the same error whether I use sparkling water via r, or integrated
into a spark app.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/h2oai/rsparkling/issues/31#issuecomment-346555334,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APNWWTwFzn-gFZ0GJjCv2ndDpaYFvM1oks5s5Sy1gaJpZM4NR-6Z
.


Fix week parsing for date parameters/arguments

It looks like #1284 broke some week and week range parsing and was noticed in https://github.com/cmu-delphi/covidcast-indicators/issues/1904.
Heres evidence of the new test failing before i applied the fix:
https://github.com/cmu-delphi/delphi-epidata/actions/runs/6564148018/job/17829935532
This solution is a bit brittle, but ~all of _params.py is a mess and needs to be cleaned up, so i think this can fly for now.

@melange396 thx for fixing that. I didn't even know that we have such time_value format


Events are not triggered

Hi,
My events are not triggered in service.
Here is my code
And does anyone have any ideas about getting tracks duration in queue?!
tracks.push({
          url: BASE_URL + value.path96.replace(/\\/g, "/"),
          id: index
});

 TrackPlayer.updateOptions({
        stopWithApp: true,
        compactCapabilities: [
          TrackPlayer.CAPABILITY_PLAY,
          TrackPlayer.CAPABILITY_PAUSE
        ],
        capabilities: [
          TrackPlayer.CAPABILITY_PLAY_FROM_ID
        ],
        playIcon: require('../../assets/images/player_play.png'),
        pauseIcon: require('../../assets/images/player_pause.png'),
      })
  TrackPlayer.setupPlayer().then(async () => {
        await TrackPlayer.add(tracks);
        TrackPlayer.play()
        this.setState({
          paused: false
        })
      })
    })

And my TrackPlayerService.js
import TrackPlayer from 'react-native-track-player'

const TrackPlayerService = async () => {
    console.log('adding event listeners')
    TrackPlayer.addEventListener('remote-play', () => {
        console.log('remote-play');
        TrackPlayer.play();
        TrackPlayer.getDuration().then((duration) => {
            setStore({
                sliderDuration: duration
            })
        })
    });

    TrackPlayer.addEventListener('remote-pause', () => {
        console.log('remote-pause');
        TrackPlayer.pause();
    });

    TrackPlayer.addEventListener('remote-next', () => {
        console.log('remote-next');
        TrackPlayer.skipToNext();
    });

    TrackPlayer.addEventListener('remote-previous', () => {
        console.log('remote-previous');
        TrackPlayer.skipToPrevious();
    });

    TrackPlayer.addEventListener('remote-stop', () => {
        console.log('remote-stop');
        TrackPlayer.destroy();
    });
};

export default TrackPlayerService;

RN: v0.58.4
OS: Android 9.0 Pie
TrackPlayer: 1.1.3

You should be setting up the player before updating the options.
You shouldn't be updating your store inside the service, that shouldn't cause issues in your case, but it might.
remote- based events are only trigger when the user presses a button in the notification or in another device. As you aren't specifying the proper capabilities in the options, I think they're not triggering because the buttons don't show up.
You can feed the duration manually and then retrieve it using getQueue(). The player won't calculate the duration (which you can retrieve using getDuration()) until you start the track.

Thanks for your response.
How can I remove notitication?
On Thu, 14 Mar 2019, 15:14 Guilherme Chaguri<EMAIL_ADDRESS>wrote:

You should be setting up the player before updating the options.
You shouldn't be updating your store inside the service, that
shouldn't cause issues in your case, but it might.
remote- based events are only trigger when the user presses a
button in the notification or in another device. As you aren't specifying
the proper capabilities in the options, I think they're not triggering
because the buttons don't show up.
You can feed the duration manually and then retrieve it using
getQueue(). The player won't calculate the duration (which you can
retrieve using getDuration()) until you start the track.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-native-kit/react-native-track-player/issues/491#issuecomment-472817629,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXphKcBVgR3g7DNlKfNxlCdl1jIL5QkKks5vWjYFgaJpZM4bsYYE
.

You can't. Android requires a notification up in order to start a foreground service

So I selected icon for pause and play but nothing happens
On Fri, 15 Mar 2019, 05:48 Guilherme Chaguri<EMAIL_ADDRESS>wrote:

You can't. Android requires a notification up in order to start a
foreground service
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-native-kit/react-native-track-player/issues/491#issuecomment-473133926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXphKVpgDRJLSQe5zt09YILpS1iRIw6Xks5vWwLlgaJpZM4bsYYE
.

Can you share the code of your app, so we can take a look at it?

Here is my code, I have removed icon options.
On Fri, 15 Mar 2019, 14:48 Guilherme Chaguri<EMAIL_ADDRESS>wrote:

Can you share the code of your app, so we can take a look at it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-native-kit/react-native-track-player/issues/491#issuecomment-473249123,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXphKYz1JOdL8HvM5pY8azTQiFDYmD6mks5vW4GJgaJpZM4bsYYE
.

I'm with the same issue
While remote-jump-forward works fine on iOS, Android for some strange reason is not firing this event.

@weslley39 Make sure you have the CAPABILITY_JUMP_FORWARD added to the options, if it's still not working, please, open a new issue.


Add london.txt

@russelljkennedy First stage of verification passed, official domain is london.ac.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@russelljkennedy Pull request merged. Thank you!


REST Response Intercept inconsistent behavior

I have an issue for REST API interception for modifying Response content. It works randomly depending on sequence of http/https calls.
Sometimes it doesn't process the URLs coming in at all, and other times I get "null" error on contents.setTextContents....
By inspecting the code it relies on "HTTP message aggregation" or some such thing, the code says in browsermob-core-littleproxy/src/main/java/net/lightbody/bmp/filters/Response/RequestFilterAdapter.java
"HTTP message aggregation will be enabled if any filter has a maximum request or response buffer size greater than 0." but this doesn't seem to always work for me. Sometimes requests just aren't processed. Other-times they are but contents is set to null and I get a null pointer error. I had this working by some fluke combination of HTTP requests but figured out over last day or so that the robustness of this functionality is like spinning plates. I haven't been able to determine exactly what is causing this, tried a whole bunch of different things and looked at debug log. It does seem to happen for more "complex" requests, whatever that means, harder to reproduce in an isolated example, but was still able to.
Any explanation of why I'm seeing this behavior would be greatly appreciated. If you need anything from me on this to reproduce, let me know.
I'm now looking into other available proxy solutions now or something that can be done at JavaScript layer, but not much is out there that I can find.
Thanks

How large are the HTTP messages you're processing? The default maximum is 2 MB, and if the messages are larger than that, you'll definitely experience problems with Javascript filters. This is done intentionally to prevent one large or malicious message from bringing down the JVM. The default can be overridden quite easily in code, though the REST API doesn't expose this functionality. It would be quite trivial to add, so if that's the problem you're facing, I'd be willing to add it to the creaky old REST API (which is going to be replaced in a forthcoming release)..
If you have any stack traces, that'd be quite helpful.

Not that large! But in my case html files with a bunch of resources within, don't think limit is the reason as sometimes it works depending on sequence of calls. I don't get any stack traces. For example to debug I tried to simply make the response filter  log all files it's processing using log.info and a bunch of files depending on http sequence just aren't listed in log output. (i actually was also trying this with snapshot compiled increase maximum to 20mb for other reasons but had no difference in effect..)
for example if I process an https request and then http, it works. but if all requests are https it doesn't work. it's so strange, but it does behave consistently, i.e. it's not random so much but behavior is dependent on specific sequence of calls and/or the web-content I'm loading (and perhaps also some network conditions.)
I ended up finding another solution for my needs which was to create my own kind of proxy http server that modifies response content. Thanks for the reply.

Do you have an example URL that will reproduce this issue? I'd like to try to solve this if possible, since other people are probably running into the same issue. In general, the request and response modification should work extremely well, as long as you're using the LittleProxy implementation (which is the default in beta-4).

is there any way i can provide the details in non-public forum, for confidentiality reasons? it's not a specific url but rather a sequence of calls, where some resources are just "skipped" and/or contents is set to null. it seems like the first url in the sequence is checked but then attempt to load same url again and another url, and both are "skipped"
i have a script where i was able to (at least locally) reproduce this fairly consistently. i'm also doing this in nodejs using browsermob-node & selenium, not sure if that makes a difference. just ran it again for example, still happening.

Can you reproduce the issue against a public website? That would be the best test case, in the spirit of transparent open source development.

Since there's no public website to test against, I'll close this issue. Feel free to reopen with more information.


⚠ Bofrost.at has degraded performance

In d0ca4ef, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1342 ms

Resolved: Bofrost.at performance has improved in de28270.


[Question]: Is there an exmaple of cost analysis that works? MockLLMPRedictor is not working

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I am looking for a Cost Analysis example that I can use without calling OpenAI, so I can get an approximate cost of a specific use case I have. I've seen the xamples and I got my code working almost to the end, but the MockLLMPRedictor library apparently is not working anymore for the newest versions.
I used this example: https://betterprogramming.pub/how-to-calculate-and-optimize-chatgpt-costs-using-langchain-9599354f0e2c
I go tit almost working except the total_token_usage, as that function is no longer working.  I also looked into the documentation, and they have the same exmaples, and none is working:
https://gpt-index.readthedocs.io/en/v0.6.8/reference/service_context/llm_predictor.html
https://gpt-index.readthedocs.io/en/v0.6.8/how_to/analysis/cost_analysis.html

The documentation you linked is very old.
The latest example of token counting is here
https://docs.llamaindex.ai/en/stable/examples/callbacks/TokenCountingHandler.html

Awesome. this indeed worked. Thanks for the support.


46.teams-auth - Sign in card not working in MS Teams

Sample information

Sample type: samples
Sample language: python
Sample name: teams-auth

Describe the bug
When pressing "Sign In" in Teams, it does nothing. When testing in web chat all works as expected.
To Reproduce
Steps to reproduce the behavior:

Start bot.
Upload manifest with token.botframework.com in validDomains to Teams.
Type anything and bot responds with card asking for sign in.
Press "Sign In" button and nothing happens. No errors thrown in bot also.

Expected behavior
Should open sign in window.
Screenshots

Additional context
Bot hosted on local machine and exposed over ngrok.
Any ideas how to troubleshoot?

Hi @rizickus
Teams also requires the domain of the bot's endpoint to be in validDomains:
"validDomains": [
   "*.azurewebsites.net",
   "*.botframework.com"
]

https://github.com/microsoft/BotBuilder-Samples/blob/main/samples/python/46.teams-auth/teams_app_manifest/manifest.json
If debugging locally using ngrok, this should include *.ngrok.io

Thanks for the reply! Will check next week.

@rizickus was @EricDahlvang's guidance able to solve the issue?

Closing this due to inactivity. If problem persist please open a new one.

@rizickus did you manage to solve your issues?


Pin this repository to top of ARMmbed Group

There is an open space at the top of the ARMmbed group. I think this repository ought to be pinned there.

ARM Internal Ref: IOTCLT-3602

Done

Thanks @MarceloSalazar ! Thank you for the feedback @loverdeg-ep .


On Stream Review

Hardcoded Values in Repo

Starting from raw data, it would be nice to find a way to not have to commit the data itself but instead create some sort of process to retrieve the data - I'm not sure how you found this data so it might not be possible, but I'm sure you understand why it's suboptimal to have a ton of data just hardcoded in your repo
A lot of places in the repo could use a combover looking to pull constants out

Urls that you make requests to
The default classes that you recommend in your "GetSuggestedCourses" DB function

DB Schema

I don't personally love how difficult it gets to work with your "courses" based around the schema of "APlus", "A", etc.
It's nice in that you can just sum the grades by summing columns but again it feels super hardcoded to the way that your school does grades / reports their distribution
In a prod environment you really wouldn't want to convey your "attribute" via N columns
It would probably be better to turn this into 2 tables that are a 1->many relationship of instructor -> grade count

Thanks for taking the time to review my code!
Regarding the data source, you're absolutely right about the bad practice of hardcoding it into the repo. I plan to enhance this in the future by implementing a feature to dynamically retrieve data from the UTD Grades repository's raw_data directory, addressing the concern you raised.
In the upcoming version 1.0.9, I have taken some constant out as per your discretion. Thanks for pointing that out!
I appreciate your suggestion about the DB schema and the potential complexity in handling grades. While I didn't fully grasp your proposed solution, I believe for the current project scope, the hardcoded SQL statements suffice. I'll keep your insights in mind for future iterations as the project evolves and features become more intricate.
Again, thank you so much for not only reviewing my code, but also for the generous $100 cloud credits giveaway!


Add CRSP cross-walk and regression verification to conversion code

This should also move to a new repo. @Jay4869 has the details on where to get the CRSP data, and will move it to Savio for now.
This will make our conversion code much more generally useful.
cc: @rdhyee

Once this is done, we should incorporate this logic in dlab-berkeley/python-taq

CRSP dataset: taq_crsp.sas7sdat produced by _TAQ_CRSPnext.sas

clean_dsenames.sas: cleaning list of SYM_ROOT SYM_SUFFIX for mering CRSP with TAQ
_TAQ_CRSP.sas: create a small version of crspa.dsf
TAQ_CRSP.sas: combines the TAQ and CRSP based on day-by-day for all stocks, and run in parallel for each day.
lusFIN.sas: created by clean_dsenames and taq_crsp

As last time Justin said, the raw-crsp is produced by dsf.sas7bdat and dsename.sas7bdat under folder "a_stock". However, I did not find the folder "a_stock" in the wrds.
Let's take time figure out that

@Jay4869 is almost there with this, but still getting a sas error. Currently, we think this might be a missing include file.
We might also do this computation on WRDS.


fixes issue #52: added hover to buttons

added hover feature to paypal button on donate page and main page buttons under Jupyter architecture section
(fixes issue #52)

Thanks  a lot @go-bears !
I think we might want the same style than on the index page ie  background-color: #DB5106; on hover more than a deep blue.
Though @ellisonbg / @cameronoelsen  ? (style of this PR on hover)

@go-bears You do not need to close the PR to update it (I saw you submitted #68) you can just push extra commit on the same branch, and the PR will be updated. You can also force push if you want to completely or rewrite history. I don't go into details here, but feel free to ask question if you are not sure of how to push/force push or use git or something else.

thanks  @Carreau for the color reference! will get to it now.

Better ! I think the drop shadow is still blue :-)

Hi @Carreau I think finally all the button issues are fixed for now.  Thank you for your guidance!


Optional paddingY

Description of change
Optional paddingY in math/src/Rectangle.js and display/src/Bounds.js
Pre-Merge Checklist

[x] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Codecov Report

Merging #6145 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6145   +/-   ##
=======================================
  Coverage   71.67%   71.67%
=======================================
  Files         213      213
  Lines       10752    10752
=======================================
  Hits         7707     7707
  Misses       3045     3045

Impacted Files
Coverage Δ

packages/display/src/Bounds.js
91.08% <ø> (ø)
:arrow_up:

packages/math/src/shapes/Rectangle.js
92.95% <ø> (ø)
:arrow_up:

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 28f2346...c75b8aa. Read the comment docs.

Consistency. You certainly know what it is.


pinchZoom: Add option to change the zoom factor

First of all, thank you for this great plugin!
We are currently working on a project where the required zoom can increase very heavily.
By adjusting the zoomFactor the mouse wheel zoom works just fine.
Unfortunately there is no such thing when zooming with pinchZoom.
Would it be possible to add a new option like pinchZoomFactor or zoomFactorPinch?

when you pinchzoom, your 2 fingers always stay on the same spot on the canvas. that means you can only zoom as much as you can spread your fingers. A zoom factor doesnt really make sense.
If you want that anyway you are welcome to create a PR. The code should be easy enough to understand.
e.g. I would start by multiplying the zoomAmount in the pinchZoom function by 2 and see if it works as you would expect


Reference app not working on pycafe

link

This simpler one does not work either. Same problem.
link

Looks like the bug you already fixed in Panel. Will cut a 1.5.4 release with a fix soon.


Split long modules / Improve directory structure

The project has some modules which are too long or have several classes in them which don't relate:

rasa_core/actions/action.py: 11 classes, > 490 lines
rasa_core/events/.__init__.py: 18 classes, > 950 lines
rasa_core/utils.py:4 classes, > 700 lines
rasa_core/evaluate.py: > 660 lines
tests/test_policies: > 600 lines

In case of the test modules it might also be worth to mirror the directory structure of the source code to ease finding tests.

@wochinge i guess this is still relevant?

yes. Also part of Rasa X

added the enhancement label so stalebot doesn't kill it :D


Implement CameraUpdate.newLatLngBounds

Please add support for CameraUpdate.newLatLngBounds. This seems to be already available in flutter/google-maps: https://github.com/flutter/plugins/blob/master/packages/google_maps_flutter/google_maps_flutter/example/lib/move_camera.dart#L84-L91. Not sure about the apple maps part though

+1


permissions issue

I keep getting this issue when I try to run the plugin on MacOS Sierra. It appears to have installed correctly. I tried a sudo install too.
find: ./.gnupg: Permission denied

I don't think it has nothing to do with git-standup. Seems like you've used sudo to do something with your GPG Keys.
Have you tried to restore permissions? Everything under your $HOME should be yours.


Add Cleverbot.io as dependency

Bot is unable to start without Cleverbot.io.

Added some explanation messages in shards as well

@cernodile Made changes as per requested.


TP-571: ADR-0012: Ordering of transactions.

An ADR that was previously a Google doc, talking about how we order transactions.

Codecov Report

Merging #221 (80b5eec) into master (cd22c3e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #221   +/-   ##
=======================================
  Coverage   61.11%   61.11%
=======================================
  Files         177      177
  Lines       10764    10764
  Branches      807      807
=======================================
  Hits         6578     6578
  Misses       4002     4002
  Partials      184      184

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 cd22c3e...80b5eec. Read the comment docs.

Does everything we need to do fit into a transaction ?
Some of the example operations take more than one transaction to achieve, does that mean the business rules are still valid if only one of those transactions was acted on ?

Does everything we need to do fit into a transaction ?
Some of the example operations take more than one transaction to achieve, does that mean the business rules are still valid if only one of those transactions was acted on, or do they have to go a via a state where they are invalid ?

No, all transactions must be valid. If an operation takes multiple transactions to achieve, the state at the end of each must be valid as well. This is the thing that makes it so tricky – in the quota order number update example, you can't issue any ordering of transactions (using only UPDATEs) that is valid, so you have to go via anothe rmechanism (DELETE then CREATE).

It feels analogous to trying to find a route from one part of a map to another (and probably similar in the explosion of combinations).
I wonder if it's easier starting from every  update being  DELETE and CREATE, then trying to turn some into updates or going the other way and attempting to find a combination where the business rules are satisfied by converting UPDATES into DELETE and CREATES?

I wonder if it's easier starting from every update being DELETE and CREATE, then trying to turn some into updates or going the other way and attempting to find a combination where the business rules are satisfied by converting UPDATES into DELETE and CREATES?

That's interesting, I will think on that further.
It's ultimately a finite space so really it should be possible to define all the possible combinations. It's just hard to get right and will take a long time!

I'm reminded of calendar scheduling algorithms (during the change from one state to another, one thing blocks another from being in a valid state).
There's an interesting graph based approach to a scheduler algorithm
https://stackoverflow.com/questions/3235306/calendar-scheduler-algorithm
Maybe some of these approaches might help.


feat: NFT #Number not showing in profiles

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Feature

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #9144

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary
copilot:summary
copilot:poem

Btw there are many issue with showing serial number that could have same implementation

Btw there are many issue with showing serial number that could have same implementation

Yup, that should be the reason why we do not apply them on the items explorer page.

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 6.73 USD/DOT ~ 4.458 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0xe851a76b5d2241427a4169c828457054604926f7e3ccd7c8239d11be61fef1ab
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


[UIKit] In iOS 12, UINavigationBar with search controller changes height abrubtly

UPDATE: finish #435 before fixing this.
Steps:

On iOS 12, tap search.
When on search, press back to navigate to index, or pull from left edge slowly.

Expected:

Height of UINavigationBar is changed with animation, smoothly - like in iOS 13+.
Screen Recording 2022-06-03 at 00.01.24.mov.zip

Actual:

Height of UINavigationBar is changed abruptly.
Screen Recording 2022-06-02 at 23.55.58.mov.zip

Changes will be done in #436


build: use npm build instead of yarn

in GitHub action, we use yarn for building, but we use npm.

Thanks @amirfefer ! :)


[Question] Where is hb-view?

When I install the dependencies thru brew I get harfbuzz v1.4.5 and the bin folder contains only the tools hb-ot-shape-closure and hb-shape.
When I try to run notodiff with the rendered option it fails because it cannot find hb-view.
What am I missing?

Probably you need to run this command if you are using linux.
sudo apt-get install libharfbuzz-bin
I have no idea either why we do not install libharfbuzz-bin automatically with the notodiff.
Reference:
https://github.com/googlei18n/nototools/issues/392

You need to build HarfBuzz with Cairo support enabled (--with-cairo) to get hb-view, by default Homebrew builds without Cairo.

Thanks Khaled. Do you have a suggestion on how to modify the installation instructions? As they stand they're incomplete.

I can’t test on Mac right now, but I think something like this should work (you might also need to uninstall harfbuzz first or force reinstall, I don’t remember exactly how):
brew install --with-cairo harfbuzz cairo pango pygtk imagemagick

I confirmed that that works. Thanks Khaled.
I've submitted a patch in this PR https://github.com/googlei18n/nototools/pull/395


[BEAM-7112] Timer race with state cleanup - take two

There was still a race condition when cleanup and user timer are triggered together in finish bundle. This is resolved by deferring state cleanup to the next bundle.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Thanks Thomas! I think we should simplify the firing logic of the cleanup timers but looks good otherwise.

Run Java PreCommit

We don't need to checkpoint the cleanup queue since it does not cross the bundle boundary. But we need to look into another issue (outside of this PR): The finish bundle callback starts a new bundle when the watermark advances, which can lead to inconsistent state snapshot.

Run Java PreCommit

The finish bundle callback starts a new bundle when the watermark advances, which can lead to inconsistent state snapshot.

I think the solution could be to checkpoint cleanup timers and delay execution of the timers for until after the checkpoint has completed. Otherwise we have to checkpoint any possible emitted elements during completing the bundle within snapshotState.

Run Java PreCommit

Run Python PreCommit

Build successful but it stuck collecting results: https://builds.apache.org/job/beam_PreCommit_Java_Commit/5674/
06:13:42 BUILD SUCCESSFUL in 21m 32s
06:13:42 966 actionable tasks: 932 executed, 34 from cache

Run Java Flink PortableValidatesRunner Streaming

Run Java Flink PortableValidatesRunner Streaming

I think this is good to merge. Let's tackle problems with the checkpoint interaction next. On Flink >= 1.6 this would be trivial to solve by flushing inpreSnapshotBarrier. For 1.5 we will have to do one of the following:

Checkpoint cleanup timers and defer execution for after the checkpoint
Buffer emitted elements during snapshot (this is already done but there is no key scope)

The issues is that a new bundles is created during finish bundle, which is actually not related to this PR. Tracked in: https://issues.apache.org/jira/browse/BEAM-7171


turn off notifications which are coming from plugin

I want to turn of all the notification which are coming from plugin, but i want to track the location every 5secs,
i made debug:false, in my code and in
componentDidMount(){
BackgroundGeolocation.on('location', (location) => {
      console.log("location123:"+JSON.stringify(location))
      // handle your locations here
      // to perform long running operation on iOS
      // you need to create background task
      BackgroundGeolocation.startTask(taskKey => {
        console.log("taskKey123:"+taskKey)

      //here i will call my api and send the locations to db
       })
  })
}

i don't those notifications which are coming from plugin and i want to store my location in db when app is in background for every 5 secs.
Please help me or give me some suggestions  how can i use this module for that.

@mauron85 please respond

I don't understand your problem. Please fill issue template, so I can help.

@mauron85 his problem is that on iOS there is a local notification constantly being sent.

Please post output of:
BackgroundGeolocation.getConfig(function(config) { console.log(config); });


Polish up APISIX metrics to follow Service concept

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

All discussion could be found here  https://github.com/apache/skywalking-showcase/pull/83#issuecomment-1272518463

route:: and node:: should be updated as route/ and node/. Because in ElasticSearch and BanyanDB search, it relies on word splitter, / is in that list, :: is not.

Update  :

One more suggestion, you could set metrics for endpoint list for now.

Like this ?

Yes.

Is upstream better than node? Is the IP upstream addr?

Is upstream better than node? Is the IP upstream addr?

I have update the tile , keep default value . IP is apisix instance .

What do you mean? Isn't 172 a remote and upstream addr?

Sorry  , Let's  change to upstream

upstream

My  metrics is  labeled  , and  other similar ui-template is  not labeled .  Couldn't  support labeled metrics ?

@pg-yang Let's continue without this.
@Fine0830 Could we support labeled value? I suppose we need an issue to discuss how to show brief values if supported.

Do we need test updates due to endpoint format changed?

It's time to dinner . Hot pot is  great

Do we need test updates due to endpoint format changed?

Needn't  , E2E only check whether endpoint has value  rather than specific value

ping 🔢


fix: The sqlite database file is not updated because the database con…

The sqlite database file is not updated because the database connection is not closed when the program exits.

https://github.com/rocboss/paopao-ce/blob/a4348f225046ddc3a7b2f93dd930e971ea4fd1ab/web/src/views/Post.vue#L8
This condition will cause the first article to be unable to display details.


Create german

This text is based on the "Washington"-version; I'd be pleased, if someone could check and improve this version.
I used german pages to OSM and reduced some parts that are only available in English language.
This brings me to a proposal: May be, it's possible to combine the URLs into the "OSM-World" somewhere for different languages, here EN and DE:
Mainpage | Hauptseite
Get Help | Hilfe
Forum | Forum
...

Thank you @jbelien for your corrections in my Welcome-text (German version); sorry, this has been my very first contribution here and with MarkDown. Thanks again for your quick fix with the LogIN / LogOut; works perfect now!
As you may find here, I'm thinking about a second Welcome-text for StreetComplete - Users  and this brings me back to my proposal here: Is there already any "Best Practice", to unify those links?

Thank you @jbelien for your corrections in my Welcome-text (German version); sorry, this has been my very first contribution here and with MarkDown.

No reason to apologize, I'm here to help. 😄

Thanks again for your quick fix with the LogIN / LogOut; works perfect now!

Happy to hear it. 👍

As you may find here, I'm thinking about a second Welcome-text for StreetComplete - Users and this brings me back to my proposal here: Is there already any "Best Practice", to unify those links?

No, there is currently not really a best practice.
Each region is completely free to add any template they judge necessary (in any language they need).


Fix disposable leak in SlideBase

Fixes the remaining part of https://github.com/google/closure-library/issues/214.

@nanaze  Who can review this?

@concavelenz any of the people currently configured in the review sharding tool. link: https://goto.google.com/bnjfj


Voice app

Have you read the Contributing Guidelines ?
Yes I have read the contributing guidelines.
Description
Added Voice to text App in Kotlin
Fixes #702

Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
https://user-images.githubusercontent.com/94545831/170314832-17e62efd-9d80-46b4-857c-b4a5023f6295.mp4

@photon149 I've done the requested changes. Kindly merge the pr


[WIP] Added rust phases diagram and explanation

[x] CoC read
[x] Included @ehuss 's diagram of rustc compilation phases and artifacts
[x] Added elaboration on stage naming convention
[ ] Added content from https://github.com/rust-lang/rust/issues/57963

The diagram was made by @ehuss. I was trying to ask in the other issue if you were comfortable by adding this to rustc-guides.

Sure, you're free to use it.  Here's the source file if you want to edit it: https://gist.github.com/ehuss/70eafa935cf21f0d67533790dcafc18e  That can be edited in https://www.draw.io/.  Just keep in mind that this is a simplification.  If there are non-host targets, its a bit different (described here), and passing flags like --keep-stage will change what happens.  Also, rustdoc can be built at different stages depending on various things.

I'll add in those caveats, thank you

Update: pushed some more content and almost done but still missing some important info and editing. Will r= when done.

r=mark-i-im
@jonas-schievink @dwijnand @mark-i-m @ehuss @Mark-Simulacrum
Please review the incorporated text, as they are based mostly off of your comments. I tried hard to be clear, non-repetitive and most of all, factually correct. I know the diff is large so if you have the opportunity maybe fetching the branch and reviewing the rendered markdown locally would be easier. Thank you for your time.

@igaray Sorry for the delay. Can you rebase? After that I think we can merge. This PR has already undergone more scrutiny than most.
Thanks for your patience!

Rebased but chalk links are broken again. Will rebase again after https://github.com/rust-lang/rustc-guide/pull/322 is merged


Connection pooling mechanism in redisson

HI Team,
I had a doubt regarding redisson connection pooling management.
I have an application with a high volume of traffic and hence get a lot of concurrent requests.
Have optimised my connection pool accordingly.But i have the following query
What if a new request arrives and all the connections are busy in case of unprecedented volume.
How does redisson handle such scenarios.Is there a connection request time out , post which it tries to create a new connection if pool is exhausted OR the request is rejected in absence of a connection in the pool .
regards,
AC

Is there a connection request time out , post which it tries to create a new connection if pool is exhausted OR the request is rejected in absence of a connection in the pool .

Redisson makes a new attempts to execute command according to retryInterval and retryAttempts settings.


cmd/cue: Input by user from prompt not parsed correctly

What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.3 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?

I was playing with cue scripting feature and wrote this little script
package foo

import (
    "tool/exec"
    "tool/cli"
)

city: "Kathmandu"

command: hello: {
    ask: cli.Ask & {
        prompt: "Your Name?"
        response: string
    }
    print: exec.Run & {
        cmd: "echo Hello \(ask.response)! Welcome to \(city)."
    }
}

I ran the following script using this command cue cmd hello
Now cue prompts me to give input
I give input with space character in it.
What did you expect to see?
I expect cue to print Hello Rabin Gaire! Welcome to Kathmandu.
What did you see instead?
Hello Rabin! Welcome to Kathmandu. and shell treats Gaire as a new command
rabingaire@dev:~/Desktop/cue
$ cue cmd hello
Your Name? Rabin Gaire
Hello Rabin! Welcome to Kathmandu.
rabingaire@dev:~/Desktop/cue
$ Gaire

Command 'Gaire' not found, did you mean:

  command 'caire' from snap caire (1.4.4)

See 'snap info <snapname>' for additional versions.

rabingaire@dev:~/Desktop/cue
$

fmt.Scan, just like any other form of reading from standard input in an interactive way, will block until a newline is entered. So, from a UX and API point of view, I don't think it makes sense to support a cli.Ask that only reads one word, because it blocks until a whole line is entered.
Looking at https://cue-review.googlesource.com/c/cue/+/7261, where the code was reviewed, I don't see that it was a conscious choice to go with fmt.Scan. I believe it was an unfortunate choice that didn't get caught during code review :)
I think we should just go with suggestion 1: change cli.Ask to read the whole line that it's blocking on. For the sake of backwards compatibility, it's worth noting that fmt.Scan skips whitespace before and after the word being read. I think we do want cli.Ask to strip the trailing newline, because most users will want some answer rather than some answer\n.
It's less clear whether we want to keep the trimming of whitespace, so that answering with foo results in foo. I lean towards cli.Ask giving the whole line as it was written by the user, including all whitespace except the trailing newline. The CUE command can always strip whitespace via https://pkg.go.dev/cuelang.org/go/pkg/strings#TrimSpace, or get the whitespace-separated words (fields) via https://pkg.go.dev/cuelang.org/go/pkg/strings#Fields.

I forgot to mention the most important point about backwards compatibility: would this change break any CUE commands? I'm inclined to say not, because any existing cli.Ask users who just want one word are still ending up with very confusing command-line UX for the end user. They can always use strings.Fields to split the answer line and do whatever they want with any number of entered words, for example, and that should work much better.
But this change would certainly warrant a mention in the changelog.

https://review.gerrithub.io/c/cue-lang/cue/+/541426 is my proposed fix per the discussion above, if any of you want to give it a try or a review.


Allow errors in TCP.start_protocol/1 pipeline to be returned.

When connecting many clients to an XMPP server quickly, closed streams
can occur. Let those errors through instead of blowing up due to
FunctionClauseError.
This could probably be done more cleanly with with, but mix.exs specifies compatibility back to Elixir 1.1.
Fixes #27

Coverage decreased (-2.3%) to 71.885% when pulling 01a3bc5d06f4b3b6c358d4e7e0c32fc6f717397b on lmarlow:send_error into 0ecef1e1d6e11f0a47a688b29e8c13c39aa6018f on scrogson:master.

Coverage increased (+1.2%) to 75.399% when pulling 84a4fe74808ccbcdf29c7a1ca2816dfe5c3246ec on lmarlow:send_error into 0ecef1e1d6e11f0a47a688b29e8c13c39aa6018f on scrogson:master.


Delete consteval constructors

This is sad. But they break compilation.
They are a constructor so they should let you construct stuff with them.
struct S { consteval S(int); };
S(1);

This works fine cuz we're calling the constructor directly. But if there's any indirection in between, the constructor becomes unusable as you now have a reference, but still remains callable even though you're now in a runtime context. And this just produces a compiler error.
One consequence is that it's impossible to construct the above S from a function that takes a variable number of arguments. Varargs are implemented as templated types, and you can specified constructible_from<S, Ts> for each Ts but you receive a Ts&& and now when you call S::S() with that reference, the compiler generates an error.
See https://sunny.garden/@blinkygal/110557258312870357 for more sadness about this.
It was really nice to compile-time convert from (signed) 1 to u32 but it just breaks in completely surprising and difficult ways to understand for a user. Better to not work at all I think.

Avoid consteval in C++20.
Here's a more coherent explanation of the deficiencies of consteval in C++20. We'll look at it from the perspective of constructors, where it's more obvious, but the same applies to any function.
tl;dr consteval does not participate in overload resolution, consteval is not visible to SFINAE, and consteval. So it breaks templated code and your users will have to work around it.
Setup
Here's a type, it can be constructed in a constant expression from an int or a float. At runtime it can not be constructed from int. To show varargs, it also has a constructor from int, float.
struct S {
    // Compile-time or run-time.
    constexpr S(int, float) {}
    constexpr S(float) {}

    // Compile-time only.
    consteval S(int) {}
};

What works
We can construct S from an int literal, which is known at compile time. We can also construct it from a constexpr variable.
auto v1 = S(1);
constexpr auto c =1;
auto v2 = S(c);

We can receive S by value in a non-constexpr function and pass an int literal to construct the argument.
void take_s(S) {}

void f() {
    take_s(1);
}

What works in one compiler
We can construct S from a constexpr template function, in clang anyway. In MSVC this is a compiler error error C7595: 'S::S': call to immediate function is not a constant expression inside each of the make_s1 and make_s2 functions. In GCC this is compiler error error: 'i' is not a constant expression inside each function.
template <class T>
constexpr S make_s1(T i) { return S(i); }
template <class T>
constexpr S make_s2(T&& i) { return S(std::forward<T>(i)); }

void f() {
  auto v1 = make_s1(1);
  auto v2 = make_s2(1);
}

What does not work
If your constexpr function has a path that calls a non-constexpr function (in fact soon all paths can be non-constexpr and will be accepted), and you forward your template type, things break.
void take_s(S s) {}

template <class T>
constexpr void make_s1(T i) { if (i < 10) take_s(i); }
template <class T>
constexpr void make_s2(T&& i) { if (i < 10) take_s(std::forward<T>(i)); }

void f() {
  // error: call to immediate function 'make_s1<int>' is not a constant expression
  make_s1(1);
  // error: call to immediate function 'make_s2<int>' is not a constant expression
  make_s2(1);
}

In clang, at least the error is at the caller of make_s1 and make_s2, whereas in MSVC it ends up inside the body of those functions.
There's no way to ask if S is constructible in a runtime context. The type_traits library only answers about compile time (constant evaluation). So there's no way to interact with the caller in SFINAE and have them choose a different path. There is only compile error.
This gets much more obvious when you want to write generic code. Here's a class that's templated on T.
template <class T>
struct Holder {
  Holder(T u) { T{u}; }
};

This works because we're receiving T by value.
void f() {
    Holder<S>(1);
}

But we may want to receive parameters that are convertible_to the template type T without converting them yet, rather than receiving them by value. This is a pretty common idiom in C++ due to the lack of trivial relocation. Now this fails to compile, because the construction of S as moved into the Holder constructor.
template <class T>
struct Holder {
  template <std::convertible_to<T> U>
  Holder(U&& u) {
    // error: call to consteval function 'S::S' is not a constant expression
    T{std::forward<T>(u)};
  }
};

void f() {
    Holder<S>(1);
}

And again, there's no way for f() to determine that it can't construct Holder in this way. There's no way for Holder to provide a different constructor overload for runtime instead of compile time.
Thus my very modern and clever type S with its use of consteval can not integrate into any code that contains non-constexpr functions. And not because S isn't usable in a runtime context, but because the language is not expressive enough to expose the consteval requirement in a dynamic way, or to provide alternate paths. We could use std::is_constant_evaluated() to use a run-time constructor if we could tell that something is runtime-constructible but not compile-time-constructible.
Not that I am wishing for yet even more constructible type traits. The situation with constant evaluation in C++ is... complicated. I'd take procedural macros instead any day. I kind of just want consteval to be something you can overload on with non-constexpr (but not with constexpr), so that you can provide a compile-time path and a run-time path. But then I'd want to = delete the runtime one sometimes and there'd still be no way to see that in type_traits.
I am going to stop using consteval for now.

Here's the specific code that works without consteval:
TEST(Successors, Example) {
  auto powers_of_10 = sus::iter::Successors<u16>::with(
      sus::some(1_u16), [](const u16& n) { return n.checked_mul(10_u16); });
  sus::check(
      sus::move(powers_of_10).collect<Vec<u16>>() ==
      sus::Vec<u16>::with_values(1_u16, 10_u16, 100_u16, 1000_u16, 10000_u16));
}

And here's what I want it to look like, using consteval constructors to reject invalid values at compile time:
TEST(Successors, Example) {
  auto powers_of_10 = sus::iter::Successors<u16>::with(
      sus::some(1), [](const u16& n) { return n.checked_mul(10); });
  sus::check(
      sus::move(powers_of_10).collect<Vec<u16>>() ==
      sus::Vec<u16>::with_values(1, 10, 100, 1000, 10000));
}

Unfortunately some() holds an int&& and its constexpr operator Option<i16>() fails to compile constructing the i16 from the int&&.
And Vec<u16> takes variadic args Ts&&... which are convertible_to<u16> except they are not when the function runs and it fails to compile.


Skip rules when gateway is down doesn't work for gateway group

When i create a rule for forward traffic to gataway group.
And all gateways in this group are down.
Feature  Skip rules when gateway is down doesn't apply to this rule so the trafficstops at this rule.
To Reproduce
Steps to reproduce the behavior:

Create gateway group with two or more gateways
Also have some default gateway not in the group
Create rule to forward some traffic to this gateway group
Enable feature Skip rules when gateway is down
Shutdown gateways in the group to make them down
Traffic stops on this rule and feature Skip rules when gateway is down doesn't apply

Expected behavior
Feature Skip rules when gateway is down should be applied to gateway group
OPNsense 24.7.10_1-amd64
FreeBSD 14.1-RELEASE-p6
OpenSSL 3.0.15

I'm not sure that's how it works. A gateway group is a set of gateways that should always have a gateway that is up so you cannot skip a rule for a gateway group because it is always active?

If you really plan to have multiple gateway groups in effect I think you actually want the rules to be static in order to not taint your other gateway group with sudden traffic from a side you don't actually want to.
Feel free to explain your use case, but I don't have high hopes this is a feature that many people need and is useful to spend the required community time on.

Feel free to explain your use case, but I don't have high hopes this is a feature that many people need and is useful to spend the required community time on.

Dictator in my country block youtube traffic so I have to force youtube traffic to VPNs.
When VPNs is down I can't use not only youtube but also google search which is not forbid for now and could continue to work through default gateway.

@vgdh doesn't this mean you could create a duplicated rule per gateway in order to emulate the group? that would turn off the rule to the gw that doesn't work and eventually turns off all rules with a gateway down?

@vgdh doesn't this mean you could create a duplicated rule per gateway in order to emulate the group? that would turn off the rule to the gw that doesn't work and eventually turns off all rules with a gateway down?

Of course this workaroud will work.  But this doesn't resolve the issue 😄

I'm considering it as a reasonable feature request but that doesn't make it relevant to be added any time soon.
The workaround is the best you got.

I'm considering it as a reasonable feature request but that doesn't make it relevant to be added any time soon.
The workaround is the best you got.

It would be great to see this feature in release some day.


STUB: App Store

What happened (or feature request):
As per the discussions on packaging and making this drop-dead simple to get started and up and running with your first site, we would like to get ddev-ui.app in the Apple App store.

Not sure if this is prioritized or on the current roadmap https://github.com/drud/ddev-ui/wiki/Roadmap. However, I had a person I wanted to demo ddev-ui to and having it available as a direct link in the app store will be nice once that's available.

@rickmanelius great to hear from you! I'm trying to make this happen. Looks like we need to accepted the updated license as well as I need access to the agent account, or need someone to create some certificates for me. However, I would need to send a CSR that you would use to create them, so we can have circle handling code signing.
Certs still needed.

Developer ID Application
Developer ID Installer


fix(docs): refer to correct "deis perms:create" command

$ deis perms:add -h
Found no matching command, try `deis help`
Usage: deis <command> [<args>...]

saw the conversation on IRC. Nice catch :)
docs LGTM!

:ship:

This is really a typo fix, and I know it passes documentation tests, so I'm merging it since Jenkins is slow and irritable today.

This is really a typo fix, and I know it passes documentation tests, so I'm merging it since Jenkins is slow and irritable today.

:thumbsup:


from opus install decoder

I downloaded the git repository and then i run:
python setup.py build
python setup.py install

after that i have this error:
>>> import opus.decoder
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
  File "<frozen zipimport>", line 259, in load_module
  File "C:\Users\Χρήστος\AppData\Local\Programs\Python\Python38\lib\site-packages\opus-0.0.1-py3.8.egg\opus\decoder.py", line 3, in <module>
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
  File "<frozen zipimport>", line 259, in load_module
  File "C:\Users\Χρήστος\AppData\Local\Programs\Python\Python38\lib\site-packages\opus-0.0.1-py3.8.egg\opus\api\__init__.py", line 5, in <module>
  File "C:\Users\Χρήστος\AppData\Local\Programs\Python\Python38\lib\ctypes\__init__.py", line 363, in __init__
    if '/' in name or '\\' in name:
TypeError: argument of type 'NoneType' is not iterable

Any advice?

This project is not maintained, you can use any of forks available, such as https://github.com/orion-labs/opuslib


Add enum support to Slider control

Duplicate Check

[X] I have searched the opened issues and there are no duplicates

Describe the requested feature
I think there should be a property (or a methodology or anything, you name it), so that, we can supply a subclass of enum.Enum to it. Then it should work like, min value would be the minimum value from the enum we supply, and the max value would be the maximum value from the enum values. And also, there should be option to choose the label as the name property of the value of the enum we give.
Suggest a solution
There should be some property, which should accept the enum.
Screenshots
No response
Additional details
No response

I don't seem to understand your idea/suggestion.
Can you roughly share how the code for the enum will look like? and perhaps some code demonstrating a usage example.


another vtype layout bug

We have two "vtype layout" tables (one without ediv and one with), and one "vtype register format" table (without ediv).  An earlier pull request from me fixed typos in the vtype layout tables with the reserved bits line, but I missed the vtype register format table which has the same bug.  This could maybe be placed in a separate file and included so we don't have two almost identical copies of the same table.

Good idea - factored into separate file.


Unable to load and access ICOS data for Macehead

On Blue Pebble, if I attempt to load the ICOS data files for macehead this seems to process the files but these do not seem to be accessible within the object store
from openghg.modules import ObsSurface
from pathlib import Path
import glob

sitename = "macehead"
sitecode="MHD"
icos_directory = Path(f"/work/chxmr/shared/obs_raw/AGAGE_GCWerks/data-icos/{sitename}/")

search_str = str(icos_directory / f"{sitecode.lower()}.*.1minute.*.dat")
icos_files = glob.glob(search_str)

mhd_results_4 = ObsSurface.read_file(filepath=icos_files,
                                    data_type="ICOS",
                                    site="MHD", network="ICOS")

When visualisting the object store this does not show the ICOS network
from openghg.objectstore import visualise_store
visualise_store()

When extracting obs_surface data this is unable to find any "co2" data for "mhd" (macehead) from the "ICOS" network
from openghg.localclient import get_obs_surface
data = get_obs_surface(site="mhd", species="co2", network="ICOS")

Looks like this error is returned when trying to read the macehead icos data:
Traceback (most recent call last):
  File "/home/rt17603/openghg/openghg/modules/_obs_surface.py", line 142, in read_file
    measurement_type=measurement_type
  File "/home/rt17603/openghg/openghg/modules/_icos.py", line 46, in read_file
    gas_data = self.read_data(data_filepath=data_filepath, species=species, site=site)
  File "/home/rt17603/openghg/openghg/modules/_icos.py", line 111, in read_data
    date_parser=date_parser,
  File "/work/rt17603/environments/openghg/lib/python3.7/site-packages/pandas/io/parsers.py", line 610, in read_csv
    return _read(filepath_or_buffer, kwds)
  File "/work/rt17603/environments/openghg/lib/python3.7/site-packages/pandas/io/parsers.py", line 462, in _read
    parser = TextFileReader(filepath_or_buffer, **kwds)
  File "/work/rt17603/environments/openghg/lib/python3.7/site-packages/pandas/io/parsers.py", line 819, in __init__
    self._engine = self._make_engine(self.engine)
  File "/work/rt17603/environments/openghg/lib/python3.7/site-packages/pandas/io/parsers.py", line 1050, in _make_engine
    return mapping[engine](self.f, **self.options)  # type: ignore[call-arg]
  File "/work/rt17603/environments/openghg/lib/python3.7/site-packages/pandas/io/parsers.py", line 1898, in __init__
    self._reader = parsers.TextReader(self.handles.handle, **kwds)
  File "pandas/_libs/parsers.pyx", line 518, in pandas._libs.parsers.TextReader.__cinit__
  File "pandas/_libs/parsers.pyx", line 742, in pandas._libs.parsers.TextReader._get_header
ValueError: Passed header names mismatches usecols


feat: add aggregateGroupByPeriod API

Summary
Currently, the library only supports the aggregateRecord function to retrieve aggregated data within a specified time range and does not support either aggregateGroupByPeriod or aggregateGroupByDuration. This PR adds the aggregateGroupByPeriod API to the library. Once this is approved, I am ready to submit another PR for aggregateGroupByDuration and update the documentation accordingly.
Related Issues:
https://github.com/matinzd/react-native-health-connect/issues/16
https://github.com/matinzd/react-native-health-connect/issues/116
Background
I'm working on an app that visualizes various health data in charts. For example, to display daily weight changes over a month, we currently need to make 30 separate aggregateRecord requests, one for each day. With aggregateGroupByPeriod, we can potentially reduce the number of requests significantly. I suspect that the following error, which I occasionally encounter, may be related to exceeding the API call quota:

android.health.connect.HealthConnectException: android.health.connect.HealthConnectException: API call quota exceeded, availableQuota: 0.0151130445 requested: 1

Tests
I've made a couple of changes to the example app to test the new functionality:

insertSampleData: Updated the sample data insertion. The insertSampleData function now inserts step count data for each day between 9 and 11 AM for the past 7 days (excluding today, as it can be before 9 AM).
AGGREGATE SAMPLE GROUP DATA button: Added the "AGGREGATE SAMPLE GROUP DATA" button, as shown below, to test the aggregateGroupByPeriod API.

Log after AGGREGATE SAMPLE GROUP DATA button is pressed
# The user timezone is Asia/Tokyo
# No sample data is inserted for TODAY (2024-09-11)

Aggregated Group:  {
  "result": [
    {
      "endTime": "2024-09-04T15:00",
      "startTime": "2024-09-03T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 1000
      }
    },
    {
      "endTime": "2024-09-05T15:00",
      "startTime": "2024-09-04T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 2000
      }
    },
    {
      "endTime": "2024-09-06T15:00",
      "startTime": "2024-09-05T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 3000
      }
    },
    {
      "endTime": "2024-09-07T15:00",
      "startTime": "2024-09-06T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 4000
      }
    },
    {
      "endTime": "2024-09-08T15:00",
      "startTime": "2024-09-07T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 5000
      }
    },
    {
      "endTime": "2024-09-09T15:00",
      "startTime": "2024-09-08T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 6000
      }
    },
    {
      "endTime": "2024-09-10T15:00",
      "startTime": "2024-09-09T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 7000
      }
    },
    {
      "endTime": "2024-09-11T10:50:12.182",
      "startTime": "2024-09-10T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 0
      }
    }
  ]
}

Hey!
Thanks for the PR :) I will try to review it this weekend if I find some free time.

@matinzd Do you have time to take a look at the PR by any chance?

Hey!
I reviewed 60% of it. There are a lot of file changes so I need to be careful with the review :) Thanks for the follow-up.

@matinzd Thank you for the review! I will apply the changes soon

@matinzd I have applied the changes accordingly. And here is the log message for the record.
Log after AGGREGATE SAMPLE GROUP DATA button is pressed
# The user timezone is Asia/Tokyo
# No sample data is inserted for TODAY (2024-10-02)

Aggregated Group:  {
  "result": [
    {
      "endTime": "2024-09-25T15:00",
      "startTime": "2024-09-24T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 1000
      }
    },
    {
      "endTime": "2024-09-26T15:00",
      "startTime": "2024-09-25T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 2000
      }
    },
    {
      "endTime": "2024-09-27T15:00",
      "startTime": "2024-09-26T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 3000
      }
    },
    {
      "endTime": "2024-09-28T15:00",
      "startTime": "2024-09-27T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 4000
      }
    },
    {
      "endTime": "2024-09-29T15:00",
      "startTime": "2024-09-28T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 5000
      }
    },
    {
      "endTime": "2024-09-30T15:00",
      "startTime": "2024-09-29T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 6000
      }
    },
    {
      "endTime": "2024-10-01T15:00",
      "startTime": "2024-09-30T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 7000
      }
    },
    {
      "endTime": "2024-10-02T12:47:21.255",
      "startTime": "2024-10-01T15:00",
      "result": {
        "dataOrigins": [],
        "COUNT_TOTAL": 0
      }
    }
  ]
}

@matinzd Just a friendly reminder that I made the suggested changes. Please take a look when you can :)

@matinzd Thanks! Will work on aggregateGroupByDuration next.

Very nice! Can't wait to see that!


Update for new fits format and plot fixes

Codecov Report

Merging #23 (051a140) into master (9f9bdbe) will decrease coverage by 43.96%.
The diff coverage is 11.42%.

@@             Coverage Diff             @@
##           master      #23       +/-   ##
===========================================
- Coverage   67.41%   23.44%   -43.97%
===========================================
  Files           9        9
  Lines         672      661       -11
===========================================
- Hits          453      155      -298
- Misses        219      506      +287

Impacted Files
Coverage Δ

stixpy/net/attrs.py
100.00% <ø> (+14.28%)
:arrow_up:

stixpy/timeseries/quicklook.py
17.20% <0.00%> (-51.31%)
:arrow_down:

stixpy/science.py
17.86% <7.69%> (-39.63%)
:arrow_down:

stixpy/data/test.py
85.71% <100.00%> (-9.53%)
:arrow_down:

stixpy/net/client.py
32.35% <100.00%> (-67.65%)
:arrow_down:

stixpy/vis/map_reprojection.py
28.07% <0.00%> (-36.85%)
:arrow_down:

Continue to review full report at Codecov.

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


fixed bug and enhanced UI of author.html and tips

Related Issue
None
Fixes:  #4521
Description
Fixed the bug of author.html and enhanced the UI of author.html and tips.html by improving the cards.

Type of PR

[x] Bug fix
[x] Feature enhancement

Screenshots / videos (if applicable)
Before:

After:

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

the box shadow effect on the cards doesnt look good .
work on other issues.


hello , god blessing you , when I run node ruff.js  ,    here  are  error   ,what do I do

win11  ruff 0.6.9

running  error

using deprecated parameters for the initialization function; pass a single object instead
Error: TypeError: fetch failed
at node:internal/deps/undici/undici:13185:13
at async __wbg_init (file:///A:/GitHub_Learning_Completed_Course/100+%20Python%20%E9%A1%B9%E7%9B%AE%E6%8C%91%E6%88%98/ruff-playgroud/black-playground/web/node_modules/.pnpm/@astral-sh+ruff-wasm-web@0.6.9/node_modules/@astral-sh/ruff-wasm-web/ruff_wasm.js:815:51)
at async runRuff (file:///a:/GitHub_Learning_Completed_Course/100+%20Python%20%E9%A1%B9%E7%9B%AE%E6%8C%91%E6%88%98/ruff-playgroud/black-playground/web/src/lib/ruff.js:12:3) {
[cause]: Error: not implemented... yet...
at makeNetworkError (node:internal/deps/undici/undici:8968:35)
at schemeFetch (node:internal/deps/undici/undici:10363:34)
at node:internal/deps/undici/undici:10212:26
at mainFetch (node:internal/deps/undici/undici:10231:11)
at fetching (node:internal/deps/undici/undici:10179:7)
at fetch (node:internal/deps/undici/undici:10048:20)
at fetch (node:internal/deps/undici/undici:13183:10)
at fetch (node:internal/bootstrap/web/exposed-window-or-worker:72:12)
at __wbg_init (file:///A:/GitHub_Learning_Completed_Course/100+%20Python%20%E9%A1%B9%E7%9B%AE%E6%8C%91%E6%88%98/ruff-playgroud/black-playground/web/node_modules/.pnpm/@astral-sh+ruff-wasm-web@0.6.9/node_modules/@astral-sh/ruff-wasm-web/ruff_wasm.js:810:26)
at runRuff (file:///a:/GitHub_Learning_Completed_Course/100+%20Python%20%E9%A1%B9%E7%9B%AE%E6%8C%91%E6%88%98/ruff-playgroud/black-playground/web/src/lib/ruff.js:12:9)
}

Are you running the code in node? If so, then you want to use the ruff-wasm-nodejs package https://www.npmjs.com/package/@astral-sh/ruff-wasm-nodejs

example is fault  , I  have to  read source to  use  api

I'm not sure I understand. Were you able to resolve the issue?


web: update dependencies

What problem does this PR solve?
Update NPM dependencies (resolve the vulnerability warnings).
What is changed and how it works?
Check List
Tests

Manual test (add detailed scripts or steps below)

the web page looks fine

Side effects
Related changes

LGTM


Problem with unrecognized argument

When I run the CodeGen, the preprocessing.py it is asking for various arguments and when I pass those argument its showing unrecognized arguments. Below are my codes.
%run codegen_sources/preprocessing/preprocess data/test_dataset 20  python obfuscation 8 500 200 400 roberta False data/bpe/cpp-java-python/vocab False data/bpe/cpp-java-python --1
usage: preprocess.py [-h] [--local LOCAL]
[--local_parallelism LOCAL_PARALLELISM]
[--langs LANGS [LANGS ...]]
[--mode {obfuscation,monolingual,monolingual_functions,obfuscation_functions}]
[--train_splits TRAIN_SPLITS] [--job_mem JOB_MEM]
[--tokenization_timeout TOKENIZATION_TIMEOUT]
[--bpe_timeout BPE_TIMEOUT]
[--train_bpe_timeout TRAIN_BPE_TIMEOUT]
[--bpe_mode {fast,roberta}]
[--fastbpe_use_vocab FASTBPE_USE_VOCAB]
[--fastbpe_vocab_path FASTBPE_VOCAB_PATH]
[--keep_comments KEEP_COMMENTS]
[--fastbpe_code_path FASTBPE_CODE_PATH] [--ncodes NCODES]
[--percent_test_valid PERCENT_TEST_VALID]
input_path
preprocess.py: error: unrecognized arguments: 20 python obfuscation 8 500 200 400 roberta False data/bpe/cpp-java-python/vocab False data/bpe/cpp-java-python --1

Hi,
Is this line the command you run in a terminal?
%run codegen_sources/preprocessing/preprocess data/test_dataset 20 python obfuscation 8 500 200 400 roberta False data/bpe/cpp-java-python/vocab False data/bpe/cpp-java-python --1

I believe that you are not passing the arguments correctly. Maybe your script omits the -- before named arguments (e.g. --local_parallelism 20)


Improve the payload properties when scheduling a job

We have updated the payload properties when scheduling/run-now a job. Take reference from Job Manager app

Not a priority. So closing this task for now.


Update package.json

Update repo URL (fix for https://github.com/OctoLinker/OctoLinker/issues/627)

Thank you!


Fix cors_allowed_origins

what

Explicitly pass variable as list despite already being list

why

My only guess is there's a bug in TF HCL validation whereby it doesn't detect the input is already a list

Tested.


Message reaction change Activity (C#)

See parent

@clearab, is this still needed?

Yes, Teams implemented that payload change and we should update the Activity dispatch accordingly.

note: this is a breaking change to the sdk behavior, and should maybe be released with a major version update?

I'm not sure if it counts as a breaking change, the way the SDK would handle and activity with an add and a remove would be wrong right now - I think it would only dispatch to the reactionAdded handler. The change should just add support for dispatching correctly in the case both were sent.
However, when I wrote this one up v5 wasn't in the works yet - I'm happy to make this part of that set of changes.

Currently, if both are sent then both handlers will execute. First Added, then Removed:
        protected virtual async Task OnMessageReactionActivityAsync(ITurnContext<IMessageReactionActivity> turnContext, CancellationToken cancellationToken)
        {
            if (turnContext.Activity.ReactionsAdded != null)
            {
                await OnReactionsAddedAsync(turnContext.Activity.ReactionsAdded, turnContext, cancellationToken).ConfigureAwait(false);
            }

            if (turnContext.Activity.ReactionsRemoved != null)
            {
                await OnReactionsRemovedAsync(turnContext.Activity.ReactionsRemoved, turnContext, cancellationToken).ConfigureAwait(false);
            }
        }

https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder/ActivityHandler.cs#L273
With this change, only the new handler will execute (breaking any existing bots with code in the previous two handlers).
        protected virtual async Task OnMessageReactionActivityAsync(ITurnContext<IMessageReactionActivity> turnContext, CancellationToken cancellationToken)
        {
            if (turnContext.Activity.ReactionsRemoved != null && turnContext.Activity.ReactionsAdded != null)
            {
                await OnReactionsChangedAsync(turnContext.Activity.ReactionsAdded, turnContext.Activity.ReactionsRemoved, turnContext, cancellationToken).ConfigureAwait(false);
            }
            else
            {
                if (turnContext.Activity.ReactionsAdded != null)
                {
                    await OnReactionsAddedAsync(turnContext.Activity.ReactionsAdded, turnContext, cancellationToken).ConfigureAwait(false);
                }

                if (turnContext.Activity.ReactionsRemoved != null)
                {
                    await OnReactionsRemovedAsync(turnContext.Activity.ReactionsRemoved, turnContext, cancellationToken).ConfigureAwait(false);
                }
            }
        }

https://github.com/microsoft/botbuilder-dotnet/pull/6089
v5 👍

Merged into the v5 brach


[improve] When pr contains only md file shouldn't trigger maven compile.

When pr contains only md file shouldn't trigger maven compile.

@wolfboys I have test in my fork project. This is test pr https://github.com/GOODBOY008/streamx/pull/1  and this commit is merged to my dev branch.

LGTM


Trimmed down permission for dashboard service account

What this PR does / why we need it:
This PR trims down the dashboard service account to the minimum possible per https://github.com/kubernetes/dashboard/wiki/Access-control#default-dashboard-privileges .
Current permission allow users to skip dashboard authentication and inherit cluster-admin priviledges nullifying the login process and RBAC resulting in an unsecure cluster.
With the proposed configuration, skipping login results in no permissions.
For development clusters, users can still reinstate previous permission manually granting cluster-admin permission to the dashboard service account.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #2425
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:
Security: skipping kubernetes dashboard's login now effectively results in no permissions.
Logged in user will receive permission per RBAC profile.

 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.:x: EPinci sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@epinci it looks like these changes were not rolled into the v1.10 version!
https://github.com/Azure/acs-engine/blob/master/parts/k8s/addons/kubernetesmasteraddons-kubernetes-dashboard-deployment.yaml

@ewok2030 Thanks! fixed here: https://github.com/Azure/acs-engine/pull/2816


handle multiple source -> one table mappings

If there are two sources that point to the same table, the topological
sort creates graph edges against just one of the two sources.  This can
result in fixture loading-order issues.
When processing sources, keep an arrayref and create edges against
all sources for a particular table.

This has been released as 0.50010. Thanks for the patch!

Awesome! Thanks!
Sent from my iPhone

On Jul 27, 2017, at 9:54 PM, Rob Kinyon<EMAIL_ADDRESS>wrote:
This has been released as 0.50010. Thanks for the patch!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


opensync: rrm support

Add rrm manager
Add rrm config params to schema

Signed-off-by: Arif Alam<EMAIL_ADDRESS>

merged into pending, Thanks !


chore: bdd-spring-1574264675 to 0.0.1

chore: Promote bdd-spring-1574264675 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Prevent double escaping of comments

See https://github.com/openshiftio/openshift.io/issues/4132

Codecov Report

Merging #2236 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #2236   +/-   ##
=======================================
  Coverage   69.84%   69.84%
=======================================
  Files         170      170
  Lines       15856    15856
=======================================
  Hits        11074    11074
  Misses       3769     3769
  Partials     1013     1013

Impacted Files
Coverage Δ

controller/comments.go
80.6% <100%> (ø)
:arrow_up:

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 c2b1184...38f1e33. Read the comment docs.

I do not like the idea of having this assertComment function https://github.com/fabric8-services/fabric8-wit/pull/2236/files#diff-a1fbe5131a7a25c59d631349582ec3ebR145 .
We could use golden files to validate the comments.

[test]


osd-18602-updated-files

Since PR 126, files from LPSRE were still pointing to the previous directory CSSRE. The files were updated to the correct path now.

/retest

/lgtm
/approve


Design for autoconversion UI

Background
We are allowing Readdy and cytosym file types to be auto converted into .simularium for ease of use
Fields below are for internal team use
User needs statement(s)
As a computational scientist, I want to load my own data without writing a Python script and instead use a UI to provide a file and enter additional info so that I can write one less script.
Size [XS, SM, M, LG, XL]
Expected level of user research (includes discovery) and design effort
Medium
Requirements

A UI that supports intake of information related to the file (different for each file type)

Deliverables (acceptance criteria)

UI + Interaction design
Thoughtful review team and internal stakeholders and users for feedback

Stakeholder(s)
Megan, Blair, Graham, Matt A (has his cytosym files but maybe not necessary to review with)
Use Case
As a user, I want to load my own data without writing a Python script and instead use a UI to provide a file and enter additional info.
Acceptance Criteria
Designs ready for development of production autoconversion UI #346
Details
Each engine that we support has it's own required data fields. Templates for the fields for each can be found here
The prototype UI will be a good reference https://github.com/simularium/simularium-viewer/issues/237

Unstyled smoldyn example form:


sweet-graduate-resume:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: This is a simple and neat graduate resume template useful for college students. It was designed for engineering students but I don't see why it wouldn't work for anyone else.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Also, the new structure is better, but not exactly what most people will expect: the code that should not be edited by users (template/template.typ, and the fontawesome copy if you decide to keep it) should not be in the template directory, but outside of it.

Hi @elegaanz . I finally understood what you meant when I saw. I've pushed the updated version.

Thanks :)


rgw: add AWS4 auth testing support

This patchset adds AWS4 auth testing support.
Signed-off-by: Javier M. Mellid<EMAIL_ADDRESS>

@jmunhoz my main issue here is whether there's a way to have all common tests run twice, for both aws2 and aws4.

@yehudasa I changed the approach to keep all headers test cases (common, aws2 and aws4) in the original file so the current behavior is running common and aws2 test cases but skipping the specific aws4 test cases as expected.

@jmunhoz  How do I run AWS4 auth tests?  When setting host = <IP_ADDRESS> and region = use-east-1 I encounter the same symptoms as boto/boto#2624:
$ S3TEST_CONF=s3proxy.conf S3_USE_SIGV4=true ./virtualenv/bin/nosetests
...
UnboundLocalError: local variable 'region_name' referenced before assignment

@andrewgaul  I found our current boto version not very reliable to test AWS4 in the past. I faced similar issues with regions and/or adding the port in an explicit way (80, 8080 and so on). Both values are used in AWS4 though. It looks our boto version is quite old and it could make sense going with the last boto3 instead of fixing/supporting the current boto version. Not sure though. In the meanwhile, I am using my local DNS to bypass these issues with boto in order to have AWS4 in place ASAP.
To test AWS4 I use one region requiring AWS4 only. In my case I map 's3.eu-central-1.amazonaws.com' to <IP_ADDRESS>. This way I don't depend on boto negotiating AWS2/AWS4 and/or using different parameters. It is useful to develop/test.
With this previous configuration you can run the following command to test AWS4:
$ S3_USE_SIGV4=1 S3TEST_CONF=your.conf ./virtualenv/bin/nosetests s3tests/functional/test_headers.py -a auth_aws4
The previous command would run test cases making sense in aws4_only.


File merge on write capabilities

Is your feature request related to a problem? Please describe.
Lula will be operating on all of the OSCAL models in some way as context is used across models to build more provenant workflows. This could mean many artifacts that are required for validation and evaluation as well as generation or evidence being required for even a single repository.
Instead of requiring a file per model - the current types already support a combined definition for all models. When lula performs any write operation to a file - it should read any existing file and merge it into the appropriate model before writing the result. Meaning that models could be co-located in the same file.
Describe the solution you'd like

Given a file exists containing some OSCAL data
When any command is executed that writes to a file (generation, validation, etc)
Then Lula supports retaining the existing data and performing a merge as appropriate.

Describe alternatives you've considered
Keep all files as distinct artifacts where each model is separate of one another.
Additional context
This should cut down requirements for 1 -> many OSCAL files for every "package"

Common function that can handle each model update in the event of existing data would be good here.


Bug fix: could select the form default values again in multiple mode with minimumInputLength 0

Codecov Report

:exclamation: No coverage uploaded for pull request head (initial-dropdown-values-fix@5dfde7f).

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 4c15e73...5dfde7f. Read the comment docs.


migrated to null safety

please have a look and let me know if any further changes required. I migrated the project to null-safety and run flutter create . command in example directory to create ios, android and web project.

Awesome! Thank you for this.
Don't know about the other changes, but it's a shame null safety hasn't been merged into the library yet.

I think we should fork and create a new package.
It is no longer active since 2019.


[5.7] Fix chevron icon in collapsed top nav bar is not visible

Rationale: Fixes a bug where the chevron icon in the nav bar disappears at a certain zoom level
Risk: Low
Risk Detail: Isolated CSS changes in the nav bar.
Reward: High
Reward Details: UI is now consistent when users zoom in/out
Original PR: https://github.com/apple/swift-docc-render/pull/179
Issue: : SR-16120
Code Reviewed By: @hqhhuang @dobromir-hristov
Testing Details: In safari, enter simulator, set browser window to 800 x 700 VP, 1x res, Safari 15.3 - macOS - Mac; Zoom out to around 85%, verify that the chevron icon in nav bar appears consistently when zooming in/out

@swift-ci test

@swift-ci test

Merging. Arthur approved.


Add support for redis 5 zpopmin and zpopmax

Description: Add support for redis 5 zpopmin and zpopmax. They behave as a simple comment as the first argument is the key of the sorted set.
Risk Level: low
Testing: N/A
Docs Changes: Updated list of supported commands
Release Notes: Added new commands to release note.

LGTM

Thank you! 🙏


Update WaitOnAdress / WakeByAddressSignle / WakeByAddressAll to refer to the correct dll

The page Windows 8 API Sets lists api-ms-win-core-synch-l1-2-0.dll.
Debugging shows it actually maps to KernelBase.dll
Attempts to get these functions from either KernelBase.dll or api-ms-win-core-synch-l1-2-0.dll work. Attempts to get them from Kernel32.dll fail.
So I'm really not sure which of KernelBase.dll or api-ms-win-core-synch-l1-2-0.dll is correct, but sure not Kerne32.dll

@AlexGuteniev : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Hi @AlexGuteniev, can you resolve the merge conflicts and resubmit? Thanks!

@AlexGuteniev : Thanks for your contribution! The author(s) have been notified to review your proposed change.

I was not able to merge it in a clean way, I reverted and re-applied changes.

@AlexGuteniev : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Unable to integrate with cocoapods. Help anyone!

have you tried running this:
``

This is a real problem. +1. I tried it with Tide Pods, and didn't get anything either


Guard SVV_DISKUSAGE behind --metadata-stats

In the medium term, I'd love to unguard this if we can prove that it doesn't have significant cost, the way the table stats do.


Add favicon

Fixes #120

Maybe the theme color shouldn't be #ffffff considering that the logo is white? Maybe we should use an orange version of the logo to ensure it's visible in both light and dark mode?
Maybe we should control the theme with meta tags instead?
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/meta/name/theme-color

Yes, orange would work! Regarding the theme, I trust your judgment, I just used a tool to generate the favicons and this came along with it.

Rebased locally and committed in 925a9c9f18b0369fe61af2f7a87b779efe4b72a2.


download Flannel releases to /opt/

/usr/ is readonly on Core OS, and /opt/ should work on Ubuntu too.
Intallation failure on Core OS was introduces in #588
/cc @unicell @danehans @stephenrlouie

@rutsky @eparis fyi this patch works in my coreos testbed. Before getting merged, I would like the inline comments addressed. Otherwise, thanks a bunch @rutsky for fixing the breakage.

@eparis, @danehans I fixed your comments, please take a look.


Facelift of the RTG Scan in the streamlit app

The RTG scan is a cool utility which produces graphs like the ones below.
I'd like to do the following:

[ ] Clean up the code a bit (maybe just split code into more discrete sections)
[ ] Start using T-Lens for the Decomposition
[ ] Enable Decomposition at different levels
[ ] The visualization of attention is crap so fix that.
[ ] sort the grid of correlations in the RTG decomp

Looking good!