Move Travis to use the Checks API

We'll need to upgrade the build at some point to do this. This one is probably something you'll need to do @ashleygwilliams

indeed. do we know how urgent this is? i've been waiting for an announcement from travis but haven't seen anything (though i may have missed it because of travel)

I mean it looks like they said 2 weeks but here we are 🤷♂ . It was not very clear. Don't think it needs to be done today, but maybe by like the release after this one?
https://blog.travis-ci.com/2018-05-07-announcing-support-for-github-checks-api-on-travis-ci-com

i believe this has been done.


Added WB schools

This PR adds support for Western Brown schools in Ohio.

@drewsucksatlife Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Fails to install/start using Podman

Describe the bug
After running podman compose up -d on the yaml file the image is downloaded and the container starts to run; however, when viewing the logs, there is a series of critical errors and the application does not finish setting up, then exits, before looping the whole thing again, and again. The error (view snippet section) mentions a permission issue on an internal folder (inside the container, not the host machine), before saying the data directory needs permissions and to use a chown command. Even once I use the chown command and try again
I HAVE used sudo to change the ownership on the host system as suggest, didn't make any difference and it was already owned by 1000:1000
To Reproduce
Steps to reproduce the behavior:
Create a compose.yaml as described in the installation steps.
Run container.
Look at logs.

Expected behavior
I expect the container to install correctly and display a webpage for me.
Screenshots
If applicable, add screenshots to help explain your problem.
Spoolman Host (please complete the following information):

Installation type: podman
Installed version: latest as of 2024-11-24
OS and Distro: Fedora
Database type: sqlite

docker-compose.yml

services:
spoolman:
image: ghcr.io/donkie/spoolman:latest
container_name: spoolman
environment:
- TZ=America/Toronto # Optional, defaults to UTC
volumes:
# Mount the host machine's ./data directory into the container's /home/app/.local/share/spoolman directory
- type: bind
source: /home/SECRET/spoolman2/data
target: /home/app/.local/share/spoolman # Do NOT change this line
ports:
- 7912:8000
restart: unless-stopped
Output
podman start -a spoolman
[spoolman] | usermod: no changes
[spoolman] | User UID: 1000
[spoolman] | User GID: 1000
[spoolman] | Starting uvicorn...
Failed to setup disk-based cache due to permission error. Ensure the path /home/app/.local/share/spoolman/cache/hishel is writable. Using in-memory cache instead as fallback.
INFO:     Started server process [1]
INFO:     Waiting for application startup.
spoolman.env               WARNING  Data directory is not writable, trying to fix it...
chown: cannot read directory '/home/app/.local/share/spoolman': Permission denied
spoolman.env               ERROR    Data directory is not writable. Please run "sudo chown -R 1000:1000 /path/to/spoolman/datadir" on the host OS.
ERROR:    Traceback (most recent call last):
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 732, in lifespan
async with self.lifespan_context(app) as maybe_state:
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 608, in aenter
await self._router.startup()
File "/home/app/spoolman/.venv/lib/python3.11/site-packages/starlette/routing.py", line 709, in startup
await handler()
File "/home/app/spoolman/spoolman/main.py", line 117, in startup
env.check_write_permissions()
File "/home/app/spoolman/spoolman/env.py", line 391, in check_write_permissions
sys.exit(1)
SystemExit: 1
ERROR:    Application startup failed. Exiting.

I've spent an hour trying to get Spoolman working with podman (which I've never used before) but there seem to be issues with permissions definitely. The only way I got it working is by simply running it with rootful podman, so like
sudo podman-compose up -d
did you try that?

Defeats the whole purpose of podman.
Podman was designed and built to be rootless.
---- On Mon, 25 Nov 2024 13:44:07 -0500 @.*** wrote ----
I've spent an hour trying to get Spoolman working with podman (which I've never used before) but there seem to be issues with permissions definitely. The only way I got it working is by simply running it with rootful podman, so like
sudo podman-compose up -d
did you try that?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Sure, but I don't use it and thus don't know how to support it. Feel free to submit a pull request with the changes needed to make it work.

Well, I know less about programming than you know about podman ;-) so ... I won't be generating a pull request.
Thanks for trying.
---- On Mon, 25 Nov 2024 16:10:10 -0500 Donkie @.***> wrote ---
Sure, but I don't use it and thus don't know how to support it. Feel free to submit a pull request with the changes needed to make it work.
—
Reply to this email directly, https://github.com/Donkie/Spoolman/issues/546#issuecomment-2499040424, or https://github.com/notifications/unsubscribe-auth/AUOUH6452JMFILPDAOVYLG32COG3FAVCNFSM6AAAAABSMLFZPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJZGA2DANBSGQ.
You are receiving this because you authored the thread.

The problem is that podman will remap the root user inside the container to the regular user outside of the container. Because of this images that are build to run as the root user by default tend to just work. However the image of spoolman does user switching for you, which causes permission issues because of this. There's the very useful --userns keep-id because of this (where the container would run under the same user id as you have outside of the container, instead of as root).. But that doesn't work either because of the user switching part. I got it to work quite easily by just ignoring the build in entrypoint that does the user switching. As in, the following command works for me.
podman run --userns keep-id --entrypoint uvicorn -v /tmp/spoolman/:/home/app/.local/share/spoolman -p 8000:8000 -it --rm ghcr.io/donkie/spoolman:latest spoolman.main:app --host ${SPOOLMAN_HOST:-<IP_ADDRESS>} --port ${SPOOLMAN_PORT:-8000} "$@"
Do however keep in mind that with this approach, defining SPOOLMAN_HOST or SPOOLMAN_PORT as environment variables inside the container won't do anything, as it's evaluated earlier already.


Plugin load fail

first time trying to use the plugin and this occurred:
15:53:07 [EROR] (cssharp:PluginManager) Failed to load plugin from C:\Users\Administrator\Documents\cs2\game\csgo\addons\counterstrikesharp\plugins\CS2-Essentials\CS2-Essentials.dll CounterStrikeSharp.API.Core.NativeException: Invalid function pointer    at CounterStrikeSharp.API.Core.ScriptContext.CheckErrors() in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/ScriptContext.cs:line 176    at CounterStrikeSharp.API.Core.NativeAPI.HookFunction(IntPtr function, InputArgument hook, Boolean post) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/API.cs:line 1064    at hvhgg_essentials.Plugin.Load(Boolean hotReload) in /home/runner/work/CS2-Essentials/CS2-Essentials/CS2-Essentials/Plugin.cs:line 103    at CounterStrikeSharp.API.Core.Plugin.PluginContext.Load(Boolean hotReload) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/PluginContext.cs:line 218    at CounterStrikeSharp.API.Core.Plugin.Host.PluginManager.LoadPlugin(String path) in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/Host/PluginManager.cs:line 125    at CounterStrikeSharp.API.Core.Plugin.Host.PluginManager.Load() in /home/runner/work/CounterStrikeSharp/CounterStrikeSharp/managed/CounterStrikeSharp.API/Core/Plugin/Host/PluginManager.cs:line 93

Update the plugin / gamedata. :-)


Support setting custom HTTP headers

This ticket requires a feature that exposes API(s) to allow setting custom HTTP headers for HttpBackend. Or exposes the underlying http_client like https://github.com/apache/opendal/issues/4389

We now offer an HTTP client builder. Perhaps we could allow direct HTTP client integration, enabling OpenDAL to work with a wider range of HTTP clients.

We can add a new API called with(client: reqwest::Client) to allow users to build HttpClient via their own client. Would you like to give it a try?

Sure, please assign :+1:

By the way, the build API should work for now. Please try add headers in ClientBuilder and pass into http backend.

We now offer an HTTP client builder. Perhaps we could allow direct HTTP client integration, enabling OpenDAL to work with a wider range of HTTP clients.

We can add a new API called with(client: reqwest::Client) to allow users to build HttpClient via their own client. Would you like to give it a try?

@Xuanwo
The current method of handling request headers seems incompatible with usage in other language bindings.

The current method of handling request headers seems incompatible with usage in other language bindings.

Yes, setting custom HTTP headers for other bindings is a seperate feature to support. Would you like to create an issue for this?


Combobox issue

Hi I am implementing the Combobox component and added the property for singleInputDisabled but it does not disable the input.

It should only disabled the read only variant and and inline listbox. Is that missing from the docs?

Yes variant is set to readonly

https://react.lightningdesignsystem.com/components/comboboxes/#examples
It's working for me.

Please provide how to duplicate and re-open. Thanks!

Is it not working for multiple select read only?


ng generate to specific file path not working

OS?
OSX El Capitan (10.11.13)
Versions.
angular-cli: 1.0.0-beta.11-webpack.8
node: 6.2.0
os: darwin x64
Repro steps.
Trying to run ng g component mycomp\thiscomp as per the CLI Reference
ng g component <path/to/component-name>

I have tried several permutations, but the path gets ignored and the generated output gets put into the root of the project.
For example
ng g component mycomp\thiscomp
installing component
  create src/app/mycompthiscomp/mycompthiscomp.component.css
  create src/app/mycompthiscomp/mycompthiscomp.component.html
  create src/app/mycompthiscomp/mycompthiscomp.component.spec.ts
  create src/app/mycompthiscomp/mycompthiscomp.component.ts

Thanks for checking this out !

my bad.
The issue is that I was using the wrong slash.
This will work:
ng g component mycomp/thiscomp

how to generate only one component file in angular, only one file
ex:  name-parent.component.ts

@Anitha-Hexaware use command flags like --inline-template --inline-style etc
here is more info


Influx CLI 2.7.0

ETA: April 5 2022
See changelog for all PRs associated with this release:
Bug Fixes

4ce9053: Updated api for deleting secrets so "influx secret delete" command works again
4046d99: Typo in alias of bucket update command

Features

625273a: Add new replication field to better show queue sync progress

Other

5546ab6: Chore: upgrade go releaser to v1.13.1
48e8f5b: Chore: update to go 1.19.6
1164ca5: Chore: use fpm for packaging
88517ea: Chore: version bump sys to support riscv
2aea3fc: Chore: update openapi

