Canto GO annotations missing from GAF?

The GAF shrunk abruptly between pombase-build-2016-09-15 and pombase-build-2016-09-18 builds:
pombase-build-2016-09-11-v1.gaf.gz	2016-09-12 12:10	783K	 [v61 release]
pombase-build-2016-09-15-v1.gaf.gz	2016-09-16 12:42	782K
pombase-build-2016-09-18-v1.gaf.gz	2016-09-19 15:54	570K
pombase-build-2016-10-17-v1.gaf.gz	2016-10-19 08:46	636K	 [latest build as of ticket opening]

I noticed because annotations from PMID:15226425/d2288b20e67eb62d aren't in the latest GAF. Spot-checking a couple of other randomly chosen sessions, and a look at the latest gaf-load-output log, raises the ugly possibility that GO annotations aren't getting loaded from Canto. It doesn't look like a general Canto problem, though because the PHAFs have stayed at consistent sizes.
I added 'high priority' because something's going wrong, but I suppose you could change that because it'll be ages before the next release.

According to the current logs and GAF its the IEA data that isn't loading?
The Canto numbers look slightly higher in the logs.
If this is the case, this should get fixed when
https://github.com/pombase/pombase-chado/issues/594
is implemented

I started looking at this last week but got distracted by fever.  It's very odd that there was a big drop and even odder that the gaf size seems to change quite a lot every load for the last month.  I'll look at this again first thing on Monday.
Recent file sizes:
724K  pombase-build-2016-10-14-v1/pombase-build-2016-10-14-v1.gaf.gz
624K  pombase-build-2016-10-15-v1/pombase-build-2016-10-15-v1.gaf.gz
612K  pombase-build-2016-10-16-v1/pombase-build-2016-10-16-v1.gaf.gz
640K  pombase-build-2016-10-17-v1/pombase-build-2016-10-17-v1.gaf.gz
804K  pombase-build-2016-10-18-v1/pombase-build-2016-10-18-v1.gaf.gz
676K  pombase-build-2016-10-22-v1/pombase-build-2016-10-22-v1.gaf.gz
644K  pombase-build-2016-10-23-v1/pombase-build-2016-10-23-v1.gaf.gz
608K  pombase-build-2016-10-24-v1/pombase-build-2016-10-24-v1.gaf.gz
620K  pombase-build-2016-10-25-v1/pombase-build-2016-10-25-v1.gaf.gz
664K  pombase-build-2016-10-26-v1/pombase-build-2016-10-26-v1.gaf.gz
4.0K  pombase-build-2016-10-27-v1/pombase-build-2016-10-27-v1.gaf.gz
664K  pombase-build-2016-10-28-v1/pombase-build-2016-10-28-v1.gaf.gz
572K  pombase-build-2016-10-29-v1/pombase-build-2016-10-29-v1.gaf.gz

Yep it does seem to be quite random ;) fun!

Sorry it's taken so long to look at this.  The first thing I've noticed is that since 2016-09-18 there are no ND line in the GAF files.  I think that's probably a symptom not the cause.

I think I've tracked down the problem.  There was an error during the GAF file writing.  The error is in the log files but it was a bit buried so it was hard to spot.  The there were a few annotations from Canto that were stored incorrectly in Chado (see #597) which caused the GAF writer to fail.
The annotations were in:
https://curation.pombase.org/pombe/curs/ccbd850b80722817/ro/
They were using FYPO:0003180 "decreased intergenic meiotic recombination" but that ID is now an alt_id for: FYPO:0002485.  That should be fine but the extension storing was going slightly wrong.  I've add #597 about the bug.
As a quick fix I've changed FYPO:0003180 to FYPO:0002485 in that session and I've started a new load.
I'll close this if that fixes things.

there are no ND line in the GAF files

The ND lines are rewritten out at the end and the writer wasn't getting that far - the GAF files were truncated.

As a quick fix I've changed FYPO:0003180 to FYPO:0002485 in that session and I've started a new load.

I forgot to check the other sessions.  Whoops.  I'll try again.

Fixed!

great thanks! Let me know  when  the stats page is next updated.

Let me know when the stats page is next updated.

It's updated now.

thanks


Error building JFoenix demo

Hello, I was trying to run the demo with fresh copy of master branch, and followed the commands given by the Wiki. However, the demo could not be built. Could someone help me figure out what the problem is? ( Windows 10 with JDK9)
Error log:
C:\Users\haoyu>cd C:\Users\haoyu\Desktop\JFoenix-master
C:\Users\haoyu\Desktop\JFoenix-master>gradlew run
Parallel execution is an incubating feature.

Configure project :jfoenix
Gradle now uses separate output directories for each JVM language, but this build assumes a single directory for all classes from a source set. This behaviour has been deprecated and is scheduled to be removed in Gradle 5.0
at build_1mkbzqlgw6ek6ucgcxrdyl9kf$_run_closure6.doCall(C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\build.gradle:35)
(Run with --stacktrace to get the full stack trace of this deprecation warning.)

