Where did the lib\wp8 directory go?

After updating to version 4.0.3 I have had alot of problems with the WinPhone Project (Silverlight 8.1).
I noticed that the WP8 directory existing in v 3.4.2 no longer exists... Instead I have WPA81 and uap10.0 directories.
I guess I should use the dll:s from WPA81? But they also gives me odd exceptions, like "The application called an interface that was marshalled for a different thread..."
Or am I missing something?

My suggestion is to go back to 3.4.2 for now.  I didn't want to carry WP81 forward because of the new functionality that was added.  In the near future, only UAP will matter anyhow.

Ok, then I know... Will revert to 3.4.2 for now.
Thank you


Special Character in Master Item Name

I used to write object names with an underscore (_) to separate words, when I have two button with these names:
tgtBtnYTD and tgtBtnYTD_PY, the configuration pane shows two: tgtBtnYTD; the underscore character is valid as a variable name, so it should not generate this problem.

Thank you for bringing this to my attention. I have changed this behavior. Now I use '~' as the sign to concatenate information within the model.  Please use version 1.0.1.
You also need to pick your Master Items again in already build grouped-containers to apply the new values.
Here you can find the new release: version1.0.1

Thank you for bringing this to my attention. I have changed this behavior. Now I use '~' as the sign to concatenate information within the model.  Please use version 1.0.1.
You also need to pick your Master Items again in already build grouped-containers to apply the new values.
Here you can find the new release: version1.0.1


Add ability to add invalid fields after a warning

Relating to the last issue #114, we also can conceive the case where a user wants to input for example two phone numbers separated by text (e.g. home 92332331 handphone 5903821). Therefore again we would like to allow the user to input this, but with a warning to inform them that this is not the typical format.

closed by #127.


Customer stories: Detail which FlowFuse products are being used

Description
We currently have a right column with details about each story, but we don't highlight which of our products are being used in each case.
We could add a new section that includes the icon(s) of each product used:

Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Updated icons


Variants choices sometimes do not output correctly in the variant inventory table

Take a look at the screenshot above. I added a variant for Size and included choices for: Small, Medium, Large.
I entered into a scenario where the Medium choice output twice in Variant inventory table, while the Large choice didn't output at all.
This seems a bit difficult to reproduce; most of the time it works fine. However, there does seem to be a bug hidden in here somewhere that we'll need to stomp out. I'll continue poking around at it to see if I can figure out the exact steps to recreate it.

We just received another report of this bug.

In all of my products that I have listed so far I have color variations, Black, Green, Red etc. With the Product SKU’s I use the Product SKU, ie; Bggy-Hrm- Followed by the color, (abbreviated).
Example;
Red Baggy Pants                 Bggy-Hrm-Red
Black Baggy Pants              Bggy-Hrm-Blk
White Baggy Pants              Bggy-Hrm-Whi
After I save the Sku’s seem to become mismatched, example:
Red Baggy Pants                 Bggy-Hrm-Blk
Black Baggy Pants              Bggy-Hrm-Whi
White Baggy Pants              Bggy-Hrm-Red
I didn’t really notice until I received an order and the product picture was for one color yet the SKU showed a different color. When I checked my listings at the time they were all mismatched on the SKU.
I could have been a mistake my end, although I can’t see how?
I hope this helps.
My open Bazaar2 page is Siam Secrets.
ob://QmVB42zLCzJqHznT5ostfsG9xuVAXzPWRioijogCXzAkxQ/store

@rmisio can you look into this? It seems to be rare, but still happens in the current code.

This is pretty old, can we take another look at it, and either close it or triage what needs to be done to fix it?


Inefficiencies

In DIscordMessageSender, a new thread is used to send a webhook, this is highly inefficient, spinning up a new thread on every message would be highly resource intensive and would be slower than actually just sending a webhook without it.
The webhook should not take too much time to send, and if it does a threadpool should be initialised upon the plugin being loaded, this will therefore means threads aren't created each message and instead reused.

Oh yeah I know what you mean. I had some weird exceptions when not doing this with bukkit. I will take a lookt at.

Alright I fixed it for you


Bug Import "dotenv" could not be resolved - Pylance

I installed the python-dotenv and the python-decouple packages to help load my env files. But Pylance can seem to resolve them at all

Thanks for the report, unfortunately, this is not enough data for us to debug.
If this continues to be a problem, would you please reopen the issue and add the information in the report as described in the troubleshooting guide?


It's actually totally impossible to install/use this software with existing documentation

Not sure how projects like these land VC, but this is frustrating. Tendermint based chains are really, really janky and hard to use. Can you please update your documentation so that people don't waste hours or days on your broken, poorly programmed software?

After endless struggle I managed to get this garbage going.


Use Ogmios Tx Monitor to improve error recovery logic for SubmitTx

Closes https://github.com/Plutonomicon/cardano-transaction-lib/issues/398
Pre-review checklist

[x] All code has been formatted using our config (make format)
[x] The integration and unit tests have been run (npm run test) locally
[x] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@errfrom Is it possible to write a test for this, one that kills the websocket?

@jy14898 Currently it's possible to test these changes by uncommenting the following lines:
https://github.com/Plutonomicon/cardano-transaction-lib/blob/031ce4a201327f169cabb7630eb028a9c2a8f257/src/QueryM.purs#L1089-L1091
However, I have not yet figured out how to approach writing tests for this

@klntsky It should work just fine, because the _onWsConnect handler will only be invoked once in case of a failure, so the code responsible for resending SubmitTx requests will work with a single mempool snapshot

@klntsky If you are referring to the case where multiple users would connect to Ogmios and use its Tx Monitor protocol, so for each user there will be a separate internal state maintained by Ogmios (AFAIK)

so for each user there will be a separate internal state maintained by Ogmios (AFAIK)

What is the user identity Ogmios uses to tie the state? IP?

@klntsky IP + port number. For each new connection, Ogmios initializes all services and handles the state separately. This is what I understood from reading the source code.
https://github.com/CardanoSolutions/ogmios/blob/a33342ba76b8424b7c2ff67f10e9a09bf81a9ebe/server/src/Ogmios/App/Server/WebSocket.hs#L154-L164

What can happen at worst in case there are multiple browser tabs (that share the state)?

@klntsky In the worst case scenario (when two clients attempt to resend their pending submit requests), one of them may potentially be blocked on AwaitAcquire request until a new mempool snapshot is available, but the odds of this happening are quite low.


Don't bail out if ssh-keygen -R fails

If ~/.ssh doesn't exist, like in the TeamCity nightly test, ssh-keygen -R will fail with "mkstemp: no such directory". This doesn't seem worth
bailing out for. Instead, just print the warning.

This change is

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

Comments from Reviewable


Change URL pattern to allow ports != 80

The port specification was at the wrong place in the URL-pattern.

Thanks for catching this! One question: Is it desired to be able to do webroot verification on a port other than 80?
https://certbot.eff.org/faq/#can-i-issue-a-certificate-if-my-webserver-doesn-t-listen-on-port-80
Seems to suggest this is not possible. We personally always use port 80 for verification, so I haven't run into this use case. Could you clarify @elektro-wolle ?

It's no longer needed for me, I've used the path based haproxy rule on /.well-known on Port 80 instead of using a different port. (Had a service directly running on port 80 without a haproxy in front of it. Therefore i thought having to use a different port, instead of changing the entry-point to haproxy)


incorrect host name

Where is the problem?
https://docs.konghq.com/gateway/2.8.x/get-started/comprehensive/load-balancing/
What happened?
the mentioned host in step number 12:

Change the Host field to upstream, then click Update.
should be example_upstream, instead of upstream only.

What did you expect to happen?
load-balancer should've worked, but since the hostname was incorrect it wasn't able to local the required host/upstream
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

Nice catch, this was an oversight when we changed the name. Fixing now.


CONTRIBUTING.md: Add section on becoming a maintainer

I got some feedback after a review on a PR (https://github.com/NixOS/nixpkgs/pull/161404#issuecomment-1117081506) that it's not very clear that adding yourself as a maintainer in maintainers.nix should be a separate commit.
It might be nice to add a small section to CONTRIBUTING explaining the (simple) process, or maybe linking from there to the manual.

The Nixpkgs Manual has that section.

Nevertheless the section in the manual is helpful, imo having a comment in CONTRIBUTING seems like a good idea.
Additionally I would suggest that both sources should mention that as per comment it might be preferrable to have the commits ordered like this:
maintainers: add <new-maintainer>
<package-name>: <commit message>

this also helps to easier squash required changes to the actual commit.

Fixed by #248013 and #250344


Warning on MacOS X

Hi,

I saw you fixed most of warnings with v0.11, great job (and fixed the assert
with uint64 too).

I still have one warning when compiling on g++ on MacOS X 10.8.3 with rapidjson
v0.11 :
rapidjson-0.11/rapidjson/include/rapidjson/reader.h:422: warning: comparison is
always true due to limited range of data type

BTW, thanks for this great library :)

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 5 Apr 2013 at 8:59

I have this problem as well:

reader.h:422:31: warning: comparison of constant 256 with expression
      of type 'Ch' (aka 'char') is always true [-Wtautological-constant-out-of-range-compare]
                                if ((sizeof(Ch) == 1 || e < 256) && escape[(unsigned char)e])

Original comment by<EMAIL_ADDRESS>on 8 Dec 2013 at 12:29

I met it too.

I think it is harmless. Author doesn't want to use template specialization
here:)

Original comment by<EMAIL_ADDRESS>on 31 Mar 2014 at 6:34

So is the svn repo obsolete now in favor of:
https://github.com/miloyip/rapidjson ?

If so, perhaps update the "source" page here. Thanks!!

Original comment by<EMAIL_ADDRESS>on 30 Jun 2014 at 4:12

I am getting this same behavior. Is there something that could be missing from
the install instructions?

Original comment by<EMAIL_ADDRESS>on 19 Jan 2014 at 4:56

We should have fixed quite a few warnings.
Currently it is tested with gcc/clang, as shown in
https://travis-ci.org/miloyip/rapidjson
We will continue to fix other warnings.

Original comment by<EMAIL_ADDRESS>on 30 Jun 2014 at 10:54

Changed state: Fixed

Original comment by<EMAIL_ADDRESS>on 20 Jun 2014 at 11:58

Changed state: Accepted

Yes. There is a big billboard in the front page :)