Thanks @jeffreyssmith2nd. Prepping everything today, although, thanks to your efforts (🙏🏻), I think this is mostly already done.

Closing as resolved by #4847


Drop Bluebird as dependency

Bluebird is faster than the default Promise implementation, but it should be up to the user which Promise implementation to use. Bluebird is not included in the browser version to save file size, and it may make sense not to make it default in Node.js as well.
If the differences are really minimal then I think it makes total sense.

Done https://github.com/fortunejs/fortune/commit/6e912299e7ef8604861528875bc05cdc8dff7480.


Create promote feature and update demote command

Describe your changes
Fix minor changes to demote command
Create Promote feature
Issue ticket number and link
Closes #114
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

Codecov Report
Attention: Patch coverage is 86.79245% with 7 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

...edu/address/logic/parser/PromoteCommandParser.java
73.33%
2 Missing and 2 partials :warning:

...a/seedu/address/logic/commands/PromoteCommand.java
94.44%
0 Missing and 2 partials :warning:

...va/seedu/address/logic/commands/DemoteCommand.java
0.00%
0 Missing and 1 partial :warning:

Files with missing lines
Coverage Δ
Complexity Δ

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)
15.00 <0.00> (+1.00)

...va/seedu/address/logic/commands/DemoteCommand.java
93.93% <0.00%> (-6.07%)
10.00 <1.00> (+1.00)
:arrow_down:

...a/seedu/address/logic/commands/PromoteCommand.java
94.44% <94.44%> (ø)
12.00 <12.00> (?)

...edu/address/logic/parser/PromoteCommandParser.java
73.33% <73.33%> (ø)
2.00 <2.00> (?)

... and 1 file with indirect coverage changes


Import: Support import of huddles.

For importing huddles we have to have unique huddle hashes. Huddle hashes are extracted from the list of users participating in a huddle. So to extract this user ids, we first use huddle id to getting the matching recipient, and then we use subscription to get the user ids from the recipient id.

@rheaparekh how have you tested this?  This is complicated enough that automated tests seem pretty high-value here.

@timabbott I have tested this manually several times, by checking if all of the Huddle features are present in the exported and then imported realm with the original realm. For writing automated tests, I would have to start from scratch and write tests for the entire import_realm module. I am planning to do that (after finishing with the remaining features of import/export), and after I have done that, I can write tests for this feature as well. What are your thoughts?

Have you seen zerver/tests/test_export.py?  It has coverage on a lot of import_realm.

@timabbott zerver/tests/test_export.py doesn't have coverage on import_realm. It only covers tests for export.py. I am planning to increase the coverage for both of these modules.

I'll be updating this with the tests after #9659 has been merged

I have updated this with tests.

I have updated this!

@rheaparekh can you do a rebase on this?

@timabbott I have rebased and updated this.

