Documentation question - How to get to the queue from a service class (outside of controller plugin)

First of all, thanks to all of the contributors on this project. This is an absolutely fantastic module.
I do have one question after pouring through the documentation and perhaps someone has an answer to this.
I have most of my core busines functionality ran from service classes that are fat and really thin controller methods (Apigility REST api -- code connected). How does one access the Queue from a service like this? My initial assumption was to make my service implement the QueueAwareInterface; however, that didn't do what I expected. In fact, nothing appeared to happen --- no queue was injected using setter method injection.
My questions are as follows:

What should I reference to get the queue to be able to push ($queue = $this->getServiceLocator()->get('queue');

Is there an X-aware interface implementation that has already been incorporated but just not documented using ZF2 initializers?

Hi !
The QueueAwareInterface, as documented here: https://github.com/juriansluiman/SlmQueue/blob/97e16e9d87b39e98184276d36faddb8dd85da23c/docs/4.QueueAware.md only works for job.
The problem by using this interface for your service is that even if you create an initializer, how the initializer could know which queue inject into a given service? It cannot know.
That's why I think injecting the queue directly is the best way:
class MyService
{
  public function __construct(QueueInterface $queue)
  {
  }
}

This way you can control which queue to inject right into your factory.
Hope it helps! ;)

And an addition, inject the proper queue with your service factory:
createService($sl) {
    $queueManager = $sl->get('SlmQueue\Queue\QueuePluginManager');
    $queue        = $queueManager->get('my-queue');

    return new MyService($queue);
}

See also https://github.com/juriansluiman/SlmQueue/blob/master/docs/2.Configuration.md#queues

Thanks, that's exactly what I was looking for. This worked like a charm!

Thanks for this. It looks like I'm about halfway there to being able to submit jobs properly.
When using invokables things appear to work just fine.
When using factory-instantiated jobs -- it's another story. I'm not sure if my application is configured oddly or I'm just doing something wrong ... but here's what I'm getting:
        $jobManager = $this->_queue->getJobPluginManager();
        $job = $jobManager->get('MyFactoryInstantiatedJob');
        $job->setContent($workload);
        $this->_queue->push($job);

error:
Sep  1 23:34:22 com Zend\Log[1360]: PHP Fatal error:  Call to protected Doctrine\ORM\EntityManager::__construct() from context 'Zend\ServiceManager\AbstractPluginManager' in /var/www/domains/com.project.example/vendor/zendframework/zend
-servicemanager/src/AbstractPluginManager.php on line 207
service instantiation in module.config.php:
'ExampleService' => function ($serviceManager) {

                $queueManager = $serviceManager->get('SlmQueue\Queue\QueuePluginManager');
                $queue = $queueManager->get('default');

                $service = new ExampleService($serviceManager->get('Doctrine\ORM\EntityManager'), $serviceManager->get('ExampleLoggerFacility'), $serviceManager->get('AcMailer\Service\MailService'), $queue);
                return $service;
            }

Any thoughts on why this would result in a doctrine error? I see that the JobPluginManager is a child on AbstractPluginManager.
Removing the code related to the JobPluginManager removes the exception -- so the exception definitely has something to do with how the job DI is being handled as far as I can tell.
Any thoughts on this would be helpful =)

I'm guessing that I need to inject in the Entity Manager using something along the lines of a worker strategy.
I've tried using the trait provided by doctrine orm module; however, that too didn't return a working entity manager.
As far as I understand the docs far it seems like I would need to:

push in the entity manager around EVENT_PROCESS_JOB w/ priority around 100.
---> somehow have that dependency available from the pluginmanager for factory instantiation because the ORM isn't available from the abstract plugin manager because it is not the true service locator.
Kill an existing DBAL connection and create a new one for each job (prevent the mysql connection lost type of errors)
re-connect to DB on job process.

But... I'm stuck. I'm not sure how or where I would pull from the shared event manager and provide that dependency into the local event manager on a per-job or per-worker basis.

If you have any dependencies in your job you should (well are encouraged) to create a factory for that particular job.
The factory must then be registered within the configuration of slm-queue for the job plugin manager (which is a service locator)
Then within your factory you should be able to pull any dependency from the service locator and pass it along to the job constructor. The global service locator can be pulled from the plugins managers service locator.
$sm->getServiceLocator()->get('doctrine.entitymanager.orm_default');
If you want I can pull up some code from one of mine as an example. Ping me. At a normal keyboard in an hour or so...

@basz
Thanks! That's actually the magic there that did the trick: adding in that parentlocator fetching tidbit.
$parentLocator = $serviceLocator->getServiceLocator()->get('doctrine.entitymanager.orm_default'); and then passing that in to my job as a dependecy.
Rockstar.


Update README.md

fixed grammar, plz sir need to get t-shirt

@closebracket 503 t-shirt service temporarily not available, please try again october

No


Full-screen break shows under current window 2nd monitor

I'm using version 1.0.0
Windows 10 x64. Dual monitor.
In Stretchly I have "Show breaks in Full Screen" set and "Show on all monitors"
On the primary screen, stetchly correctly draws on top of other windows.
On the secondary screen, stretchly draws underneath other windows. See attached screenshot.

Agree. Just want to post this. Experience the same thing here.
Win 10 2004, Dual Monitor in Extend Mode
Stretchly 1.0

Thanks for the report! How is it behaving when you show breaks not fullscreen? Is it also behind the other apps?
My workaround for 1.0 was to basically don't use fullscreen functionality (as it was broken) but to basically show normal window but make it as big as the screen. So I wonder if the "Show breaks in Window" works the same.

The window mode works well. But how to change the size of it? I tried dragging the edges or corners and it doesn't work. (still prefer fullscreen btw)

It works fine in "window" mode, where the window draws over the top of other windows.  Maybe a work around would be to set the size of the "window" mode to like 95% :D

@pro-logic in next release, I plan to make break "window" to be 70-80% of the screen-size
@XuShenLZ size of the break window can't be changed. What I meant was workaround in the code, for the fullscreen, is that I make regular window the size of the screen, thus pretending that it's fullscreen. That's why I am surprised why it does not work, as regular window works. Will have to check the code to see if there's another difference.

Thanks! I am actually confused by the bug's pattern. During my use these days (fullscreen, extend on 2nd monitor), sometimes it will not be hidden (have saw it on top of Matlab when that window is not the focus), but sometimes it will be covered (when using Edge browser, Scretchly will normally be covered no matter Edge is my current focus or not). Haven't found any clear clue, but the message is that the behavior is "unstable"...

Yeah, with electron the message is always "it's unstable" :smile:

Maybe this will be interesting to test: https://github.com/electron/electron/issues/20933#issuecomment-685136742

Hi. thanks for your great app. I recently installed V1.1.0 on MS win10. there is no problem in "Window" mode . but when choose "Fullscreen" , if a window is open , breaks is not shown.I tried to minimize all windows and then every thing works well.
As you said, I changed window size and see that break is running behind  the all windows in first monitor.
Interestingly, this problem does not exist in the second monitor and breaks display at top of all open windows

Hey guys, I can't build on Windows for some reason, but I created branch with simple fix mentioned in Electron https://github.com/electron/electron/issues/20933#issuecomment-685136742. If anyone is running Stretchly from source on Windows, could you try branch fix/win-fullscreen?


AirISP command not found in mac

AirISP command not found in mac
use mac ArduinoIDE to flash rom to air001:
user@MacBook-Pro <IP_ADDRESS> % "/Users/user/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP" --chip "auto" --port "/dev/cu.usbserial-0001" --baud 115200 --before default_reset --after hard_reset write_flash -e -p 0x08000000 "/private/var/folders/z5/7h2gkqpn3x70bm28dp_l5ktw0000gn/T/arduino/sketches/63087252B0F3D75209FD63EB4F210E8B/sketch_jul17a.ino.bin"
zsh: permission denied: /Users/user/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP
user@MacBook-Pro <IP_ADDRESS> % sudo "/Users/user/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP" --chip "auto" --port "/dev/cu.usbserial-0001" --baud 115200 --before default_reset --after hard_reset write_flash -e -p 0x08000000 "/private/var/folders/z5/7h2gkqpn3x70bm28dp_l5ktw0000gn/T/arduino/sketches/63087252B0F3D75209FD63EB4F210E8B/sketch_jul17a.ino.bin"
then i into sudo：
sudo: /Users/user/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP: command not found
user@MacBook-Pro <IP_ADDRESS> % sudo: /Users/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP: command not found
My ArduinoIDE version：2.0.4
My OS version：13.0 (22A380)

Please try to save the project to any user directory first and then try again.

it's also have the same problem:
MacBook-Pro <IP_ADDRESS> % sudo "AirISP" --chip "auto" --port "/dev/cu.usbserial-0001" --baud 115200 --before default_reset --after hard_reset write_flash -e -p 0x08000000 "/private/var/folders/z5/7h2gkqpn3x70bm2_l5ktw0000gn/T/arduino/sketches/0F5B328EE29737EFCB54222A4873D071/sketch_jul17a.ino.bin"
sudo: AirISP: command not found

That's odd, if it's easier for you, can you download the AirISP tool separately to run it? Use in the terminalhttps://github.com/Air-duino/AirISP/releases/tag/<IP_ADDRESS>
AirISP -v

command in the terminal.

I downloaded (AirISP-osx-arm64, AirISP-osx-x64) and tried the following commands in the terminal, but none of them seemed to work. Is my command wrong?
casillas@MacBook-Pro AirISP-osx-x64 % pwd
/Users/casillas/Downloads/AirISP-osx-x64
casillas@MacBook-Pro AirISP-osx-x64 % AirISP
zsh: command not found: AirISP
casillas@MacBook-Pro AirISP-osx-x64 % ./AirISP
zsh: permission denied: ./AirISP
casillas@MacBook-Pro AirISP-osx-x64 % sudo ./AirISP
sudo: ./AirISP: command not found
casillas@MacBook-Pro AirISP-osx-x64 % AirISP -v
zsh: command not found: AirISP
casillas@MacBook-Pro AirISP-osx-x64 % sudo ./AirISP -v
sudo: ./AirISP: command not found

I tried it in window10 and it works fine. ^ - ^