https://code.google.com/p/rapidjson/

Original comment by<EMAIL_ADDRESS>on 30 Jun 2014 at 4:41

Haha, that's funny. Yeah I never went to that page.

FYI I followed the link for this issue, then clicked on the "Source" tab, which
still references the svn repo.

Thanks for all your great work!!

Original comment by<EMAIL_ADDRESS>on 30 Jun 2014 at 4:44


New Grid

SMDBGrid component. [Delphi] The successor TDBGrid with the extended features. Is able to display multiline wordwrap column titles, checkboxs for boolean fields, a convenient select of records from the keyboard and mouse via checkboxs, extanded Indicator column, fixing of columns, an opportunity to exclude insert and delete of records in the DBGrid, own standard PopupMenu, save/restore of a column states, processing of additional events etc. Multilanguage resources.

Thanks! Added, sorry for delay


Add data-skip-router

Allow skipping the router by specifying a special
data-skip-router attribute on a local link
In some cases it's useful to have an a element that doesn't go through the router, because it is handled in a different or special way.
In my use case normal routing calls the server (via ajax) to fetch data for the next page, but some pages are closely related (for example multiple tabs), so when a user enters one tab I load all data for all tabs. Changing tabs now no longer has to go through the normal router middleware to fetch all data.
This change allows me to mark the tab links as a elements that skip the router, AND handle them separately with onClick handler.

Im not sure that this change is required to get behavior like you describe.  I think if you bind directly to the links for your onClick and stopPropagation on it, then Nighthawk should never see the click, and then not do anything.  If you can show that this is not the case let me know and we can explore this further.

Hi Wes,
The following test fails. Is that what you had in mind? something else?
	it('should not intercept click if stopPropagation was called', function () {
		router._processRequest = function () {
			throw new Error('Should not have been called!!');
		};

		evt.target = document.createElement('a');
		evt.target.setAttribute('href', '/foo');
		evt.target.addEventListener('click', function (e) {
			e.stopPropagation();
		});

		router.onClick(evt, evt.target);
	});

Not exactly, because the stopPropagation is a browser event thing, this test does not behave the same way.  The tests for this package do not try to actually mimic browser events, they just are testing internal behavior.
An actual test would be to create a test page with code like that above, and try clicking the link.  The only other way would be to use a testing tool like selenium to actually imitate a real click event.  But personally I try to avoid those kind of tests because they are hard to write and maintain.

Thanks! After double checking using stopPropagation did solve my issue and prevented routing. No need for this PR


Arduino external sync acting as Master produces unexpected capture behaviour with two devices in Subordinate mode

DESCRIBE THE BUG
We are trying to use an Arduino Uno external sync as a Master device, controlling the captures of two Subordinate AK sensors with the latest firmware. We have programmed the Arduino to emit interlaced, periodic pulses, emulating Sync Out capture pulses, out of two separate pins, and have wired these pins to aux cables plugged into the AK devices via Sync In.
When testing to see if the captures were behaving as expected, we also programmed the Arduino to control two LEDs, such that one LED was always high at any time. The LED is set to turn on at the rising edge of the capture pulse of the first Kinect, and turn off at the rising edge of the subsequent capture pulse controlling the other Kinect, and vice versa.
The expected behaviour was that each AK would be triggered to capture when it received its pulse from the Sync In port, and therefore would also capture its respective LED being lit up. In turn, when the second AK captures, in the delay period of the first AK, the second LED was expected to be lit up. In reality, it appeared that the same LED was bright for both AK captures, indicating that they are happening either at the same time or in quick succession, which contradicts the input signal being fed to it. In addition, when we unplugged the Sync In from one AK and plugged that same signal into the other AK, instead of showing the same LED being lit up (as would be expected), the opposite LED was shown to be lit up. We also noticed that when holding both Kinects, and moving them with the LEDs still in view, there was occasional flickering between which LED was lit up and which was dim. This indicates that the capture rate is unstable and that the AK is not listening to the input signal when timing its captures. This shows unexpected timing behaviour of the AK when used with an external trigger and no Master device.
As a further experiment, we tried assigning one AK as the Master and one as Subordinate, with a normal aux cable between their respective Sync Out and Sync In, and programmed the k4aviewer 'delay off master' to the time period between 60fps captures. This also resulted in similar behaviour - both AKs only captured the same LED as high.
TO REPRODUCE

Set up PC and AKs according to Desktop information below
Program an Arduino to produce interlaced, periodic, active-high pulses of 5V, 10us time periods, such that the time between pulses is around 16.7ms (1/60Hz). We tested with an oscilloscope to ensure accurate timing and minimal cross-channel ringing

Arduino code:
`// 60 FPS interlaced
void loop() {
PORTD |= 1<<LED_1; // LED_1 high
PORTD |= 1<<PIN_1; // PIN_1 high (the delay introduced by this instruction is only one cycle for a 8 MHz clock in the UNO)
_delay_us(10); // capture pulse must be 8-14us
PORTD &= ~(1<<PIN_1); // PIN_1 low
_delay_us(16000); // total capture period calculated as 1/60Hz
_delay_us(652);
PORTD &= ~(1<<LED_1); // LED_1 low
PORTB |= 1<<LED_2; // LED_2 high
PORTB |= 1<<PIN_2; // PIN_2 high
_delay_us(10);
PORTB &= ~(1<<PIN_2); // PIN_2 low
_delay_us(16000);
//_delay_us(652);
_delay_us(465); // lower delay value here to account for delay in system
PORTB &= ~(1<<LED_2); // LED_2 low
}`
Oscilloscope reading (input capture signals to AKs):

Cut an aux cable in half, separate out the wires. Using a breadboard, connect the signal wires of the aux to the signals from the Arduino, and the ground wires of the aux to the grounds of the Arduino. For our setup, each piece of aux cable was just under 1m long.

Place two red LEDs on the breadboard, in series with resistors, and connect them to the LED-programmed pins of the Arduino respectively

Physical circuit:
(Note: the ceramic resistors were used as wires for obtaining oscilloscope readings)

Open two instances of k4aviewer on the PC connected to the AKs, and place the AKs such that both LEDs are in full view for both sensors. Configure both viewers to 1536p, NFOV Unbinned, 500us manual exposure, 30fps, imu OFF, external sync = Subordinate. The viewers will wait for the master signal from the Arduino to start
Plug in the Arduino power, and assess the captures of each AK and its respective LED

k4aviewer output stream:
The images below are of LED_1 being covered, then LED_2 being covered. They show that both AKs are seeing the same LED being lit up, despite receiving different capture signals.

Pick up and move the AK devices, keeping the LEDs in frame. We observed a change in which LED was lit up when moving the AKs. Also remove the Sync In cables from each AK and switch them around. Both AKs will have the same LED lit up when both Sync Ins are swapped, showing that the Sync In cables are giving the opposite outcome for each AK, with the same signal.

EXPECTED BEHAVIOUR

The AKs capture at 30fps each, but the interlacing of the captures results in an effective 60fps capture rate of the system
Each AK only sees its corresponding LED as high, as both the AK capture and the LED brightness are controlled by the same signal
Unplugging the Sync In cable from AK1 into AK2 should result in AK2 seeing the same LED as high as AK1. This is because its capture period should mimic the signal coming into its Sync In port
Moving the AK devices does not affect which LED is lit up and which is dim, as the capture rate is set by Sync In

DESKTOP INFORMATION

Ubuntu: 18.04
SDK Version: 1.4.1
Firmware version: AzureKinectDK_Fw_1.6.110079014.bin