I gave this a careful read, and the logic looks great.  I made a small change to some variable names in the tests for clarity (pluralization), and added a TODO comment for migrating this new test to use assert_realm_values (since it doesn't), and merged this.  Thanks @rheaparekh!

Oh I forget to rebase it after the refactor got merged yesterday. I'll add another PR for changing the tests to assert_realm_values method. @timabbott @neiljp Thank you for the reviews!


bug(node-trace-sdk): could not load plugin errors are polutting stdout

Since the express module is in the list of default plugins, the plugin loader try to load it when started:
PluginLoader#load: could not load plugin @opentelemetry/plugin-express of module express. Error: Cannot find module '@opentelemetry/plugin-express'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /home/node/src/index.js

However in my case i don't want the express plugin, and i also don't want to those errors because it's expected.
I would don't think we should log if the module isn't found in this case, WDYT ?

Can you just add { express: { enabled: false } } to the plugin config?

Yeah i can do it as a workaround, however i don't think we should require users to disable plugins they don't want (since they didn't install them in the first place).

I think we should really fix this, its really common for people to think the sdk is not working when seeing those logs. Could we just log those as debug ? cc @open-telemetry/javascript-approvers

FWIW I do find them useful to know which plugin I forgot to install. But it does not merit Error level as @vmarchaud  mentioned. Warn, Info or Debug are all more appropriate than Error.

It's probably more Warn worthy for non default plugins that are explicitly enabled but not found, but implementing that check for explicitly enabled might be more logic than its worth. Let's just choose A lower level for 'em all.

Yeah my main issue is that when a default plugin is added, i need to add an entry in the tracing sdk to ignore this plugin specially. There is currently 13 default plugins, so by default if someone just want to use the http one, there will be 12 error logs at startup.

It's probably more Warn worthy for non default plugins that are explicitly enabled but not found

That would have been the best solution but since the config are merged there no way at the plugin loader level to know if the plugin was explicitly enabled or not :/

FYI I'm refactoring default plugin instrumentation (started by @dyladan ) once this is done this might go away as the approach for plugin will be different. If this is not urgent you might think of delaying that until then.

That's fine for me, waiting for your PR then

Closing since it will be fixed with the instrumentation class


remove starport stubs

remove starport stubs from Neutron
format code

This is mainly a cosmetic PR, and can safely be reviewed and merged post launch.

I would really hope that we don't.  Super bigly ser.

It looks like this PR completely breaks ignite

it totally should break ignite.
That's the point....


CodeSignatureIdentifier update for XMind.download.recipe

A client asked to add XMind to SelfService so I did an autopkg search and found your two recipes for download and pkg. Thanks for providing them.
However when running the recipe I got an CodeSignatureVerifier error:
Error in com.github.ahousseini-recipes.pkg.XMind: Processor: CodeSignatureVerifier: Error: Code signature verification failed. Note that all verifications can be disabled by setting the variable DISABLE_CODE_SIGNATURE_VERIFICATION to a non-empty value.
I put XMind in Applications and run a codesign against it. It turns out that they changed the Signature to:
identifier "net.xmind.vana.app" and anchor apple generic and certificate 1[field.1.2.840.1136<IP_ADDRESS>.6] /* exists */ and certificate leaf[field.1.2.840.1136<IP_ADDRESS>.13] /* exists */ and certificate leaf[subject.OU] = "4WV38P2X5K"
I made a copy of your two recipes, changed the identifier for both, replaced the CodeSignatureVerifier requirement with the one above and I can now successfully build XMind.
Hope this helps,
Eric

Thanks, I updated the recipe: https://github.com/autopkg/ahousseini-recipes/commit/fcb3ae95a14713e062c18f05a7d9933108532cf1


db_download error

System: MacOS Ventura, Python version:3.9.1 crabs:0.1.4
Code:
crabs db_download --source ncbi --database nucleotide --query '12S[All Fields] AND ("1"[SLEN] : "50000"[SLEN])' --species zhuangjiang_fish.csv --output zhujiang_fish_ncbi.fasta --keep_original yes --email myemail --batchsize 500000
error report:
Traceback (most recent call last):
File "/Users/liuzifang/reference_database_creator/crabs", line 1428, in
main()
File "/Users/liuzifang/reference_database_creator/crabs", line 1425, in main
args.func(args)
File "/Users/liuzifang/reference_database_creator/crabs", line 80, in db_download
speciesTempFileList = wget_ncbi_species(QUERY, DATABASE, EMAIL, BATCHSIZE, OUTPUT, speciesName)
File "/Users/liuzifang/reference_database_creator/function/module_db_download.py", line 111, in wget_ncbi_species
print(f'downloading {seqcount} sequences from NCBI using the searchterm: {speciesSearchTerm}')
UnboundLocalError: local variable 'seqcount' referenced before assignment
I changed the script in module_db_download.py by adding seqcount=0 at line 102, also tried seqcount='', and both worked.  When I used the newest version of Crabs on an Ubuntu desktop, I didn't encounter the same error. So I guess it's possibly just happened in the older version of crabs, but I just think it could be helpful for people who encounter similar problems using the old versions.

It's happening the same to me, crabs version: 0.1.8

Hi Fabio,
It happened to me after I updated the crabs to v0.1.8 too. Try add
seqcount=0 at line 102, it will fix the problem roughly.
Best regards,
Zifang
On Sat, 25 Nov 2023 at 00:29, FabioGAmaral @.***> wrote:

It's happening the same to me, crabs version: 0.1.8
—
Reply to this email directly, view it on GitHub
https://github.com/gjeunen/reference_database_creator/issues/42#issuecomment-1825885932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMK5ZD6OWMBA2BJYWE4GPKDYGDDNPAVCNFSM6AAAAAA66EB7NKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVHA4DKOJTGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm getting this same error but unsure how to fix when using Docker bc unsure how to edit the module_db_download.py script

Hello @saraswaminathan,
Thank you for your message.
We're introducing a new version of CRABS tomorrow, which represents a full overhaul of the code, with added functionality, increased speed, improved file and error handling, enhanced flexibility, and simplified installation through fewer dependencies. This version will first be available by cloning the GitHub repository, with the other installation options following shortly after. The new version should also fix the error you are currently encountering.
Best wishes,
Gert-Jan


Without Google play services

We have custom built AOSP without google play services, can we use the mixpanel library without the google play services?  Is it recommended to use the mixpanel-java library instead of mixpanel-android.
Thanks,
-Kuppa

You can perfectly use mixpanel-android library without google play services. We use google play services to send push notifications so you customers won't be able to receive push notifications through our library but you can use any other features :)


Raspberry Pi 4 and Intel RealSense SDK 2.0

Required Info

Platform
Raspberry Pi

SDK Version
{  2.36.0 }

Language
{C++,python }

Segment
{installation}

Issue Description
is there a way to install the sdk (not the viewer) on the raspberry pi 4?
i tried following the instructions in the following link:
https://github.com/acrobotic/Ai_Demos_RPi/wiki/Raspberry-Pi-4-and-Intel-RealSense-D435
but its seems that the fact that the raspberry is using arm and not x isa dose not allow me to properly install the sdk.
i tried to look for a proper solution but all i could find is a work arounds at best.
let me know if its possible or if i need to change my sbc.
thank you
netanel

Hi @netanelfar   The RealSense 400 Series cameras can work with any Intel or Arm processor.  Librealsense certainly can be installed for Arm devices such as the Pi.  The Acrobotic installation guide for Pi 4 is actually similar in content to the official Intel installation guide for Pi 3.
For a quick setup, Intel have a pre-built SD card image for Pi 4 that can be downloaded and installed.  This can act as a reference when learning how to install librealsense on Pi 4 yourself.
https://dev.intelrealsense.com/docs/open-source-ethernet-networking-for-intel-realsense-depth-cameras#section-2-3-preparing-the-sd-card
An easier method of installation than the guides mentioned above may be to use a method called libuvc-backend.
https://github.com/IntelRealSense/librealsense/blob/master/doc/libuvc_installation.md
Could you try this method please and see how it goes?

thank you marty.
another question when using frame.get_data what is the type of each pixel?(what is the size of it).
if i want to split the frame bit by bit and put it in a matrix.
thanks again

The question of size may be covered by get_data_size(), which is described in the C++ discussion in the link below.
https://github.com/IntelRealSense/librealsense/issues/5718
get_data_size() retrieves data size from the frame handle and returns the number of bytes in a frame.

Hi @netanelfar   Do you require further assistance with this case, please?   Thanks!

@MartyG-RealSense Thank you for now i am good.
thanks again for all the help!

Thanks @netanelfar for the update!

@MartyG-RealSense
Howdy Marty?
Does the rpi image get updated?
Or as for today there is no somewhat recent rpi image with realsense support pre-built by intel? Thanks

@AndreV84   As far as I am aware, the pre-built Pi SD card image in the open source ethernet white paper does not get updated from its original support for librealsense 2.34.0.

so we would just build like that https://github.com/IntelRealSense/librealsense/blob/master/doc/libuvc_installation.md ?
will it address the metadata for d455 on raspberry?
e.g. on arm jetson we would apply Jetson librealsense L4T patch
how to address it on rpi?

@MartyG-RealSense

The build script 'libuvc_installation.sh' uses the CMake build flag -DFORCE_LIBUVC=true and so metadata support should be included in the build (as DFORCE_LIBUVC=true is the predecessor of FORCE_RSUSB_BACKEND=true)

@MartyG-RealSense
Yes, but on Jetson we wold aditionally patch the kernel for d455
with librealsense script L4T
in addition to building with RSUSB

Intel's official Jetson installation instructions offer a choice of building librealsense with the RSUSB Backend by using  the 'libuvc_installation.sh' script so that you do not need to apply a patch, or using the Native Backend and patching the kernel with the 'patch-realsense-ubuntu-L4T.sh' script.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_jetson.md#building-from-source-using-rsusb-backend
If you build with LIBUVC / RSUSB then you should not be running a kernel patch script.


”Armeniș” is listed in the drop-down menu

Description:
"Armeniș" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Armeniș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Armeniș, Caraș-Severin".


Add waiting for loading search results

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


AMBARI-24635. Web Client Chooses Wrong Version When Reverting Configs

What changes were proposed in this pull request?
STR:
Setup a simple cluster with ZK, HDFS, YARN, Hive, Pig

Create at least 5 different configuration versions for Pig
Load the Pig configuration page and select v4
Now select V3 and then choose "Make Current"
At this point, the web client will populate a dialog that says Created from service config version V4 and indeed it restores v4, not v3.

How was this patch tested?
21814 passing (48s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3967/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3971/
Test PASSed.


IP address anonymization possible?

Is it possible to use this library to parse IPv4 and IPv6 addresses and anonymize them like Google Analytics does?
That is:

The IP anonymization feature in Analytics sets the last octet of IPv4 user IP addresses and the last 80 bits of IPv6 addresses to zeros

For example, an IP address of <IP_ADDRESS> would be changed to <IP_ADDRESS>. (If the IP address is an IPv6 address, the last 80 of the 128 bits are set to zero.)

bitwise AND should do it:
>>> from iptools import ipv4, ipv6
>>> ipv4.long2ip(ipv4.ip2long('<IP_ADDRESS>') & ipv4.ip2long('<IP_ADDRESS>'))
'<IP_ADDRESS>'
>>> ipv6.long2ip(ipv6.ip2long('2607:f8b0:400e:c08::8b') & ipv6.ip2long('ffff:ffff:ffff::'))
'2607:f8b0:400e::'


Improvements of reddit scraper

Additions and modifications:

Enriched submission result and comments with additional data
Added submission and comment author data to result
Added docker env to config; based on env var scraper can now be started locally or dockerized
Added pre-commit to requirements; adjusted some pre-comment config parameters
Reddit scraper now fetches up to 100 hot submissions per subreddit instead of just 100 hot submission of all given subreddits
Added docstrings to RedditDataCollector class
Commented in subparser statements for reddit and twitter that were removed before

Necessary /.vscode/launch.json to run changes made to config.json in this branch with Visual Studio Code:
{
      "name": "Reddit Debug Local",
      "type": "python",
      "request": "launch",
      "program": "<path_to_project>/scraping-scripts/src/data_collection.py",
      "console": "integratedTerminal",
      "justMyCode": false,
      "args": ["--config", "<path_to_project>/scraping-scripts/config.json", "reddit"],
      "env": {
          "SCRAPER_ENV_LOCAL": "true",
      }
  },
  {
      "name": "Reddit Debug Docker",
      "type": "python",
      "request": "launch",
      "program": "<path_to_project>/scraping-scripts/src/data_collection.py",
      "console": "integratedTerminal",
      "justMyCode": false,
      "args": ["--config", "<path_to_project>/scraping-scripts/config.json", "reddit"],
      "env": {
          "SCRAPER_ENV_LOCAL": "false",
      }
 },

Fixed unit tests by adding a pytest.ini to project root an defining pathpath (and testpaths). See pytest configuration and pytest configuration options for reference.

Please remove any tokens or secrets from your config.json before committing

@lukasgraef i'll remove them for now. However, this should be handled entirely different, i.e. by setting config.json's contents upon start of data_collection.py and reading sensitive data from a .env, or similar way.

@dim35216: The number of workers is bounded by max_workers, which is currently set to 20 workers. You can change this variable in config.json.
The number you're refering to is the total amount of futures created. Those get distributed among the 20 workers of our Threadpool, so I think our current implementation should be alright.


[Regression] node.external.partitions returned as string instead of dict

Is this a regression in a recent version of dbt-core?

[X] I believe this is a regression in dbt-core functionality
[X] I have searched the existing issues, and I could not find an existing issue for this regression

Current Behavior
In dbt 1.8.9 it seems like the behavior of node.external.partitions has changed. It used to return a list of dicts, but now it seems to return a list of json strings.
This seems to be causing problems for dbt-external-tables when trying to create external tables with partitions defined.
I'm using this macro to compare the behaviors:
{% macro get_partitions() -%}
    {% set source_nodes = graph.sources.values() if graph.sources else [] %}
    {% for node in source_nodes %}
        {% if node.external %}
            {% if node.external.partitions %}
                {{print(node.external.partitions)}}
            {% endif %}
        {% endif %}
    {% endfor %}
{%- endmacro %}

dbt 1.8.8:
dbt run-operation get_partitions
11:23:26  Running with dbt=1.8.8
11:23:27  Registered adapter: snowflake=1.8.4
11:23:31  Found 1412 models, 2 snapshots, 6 seeds, 5 operations, 129 data tests, 619 sources, 2 exposures, 730 macros
[{'name': 'dl_partition', 'data_type': 'string', 'expression': "<redacted>"}]

dbt 1.8.9:
dbt run-operation get_partitions
11:21:38  Running with dbt=1.8.9
11:21:38  Registered adapter: snowflake=1.8.4
11:21:43  Found 1396 models, 2 snapshots, 6 seeds, 5 operations, 129 data tests, 611 sources, 2 exposures, 735 macros
['{\'name\': \'dl_partition\', \'data_type\': \'string\', \'expression\': "<redacted>"}']

Expected/Previous Behavior
I expect node.external.partitions to be a list of dicts so that the values can be accessed by looping the list and addressing with i.e. partition.name
Steps To Reproduce

Install dbt 1.8.9
Create a yaml source file called testtable.yml with external and partition set. I.e.:

version: 2
sources:
- name: TEST
  schema: STAGE
  tables:
  - name: TABLE
    external:
      partitions:
      - name: dl_partition
        data_type: string
        expression: split_part(METADATA$FILENAME, '/', 2)

Create a test macro called get_partitions.sql:

{% macro get_partitions() -%}
    {% set source_nodes = graph.sources.values() if graph.sources else [] %}
    {% for node in source_nodes %}
        {% if node.external %}
            {% if node.external.partitions %}
                {{print(node.external.partitions)}}
            {% endif %}
        {% endif %}
    {% endfor %}
{%- endmacro %}

Run the macro:
dbt run-operation get_partitions

Relevant log output
No response
Environment
- OS: Windows 11 23H2/Ubuntu 22.04.5 LTS
- Python: Python 3.12.4
- dbt (working version): 1.8.8
- dbt (regression version): 1.8.9

Which database adapter are you using with dbt?
snowflake
Additional Context
No response

Thanks for reporting this @thomasgjerdekog !
I'm not seeing an obvious code difference between dbt-core 1.8.8 and 1.8.9 that explains what you're seeing, so I'm guessing it is via some dependency.
Could you share all the dbt versions for your dependencies in each of those different environments?
Assuming that you're using pip to install dbt-core, you can run something like this to get those versions:
pip list | grep '^dbt-'

Output might look something like this:
dbt-adapters               1.4.0
dbt-common                 1.7.0
dbt-core                   1.8.8
dbt-postgres               1.8.2
dbt-semantic-interfaces    0.5.1

Hi @dbeatty10
Sure!
Old (working):
pip freeze | grep "^dbt-"
dbt-adapters==1.7.0
dbt-common==1.10.0
dbt-core==1.8.8
dbt-extractor==0.5.1
dbt-osmosis==0.12.8
dbt-semantic-interfaces==0.5.1
dbt-snowflake==1.8.4

New (not working):
pip freeze | grep "^dbt-"
dbt-adapters==1.10.2
dbt-common==1.13.0
dbt-core==1.8.9
dbt-extractor==0.5.1
dbt-osmosis==0.12.8
dbt-semantic-interfaces==0.5.1
dbt-snowflake==1.8.4

I tried experimenting a bit with downgrading the dbt-adapters and dbt-common packages prior to submitting this issue, but I wasn't able to resolve the install conflicts since dbt-core 1.8.9 depends on protobuf >= 5.0 while versions of dbt-adapters prior to 1.10.0 depend on protobuf < 5.0

Running into the same issue on dbt-core 1.7.18 and dbt-redshift 1.7.7

Pinning mashumaro also worked for us.

Weird. Pinning mashumaro to 3.13.1 still produces the partitions as strings for me:
pip freeze | grep "^dbt-\|mashumaro"
dbt-adapters==1.10.2
dbt-common==1.13.0
dbt-core==1.8.9
dbt-extractor==0.5.1
dbt-osmosis==0.12.8
dbt-semantic-interfaces==0.5.1
dbt-snowflake==1.8.4
mashumaro==3.13.1

dbt run-operation get_partitions
13:33:13  Running with dbt=1.8.9
13:33:14  Registered adapter: snowflake=1.8.4
13:33:18  Found 1394 models, 2 snapshots, 6 seeds, 5 operations, 129 data tests, 611 sources, 2 exposures, 738 macros
['{\'name\': \'dl_partition\', \'data_type\': \'string\', \'expression\': "<redacted>"}']

flagging that multiple users of dbt-external-tables are experiencing this and reported in https://github.com/dbt-labs/dbt-external-tables/issues/325


Add a decrement button to counter page

We can increment the counter but not decrement it please fix this.

Job Done


Battle descriptions disappear on refresh

From #37
Opening this for discussion, but currently no game history is stored on the server, and this is entirely client-side. We'd either need to change that, or store it in localstorage. (In the future, moving between devices will require storing this somewhere on the backend anyway, though, but ideally not in our hot game storage)

We now have a database that stores sequences of moves for single player games. Storing this for all games and replaying them to get the battles again seems like the best course of action.


fix(gantry, head): Updated EVT tmc2160 register settings, plus lowering microstepping

Context
Big discovery, David Gomez found our EVT units had the wrong sense resistors. This was why there were SO DAMN LOUD. So now, even at 32 microstepping, the gantry sounds just fine.
We found that at 64 microstepping, we aren't yet able to reach the physical max speeds of the gantry. If we lower it to 32, then we can.
David has also been doing testing using a scope to try and get our motor waveforms close to ideal.
This PR
Makes the following changes:

Decreases gantry microstepping from 64 to 32
Decreases head microstepping from 32 to 16
Explicitly disables Stealth-Chop on both head and gantry

@djgomez-opentrons is my description accurate?


is not between

I want to select data where workflow is not between 3000 and 3100. Is there any method  for is not between?

Using workflow.isBetweenVaues(3000, 3100).not() should work here.


Recently.

EOFoading LFS objects: 100% (1/1), 232 KB | 0 B/s
Maybe just a need to update? Don't have time ot handle this this week.

I was just using the wrapper instead of the executable. Made the wrapper handle outside repos.


C-API: how to exchange scalar data or arrays or lists

Thank for this clever python alike library! This could be useful for many applications, not only gaming.
Sorry for my ignorance: What is the easiest way to exchange scalar data (float, int) beside of chars with the help of the C-API? And is there a way to exchange arrays of data of uniform data types?
And as I'm curious: what is the meaning of pkpy_setup_callbacks?
I'm thinking of using pocketpy as a kind of programmable input-file for data processing.

The C-API is designed for communicating with languages other than C/C++(as a dynamic library). There is an internal protocol to transfer basic types. But the functionality is limited.

If you are using C or C++, just use C++ apis to do integration. CAST and VAR macros can do convert between Python objects and c types. For array, use List.

But, yes, the docs are a bit misleading...

Another solution is to get a 3rd json library.
You can use pkpy_vm_eval to run something and return json string and parse it via that library.

For the sake of simplicity I will try some kind of serialization.


Parsing Prompt Node output (e.g. via JavaScript) and passing onto subsequent chains

Is there a way to parse the output of an LLM (i.e. a Prompt Node) so that this can be used as an input to another chain. So for example I can use an evaluator and parse a JSON generated by an LLM to only return a specific field of the generated JSON. I can return this string from the evaluator and connect it to a TextField Node, but when evaluating or looking at a preview of the prompt, the variable is not replaced by the output of the evaluator node. Is this correct behaviour? Is there something else I can use to parse outputs of LLM nodes to be passed on to further chains? Thanks!

Thanks for opening this issue. You are correct that evaluators are not the proper place for this function. Evaluators do not process responses, but merely annotate them.
What you're asking for is a kind of "processor" node. The outcome of our recent user study (https://arxiv.org/abs/2309.09128) suggested "Processor" nodes as the most-requested feature by real users, so we're definitely getting to them. The Issue https://github.com/ianarawjo/ChainForge/issues/132 is also a processor node.
I'll keep this open until we get to them. Although it seems like a simple problem, there will need to be special considerations for a code-based processor --for instance, is related metadata still attached? Do you have access to, for instance, changing the key of the LLM? Maybe there's multiple variants of a processor --easiest variant is just equal to the evaluate function. But, another variant lets you analyze all responses at once across all LLMs. Maybe another variant lets you analyze responses only within each LLM. ...at any rate, it's a nontrivial design problem.

Realizing this is a duplicate issue to https://github.com/ianarawjo/ChainForge/issues/119
Keeping this open atm for context.

Ah ok, I’ll take a look later. For now just use the built-in Inspect Responses footer button. Thanks!

Thank you so much for bringing this major issue to our attention. It has now been fixed in <IP_ADDRESS>, with changes pushed to the /play site.


Unexpected scaling down to 0

When a vertex has all the messages read and processed, and keeps retrying to write to next ISB (due to back pressure), the pending messages calculation only counts number of pending, which is 0. This will lead to a scaling down to 0, thus a pod termination will happen (even though the code logic tries to do clean shutdown after receiving SIGTERM, the pod has terminationGracePeriodSeconds which will do a force shutdown), which will end up with ack pending messages not 0 in the buffer information of the UI, while the pod number is 0.

@sarabala1979


'EXPORTERS', 'face-detection', 'scrfd') not found in ast index file

use model-scope to export scrfd onnx model error

Hi, you can just use official tools scrfd2onnx.py, but need to comment line 149, and set opset_version to 9.


Added ability to set icon to ActionsMenu.

I added ability to set the icon to the FloatinActionsMenu.
Add new attribute fab_addButtonIcon and fab_addButtonIconPressed.
Only when fab_addButtonIcon is set the icon is displayed, and if not, you can use the plus button in the same way as up to now.
If you set both fab_addButtonIcon and fab_ButtonIconPressed,  a switching animation of icon is enabled.
Please see the sample below.

nice, exactly what I need

Having this feature in the master branch would be lovely. I would need it too

Exactly what I wanted and was going to create a PR myself. Thank you.

Hey, do you have a version with this implemented?

Exactly what I wanted and was going to create a PR myself. Thank you.

Haha same. It's a shame this repo is not maintained anymore.


changelogs: Pre-release cleanups

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/docs
apologies for fp it was just a format change

pushing through to expedite release


tensorflow estimator error in v2.16

I am trying to use your API with TF2.16 but I got this error:
from neural_structured_learning import estimator
File "/data2/users/angeloso/anaconda/ajsoto/lib/python3.11/site-packages/neural_structured_learning/estimator/init.py", line 7, in
from neural_structured_learning.estimator.adversarial_regularization import add_adversarial_regularization
File "/data2/users/angeloso/anaconda/ajsoto/lib/python3.11/site-packages/neural_structured_learning/estimator/adversarial_regularization.py", line 27, in
from tensorflow import estimator as tf_estimator
ImportError: cannot import name 'estimator' from 'tensorflow' (/data2/users/angeloso/anaconda/ajsoto/lib/python3.11/site-packages/tensorflow/init.py)
Are you planning update your API soon?

Thank you for reaching out!  Neural Structured Learning (NSL) is currently in a maintenance phase, which means there are no immediate plans for major updates or active API development. We value your interest and will keep you informed of any changes in the future.

Ok. Thank you.


qbs: 1.17.0 -> 1.18.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/qbs/versions.
meta.description for qbs is: "A tool that helps simplify the build process for developing projects across multiple platforms"
meta.homepage for qbs is: "https://wiki.qt.io/Qbs"
meta.changelog for qbs is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.18.0 with grep in /nix/store/k16gh8xc0n0qysif0w6gbahjygxrib3d-qbs-1.18.0

directory tree listing: https://gist.github.com/589eebdd67b0d99c71018b6c73e6a1ef

du listing: https://gist.github.com/7e268ee2eb21a7244d326b0be9a8d39b

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
qbs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k16gh8xc0n0qysif0w6gbahjygxrib3d-qbs-1.18.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A qbs https://github.com/r-ryantm/nixpkgs/archive/10a3daee9e9a301259f73249b61e307a49606a56.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k16gh8xc0n0qysif0w6gbahjygxrib3d-qbs-1.18.0
ls -la /nix/store/k16gh8xc0n0qysif0w6gbahjygxrib3d-qbs-1.18.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

qbs

Maintainer pings
cc @expipiplus1 for testing.

lgtm!


behavior tests

TL;DR
This is a working example of behavior test. Consider workerman as a black box server and verify tests as clients.
As discussed in #904 , I mentioned the behavior test is not doable because pest lacks supporting for isolation process tests. However, after experiments with PhpUnit, I realized @runinseparateprocess doesn't help as well. The only programming way to kill the service itself from inside is to kill the whole process, but even for PhpUnit it still requires to do certain jobs after the test method(s). Killing the proces is considered as an error and will leads to a test failure.
This PR proposed a different way for such tests. Instead of trying to asset values on both side, we consider the server side as a black box and only pay attention to the value we sent and received as a client.  To archive this, we create a new process during beforeAll/setUp (with the help of symfony's PhpProcess) and stop it during afterAll/tearDown. Thus all tests in same file/class can communicate with the server and checking the server's behavior.

p.s. a not-so-small suggestion. Tests are not just about adding more test cases. Workerman also needs to be changed/modified to enable better and more comprehensive tests. e.g. When I tried to start workerman, the only way I found is to override $argv. There is a static::$command and getArgv in Worker so I guess the author does have an intention to support such use case. However, parseCommand still only use $argv as the only input, which make $command much less useful.

Tests are not just about adding more test cases. Workerman also needs to be changed/modified to enable better and more comprehensive tests. e.g. When I tried to start workerman, the only way I found is to override $argv. There is a static::$command and getArgv in Worker so I guess the author does have an intention to support such use case. However, parseCommand still only use $argv as the only input, which make $command much less useful.

That's ok. I Removed unnecessary $argv.
Thank you for your post.

I have a suggestion before, that is let workerman can run only once like a command, and no master and childprocess, only one process, the process exit after something finished.
Consider to implement that? @walkor

I have a suggestion before, that is let workerman can run only once like a command, and no master and childprocess, only one process, the process exit after something finished.

I think what you described is how Workerman works under Windows.
However, I don't think it's a good idea for testing purpose. We should try to test in an enviroment closing to the actual production.


KFLUXINFRA-651: Adding Instance Types for Multi-Platform Builds

Adding two types of instances

Higher Memory (1:4 ratio for cpu: memory)
Higher CPU (1:2 ratio for cpu: memory)

Note Some users may require more memory and less CPU, making Memory Optimized Instances the better choice for them. On the other hand, some users may need more CPU power but less memory, in which case CPU-optimized instances would be a good fit. Therefore, a combination of memory-optimized and CPU-optimized instances is essential to cater to different user needs. In the label naming, m refers to memory optimised and c refers to compute optimised.

Memory Optimised (not aws terminology, their memory optimized class starts from r)

Multi-Platform Label
Instance Type
Architecture
CPU
Memory(GB)

linux-mlarge/amd64
m5a.large
AMD64
2
8

linux-mxlarge/amd64
m6a.xlarge
AMD64
4
16

linux-m2xlarge/amd64
m6a.2xlarge
AMD64
8
32

linux-m4xlarge/amd64
m6a.4xlarge
AMD64
16
64

linux-m8xlarge/amd64
m6a.8xlarge
AMD64
32
128

linux-mlarge/arm64
m6g.large
ARM64
2
8

linux-mxlarge/arm64
m6g.xlarge
ARM64
4
16

linux-m2xlarge/arm64
m6g.2xlarge
ARM64
8
32

linux-m4xlarge/arm64
m6g.4xlarge
ARM64
16
64

linux-m8xlarge/arm64
m6g.8xlarge
ARM64
32
128

CPU Optimised (again not aws terminology)

Multi-Platform Label
Instance Type
Architecture
CPU
Memory(GB)

linux-clarge/amd64
c6a.xlarge
AMD64
4
8

linux-cxlarge/amd64
c6a.2xlarge
AMD64
8
16

linux-c2xlarge/amd64
c6a.4xlarge
AMD64
16
32

linux-c4xlarge/amd64
c6a.8xlarge
AMD64
32
64

linux-clarge/arm64
c6g.xlarge
ARM64
4
8

linux-cxlarge/arm64
c6g.2xlarge
ARM64
8
16

linux-c2xlarge/arm64
c6g.4xlarge
ARM64
16
32

linux-c4xlarge/arm64
c6g.8xlarge
ARM64
32
64

Suggestion We can minimize code duplication by utilizing default values for common parameters like imageId, subnetId, region, etc, unless specified otherwise.

One thought: we will have quay.io with AWS internal communication right? otherwise how about using enhanced network enabled instances (with *n) for better push pull speeds?

One thought: we will have quay.io with AWS internal communication right? otherwise how about using enhanced network enabled instances (with *n) for better push pull speeds?

ENA is enabled for the instance types we are using m6a.xxxxx, m6g.xxxxx, c6a.xxxxx and c6g.xxxxx. Reference AWS Document -- https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/enhanced-networking-ena.html

/lgtm


⚠ Processors has degraded performance

In 739ad80, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 61 ms

Resolved: Processors performance has improved in 9640b22.


Pack incompatible with tns version 4.2.1

Make sure to check the demo app(s) for sample usage
Followed the tutorial here line for line https://docs.nativescript.org/plugins/building-plugins
Make sure to check the existing issues in this repository
No issues present
If the demo apps cannot help and there is no issue for your problem, tell us about it
Please, ensure your title is less than 63 characters long and starts with a capital
letter.
Unable to pack plugin. It ends with this error
`
nativescript-version-number/publish/src/node_modules/semver/semver.js:310
throw new TypeError('Invalid Version: ' + version);
^
TypeError: Invalid Version: 4.2.1
`
Which platform(s) does your issue occur on?

iOS/Android/Both
iOS/Android versions
emulator or device. What type of device?
It happens during packing

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)
4.2.1

Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project)

4.2.0

Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project)
tns-android: 4.0.0
tns-ios: 4.0.0