I downloaded (AirISP-osx-arm64, AirISP-osx-x64) and tried the following commands in the terminal, but none of them seemed to work. Is my command wrong?
casillas@MacBook-Pro AirISP-osx-x64 % pwd /Users/casillas/Downloads/AirISP-osx-x64
casillas@MacBook-Pro AirISP-osx-x64 % AirISP zsh: command not found: AirISP
casillas@MacBook-Pro AirISP-osx-x64 % ./AirISP zsh: permission denied: ./AirISP
casillas@MacBook-Pro AirISP-osx-x64 % sudo ./AirISP sudo: ./AirISP: command not found
casillas@MacBook-Pro AirISP-osx-x64 % AirISP -v zsh: command not found: AirISP
casillas@MacBook-Pro AirISP-osx-x64 % sudo ./AirISP -v sudo: ./AirISP: command not found

Can you try to set the default rwx using the chmod command? I think it might be a permissions issue that is causing this.

after set the default rwx using the chmod command, a new problem arises:
casillas@MacBook-Pro <IP_ADDRESS> % ls -l
total 25704
-rwxrwxrwx@ 1 casillas  staff  13063691  7 17 18:44 AirISP
-rwxrwxrwx  1 casillas  staff     16416  7 17 18:44 AirISP.pdb
-rwxrwxrwx@ 1 casillas  staff     70464  7 17 18:44 libSystem.IO.Ports.Native.dylib
casillas@MacBook-Pro <IP_ADDRESS> % sudo "/Users/casillas/Library/Arduino15/packages/AirM2M/tools/AirISP/<IP_ADDRESS>/AirISP" --chip "auto" --port "/dev/cu.usbserial-0001" --baud 115200 --before default_reset --after hard_reset write_flash -e -p 0x08000000 "/private/var/folders/z5/7h2gkqpn3x70bm28dp_l5ktw0000gn/T/arduino/sketches/0F5B328EE29737EFCB54222A4873D071/sketch_jul17a.ino.bin"
zsh: killed     sudo  --chip "auto" --port "/dev/cu.usbserial-0001" --baud 115200 --before


Change color of cache description to make it readable

When opening the details of the mentioned cache (GC2EAZW) it shows just empty lines (or black on black ?).
I am using 2011.08.14 nightly, but I dont know wether this occured before.
On the website the description is ok.

I guess I have another example: GC12G98
THe text description of Waypoints 7 to 9 is just "empty" (black on black) in c:geo.

I currently can see the descriptions of all those caches by removing the black colors like this in parseLongDescription:
if (!Settings.isLightSkin() && description.contains("color="#000000")) {
description = StringUtils.replace(description, "color="#000000", "");
}
It only does not work for one paragraph in one of the examples, where the font color setting has a line break in the source like this:
color=
"#000000"
I'm not exactly sure if my change could have other side effects (because it does not test for the font tag, but only for the color attribute). Anybody with more insight in HTML to give this some tests?

Problem: If the cited code is complete it would leave a double quote in the tag. Most html interpreters are foult tolerant enough to cope with this, but I'd prefer not to produce something invalid.
Regarding the question, it can of course produce issues, as the cache owner can be as creative as he wishes with his description (e.g. place a red box, a white one on top, and then black text). In order to not produce more problems with a, probably smaller, number of other caches I'd prefer to switch the description background to white if we detect that the owner used colour attributes in his description.

I've used gray now. One of the above caches had black and white text color...


support rocketmq request-reply mode

Rocketmq already support request-reply mode since 4.6，how about add a rocketmq plugin in dubbo-remoting，so that a rocketmq consumer can expose a service to dubbo registry and can be invoked by a dubbo consumer.

Please go ahead and submit your idea or pull request.


New installation instruction for Gentoo/Funtoo

Untested in Funtoo, but I think it should work.

:+1:


Fix locale issues

bg-x locale was accidentally commited.
yo locale didn't have the right comment syntax, and wasn't inserted on the moment website.

Merged in 372dfd1f39f159b391762fe192e3ac85e9340d1a


VHDL error - Cannot open load file: Permission denied, c:/emacs/bin

Description
I can not use the Beautifier on Windows for VHDL because I keep getting the following error:

Steps to Reproduce

Windows 10
Beautifier configurations for VHDL:

Emacs is already added to the PATH in Windows

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


Problem with oscilating loop & new codes for EGLO fans

Home Assistant version
2023.7.3
SmartIR version
1.17.7 (latest as of today)
SmartIR configuration

smartir:
fan:
  #Вентилятор в гостиной
  - platform: smartir
    name: Вентилятор
    unique_id: lr_fan
    device_code: 1230
    controller_data: remote.broadlink_rm4_pro
  #Вентилятор в спальне
  - platform: smartir
    name: Вентилятор
    unique_id: br_fan
    device_code: 1231
    controller_data: remote.broadlink_rm4_pro

Describe the bug
I have 2 identical fans with different remotes and set of commands to control them individually. Both of them have 5 speeds and revert button (oscillate). When I just manually send commands, I can send oscillate only once, and then change speeds, turn them on/off and fan will stay.
It's enough to send oscillate command once, and it will be stored even after power outage.
If I turn on oscillate through fan entity created by SmartIR, every speed change will revert this fan, like I send oscillate command every time i change any setting on it
Additional context
Adding 2 files with codes for this model. People can relearn their fans to understand this codes instead of their own remote to save time reading these on their own.
Архив.zip

I haven't done a pull request since 2 identical fans gave me 2 different codesets, and this was made by manufacturer on purpose, so that each remote controls only the fan it's binded to, so I have no idea how to store 2 versions of this codeset by the project rules

Any news about the bug?


Next release?

Readme describes the use of the 1.2 image, but also describes the scheduling parameters which, as I just found out, are not included in release 1.2.
It's been a while, so when can we expect a new release?

I deployed it directly from master and it is working fine for the last 1 month. Wonder why the release was not created?


[R-package] Reduce duplication in configure.ac

I've been using xgboost's configure.ac as a reference, and noticed an opportunity for reducing duplication. Right now the location for libomp and the OPENMP_CXXFLAGS are hard-coded in two places.
This small PR proposes removing the second hard-coding to reduce duplication and the risk of mistakes if that location or those flags are updated in the future.

Do we need to re-generate the configure script using autotools?

Do we need to re-generate the configure script using autotools?

Ah yes I can do that if you'd like! I'd be sure to use automake 2.62 since I see that's the version you used before.
Let me know if you'd like me to do that.

Sure.  Please help generating the config file.

Updated in https://github.com/dmlc/xgboost/pull/5693/commits/d9a92f2ebc5734e00ee31e7ed4dd386286619133.
I originally used autoconf 2.62., but it looks like the last time this file was updated, autoconf 2.69 was used https://github.com/dmlc/xgboost/blob/cb3ed404cff3636248b04e7081bfccfefec3e3d8/R-package/configure#L3
So I decided to use 2.69, and I updated the AC_PREREQ call in configure.ac.
I updated using autoconf 2.69 installed in an Ubuntu 18.04 environment, using this command
autoconf \
  --verbose \
  --output configure \
  configure.ac

Codecov Report

Merging #5693 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5693   +/-   ##
=======================================
  Coverage   82.56%   82.56%
=======================================
  Files          11       11
  Lines        2598     2598
=======================================
  Hits         2145     2145
  Misses        453      453

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 646def5...d9a92f2. Read the comment docs.


Shaunna l patch 1 4

This page seems to go out of date a lot due to updates to Azure and SmarterU. I manage the content at help.smarteru.com and we make all updates there. Would be easiest to redirect to that page so documentation is always current in both spots.

Fixed conflicts.

@SyntaxC4 - Could you review the proposed updates to the README.md file for the azure-docs repo?
@jeevansd - Could you review the proposed updates to the SmarterU SaaS tutorial?
Thanks.

@ktoliver Don't remove the app side instructions here. Please add them back.

@ktoliver Don't remove the app side instructions here. Please add them back.

@jeevansd I think this is meant for @ShaunnaL, the pull request contributor. @ShaunnaL could you update the PR based on this feedback? Thanks!


Example of project with logback.xml

Hi!
Could you please provide some Quarkus app example where logback.xml is handled by application and Slf4j?
I have done all according documentation where I was supposed to include it dependency, but it's not working for me.

Can you provide a reproducer where it is not working so I can see what you are trying to do?

@stuartwdouglas , I've fixed problem. It was due to wrong configuration of maven-resources-plugin. I deleted last one.
But I have strange notification on start:
2021-12-15 16:23:12,403 WARN  [io.qua.dep.ste.ClassTransformingBuildStep] (build-28) Could not removed configured resources from the following artifacts as they were not found in the model: [ch.qos.logback:logback-classic::jar]
What does it means? How to solve it.
Repo: https://github.com/SMakhrov/quarkus-logback

That error will be fixed by: https://github.com/quarkusio/quarkus/pull/22272

I have a question.  I'm usiong Quarkus 2.5.4.Final and I'm using
<dependency>
            <groupId>io.quarkiverse.logging.logback</groupId>
            <artifactId>quarkus-logging-logback</artifactId>
            <version>0.8.0</version>
        </dependency>

when I launch the application with mvn quarkus:dev the logs looks OK, but when I launch my project in junit, the configuration in application.properties are ignored because I don't see my debug logs when I run my junits
I asked the question here too : https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Quarkus.20-.20Logging.20Logback


Cannot load schema into Scala schema repo

I run the Scala static repo and run it as follows:
java \
-Dconfig.file=/path/to/application.conf \
-jar /path/to/iglu-server-0.1.0 com.snowplowanalytics.iglu.server.Boot

I then create the super API key as per the documentation, in PSQL. (Note that Postgres is running on the same box as the server repo):
insert into apikeys (uid, vendor_prefix, permission, createdat)
values ('63235577-b617-4a62-a0f0-df11ee3009a7', '.', 'super', current_timestamp);

I use it to  create a key for com.example_company, using cURL:
$ curl \
>   localhost:8080/api/auth/keygen \
>   -X POST \
>   -H "api_key: 63235577-b617-4a62-a0f0-df11ee3009a7" \
>   -d "vendor_prefix=com.example_company"
{
  "read" : "3e2b35e8-11eb-4954-b729-0c7f0e8d1220",
  "write" : "08cd8c22-800f-4f35-9b72-f8b7915c2fc6"
}

I then want to add an example schema, using the write key created above:
curl \
  localhost:8080/api/schemas/com.example_company/example_event/jsonschema/1-0-0 \
  -X POST \
  -H "api_key: 08cd8c22-800f-4f35-9b72-f8b7915c2fc6" \
  -d "{
  \"$schema\": \"http://iglucentral.com/schemas/com.snowplowanalytics.self-desc/schema/jsonschema/1-0-0#\",
  \"description\": \"Schema for an example event\",
  \"self\": {
    \"vendor\": \"com.example_company\",
    \"name\": \"example_event\",
    \"format\": \"jsonschema\",
    \"version\": \"1-0-0\"
  },

  \"type\": \"object\",
  \"properties\": {
    \"exampleStringField\": {
      \"type\": \"string\",
      \"maxLength\": 255
    },
    \"exampleIntegerField\": {
      \"type\": \"integer\"
    },
    \"exampleNumericField\": {
      \"type\": \"number\",
      \"maxDecimal\": 3
    },
    \"exampleTimestampField\": {
      \"type\": \"string\",
      \"format\": \"date-time\"
    }
  },
  \"minProperties\":1,
  \"required\": [\"exampleStringField\", \"exampleIntegerField\"],
  \"additionalProperties\": false
}"