Hey that's so similar to my problem but I couldn't fix it either :( good luck!

Hey that's so similar to my problem but I couldn't fix it either :( good luck!

Could you give some more details, please?


automate pypi releases

This will do the following whenever a release is created:

update the project version to match the release tag
create a release and upload to pypi

PYPI_API_TOKEN has been added to the repository secrets

I think it would be good to set this up as a trusted publisher deployment. More like we do with PyBaMM and pybamm-eis
https://github.com/pybamm-team/pybamm-eis/blob/main/.github/workflows/release.yml


Error when installing plugin from folder with whitespace

I tried to install a plugin/theme from a folder that contains a whitespace but sadly fisherman exits with an error.
Folder:

/Users/Test/Downloads/theme master/

Output: fisher install .

Installing >> master theme master
fatal: repository '/Users/Test/Downloads/theme' does not exist
No plugins were installed.

I can reproduce this.

@iONsky Thanks. The validation should catch this is an invalid name and provide a better error message.

@bucaran You mean the name or the folder can't contain any whitespace?

@iONsky At the moment, it can't... but if we can figure out a way to extract the name of the plugin from the directory's name, then sure.
In this case, what do you want to use as the name for your plugin?

In my opinion, while fisherman can be accommodating with folder names like these, I think the appropriate action here is to only allow certain folder naming conventions and provide readable error messages if otherwise. Having a convention for things like these will ultimate minimize work done to provide support, so we can focus on the key features of fisherman.

Already fixed locally, ships in 1.0. Basically, we detect the invalid path and print an error message.

I noticed something else...the problem occurs even if the path contains a whitespace.
For example /Users/Test/Development Tools/theme/

@iONsky Nice catch. Ah, this is tricky. It's fixed now and ships with 1.0, but if please keep reporting problems if you keep finding them.

@bucaran Awesome :+1:


Error notice in new 1.20.1 release

Output
mark@osx:~$ phpbrew install 5.6.21 +default +mysql +gettext=/usr/local/opt/gettext +iconv +ftp +exif +dba +openssl +soap +apxs2=/usr/local/bin/apxs
===> phpbrew will now build 5.6.21
===> Loading and resolving variants...
sh: ../Cellar/mysql/5.7.12/bin/mysql_config: No such file or directory
PHP Notice:  Undefined offset: 0 in phar:///usr/local/Cellar/phpbrew/1.21.1/libexec/phpbrew/src/PhpBrew/VariantBuilder.php on line 392

Expected Result
No error messages

Command
phpbrew install 5.6.21 +default +mysql +gettext=/usr/local/opt/gettext +iconv +ftp +exif +dba +openssl +soap +apxs2=/usr/local/bin/apxs

Platform
OSX 10.11.4
Running PHP:
PHP 5.6.20

Installing PHP:
PHP 5.6.21

It seems Utils::findBin('mysql_config') get the symbolic information through readlink, but readlink return a related path instead of absolute path.
mysql_config@ -> ../Cellar/mysql/5.7.12/bin/mysql_config

Maybe you should consider use the following script to get real path to avoid this kind of errors.
realpath(readlink('/usr/local/bin/mysql_config'))

This will return /usr/local/Cellar/mysql/5.7.12/bin/mysql_config, instead of ../Cellar/mysql/5.7.12/bin/mysql_config

thanks for your report, we will fix this bug in next release.

Yes, thanks @markwu for reporting this 👍

@jhdxr could you run make to update the phar on github? thanks!  (I am outside right now, don't have a laptop to update the release)

The composer.lock file was updated for release, so you could do composer
install and we could have the same dependencies
xiami<EMAIL_ADDRESS>於 2016年4月30日 星期六寫道：

thanks for your report, we will fix this bug in next release.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/phpbrew/phpbrew/issues/716#issuecomment-215973782

@c9s  I have updated the phar file without bumping version, is it ok?

yes, it should be fine. we can bump the version later.

thank you all :)

I confirmed the bug fixed in new released phpbrew.phar, thanks for your hard works.


put camera/GPU IRQs on core 3

Considering there's nothing pinned to core 3 and it's always online, this should be a strict upgrade from core 0. The USB interrupts, WLAN, cdsp are still on core 0.
I'm not sure you want this on the same core as camerad due to priority inversion issues.

Do these showup before starting camerad? Some of the camera ones don't IIRC. Also did a few more like the encoders in https://github.com/commaai/openpilot/pull/24088.

Closed for #24088


OS compatibility/portability information

Would it be possible to implement a info section for OS compatibility/portability for packages on the NuGet gallery website.
This information would be provided by community feedback or the original package developer upon submission. There is some way of scanning libraries for calls to platform specific APIs also.
For example one library may reference win32 API specific stuff which doesn't work on Linux even though the library maybe targeting a cross-platform framework or so.
Currently the consumers have to trial and error test or we rely on documentation from the developer on their Github.
Just an idea. I will try find more examples of this.
Related MS docs:
https://docs.microsoft.com/en-us/dotnet/standard/native-interop/
https://docs.microsoft.com/en-us/uwp/win32-and-com/win32-and-com-for-uwp-apps
Example of platform specific libraries that may catch people out:
https://www.hanselman.com/blog/HowDoYouUseSystemDrawingInNETCore.aspx
Alternative FOSS cross platform library:
https://github.com/SixLabors/ImageSharp

@anangaur

Perhaps this is like Microsoft.DotNet.Analyzers.Compatibility which seems to be a new preview package. Can't we apply something like this to automatically perform static analysis on nuget packages and display this information on the nuget package information when browsing.

With .NET 5, I am assuming packages will be able to state the supported platform in a much more concrete way than today (RIDs). Tagging @terrajobst for his thoughts on this.

https://docs.microsoft.com/en-us/dotnet/standard/analyzers/portability-analyzer
https://docs.microsoft.com/en-us/dotnet/standard/analyzers/platform-compat-analyzer
https://docs.microsoft.com/en-us/dotnet/standard/analyzers/api-analyzer
A few more relevant links.
.NET 5 may help this but there is lots before we can get there. Many packages will have hidden platform specific win32 API calls or other non portable code. I guess UWP falls into this too.
Or as you state we rely on some concrete way for devs to state it's portable. Although thats subject to their interpretation of portable. I guess VS tools mayble able to do static analysis if MS implemented something along those lines.
Alternatively there is a new class library type which forces fully portable APIs within .NET 5.

For example today I need to use: https://www.nuget.org/packages/System.IO.Ports/
I have no way of knowing if this works on Linux without trying it myself (which becomes tedious) or Googling it. I did the latter and found: https://www.hackster.io/sxwei123/serial-communication-with-net-core-3-0-on-rpi-linux-0f2ed4
This doesn't seem like a great way of doing things. Googling each NuGet package. So ideally either NuGet gallery can implement the static analysis or MS provide some method for the package to describe it's platform portability/support. Ideally with some static analysis on client side VS to ensure that's correct.

/cc: @JonDouglas


Use better column names in join table for implicit m-n relations

Problem
I have the following schema which includes an implicit many-to-many relation:
model Permission {
  id    Int    @id @default(autoincrement())
  name  String
  users User[]
}

model User {
  id             Int          @id @default(autoincrement())
  userName       String       @unique
  hashedPassword String
  permissions    Permission[]
}

Prisma creates a join table with columns called A and B. When I inspect this database in some database tool, it is not clear which column represents which table in this relation.
Suggested solution
Prisma should use a combination of a table name and an ID column name when naming columns of the join table. In this case, these columns would be called permission_id and user_id.
Alternatives
Or Prisma could allow us to specify column names in @relation attribute:
model Permission {
  id    Int    @id @default(autoincrement())
  name  String
  users User[] @relation(map: ["permission_id"])
}

model User {
  id             Int          @id @default(autoincrement())
  userName       String       @unique
  hashedPassword String
  permissions    Permission[] @relation(map: ["user_id"])
}

Additional context
I use Prisma 3.4.1 with SQLite database.
Related issues:

https://github.com/prisma/prisma/issues/3381
https://github.com/prisma/prisma/issues/5234
https://github.com/prisma/prisma/issues/5369

Please take a look at this feature request since this is basic Object Relational functionality. The default column name convention Prisma is proposing for these columns is not standard and should be customizable to support any other convention.  Declaring models for these middle tables is overkilling and impacts not only all related query DSL but also domain logic code when retrieving values from objects returned by queries.
It's not the end of the world if it's not implemented but this as well as a few other limitations observed during only 3 weeks of exploration is making me think that we are still 1 or 2 years away from considering this tool as an all-terrain ORM
Hoping this will not be so hard to achieve.
Congratulations to all of you that have put some effort into building this toolset that looks promising.

I second this approach. Prisma does not always take the most efficient approach at complex queries, so queryRaw comes in handy pretty frequently. Being forced to use this A and B column structure is confusing when writing raw queries. At a bare minimum could we at least name the columns the same as the Model's mapped name?

The decision of what side of the relationship is A/B should also be configurable at least with the current approach.
Right now the decision is driven by the alphabetical order of the model names which is not so deductive.

I agree with @jjosef suggestion to name the column Permission and User (referring to the example provided in this issue), rather than A and B. The A and B convention is not very refactoring friendly/future proof. For example, in our prisma model, we have one specific N-M association table that is implicit, but we fear that one day we will need to add an additional column for that association table, which would force us to make the implicit table implicit in the model, and obviously, we do not want columns to be called A and B, but rather called Permission and User (or similar). One way or another, if we need to do so, we will manage (make a new table rather than using the current one and do a data migration, use the new table instead, and finally drop the implicit table), but it would have been nicer to be able to switch from implicit to explicit association tables (nearly) seamless...
IMHO, generated code should be  as close as possible to code written by an expert. Of course, code generators need to be systematic, etc, cannot cope with each and every opinions and conventions experts may have and use. And @jjosef suggestions seems to be a nice step in that direction. I would suspect it would not incur any significant overhead for any of the DB supported by Prisma. Also, code generators should ideally not come with a vendor lock-in i.e., if one day one wants to go SQL-only rather than using Prisma (which I would not recommend😺), the SQL code needs to be as explicit as possible, not arbitrarily obscure for no good reasons.


fix: [#124] Make variables that are required, mandatory

Ensure that variables are set by using requires. If not then bad things can happen.

There was a conversation and the requires will be used only when it is really needed to prevent that disastrous things will happen on a machine if some variables would be omitted.


Exceptions thrown in errorHandler halt app

Version
2.4.4
Reproduction link
https://jsfiddle.net/qok5vcnf/1/
Steps to reproduce
Add an errorHandler that throws an exception, and notice that the app halts badly.
What is expected?
Exceptions thrown from errorHandlers should use the console to print something, or fail silently.
What is actually happening?
The whole app rendering halts.

Was using the raven-js plugin for Vue, which triggers this behavior.  I have filed an issue over there with my feedback (https://github.com/getsentry/raven-js/issues/1061), but I thought it might save some future developers time to not fail quite so catastrophically in this case.

Why don't you console.error the error instead of throwing it inside of the errorHandler?
What's the point of an error handler if you throw the error again?

That is the patch I submitted to Raven, they were doing a lot of logic in their errorHandler (to increase the value of the reported error - good stuff, actually!) without careful try/catches, not to mention some things that made me suspect they hadn't run their Vue plugin in production much.  That being said, the failure mode was so silent, catastrophic, (and hard to debug!) that I figured a quick try/catch around the errorHandler might save some people in the future some headache.

But I think you shouldn't throw errors in the errorHandler, you should treat all of them but not generate new one or ignore them (throwing them)

I agree with you.  However, if you don't do what you should, I think the app shouldn't crash, especially since it's so easy to prevent that error from becoming catastrophic.

But that's because throwing errors in js make apps crash...

Why would I need this? Because I'm testing my vue.js app in a google chrome using cypress and I want to let a test fail when there's an error. But with the change, this is  not possible anymore. So how would I actually raise an error which happens inside a vue component?

@siegy22 - Some more context would be helpful.  My guess is that you may be able to set your own error handler, and hook that up to cypress.  Letting exceptions "escape" is usually not ideal - they should be caught and reported somewhere, and that's what the error handler is for.


Suppress "closed" feature label_position until zoom 17+

#18/37.87298/-122.51169
The Best Buy in this mall has closed, so someone noted that by adding " (Closed)" to the name. Kinda makes sense for recently closed stores, but less so for stores that have been closed more than 1 year?
So zoom 15 is too earlier to see these closed features:

Does make some sense to show them at zoom 17+ (here at zoom 18):

Related: #291

I started on the implementation of this in sql, but it seems that the discussion in https://github.com/mapzen/vector-datasource/issues/291 suggests that we might want to do it in a transform due to tag rewriting.
I also noticed some buildings that have the suffix (closed) as well, eg http://www.openstreetmap.org/way/395475802.

@nvkelso sounds like we would want to just update the zooms here too then, and not alter any of the tags. Do we want to do the same for the buildings layer?

sounds like we would want to just update the zooms here too then, and not alter any of the tags.

Three options:

Don't alter the tags.
Do the same as the  (historical) features in the other issue where we'd sub out so kind:closed.
Same as 2, but throttled by time. Do we know when it was marked closed? If it's recently closed, keep the original kind=*, but if it's a year+ closed then transform to kind:closed?

Do we want to do the same for the buildings layer?

Yes

How can we tell if it was closed recently? Is there a way of encoding that in a separate tag? I searched the osm wiki quickly but the closest thing I found was for opening hours.
My vote would be for option 2 to be consistent with the (historical) features, and do the exact same thing for the buildings layer.

:+1:

Verified:


Implement a blocking many-client

Description
This is done to simplify the usage of many-client in non-async binaries.
Checklist:

[X] I have read and followed the CONTRIBUTING guidelines for this project.
[X] I have added or updated tests and they pass.
[X] I have added or updated documentation and it is accurate.
[X] I have noted any breaking changes in this module or downstream modules.

Codecov Report

Merging #198 (214b58a) into main (d97a272) will decrease coverage by 0.33%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #198      +/-   ##
==========================================
- Coverage   68.61%   68.27%   -0.34%
==========================================
  Files          68       69       +1
  Lines        7860     7897      +37
==========================================
- Hits         5393     5392       -1
- Misses       2467     2505      +38

Impacted Files
Coverage Δ

src/many-client/src/client.rs
73.62% <ø> (ø)

src/many-client/src/client/blocking.rs
0.00% <0.00%> (ø)

src/many-identity/src/address.rs
79.94% <0.00%> (-0.27%)
:arrow_down:

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


Support for username auto-completion when editing message.

Currently, the username auto-completion feature is only implemented for the new message compose box, but not in message editor.

Thanks for reporting this!  This is a duplicate of https://github.com/zulip/zulip/issues/195.
If you're interested in working on this area, I think one thing we don't have an open issue for is being able to autocomplete the #**stream_name** syntax in the message editor.  Opened that as https://github.com/zulip/zulip/issues/2577


This software needs to be updated. Contact the developer for more information.

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: macOS
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
dotnet-sdk-3.0.101-osx-x64.pkg” can’t be opened because Apple cannot check it for malicious software.
It is not updated to the latest macOS Catalina software update.

Can you provide your exact OS version and the commands you're using to install the SDK? Thanks.
CC @aik-jahoda

Having same issue while trying to download dotnet-sdk-3.0.101-osx-x64.pkg
OS Version: Catalina v. 10.15.1
just following these install guides: https://dotnet.microsoft.com/download/dotnet-core/thank-you/sdk-3.0.101-macos-x64-installer and getting the above error message

There's a workaround and more info posted here: https://github.com/dotnet/core/issues/3685#issuecomment-547411254

Show the file in the Finder and open context menu by right click on the downloaded package. Then select Open and a similar dialog will appear however there will be Open button available. When you open it in this way it will execute the package.

@leecow @jcagme do we currently expect this issue to be popping up? Is it documented in a place people can find? Context: "dotnet-sdk-3.0.101-osx-x64.pkg can’t be opened because Apple cannot check it for malicious software." I see a lot of discussion online for other software installers that need this workaround, but people will naturally ask us when our instructions lead them down this path.
/cc @aik-jahoda

Still having this issue when downloading .NET Core SDK 3.1.100: https://dotnet.microsoft.com/download/dotnet-core/thank-you/sdk-3.1.100-macos-x64-installer
macOS 10.15.1 (19B88)

@gastonmuijtjens Did you follow https://github.com/dotnet/core/issues/3943#issuecomment-560578020 comment? If it doesn't help, can you share your error message and os version?

This message is occurring due to new security features in Catalina. We have been working with Apple to understand what needs to be done and are working toward a resolution. This will take place over the next few months for 2.1, 3.0 and 3.1. See the 3.1 Known Issues document for more info on installing on Catalina if you get this error.

Still happening here. Seriously, anyone have tried this before releasing such an important package? Catalina has been around for months now....

I tested it on Catalina and it works fine. You need to follow this comment https://github.com/dotnet/core/issues/3685#issuecomment-547411254 ?

Show the file in the Finder and open context menu by right click on the downloaded package. Then select Open and a similar dialog will appear however there will be Open button available. When you open it in this way it will execute the package.

It works following https://github.com/dotnet/core/blob/master/release-notes/3.1/3.1-known-issues.md#net-core-installation-on-macos-catalina-1015
But it is annoying that such a distribution does not work out of the box.

thanks @aik-jahoda  - right clicking  on file and open worked for me anyway on Catalina.

In order to install Package.dmg on Big Sur, you have to do it by the command line.

Open the Package.dmg
Open the console and type:
sudo /usr/sbin/installer -pkg  -target /


Use GitHub's forms for issue templates

Do you have a screenshot of how it looks like in real life?

Ok, nevermind, I can try myself here 😊  https://github.com/BoD/apollo-kotlin


helm 3: support writing multiple resources to the same file

What this PR does / why we need it:
Templates can contain multiple manifests. As I made a mistake while implementing --output-dir option in #5762  for helm template only the last resource is written to that file.
This PR fixes that.
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Can you write some unit tests to cover this case?

Where should these test be added?
go test -timeout 30s helm.sh/helm/cmd/helm is broken on dev-v3 branch. Are those executed?
Or should the unit tests be placed in pkg/action/...

@bacongobbler Should be ready for review.

@torstenwalter To clarify here, this is to fix the issue whereby you have more than one resource in the same template file and the output just writes the last resource to the output manifest file? I am  asking because you mention "Templates can contain multiple manifests." in the description and it works for this scenario. I would think that this would also need to be fixed for v2.

@torstenwalter To clarify here, is this is to fix the issue whereby you have more than one resource in the same template file and the output just writes the last resource to the output manifest file? I am asking because you mention "Templates can contain multiple manifests." in the description and it does already work for this scenario.
I think that this would also need to be fixed for v2.

I think it works in v2.

@hickeyma squashed and rebased on dev-v3

I think it works in v2.

Yes, it does.

@hickeyma, @bacongobbler Thank you for considering this PR and reviewing it.


Dask implementation for distributed computation

The idea of using distributed computing to tackle the simulated annealing task has come up.
A first test using Dask is being worked on, the first results will come in shortly, but working on that it becomes evident that some work should be done directly on the library to make it more compliant with this kind of implementation.

Yes, I think that to get an actual performance benefit from a distributed (Dask-based) version of Simulated Annealing, we need to think about a custom implementation of the algorithm instead of using the SimulatedAnnealingSampler by D-Wave, which is not meant to run in a distributed setting.
Taking a quick look at the literature, although designing a distributed/parallel SA algorithm is apparently not as straightforward as I thought, there are several ideas to draw inspiration from. For reference, see the following paper:
Janaki Ram et al. - Parallel Simulated Annealing Algorithms

Simulated annealing (SA) has been considered a good tool for complex nonlinear optimization problems. The technique has been widely applied to a variety of problems. However, a major disadvantage of the technique is that it is extremely slow and hence not suitable for complex optimization problems such as scheduling. There are many attempts to develop parallel versions of the algorithm. Many of these algorithms are problem dependent in nature. We present, in this paper, two general algorithms for SA. The algorithms have been applied to job shop scheduling problem (JSS) and the traveling salesman problem (TSP) and it has been observed that it is possible to achieve superlinear speedups using the algorithm.

An easier short-term approach would be instead to implement the parallelization of the SA algorithm at the sampler reads level.
Suppose for example the problem requires you to run the SimulatedAnnealingSampler with num_reads=1000 (the number of reads is simply the number of times you execute the whole global optimization algorithm). Each read is starting from exactly the same BinaryQuadraticModel and it is completely independent from any other read. So, if you have for instance 10 worker nodes available in your distributed computation setting, you can run in parallel just 100 reads per node getting roughly a 10x speedup compared to a single-node execution.


pandownload: Fix hash

Closes #10681

[x] I have read the Contributing Guide.

/verify

/verify


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for basedeconhecimento-bigshop processing.

Name
Link

🔨 Latest commit
19073be14b0e4de4584eff9f3174aa095065f156

🔍 Latest deploy log
https://app.netlify.com/sites/basedeconhecimento-bigshop/deploys/635ffd7aacf8620008ed9912


The example from the description doesn't work

Hi.
I tried the the following example:
Inserted the following content to the settings file and restarted VScode.
"toolbar.toolboxs": [
  {
    "name": "My Box",
    "activate": true,
    "view": "mars",
    "layers": [
      {
        "name": "My Layer",
        "icon": "person",
        "tools": [
          {
            "name": "Open Extensions Folder",
            "icon": "folder-library",
            "actions": ["workbench.extensions.action.openExtensionsFolder"]
          }
        ]
      }
    ]
  }
],

But nothing happened.
You can see it in the following screenshot:

My apologies, my bad. It works!


stm32f072

What I have to change to use with stm32f072?
thank you

I'm not familiar with STM32F072. As first try you can take the code without changes.

works fine on STM32F072 without change.
thank you


Whitelist inbound service warmup

https://ocd.appscale.com:8080/job/Daily Build/4698/
4 node hawkeye failed to deploy application because it took too long
AppScale/hawkeye#136
AppScale/appscale-testing#281


Manga/Anime that actually exists in library disappears temporarily

Steps to reproduce
Uhh, not sure about that. It happens randomly sometimes. But I guess it happens when you make global update,  leave from the app, and check it later while it's still open in background
Expected behavior
The library should look fine
Actual behavior

The library looks empty as you can see on the screenshot. When you start to scroll, it turns back to normal though
Crash logs
No response
Aniyomi version
<IP_ADDRESS> - r7714
Android version
Android 14
Device
Galaxy S23
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extension's repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

what's your sort type in filter? https://cdn.discordapp.com/emojis/1250545322946592829.webp?size=64&quality=lossless&name=filter~1


[BUG] Openshift Local Ver.2.5.1 after every reboot of the PC a terminal window shows with some HTTP Requests

General information

OS: Windows
Hypervisor: Hyper-V
Did you run crc setup before starting it (Yes/No)? Yes
Running CRC on: Laptop

CRC version
CRC version: 2.5.1+5b02b826
OpenShift version: 4.10.18
Podman version: 4.1.0

CRC status
CRC VM:          Running
OpenShift:       Running (v4.10.18)
Podman:
Disk Usage:      15.99GB of 32.74GB (Inside the CRC VM)
Cache Usage:     17.98GB

CRC config
- consent-telemetry                     : yes
- cpus                                  : 6
- memory                                : 32256

Host Operating System
Host Name:                 TH-P1-W10
OS Name:                   Microsoft Windows 11 Enterprise
OS Version:                10.0.22000 N/A Build 22000
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Member Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:          Red Hat User
Registered Organization:   Red Hat Inc.
Product ID:                00329-00000-00003-AA136
Original Install Date:     03/08/2022, 12:33:34
System Boot Time:          07/03/2022, 11:11:52
System Manufacturer:       LENOVO
System Model:              20TJS2F40Y
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 165 Stepping 2 GenuineIntel ~2310 Mhz
BIOS Version:              LENOVO N2VET38W (1.23 ), 02/10/2022
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume4
System Locale:             he;Hebrew
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC+02:00) Jerusalem
Total Physical Memory:     65,289 MB
Available Physical Memory: 24,119 MB
Virtual Memory: Max Size:  75,017 MB
Virtual Memory: Available: 30,932 MB
Virtual Memory: In Use:    44,085 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    win.redhat.com
Logon Server:              \\AMS2-DC02
Hotfix(s):                 4 Hotfix(s) Installed.
                           [01]: KB5013889
                           [02]: KB5007575
                           [03]: KB5014697
                           [04]: KB5014034
Network Card(s):           4 NIC(s) Installed.
                           [01]: Viscosity Virtual TUN Adapter
                                 Connection Name: Tel Aviv (TLV)
                                 Status:          Hardware not present
                           [02]: Intel(R) Wi-Fi 6 AX201 160MHz
                                 Connection Name: Wi-Fi
                                 DHCP Enabled:    Yes
                                 DHCP Server:     <IP_ADDRESS>
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                                 [02]: fe80::d11b:5a94:fd02:ca5e
                           [03]: Hyper-V Virtual Ethernet Adapter
                                 Connection Name: vEthernet (crc)
                                 DHCP Enabled:    Yes
                                 DHCP Server:     <IP_ADDRESS>
                                 IP address(es)
                                 [01]: <IP_ADDRESS>
                                 [02]: fe80::800d:e8a2:1f7d:6d86
                           [04]: Bluetooth Device (Personal Area Network)
                                 Connection Name: Bluetooth Network Connection
                                 Status:          Media disconnected
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

Steps to reproduce

after install of CRC 2.5.1 finish the setup
reboot the machine / Laptop
a terminal windows with \.\pipe\crc-http popup every time

Expected
Actual
Logs
Before gather the logs try following if that fix your issue
$ crc delete -f
$ crc cleanup
$ crc setup
$ crc start --log-level debug

Please consider posting the output of crc start --log-level debug  on http://gist.github.com/ and post the link in the issue.

@Tal-Hason This terminal windows comes for short time period (flashes) because of the schedule task and we already have https://github.com/code-ready/crc/issues/3089 for tracking purpose.

Also notice after closing the terminal window the CRC is stoping by it self and in the log is shows that its running
listening vsock://00000400-FACB-11E6-BD58-64006A7986D3
Checking Windows 10 release
Checking Windows edition
Checking if crc-users group exists
Checking if Hyper-V service is enabled
Checking if vsock is correctly configured
CRC VM is running
Check internal and public DNS query...
Check DNS query from host...
Failed to query DNS from host: lookup foo.apps-crc.testing: no such host
Verifying validity of the kubelet certificates...
Starting OpenShift cluster... [waiting for the cluster to stabilize]
Operators are stable (3/3)...

but in the Hyper-V is still running

It feels like he reports this as being persistent (not closing).

yes

its RadHat CSB Windows ... :-\

I will have a look at this.

Maybe this is the issue:
Specifies the state of the window that is used for the new process. The acceptable values for this parameter are: Normal, Hidden, Minimized, and Maximized. The default value is Normal.

**You cannot use the WindowStyle and NoNewWindow parameters in the same command.**

The parameter does not apply for non-Windows systems. When using on non-Windows systems, you never get a new window.

maybe instead of using "Windows Task Scheduler" put a script in the "shell:startup"

A service can ran as local system or a network service and wont need any special premissions

You can check our commit log; as this is what we did earlier. It wasn't ideal for us.
We used network service and local system, but that is actually too many privileges. This is a security risk.

as you create a "task" in the task scheduler" with my logon

you can create a service to run with my AD user. i guess most user that install the OCP Local will have Local Admin Permissions
obj= { | }

A service to run as user needs credentials. These, as enforced by many policies will change over time and become invalid.
Also, "Local Admin Permissions" is not a given. We mostly see this for .NET developers, but not for Java developers.

This might have to do with the Windows Terminal. In a normal situation, powershell should be run from a regular command prompt and this 'disappears'. But as I can see in the screenshot, powershell is hosted inside the Windows Terminal application.
Does a Win + R -> powershell also open in the Windows Terminal?

changed the the setting for command line to from the windows terminal to "windows console host"
rebooting the machine and check

Perhaps this could be solved by force running powershell.exe in conhost.exe instead of allowing the OS to use the default configured console host. For example, if I run conhost powershell.exe -NoExit Get-Date it will open in a non-Windows Terminal host. Obviously, the code would not use -NoExit. I used that parameter to prevent the window from closing immediately. From my testing, all the parameters still get passed into the powershell executable correctly. A few changes in powershell_windows.go could have this issue addressed.
For those looking for more information, Scott Hanselman describes what conhost vs Windows Terminal host is here: https://youtu.be/tuhzVDc0Slg?t=804
It may not be worth the effort trying to determine which default console host is configured. If I recall correctly, conhost.exe is not going anywhere.

I think I found an end user work around. If you set powershell.exe to use "Windows Console Host" by default, the CRC log window does not display.  This change does not require the user to change their default terminal application.
Steps:

Run powershell.exe via conhost: Win+R and run conhost powershell.exe -NoProfile -NonInteractive
Right click on the title bar and choose "Properties"
Change the" Default Terminal Application" to Windows Console Host

You can test by running powershell.exe -NoProfile -NonInteractive using Win+R

this was fixed in #3901


Program not working after the update

Hello, the recent updated XivAlexander does not seem to work with Korean server for some reason?
Nothing shows up on my Log screen

Are you using this without installing? In that case, I forgot to include a file in the archive. Try downloading it again, and it should work. Otherwise, if you did install, check which files exist in %APPDATA%\XivAlexander.

Are you using this without installing? In that case, I forgot to include a file in the archive. Try downloading it again, and it should work. Otherwise, if you did install, check which files exist in %APPDATA%\XivAlexander.

Works amazing again after re-installing!! Thank you for the help!

Hi does Version matter ? I just downloaded the one on the github and my
region is japan . Do I need to change something ?
On Sun, 19 Sep 2021, 16:50 justin767, @.***> wrote:

Are you using this without installing? In that case, I forgot to include a
file in the archive. Try downloading it again, and it should work.
Otherwise, if you did install, check which files exist in
%APPDATA%\XivAlexander.
Works amazing again after re-installing!! Thank you for the help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Soreepeong/XivAlexander/issues/148#issuecomment-922457213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVWIXUR2JPK6YWMFQAG5XU3UCXBQBANCNFSM5EKHBG3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I reinstalled and updated the files, to which it stopped working, it says it can't verify both x32 and x64 dll and both loaders, then my antivirus recognizes XivAlexanderLoader32.exe as a trojan file:
Trojan:Win32/Sabsik.FT.A!ml
So what's going on about that? Whatever the response may be, should I use a different release instead?

Version does not matter as long as it works, and I don't really care about antiviruses detecting it anymore.

I reinstalled and updated the files, to which it stopped working, it says it can't verify both x32 and x64 dll and both loaders, then my antivirus recognizes XivAlexanderLoader32.exe as a trojan file:
Trojan:Win32/Sabsik.FT.A!ml
So what's going on about that? Whatever the response may be, should I use a different release instead?

Try disabling your anti-virus software during XivAlexander installation, and add the software as an exception for its virus detection? If you are using Windows defender, I had no problem after setting it up like this.


Automatic residental-address detection

It would be helpful to have automatic residential-address detection. @4kbt has explored SmartyStreets capabilities (at the suggestion of @marcuscollins) , and it looks plenty capable for our needs.
Why is it useful? Many of our recipients list residential addresses. Reasons vary, but they include increasing accessibility to the community, making drop-off safer for everyone, or perhaps avoiding employer restrictions on PPE donations.
In any case, it would be useful to surface a small note on residential donations like : "This appears to be a residential address. Many legitimate recipients list a home address to increase accessibility to the community."  or some such, just so a donor is not confused by "NYC Hospital" appearing in New Jersey, when in fact it is a doctor who works at NYC Hospital making the request.

We have attempted to do this detection by hand, but it takes too much time for us to do it with our current level of moderator support.

@4kbt - were you able to secure free API access?

I reached out to lob.com and easypost who have address validation apis that can do the same thing.

Never heard back from them, which is a real bummer. They were super
responsive on their first interaction. They hand-processed our entire
dataset, recorded a movie...  really cool.
On Sun, Apr 12, 2020 at 1:02 AM Saikat Bhadra<EMAIL_ADDRESS>wrote:

I reached out to lob.com and easypost who have address validation apis
that can do the same thing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/findthemasks/findthemasks/issues/391#issuecomment-612579134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6L2GAIFZROKIGHO4ZSBG3RMFYQHANCNFSM4LYG64YQ
.

An update -- got a reply from SmartyStreets today. Following up.

Continued traction -- they're going to provision a findthemasks account with quota.

Nate Burt: Should these fields be added back to the spreadsheet or just needed in data.json?
Charlie Hagedorn :
We'd want to add them to the spreadsheet (and perhaps many of the other SmartyStreets fields).   I've coordinated unlimited SmartStreets API calls for the next three months, and have an account ready to go. I just don't have the dev knowledge necessary to efficiently make it all go.
Have to make sure that the Sheet doesn't exceed 5 million cells, too. (probably not a problem, but the SmartyStreets data are rich).
The simplest approach is apt to be asking for a full smarty-streets dump for every Final Address into an independent tab, then pulling the columns we need into Combined for moderator use and packaging into data.json.

Necessary credentials are in dev-secrets.

Currently, the US has a cloud function called "smarty_streets_analysis" to look up smarty streets data for an address and write it to a separate tab in sheets.

Update this cloud function so that it also adds the "rdi" column to the US Combined tab.
On the FE, if the rdi column is "Residential" add "Residential Address. (why residential?)" to the listing + infowindow popup. "why residential?" should link to a popover with a title of "Why residential" and a body of:

"About 15% of our recipients list home addresses. This may be because it is safer/easier for donors to drop off at a nearby location, because their office hours are limited, or because they are having trouble getting PPE at work.
If you prefer to donate directly to a healthcare institution, please select another entry from among our thousands of recipients in need!"

Closing since the US is live, opened a separate issue for international: https://github.com/findthemasks/findthemasks/issues/762

So good.
On Wed, Apr 29, 2020 at 3:34 PM Nathanael Burt<EMAIL_ADDRESS>wrote:

Closing since the US is live, opened a separate issue for international:
#762 https://github.com/findthemasks/findthemasks/issues/762
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/findthemasks/findthemasks/issues/391#issuecomment-621502119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6L2GBUMS6QV3QT5GQSCQ3RPCTKXANCNFSM4LYG64YQ
.


v1.8 docs release endgame

Update all SDKs docs

[x] All SDK maintainers ACK that docs are ready
[x] Update all the SDK submodules and triage changes with maintainers

v1.7 Branch - use this for reference: **** https://github.com/dapr/docs/pull/2345
In pending PR:

[x] Add v1.8 (latest) to nav bar
[x] Archive notice
[x] Update root README.md
[x] Update URL in config.toml
[x] Delete workflow file

Day of:

[x] Push v1.7 PR

v1.8 Branch - use this for reference: **** https://github.com/dapr/docs/pull/2344
In a pending PR:

[x] Update README.md
[x] Update version dropdown in nav bar
[x] Supported versions section
[x] All output logs change 1.7 to 1.8 (change dapr-latest-version.html)
[x] Update URL in config.toml
[x] Update workflow for docs.dapr.io to v1.8 branch

Day of:

[x] Push v1.8 PR

GitHub - After release

[x] Make v1.8 the default branch
[x] One last upmerge

Website

[x] Create new website for v1.7, v1.8, v1.9
[x] Add DNS entry for new website - @greenie-msft  to ask CNCF to create this. Let's as for v1-7.docs, v1-8.docs, v1-9.docs.
[x] Add custom domain & generate SSL certificate - Needs to happen after the DNS entry is created

v1.9 Branch

[x] Create v1.9 branch
[x]  Update website workflow file - See this PR https://github.com/dapr/docs/pull/2156 for the Nav bar
[x]  Upmerge from release branch to preview branch

Release announcements

[x] Release notes/blog
[x] Tweet release announcement
[x] Discord release announcement

Shipped!!


Link Rot: www.soundcloud.com

As of 08/11/2017, links to SoundCloud are timing out. This tracks that issue.
If the service isn't fixed in one month, I'll remove the links.

It's hard to determine what's really down. It's not down for my browser, and when running rake test locally, the tests pass. Reverted via 327ec2dfe216250ba6889d2b382a98dbb39f89db because TravisCI sees SoundCloud as down.

Finally we're good.


add placeholder

@MikeTschudi this is a simple one...going to merge so I can do one more PR to IA tonight. Will update if you see anything


Windows MSYS2 clang32 and clang64

Re-enable the github ci tests for Windows MSYS2 clang32 and clang64.
Currently webp fails to build

Perhaps we could disable webp for a while on MSYS2 platform.
(e.g. same thing happens here: https://github.com/google/oss-fuzz/issues/8366)

Any reason to not use already existing binary webp packages from MSYS2? They're available for both clang32 and clang64, don't know if the features switched off made the difference...

We do - fetch them with pacboy...

Problem happens on linking time -> it is problem with webp package. Not unlikely it is missing some transitive dep.

Ah, the new webp 1.3.0 introduced a new dependency - need to link also libsharpyuv.

Not sure what https://github.com/libjxl/libjxl/blob/0a6bafeead372d59329c0ea390529b70befd4b73/tools/CMakeLists.txt#L246 is doing re transitive deps and constructing that IMPORTED_TARGET, you might have better luck with find_package(WebP CONFIG) if you know you're using a more recent version that ships the CMake config...

In https://github.com/libjxl/libjxl/commit/5a859bf0619d5b4483de4cb4a18b07b539b2657a it looks like you're handling the static linking specially and on your own, so there might not be any problems with the webp package and source after all, no?

Thanks for noticing that. Will land proper fix soon.

FYI, here's the list of internal variables pkg_check_modules(WebP IMPORTED_TARGET libwebp) defines for me:
WebP_CFLAGS:INTERNAL=-IC:/msys64/ucrt64/include/webp
WebP_CFLAGS_I:INTERNAL=
WebP_CFLAGS_OTHER:INTERNAL=
WebP_FOUND:INTERNAL=1
WebP_INCLUDEDIR:INTERNAL=C:/msys64/ucrt64/include
WebP_INCLUDE_DIRS:INTERNAL=C:/msys64/ucrt64/include/webp
WebP_LDFLAGS:INTERNAL=-LC:/msys64/ucrt64/lib;-lwebp
WebP_LDFLAGS_OTHER:INTERNAL=
WebP_LIBDIR:INTERNAL=C:/msys64/ucrt64/lib
WebP_LIBRARIES:INTERNAL=webp
WebP_LIBRARY_DIRS:INTERNAL=C:/msys64/ucrt64/lib
WebP_LIBS:INTERNAL=
WebP_LIBS_L:INTERNAL=
WebP_LIBS_OTHER:INTERNAL=
WebP_LIBS_PATHS:INTERNAL=
WebP_MODULE_NAME:INTERNAL=libwebp
WebP_PREFIX:INTERNAL=C:/msys64/ucrt64
WebP_STATIC_CFLAGS:INTERNAL=-IC:/msys64/ucrt64/include/webp
WebP_STATIC_CFLAGS_I:INTERNAL=
WebP_STATIC_CFLAGS_OTHER:INTERNAL=
WebP_STATIC_INCLUDE_DIRS:INTERNAL=C:/msys64/ucrt64/include/webp
WebP_STATIC_LDFLAGS:INTERNAL=-LC:/msys64/ucrt64/lib;-lwebp;-lsharpyuv
WebP_STATIC_LDFLAGS_OTHER:INTERNAL=
WebP_STATIC_LIBDIR:INTERNAL=
WebP_STATIC_LIBRARIES:INTERNAL=webp;sharpyuv
WebP_STATIC_LIBRARY_DIRS:INTERNAL=C:/msys64/ucrt64/lib
WebP_STATIC_LIBS:INTERNAL=
WebP_STATIC_LIBS_L:INTERNAL=
WebP_STATIC_LIBS_OTHER:INTERNAL=
WebP_STATIC_LIBS_PATHS:INTERNAL=
WebP_VERSION:INTERNAL=1.3.0
WebP_libwebp_INCLUDEDIR:INTERNAL=
WebP_libwebp_LIBDIR:INTERNAL=
WebP_libwebp_PREFIX:INTERNAL=
WebP_libwebp_VERSION:INTERNAL=

(This is w/ upcoming libwebp-1.3.0-2 w/ patched "Requires.private".)


Conflict lua <--> luajit

Z:\vcpkg>vcpkg install luajit sol2
The following packages will be built and installed:
    luajit[core]:x64-windows-static
    sol2[core]:x64-windows-static
Starting package 1/2: luajit:x64-windows-static
Building package luajit[core]:x64-windows-static...
-- Using cached Z:/vcpkg/downloads/LuaJIT-LuaJIT-v2.0.5.tar.gz
-- Using source at Z:/vcpkg/buildtrees/luajit/src/v2.0.5-c87550d73d
-- Applying patch Z:/vcpkg/ports/luajit/001-fixStaticBuild.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Building x64-windows-static-dbg
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/debug/tools/luajit.exe
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/debug/lib/lua51.lib
-- Building x64-windows-static-rel
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/tools/luajit.exe
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/lib/lua51.lib
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/lua.h
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/luajit.h
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/luaconf.h
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/lualib.h
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/lauxlib.h
-- Installing: Z:/vcpkg/packages/luajit_x64-windows-static/include/lua.hpp
-- Performing post-build validation
-- Performing post-build validation done
Building package luajit[core]:x64-windows-static... done
Installing package luajit[core]:x64-windows-static...
The following files are already installed in z:/vcpkg/installed/x64-windows-static and are in conflict with luajit:x64-windows-static

    include/lauxlib.h
    include/lua.h
    include/lua.hpp
    include/luaconf.h
    include/lualib.h

It appears (from the above output) that luajit uses 5.1, while lua is 5.3.

This is why we recommend creating a new directory to install headers.

Thanks for your feedback. this issue has fixed in https://github.com/Microsoft/vcpkg/pull/5863/

Thanks.


Instructions on accepting invitation and receiving credentials to show the features of the app

Hello all,
I've been working on the development branch of the British Columbia Indy network, using this as a mobile application. Ive been running into some issues on accepting the connection from the issuer agent. If you have time/ or have some type of tutorial for using this mobile agent to interact with the issuer agent (accept the connection and receive credentials) I would be really interested in this.
Ignace Loomans

Hi @ILoomans, we don't have a written tutorial. We do have a video that uses the mobile agent with an ACA-Py issuer agent (https://www.youtube.com/watch?v=mmFSgSR5yPA). It is however a bit outdated.
Is there a specific issue you're running into?
We're planning to deprecate this app and collaborate on the aries bifold app (https://github.com/indicio-tech/aries-mobileagent-react-native)

Thanks for the response @TimoGlastra ,
I have a deployed issuer, we can generate the invitation_url, decode it and accept invitation on the app, but the state just stays on 'REQUESTED' after that. I think it's a problem on the backend but I'm not certain.
Ignace

Hmm hard to say why it's not working.

Is the issuer an ACA-Py or AFJ agent?
Are you using an ngrok tunnel for both the issuer and the mediator. Are the agents able to reach each other?

If you can share your exact setup I may be able te replicate the issue.

The issuer agent is aca-py, mediator is AFJ, i found that i was only able to establish a succesful connection using the ngrok tunnel for the mediator on AFJ, I have started using the new app (indico) to test it and i am able to establish connection with the aca-py demo faber agent, however the app isnt able to receive the offer-credentials (sorry i switched up the issue a bit here). On the app i am getting a mediator error.
Mediation Error:  503 ngrok gateway error
The server returned an invalid or incomplete HTTP response.
Do you know of a publicly deployed mediation endpoint i could try this with?


C++11 support needed for blt_cuda_mpi_smoke

I'm running into trouble when trying to build the blt_cuda_mpi_smoke (rev 9383457) test, but I'm not sure if this is a blt issue or a local configuration problem, and I was looking for some advice.
The platform is Power9+Tesla, running
Linux 4.14.0-115.7.1.el7a.ppc64le #1 SMP Thu Mar 28 16:03:17 UTC 2019 ppc64le ppc64le ppc64le GNU/Linux

and compiling with XL-<IP_ADDRESS>, CUDA 10.1, and OpenMPI 4.0.1.
If I attempt to compile a BLT project with this configuration, I get while compiling blt_cuda_mpi_smoke.cpp during the include of <cstdint> about it needing C++11 support:
In file included from /home/phenning/Projects/UTA-Proj/UTA/blt/tests/smoke/blt_cuda_mpi_smoke.cpp:42:
In file included from /projects/opt/ppc64le/gcc/4.9.3/lib/gcc/powerpc64le-unknown-linux-gnu/4.9.3/../../../../include/c++/4.9.3/cstdint:35:
/projects/opt/ppc64le/gcc/4.9.3/lib/gcc/powerpc64le-unknown-linux-gnu/4.9.3/../../../../include/c++/4.9.3/bits/c++0x_warning.h:32:2: error:
      This file requires compiler and library support for the ISO C++ 2011
      standard. This support is currently experimental, and must be enabled with
      the -std=c++11 or -std=gnu++11 compiler options.
#error This file requires compiler and library support for the \
 ^
make[2]: *** [blt/tests/smoke/CMakeFiles/blt_cuda_mpi_smoke.dir/blt_cuda_mpi_smoke.cpp.o] Error 1

This is weird to me for several reasons that make me suspect local XL configuration problems.  First is that I thought <cstdint> (for the most part) was available pre-C++11. And second, I don't know why XL is looking up such an old gcc version. However, I find that adding a set(BLT_CXX_STD "c++11" ... command fixes this, so I don't know if this is something peculiar to the XL family of compilers that should be handled in BLT.  Thoughts?

thats a fun one, not sure how critical any use of cstdint functionally is -- given the compiler isn't happy, maybe we should refactor to avoid using it. This test is actually quite a bit more complex than I would suggest for a smoke test.

<cstdint> was made available with C++11 support in XL and GCC. With XL, and you can enable C++11 support with -std=c++11. To point XL to a newer version of GCC, please read Configuring IBM XL C/C++ for Linux, V16.1.1 on Knowledge Center.

I will refactor this smoke test to be much simpler hopefully today.


TBMQ fails to login when placed behind haproxy for SSL termination

Component

Core

Description
Our organisation uses Thingsboard PE and Trendz extensively and we are in the process of testing TBMQ with the view of migrating our MQTT servers (mosquitto currently) to TBMQ. I have the tbmq UI working well with http but when I put haproxy in front of the tbmq for HTTPS/SSL termination I get errors in the tbmq logs showing issues with CORS at the login page when I enter my username and password. This extract is from the docker logs:
java.lang.IllegalArgumentException: When allowCredentials is true, allowedOrigins cannot contain the special value "*" since that cannot be set on the "Access-Control-Allow-Origin" response header. To allow credentials to a set of origins, list them explicitly or consider using "allowedOriginPatterns" instead.
The tbmq documentation shows the environment variables to configure tbmq with and under the section "Spring MVC/Resources parameters" it shows the CORS parameters but there is no environment variable in the middle column to allow us to configure this parameter.
Is there a reason why tbmq uses CORS? Thingsboard and Trendz do not require CORS and live happily behind haproxy. We have a mandate to use haproxy in our organisation for SSL termination since it gives a lot of additional features by default to secure the connection better than using SSL directly in the end product.
Environment
Ubuntu 22.04.2LTS, Docker 24.04, Docker compose 2.19, haproxy 2.4.22 with Lua 5.3.6

OS:  Ubuntu 22.04.2
TBMQ: 1.01 docker
Browser: Chrome

hi @DavidHill0809,
Thank you for reporting the issue. We will work on the fix for this and include it in the next release.


On Device Translation crashes app when translating multiple words

Rather than crashing the app with no information about why I would expect to get an error message back or an exception thrown that could be caught with a try/catch.
This should be reproducible in the sample project. That is where I found it.

Hi @fbernaly
Can i handle this.. seem to be a try-and-catch block needed

@bensonarafat : go ahead, give it a try, make sure to provide testing steps in the PR.

great

@SuperWes : could you confirm that you reproduce this in the simulator? or in a real device?

I can confirm this happens on a real device and even on android

Any updates on this? I am facing the same issue on iOS simulator.

Sorry, I have been busy with work, I do not have an ETA for now. However, contributions are always welcome. Submit your PRs and I will review them.

Hello, I will try to find the right solution as soon as I get some free time but for the time being I used this nasty hack. I think it's not that good from performance perspective but does the job for now.
Since this only breaks on iOS, i did a platform check and then iterated over the whole string word by word, then translated each word instead of the whole line string at once.
Here's the code if someone wants to use it until we get a proper fix:
// it crashes on iOS when multi-word line is passed
if (Platform.isIOS) {
  String translated = "";
  for (var substr in text.split(" ")) {
    translated =
        "$translated ${await onDeviceTranslator.translateText(substr)}";
  }
  return translated;
} else {
  return await onDeviceTranslator.translateText(text);
}


Activity feed: handling no records

Handle no records in the activity feed. Suggest where to go to create records.

If the user has no records we should be showing them the getting started component. Only when they have records can that be hidden.


Added traces to CRI API pertaining to containers and sandboxes

Sandbox and Container CRI commands now log debugging information about the infra-container or container they've just operated upon.
Including adding function in util that translates labels in a sandbox run/container create request into a description. This allows the user to see requests to create a container even if it's failed.
Signed-off-by: Peter Hunt<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

/ok-to-test

/test e2e_rhel

/test e2e_rhel

/test e2e_rhel

/test e2e_rhel

Rebasing to master should fix the tests now.

/test integration_rhel

@mrunalp happy tests!

/lgtm

/approve


Add support for including files in linkerscript

Files can now be included with the following syntax:
INCLUDE "path.link"

The maximum include depth is 5.
Man page updated.
Fixes https://github.com/rednex/rgbds/issues/147

Any particular reason why you chose a maximum depth of 5? It sounds a little low to me… I'd personally go with more like 16 myself.
Also, it could quite easily be possible to dynamically allocate the includes stack, which would allow for as many includes as you'd like (although you'd have to be careful of infinite recursion, of course).

5 levels of depth seems like a lot more than people will use... I really don't expect anybody to use more than 3.

Sure, but it still feels like a rather arbitrary limitation, and I'd don't think that exceeding this limit is outside the realms of possibility.

Supposing each file of level 5 defines sections for 10 banks and each level includes 10 files of the following level, the result is 10^5 banks. Not even the disassemblies of Pokemon Crystal or similar need such a high depth, at most they need 2 more levels than the base file.
Of course, if someone finds this is too restrictive the define can be changed and that's it, but I can't think of a case in which making this bigger makes actual sense.

This should do it. I've also increased the number to 8, and that's a lot more than anyone could need.


Implement DictSquashCopyDict hint

This PR implements DictSquashCopyDict hint, which prepares arguments for creating a new dictionary with dict_new function. It copies the existing squashed dictionary in the  current scope
Closes #292

@har777  i'm not 100% sure i'm doing write this python code
dict(__dict_manager.get_dict(ids.dict_accesses_end))

could you check this?
I guess this code only retrieves from operanders the address dict_accesses_end , uses GetDictionary to retrieve the dictionary (a copy) and assign it in scope?

could you resolve conflits? It will be easier to review ✅

@har777  @MaksymMalicki  just updated, now making a deep copy of the ZeroDict before assign it in the new scope.
I added only one test for now with an empty dictionaryManager, will add other tests tomorrow

currently FreeOffset field of the ZeroDict struct is of type *uint64 , any reason?

@har777  ready for review, i think this is the last dict hint

i guess we should use FreeOffset uint64 in ZeroDictionary, there is no benefit of using a pointer here, we can modify this later

@har777  now only the dict Data is copied and I inlined the copy


check for newuidmap before reexec

Without this check img subcommands will fail:
nsenter: failed to execv: No such file or directory
nsenter: failed to sync with parent: SYNC_USERMAP_ACK: got 255: Success
nsenter: failed to use newuidmap: Success
With it, you get this instead:
FATA[0000] newuidmap not found (install uidmap package?): exec: "newuidmap": executable file not found in $PATH·

I realized as soon as I submitted this that the original error DOES say that newuidmap failed, but I missed it the first time not realizing that newuidmap was a binary that I needed to install and focused on trying to figure out why nsenter wasn't working right.
I had ran strace -f -e execve which gets lost somewhere and just shows
[pid 12376] execve("/proc/self/exe", ["/proc/self/exe", "build", "-t", "foo", "."], 0xc000326a80 /* 54 vars */) = 0
strace: Process 12377 attached
strace: Process 12378 attached
[pid 12378] execve(0x13c0408, 0x7fff4bcae5d0, 0x7fff4bcae5c8) = -1 ENOENT (No such file or directory)
nsenter: failed to execv: No such file or directory

which was no help..
in any case, img works great once you read the documentation and install uidmap.

great thanks :)


FilePicker Site document libraries - tab allows to select a file from Other Sites Libraries

FilePicker control
The documentation says:Site document libraries - tab allows to select a file from the existing site document libraries
Category
[X] Enhancement
[ ] Bug
[F] Question
Version
Please specify what version of the library you are using: [  3.1.0  ]
Expected / Desired Behavior / Question
Ability to select libraries in any Site with permissions.
Is there a property I can set to get files from any site?
Can I change the context to be of other site to be able to get files from other sites?
Thanks! FilePicker is amazing

Hi @Ofer-Gal , I don't think you can change the context for that and there is currently no property to allow the URL of a site to be specified.
I had a quick look a the FileBrowserService and seems that adding support for this would not be a complex task. Would you be interested in giving it a go and submitting a PR with the improvement?

I never learned how to submit a PR. I will give it a try :-)

@Ofer-Gal sorry for taking so long to reply. Submitting PRs is actually simpler than it may look like at first.
At a very high level (it can also be done in different ways):
-Fork the repository
-If already forked previously, ensure the dev branch in your fork is in sync and contains all the latest changes
-Create a new branch (from dev) for implementing the changes
-Once changes are complete, ensure everything is committed
-Go to GitHub and open the relevant repository
-A notification should appear asking you if you want to create a PR

Complete the form following the instructions provided and it's done

There is a detailed guide here:
https://tahoeninjas.blog/2019/08/18/my-github-cheat-sheet-for-pnp-contributions-an-interactive-cheat-sheet/
You can also join for one of the awesome PnP sessions for first-time contributors!
https://pnp.github.io/sharing-is-caring/#pnpsicsessions

I did try :-)
I see I need to make an equivalent of public getListItems in line 27 of /service/FileBrowserService.ts
But I do not know were the replacement URL for this.context.pageContext.web.absoluteUrl will com from.
There should be a textbox for that somewhere. Maybe at the top of the render() in /controls/FileBrowser.tsx
I have no experience with classes. I started react with hooks.
I rather leave this with better developers than I am.
Thank you for the suggestion though.