Plugin(s): (look for the version numbers in the package.json file of your
project and paste your dependencies and devDependencies here)
{   "name": "nativescript-version-number",   "version": "1.0.0",   "description": "Your awesome NativeScript plugin.",   "main": "version-number",   "typings": "index.d.ts",   "nativescript": {       "platforms": {           "android": "4.0.0",           "ios": "4.0.0"       }   },   "scripts": {       "tsc": "tsc",       "build": "npm run tsc && npm run build.native",       "build.native": "node scripts/build-native.js",       "postclone": "npm i && node scripts/postclone.js && cd ../demo && npm i && npx rimraf -- package-lock.json && cd ../src",       "test.android": "npm run tsc && npm run tslint && cd ../demo && tns build android && tns test android --justlaunch",       "test.ios": "npm run tsc && npm run tslint && cd ../demo && tns build ios && tns test ios --justlaunch",       "tslint": "cd .. && tslint \"**/*.ts\" --config tslint.json --exclude \"**/node_modules/**\"",       "plugin.tscwatch": "npm run tsc -- -w",       "demo.ios": "npm run tsc && cd ../demo && tns run ios --syncAllFiles --emulator",       "demo.android": "npm run tsc && cd ../demo && tns run android --syncAllFiles --emulator",       "demo.reset": "cd ../demo && npx rimraf -- hooks node_modules platforms package-lock.json",       "plugin.prepare": "npm run build && cd ../demo && tns plugin remove nativescript-version-number && tns plugin add ../src",       "clean": "npm run demo.reset && npx rimraf -- node_modules package-lock.json && npm i",       "ci.tslint": "npm i && tslint '**/*.ts' --config '../tslint.json' --exclude '**/node_modules/**'",       "prepack": "npm run build.native"   },   "keywords": [       "NativeScript",       "JavaScript",       "Android",       "iOS"   ],   "author": {       "name": "Your Name",       "email"<EMAIL_ADDRESS> },   "bugs": {       "url": "https://github.com/Rjgoolsby/nativescript-version-number/issues"   },   "license": "Apache-2.0",   "homepage": "https://github.com/Rjgoolsby/nativescript-version-number",   "readmeFilename": "README.md",   "devDependencies": {       "tns-core-modules": "^4.2.0",       "tns-platform-declarations": "^4.2.0",       "typescript": "~2.8.2",       "prompt": "^1.0.0",       "rimraf": "^2.6.2",       "tslint": "^5.11.0",       "semver": "^5.5.0"   },   "dependencies": {},   "bootstrapper": "nativescript-plugin-seed" }