Task :jfoenix:compileJava
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:48: error: constructor ComboBoxBaseBehavior in class ComboBoxBaseBehavior cannot be applied to given types;
super(colorPicker, JFX_COLOR_PICKER_BINDINGS);
^
required: ComboBoxBase
found: ColorPicker,List
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:66: error: method does not override or implement a method from a supertype
@Override
^
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:73: error: cannot find symbol
super.callAction(name);
^
symbol: method callAction(String)
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:83: error: method does not override or implement a method from a supertype
@Override
^
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:85: error: cannot find symbol
ColorPicker colorPicker = (ColorPicker) getControl();
^
symbol:   method getControl()
location: class JFXColorPickerBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXColorPickerBehavior.java:89: error: method onAutoHide in class ComboBoxBaseBehavior cannot be applied to given types;
super.onAutoHide();
^
required: PopupControl
found: no arguments
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXDatePickerBehavior.java:45: error: constructor ComboBoxBaseBehavior in class ComboBoxBaseBehavior cannot be applied to given types;
super(datePicker, JFX_DATE_PICKER_BINDINGS);
^
required: ComboBoxBase
found: DatePicker,List
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXDatePickerBehavior.java:57: error: cannot find symbol
JFX_DATE_PICKER_BINDINGS.addAll(COMBO_BOX_BASE_BINDINGS);
^
symbol:   variable COMBO_BOX_BASE_BINDINGS
location: class JFXDatePickerBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXDatePickerBehavior.java:66: error: method does not override or implement a method from a supertype
@Override
^
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXDatePickerBehavior.java:68: error: cannot find symbol
DatePicker datePicker = (DatePicker) getControl();
^
symbol:   method getControl()
location: class JFXDatePickerBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXDatePickerBehavior.java:72: error: method onAutoHide in class ComboBoxBaseBehavior cannot be applied to given types;
super.onAutoHide();
^
required: PopupControl
found: no arguments
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTimePickerBehavior.java:45: error: constructor ComboBoxBaseBehavior in class ComboBoxBaseBehavior cannot be applied to given types;
super(timePicker, JFX_TIME_PICKER_BINDINGS);
^
required: ComboBoxBase
found: JFXTimePicker,List
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTimePickerBehavior.java:57: error: cannot find symbol
JFX_TIME_PICKER_BINDINGS.addAll(COMBO_BOX_BASE_BINDINGS);
^
symbol:   variable COMBO_BOX_BASE_BINDINGS
location: class JFXTimePickerBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTimePickerBehavior.java:66: error: method does not override or implement a method from a supertype
@Override
^
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTimePickerBehavior.java:68: error: cannot find symbol
JFXTimePicker datePicker = (JFXTimePicker) getControl();
^
symbol:   method getControl()
location: class JFXTimePickerBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTimePickerBehavior.java:72: error: method onAutoHide in class ComboBoxBaseBehavior cannot be applied to given types;
super.onAutoHide();
^
required: PopupControl
found: no arguments
reason: actual and formal argument lists differ in length
where T is a type-variable:
T extends Object declared in class ComboBoxBaseBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTreeTableCellBehavior.java:40: error: cannot find symbol
final TreeItem treeItem = getControl().getTreeTableRow().getTreeItem();
^
symbol:   method getControl()
location: class JFXTreeTableCellBehavior<S,T>
where S,T are type-variables:
S extends Object declared in class JFXTreeTableCellBehavior
T extends Object declared in class JFXTreeTableCellBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\controls\behavior\JFXTreeTableCellBehavior.java:42: error: cannot find symbol
final Node disclosureNode = getControl().getTreeTableRow().getDisclosureNode();
^
symbol:   method getControl()
location: class JFXTreeTableCellBehavior<S,T>
where S,T are type-variables:
S extends Object declared in class JFXTreeTableCellBehavior
T extends Object declared in class JFXTreeTableCellBehavior
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\skins\JFXProgressBarSkin.java:132: error: cannot find symbol
if (getSkinnable().impl_isTreeVisible()) {
^
symbol:   method impl_isTreeVisible()
location: class ProgressIndicator
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\skins\JFXSpinnerSkin.java:67: error: BehaviorBase is abstract; cannot be instantiated
super(control, new BehaviorBase(control, Collections.emptyList()));
^
C:\Users\haoyu\Desktop\JFoenix-master\jfoenix\src\main\java\com\jfoenix\skins\JFXSpinnerSkin.java:122: error: cannot find symbol
if (getSkinnable().impl_isTreeVisible()) {
^
symbol:   method impl_isTreeVisible()
location: class JFXSpinner
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
21 errors

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':jfoenix:compileJava'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Get more help at https://help.gradle.org

BUILD FAILED in 2s
4 actionable tasks: 2 executed, 2 up-to-date

I'm not experienced with the JDK9 build.
Try building the JDK 8 demo. If that works then we can confine the problem to the JDK 9 branch.

@TurekBot Sorry JDK version should come to my mind first but I forgot. Just tried JDK8 out and it runs perfectly, seems that the issue is related to the newer JDK. Thank you.

Hello,
JFoenix demo is not ported to Java 9 yet, only JFoenix library is ported to java 9.
It's because the demo contains some dependencies (DataFX) that are not supported in java 9 yet.
Regards,

@jfoenixadmin Thank you for letting me know!


Recursion in CDBResult class>>#fromDictionary:

CDBResultSet inherits from CBDResult and does not implement #fromDictionary:

so the code in CDBResult class>>#fromDictionary:
fromDictionary: dict

	^ ((dict at: #ok)
		   ifTrue: [ CDBResultSet ]
		   ifFalse: [ CDBErrorResult ]) fromDictionary: dict

leads to a recursion: when #ok is flagged in the dict then CDBResultSet is picked as the class and again #fromDictionary: is sent which is again found in the superclass CDBResult class>>#fromDictionary:
You can already reproduce with Pharo 11 or 12 and the sample given in the README

Refactor till death ;-) Or: True, oh so true.
I wanted the trait-introduced functionality to be inherited, and didn't realize the  pre-existing constructor method broke it.
Instead of using the trait in subclasses, I decided to rename the factory method to fromResponse:, as the cozo response is pretty specific dict anyways.
Fix incoming.

Message to self: although the method was deleted, is still shows in search under original class with original text. Pinpoint & create issue.


Speakers' SNS links could be displayed by default

Overview (Required)
Now, once a speaker view is opened, one has to click once more on the user icon to display its SNS links. I don't think that's really intuitive and I didn't know we could do that before I saw this tweet. How about just always displaying it as a bottom view?

I think so
If we can implement cool ui, we should do it👍


which paper related this project?

https://arxiv.org/abs/1703.03130
A Structured Self-attentive Sentence Embedding


Energy sensor created from power sensor does not seem to work.

Hi,
followed the documentation, and created energy sensors based on existing power sensors, but they do not seem to increase. by comparison, i have some Riemann sum sensors set up already, and they increase as expected, as do all the energy sensors Powercalc created on it's own.
i have not seen anything in the logs that would indicate malfunction.
additional suggestion: when setting up the riemann sensors, i had to add a constantly changing attribute to some of my power templates, otherwise riemann did not update.
please tell me if i can provide any further data, log or other form of debugging.

What does your configuration look like? What kind of integration provides this existing power sensor you are mentioning?

i think it's worth to mention that when i used the same power sensors via a YAML configured Riemann integration, it did not update even after 10 hours, but when i created a Riemann integration sensor via the GUI, it is updating in every 2-3 minutes. i have no idea why, everything was correct based on what i read in the docs and found as examples all over the internet.
example:
  - platform: integration
    name: 'Kitchen refrigerator energy'
    unique_id: eb6d59cf-ca3d-4b3e-b94a-f49df4ebf380
    source: switch.kitchen_refrigerator_power
    unit_prefix: k
    method: left

currently i do not know how to check what the GUI created, so i can't compare.

I have no idea why the riemann sensor created with YAML is having problems on your machine. I'm glad you actually have the same behaviour with the riemann sensor you created yourself and the one created with powercalc, so it seems to be an HA core issue and not a powercalc issue per se.
Could you have a look in the developer tools and compare attributes for the one you created with GUI and with YAML?

Closed due to inactivity. Let me know if you still need any help.


Assign the appropriate stackPipe or unitId

Issue:
When a user is updating existing Test Data or adding new Test Data, the system throws a validation error or check that the end year is not applicable even when the end year is within the applicable range of 1993-2022. This is happening even when the End Date is correct or within the specified range.
Upon investigation, it was discovered that when creating a test summary record, the payload should have the appropriate stackPipe or unitId assigned. However it is currently only assigning unitId even though the location is a stackPipeId

Bug Fixed


Access to the ICA dataset

Hello,
I was wondering if you were perhaps planning on making the ICA annotations of the OASIS-3 scans publicly available.
I am working on test time adaptation for segmentation, and having access to a labeled ICA dataset would be a great help.
Best regards,
Juan

Hi Juan,
Thank you for your interest. Unfortunately, we’re not able to make the ICA annotations of the OASIS-3 scans publicly available at this time. For the time being, I would recommend testing your methods on the dataset provided by TopCoW2023. It also focuses on multi-class segmentation tasks on cerebral vasculature.
Best regards,
Pengcheng

Hi Pengcheng,
I understand, thanks for the information on the TopCoW challenge!
Best,
Juan


Support for reqwest 0.12 ?

The graphql_client optional dependency for reqwest-create seems locked to 0.11
https://github.com/graphql-rust/graphql-client/blob/main/graphql_client/Cargo.toml#L23
What's needed to support use with 0.12?
I currently have issues because of conflicting versions if I try to use 0.12 in a project with reqwest and graphql_client

Any progress/plans on this?

Nobody is working on it, but I could review a PR. We need different feature flags for different versions of reqwest.

Ahh, I opened a PR #490 but I'll add feature flags

PR https://github.com/graphql-rust/graphql-client/pull/494 is better, should be able to merge as is and release a patch version


Newly appointed admin has incorrect admin status

In PROD:

created new circle (circle 11). circle admin is user 179.
then I use the join URL for circle 11 and add another user (no approval required).
the new helper is successfully added to the circle as user 180.
then I have the current admin also add the new user for the role of admin. The status of this role is 'nil' hence the display is broken.
Also the button to remove the admin role from user 180 is not displayed, which should be possible since we still have one valid admin.

Todos:

[x] set a new admin role to active by default
[x] only the last active admin role can't be deleted; blocked admin roles can be deleted.

@dottorer implemented; can be reviewed on staging.

@phillipoertel Tested in STG in circle 56. Works great now. Did we fix the existing PROD, DEMO and STG instances via SQL?

All instances have been fixed in STG, PROD and DEMO by SQL. Closing.


Add support for zh_cn language

Since first local tests of adding support for zh_cn seem promising, I'd like to make the following request: could you please be so kind as to provide me with proper values for CollisionPrefixes and CustomTranslations? Just orient yourself on their English counterparts; it would also be great if the custom translations don't collide with any other names (materials, effects, enchantments, ...), but if that's impossible, there are collision-prefixes after all, so it wouldn't be a big deal. Thanks for the effort in advance! :)

Let me provide some additional context. The translations for negation (not), disjunction (or) and conjunction (and) are logical operators and thereby self-explanatory, while amount refers to the stack-size; deterioration is documented here, and you can read up on exact-mode here - this should help to get a better idea of what CustomTranslations are referring to.
CollisionPrefixes are also quite straightforward, with possibly the exception of forInstruments, which refers to the just recently introduced MusicInstrument; even though it's only used for goat-horns as of now, I'd not refer to that in the translation - it may be extended in the future.
For your reference, here's the corresponding language-file, in case you want to check against collisions by searching through it zh_cn.json; used keys mainly start with enchantment.minecraft, item.minecraft. or block.minecraft., effect.minecraft. and instrument.minecraft..

My brain tries to understand but fails anyway
How do you think #3 looks?

Honestly, I was really surprised by how perfectly you've implemented those changes first-try - great job! I'm super happy with it.
The only question I have would be whether there's no proper translation for "Music Instrument"? If that's the case, I'm fine with accepting your PR the way it currently is, :). We just really shouldn't change it down the road - these values are intended as constants.

Honestly, I was really surprised by how perfectly you've implemented those changes first-try - great job! I'm super happy with it.老实说，我真的很惊讶你们在第一次尝试时就如此完美地实施了这些更改 - 干得好！我对此非常满意。
The only question I have would be whether there's no proper translation for "Music Instrument"? If that's the case, I'm fine with accepting your PR the way it currently is, :). We just really shouldn't change it down the road - these values are intended as constants.我唯一的问题是“乐器”是否没有合适的翻译？如果是这样的话，我可以按照目前的方式接受你的 PR，:)。我们真的不应该在以后改变它——这些值被设计为常量。

Where would this value be applied? I'd like to test it.

Honestly, I was really surprised by how perfectly you've implemented those changes first-try - great job! I'm super happy with it.
The only question I have would be whether there's no proper translation for "Music Instrument"? If that's the case, I'm fine with accepting your PR the way it currently is, :). We just really shouldn't change it down the road - these values are intended as constants.

Never mind, Just finished PR
These translations may be modified by future users, as they don't look very good to me

I've written a few lines about collision prefixes on a documentation page. Since we're still in very early development, I think that it's okay if we need to change them before the proper release. Just later on, when predicates actually become persisted, like on Pipe-Filters, they shouldn't necessarily change anymore, as people might depend on them, and would have to update their inputs manually otherwise.
Google translator responds with spot-on english equivalents, so it looks good to me, LOL. I'll merge it and provide you with a dev-build in just a second, so you can test it out before I make an actual release. Maybe there are still oddities to fix.

Please give the latest release a thorough inspection and try to search materials, enchantments, potions, etc. on your local server in Chinese; feel free to report any odd behavior! :)

Seems everything is working now!


Adds 'aad user license list' command. Closes #4502

Adds aad user license list command. Closes #4502

Great work @nanddeepn! We'll try to review it ASAP!

Merged manually. Awesome work! Thank you for this contribution! 👏


GM: don't consider soft disabling as ACC fault

In dashcam mode, if stock ACC is braking for a car and the user presses the cancel button (taps or holds), ACC stays engaged for up to 2.8 seconds (observed) and a warning sound and lights are played (FCW severity) while the car tries to bring you to a stop before it disengages.
The entire time, the cruise state is "faulted," however we usually take faulted to mean we caused an issue with the car. In this instance though, faulted is a state where the car is soft disabling and continuing longitudinal actuation as normal.