Hey guys,
So, I stumbled on this issue a couple days ago because I needed this feature for my project. As @joelfmrodrigues said, it was not a complex task, but I had to add a new method to fetch the site Title to display it in the breadcumb.
I have submitted a PR #1081 to add this feature. If you find any problem with the implementation, feel free to contact me.

Thanks to @kadu-jr this has been implemented and will be included in the next release.

I upgraded to "@pnp/spfx-controls-react": "^3.6.0-beta.1671868" but I don't see a way to tell the tab (site tab?) to show libraries from other sites then the current.
Also, Is there a way to change the order of the tabs. (Upload files first, OneDrive second etc..)
Thank you

@AJIXuMuK can you please tell me how to tell the tab (site tab?) to show libraries from other sites then the current.?

When I use webAbsoluteUrl={`${window.location.origin}/teams/${dpcRequest.projectNumber}`} it works fine but I want to show the "General" folder under Shared%20Documents (we renamed it "Working Papers")
So I added defaultFolderAbsolutePath={`${window.location.origin}/teams/${dpcRequest.projectNumber}/Shared%20Documents/General`}
But it errors with : status='404'' dist/node_modules/@pnp/spfx-controls-react/lib/services/FileBrowserService.js:148 [FileBrowser._getListItems]: Something went wrong when executing request. Status='[HTTP]:404 -  [CorrelationId]:885b20a0-a02b-1000-97bf-669cb5b3b6d9 [Version]:<IP_ADDRESS>26' and shows "This folder is empty" (it always have 6 folders)
What is the right way to set the default folder?