Please, tell us how to recreate the issue in as much detail as possible.
Describe the steps to reproduce it.
Simple follow the tutorial
Is there any code involved?

provide a code example to recreate the problem
(EVEN BETTER) provide a .zip with application or refer to a repository with application where the problem is reproducible.

Here's proof
in /nativescript-version-number/src/node_modules/semver/semver.js line 302 hardcode the version as4.2.0 (this isn't a fix but just proves the incompatibility) and it runs just fine. If you don't set it then it comes in as 4.2.1 and it throws an invalid version.
See code below
`
function SemVer(version, loose) {
if (version instanceof SemVer) {
if (version.loose === loose)
return version;
else
version = version.version;
} else if (typeof version !== 'string') {
throw new TypeError('Invalid Version: ' + version);
}
if (version.length > MAX_LENGTH)
throw new TypeError('version is longer than ' + MAX_LENGTH + ' characters')
if (!(this instanceof SemVer))
return new SemVer(version, loose);
debug('SemVer', version, loose);
this.loose = loose;
version = "4.2.0";
var m = version.trim().match(loose ? re[LOOSE] : re[FULL]);
if (!m)
throw new TypeError('Invalid Version: ' + version);
this.raw = version;
// these are actually numbers
this.major = +m[1];
this.minor = +m[2];
this.patch = +m[3];
if (this.major > MAX_SAFE_INTEGER || this.major < 0)
throw new TypeError('Invalid major version')
if (this.minor > MAX_SAFE_INTEGER || this.minor < 0)
throw new TypeError('Invalid minor version')
if (this.patch > MAX_SAFE_INTEGER || this.patch < 0)
throw new TypeError('Invalid patch version')
// numberify any prerelease numeric ids
if (!m[4])
this.prerelease = [];
else
this.prerelease = m[4].split('.').map(function(id) {
if (/^[0-9]+$/.test(id)) {
var num = +id;
if (num >= 0 && num < MAX_SAFE_INTEGER)
return num;
}
return id;
});
this.build = m[5] ? m[5].split('.') : [];
this.format();
}
`

The latest NativeScript CLI version is 4.2.3. Can you update your environment and try again?
I tried following the steps in the tutorial and saw no errors when I packed the plugin. Here are the commands I executed:
git clone https://github.com/NativeScript/nativescript-plugin-seed nativescript-version-number

cd nativescript-version-number/src

npm run postclone

npm run plugin.tscwatch

# in another terminal again in the src directory
npm run demo.android

# stop both commands in the two terminals using Ctrl-C after making sure the demo application works
# then proceed to pack the plugin
cd ../publish
./pack.sh

This produces a file called nativescript-version-number-1.0.0.tgz in the publish/package directory with the packed plugin.

Ahh. I nailed down the issue. I was reading the error wrong.
   TypeError: Invalid Version:     Support for macOS 10.11 is deprecated and will be removed in one of the next releases of     NativeScript. Please, upgrade to the latest macOS version.    4.2.3
It's not saying the TNS version is wrong. It's saying my macOS version isn't supported. I'm using my work computer and we haven't been allowed to upgrade to 10.13 (don't ask). I tried this on my personal mac and everything runs just fine. This is an OS issue not plugin issue. Sorry for the inconvenience