If the data checks out, does this sound like an appropriate fix to you @adeebshihadeh?
The alert from the car is another issue I think, and that involves finding a different cancel command (or looking into what threshold user brake disengages ACC, is it really adjusted by how much the car is currently braking?)

These aren't considered in the MTBF report


Go keywords for C function

Add test for next function name:
chan
defer
fallthrough
func
go
import
interface
map
package
range
select
type
var
_

Is this now fixed with #419 ?


Add test for CASSANDRA-10806

Added test to check sstableloader works with upper case keyspace name.
This is test for CASSANDRA-10806.

After addressing my comments, I believe the code will be fine. It looks like CASSANDRA-10806 is on track to be merged soon, so I think we should hold off on merging this PR until then?

thanks for review. I will commit 10806 first then.

Merged CASSANDRA-10806.

Great. I've run this locally and it passes; if you'll address my comments and push your changes, I can run again locally and merge if they work.

Still passing after the changes, merging.


PhpRedisConnector is used instead of PhpRedisSentinelConnector

By using below definition for default and expecting it to use the PhpRedisSentinelConnector, I am getting error
In PhpRedisConnector.php line 232: Undefined array key "host" , so it seems that the new connector is not used.
Is there anything additional that needs to be set up?
    'default' => [
        'host' => env('REDIS_SENTINEL_HOST', '<IP_ADDRESS>'),
        'port' => env('REDIS_SENTINEL_PORT', 26379),
        'sentinel_service' => env('REDIS_SENTINEL_SERVICE', 'mymaster'),
        'sentinel_timeout' => env('REDIS_SENTINEL_TIMEOUT', 0),
        'sentinel_persistent' => env('REDIS_SENTINEL_PERSISTENT'),
        'sentinel_retry_interval' => env('REDIS_SENTINEL_RETRY_INTERVAL', 0),
        'sentinel_read_timeout' => env('REDIS_SENTINEL_READ_TIMEOUT', 0),
        'sentinel_username' => env('REDIS_SENTINEL_USERNAME'),
        'sentinel_password' => env('REDIS_SENTINEL_PASSWORD'),
        'password' => env('REDIS_PASSWORD'),
        'database' => 0,
        'client' => 'phpredis-sentinel',
    ],

I use a separate single instance for session that continues to use the phpredis client
    'session' => [
        'url'      => env('REDIS_SESSION_URL', env('URL')),
        'host'     => env('REDIS_SESSION_HOST', env('HOST', '<IP_ADDRESS>')),
        'password' => env('REDIS_SESSION_PASSWORD', env('PASSWORD', null)),
        'port'     => $redis_port,
        'database' =>  env('DATABASE', 0),
        'client' => 'phpredis',
    ]

seems that if client is placed above session and default and not inside their blocks it works, so using different clients will not be so straight forward.

Yes, that's not easy to achieve. The RedisManager is itself instantiated with a single driver that is used for all connections:
https://github.com/laravel/framework/blob/79c9874700427e0e469fb3f56c5f072342263e73/src/Illuminate/Redis/RedisManager.php#L69-L74
You could in theory inject a custom, second RedisManager with a different driver into the service container. However, using this other connection manager in other parts of the framework (e.g. cache) would be very challenging in my opinion.

After setting universal client as seen below:
'redis' => [
'client' => env('REDIS_CLIENT', 'phpredis-sentinel'),
    'default' => [
        'sentinel_host' => env('REDIS_SENTINEL_HOST', '<IP_ADDRESS>'),
        'sentinel_port' => (int) env('REDIS_SENTINEL_PORT', 26379),
        'sentinel_service' => env('REDIS_SENTINEL_SERVICE', 'mymaster'),
        'sentinel_timeout' => env('REDIS_SENTINEL_TIMEOUT', 0),
        'sentinel_persistent' => env('REDIS_SENTINEL_PERSISTENT'),
        'sentinel_retry_interval' => env('REDIS_SENTINEL_RETRY_INTERVAL', 0),
        'sentinel_read_timeout' => env('REDIS_SENTINEL_READ_TIMEOUT', 0),
        'sentinel_username' => env('REDIS_SENTINEL_USERNAME'),
        'sentinel_password' => env('REDIS_SENTINEL_PASSWORD'),
        'password' => env('REDIS_PASSWORD'),
        'database' => 0,
    ],
    ....

]
I am getting this error:
ERROR:  In PhpRedisConnector.php line 161:
php_network_getaddresses: getaddrinfo for redis-0.redis failed: Name or service not known
Why PhpRedisConnector is used while I have defined the phpredis-sentinel client?
Shouldn't only the  PhpRedisSentinelConnection.php be used?

Looks like your sentinels return incomplete FQDNs, maybe check their config and add the .svc.cluster.local part.
PhpRedisConnection being called is fine. PhpRedisSentinelConnection is just a wrapper around the built-in driver that first resolves the Redis address from the Sentinels, then falls back to the default implementation using that address.


Argument 1 passed to League\OAuth2\Server\Entity\AbstractTokenEntity::__construct() must be an instance of League\OAuth2\Server\AbstractServer, null given

I am trying to implement an oauth2 in lumen.
Everything is working fine.
I am trying to revoke a token and I have concluded that to revoke a token I need to remove all the token related to refresh token as well.
I created an instance of  League\OAuth2\Server\Storage\RefreshTokenInterface
I added this to the config file;
'refresh_token' => [         'class' => 'League\OAuth2\Server\Storage\RefreshTokenInterface'     ]
$config = Config::get('oauth2');         $class = $config['refresh_token']['class'];         $input = app()->make('request')->all();         $refreshToken = $input['token'];         $refreshTokenObj = app()->make( $class )->get( $refreshToken );
I am trying to get the instance of the refreshtoken. I coudl directly write a query but I am trying to do this the right way.
I get this error on get() method
Argument 1 passed to League\OAuth2\Server\Entity\AbstractTokenEntity::__construct() must be an instance of League\OAuth2\Server\AbstractServer, null given
Also, i get this error if I try to delete the refresh token
Argument 1 passed to LucaDegasperi\OAuth2Server\Storage\FluentRefreshToken::delete()
Please help.

You should implement token deletion all by yourself, there's no delete method on the FluentRefreshToken as it is not an Eloquent record.

I tried to do this like this because there seems to be a class
public function get($token)
{
    $result = $this->getConnection()->table('oauth_refresh_tokens')
            ->where('oauth_refresh_tokens.id', $token)
            ->where('oauth_refresh_tokens.expire_time', '>=', time())
            ->first();

    if (is_null($result)) {
        return null;
    }

    return (new RefreshTokenEntity($this->getServer()))
           ->setId($result->id)
           ->setAccessTokenId($result->access_token_id)
           ->setExpireTime((int)$result->expire_time);
}

there is this class FluentRefreshToken and I am trying use this get method and by my code I see that the query is successfully executed and it is just that it is having a hard time creating this
new RefreshTokenEntity($this->getServer())  class. If this accomplished I could use the delete method in the same class.I am trying to do this like this to consolidate my codes. Is this possible. is there any solution or do I have to do this all by myself? I i could just get the RefreshTokenEntity class to successfully create, I seem to be able to do this.

It is having a hard time because you would need to pass an AbstractServer instance in the constructor.

Not exactly, I looked into the codes and I see no way that I can pass the server instance, it seems to fetch it itself using some other class. There is nothing I can do without changing the source code. The class is failing to create the server instance. If that somehow can be fixed, my code would work flawless but i am not sure how to do that.

In the first place why are you revoking an access token by using a refresh token? to extend the functionality of the provided storage classes try to subclass them and register them in the IoC to replace the one you are extending.

first things is I am totally new to this implementation and I am learning as I go.
I am using this in one of my angular apps and using angular-oauth2 for that. When it hits the revokeAccessToken method, it seems to send the token as post. I have the access token in header. My first instinct was that I would simply delete the access token from the table but since I got refresh token in post as 'token', i thought it should be "right" to delete that to delete all the tokens related to that( they seem to cascade).
Like I said, I am learning as i am implementing this. I tried to google the "correct" way to revoke an access token but could not find a good resource on that so I thought this is the "right" way because it made sense to me and trying to implement that and also I looked around in the classes and there were methods and classes that did that, so just trying things out, the things that seemed better.

The refresh token is just a "temporary" type of token and is always associated to an access token, in order to revoke them both, you would need to delete the session associated with the access token. Plus there's a specification by the IETF for explicitly revoking tokens. http://tools.ietf.org/html/draft-ietf-oauth-revocation-06

Thanks for the link, its helpful
I will create a class/system based on this.
And thanks for the quick replies. You are the best :)


How to replace Example from XBehave with Autofixture's AutoMoqDataAttribute

Hello,
I am trying to use an AutoMoqDataAttribute with my XBehave scenarios but it seems that when I do this I use the benefit of having different step information showing up on the Resharper or MsTest windows.

usually to provide data to your tests in Xbehave you use an Example attribute but I want to be able to provide already setup test data using autofixture.
I can see someone has had a similar issue here and they have created this gist, but I am not sure how this will help or how it is supposed to be wired up to auto fixture.
here is my code:
[Scenario]
        [AutoMoqData]
        public void CorrelationId_WhenCorrelationIdIsPassedIn(
            string result,
            string correlationId,
            [Frozen(Matching.DirectBaseType)] DefaultHttpContext httpContext,
            [Frozen] Mock<IHttpContextAccessor> httpContextAccessorMock,
            CorrelationIdProvider sut)
        {
            $"Given a correlation id header value of {correlationId} has been passed in"
                .x(() =>
                {
                    httpContext.Request.Headers["x-Correlation-ID"] = correlationId;
                    httpContextAccessorMock.Setup(x => x.HttpContext).Returns(httpContext);
                });

            "When the correlation id is retrieved"
                .x(() => result = sut.CorrelationId);

            "Then the correlation id is not empty"
                .x(() => result.Should().NotBeEmpty());

            $"And the correlation id matches {correlationId}"
                .x(() => result.Should().Be(result));
        }

 public class AutoMoqDataAttribute : AutoDataAttribute
    {
        public AutoMoqDataAttribute() :
            base(() => new Fixture().Customize(new AutoMoqCustomization()))
        {
        }
    }