HHH-18629 Fix inconsistent column alias generated while result class is used for placeholder

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

https://hibernate.atlassian.net/browse/HHH-18629

Problem is likely https://github.com/hibernate/hibernate-orm/pull/9075#issuecomment-2491291737
Checking...

The changes fail on valid use cases.  Looking...

Sorry, just meant to comment :)


[BUG]

Describe the bug
I execute a configure_dataset_settings for each workspace where each workspace belong a one unique user. This form each dataset/workspace  supports the labels configurations. The records is shipped separately for each user. However in listener, only is recognized the shipped for the last workspace configured, i believe this could happend because the client.active_client() support just one workspace at same time. The listener is not oriented for work with many workspace where each one work separately from the other.
Stacktrace and Code to create the bug
rg.client.active_client().set_workspace(workspace)
rg.configure_dataset_settings(name=dataset.name, settings=settings, workspace=workspace)
rg.log(records=records, name=dataset.name, workspace=workspace)

@listener(
        dataset=dataset.name,
        query="status:Validated AND metadata.batch_id:{batch_id}",
        condition=lambda search: search.total == dataset.num_samples,
        execution_interval_in_seconds=3,
        batch_id=0,
        isInitial=True,
        qtd=dataset.num_samples,
)
def active_learning_loop(records, ctx):