cc1plus: error: unrecognized command line option "-Wno-weak-vtables"

Built on x86 Debian 5.x with kernel 2.6.18 with GCC 4.3.2. Issue #991 discusses this as a warning, however now it appears to manifest an error with both the git current 3.7dev build and the 3.8 release. I note that same changes were made to the build as a result of #991.
Is there any workaround for this? Thanks.
mv -f src/CppUTest/.deps/lib_libCppUTest_a-Utest.Tpo src/CppUTest/.deps/lib_libCppUTest_a-Utest.Po
g++ -DHAVE_CONFIG_H -I.   -include ./include/CppUTest/MemoryLeakDetectorMallocMacros.h -I ./include     -include ./include/CppUTest/MemoryLeakDetectorNewMacros.h  -Wno-c++11-long-long -Wno-long-long -Wall -Wextra -Wshadow -Wswitch-default -Wswitch-enum -Wconversion -pedantic -Wsign-conversion -Woverloaded-virtual -Wno-disabled-macro-expansion -Wno-padded -Wno-reserved-id-macro -Wno-keyword-macro -Wno-global-constructors -Wno-exit-time-destructors -Wno-weak-vtables -Wno-old-style-cast   -g -O2 -MT src/Platforms/Gcc/lib_libCppUTest_a-UtestPlatform.o -MD -MP -MF src/Platforms/Gcc/.deps/lib_libCppUTest_a-UtestPlatform.Tpo -c -o src/Platforms/Gcc/lib_libCppUTest_a-UtestPlatform.o test -f 'src/Platforms/Gcc/UtestPlatform.cpp' || echo './'src/Platforms/Gcc/UtestPlatform.cpp
src/Platforms/Gcc/UtestPlatform.cpp: In function ‘int IsNanImplementation(double)’:
src/Platforms/Gcc/UtestPlatform.cpp:244: warning: conversion to ‘float’ from ‘double’ may alter its value
src/Platforms/Gcc/UtestPlatform.cpp: In function ‘int IsInfImplementation(double)’:
src/Platforms/Gcc/UtestPlatform.cpp:249: warning: conversion to ‘float’ from ‘double’ may alter its value
At global scope:
cc1plus: error: unrecognized command line option "-Wno-weak-vtables"
cc1plus: error: unrecognized command line option "-Wno-exit-time-destructors"
cc1plus: error: unrecognized command line option "-Wno-global-constructors"
cc1plus: error: unrecognized command line option "-Wno-keyword-macro"
cc1plus: error: unrecognized command line option "-Wno-reserved-id-macro"
cc1plus: error: unrecognized command line option "-Wno-disabled-macro-expansion"
cc1plus: error: unrecognized command line option "-Wno-c++11-long-long"
make[1]: *** [src/Platforms/Gcc/lib_libCppUTest_a-UtestPlatform.o] Error 1
make[1]: Leaving directory `/home/anthony/downloads/tools/cpputest-3.8'
make: *** [all] Error 2

Is this still an issue ?

Actually, this is a bug in gcc. The real warning is the conversation from float to double. When there is a real warning, gcc gives these weird additional warnings. Unfortunately, I got them too. We try to get rid of them by fixing the real warning. Closing this issue.


Implement alignment (from whisperx)

Implement alignment service (from whisperX repo)
Add an extra post-process to check all words have start/end values
Add alignment as a request parameter to choose to use it or not
Add experimental function for dealing with numbers to words for alignment (not used right now). Need more work

The Coverage failed test isn't a "real" problem. It just means we need to write more testing functions. I'm on it for the utils functions. I will push them directly to main after this PR merge.


{Misc.} Remove unused dockerfile

Dockerfile.spot was added in https://github.com/Azure/azure-cli/pull/6716.
It's used by vscode-spot extension, which was archived years ago.
Dockerfile.nanoserver was added in https://github.com/Azure/azure-cli/pull/6070 to support Windows container, but it has never been released. Now, Linux containers have become the mainstream.

Misc.


Tags have duplicates, and tags are highlighted after changing the tags on an item

Issue 1
When selecting the tag in the tag manager, duplicate tags by name (and not by color) are allowed.
On press, both tags of the same name are highlighted.
Issue 2
When pressing the back button on the tag manager after, tags applied to the item are highlighted in the details view and edit view when they should not be.
Replication
From the list view activity, click on an item with tags, then click the edit button, then click the + button on the tags slide, and then click back. Clicking confirm when you are on the tags slide resets the highlighted state of the tags on the item.

Found a fix for issue 2

@waridh are you still seeing these issues?

I saw issue one earlier today. Isn't it still there?

heck

Ok PR https://github.com/CMPUT301F23T01/Sigma-Blue/pull/220 should fix this


feat: Add mini.statusline highlights

Adds highlights for mini.statusline. Let me know if other colors would would be preferable

Looks great, thank you!


Continue extension visual studio code does not initiate extension host

Before submitting your bug report

[ ] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[ ] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: ubuntu
- Continue version:0.8.60linux
- IDE version:VSCode1.95.13
- config.json:{
  "_name_or_path": "all-MiniLM-L6-v2",
  "architectures": ["BertModel"],
  "attention_probs_dropout_prob": 0.1,
  "classifier_dropout": null,
  "gradient_checkpointing": false,
  "hidden_act": "gelu",
  "hidden_dropout_prob": 0.1,
  "hidden_size": 384,
  "initializer_range": 0.02,
  "intermediate_size": 1536,
  "layer_norm_eps": 1e-12,
  "max_position_embeddings": 512,
  "model_type": "bert",
  "num_attention_heads": 12,
  "num_hidden_layers": 6,
  "pad_token_id": 0,
  "position_embedding_type": "absolute",
  "transformers_version": "4.29.2",
  "type_vocab_size": 2,
  "use_cache": true,
  "vocab_size": 30522

}

Static hostname: sakaki-Lenovo-B50-45
       Icon name: computer-laptop
         Chassis: laptop
      Machine ID: f38febb124694cb19793e48116d019bd
         Boot ID: 8e5de087fe2b4be3aabbc2942149d78a
Operating System: Ubuntu 22.04.5 LTS
          Kernel: Linux 6.8.0-49-generic
    Architecture: x86-64
 Hardware Vendor: Lenovo
  Hardware Model: Lenovo B50-45

Description
I installed the continue extension in Visual studio code, and it says an error of extension host terminated unexpectedly. I only have the continue etension installed right now, so it must be something with it. I uninstalled all other extensions. I've tried reinstalling it, reinstalling code, installing node.js, and updating my os. I also have the logs f the error:
It does not even show the continue icon in visual studio code to the left side. Thank you and hope there's a solution for this
To reproduce
Install the visual studio code continue.continue extension. Specifically Continue - Codestral, Claude, and more
v0.8.6. Then reopen code and it will say Extension host terminated unexpectedly.
Log output
RR Extension host (LocalProcess pid: 21977) terminated unexpectedly. The following extensions were running: vscode.emmet, vscode.git-base, vscode.git, vscode.github, vscode.debug-auto-launch, vscode.merge-conflict, Continue.continue, vscode.github-authentication
workbench.desktop.main.js:sourcemap:3086 Extension host terminated unexpectedly 3 times within the last 5 minutes.

Could you try installing a different version of Continue and reporting on the results?

Also, feel free to check out our Discord, it's quite active and you might be able to find a solution or someone to help you there.

I have the same issue, please help me !!!

@slip2PTY can you share your machine/platform/extension/config by chance?

Attached my machine specs and the "defaultSettings.json"

defaultSettings.json

@Patrick-Erichsen did you mention you might work on a fix for indexing crashing the extension?

Hey thanks for pulling me in here @RomneyDa , yes, this is something we are looking into this week.
For context: https://github.com/continuedev/continue/issues/940#issuecomment-2499649674

mmm... No, I never mention that, I'm new here :)


Support non-integer modulus in ModulusAnimatedNode on Android

Motivation
Animated.modulo(value, modulus) supports a non-integer modulus in the iOS and JS implementations but crashes on Android when useNativeDriver is set to true.
Test Plan
Unfortunately, I'm not fluent enough with this codebase to add a Java test for this fix - especially as I couldn't find any analogous tests to extrapolate from. However, the fix itself seems straightforward enough.
Related PRs
None needed
Release Notes
[ANDROID] [BUGFIX] [Animated] - Support non-integer modulus in .modulo()

Thanks for fixing this!
@facebook-github-bot shipit

For future reference native animated tests are located here but sadly there isn't one for the module node. It should be possible to add one that's similar to the addition / multiplication ones, if you ever feel like writing android tests 🙂.


Tool to generate chart revisions stopped working

Since last week I've noticed that the etl-chart-suggester tool to generate chart revisions stopped working (it fails both when used on local grapher and on live grapher). I haven't looked into the reasons.
For example, after having generated a mapping file for the IRENA dataset on costs (etl-match-variables -old "Renewable energy costs (IRENA, 2020)" -new "Renewable power generation costs" -f local_mapping.json), I run etl-chart-suggester -f local_mapping.json -n irena -v 2022-10-20 -d renewable_power_generation_costs and get the following error:
  0%|                                                     | 0/1 [00:00<?, ?it/s]2022-10-24 10:26.12 [error    ] Error encountered for chart 4558: min() arg is an empty sequence
100%|████████████████████████████████████████████| 1/1 [00:00<00:00, 542.32it/s]
2022-10-24 10:26.12 [error    ] Error encountered during import: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n                        GROUP BY chartId\n                        ORDER BY c DE' at line 5")
2022-10-24 10:26.12 [error    ] Rolling back changes...
2022-10-24 10:26.12 [error    ] INSERT operation into `suggested_chart_revisions` cancelled. Error: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n                        GROUP BY chartId\n                        ORDER BY c DE' at line 5")
2022-10-24 10:26.12 [info     ] 0 of 0 suggested chart revisions inserted.

2022-10-24 10:26.12 [error    ] (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n                        GROUP BY chartId\n                        ORDER BY c DE' at line 5")
(.venv) ➜  etl git:(include-additional-irena-costs) ✗ etl-chart-suggester -f /Users/prosado/Desktop/local_mapping.json -n irena -v 2022-10-20 -d renewable_power_generation_costs
  0%|                                                     | 0/1 [00:00<?, ?it/s]2022-10-24 10:26.32 [error    ] Error encountered for chart 4558: min() arg is an empty sequence
100%|████████████████████████████████████████████| 1/1 [00:00<00:00, 579.40it/s]
2022-10-24 10:26.32 [error    ] Error encountered during import: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n                        GROUP BY chartId\n                        ORDER BY c DE' at line 5")
2022-10-24 10:26.32 [error    ] Rolling back changes...
2022-10-24 10:26.32 [error    ] INSERT operation into `suggested_chart_revisions` cancelled. Error: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n                        GROUP BY chartId\n                        ORDER BY c DE' at line 5")
2022-10-24 10:26.32 [info     ] 0 of 0 suggested chart revisions inserted.

2022-10-24 10:26.32 [error    ] (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')\n

@pabloarosado this looks like your data_values was empty. I was able to replicate it with empty data_values and it worked when I upserted data.
This probably happened during the period when we switched from data_values to parquet files. Chart revisions together with compare commands would have to be updated to work with parquet files too. I haven't realised this before :(.

Hi @Marigold yes, that was the root of the issue. Now it's fixed, thanks!

For future reference, this happened to me, and it was because I was using the wrong variable IDs.
In particular, I ran etl-match-variables using my local environment but then etl-chart-suggester on live.

I guess at some point we could make the tool a bit smarter to be able to know the reason why it fails.


refactor settings, add AwardYear scope

reviewing

@jaleszek I just thought it's worth adding one test for email notifications.


assets are getting load over http

Laravel Version: 5.6
PHP Version: 7.3
Laravel-admin: 1.5

Description:
Laravel is not loading my http asset files. How can I change that admin_asset() function to load asset files over https
Steps To Reproduce:

Same problem

@tfarneau thankyou for the help

haha thanks @tfarneau ;)


Don't hardcode $BUILD_DIR in maven profile

/bin/test-compile is hardcoding $BUILD_DIR into $HOME/.profile.d/maven.sh
This patch will potspone use $HOME instead.

@nolith thanks for making this change! Don't worry about CI, I'm rerunning it now.

@jkutner how can I easily debug the CI failure in my own?
The error itself isn’t that helpful.

Reading the test suite code was enough to spot the problem. Now It's green.

@nolith thanks for cleaning that up! I'll get this released soon, but it might not be until the end of next week (we have a deployment freeze in the first part of the week).


Pickup data adventure: webpack-dev-server requires localhost

This is incredibly creative!  I played for a while, thought I was taking too long to finish the game, so I gave in a looked at the code. :)
I hit one issue trying to run inside gitpod.io.  webpack-dev-server serves up "Invalid Host header" if you access it on a host other than localhost.  gitpod wants to proxy things, so I had to add an entry to the webpack.config.js.
  devServer: {
    compress: true,
    port: 9000,
    allowedHosts: ['9000-jade-gerbil-o8z16ckn.ws-us23.gitpod.io'],
    ...
  },

The hosts that gitpod creates are random/unique so there's isn't a simple fix for it.  Maybe just a mention in the README.md?
Otherwise works great!

wdyt @krisajenkins - PR for the readme note as suggested?

Thanks @russau - I'm glad you've enjoyed it! I need to flesh out the map and puzzles a bit, but I think the game engine itself is pretty satisfying. :-)
As to the issue, I've just had a look at this gitpod.io guide and they suggest whitelisting the whole .gitpod.io domain will work, so I'm going to try that.


[build] Bump minor version when patch versions reach 100

Bump minor version when patch versions reach 100

Managing too many patch versions is hard. Bump the minor version when the patch version reaches 100.
How was this PR tested?

Tested locally:
$ ./make_tag.py
Using remote: upstream
From github.com:linkedin/venice
 * branch                main       -> FETCH_HEAD
From github.com:linkedin/venice
 * branch                main       -> FETCH_HEAD
From github.com:linkedin/venice
 * branch                main       -> FETCH_HEAD
Current branch tagBumpBuild100 is up to date.
New tag is 0.5.0. Continue? [y/N]:

Does this PR introduce any user-facing changes?

[X] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Why do this...? Version bumps should be driven by semver, not by numerology... the number of digits should not matter 🤔 ...

@FelixGV I kind of agree with you, but this seemed to be the consensus during VS3. We can discuss more and decide on this. I can revert it in the meantime

If we feel the current minor version has gone on for too long, we can certainly bump it manually, now, once. We did accumulate a lot of changes since the last minor bump. But automating it based on number of digits is not right, IMHO.


Add the ability to un-mute Sonos

The desired behaviour is that if the user press play on the Nuimo is should un-mute the song rather than pause it.
This would require the hub knowing that the Sonos is in a muted state.

Closed by PR #389


An alternative to the setMediaSources API in the Exoplayer in the new media3 API

I used to set manually different sources for different items in the Exoplayer by calling setMediaSources on the Exoplayer instance, to support offline playing since some episodes are downloaded and some are not. I had two different data source factories, one for online playing that is a simple CachedDataSource and an offline data source that is also CachedDataSource but it never expired, something like this:
 val dataSourceFactory =
            CacheDataSource
                .Factory()
                .setCache(get(named("downloads-cache")))
                .setUpstreamDataSourceFactory(get<DefaultHttpDataSource.Factory>())
                .setCacheWriteDataSinkFactory(null)
        dataSourceFactory

What is the equivalent of this in the media3 API? Also, episodes can be downloaded while they are in the queue, so potentially it would be ideal to switch data sources just before the episode begins to play

Why do you need to handle these differently? When the cache finds the data cached on the device it will deliver it from there. If the episode isn't downloaded then it delegates to the upstream data source.  Can't this be solved with the very same DataSource?
If you really need to handle this differently, then you can inject your own MediaSourceFactory. See #606 for a code sample.

episodes can be downloaded while they are in the queue, so potentially it would be ideal to
switch data sources just before the episode begins to play

I don't think there is a way to swap out the data source dynamically. Like I said above. I think you can just handle them the same and if it's not downloaded the CacheDataSource will delegate to upstream. CAn you elaborate a bit on why this isn't possible for your use case?

perfect, I'll try that


Websocket

Make websocket  implementation  for https://github.com/OpenPoGo/OpenPoGoWeb or https://github.com/OpenPoGo/OpenPoGoUI

please don't merge to master manually.


set git branch to main for new org git repo

Four days ago, 0b0dcf8c19facd5f546e606d528408fb40e7c2d update the git repository for org mode.  The new git repository does not have "master" branch, but it has "main" branch.
This PR is to set the git branch to "main" to deal with this fact.

Something went wrong.  This PR should consist of just one line. Closing it so that I can recreate it.

.

The issue is that you based the pull off our "master" branch. In the future, you can change that by clicking the "Edit" button to the right of the title of the pull request:

It will turn the base branch (in this case "raxod502:master") into a button which will allow you to change the branch.


⚠ Jenkins Plugins Artifactory has degraded performance

In dea5bbc, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1292 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3fcde76 after 12 minutes.


Angular 12 update

MAJOR UPDATE: MIGRATE FROM ANGULAR 8 to ANGULAR 12

Updated browserslist format: deleted browerslist and replaced with browserslistrc
Added /.angular to gitignore
Various changes to angular.json
Updated karma config
package json changes:

add pre/post install scripts to deal with resolutions of vulnerable packages
bumped Angular version from 8 to 12
bumped various packages including:

Nebular to version 8
@auth0/angular-jwt to version 5
ng-bootstrap to version 10
ngx-charts to version 20
bootstrap to version 5
Removed various unused packages

Changed import syntax for Material throughout
Fixed various lint issues
Update tests from Testbed.get to Testbed.inject syntax

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Allan MacLean seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Research how layers will work

After trying to refactor into a usable and likely more-correct system from the current nonsense that this sketch is using, I quickly realized that I've got the hardest part of the problem ahead of me:

https://github.com/react-bootstrap/react-bootstrap/issues/27
https://github.com/facebook/react/issues/379
https://github.com/pieterv/react-layers
http://khan.github.io/react-components/#layered-component

@stresslimit So the current takeaways:

Indeed the popover is not supposed to be rendered around its target as-far-as the DOM tree is concerned
Another concept is required altogether which goes along the lines of "layers" but has different implementations.

Example from React Bootstrap: http://react-bootstrap.github.io/components.html#popovers
    <OverlayTrigger trigger='click' placement='left' overlay={<Popover title='Popover left'>Holy guacamole! Check this info.</Popover>}>
      <Button bsStyle='default'>Holy guacamole!</Button>
    </OverlayTrigger>

The good news is I am not crazy; all my concerns and notes about the problem I'm seeing unfold/thinking-through are leading to the right place. For example I basically came to this conclusion on the way home repeated by someone else way before:
https://github.com/react-bootstrap/react-bootstrap/issues/27#issuecomment-34431648

Attach the modal pane to the document body, and wire it up to the trigger via refs and callbacks. The communication chain in React will be fiddly, but this will be an implementation-internal issue. I think this will involve some black magic, but will be a more solid implementation.

To paraphrase, have popover completely pass-through its children in its render and then render its own popover component to the body.
It seems that all discussion culminate in the OverlayTrigger component in Bootstrap React and the dedicated repo linked above pieterv/react-layers.
Lastly, in case readers do not read the links, keep in mind these links include discussions and references from the core community so this issue is taking from "trusted" minds in this domain.

Hmpf. Definitely react doesn't solve all problems :)  And no you're not crazy thankfully. Let me know how it goes!

Ok the three implementations that seem to "matter" are:

https://github.com/Khan/react-components/blob/master/js/layered-component-mixin.jsx
https://github.com/react-bootstrap/react-bootstrap/blob/master/src/OverlayTrigger.js
https://github.com/pieterv/react-layers

Observations:

There seems to generally be an assortment of mixins and components that use those mixins.
The simplest source code seems to be Khan, then BS, then react-layers; correspondingly their features seem to increase as well so its not, so far, "obviously", about which is best.
No other serious code references beyond these that I am aware of.
React BS We cannot use its code since its coupled to its framework; we can just be inspired by it and port it. It has a lot of opinions about fade-in/out etc. that deviates heavily from our specs.
react-layers offers three abstractions: layers, portals, containers. The source code quality is so-so; there are no docs and the project doesn't appear to be actively developed anymore so other than inspiration I cannot see how this is a directly usable library for us.
Kahn Academy's implementation is very small, to the point that we can understand the concept and rewrite it to suit our needs better.
I don't know exactly how long it would take but for me to research a solution up to the point these people have would not be fast, so I should learn from them. This is good, its something, but I am pretty sure that using their source code is not going to be feasible. So we just have to learn this stuff and write our own.

Thanks to conversation on IRC I got some nice help and a tip from @jquense (thanks!) leading to his repo https://github.com/jquense/react-layer

The conclusion is the following:

Create a React Mixin that hooks into component-lifecycle hooks. It relies on an implementation of renderLayers which acts like a render except that the returned value is sent to a different DOM point completely unrelated to the render tree of render. So in sum the render and renderLayers share an owner that provides a "common control context" but do not share in any way their render trees.
We an integration challenge to integrate renderLayer ref into owner refs. A workaround for some use-cases is to expose the return value of React.render on the owner instance. This allows the owner to get a hold of the layer DOM Node via React.findDOMNode.
We have a design decision to make regarding how many layers a component may have. Currently only one but it seems that the facebook team internally used a system wherein renderLayer could return multiple layers. They modelled this by allowed renderLayer to return an object of views rather than a single view. So far our system is simple, making renderLayer act like render.
We have a design decision to make about what life-cycle methods if any layer will get. For instance should there be layerWillUnmount and layerDidMount etc.?

Overall we have a system that just-works, leaving various design tradeoffs un-studied and refs unresolved. Our shortcuts unblock us in the small but a wise product development cycle would provide time to fully design test and extract this layer component into its own module. For that matter it could make a good presentation topic at the NYC React Meetup.

I've spent some sensible time to research on this. As a result I end up with the library which permanently solved this problem for me. Check out, please, would be good to know the impressions and everything https://github.com/AlexeyFrolov/react-layer-stack


Can't use the gradle plugin in an Android project

The instructions suggest requiring the java plugin as a dependency. The plugin seems to do that itself, too.
A gradle project can't use both the java plugin and the android plugin.
I do not need to run JTE on Android. I need to run it during my Android project's build. I want to precompile my templates,
and use them in an annotation processor that also runs during the build, not on the target phone.
Any ideas?

What are you trying to do with jte then? Generate code within the build process?

Yeah. The build process runs an annotation processor, which then uses JTE to generate code from those annotations. To be more specific, this is https://github.com/villevoutilainen/Jenny

I don't know about Android builds.
I'm imagining your annotation processor depends on JTE plugin at build time. Then the main code depends on the annotation processor at its build time. So the main code build doesn't depend on the JTE plugin.
Does your annotation processor need to be built with the Android toolchain?


Fix html build

Was looking for a way to build html sildeshows in Haskell --- this fix makes the compile work for html.

Nice thanks! Can you tell me what exactly happens now that didn't earlier? (I thought it has to do with slide shows but there don't appear to be any changes beyond the toc.hs file? Did you forget to add some other files to the pr?)

On Aug 20, 2015, at 11:35 PM, Matt Chan<EMAIL_ADDRESS>wrote:
Was looking for a way to build html sildeshows in Haskell --- this fix makes the compile work for html.
You can view, comment on, or merge this pull request online at:
https://github.com/ucsd-progsys/lh-workshop/pull/1
Commit Summary
get the thing building html
File Changes
M assets/filters/Toc.hs (9)
Patch Links:
https://github.com/ucsd-progsys/lh-workshop/pull/1.patch
https://github.com/ucsd-progsys/lh-workshop/pull/1.diff
—
Reply to this email directly or view it on GitHub.

It fixes a type error when you run make html. The slideshows (at least for 01) are still messed up.

Thanks @themattchan!


Auth.java:34-36: Implement a new auth named 'Token' that...

The puzzle 99-9320c256 from #99 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/ab0d78ec783de53306d63a66dd459598d58dc8c4/src/main/java/com/amihaiemil/docker/Auth.java#L34-L36
The puzzle was created by rultor on 23-May-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@amihaiemil please pay attention to this new issue.

@amihaiemil/z please, pay attention to this issue

Job #171 is now in scope, role is DEV

The job #171 assigned to @llorllale/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@llorllale/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait because I submitted PR #188 for this

@0crat wait because I submitted PR #188 for this (here)

@llorllale The impediment for #171 was registered successfully by @llorllale/z

The puzzle 99-9320c256 has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil


Add test_helpers and report_id keys to report header

This shall be merged when this is merged: https://github.com/TheTorProject/ooni-probe/pull/381

I am going to proceed in merging this.


run management container as non-root (#19)

Fixes #19 (wis2-gdc-management container now runs as user wis2-gdc).
@nbuttdwd can you test?

i tried it but the cronjob isn't working ...
wis2box@158bf5293a20:/$ service cron status

cron is not running
the cronjob's content: /usr/local/bin/wis2-gdc archive /data/wis2-gdc-archive.zip > /home/wis2box/logs/create_metadata_zip_cron.log 2>&1
cat /home/wis2box/logs/create_metadata_zip_cron.log
Archiving metadata from GDC https://wis2.dwd.de/gdc

it's not executing the command.

Thanks for the feedback @nbuttdwd.  I've pushed a fix which adds the wis2-gdc user to /etc/sudoers and starts cron via sudo accrodingly.  Let me know if that works, thanks.

Hi Tom, it works! Thank you.


feat: add watch functionality to dev command

Files that we watch by default:

oranda.json or custom config path
Readme file (either default or overriden in config)
mdbook directory (currently not recursively as the HTML output gets put in that directory too)
Additional pages as specified in the config
Project manifest files as found by axoproject
User-supplied paths in the -i, --include-paths CLI argument

Closes #246

@ashleygwilliams that does sound like a good idea, but if we do that, we should at the same time think about how we do excludes, since notify doesn't really have a mechanism for this. the easiest thing that comes to mind is to walkdir down any directories passed in, filter them against an exclude list, and then pass the result to the file watcher... we want this if, for example, people have a mdbook structure that includes subfolders, which don't currently get watched by our watcher here


Add user as collaborator on CreateOrganization

Scope of changes
This fixes a bug where the user tries to switch organizations and gets a 401 error due to not being a collaborator on the organization. The fix is to add them as a collaborator when they create the organization.
SC-11488
Type of change

[ ] bug fix
[ ] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report
Base: 44.45% // Head: 44.48% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (9e90718) compared to base (7105617).
 Consider uploading reports for the commit 03f8d73 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #915      +/-   ##
==========================================
+ Coverage   44.45%   44.48%   +0.02%
==========================================
  Files         679      678       -1
  Lines       24457    24452       -5
  Branches     1541     1531      -10
==========================================
+ Hits        10872    10877       +5
+ Misses      12196    12185      -11
- Partials     1389     1390       +1

Flag
Coverage Δ

unittests
44.48% <ø> (+0.02%)
:arrow_up:

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

Impacted Files
Coverage Δ

....com/trisacrypto/directory/pkg/bff/organization.go
63.80% <0.00%> (-0.20%)
:arrow_down:

...mponents/CertificateDetails/CertificateDetails.tsx
59.09% <0.00%> (ø)

web/gds-user-ui/src/hooks/useGetCertificates.ts

...-ui/src/components/CertificateManagement/index.tsx
16.66% <0.00%> (+6.14%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Allow passing typed search attributes to child workflow

With the Java SDK one can do:
  val sa = Workflow.getTypedSearchAttributes
  val optionsBuilder = ChildWorkflowOptions.newBuilder().setTypedSearchAttributes(sa)

With the current zio-temporal API, this is not possible, or at least I couldn't find a way to do that

TODO: Add ZChildWorkflowStubBuilder.withSearchAttributes method overload that accepts plain SearchAttributes (not a Map later encoded by zio-temporal).
Other methods may require such an overload as well

I think I can work on it.

@grouzen Cool, I appreciate your help! Here are a few links to simplify your work:

I'd use ZSearchAttributes wrapper instead of the java class.
To make the interop better, we might add ZSearchAttributes.fromJava method that allows passing search attributes constructed via Java SDK directly (in case teams have mixed Java/Scala code).
It would be great to add this overload to other places as well (for instance, here)

Great! Thanks for the clear instructions!

@oridag, how urgently do you need this to be released? The next planned 0.5.0 release is pretty small and contains a single little breaking change related to protobuf payload converter (as we dropped an old java dependency).
My upcoming work on #121 breaks a lot to support more Java SDK functionality. Therefore, I wonder what's the right time to release this feature

I don't mind waiting a week or two, but if it's longer than that I'd appreciate a release, thanks!