But get the following error message:
The requested resource could not be found.

Hey guys, any progress on this? I'm experiencing the 'Unrecognized token 'not'' error too.

No progress yet

I'll try to have a look.

Cheers guys, I'm happy to look at trying out the static repo option in the meantime :-)

Cheers @BenFradet, @ashramsey. In the meantime, yes the static repo works pretty well...

In the second case (vendor name which doesn't contain an underscore) the problem seems to be this: when the schema is uploaded, the server attempts to validate it against the com.snowplowanalytics.self-desc/schema/jsonschema/1-0-0 schema. But that schema doesn't exist in the database, so thequery attempting to get it returns the string "not found", which causes the error
Unrecognized token 'not': was expecting 'null', 'true', 'false' or NaN

Ah! Good sleuthing. So that's a schema which is typically available in an embedded schema repo in an iglu client. I guess we need to "bootstrap" a new Iglu scala repo with all the schemas found in an embedded schema repo...

If I manually insert the self-desc schema, everything seems to work as expected! Bootstrapping is the way to go.
When the vendor name does contain an underscore, the problem is down to the first regex in this line, which doesn't allow underscores. There are several duplicates of these mistaken regexes in the repo. They should all be replaced with something based on https://github.com/snowplow/snowplow/issues/1998.

That's encouraging!

Splitting this into #80 and #81

Sorry I haven't been able to look into this, but have you looked into the unmerged stuff and particularly #65 and 3c66f0d .

Thanks @BenFradet - I wasn't aware of that branch! Do you have any idea why the tests don't pass in either branch?

You mean all the tests?

Some of them pass but others don't.
I'm getting:
[error] Error: Total 263, Failed 134, Errors 6, Passed 123
[error] Failed tests:
[error] 	com.snowplowanalytics.iglu.server.test.service.ApiKeyGenServiceSpec
[error] 	com.snowplowanalytics.iglu.server.test.service.SchemaServiceSpec
[error] 	com.snowplowanalytics.iglu.server.test.service.ValidationServiceSpec
[error] Error during tests:
[error] 	com.snowplowanalytics.iglu.server.test.actor.SchemaActorSpec
[error] 	com.snowplowanalytics.iglu.server.test.actor.ApiKeyActorSpec
[error] (test:test) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 47 s, completed 11-Nov-2015 15:34:51

Most of the failures seem to be caused by an unexpected response of 401 Credentials Rejected.

Is this on master or feature?
Because it might be linked to #63 and 8cdc771.

This is on both master and feature/scala-repo-server-0.2.0. (In both cases, after I have done #78 in that branch to get it compiling.) I notice that in feature/scala-repo-server-0.2.0, a super API key is created automatically because of the commit you linked. In master I created the super key myself.

Weird, all tests were passing when I stopped developing the project.
I think it has something to do with this since I give back a credentials rejected only there.

I think I see part of the problem - some of the tests (like this one) use API keys like 6eadba20-9b9f-4648-9c23-770272f8d627 which don't ever seem to be explicitly created.

I now remember that I used to do some manual initialization for my unit tests like adding some schemas and api keys as detailed here, sorry.

That would explain it!

@yalisassoon When you execute the command

thanks @fblundun ! Great to know...


Support structs as mro args/outs

Now we can write code like this:
#[derive(Serialize, Deserialize, MartianStruct)]
struct RnaChunk { chunk_id: u8 }

#[derive(Serialize, Deserialize, MartianStruct)]
struct SampleDef { read_path: PathBuf }

#[derive(Serialize, Deserialize, MartianStruct)]
struct SI { sample_def: SampleDef }

#[derive(Serialize, Deserialize, MartianStruct)]
struct SO { chunks: Vec<RnaChunk> }

pub struct SetupChunks;

#[make_mro]
impl MartianMain for SetupChunks {
    type StageInputs = SI;
    type StageOutputs = SO;
    fn main(&self, _: Self::StageInputs, _: MartianRover) -> Result<Self::StageOutputs, Error> {
        unimplemented!()
    }
}

which would produce the following mro:
struct RnaChunk(
    int chunk_id,
)

struct SampleDef(
    path read_path,
)

stage SETUP_CHUNKS(
    in  SampleDef  sample_def,
    out RnaChunk[] chunks,
    src comp       "my_adapter martian setup_chunks",
)

@adam-azarchs Is there any sort order for struct definitions in the mro? Currently, I am outputting them in alphabetical order.

If your question is whether mrf will reorder fields, the answer is no.  But sorted is always a good choice, especially because struct literals will have their keys sorted.

What if there are more than one struct definitions in the mro. Are they reordered in any way by mrf? Empirically, it does not seem like it.

No, in general mrf will not reorder declarations of structs, stages, or pipelines (though it will reorder structs to be before stages or pipelines, and it will always put the top-level call last).

Actually now I'm wondering a bit if it might be a good idea to expose the formatting functionality from martian's go code as a library that the rust code could link in, so that we could guarantee consistency of formatting.  It would also be potentially helpful for example for rust tools which needed to generate invocations, so they wouldn't have to shell out to mrg.  Go does make it pretty easy to produce a .a+.h, from whence you could either use bindgen or just manually write the signatures (since they're not that complicated).

Oh! I love the idea of having an API that exposes martian's go code. It would be great to get to a world where Rust code would populate a data structure representing a stage/pipeline/call etc and the martian API can create the mro.


Add deprecation warning to tf.gfile.FastGFile.

Fixes #12663.

Hi @rasmi, it's probably better to keep this interface until we implement it properly in open source. I would rather we didn't submit this. WDYT?

Nagging Assignee @drpngx: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


301 all the things

connections should be pointed @ sources/{blah}/integrations/{blee}

closed with https://github.com/silverton-io/buz-docs/pull/39


Upgrade FPS project to 0.1.2 release

Description
Add error message parameter to ClientWorkerConnector.HandleWorkerConnectionFailure that was added in: https://github.com/spatialos/gdk-for-unity/pull/576

Going to create a new branch for the upgrade


[WIP] Bump puppetlabs-apt dependency to permit 7.x

Fixes #11

Coverage remained the same at ?% when pulling 3ea493db4949d9abc60ec5437317ee2e11b8dc05 on optiz0r:11_apt_dependency into aea1e2141e735470c8efccda414700187913c443 on bodgit:master.

Coverage remained the same at ?% when pulling 3ea493db4949d9abc60ec5437317ee2e11b8dc05 on optiz0r:11_apt_dependency into aea1e2141e735470c8efccda414700187913c443 on bodgit:master.

As far as I can tell, I'm only directly using the apt::ppa defined type so this seems reasonable.

Tests seem to be failing/going slowly for Puppet 4.6.x and 4.7.x on Travis but I've run them manually and they pass fine so not sure what's going on there.


Remove the custom "is" matcher in favor of "toEqual"

jest has dramatically improved over the years, and supports Immutable values directly! No need for our custom checker.

This is great to know for nteract's code base. Thanks @leebyron! /cc @lgeiger


RTL support for Checkbox

We should check, but I think the checkmark flips around.

Turns out they don't.
Done!


chore(npm): Update dependencies

Update to webpack 4, babel 7, storybook 4 and latest flow
Closes https://github.com/gymnastjs/gymnast/issues/397

:tada: This PR is included in version 16.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix various issues in docs

[ ] Tests added for any new code
[ ] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

This PR fixes next issues in docs:

broken links
grammar mistakes
wrong punctuation
stylistic mistakes
long lines in some files (this commit contains only changes in spaces and newlines, no other changes there)
wrong numbers in lists (all 1s)
improperly formatted tables

I'm not a native speaker so some stylistic or punctuation mistakes may be not mistakes after all, so feel free to discard them.
I split these issues into separate commits, to simplify the review, though some changes may be mixed in the same commit (e.g. punctuation and long lines formatting).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0549ef7) 78.89% compared to head (16e8daf) 78.88%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2825      +/-   ##
==========================================
- Coverage   78.89%   78.88%   -0.02%
==========================================
  Files         467      467
  Lines       15950    15950
  Branches     2586     2586
==========================================
- Hits        12584    12582       -2
- Misses       3366     3368       +2

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

This is a tremendous help! Thank you for the careful review and updates and fixes to the docs. It will take me a bit of time to work thru review, but this contribution is much appreciated!

Thank you so much for this great contribution!


TypeScript ComponentOptions render return type is restricted to VNode, but Vue allows VNode[] too

Version
2.6.10
Reproduction link
https://codesandbox.io/s/vue-typescript-example-ptkbu?module=%2Fsrc%2Fcomponents%2FExample.vue
Steps to reproduce
This is a simple reproduction and the codesandbox might make more sense than this explanation.
Returning scoped slot like this works fine, when not using TypeScript:
// This works fine without TypeScript
render (h) {
  return this.$scopedSlots.default({
    action: this.action
  })
}

In TypeScript, this equivalent code fails. defaultSlot is a VNode[], but the render function only allows a VNode. Here's the example with checking all of the possible undefineds:
// The type-safe equivalent in TypeScript is
render (h) {
  if (this.$scopedSlots && this.$scopedSlots.default) {
    const defaultSlot = this.$scopedSlots.default({
      action: this.action
    })

    if (defaultSlot) {
      return defaultSlot // This is VNode[] but render demands a VNode
    }
  }

  return h('div') // for this example proof of type safety
}

The error statement:
No overload matches this call.
  The last overload gave the following error.
    Argument of type '{ props: { project: { type: ObjectConstructor; required: true; }; }; methods: { deleteProject(): void; }; render(h: CreateElement): VNode[] | VNode; }' is not assignable to parameter of type 'ComponentOptions<Vue, DefaultData<Vue>, DefaultMethods<Vue>, DefaultComputed, PropsDefinition<Record<string, any>>, Record<...>>'.
      The types returned by 'render(...)' are incompatible between these types.
        Type 'VNode[] | VNode' is not assignable to type 'VNode'.
          Type 'VNode[]' is missing the following properties from type 'VNode': isRootInsert, isComment