Logging... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
5 records logged to http://<IP_ADDRESS>:6900/datasets/victor/papers
Logging... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
5 records logged to http://<IP_ADDRESS>:6900/datasets/auditor/papers

Expected behavior
Recognize data from the auditor and victor workspaces and treat each separately. The Communication works only for the last workspace (auditor).
Environment:

Argilla Version [e.g. 1.12.1]:
ElasticSearch Version [e.g. 7.10.2]:
Docker Image (optional) [e.g. argilla-quickstart:v1.12.1]:

Additional context
Add any other context about the problem here.

Hi @LordWaif and thanks for reaching out.
Exactly, the current client connection works based on a preset session workspace. We plan to work on this to support contextualized workspace support.

thank you for the explanation


Create Cu_p_s

Thanks for the contribution!
any chance you could update the file name, such the last character represents a charge state.
Mattias


Hotfix/azure.resource manager.cosmos db 1.3.1

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.CosmosDB


Confusing prompt copy when cancelling an export

Platform:
17.4.0 Darwin Kernel Version 17.4.0: Sun Dec 17 09:19:54 PST 2017; root:xnu-4570.41.2~1/RELEASE_X86_64 x86_64
Kap Version:
Version 2.1.1 (<IP_ADDRESS>7)
Steps to reproduce