Hi @ftaran, thanks for raising this. I'm not entirely sure I understand the problem. As long as the attribute you are using derives from xunit's DataAttribute, everything should just work.

@adamralph
Is it possible to unseal the example attribute so that I can extend it? This way I will be able to automatically generate test data and get detailed info in the test window.

@ftaran you can inherit directly from Xunit.Sdk.DataAttribute.
As you can see from the source, ExampleAttribute is a trivial derivation of DataAttribute.

Ah hey. Yes, I wrote that Gist.
So I extended AutoData:
[AttributeUsage(AttributeTargets.Method, AllowMultiple = true)]
public class CustomAutoDataAttribute : AutoDataAttribute
{
  public WebhostAutoDataAttribute(params object[] data) : base(() =>
  {
    var fixture = new Fixture();

    fixture.Customize(new ExampleAutoDataSpecimenBuilder(data));
  }
}

with ExampleAutoDataSpecimenBuilder being from the above-mentioned gist. ExampleAutoDataSpecimenBuilder infers the parameter position in the target method then uses that to find the data by index in the object[] provided.

@ftaran have you managed to acheive what you want?

@ftaran I assume that, given the above info, you have enough to things working for you now. If not, please feel free to re-open this issue or raise another one.


Adding support for DynamoDB backend

Picking up where https://github.com/hashicorp/terraform-aws-vault/pull/77 left off to resolve https://github.com/hashicorp/terraform-aws-vault/issues/76

Adds functionality for DynamoDB HA storage backend in the run-vault script.
Documented usage in README.
Adds Dynamo example implementation

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

Stephen Coe seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.
Have you signed the CLA already but the status is still pending? Recheck it.

Are there any Consul params that now need to be made optional for users who go with DynamoDB?

The main vault-cluster module doesn't contain any specifics for consul. The root example does but since this is only an example I didn't see the need to make any changes

Getting test failures such as:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:51Z save_test_data.go:168: Storing test data in /tmp/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami311063910/terraform-aws-vault/examples/vault-dynamodb-backend/.test-data/TerraformOptions.json so it can be reused later
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:51Z retry.go:72: terraform [init -upgrade=false]
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:51Z command.go:87: Running command terraform with args [init -upgrade=false]
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:58Z command.go:158: Initializing modules...
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:58Z command.go:158: - backend in
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: - vault_cluster in ../../modules/vault-cluster
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: Error: Unreadable module directory
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: Unable to evaluate directory symlink: lstat backend: no such file or directory
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: Error: Failed to read module directory
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: Module directory  does not exist or cannot be read.
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: - vault_cluster.security_group_rules in ../../modules/vault-security-group-rules
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158:
TestMainVaultCluster/group/TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu18Ami 2020-02-12T11:47:59Z command.go:158: Error: Unreadable module directory

Hey @brikis98 👋
Thanks for running this through CI! I have fixed the issue and run those tests again locally to verify.
I had an issue running the tests originally, the AMI would come up in say us-west-1 but when it came to run the apply it was reading the region from my ~/.aws.config, in my case eu-west-1.
Looking at the code the same region is passed through. I ended up deleting my local region but wondered if it was something you had come across in terratest?

Strange. We explicitly pick a random region for each test run, and we set it as an environment variable. Perhaps local AWS config overrides the environment variable? I'm not sure.
I'll kick off the tests again shortly.

The yaks continue. This time, it looks like the test suite timed out after 45 min and Go killed it off 😢
The test suite used to take 25 - 35 min... Perhaps the new tests pushed something over the edge so it's a lot slower now?
That said, the panic stack trace includes specifically the new tests:
goroutine 63 [chan send, 15 minutes]:
testing.tRunner.func1(0xc00be57300)
	/usr/local/go/src/testing/testing.go:822 +0x265
runtime.Goexit()
	/usr/local/go/src/runtime/panic.go:397 +0xef
testing.(*common).FailNow(0xc00be57300)
	/usr/local/go/src/testing/testing.go:590 +0x39
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/stretchr/testify/require.NoError(0x18b56a0, 0xc00be57300, 0x18ad8c0, 0xc000020150, 0x0, 0x0, 0x0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/stretchr/testify/require/require.go:800 +0xe8
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/terraform.Destroy(0xc00be57300, 0xc0006f39a0, 0x7e, 0xc0006f39a0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/terraform/destroy.go:12 +0x8c
github.com/hashicorp/terraform-aws-vault/test.teardownResources(0xc00be57300, 0xc0006f0080, 0x7e)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_helpers.go:72 +0x5b
github.com/hashicorp/terraform-aws-vault/test.runVaultWithDynamoBackendClusterTest.func1()
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_cluster_dynamodb_backend_test.go:27 +0x3c
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure.RunTestStage(0xc00be57300, 0x16ca438, 0x8, 0xc008261e20)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure/test_structure.go:24 +0x208
runtime.Goexit()
	/usr/local/go/src/runtime/panic.go:397 +0xef
testing.(*common).FailNow(0xc00be57300)
	/usr/local/go/src/testing/testing.go:590 +0x39
testing.(*common).Fatal(0xc00be57300, 0xc008260dc0, 0x1, 0x1)
	/usr/local/go/src/testing/testing.go:628 +0x6f
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/aws.GetInstanceIdsForAsg(0xc00be57300, 0x0, 0x0, 0xc000457c60, 0xc, 0xc008260e58, 0xa10ef2, 0x7f41789591c0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/aws/asg.go:59 +0xdd
github.com/hashicorp/terraform-aws-vault/test.getIpAddressesOfAsgInstances(0xc00be57300, 0x0, 0x0, 0xc000457c60, 0xc, 0x0, 0xd08, 0x100)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/aws_helpers.go:11 +0x74
github.com/hashicorp/terraform-aws-vault/test.findVaultClusterNodes(0xc00be57300, 0x16da8d3, 0x16, 0x16c8dfd, 0x6, 0xc00be79220, 0xc000457c60, 0xc, 0xc00044ffb0, 0x0, ...)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_helpers.go:194 +0xc7
github.com/hashicorp/terraform-aws-vault/test.writeOutVaultLogs(0xc00be57300, 0x16da8d3, 0x16, 0x16c8dfd, 0x6, 0xc00be79220, 0xc000457c60, 0xc, 0xc00044ffb0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_helpers.go:153 +0xde
github.com/hashicorp/terraform-aws-vault/test.getVaultLogs(0xc00be57300, 0x16e5160, 0x1d, 0xc00be79220, 0xc000469040, 0x15, 0xc000457c60, 0xc, 0x16c8dfd, 0x6, ...)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_helpers.go:124 +0xe9
github.com/hashicorp/terraform-aws-vault/test.runVaultWithDynamoBackendClusterTest.func2()
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_cluster_dynamodb_backend_test.go:34 +0x153
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure.RunTestStage(0xc00be57300, 0x16c75ff, 0x3, 0xc008261ec0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure/test_structure.go:24 +0x208
runtime.Goexit()
	/usr/local/go/src/runtime/panic.go:397 +0xef
testing.(*common).FailNow(0xc00be57300)
	/usr/local/go/src/testing/testing.go:590 +0x39
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/stretchr/testify/require.NoError(0x18b56a0, 0xc00be57300, 0x18ad8c0, 0xc0002ecd00, 0x0, 0x0, 0x0)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/stretchr/testify/require/require.go:800 +0xe8
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/terraform.InitAndApply(0xc00be57300, 0xc00be97e00, 0x7e, 0xc00be97e00)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/terraform/apply.go:14 +0x8c
github.com/hashicorp/terraform-aws-vault/test.deployCluster(0xc00be57300, 0xc000469040, 0x15, 0xc000457c60, 0xc, 0xc0006f0080, 0x7e, 0xc0000115a0, 0x6, 0xc008261b80)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_helpers.go:120 +0x4f7
github.com/hashicorp/terraform-aws-vault/test.runVaultWithDynamoBackendClusterTest.func3()
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_cluster_dynamodb_backend_test.go:41 +0x198
github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure.RunTestStage(0xc00be57300, 0x16c8a97, 0x6, 0xc0006b1e40)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vendor/github.com/gruntwork-io/terratest/modules/test-structure/test_structure.go:24 +0x208
github.com/hashicorp/terraform-aws-vault/test.runVaultWithDynamoBackendClusterTest(0xc00be57300, 0xc000469040, 0x15, 0xc000457c60, 0xc, 0x16c8dfd, 0x6)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_cluster_dynamodb_backend_test.go:37 +0x2ac
github.com/hashicorp/terraform-aws-vault/test.runTestsOnDifferentPlatforms.func1(0xc00be57300)
	/home/circleci/.go_workspace/src/github.com/hashicorp/terraform-aws-vault/test/vault_main_test.go:156 +0x1f8
testing.tRunner(0xc00be57300, 0xc00003b140)
	/usr/local/go/src/testing/testing.go:827 +0xbf
created by testing.(*T).Run
	/usr/local/go/src/testing/testing.go:878 +0x353

So perhaps something in the new tests is hanging?
At any rate, to avoid Go doing a panic in the future, could you update the call to run-go-tests here with  --timeout 90m?

It does look like a timeout but yes suspicious it stopped at those tests. I can see it passed for the previous AMIs, TestVaultWithDynamoDBBackendWithvaultOpenSourceOnUbuntu16Ami as an example. I will do some more checking locally against the AMIs specifically the one that failed to see if anything breaks before changing the timeout

OK, I found what I think is the cause of the panic. Non-unique names for dynamo were being used which is causing an error when running multiple AMI types.

Thx! I'll kick off tests again shortly.

Tests passed! I'll merge this now. Thanks!

https://github.com/hashicorp/terraform-aws-vault/releases/tag/v0.13.6

Hello @stephencoe, could you please add the architecture-with-dynamodb.png under _docs folder. This will make the README.md even better !
Best,


Fix #2272: Write unit tests for the IR printer.

This includes a couple of tweaks to what is actually printed, to make things more beautiful.

I guess @gzm0 will want to review this :p

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3036/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3940/
Test FAILed.

Thanks for doing this! I'll try and review this tonight.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3037/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3941/
Test FAILed.

That's all. Thanks!

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3039/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3943/
Test PASSed.

Updated.

LGTM. Thanks!

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3040/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/3945/
Test PASSed.


Inference with BOP Challenge 2024 HOPE Dataset - Missing keys_to_shard.json

Hi,
Thank you for your great work.
I was trying to inference your model with the BOP Challenge 2024 dataset "HOPE" but it does not seem to have the keys_to_shard.json file (possibly due to dataset version changes?).
As a result, the inference code does not work.
Is there any way or suggestions to use your model with the 2024 HOPE dataset?
If I've misunderstood something or if there are additional steps I need to follow, your guidance would be greatly appreciated.
Thank you for your help!

I think even lmo dataset does not have keys_to_shard.json file too

Hi, Seunggyu
Could you check how the dataset was obtained? If it was downloaded directly through Hugging Face or the BOP website, it’s normal for it not to be included.
When you download it using the script below, as mentioned in the ReadME, it calls two conversion scripts, and one of them, convert_imagewise_to_webdataset.py, generates the key_to_shard.json file.
python -m src.scripts.download_test_bop23


Raster clipping service [EP-3541]

By submitting a PR to this repository, you agree to the terms within the Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.
Description

Describe the purpose of this PR along with any background information and the impacts of the proposed change. For the benefit of the community, please do not assume prior context.
Provide details that support your chosen implementation, including: breaking changes, alternatives considered, changes to the API, etc.

References

Include any links supporting this change such as a:

GitHub Issue/PR number addressed or fixed
StackOverflow post
Support forum thread
Related pull requests/issues from other repos

If there are no references, simply delete this section.

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.
Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.
Also include details of the environment this PR was developed in (language/platform/browser version).

[ ] This change adds test coverage for new/changed/fixed functionality

Checklist

[x] I have added documentation for new/changed functionality in this PR or in /docs
[x] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used, if not master

:tada: This PR is included in version 1.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Merge client fix to dev

Merge if client fixes are complete in preparation for heroku setup? @Dlearn

I have merged it, but I'm still not sure exactly how branches work in github.

github is just a remote repo for git with added flavour like pull requests and comment threads like these


Remove axis kwarg support from linalg.matrix_norm

This PR

resolves https://github.com/data-apis/array-api/issues/318 by removing support for the axis kwarg in linalg.matrix_norm. The current specification derives its signature from a previously unified norm API which had to cater to both vector and matrix use cases. In the current specification, matrix_norm is unique among linalg APIs in supporting an axis kwarg for an API operating on stacks of matrices. And as this API is relatively new among array libraries, seems most prudent to hold off including an axis kwarg until array libraries have come to a consensus that providing axis support is desirable. In general, easier to add to the standard, rather than take something away.

cc @asmeurer for visibility.

Will merge. We can revisit the inclusion of an axis kwarg in the 2022 revision of the standard.


Method History Browser improvements

Speed up change file scanning; reuse existing source when restoring methods. Plus associated functional and cosmetic changes.
Closes #999.

Sure, no problem. Will resubmit with a bugfix and tests.


Update active-directory-aadconnect-troubleshoot-objectsync.md

Hi - I added a link to our youtube video - can you please review and approve the PR?

@RobdeJong : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.

#sign-off

@RobdeJong : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.


All classification results are 0.5s

Hi, I am new in learning Xgboost, and i have a question like this:
when I try to use xgboost to classify some data ,such as when [0,0,0,0,0,0] label is [1]
when [1 1 1 1 1 1],label is [0],but all the classificaion  resulta are 0.5s.
I know it must be my mistakes when use it, it has maken me confused for 1 week, Thank you for your tiime. ,please help me about this when you are convenience.
And this is my code:
`import  xgboost as xgb
import numpy as np
from sklearn.cross_validation import train_test_split
from sklearn.metrics import f1_score
random_seed=333
params={
'booster':'gbtree',
'objective': 'binary:logistic',
'early_stopping_rounds':1000,
'gamma':0.1,
'max_depth':10,
'lambda':300,
'alpha':100,
'subsample':1,
'colsample_bytree':1,
'min_child_weight':5,
'eta': 0.001,
'seed':random_seed,
'silent':1
}

def evalerror(y,t):
t = t.get_label()
y_bin = [1 if y_cont > 0.5 else 0 for y_cont in y] # binaryzing your output
return 'f1',f1_score(t,y_bin)
sample_r_array=[[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[1,1,1,1,1,1,1,1,1,1,0],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1],
[0,0,0,0,0,0,0,0,0,0,1]]
train_set=np.array(sample_r_array)
test=[1,1,1,1,1,1,1,1,1,1]
X_train,X_val,y_train,y_val=train_test_split
(train_set[:,0:-1],train_set[:,-1],test_size=0.3,random_state=random_seed,feval=evalerror)
dval = xgb.DMatrix(X_val,label=y_val)
dtrain = xgb.DMatrix(X_train, label=y_train)
dtest=xgb.DMatrix(test)
watchlist  = [(dval,'eval'), (dtrain,'train')]
model = xgb.train(params,dtrain,num_boost_round=10,evals=watchlist)
a=model.predict(dtest)
print a`

Do not expect xgboost to work with every possible crazy combination of parameters you can throw at it.
Did you even try running it with different parameters?
The l1 and l2 regularizations are way too high in params, and also that min_child_weight might be too large for such a small sample.


Define array of values to be used as steps.

Sometimes I want to use a range with a non-linear list of steps.
Say I have a range from 0 to 60, and instead of equally sized steps, I want to step 5 by 5 until 30, and then 10 by 10 until 60.
I could manually define an array with these values: [5, 10, 15, 20, 25, 30, 40, 50].
There is a fork of an old version that does this, with a rz-slider-step-array option: https://github.com/hrishi1183/angular-slider-custom-rzslider-
Demo here: http://plnkr.co/edit/sUFdW4lulu2NQBvn9fq0?p=preview

The fun fact is that I was actually thinking about this something like 5 hours ago :D
We can even imagine that the steps array could contain strings as step value: ['A', 'B', 'C']...
This could be added to the list of options one the refactoring of this library is merged into master (see ReadMe).

Nice!
I just subscribed to the refactoring pull-request, will be following how things go.

OK, what's your opinion on it?

I agree it should be added to the list of options, after the refactoring.
About the implementation, I think there should be a separate option (maybe just step-array), where we would define a list of strings.
The fork I linked to uses a ng-repeat to show the ticks, but I don't know how that compare with the updateTicksScale approach.

Actually, I was refering to your opinion abour the PR. ^^ I already have ideas for the implementation of step-array.

I have started to implement it, it's pretty easy with the new version. Do you think it should be possible to give a custom translate function when providing a steps-array? As far as I think, I don't see a valid use case for it. In my current implementation, the internal translate function just returns the correct step from the array.

Will be in 2.0.0 release.

Sorry for the delay.
I agree with you, it doesn't seem necessary to have a custom translate, I think steps-array will already provide enough flexibility.


Delete stale branches

Most of the stale branches at https://github.com/rtfd/readthedocs.org/branches/stale?page=2 have already been merged so they can safely be removed now.

+1

Can I do this? With merged branches at least :). I really like to see the project clean :grin:

@stsewd I suppose that merged branches are safe to remove.


Add permission block for Code QL workflow

Adding the security-events: write permission to the code ql workflow. This should fix the errors we've seen in the workflow.

Thanks for looking into this! Do we know why it needs write access? Seems odd to me for Code QL...

Probably because it's making a PUT request:


Patch 1 gmbal api.version 4.0.2

invalid gmbal-api.version 4.0.3, the last one is 4.0.2

4.0.3 has just not been sent to central, see staging

4.0.3 has just not been sent to central, see staging

Hi lukasj :-)
thanks for your hint.
But it's more than "just" a problem, e.g. for all developers which have to use artifactory/nexus.
We are only able to download jar-files from maven-central.
So 4.0.3 is unreachable for us.
Do you have an idea in how many days 4.0.3 will be available in central?

We are only able to download jar-files from maven-central.

an option for the time being is to grab com.sun.xml.ws:jaws-ri and take everything from there

Do you have an idea in how many days 4.0.3 will be available in central?

no, that is out of my control but it should not take long

Hi Lukas,
yes, I already know about this possibility, but zip-files are decline in my
authority.
So we are only able to download jar files from the central. :-(
So it basically possible to download jaxws-rt and jaxws-tools in 2.3.3, but
not 2.3.4 with the missing gmbal-version.
Lukas Jungmann @.***> schrieb am Mo., 19. Apr. 2021,
15:23:

We are only able to download jar-files from maven-central.
an option for the time being is to grab com.sun.xml.ws:jaws-ri and take
everything from there
Do you have an idea in how many days 4.0.3 will be available in central?
no, that is out of my control but it should not take long
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eclipse-ee4j/metro-jax-ws/pull/217#issuecomment-822463221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHJI7TR7HH3F3DBMSE2A56LTJQVF5ANCNFSM43FYEI4A
.


Two active forks of Exceptional plugin

There are two active forks of the original Exceptional plugin pushed to JetBrains Marketplace
https://plugins.jetbrains.com/plugin/19461-exceptional-continued/
https://github.com/RobertWeinmeister/ExceptionalReSharper
by @RobertWeinmeister
and
https://plugins.jetbrains.com/plugin/14928-exceptional-by-mf
https://github.com/hhu-mahmoud/ExceptionalReSharper
by @hhu-mahmoud
It leads to an installation error, when both of these plugins are selected.
At the moment I am trying to find right way to resolve this issue.
@RobertWeinmeister is there are any changes in the feature set comparing to the fork of @hhu-mahmoud?
Because of the installation error we need to have only one version of the plugin in the Marketplace and unlist the other one.
Please let us know as soon as possible about the differences between the versions.

What is the problem with two forks at the same time? They have different names, or?
https://github.com/hhu-mahmoud/ExceptionalReSharper was abandoned for some time so I created this one. At the moment it has one additional functionality, it allows to ignore certain namespaces.
Is there a way to keep both on the marketplace?

@RobertWeinmeister thanks you for quick response.
There are no problem with the two forks, there is problem with two plugins with the same dll name in the JetBrains Marketplace.
As I mentioned above, it leads to errors because these plugins cannot be installed simultaneously. Due to plugin's installation model installer attempt to copy two dll with the same name into the same directory.

was abandoned for some time so I created this one
Did you try to contact the maintainer before creating another fork?

My priority was not to fork the plugin, I wanted to get it working again. I contacted @hhu-mahmoud back then and did not get a response, so I set it up myself. I do not mind merging them again or something similar.

@hhu-mahmoud thanks for joining the discussion and clarifying the situation!

I'm not sure with 2 installs, cus the last instal will overwrite the first one. I think!

Unfortunately, It won't. Installation doesn't work that way. We are treating each plugin equal and the installer will just fail because it cannot select one dll from many possibilities. We do consider plugins equal to the our products as well, so this check ensures that, for example, there is no dll with the same name from ReSharper and dotTrace, which are developed by different team, but share the integration into Visual Studio (and the installation folder)
This is real issue, because we are receiving customer requests regarding failed installation.
There is also another way to resolve this issue and having two active forks: rename dll in one of the forks and ensure that ReSharper isn't broken then two plugins are installed. But this solution far from ideal.


What's the correct way to list dependencies of NuGet packages?

AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet#1573 and dotnet/SqlClient#1137.
Currently based on my experience, we should list all top level dependences on NuNet package page, such as https://www.nuget.org/packages/Microsoft.Data.SqlClient/3.0.0. However this package only list Azure.Identity as required dependency. I suppose the Azure.Core also be listed as a required dependency, as the Microsoft.Data.SqlClient reference it directly.
.NETFramework 4.6.1

Azure.Identity (>= 1.3.0)

I just wonder what's the correct way to list the dependencies.
Imagine that we have a libA which references packageB and packageC. And packageB require PackageC.
I think the libA should list both packageB and packageC as dependencies, but from the above mentioned Microsoft.Data.SqlClient package, it only list packageB. Which one is the correct one?

Maybe you have a rule book about this? I have tried to Google it, but no lucky.

Hello @chucklu, judging based on the other issue, it seems like you may have come to the conclusion already. But just in case, I'll try to clarify here.
The dependency information shown on nuget.org is direct package dependencies. This is not necessary the same as assembly references for several reasons:

When you depend on a package, it can pull in many transitive package dependencies which themselves may bring other assemblies. At build time, you can reference any of those assemblies brought in packages either directly or indirectly. For the Azure.Core example, this is a transitive dependency of Microsoft.Data.SqlClient because Azure.Identity (a direct dependency) itself depends on Azure.Core. This means that any assembly in the Azure.Core package can actually be an assembly reference of the SqlClient DLL or even your own project that uses the Microsoft.Data.SqlClient package.

Not all packages actually bring assemblies. This means you might see a name pop up in the nuget.org UI of a package that has not matching assembly name. Some packages are so-called "meta-packages" that do not contain assemblies but instead contain only dependencies on other packages.

Sometimes the package ID does not match the assembly name. This can further confuse the situation that you were investigating in the other issue.

I hope this clears things up. In short, this is expected due to how transitive package dependencies work in NuGet.

@joelverhagen  Thanks for your explanation.


Review IG logging config

IG produces logs with a pattern that is hard to read, it also lacks useful time information.
Example snippet:
main] INFO  o.f.openig.launcher.Launcher -
                Reading the configuration from /var/ig/config/admin.json
[main] DEBUG io.vertx.core.logging.LoggerFactory -
                Using io.vertx.core.logging.SLF4JLogDelegateFactory
[main] DEBUG i.n.u.i.l.InternalLoggerFactory -
                Using SLF4J as the default logging framework

Review the logback.xml

The strange characters are ANSI color codes, if you view the log in something that understands them then you get the above output.
However, I don't find it very readable as it always puts the log message indented on the next line which makes it very hard to scan the file


Composer plugin freezing

When I am trying to build and application it's just freezing on this stage:
[2016-12-21 17:42:58] RunCommand.INFO: a5fa605b7446fffbb0227dceafa5bc5568267b8f {"buildID":"121"} []
[2016-12-21 17:42:58] RunCommand.INFO: Working copy created: /home/env/domains/phpci.env/PHPCI/build/121_9d602/ {"buildID":"121"} []
[2016-12-21 17:42:58] RunCommand.INFO: RUNNING PLUGIN: \PHPCI\Plugin\Composer {"buildID":"121"} []
[2016-12-21 17:42:58] RunCommand.DEBUG: Looking for binary: composer {"buildID":"121"} []
[2016-12-21 17:42:58] RunCommand.DEBUG: Found in : composer {"buildID":"121"} []
Seems like composer just not running, but application is freezes.
Command that are executed is (i removed --no-ansi --no-interaction for test):
[2016-12-21 18:00:14] RunCommand.INFO: /usr/local/bin/composer --working-dir="/home/env/domains/phpci.env/PHPCI/build/122_a3ed5/" install {"buildID":"122"} []

Problem was in my env, maybe better to add additional option to \PHPCI\Helper\BaseCommandExecutor to proc_open function, this way it outputs errors:
$process = proc_open($command, $descriptorSpec, $pipes, $this->buildPath, [
'suppress_errors' => true,
]);
What is the idea to be turned off?

I have not looked at the code but my guess is that stdout and stderr are
suppose to be captured by the pipes.
It will be easier for someone to debug if they are able to reproduce the
issue. What was the issue with composer in your environment?
On Thu, Dec 22, 2016, 12:54 PM Maxim<EMAIL_ADDRESS>wrote:

Problem was in my env, maybe better to add additional option to
\PHPCI\Helper\BaseCommandExecutor to proc_open function, this way it
outputs errors:
$process = proc_open($command, $descriptorSpec, $pipes, $this->buildPath, [
'suppress_errors' => true,
]);
What is the idea to be turned off?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Block8/PHPCI/issues/1286#issuecomment-268853988, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPd3fo8CGWAQFwZNyJGckeF5R2OJlj-ks5rKrlLgaJpZM4LTCOW
.

--
Sincerely,
Pablo Tejada
From Mobile

Problem is that in my env there was no directory /usr/local/bin, php was installed in that and composer can't find this (i solved it by ln).
But there still an issue about HOME.
The problem is that without suppress_errors you can't debug: program just doing nothing for a lot's of time.

My fault, replaced env in proc_open function because of first issue with PHP (which i solved via ln).


Add proxy settings for BrowserstackLocal

Added the ability to set the proxy-settings on the BrowserstackLocal-tunnel. I needed those to be able to use the plugin from behind a corporate proxy.
Sorry for the resubmit, I did use the wrong email-address for the commit and ran into issues with the google-licensing bot.

Sorry, forgot about the pull request and then updated my fork. Would be great to have this in upstream, so I could get rid of the fork.

Thanks, looks good now. One more thing could you squash all your commits into one and follow our commit message guidelines please? Then I can merge it :)

done

Thanks a lot :octocat:


PowerBI Actions issue

Dear Sir/Madam,
I love this extension, saved my time a lot.
I got a repository with many report and groups, it's working with most of the reports.
but there is a particular group which I only can upload 7 reports out of 9 reports.
as you see in the below screen:

also, I have a group with only one file which it can't upload:

** for both cases, I uploaded the files manually and they are uploading!
** what I suggest:**
I wonder if there was a way for you guys to get more details regarding the error messages and pass it to the logger.

Hi Farzad-Jalali, do you still encounter the issue?

Hi Maik,
I asked the BI team to rebuild the reports from scratch, also making a new group.
I think the mix of two actions actually fixed the problem and it's working just fine.


PlacePicker needed. Editable GPS Field Causing issues

Subject of the issue
As warned, making GPS field as editable is causing havoc as many users are entering random URLs/addresses to the location. Refer the screenshot for more information

This is caused by  #160
Steps to reproduce
Open keralarescue.in/requests/ and submit for a district.. many rows have accuracy and maps link as broken!
Expected behaviour
Maps co-ordinates should only be comma separated latitude and longitude.
Actual behaviour
People can type in anything causing the issue in buttons/display
Proposed solution
Revert back to old method of making it read only and only allowing it for self requests (as done in #122 )

Currently the code is requesting openmaps to do the reverse lookup.
Could  disabling reverse lookup could be the solution ?
var success = function(pos) {
			var crd = pos.coords;
				gpsField.value = crd.latitude + "," + crd.longitude;
				gpsAccuracyField.value = crd.accuracy + ' Meters';
				$.get('https://nominatim.openstreetmap.org/reverse?format=jsonv2&lat=' + crd.latitude + '&lon=' + crd.longitude + '&zoom=18&addressdetails=1')
					.done(function(response) {
						var inputElem = $('<input class="form-control" id="id_latlng_display" value="' + response.display_name + '"</input>')
						$(gpsField).before(inputElem);
						$(gpsField).hide();
						console.log(response.display_name);
					})
					.fail(function(err){
						console.log(err);
					});

Could disabling reverse lookup could be the solution ?

No, it will only cause more confusion as layman would not be familiar with concept of GPS co-ordinates. The best way is to

Implement auto detection when requesting for self.
Implement Place Picker when requesting for others

#232  disables input if requesting for others which is a temporary fix until a PlacePicker is created.

@naveenpf , please do merge #232 .

This is temporarily solved as of now, but a placepicker is of urgent requirement.

L:
If anyone's working on it, preferred workflow would be
User clicks on the GPS field -> modal opens -> types address -> co-ordinates are stored.
(Ability to mark the location on map itself would be a plus)
If you're having API restriction issues, Do look into
http://wiki.openstreetmap.org/wiki/Nominatim by Openstreetmap.

Not sure if making read only is the ideal solution.
I recieved a request in WhatsApp with location but I couldn't enter the location coordinates.
May be a warning to edit this field only if the user knows what he/she is doing will be better?

Making it readonly is never intended as a working solution, few people are already working on implementing the PlacePicker.

This should be fixed in PR #266

is anyone working on adding a place picker? - I can add this one - https://developers.google.com/maps/documentation/javascript/examples/places-autocomplete

please check #248 as well.

suggestion for the location picker : Do like ola/uber. Keep a crosshairs/marker fixed, centered over the map. Take the map's center as the lat-long.User only has to move the map around and position it below the marker/crosshairs. Much easier to use on mobiles.

will be fixed with #266

Automatic location picking is already done in #29 and is still supported in #289. If possible, test and verify #289 for faster merging.

Hi I've made a simple location picker / placepicker on ola/uber model : move the map, the marker is in the center. I don't think I can build a PR for this as I don't understand the rest of the code and how to push mine in.
Live: https://answerquest.github.io/leaflet-center-location-picker.html
Code: https://github.com/answerquest/answerquest.github.io/blob/master/leaflet-center-location-picker.html

@answerquest I have made the changes you requested in #266  itself, it behaves like how you described here now, the marker moves with center of the map

@ashwinkjoseph that's great! Thanks!

need to test #266 : please test and report so that this can be merged.


fix duplicate URL Encode on RelayState

The RelayState is URL Encoded twice when using urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST.

This is because Form Data is by default already URL Encoded.
Additionally, for HTTP-Redirect, the RelayState is already Query Escaped: https://github.com/zitadel/saml/blob/main/pkg/provider/redirect.go#L120

This bug results in the ACS receiving the incorrect RelayState. This PR fixes the issue.

thanks for the PR @coelho
@stebenz can you have a look pls?

The RelayState is URL Encoded twice when using urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST.

This is because Form Data is by default already URL Encoded.
Additionally, for HTTP-Redirect, the RelayState is already Query Escaped: https://github.com/zitadel/saml/blob/main/pkg/provider/redirect.go#L120

This bug results in the ACS receiving the incorrect RelayState. This PR fixes the issue.

Thank you for your contribution, it seems as if I totally overlooked this as it worked with crewjam and other SPs.


Remote profile / target is confusing without more context

Describe the bug
I created a new install of v0.8.2 with a remote host on Linux and the client on Windows. The Linux host can create / list / etc new workspaces. The Windows client can list and connect to existing workspaces but cannot create them. In the creation menu the remote target is not available. Even though the profile is set to default.
To Reproduce
Steps to reproduce the behavior:

Install Daytona on Linux and start the server with a new API key
Install Daytona on Windows, add the Linux server as a new profiles with the API key
Set the remote profile to default
Try to create a workspace on the remote Linux host from the Windows profile
See that the remote target isn't

Expected behavior
I expected to be able to create workspaces on Widows.
Desktop (please complete the following information):

OS: Remote Linux Ubuntu 22.04 Windows 11
Daytona Version: v0.8.2

@metcalfc there shouldn't be a remote target in the process you described.
When you add a profile, every action is executed on the remote server and all targets you see are those of the remote server.
When creating a workspace with a profile that goes to a remote server, the local target is that of that remote server.
Targets are not managed by the client, they are managed by the server and live where the server lives.

Closing this in favor of opening: #296, #297, #298, #299.


fix: make possible mutate provider-id

We should allow changing the Provider ID string in CCM. And add label key/value validation.
Pull Request

What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[ ] you linted your code (make lint)
[ ] you linted your code (make unit)

See make help for a description of the available targets.

/m


Add support for building OV 2021.04

There will be a tweak in build.py to build both the OV backends. I will raise a PR soon.


Auto label with raw data

Hi, I have a bag that includes lidar data and related info. I want to label with the auto-label tool. I think I should create nuscenes format (example dataset) from raw bag data without labels and label with the auto label tool, is it right?

What we did for processing our rosbags is to divide up the bag into 20s sequences, then export each frame of the sequence into pcd files. You can do longer than 20s sequences if you wish.
The file structure and auto-labelling instructions can be found in our custom dataset readme.
Let me know if you have any other questions!

What we did for processing our rosbags is to divide up the bag into 20s sequences, then export each frame of the sequence into pcd files. You can do longer than 20s sequences if you wish.
The file format/structure and auto-labelling instructions can be found in our custom dataset readme.
Let me know if you have any other questions!

Thanks for quick answer, i will try

We used the C++ PCL function, pcl::io::savePCDFileASCII, for exporting each frame of our rosbag to pcd. This should make it directly readable by open3d (which is how we load point cloud files in our custom dataset template).

Hi @darrenjkt,
I try to label raw bag data with custom_dataset instructions. I did but i got different result according to the other dataset.
2023-10-18 11:52:19,683   INFO  recall_roi_0.3: 0.000000
2023-10-18 11:52:19,683   INFO  recall_rcnn_0.3: 0.000000
2023-10-18 11:52:19,684   INFO  recall_roi_0.5: 0.000000
2023-10-18 11:52:19,684   INFO  recall_rcnn_0.5: 0.000000
2023-10-18 11:52:19,684   INFO  recall_roi_0.7: 0.000000
2023-10-18 11:52:19,684   INFO  recall_rcnn_0.7: 0.000000
2023-10-18 11:52:19,684   INFO  Average predicted number of objects(399 samples): 30.757
I have no annotation info. is this result related to label information?

Hi @darrenjkt again,
I use custom_dataset_da.yaml but i got error about kitti eval
ile "../pcdet/datasets/kitti/kitti_object_eval_python/eval.py", line 661, in get_official_eval_result
current_classes_int.append(name_to_class[curcls])
KeyError: 'Vehicle'
I use default yaml config
CLASS_NAMES: ['Vehicle', 'Pedestrian', 'Cyclist']
CLASS_MAPPING: {'car': 'Vehicle',
'truck': 'Vehicle',
'bus': 'Vehicle',
'Vehicle': 'Vehicle',
'motorcycle': 'Cyclist',
'bicycle': 'Cyclist',
'Cyclist': 'Cyclist',
'pedestrian': 'Pedestrian',
'Pedestrian': 'Pedestrian'}

This is because the custom dataset expects class names to be ['Vehicle', 'Pedestrian', 'Cyclist']. The labels you've given it from kitti raw are car, truck, pedestrian etc hence why it's throwing an error. I suggest either changing the CLASS_NAMES/CLASS_MAPPING to keep your kitti labels unchanged or modify your kitti labels to be the same as ['Vehicle', 'Pedestrian', 'Cyclist'].

thanks for reply @darrenjkt, I adjust class mapping in terms of kitti but I got error again,

should I comment this section?


[Tb 3369] Add bottom sheet cancel/close background color

What 🕵 🔍

add color for cancel in bottom sheet

Codecov Report

Merging #74 (d60fec8) into main (f480f93) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
- Coverage   47.16%   47.09%   -0.08%
==========================================
  Files          57       57
  Lines        2012     2015       +3
==========================================
  Hits          949      949
- Misses       1063     1066       +3

Impacted Files
Coverage Δ

lib/src/linden/xcolors.dart
29.57% <0.00%> (-0.64%)
:arrow_down:

Continue to review full report at Codecov.

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

why do we need to allow multiple sheets to be shown on top of each other? :)

There is no need for stacking bottom sheets now. We might at some point need to show the generic error bottom sheet when the bottom sheet was open.


Greatly increase test coverage

Thanks for approving. I'm actually gonna add a lot more tests. Would appreciate another review later 😉


kill: Differences from GNU 9.5

Found while checking #6201. Feel free to grab any/all of these, @m-haisham :)
Table (-L) format is wrong
Should be an actual table, not a grid
--- STDOUT_GNU	2024-04-07 19:12:25.820911283 +0200
+++ STDOUT_UUTILS	2024-04-07 19:12:25.824911280 +0200
@@ -1,62 +1,5 @@
- 1 HUP      Hangup
- 2 INT      Interrupt
- 3 QUIT     Quit
- 4 ILL      Illegal instruction
- 5 TRAP     Trace/breakpoint trap
- 6 ABRT     Aborted
- 7 BUS      Bus error
- 8 FPE      Floating point exception
- 9 KILL     Killed
-10 USR1     User defined signal 1
-11 SEGV     Segmentation fault
-12 USR2     User defined signal 2
-13 PIPE     Broken pipe
-14 ALRM     Alarm clock
-15 TERM     Terminated
-16 STKFLT   Stack fault
-17 CHLD     Child exited
-18 CONT     Continued
-19 STOP     Stopped (signal)
-20 TSTP     Stopped
-21 TTIN     Stopped (tty input)
-22 TTOU     Stopped (tty output)
-23 URG      Urgent I/O condition
-24 XCPU     CPU time limit exceeded
-25 XFSZ     File size limit exceeded
-26 VTALRM   Virtual timer expired
-27 PROF     Profiling timer expired
-28 WINCH    Window changed
-29 POLL     I/O possible
-30 PWR      Power failure
-31 SYS      Bad system call
-34 RTMIN    Real-time signal 0
-35 RTMIN+1  Real-time signal 1
-36 RTMIN+2  Real-time signal 2
-37 RTMIN+3  Real-time signal 3
-38 RTMIN+4  Real-time signal 4
-39 RTMIN+5  Real-time signal 5
-40 RTMIN+6  Real-time signal 6
-41 RTMIN+7  Real-time signal 7
-42 RTMIN+8  Real-time signal 8
-43 RTMIN+9  Real-time signal 9
-44 RTMIN+10 Real-time signal 10
-45 RTMIN+11 Real-time signal 11
-46 RTMIN+12 Real-time signal 12
-47 RTMIN+13 Real-time signal 13
-48 RTMIN+14 Real-time signal 14
-49 RTMIN+15 Real-time signal 15
-50 RTMAX-14 Real-time signal 16
-51 RTMAX-13 Real-time signal 17
-52 RTMAX-12 Real-time signal 18
-53 RTMAX-11 Real-time signal 19
-54 RTMAX-10 Real-time signal 20
-55 RTMAX-9  Real-time signal 21
-56 RTMAX-8  Real-time signal 22
-57 RTMAX-7  Real-time signal 23
-58 RTMAX-6  Real-time signal 24
-59 RTMAX-5  Real-time signal 25
-60 RTMAX-4  Real-time signal 26
-61 RTMAX-3  Real-time signal 27
-62 RTMAX-2  Real-time signal 28
-63 RTMAX-1  Real-time signal 29
-64 RTMAX    Real-time signal 30
+ 0 EXIT     1 HUP      2 INT      3 QUIT     4 ILL      5 TRAP     6 ABRT
+ 7 BUS      8 FPE      9 KILL    10 USR1    11 SEGV    12 USR2    13 PIPE
+14 ALRM    15 TERM    16 STKFLT  17 CHLD    18 CONT    19 STOP    20 TSTP
+21 TTIN    22 TTOU    23 URG     24 XCPU    25 XFSZ    26 VTALRM  27 PROF
+28 WINCH   29 POLL    30 PWR     31 SYS

Missing RTMAX/RTMIN on Linux
I'm not entirely sure what they mean, so perhaps we simply don't want to support it, but I wanted to mention it.
Broken support for querying multiple signal names
$ ../gnu/src/kill --version | head -n1
kill (GNU coreutils) 9.5
$ ../gnu/src/kill -l SIGUSR1 SIGUSR2
10
12
$ cargo run --all-features kill -l SIGUSR1 SIGUSR2
10
$

Note that the bash built-in kill also supports querying multiple names at the same time, but the Debian default binary (procps-ng 4.0.4) does not.

Noticed that the gnu starts with 1 while uutils starts with 0.
--- STDOUT_GNU
- 1 HUP      Hangup
+++ STDOUT_UUTILS
+ 0 EXIT

I'll change uutils to start with 1 as well if that's acceptable.

I'll post a pr for third and a partial fix for first after #6201 has been merged.
The signal descriptions does not seem to be available for use in uucore atm. So probably best to add them in another pull request.

@BenWiederhake, what's the status here. Can we close this with #6216 ?

Yeah, you're right, it should be separate issues. I was a bit too lazy when I created this issue :)


3.0 - Final Phase, stage continued #13

Add the useful ability to use a string for $where parameter in Engine's 'delete' method.

@daveismyname Please merge this pull-request, as its objectives being completed.


The lastest version of pyzx breaks zxlive

If you install the latest version of pyzx (this commit) you cannot add edges in edit mode anymore - if you try to connect two vertices by an edge, the edge simply does not show up. This is likely due to the new implementation of hyper edges.
Error message in console:
Traceback (most recent call last):   File ".../zxlive/mainwindow.py", line 263, in _undo_changed     if self.active_panel:   File ".../zxlive/mainwindow.py", line 218, in active_panel     current_widget = self.tab_widget.currentWidget()   File ".../zxlive/commands.py", line 260, in redo     self.update_graph_view()   File ".../zxlive/commands.py", line 48, in update_graph_view     self.graph_view.update_graph(self.g, select_new)   File ".../zxlive/graphview.py", line 118, in update_graph     self.graph_scene.update_graph(g, select_new)   File ".../zxlive/graphscene.py", line 141, in update_graph     e_item = EItem(self, e, self.vertex_map[s], self.vertex_map[t]) KeyError: (8, 9)

I have fixed it in this commit d8e50d68ca0e8c3364aed559463e00ae70893313 but now you need to use the latest version of pyzx

I'm closing this, since it is fixed, and zxlive now in the requirements always asks for the github version of pyzx.

There are several things that still do not work such as saving/loading #308 and graph-like rewrites/other routines #256. But they are already tracked in their own issues so it's okay to close this.


Add accessibility tools to base SIFT VM

To support upcoming accessibility requirements, please ensure the following are installed on the base public SIFT VM.  I've verified the commands below on the 18.04 VM.

Magnus: On-screen magnification tool

https://snapcraft.io/magnus
sudo snap install magnus

Onboard: Alternative input option for motor accessibility

sudo apt-get install onboard

Orca: Text-to-speech tool for audible system navigation

This was already installed via apt-get on the 18.04 version

So snaps are terrible. I suggest we avoid as long as possible. I’ll see if we can install it another way.
Sent from my iPhone

On Dec 10, 2020, at 19:35, Phil Hagen<EMAIL_ADDRESS>wrote:

To support upcoming accessibility requirements, please ensure the following are installed on the base public SIFT VM. I've verified the commands below on the 18.04 VM.
Magnus: On-screen magnification tool
https://snapcraft.io/magnus
sudo snap install magnus
Onboard: Alternative input option for motor accessibility
sudo apt-get install onboard
Orca: Text-to-speech tool for audible system navigation
This was already installed via apt-get on the 18.04 version
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

No worries at all!  I didn't mean to specify that as a solution, just something that WFM.

No worries! Just sharing knowledge. I’ll get it installed however we can.
Sent from my iPhone

On Dec 10, 2020, at 20:21, Phil Hagen<EMAIL_ADDRESS>wrote:

No worries at all! I didn't mean to specify that as a solution, just something that WFM.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Trying to understand how this is going to be done; will these get added as salt packages so that they are pulled and installed as part of sift install.
Thank You

Trying to understand how this is going to be done; will these get added as salt packages so that they are pulled and installed as part of sift install.
Thank You

Correct. Look for them in the next couple of weeks.
On Sun, Jan 17, 2021 at 8:28 AM gnittala<EMAIL_ADDRESS>wrote:

Trying to understand how this is going to be done; will these get added as
salt packages so that they are pulled and installed as part of sift
install.
Thank You
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/500#issuecomment-761830353, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALZSNGNRBV3OIG2RMJUZLS2L623ANCNFSM4UV5SXBA
.

Correct. Look for them in the next couple of weeks.
On Sun, Jan 17, 2021 at 8:28 AM gnittala<EMAIL_ADDRESS>wrote:

Trying to understand how this is going to be done; will these get added as
salt packages so that they are pulled and installed as part of sift
install.
Thank You
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/500#issuecomment-761830353, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALZSNGNRBV3OIG2RMJUZLS2L623ANCNFSM4UV5SXBA
.


Update service performance dashboard to accept 2022 as a year value

Context
2021 to 2022 (starts 2022) values are currently not rendering on the Service performance dashboard as the controller checks to see if the selected year appears in an array of year values and 2022 is currently missing from this array.
Changes proposed in this pull request
2022 added into:
year = params[:year] if %w[2020 2021].include?(params[:year])
Guidance to review

Link to Trello card

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

Good catch!

Should this use RecruitmentCycle.years_visible_in_support?

Should this use RecruitmentCycle.years_visible_in_support?

that's a better idea.

Thank you @JR-G! merging this now so it goes out tonight


Python: fix for streaming openai responses, and first parts of fixes for Chat With Your Data

Motivation and Context

Description

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Python 3.8 Test Coverage Report • FileStmtsMissCoverMissingsemantic_kernel   kernel.py3162891%190–194, 196–200, 202–206, 214–216, 235, 241, 243–247, 249–253, 255–260, 262, 307–311, 328–333, 340, 344–348, 361semantic_kernel/connectors/ai   chat_completion_client_base.py12192%18semantic_kernel/connectors/ai/ollama/services   ollama_chat_completion.py62297%114, 180semantic_kernel/connectors/ai/open_ai/contents   open_ai_streaming_chat_message_content.py311939%49–71semantic_kernel/connectors/ai/open_ai/services   open_ai_chat_completion_base.py1606261%107–108, 133–139, 154–157, 166–169, 179–202, 208–210, 227–229, 252, 260, 266–272, 284–290, 302–303, 317, 344–347, 349–352   open_ai_text_embedding_base.py19195%51semantic_kernel/contents   streaming_chat_message_content.py241154%42, 45, 53–61semantic_kernel/functions   kernel_function.py57296%119, 138   kernel_function_from_prompt.py123398%166, 193, 289TOTAL480592781%
Python 3.8 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1026
11 :zzz:
0 :x:
0 :fire:
19.984s :stopwatch:


Allow to run on rails 7

Updated .gemspec
Allow it to run under rails 5.2 =< {rails version} < 8
Closes #113

@richardvenneman please check it out. Feel free to ask if you have any questions.

Great thank you @alxekb 👍🏻
It would be fantastic if you could add that for Rails 7 support @richardvenneman. Thank you very much for your effort!

Thanks for this, sorry for the wait!

Hi @richardvenneman
Great, thank you very much! I am very happy that Rails 7 support is now available.
Could you please also release a new version of the Gem? Thank you 😃