What is expected?
The render return type allows all valid return types. In this case, it's at least VNode[] | VNode.
-  render?(createElement: CreateElement, hack: RenderContext<Props>): VNode;
+  render?(createElement: CreateElement, hack: RenderContext<Props>): VNode | VNode[];

There might be others valid types?
What is actually happening?
return defaultSlot is expecting a VNode[] but the render typing only accepts a VNode.
The only way to work around this now is to cast to any when returning a value, which is a hack.

Source line for render function: https://github.com/vuejs/vue/blob/4821149b8bbd4650b1d9c9c3cfbb539ac1e24589/types/options.d.ts#L84

The error makes sense, you cannot return an array of vnodes, only an array of one vnode (https://github.com/vuejs/vue/issues/8056). This was to avoid breaking changes. Regarding types, however, it makes sense to make it strict.

That's very odd behavior. Thank you for the history, I can see how it ended up that way. Seems like the correct code is to return defaultSlot[0].


Investigate increasing developer productivity with pull requests environments

I would like to make life easier for existing and new Shynet contributors including the maintainer by implementing Uffizzi previews.
Disclaimer: I work on Uffizzi
Uffizzi is a Open Source full stack previews engine and is free for Shynet. This will provide newcomers with previews of their PRs in the cloud, allowing them iterate faster, without having to first set up a complete development environment themselves.
TODO:

[ ] Intial POC

I feel like setting up a GitHub codespace environments would work here? Anyway, sorry, I don't think this is a good fit for the project, and I've learned my lesson on scope creep/taking on additional maintenance burdens.


⚠ Bofrost.lu has degraded performance

In ae6c178, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Bofrost.lu performance has improved in 1317398 after 56 minutes.


undeclared identifier 'SIZE_MAX'

I am now getting this error building via clang on linux.
I assume it's since the 1.2.0 update since it did not occur previously.

yeah, I also got this error

I'll submit a bugfix in a few minutes.

It should compile now.

Thank you


[stable/elastic-stack] upgrade to ES 6.6.0

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

/ok-to-test
/assign


Generate uuid for pk ID

Hello I'm trying to use the default tag as bun:default:gen_random_uuid() in order to generate new uuid for the pk, but each time I run the query error with message "ERROR: null value in column "id" violates not-null constraint (SQLSTATE 23502)" is shown. Do you have any idea what am I missing here ?

default:gen_random_uuid() makes CreateTable query to set a default expression for the column. I guess you've created the table manually and the column does not have the DEFAULT gen_random_uuid() expression.

Right, this makes a lot of sense, thank you.


Code Whitespace

Whitespace can be helpful for reading complex code. Please add some.

We have fairly good adoption of whitespace now.
@philknows Can you add this to the contributing doc or wherever it makes sense for long term persistance?


Rename ServiceCollectionExtensions to AntiforgeryServiceCollectionExtensions

Rename ServiceCollectionExtensions to AntiforgeryServiceCollectionExtensions so that the type name is not ambiguous across other assemblies.

@danroth27 FYI

54d9860eeae05baf39006b499c9c7af3f1350814


NTR: cortical rotation

Please provide as much information as you can:

Suggested term label:
cortical rotation

Definition (free text)
A process by which the cortical layer of an oocyte/egg's cytoplasm rotates with respect to the deeper layers of an oocyte/eggs cytoplasm. In amphibian eggs, the cortical layer rotates c. 30o [degrees] with respect to the heavier, deeper yolk so that the cortex moves towards the animal pole on the dorsal side.
(or something like this)

Reference, in format PMID:#######
PMID:37983969
Azbazdar and De Robertis 2024 The early dorsal signal in vertebrate embryos requires endolysosomal membrane trafficking. Bioessays 2024 Jan 01;461:e2300179. doi: 10.1002/bies.202300179.
Also:
PMID: 1879356
PMID: 332452
PMID: 24798204
PMID: 9037034
PMID: 3289985
PMID: 332453

Gene product name and ID to be annotated to this term

ctnnb1.L Xenopus Gene ID:399274(http://www.ncbi.nlm.nih.gov/entrez/query.fcgi?db=gene&cmd=Retrieve&dopt=full_report&list_uids=399274). (Xenbase:XB-GENE-6254184)

Parent term(s)
GO: BP
GO_0030866: cortical actin cytoskeleton organization

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information

Add fish PMID:22949618 and worm PMID:25077289 references. IS_A GO:0048869 cellular developmental process.

[Term]
id: GO:0160174
name: cortical rotation
namespace: biological_process
def: "A cellular developmental process by which the outer layer of the cell (the cortex) rotates relative to the inner cytoplasm." [PMID:22949618, PMID:25077289, PMID:37983969]
is_a: GO:0048869 ! cellular developmental process
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/27366" xsd:anyURI
created_by: rynl
creation_date: 2024-03-22T21:44:06Z


[SMALLFIX] reset mEOFSent properly

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12971/
Test PASSed.


Design naming transformation and namespaces

Tables in a destination should not conflict in advanced scenarios. Think out possible collisions with multi sites sync to the same destination.

Some enhancement has been applied to how the name is mapped to destination table.
Also, an explicit alias for the table is added as optional spec parameter.
So far the naming is OK. Yet, I expect some potential changes based on users feedback.


Update sensor_database.csv

Added Panasonic Lumix DMC-FZ300 Small Sensor Superzoom camera: Panasonic,Panasonic Lumix DMC-FZ300,6.17

From another source, there is i little difference: width 6.16. Closing this and starting new pull request

From another source, there is i little difference: width 6.16. Closing this and starting new pull request


Paginator transformer problem

Hi,
i would like to transform objects returned in a paginator :
i have a thread model and a participant model and each thread has only two participant including the one asking the data.
So i would like to return the thread objects with a property 'participant' as a single User object instead of an array of the two users.
i have this code to get the data :
$threads = Thread::forUser($currentUserId)
        ->with('participants.user')
        ->with(['participants' => function ($query) {
            $query->where('user_id', '!=', $this->user->id);
        }])
        ->latest('updated_at')
        ->Paginate(2);

it selects the threads with only the other participant.
My problem is the formatted threads contains an array with the single user instead of the user only…
How can i use a transformer to do this please ?
Is it also possible to customize how the paginator is formatted ?

some clarification :
Thread model has a has_many relationship 'participants'
Participant model has a user property

@mushu8 Some of your question is a bit reliant on knowing your projects domain/context, and we may not be able to fully help there. But I would recommend checking out Fractal's page on pagination which includes a section on how to use it with Laravel's Paginator

Closing due to issue age


JavaScript

#JavaScript
##User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
##Acceptance Criteria
GIVEN Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned along with a suggestion on what I should study first

I got help from the tutor to push and pull the request to main.


Fix workflows with breaking changes and Set/modify otel propagators in the build scripts

Description:
This PR updates the scripts and adjust the changes related to the breaking changes after sub module update

Users (of tracer layers that have implemented xray-lambda) that send traces to X-Ray, especially if they enable Active Tracing, should override this value to OTEL_PROPAGATORS=tracecontext,baggage,xray-lambda to ensure proper span relationships within X-Ray.

set otel propagators in the scriptsref here
expands nodejs runtimes for collector layer in the tf files to be inline with supported lambda runtimes
Update go sample build script to add go mod tidy to fix the inconsistent go mod file upstream.
Logging exporter is deprecated, use debug exporter in the configuration.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This PR seems to be doing more than it says and it's not clear why.

I have updated the description related to the changes.


make some text localizable, mainly in dashboard, explore ...

make some text localizable, mainly in dashboard, explore, datasource editor, webpage title.

Codecov Report

Merging #5685 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #5685      +/-   ##
=========================================
+ Coverage   63.49%   63.5%   +<.01%
=========================================
  Files         360     360
  Lines       22914   22920       +6
  Branches     2551    2551
=========================================
+ Hits        14550   14556       +6
  Misses       8349    8349
  Partials       15      15

Impacted Files
Coverage Δ

...src/explore/components/controls/VizTypeControl.jsx
89.58% <ø> (ø)
:arrow_up:

...t/assets/src/dashboard/components/AddSliceCard.jsx
71.42% <ø> (ø)
:arrow_up:

...perset/assets/src/explore/components/SaveModal.jsx
94.04% <ø> (ø)
:arrow_up:

...hboard/components/gridComponents/new/NewHeader.jsx
100% <100%> (ø)
:arrow_up:

...hboard/components/gridComponents/new/NewColumn.jsx
100% <100%> (ø)
:arrow_up:

...uperset/assets/src/datasource/DatasourceEditor.jsx
72.92% <100%> (ø)
:arrow_up:

...dashboard/components/gridComponents/new/NewRow.jsx
100% <100%> (ø)
:arrow_up:

superset/views/core.py
73.93% <100%> (ø)
:arrow_up:

...ashboard/components/gridComponents/new/NewTabs.jsx
100% <100%> (ø)
:arrow_up:

...board/components/gridComponents/new/NewDivider.jsx
100% <100%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


also install pkginfo

A dependency change for dev scripts last night was not accounted for. This quick fix unblocks the pipeline, but a longer term fix should be considered: https://github.com/Azure/azure-sdk-for-python/issues/9837

/azp run python - smoke-tests

Passing smoke test run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=259771&view=results


Terminal: Underline is not continuous

Testing #158827
Version: 1.71.0-insider
Commit: bd6107301dc9a6200bd8d4f64a9b198438b3f1d6
Date: 2022-08-23T05:19:02.563Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0
Sandboxed: Yes

set setting terminal.integrated.gpuAcceleration: on
in a terminal type: echo -e "\x1b[4:2m"  # double underline followed by echo -e "\x1b[4:3m"  # curly underline and a new line

Especially under the w there's a lot of space so I would say it can't be antialiasing that interferes

Good find, the stroke seems oddly large on the w here.

Upstream: https://github.com/xtermjs/xterm.js/issues/4059

Looks better on the Mac now.


adds in memory kvstore

Not needed anymore? it turns out the old previously accepted PR that was merged is BACK. so we will just use that one? again, I'm sorry. not sure what happened with github. i'm wondering if there is some private -> public bug we hit.


Allow CLI options to be read from the configuration file

Allow CLI options to be read from the configuration file.

Allow CLI options to be read from the configuration file #29


sqlalchemy-oauth example not working with sqlalchemy 2.0

Describe the bug
When running the oauth example from the github repo it fails due to new type. The error message is the following:
sqlalchemy.exc.ArgumentError: Type annotation for "OAuthAccount.id" can't be correctly interpreted for Annotated Declarative Table form.  ORM annotations should normally make use of the ``Mapped[]`` generic type, or other ORM-compatible generic type, as a container for the actual type, which indicates the intent that the attribute is mapped. Class variables that are not intended to be mapped by the ORM should use ClassVar[].  To allow Annotated Declarative to disregard legacy annotations which don't use Mapped[] to pass, set "__allow_unmapped__ = True" on the class or a superclass this class. (Background on this error at: https://sqlalche.me/e/20/zlpr)

Referencing: https://github.com/fastapi-users/fastapi-users/blob/master/examples/sqlalchemy-oauth/app/db.py#L23
To Reproduce
Steps to reproduce the behavior:
Run the example in https://github.com/fastapi-users/fastapi-users/blob/master/examples/sqlalchemy-oauth/
Configuration

Python version : 3.8.10
FastAPI version : 0.94.0
FastAPI Users version : 10.4.0
SQLAlchemy: 2.0.3

Can you check that fastapi-users-db-sqalchemy dependency is at least on version 5.0.0?

@frankie567 it seems fastapi-users-db-sqlalchemy was not installed.
Before I used
fastapi[all]==0.94.0
fastapi-users[sqlalchemy,oauth]==10.4.0

But this did not install fastapi-users-db-sqlalchemy. After installing it, it is starting without any problems.


Has anyone been able to get 1554 release to work with Bluestack 5?

Preparation

[X] I read the Troubleshooting Guide
[X] My question isn't answered in any other issues (even the closed ones)

Your question
As title says, I got it to work on phone struggling on emulator. If anyone is able to help, thank you.

See #623 and #953


How to stop login from retrying once connection is lost

It's good idea to provide an option at the Connect preferences that disables retrying connections on connection lost.
Discussed in https://github.com/orgs/hawtio/discussions/3123

Originally posted by ilxyz December  7, 2023
Hello,
I noticed once a session expires and you are already logged into jolokia/remote jmx inside hawtio, a popup appears with the message "Connection lost. Retrying...", which keeps attempting login to hawtio with the jolokia/remote jmx credentials. This is generating a login error message per minute, which I am trying to stop from happening.
Is there a way to NOT have hawtio keep retrying the connection every minute and completely kill the session (in all windows)? Or a way to set a default refresh rate across the board in hawtio?

related:

#635
hawtio/hawtio#3178

With improvements from #906 I'll simply disable the toast. Failed connection will be indicated by red "disconnected" icon in the top bar.

PR: #936
I will continue the fix for credential mismatch here: hawtio/hawtio#3178


[Question] How much of my dev dependency chain can ruff remove?

Hi @charliermarsh! Thanks for the work you're doing with ruff.
I'm curious about where ruff fits into the formatting, linting, and type checking ecosystem. I have to install quite a few packages to handle formatting, linting, type checking, et cetera (see here) for python projects and would really like to trim that list down. Is this something ruff can help with? If so, what areas would it cover/ packages would ruff replace?
Thanks so much!

This is a great question! Thank you for asking it. I've used identical setups in the past and so am very familiar with the workflow you're describing. (Apologies in advance for the long answer.)
Today, ruff can be used as a drop-in replacement for flake8, when used alongside Black, and without plugins. Your setup uses flake8-docstrings, so the rules you'd be losing would be those focused on docstring enforcement.
The plugin thing is important: ruff doesn't yet support any kind of plugin architecture, though I am interested in building-in top 2-3 flake8 plugins into ruff directly to enable wider adoption. (For example, we could just add the flake8-docstrings logic to ruff; then you could replace flake8 and flake8-dodcstrings.)
In the short-term roadmap, I'd like ruff to support all of the autofixing behaviors that you get from autoflake. At that point, you could replace flake8, flake8-docstrings, and autoflake with ruff. We're not there yet, but the work is clear to me and very tractable. (ruff currently supports autofixing for a small number of rules, but not yet those enforced by autoflake.)
In the medium-term roadmap, I'd like ruff to support auto-formatting too. You could then imagine using a single, unified tool to perform linting, autofixing, and autoformatting in lieu of flake8, flake8-docstrings, autoflake, black, and even isort. Combining a linter with an autoformatter makes for a very powerful combination, and something I'm very interested in. This work isn't scoped yet, and doesn't have a clear timeline, but it does fit with the broader vision behind ruff.
In the longer term, I have an even bigger vision that touches on some of the other tools in the toolchain... But I think that's enough for now :)

One follow-up: it's important to me that even if ruff grows in scope, it can be used and adopted incrementally. For example, even if ruff grows to support autoformatting, I want it to be just as useable as a standalone linter alongside Black.

👍 Stay tuned!


[XamariniOSV2] Implement support of provisioning profile for app extension

Required Information
Question, Bug, or Feature?
Type: Feature
Enter Task Name: XamariniOS
Issue Description
In the current implementation of the Xamarin iOS task, there is a limitation that users can provide only one provisioning profile. But to sign an application that contains an extension requires two provisioning profiles. One for applications and another for an extension.

Reopening this since there's still no workaround for this issue.

Reopening this since there's still no workaround for this issue.

Any updates on this?

Hi @alexmartinezm sorry for delay - we are working on more prioritized issues at the moment, but will get back to this once be able to.

Please reopen!

Please reopen

This is not fixed. I am aware, that other things need to be taken care of first. But as this is not fixed: Please reopen it!


Forms are not XML/HTML encoded

Describe the Bug

Forms that define xml tags in the label, description, etc are not xml/html encoded.
Steps to Reproduce

create a new form
add a text field
write in the description: Hello, <world>!
go to json view
copy json
paste into a Camunda Cloud user task
go to xml view
see that it breaks the xml

Expected Behavior

Values defined in the form editor should be xml/html encoded.
For example, Hello, <world>! should be encoded to Hello, <world>! (I think)
Environment

OS: macOS 11.4
Camunda Modeler Version: 4.8.1 (<IP_ADDRESS>69)
Execution Platform: Camunda Cloud (but probably also on Camunda Platform)

I can reproduce the issue.

Congratulations by the way. You surfaced a low-level XML serialization bug. We don't have many of these around, anymore :1st_place_medal:

Actually just a zeebe-bpmn-moddle meta-model bug :crying_cat_face:.

Closed via https://github.com/camunda/camunda-modeler/pull/2369


##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work.

Question / Bug
Enter Task Name: VSTest
Environment

Server - Azure Pipelines or TFS on-premises?
https://dev.azure.com/cloudbilling/CloudBilling/_build/results?buildId=18617&view=logs&j=bf94e6da-eeef-50d1-100b-0ed7f4d964f2&t=2c85ca81-ba16-5100-d932-92222da60a57

Agent - Hosted or Private:

Agent name: 'Azure Pipelines 5'
Agent machine name: 'fv-az78-882'
Current agent version: '2.193.1'
Operating System
Microsoft Windows Server 2019
10.0.17763
Datacenter
Virtual Environment
Environment: windows-2019
Version: 20211018.0

Issue Description
Since today my VSTest task is printing this warning:

##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work.

There's no code change on my end, nor have I changed the build pipeline. I have cross-verfied against a prior commit for which this warning was not printed: running the pipeline against that commit now yields that warning.
I have no idea what this warning means. I know I do want code coverage.

We have seen the same issue on our pipelines. For what its worth our build agent is using VS2019 and the pipeline is using the "Visual Studio test platform installer" task which was pulling the latest (v17) test tools that are for the upcoming release of VS2022

Yep we're seeing this also, I have exactly the same setup as @andyfisher100

Just for clarification. We set the Visual Studio test platform installer task to use v16.11.0 of the test tools and the issue went away

Downgrading 17 to 16 resolved this problem for me as well. For reference the relevant build step:
    - task: VisualStudioTestPlatformInstaller@1
      inputs:
          packageFeedSelector: 'nugetOrg'
          versionSelector: 'specificVersion'
          testPlatformVersion: '16.11.0'

Does this mean that the new version of VS and DevOps will no longer support code coverage with test runs?

Does this mean that the new version of VS and DevOps will no longer support code coverage with test runs?

I'd assume it all works fine if the agent is running VS2022 and the latest test tools.

@tomaustin700 Nope. Still the same error when running VS2022 and the latest test tools ...

On VS 2022 and TestPlatform 17.0 Intellitrace is no longer needed to generate code coverage so maybe this warning should be removed/ignored. Could you please confirm that code coverage is really not generated? I saw 1 pipeline where code coverage was generated and this warning was in logs.

I also encountered this problem today.
We wanted to migrate from VS2019 vstest.console.exe because it is still compiled against .NET 4.5.1 and still use TLS 1.0 when running tests, but now that we use v17 of vstest.console.exe we now receive this warning.
Perhaps we could suggest a PR to remove this warning? Do you know if there is a way to get rid of this warning when invoking vstest.console.exe?

Ok I just found the resource containing this warning. Maybe it's just this task which should be tweaked.

And the function which emits a warning if the dll Microsoft.IntelliTrace.ProfilerProxy.dll is not found.
Because @jakubch1 said that the IntelliTrace is not expected to be found when using v17, how this function could be updated to handle this case?
There is also a reference to this resource in helpers.ts.

Hello!
I've faced up with the same issue in "Visual Studio test platform installer" task.
Our smore tests are not worked correctly, when I select "Latest Stable" in Version field of this task, but when I select "Specific version" and "Test Platform Version" 16.11.0 - all works fine.
May you fix it in the last stable version

Ran into this issue after updating VisualStudioTestPlatformInstaller has any progress been made regarding this?

Running into the same issue. Any progress on this yet?

@andy-mishechkin @kaylumah @ashish-tamhane is code coverage really missing or you are just getting this warning?

@andy-mishechkin @kaylumah @ashish-tamhane is code coverage really missing or you are just getting this warning?

I am just getting this warning. Code Coverage is being generated perfectly.

@phanikmmsft could you please simply remove this warning from your code? Code coverage is using now CLR Instrumentation Engine and Intellitrace files are not needed anymore so this warning is useless.

Still a problem

Still a problem in 17.4.1...is anybody going to fix this?

@jakubch1 could you please update the assignee for this issue? it seems @phanikmmsft does not work on this project anymore?

Still occurring with 17.5.0-preview

@abhadauria1 any update on removing this warning?

ok thanks for the help.

This issue is still occurring. Bumping thread.

We are experiencing this superfluous warning, as well.

I'm seeing this warning three times in my output. I have a single test task with a single test. It warns three times. This issue has been open for two years. Is it really possible that there's no workaround and no fix for the latest version of the standard testing task?

I have the same issue, when that will be fixed???

For what it's worth, my workaround is to roll back to 16.11.0, which is the last release of the VS2019 test runner. It seems every version of the VS2022 test runner has the warnings.
Include the following in the pipeline (relevant items only; not a complete pipeline definition).
- task: VisualStudioTestPlatformInstaller@1
  inputs:
    packageFeedSelector: 'nugetOrg'
    versionSelector: 'specificVersion'
    testPlatformVersion: '16.11.0'

- task: VSTest@2
  inputs:
    vsTestVersion: 'toolsInstaller'

This issue is still occurring 22-months after the initial report
2023-08-14T17:22:53.1189389Z Task         : Visual Studio Test 2023-08-14T17:22:53.1189480Z Description  : Run unit and functional tests (Selenium, Appium, Coded UI test, etc.) using the Visual Studio Test (VsTest) runner. Test frameworks that have a Visual Studio test adapter such as MsTest, xUnit, NUnit, Chutzpah (for JavaScript tests using QUnit, Mocha and Jasmine), etc. can be run. Tests can be distributed on multiple agents using this task (version 2). 2023-08-14T17:22:53.1189930Z Version      : 2.224.0 2023-08-14T17:22:53.1189983Z Author       : Microsoft Corporation 2023-08-14T17:22:53.1190063Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/test/vstest 2023-08-14T17:22:53.1190330Z ============================================================================== 2023-08-14T17:22:56.0000189Z Running tests using vstest.console.exe runner. 2023-08-14T17:22:56.0000747Z ====================================================== 2023-08-14T17:22:56.0001762Z Test selector : Test assemblies 2023-08-14T17:22:56.0002114Z Test filter criteria : TestCategory=Adhoc 2023-08-14T17:22:56.0002435Z Search folder : D:\a\1\s 2023-08-14T17:22:56.2922294Z Run in parallel : false 2023-08-14T17:22:56.2922609Z Run in isolation : false 2023-08-14T17:22:56.2922930Z Path to custom adapters : undefined 2023-08-14T17:22:56.2924324Z Other console options : /logger:trx;LogFileName=rxwelltests-tst-edge-adhoc.trx /framework:".NETCoreApp,Version=v7.0" 2023-08-14T17:22:56.2924736Z Code coverage enabled : false 2023-08-14T17:22:56.2925299Z Diagnostics enabled : false 2023-08-14T17:22:56.2929939Z Run the tests locally using vstest.console.exe 2023-08-14T17:22:56.2930121Z ======================================================== 2023-08-14T17:22:56.2930747Z Source filter: D:\a\1\s\QAAutomation\UITests\RxWellTests\bin\Release\net7.0\*tests.dll 2023-08-14T17:22:56.6963233Z ##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work. 2023-08-14T17:22:56.7306337Z ##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work. 2023-08-14T17:22:56.8691061Z ##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work.

For what it's worth, my workaround is to roll back to 16.11.0, which is the last release of the VS2019 test runner. It seems every version of the VS2022 test runner has the warnings.
Include the following in the pipeline (relevant items only; not a complete pipeline definition).
- task: VisualStudioTestPlatformInstaller@1
  inputs:
    packageFeedSelector: 'nugetOrg'
    versionSelector: 'specificVersion'
    testPlatformVersion: '16.11.0'

- task: VSTest@2
  inputs:
    vsTestVersion: 'toolsInstaller'

While the comments above offer a valid workaround, one has to ask how is it that there have been so many newer versions published (latest stable version is 17.6.3) yet this issue has never been addressed?

I just tried the latest stable version 17.6.3, the warning is still there, now our pipeline raised the warning more than 300 times, other problems are hidden in the warning flood...

Same issue here. All releases after 16.11 will result i a flood of warnings. Tryed out new xunit 2.5 and newest vstest runner.

@jakubch1 could you please update the assignee for this issue? it seems @abhadauria1 does not work on this project anymore?

With VsTest task version 2.235.0, this issue should be resolved while consuming the latest test platform versions ( > 17.0.0).

The changes were merged yesterday only. It would be gradually rolled out to all the Azure DevOps organization within next 15-20 days. I'll update the thread if things go south.

The latest VsTest Task version (>=2.235.0) is working as expected and should be available to consume where logging of unnecessary warnings isn't happening. Hence, closing the issue. Please feel free to reopen the issue if it persists.

Hi @adityashahms, was this change deployed to vstest@3 as well? I'm currently on
Starting: VSTest
==============================================================================
Task         : Visual Studio Test
Description  : Run unit and functional tests (Selenium, Appium, Coded UI test, etc.) using the Visual Studio Test (VsTest) runner. Test frameworks that have a Visual Studio test adapter such as MsTest, xUnit, NUnit, Chutzpah (for JavaScript tests using QUnit, Mocha and Jasmine), etc. can be run. Tests can be distributed on multiple agents using this task (version 2).
Version      : 3.231.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/test/vstest
==============================================================================

and still getting this error
##[warning]Test Impact (Run only Impacted tests) and Code Coverage data collector will not work.

Not sure if I should downgrade to vstest@2 as changes are deployed there or what's going on with those task versions.

@quasarea , please downgrade to VsTest@2, since VsTest@3 isn't getting active updates as much as VsTest@2.

FYI - At the top of the page I see that this issue is marked as closed, so unsure if anyone is really paying attention. Might be a good idea to open a new case for the VsTest@3.


Create frontend webpage

Create the HTML, CSS and JS(?) needed for the website frontend. This should consist of multiple pages: A home menu, a page allowing a user to supply an image file, message, and key to get an encoded image, and a page allowing the user to supply an encoded image and a key to get the original message. Only take this if you are comfortable with using HTML, CSS, and JS(?).

Please place these files in a folder named assets in the src folder.


Use GUN for Profiles Instead - tests passing on my machine

What kind of change does this PR introduce?:
Uses GUN instead for profiles. :) Text search later.
Did you add tests for your changes?:
Test ran successfully on my machine.
Summary:
Uses GUN instead for profiles.
Does this PR introduce a breaking change?:
Possibly on the parts IPFS got disabled.
Other:
We need to chat later about how to make sure it is properly running on identi.fi and in browser so demo works via GUN.

I'm getting "ERROR: Radisk needs opt.store interface with {get: fn, put: fn, list: fn}!" from tests


Rename "Discourse" link in top navigation to "Forum" or "User Forum"

Proposed documentation enhancement
When hovering over the Discourse icon in our navigation, the alt / link text that pops up is "Discourse". Now, if one doesn't know that this is meant to be a forum, one will only find out after clicking on it (if one dares to click). I'd like to propose renaming the link to "Forum" or "User Forum" for clarity.

👍 for "Forum".

I'm fine with "Forum" or "User Forum". Slight preference for "Forum" because we also want Discourse to be a place for contributors to ask questions.


Make it work

This PR adds the boilerplate to make this project work as a proper tmux plugin.

Cool so I got the whole make it a bash executable file with the shebang at the top. That said couple things I'm curious about...
Does it need to be a .sh extension or will the shebang be fine?

Does it need to be a .sh extension or will the shebang be fine?

It's fine with the shebang. Also we need the file to have *.tmux extension so it works as a tmux plugin compatible with tpm.

it forces the user to use the Meta key instead of something user prescribed? Is there a way to set the optional meta key to a user prescribed variable?

A constraint to this is terminal. From what I know it only recognizes these character and char-combos:

ascii characters
ctrl + ascii characters
meta + ascii char
4.. meta + ctrl + ascii char

First 2 groups above are already used by a lot of terminal applications (eg vim). That leaves you with meta char groups, and that is what this plugin is doing. Meta is usually alt character on the keyboard.
I think the only way to change this is if the user re-maps alt to some other key on the keyboard eg F2. Leaving alt as-is is probably the best though.
tl;dr; meta/alt key is a sensible way to go.


Rethink _d

I'm wondering if we really need to track a Date object in _d.

Loads of questions are raised from folks logging a moment instance to the console, seeing _d and not understanding how to interpret it.  While ._ convention is common for internal/private fields, it's still not something the common user understands.  Also in the console, the string it displays is implementation dependent.  Most browsers show the local time, but others (like Firefox) show the UTC time.  So a lot of misunderstandings are when the string representation of _d doesn't align with the date emitted by .format().
Currently, the value of _d.valueOf() is the same as moment.valueOf() only when the moment is either local or UTC.  As soon as there is an offset set in _offset, then the _d value no longer represents the same instant in time, but rather the instant shifted by the offset.  Our valueOf() function (and many others) have to take _offset into account and shouldn't.  This is both an efficiency problem, and an easy source of errors.  It also isn't understood when users look at _d in isolation (per my first point).
It would make a lot more sense (IMHO) to keep just a timestamp as a number, which is always the UTC-based timestamp of the specific instant in time that the moment is about.  Perhaps in _ts (for timestamp). The _offset would be applied to _ts for the functions that need it.
We may find we can actually get rid of several functions that use the Date object.  For example, I have recently re-implemented functions for date-part to timestamp and vice-versa in pure JavaScript, and they perform faster than native code.  This could be pulled into moment, and other similar functions can probably be written that manipulate the timestamp directly rather than calling functions like setMonth() on the Date object.  This would also have the benefit of not being browser-dependent on DST transition adjustment behavior.
I think this will also make it easier for some of the moment-timezone integration.

I wouldn't base my decision on folks debugging moment and not understanding how it works.
About the performance increase -- you can convert some methods to use timestamp but not all, unless you store a date object yourself and do all the nasty things Dates do when you set month, day, hour etc. I actually started a pure javascript Date implementation that was more predictable, but I  haven't made progress on it for some time.
All of the set methods will be super slow if you just do timestamp, because you have to explode to fields, then manipulate them and then convert back to timestamp. If you store fields directly it uses more memory.
https://github.com/ichernev/utc_date if you want to work on that, be my guest :) At the least we need to implement (almost) all Date manipulation methods (mostly getters, setters, convert to unix, no parsing), and a way to plug DST source (moment-timezone IANA timezone DB or Date (for local zones only)). That is actually mostly done in the current code.
Handling DST on creation is not done yet -- it has to decide what to do if its a hole (no possible UTC), if its an overlap (2 possible UTC values for one local one), possibly via configuration / callbacks (configurable defaults).
I still haven't decided how to handle DST weird cases in general -- we can have a config on what to do, but also callbacks that will notify the user of these things happening (some might use that). Its just complicated :)

What is the difference between _i and _d?  I have seen many situations where these values don't match under clone() in 2.5.2 and was digging through _d and discovered it called the underlying new Date() constructor, and then found this thread.  I am wondering if there is some documentation on these private fields somewhere...?

@jzabroski - I wholeheartedly agree that they are underdocumented.  They shouldn't be in the public API docs, but should be on a developer/contributor doc page somewhere.
The i in _i stands for input.  Whenever you create a moment by passing it something, that gets copied into _i.  I'm quite sure why this is necessary though.  _i might contain a string, a number, or an array, or it might not exist at all if you just use moment().
_d is the Date object underlying the moment.  It is sometimes read directly, and sometimes combined with _offset - depending on the circumstances.

A good minifier is supposed to eliminate comments from moment.js proper.  There is no reason for a separate "doc page".

http://stackoverflow.com/questions/28126529/momentjs-internal-object-what-is-d-vs-i

SIgned up to CodeTriage.com to make the best of some unexpected free time and it sends me this as my first go. :joy: As there's no activity since 2015 could this issue be closed?

See https://momentjs.com/docs/#/-project-status/


WIP: Updating UV to 2.0.1

Fixes #9

N.B., this PR is not working (because I wasn't able to complete the grunt build step to build the UV files)

Closing in favor of #11


✏ Rename Toc -> TOC

This is a clean-up PR, before we start doing surgery on TOC handling.

Thanks Franklin! I'm taking that as merge approval :D

@agoose77 bug I found:
myst init --write-toc

no longer works.
Will make a patch.


The tilde additional feature settings are treated as one

Trying to separate the handling (commented out code) causes null reference.

Fixed.


New VSAN page, TOC Update, format updates

New VSAN Run Command Page
TOC Rearrangement and syntax correction
Storage Policy checklist items added and format corrections.

@jjaygbay1

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE: New articles can't be merged via the public repo. If you'd like to accept this new article, the commits will have to be moved into the private repo azure-docs-pr . We can help with this process; just let us know.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


"select several answers" on page /ensemble/ensemble_quiz_m6_01.html

Question 1 et 2 : doit-on ajouter Select several answers ou remplacer par une question à choix unique ?

We are going to open a meta-issue where we will point out all the places where we want to add "multiple/single choice" info.
Basically, we want to add this note everywhere to be consisent.

@glemaitre I find the phrasing "select several answers" problematic. I think it should be "select one or more answers" for multiple choice questions.

@ogrisel , I agree, "select several answers" is ambiguous. We may also used "Several answers possible".

+1 for "Several answers possible".

"Select a single answer"
"Several answers possible"

Closing this one in favour of #546.


[Explicit Module Builds] Handle product dependencies by expanding them into constituent targets.

Targets that depend on products were not previously handled in Explicit Module Builds.
This patch adds support for product dependencies by expanding such dependencies into dependencies on targets that make up a given product, for the sake of propagating target build planning artifacts (binary module path and dependency graph) to depending targets' build planning in the integrated driver.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


First cut of laying out options

I've grabbed the options as suggested in #239 by @mikeday, and put them into this simple configuration.  While the flags work pretty much as described in that issue, the output of --help leaves a lot to be desired and i'm not sure that structopt actually can accommodate everything we want.
Things i'd still like to fix:

Make it clear that --format and --format-file are mutually exclusive, instead of exploding if a user tries to specify both.
Sensibly group the format- and output-related options in --help.  Currently they're just sorted alphabetically.

I don't want to waste too much time on this, i'll revisit it once this package has managed to integrate with fathom a little.

Looks good to me.
One note: clap upon which structopt is built is (hopefully) in the latter stages of merging in the structopt functionality. It might be worth considering building the CLI on top of the clap 3.0 beta with the assumption that by the time fathom is ready so will clap 3.

Ah, that's a really helpful remark.  Thanks.  For now i think i'll just keep this as it's "good enough" for testing, but i'll avoid investing too much time into this approach then.


flashplayer: <IP_ADDRESS>4 -> <IP_ADDRESS>0 [Critical security fixes][19.09]

Motivation for this change
#79915 for release-19.09.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Would like to see this merged too!

I would like to take this change to point to #59276 again - as long as flashplayer is a component of a system it has been one of the largest sources of broken builds on a release branch I've had to deal with.


Fixed - Form.Field - Cannot set ID when contains child elements or content prop

Fixes https://github.com/Semantic-Org/Semantic-UI-React/issues/3900
Fixed issue where ID cannot be set when contains child elements or content props for FormField

Codecov Report

Merging #3901 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3901      +/-   ##
==========================================
+ Coverage   99.84%   99.84%   +<.01%
==========================================
  Files         178      178
  Lines        3150     3151       +1
==========================================
+ Hits         3145     3146       +1
  Misses          5        5

Impacted Files
Coverage Δ

src/collections/Form/FormField.js
100% <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 2dbab6b...bcf4ca5. Read the comment docs.

@levithomason
Hi Levi, could you please address this PR?

Codecov Report

Merging #3901 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3901      +/-   ##
==========================================
- Coverage   99.84%   99.84%   -0.01%
==========================================
  Files         183      178       -5
  Lines        3157     3151       -6
==========================================
- Hits         3152     3146       -6
  Misses          5        5

Impacted Files
Coverage Δ

src/collections/Form/FormField.js
100.00% <100.00%> (ø)

src/modules/Dropdown/Dropdown.js
100.00% <0.00%> (ø)

src/modules/Transition/Transition.js
100.00% <0.00%> (ø)

src/behaviors/Visibility/Visibility.js
100.00% <0.00%> (ø)

src/modules/Transition/TransitionGroup.js
100.00% <0.00%> (ø)

...ddons/TransitionablePortal/TransitionablePortal.js
100.00% <0.00%> (ø)

src/modules/Dropdown/utils/getSelectedIndex.js

src/modules/Dropdown/utils/getMenuOptions.js

src/modules/Transition/utils/childMapping.js

src/modules/Transition/utils/computeStatuses.js

... and 3 more

Continue to review full report at Codecov.

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

I was a bit confused with a use case for this as according to spec input elements should have id (https://www.w3.org/TR/WCAG20-TECHS/H44.html) to be linked with labels. However, at the same time it's frustrating that id is passed nowhere.
Thanks for addressing this 👍


Deserialization of TweetSearchResponse throws an NPE in an environment where gson conflicts with other deps

NPE when deserializing TweetSearchResponse in a web app environment with many dependencies (not in a simple Java SE environment with few dependencies)
Expected behavior
Deserializing a json string into a TweetSearchResponse object
Actual behavior
The deserialization throws an NPE
Steps to reproduce the behavior
Create a String example with the text in tyhe attached file, which is the serialization of a TweetSearchResponse
tweetRecentSearch.txt
Then do:
TweetSearchResponse tweetSearchResponse = TweetSearchResponse.fromJson(example);
In the context of a Java SE app with few dependencies, no exception is thrown.
The pom.xml of this simple app:
https://github.com/seinecle/nocodefunctions-as-api/blob/master/pom.xml
In the context of a web app with possibly conflicting dependencies (Google librairies relying on GSON as well), an NPE is thrown. The pom.xml of this bigger app:
https://github.com/seinecle/nocodefunctions/blob/master/pom.xml
Proposed solution
Do not rely on the TweetSearchResponse.fromJson() method and use the Json Binding API instead:
dependencies: https://javaee.github.io/jsonb-spec/users-guide.html
code:
Jsonb jsonb = JsonbBuilder.create();
TweetSearchResponse tweetSearchResponse = jsonb.fromJson(example, TweetSearchResponse.class);

Broader suggestion
It could be interesting to drop the dependency on gson and rely on a more conventional json-b API and implementation:
https://javaee.github.io/jsonb-spec/index.html

Hi,
I understand the problem but didn't manage to re-create it.
Does this link help?
https://www.baeldung.com/maven-version-collision#:~:text=Excluding a Transitive Dependency From an Artifact
something like
```

com.twitter
twitter-api-java-sdk
1.2.0

com.google.code.gson
gson

Very clear thank you.


Optimize the /cart page in Merch to better handle large Carts with many Customizations:

Add missing indexes on tables SpreeProductCustomizedOptions and AdHocOptionValueLineItems on their foreign keys

https://www.pivotaltracker.com/story/show/179425908

Merging.


Researching Cosmos

"Buying atoms"

@faddat i'll fill out the issue for you. :)

Add the the following issue description and tasks for @faddat
Description
As a result of deciding to go with Cosmos for our recovery app, @faddat will need to research some unknown unknowns.
Tasks

[ ] Find out how transactions work.
[ ] Find out how to acquire Atoms as we are in New York state and it's difficult to purchase them due to the bit license.

@faddat were you able to purchase any Atoms?

From Slack for reference.
Guy Lepage 8:00 AM
@faddat good morning sir.. how did that purchase of $atoms go?
seems to be quite difficult to  purchase $atoms.
that’s a bit of an issue to discuss when talking about building on cosmos. can you put a pin on this?

Atoms and NYS  / bitlicense
Impossible at this point.
Buying Atoms
Buying $atom went like this:

https://openmarketcap.com/cryptocurrency/atom-usd
vs
https://coinmarketcap.com/currencies/cosmos/#markets

Due to OMC exchanges not operating in NYS, use hotbit from CMC, a KYC-free (ish) exchange (supposedly) based in mainland china.

On Hotbit, exchange BTC for $atom at market rate

Once done on Hotbit, coins can (maybe) be stored on an Android cosmostation wallet or a ledger nano S.  Our coins aren't stored just yet.


Fix: variable and function naming and duplication

variable x2 should be x1
function f(x) should be `g(x)
expression zy = zx + 1 is duplicated

Thanks for your pull request, but this repo does not accept outside contributions unless they are under contract. If you think this is an error, please contact @nedbat.


adding a favicon

Greetings, I want to add a favicon to your website

please assign this issue to me I will be solving this issue under GSSoC'23. I will add the favicon a mentioned.

I want to work on this issue under GSSOC '23 . I have the necessary skills required to get the issue fixed, so please approve my request.


Implement FederationProvider for Spark Connect

REMEMBER, BE SMART!
S: Specific
M: Measurable
A: Achievable
R: Relevant
T: Time-Bound
Goal-State/What/Result
Implement the FederationProvider for Spark Connect-based connectors (spark and databricks).
Why/Purpose
This fully implements support for federated push-down of all DataConnectors, as a follow on from #1318
By When
A target date for done-done completion of the entire enhancement.
Issue/Spec written and reviewed:
Done-Done:
Done-Done

[ ] First Principles Driven
[ ] The Algorithm
[ ] PM/Design Review
[ ] UX Review
[ ] Threat Model / Security Review
[ ] Tests
[ ] Documentation
[ ] Samples/Quickstarts/Tutorials
[ ] Release Notes

The Algorithm

[ ] Every requirement questioned?
[ ] Delete (Scope) any part you can.
[ ] Simplify.
[ ] Break down into smaller iterations/milestones.
[ ] Opportunities for automation.

Spec
Basic specification of the enhancement.
How/MAP
A clear and concise plan of how this should be implemented.
Implementation (issues)

QA

@phillipleblanc to sync with @digadeesh on the priority

@phillipleblanc to work with @digadeesh to assign to someone else. They should rework this as an enhancement ticket.

This is done


Android Basics: polished-user-experience

URL of codelab
In which task and step of the codelab can this issue be found?
Describe the problem
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.
Data binding is not explained but being used as if user knows already. Go through Codelab no. 3 section 3.

This(polished user experience) codelab builds on the previous codelab(Tip Calculator) in which view binding is explained. That must be why it's assumed known in this codelab since it's a requirement to have Tip Calculator app already, ie.
What you need

    A computer with Android Studio installed.
    Code for the Tip Time app from completing the previous codelabs


Error: "boxplot" is not a registered controller when using plugin with PrimeVue Charts

"@sgratzl/chartjs-chart-boxplot": "^4.2.7"
I've read other issues with this error, but still cannot understand how to properly init BoxPlot plugin when Chart.register() is not exposed.
Their canonical way to attach plugins is:
  <Chart
    :data="boxplotData"
    :options="{ responsive: true }"
    :plugins="[BoxPlotChart]"
  />

And it seem to work with other plugins, but not with BoxPlot.
Full repro:
https://stackblitz.com/edit/mgfvrw-3xiqfx?file=src%2FApp.vue
(the page is empty as it fails at setup stage, please see console for errors)
I ended up using vue-chartjs: "^5.2.0" for charts only, so in general this plugin works fine within vue, but only with .register() method.
As it's not clear whether it's a bug or usage specifics, so I crossposted it their repo also:
https://github.com/orgs/primefaces/discussions/187#discussioncomment-7510393
Thanks heaps for the plugin! Very useful.

this chart js extension is not a plugin it but a handful of components that need to be registered if used in ESM module .
i..e
import { Chart, LinearScale, CategoryScale } from 'chart.js';
import { BoxPlotController, BoxAndWiskers } from '@sgratzl/chartjs-chart-boxplot';

// register controller in chart.js and ensure the defaults are set
Chart.register(BoxPlotController, BoxAndWiskers, LinearScale, CategoryScale);

Yes, apparently PrimeVue chartjs component does not expose Chart.register() call, so I'll raise it with them.


partial data not working for non-year calcs?

WMA internal wiki was added and has a partial month:

but the marker style for month makes it look like it has at least 90% data coverage for that time period:

ah ha! process.fill_missing was assigning the same date range (n_actual) to all viewIDs. just need a group_by() in there so that ranges are computed for each viewID separately.

Good find!

yep, that did it.
from:

to:


Optimize build matrix

Fixes #40.
GitHub workflows have some limitations on setting up "constants" or "global variables".
There are two solutions:

using a placeholder and then replacing it when needed with a sort of ternary operation:${{ matrix.browser == 'default' && env.DEFAULT_BROWSER || matrix.browser }}

It's quite verbose, not so readable and "default" will show up in the CI steps (not so nice).
See run: https://github.com/minkphp/webdriver-classic-driver/actions/runs/11645388813
using a previous step to set up defaults, as mentioned in the documentation.
On the whole this seems to work better and is less verbose. The extra step might be confusing and initially, it will not properly list all steps until that first one is executed.
See run: https://github.com/minkphp/webdriver-classic-driver/actions/runs/11645451262

@aik099 the change in tests has revealed a "bug" in the browser name test where the input is not matching the output (chromium -> chrome and edge -> MicrosoftEdge). Not sure how to solve this TBH. :(

You're getting a double build count (on pull_request and on push) because you're creating branches for your PRs in the minkphp/webdriver-classic-driver repo instead of your fork repo uuf6429/webdriver-classic-driver.
Here is my setup:
Do once:

clone the minkphp/webdriver-classic-driver repo (that is origin remote)
fork the minkphp/webdriver-classic-driver repo as aik099/webdriver-classic-driver
add aik099 remote that points to the fork

Do each time:

create a PR branch
make code changes & commits
push that branch into aik099 remote with --set-upstream option so they're linked now
create a pull request from just pushed branch on the aik099/webdriver-classic-driver

You can see for yourself by comparing the build in mine & your PRs:

mine: https://github.com/minkphp/webdriver-classic-driver/pull/39/checks - only pull_request part
yours: https://github.com/minkphp/webdriver-classic-driver/pull/38/checks - both push and pull_request part
.

You're getting a double build count (on pull_request and on push) because you're creating branches for your PRs in the minkphp/webdriver-classic-driver repo instead of your fork repo uuf6429/webdriver-classic-driver.

I was under the impression that a fork is needed for non-maintainers to contribute. As a maintainer, why should I have a fork? In that case I believe the workflow should be better adjusted.. Isn't it also more convenient for you to work on the main repo directly?

@aik099 the change in tests has revealed a "bug" in the browser name test where the input is not matching the output (chromium -> chrome and edge -> MicrosoftEdge). Not sure how to solve this TBH. :(

in the driver constructor just do $this->browserName = $browserName;
in other places (\Mink\WebdriverClassicDriver\WebdriverClassicDriver::getBrowserSpecificCapabilities and \Mink\WebdriverClassicDriver\WebdriverClassicDriver::initCapabilities execute the self::BROWSER_NAME_ALIAS_MAP[$this->browserName] ?? $this->browserName instead of $this->browserName

This way developer will get the same browser as he provided at driver construction time, but the mapping (to satisfy WebDriver) would be purely driver internal stuff.

Out of curiosity, I checked how PHPStan does it. It looks similar to something I saw in the GH docs, wouldn't this make more sense?
on:
  pull_request:
  push:
    branches:
      - "main"

Out of curiosity, I checked how PHPStan does it. It looks similar to something I saw in the GH docs, wouldn't this make more sense?
on:
  pull_request:
  push:
    branches:
      - "main"

Belive me. Having PR branches on the main repo is a bad idea. Even worse these PR branches are forked to the contributors repos as well.

You're getting a double build count (on pull_request and on push) because you're creating branches for your PRs in the minkphp/webdriver-classic-driver repo instead of your fork repo uuf6429/webdriver-classic-driver.

I was under the impression that a fork is needed for non-maintainers to contribute. As a maintainer, why should I have a fork? In that case I believe the workflow should be better adjusted.. Isn't it also more convenient for you to work on the main repo directly?

I use the same approach for all repos regardless of the fact, that have push access to them (means I'm a maintainer there) 😄 . As I saw @stof does the same in https://github.com/minkphp/webdriver-classic-driver/pull/3 .

Out of curiosity, I checked how PHPStan does it. It looks similar to something I saw in the GH docs, wouldn't this make more sense?
on:
  pull_request:
  push:
    branches:
      - "main"

I'm not saying no. Let's hear other opinions: @stof , @oleg-andreyev , @mvorisek .

Thank you for asking me. I am strongly for running CI on all pushes - I very often want to run CI on non-PR branch - like before I submit PR, for experiments, ...

Thank you for asking me. I am strongly for running CI on all pushes - I very often want to run CI on non-PR branch - like before I submit PR, for experiments, ...

That sounds like you want to run it manually, right? In that case it would make sense to also keep on: workflow_dispatch.

I simply fork them, enable CI for that repo and push. I prefer CI on every push instead having to trigger it manually.

I simply fork them, enable CI for that repo and push. I prefer CI on every push instead having to trigger it manually.

@mvorisek , in this scenario you'll also get double builds:

when pushing to the forked repo (build in a forked repo on the push event)
when commit gets added to a pull request (build in origin repo on the pull_request event)

The problem, that was supposed to be solved by restricting push builds on the main branch was double builds, when maintainers create PR branches directly in the repo itself and not on their fork (see https://github.com/minkphp/webdriver-classic-driver/pull/41#issuecomment-2453116465 ).
What do you recommend?

I am maintainer of atk4 project and we run such CI configuration for years - https://github.com/atk4/ui/blob/5.2.0/.github/workflows/test-unit.yml#L4-L7. So far the CI is great. We even run CI every 2 hours using cron, so we can easily notice dependencies issues.
I found the CI run two times is actually useful, as you can easily notice a) difference of branch vs. rebased branch on PR branch (that's how pull_request is tested), b) randomly failing tests.


Add missing nil checkers to check-response-mediatype-enum-value-remove

PR to add missing nil checkers to check-response-mediatype-enum-value-remove

Codecov Report

Merging #217 (bfcf2f6) into main (e696d71) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   76.28%   76.24%   -0.04%
==========================================
  Files         137      137
  Lines        7772     7776       +4
==========================================
  Hits         5929     5929
- Misses       1561     1563       +2
- Partials      282      284       +2

Flag
Coverage Δ

unittests
76.24% <0.00%> (-0.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ker/check-response-mediatype-enum-value-removed.go
52.77% <0.00%> (-6.60%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


binaryninja.Architecture.assemble pairity

To assemble an instruction in binaryninja API, you call Architecture.assemble.
To disassemble an instruction in binaryninja API, you call Architecture.disassemble.
...I'm kidding. You call Architecture.get_instruction_text. Can we get a disassemble alias for get_instruction_text, so it's a bit more intuitive for plugin developers?

The problem is that there is only one way to assemble and multiple ways to disassemble, do you just want the raw text of the disassembly or do you want the tokenized version. We have documentation now explaining the APIs so theres less confusion.


Uploads: Show modal to delete detached attachments after message edit.

Fixes: #25525

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@zulipbot add "product review"

@zulipbot add "buddy review"

@palashb01 @Joelute , Could you review it only for the approach for now, It's missing many things but if approach is confirmed than I will work further on it.

This looks broadly reasonable; needs some cleanup.

Let's change "attachments" -> "uploaded files" in the error message for consistency.

I reviewed the screenshots and responded to the questions above; I haven't tried to test the PR due to merge conflicts.

Thanks alya for confirmations, I'm working on tests of this PR, will push the new code soon.

Let's change "attachments" -> "uploaded files" in the error message for consistency.

We decided to go with One or more files could not be deleted. in the CZO discussion.

Closing in favor of the more current https://github.com/zulip/zulip/pull/29699 for this issue. Thanks for all the work on this @brijsiyag!