Make a recording,
Export as .gif
Click on Cancel Export button
Read copy on the confirm cancellation modal

Current behaviour
Confirm modal says Are you sure you want to cancel the export? and the two options of Continue and Cancel Export. It is unclear if Continue means I want to continue with cancelling the export or continue with the export.

Expected behaviour
Confirm modal says Are you sure you want to cancel the export? and the two options should say something like Yes and No or Do not Cancel and Cancel.

Ah, yes. We should change this to:
Are you sure you want to cancel the export?
Cancel cancelling or Cancel
On a more serious note; Yes or No is simple and obvious, but I'm not sure we need this prompt at all? You don't lose the original recording if you cancel the export.

but I'm not sure we need this prompt at all?

@skllcrn Maybe it helps if the user clicks [x] to close the dialog during a long export instead of minimizing it? If explicitly clicking Cancel Export I don't think it is needed.

I think it's useful to have an explicit option to cancel, this is in part to future proof for queued exports. The button would allow you to cancel the currently active one and skip to the next in the queue, while closing the window would cancel all of them.

I think it's useful to have an explicit option to cancel

@skllcrn Agreed. Perhaps I should have worded my suggestion better:

@skllcrn Maybe the confirm modal helps when the user accidentally clicks [x] to close the export dialog during a long export instead of minimizing it? If explicitly clicking Cancel Export I don't think the confirm modal is needed.

I like your suggestion to get rid of the confirm modal for the cancel button, but am think it might be worth keeping the confirm modal when cancelling the export through the title bar.

I think the buttons should be [Continue Export] and [Cancel Export]. Yes/No buttons are not recommended by the macOS Human Interface Guidelines: https://developer.apple.com/macos/human-interface-guidelines/windows-and-views/alerts/

I like your suggestion to get rid of the confirm modal for the cancel button, but I think it might be worth keeping the confirm modal when cancelling the export through the title bar.

This sounds like a good solution to me, with the changes @sindresorhus suggested.

@skllcrn It looks like the prompt already only appears when clicking [x], I've updated the text in https://github.com/wulkano/kap/pull/438


u-boot-iot2050: Lift to 2023.10-rc4

Most of our patches are merged now, just one DT change and a small fix for the signing script are remaining.
In this latest upstream version, we folded PG1 and PG2 U-Boot configs into a single one again, generating both artifacts in the same run. Consequently, we can simplify the build as there is now only one recipe as well.
Tests are pending.

Now tested a bit better. Two more patches were needed, both sent upstream already.

2023.10 has been released some days ago. Update to that final version will follow (one patch was merged in time).


fix: Fix the SensitiveInput  the browser thought it was a password field

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Fixed the SensitiveInput component so that Chrome does not mistakenly recognize inputs as passwords.

Changed

SensitiveInput now uses -webkit-text-security: disc instead of type=password, causing the browser to not recognize the field as a password.

Additional Information

I've already checked, -webkit-text-security works with all browsers from the last 4 years, and Open WebUI itself doesn't work in a version so old that -webkit-text-security isn't supported. I tested it.

Screenshots or Videos
Before
https://github.com/user-attachments/assets/a516daa0-4f06-431b-b739-2368a119a608
After
https://github.com/user-attachments/assets/da82e099-3eae-4c87-b1fb-d887d2561cbe

Hi, can you explain to me why there was an error in @github-actions
Frontend Build / Format & Build Frontend (pull_request) if not it is a totally different commit

Thanks!


Replace references to expired ezeljs.com domain

I'm thoroughly enjoying the all the posts. Happened across some references to ezeljs.com which seems to be expired and points to a parked domain.

@craigspaeth might want to revive ezeljs


gRPC api service implementation

Note: there's no tests for the grpc service yet and the tests that were already there are probably broken... I'll go back and figure out how to add them once I get some feedback

Couple of things I'm not sure about:

There are 4 implementations of an Item:

Hacker news client
Database model
Protobuf
Gateway response

I'm not sure if this is the best way to approach it? It seems like there needs to be that many versions because it's used for different purposes in each place. e.g. Hacker News client contains the Dead and Deleted fields which the DB doesn't care about as they're filtered. Likewise the Protobuf implementation is kind of unavoidable. Any thoughts would be appreciated.
Tangentiality, when converting the item from one type to the other I've followed a similar pattern I've seen in Go (e.g Atoi) but I'm not sure on the placement of these util like functions. For the gateway service I've put it next to the Item struct but for the api service I've put it next to the handler functions.

Worth noting I've created another PR which adds a caching layer to the API service. The PR includes refactoring the structs relying on concrete types to use interfaces instead.  https://github.com/alexdunne/gs-onboarding/pull/4/files

I think you could align your Item model, as its pretty much the same apart from int32 in some places and int in others
Maybe pull it out into a models package

I've made those changes to this branch. Going to close this PR as I've rebased them on to the other branch https://github.com/alexdunne/gs-onboarding/pull/4