ondex-mini - Building plugins with new changes fails [sequence.jar]

Made some changes to the fasta_gff3 parser in ondex-knet-builder repo under modules/sequence and tried to take sequence.jar and use it as follows:

with existing older Ondex desktop and Ondex-mini installed on Windows and those failed.

Testing the new sequence.jar on Linux (by moving it under plugins/ in an existing ondex-mini install) gave the following error:
Invalid constant pool index 319 in class file net/sourceforge/ondex/export/fasta/Export

Assuming it's due to recent ondex-knet-builder Java  8 and Maven assembly plugin (3.3.0 -> 3.3.9) upgrades, I then built the ondex-mini module in the ondex-knet-builder repo and tested that on both Windows and Linux and got the following error on both:
 Exception in thread "main" java.lang.NoClassDefFoundError: net/sourceforge/ondex/WorkflowMain
         at net.sourceforge.ondex.OndexMiniMain.main(OndexMiniMain.java:7)
 Caused by: java.lang.ClassNotFoundException: net.sourceforge.ondex.WorkflowMain
         at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
         at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
         at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
         at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
        ... 1 more

Note: Other users also report seeing the same error.

On Linux, tested using ./runme.sh "dir/workflow.xml", i.e., via:
 ./runme.sh "testWorkflow_fastaGff3_parser230317/testWorkflow_fastaGff3.xml"

This is  likely caused by #4 and I'm working on it.

I've fixed #4 and now it should work. Please build ondex/test it again and let me know.

Thanks @marco-brandizi , it works fine for me now, tested on Linux (lily) by rebuilding ondex-mini and using that.


Fix/auto assign pr to author

Description
Fixing the action that automatically assigns the author of the pull request as the assignee.

If it still doesn't work it may be because you may not have created the GITHUB_TOKEN in the repo secrets.

the GITHUB_TOKEN  is automatically created by github itself.

LGTM, lets try if this works


Integration Test - Release - 2019/05/16 14:08:46

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#backlog'

#pri2

#backlog

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


[SPARK-13478] [yarn] Use real user when fetching delegation tokens.

The Hive client library is not smart enough to notice that the current
user is a proxy user; so when using a proxy user, it fails to fetch
delegation tokens from the metastore because of a missing kerberos
TGT for the current user.
To fix it, just run the code that fetches the delegation token as the
real logged in user.
Tested on a kerberos cluster both submitting normally and with a proxy
user; Hive and HBase tokens are retrieved correctly in both cases.

Test build #51921 has started for PR 11358 at commit 7fcec4a.

Test build #51921 has finished for PR 11358 at commit 7fcec4a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51921/
Test PASSed.

So I have not tested using the keytab-based login with proxy user stuff at all. We get delegation tokens even there - does this issue affect that as well?

This looks like it might affect HDFS tokens as well and error that looks like this might come up during the initial token renewal:
WARN UserGroupInformation: PriviledgedActionException as:hari (auth:PROXY) via hdfs@EXAMPLE (auth:KERBEROS) cause:org.apache.hadoop.security.AccessControlException: hari tries to renew a token with renewer hdfs

In addition to the code that gets the new tokens, I think the getTokenRenewalInterval method also needs to be be run as the real user.

keytab-based login with proxy user stuff at all

That is not expected to work. Just use the proper user's credentials if you're going to provide a principal and a keytab.

the code looks fine to me.  Do we need any more documentation to explain proxy user vs keytab?

Do we need any more documentation to explain proxy user vs keytab?

Ok, ok, I'll add something.

Test build #52058 has started for PR 11358 at commit 0159499.

looks fine.  +1 pending jenkins.  @harishreedharan any other concerns?

LGTM

Test build #52058 has finished for PR 11358 at commit 0159499.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52058/
Test PASSed.

Merging this.

hi @vanzin, how about spark sql in this issue, in your view? as in spark sql it will revoke SessionState.start in which will finally connected to metastore in Hive.

@WangTaoTheTonic please don't comment on closed PRs. If there's a bug in Spark SQL, file a bug.


HALON-309: Implement keepalive mechanism for SSPL-HL.

Created by: qnikst
Currently amqp library may silently loose connection to rabbit-mq.
In this case halon will not reply to SSPL-HL requests. In order to
fix this problem we eventually send explicit keepalive messages to
the halon exchange, in this case if HALON does not see keepalive
in TIMEOUT seconds it restarts connection.

Closing as an obsolete


enarx binary reports wrong or misleading version when logging

From enarx created by MikeCamel: enarx/enarx#1098
v0.1.2 reports version 0.1.0 in first log output.
enarx --version outputs (correctly) enarx 0.1.2.
Observed:
[mikecamel@coffeelake timestamp]$ ENARX_BACKEND=sgx enarx run target/wasm32-wasi/release/timestamp.wasm
[INFO  wasmldr] version 0.1.0 starting up
...

Expected
[mikecamel@coffeelake timestamp]$ ENARX_BACKEND=sgx enarx run target/wasm32-wasi/release/timestamp.wasm
[INFO  wasmldr] version 0.1.0 starting up
...

Suggest auto-filling from a known file for all possible outputs of versioning (and other related metadata).

This is actually the wasmldr component - so it's misleading, not wrong.
Options:

keep wasmldr in step with versioning.  Probably the wrong approach.
have the enarx component output a separate entry.  Easier, more clear.


python311Packages.pyswitchbee: 1.7.21 -> 1.7.26

Diff: https://github.com/jafar-atili/pySwitchbee/compare/refs/tags/1.7.21...1.7.26
Changelog: https://github.com/jafar-atili/pySwitchbee/releases/tag/1.7.26
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 230202 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.switchbee
python310Packages.pyswitchbee
python310Packages.pyswitchbee.dist
python311Packages.pyswitchbee
python311Packages.pyswitchbee.dist


chore: Fix error in verify_doc_links.py script

Motivation
In its current version the verify_doc_links.py script excludes any URLs that contain a colon : character.
This was meant to exclude any local URLs with a port like ":8008". Instead it excludes any URLs that start with "http:" or "https:". The "rookie" who created that script! 🙄
Modifications
Remove the ":" from the list of patterns to identify URLs that should be excluded from the link verification.
Result
220 links found instead of 211 before.

/lgtm


PageTranslator doesn't work !

Dear Jeremiah, thanks for your wonderful ad-on!
Unfortunately, it has not worked for a few days: Firefox 62.0.2, Mac OS X 10.11.6.

Hi @Armand68, I am running Firefox 62.0.2 on macOS 10.13.6 and not able to reproduce. Can you give me more information to help troubleshoot?
Is there a particular URL that it is failing to load on? What is your primary preferred language setting?
Have you tried removing the add-on at about:addons and reinstalling it from here?

Hi @Armand68, I am running Firefox 62.0.2 on macOS 10.13.6 and not able to reproduce. Can you give me more information to help troubleshoot?

I just click on the icon in the URL bar, and nothing happens.

Have you tried removing the add-on at about:addons and reinstalling it from here?

Yes, but it doesn't work.

Is there a particular URL that it is failing to load on?

No : all of them, included some pages (in english) in web sites I regulary visit.

What is your primary preferred language setting?

French

I found it! The issue is provided by a wrong interaction with Ghostery...
If I desactivate the Ghostery “Intelligent blocking” function (I dont know what is exactly the english name of this feature, « blocage intelligent » in french), Page Translator works fine.
Jeremiah, if you want somme other test or precision, feel free to ask me.
And thanks again for your marvellous add-on.

Ghostery is likely blocking Google-provided services. Ghostery users may need to temporarily disable to permit Google's Translation widget to operate.


random fail on service upgrade

Hello,
Thank you dude nice job with this tool !
Just to tell you sometime y have this error and i don t know why..
`$ upgrade --stack jankenpon --service jankenpon --no-start-before-stopping --environment lab
Upgrading JanKenPon/jankenpon in environment Lab...
Upgrade started, waiting for upgrade to complete...
Traceback (most recent call last):
File "/usr/local/bin/upgrade", line 11, in
load_entry_point('rancher-gitlab-deploy==1.2', 'console_scripts', 'rancher-gitlab-deploy')()
File "/usr/local/lib/python2.7/site-packages/click-6.7-py2.7.egg/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/click-6.7-py2.7.egg/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python2.7/site-packages/click-6.7-py2.7.egg/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python2.7/site-packages/click-6.7-py2.7.egg/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/rancher_gitlab_deploy-1.2-py2.7.egg/rancher_gitlab_deploy/cli.py", line 200, in main
api, environment_id, service['id']
File "/usr/local/lib/python2.7/site-packages/requests-2.13.0-py2.7.egg/requests/api.py", line 70, in get
return request('get', url, params=params, **kwargs)
File "/usr/local/lib/python2.7/site-packages/requests-2.13.0-py2.7.egg/requests/api.py", line 56, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python2.7/site-packages/requests-2.13.0-py2.7.egg/requests/sessions.py", line 488, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python2.7/site-packages/requests-2.13.0-py2.7.egg/requests/sessions.py", line 609, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python2.7/site-packages/requests-2.13.0-py2.7.egg/requests/adapters.py", line 473, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', BadStatusLine("''",))`
if i retry it s works
thank again.
`$ upgrade --stack jankenpon --service jankenpon --no-start-before-stopping --environment lab
Upgrading JanKenPon/jankenpon in environment Lab...
Upgrade started, waiting for upgrade to complete...
Finishing upgrade...
 It looks like the request response gets broken somehow.

Hello
Yes it s ok now, without load balancer
Thank you so !!!


Remove stac_api_version from root catalog

Conformance classes indicate all the info needed about versions support by the API, and stac_api_version field is not part of the spec

Submitted pull request for this.


Unable to deploy new environment with a CDK override modifying a slotted resolver's datasource

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
v18.13.0
Amplify CLI Version
10.7.2
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Added some HTML templates for sending out messages/verifications through Cognit, and also on Cognito I defined a couple of Cognito triggers.
Describe the bug
I'm trying to remove and re-create my Cognito 'add' by removing it and adding it again on my Amplify project as I want to add some fields and make 2FA optional.
What I want to do is create a test environment to make this changes and test that it works, but whenever I'm trying to create a new environment and then push it to the cloud I'm gettin this error without any other data:
🛑 The following resources failed to deploy:
🛑 Resource is not in the state stackUpdateComplete

Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/

Session Identifier: a0dd76bb-4b4e-42e0-af8c-e79c6a246d15

On the terminal there's a ton of text, but I can see this deployment failures:
.
.
myApi                AWS::CloudFormation::Stack     CREATE_FAILED                  Tue Feb 14 2023
User                           AWS::CloudFormation::Stack     CREATE_FAILED                  Tue Feb 14
.
.

I can see that the environment gets created with no issues, but when I try to push it there, that's where I see all these issues. I've also tried doing the cloning of the project using the web UI in the console, but it has the same effect, the environment gets created, but when it tries to start creating the resources the deployment fails.
Expected behavior
To be able to create a new environment to test out new features or changes.
Reproduction steps

Create a new environment using: amplify env add qa
Trying to push to the cloud with: amplify push

Project Identifier
No response
Log output

Deploying root stack appname [ ====================-------------------- ] 8/16
	amplify-appname-qa-142556    AWS::CloudFormation::Stack     UPDATE_ROLLBACK_COMPLETE       Tue Feb 14 2023 14:40:17…
	functionpreAuthenticationAppName AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:48…
	functionappsyncOperations      AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:37…
	functionsendAnexNineEmail      AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:48…
	functionsendUnusualActivityMa… AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:48…
	authappnameAuth              AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:40:17…
	apiappnameApi                AWS::CloudFormation::Stack     CREATE_FAILED                  Tue Feb 14 2023 14:36:51…
	authuserPoolGroups             AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:59…
	storageappnameStorage        AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:37…
Deploying api apiHookMati [ ---------------------------------------- ] 0/5
Deploying api appnameApi [ =========================--------------- ] 10/16
	GraphQLAPI                     AWS::AppSync::GraphQLApi       DELETE_IN_PROGRESS             Tue Feb 14 2023 14:39:22…
	GraphQLAPITransformerSchema3C… AWS::AppSync::GraphQLSchema    DELETE_COMPLETE                Tue Feb 14 2023 14:39:20…
	GraphQLAPIDefaultApiKey215A6D… AWS::AppSync::ApiKey           DELETE_COMPLETE                Tue Feb 14 2023 14:37:30…
	GraphQLAPINONEDS95A13CF0       AWS::AppSync::DataSource       DELETE_COMPLETE                Tue Feb 14 2023 14:39:21…
	FunctionDirectiveStack         AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:37:50…
	Company                        AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:39:18…
	User                           AWS::CloudFormation::Stack     CREATE_FAILED                  Tue Feb 14 2023 14:36:15…
	Category                       AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:38:30…
	FinancialStatements            AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:38:16…
	ProjectUpdate                  AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:38:16…
	Project                        AWS::CloudFormation::Stack     DELETE_COMPLETE                Tue Feb 14 2023 14:38:16…

🛑 The following resources failed to deploy:
🛑 Resource is not in the state stackUpdateComplete

Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/

Session Identifier: a0dd76bb-4b4e-42e0-af8c-e79c6a246d15

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @jerocosio I was able to reproduce this using the following steps:

amplify init -y
amplify add api > GraphQL
add API Key, Cognito auth
use the following schematype Todo
  @model
  @auth(rules: [{ allow: public, operations: [read] }, { allow: owner }]) {
  id: ID!
  name: String!
  description: String
}

amplify override api > copy/paste the following overrideimport { AmplifyApiGraphQlResourceStackTemplate } from '@aws-amplify/cli-extensibility-helper'

export function override(resources: AmplifyApiGraphQlResourceStackTemplate) {
  resources.models['Todo'].appsyncFunctions[
    'MutationupdateTodoauth1FunctionMutationupdateTodoauth1Function.AppSyncFunction'
  ].dataSourceName = 'TodoTable'
}

amplify push -y
observe successful
amplify add env qa
amplify push -y
observe error

As a workaround we can comment out the override code, run amplify push -y, then uncomment and re-push.
It's important to note I was not able to reproduce then when attempting to override the datasource for the generated auth0 resolver, but if I add another resolver Mutation.updateTodo.auth.2.req.vtl and apply the override I am able to reproduce this consistently. Marking as a bug 🙂

Hey @josefaidt, thank you so much for looking into this, I followed the workaround and it worked great. I think it would be even better if there was an easier way to add specific resources or Tables into a resolver, as you can see right now the process sin't really straight forward and it's hard to get the right appsync function name.
I also edited my models to remove information, but just left enough in case someone's having the same issue and googling it.

Facing the same issue where we have added a CDK override to create an API gateway WAF and hence a dependency on REST API like so
// Access other Amplify Resources
    const dependencies: AmplifyDependentResourcesAttributes =
      AmplifyHelpers.addResourceDependency(
        this,
        amplifyResourceProps.category,
        amplifyResourceProps.resourceName,
        [{ category: 'api', resourceName: MyApi' }]
      );
    const apiId = cdk.Fn.ref(dependencies.api.MyApi.ApiId);

But it fails with an error where it cannot find the apiId ref.


On debug mode : Can't see the all action bar ( Play, Restart, and navigators)

Can you be more explicit ?

I change the title.
It's only happen on debug mode.
On normal mode, all work fine

There is no code about devmode are you sure this is not related to your implementation ? You can see the player but no the controls ?


Upgrade helix to 0.9.10

Description
Upgrade helix dependency to 0.9.10 for fixing the issues related to ZNode size limit.

Stripe has tested out helix 0.9.9 on private build and surpassed the ZNode limit. (David Yang at Stripe)
The fix in Helix 0.9.9 unblocks the case when the ZNode is larger than 1MB even after compression.

The code change in 0.9.9 is here  while the limit is applied when we update Zookeeper nodes from the same ZkClient. It happens when the Pinot table has lots of segments with long hostnames, for IdealStates node.
Release Notes
Upgraded helix dependency to 0.9.10 for fixing the issues caused by ZNode size limit.

(cannot add labels in the UI, and somehow cannot find how to markdown a label)

Codecov Report

Merging #8506 (3e49272) into master (fd3ecea) will decrease coverage by 11.71%.
The diff coverage is n/a.

:exclamation: Current head 3e49272 differs from pull request most recent head dc6c7dd. Consider uploading reports for the commit dc6c7dd to get more accurate results

@@              Coverage Diff              @@
##             master    #8506       +/-   ##
=============================================
- Coverage     25.83%   14.11%   -11.72%
- Complexity        0       84       +84
=============================================
  Files          1660     1627       -33
  Lines         87129    85576     -1553
  Branches      13203    13037      -166
=============================================
- Hits          22513    12083    -10430
- Misses        62459    72584    +10125
+ Partials       2157      909     -1248

Flag
Coverage Δ

integration2
?

unittests2
14.11% <ø> (?)

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

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

Since 0.9.9 and 0.9.10 are the same, why do we upgrade to 0.9.10 instead of 0.9.9? I don't see 0.9.10 as an official release in Helix

@dongxiaoman  I agree with @jackjlli . We do have an impending change to Helix 1.x. Helix version upgrades in the past have required a good bit of testing, and we are going through it in a thorough manner in the 1.x case. I would recommend to wait until then, and use the current methods available to enable compression, etc.
What is the use case you are trying to solve? Is it idealstate getting to be too big? currentstate? We can share our helix configs if you like. We have a cluster with 4k+ tables of all shapes and sizes.

@mcvsubbu @jackjlli (Let me update that part in the description)

The fix in Helix 0.9.9 unblocks the case when the ZNode is larger than 1MB even after compression. The code change in 0.9.9 is here  while the limit is applied when we update Zookeeper nodes from the same ZkClient. It happens when the Pinot table has lots of segments with long hostnames, for IdealStates node.
Right now it will be awesome to have Helix 1.0+ in; if it is not catching the next release, it will be good for the community to upgrade 0.9.9 since this Helix back-port patch is actually requested by us Pinot
For @Jackie-Jiang  I will roll back to 0.9.9 ; mostly I found the 0.9.10 in Maven Repo so I thought it is released

Let me also design an integration test case where the ZNode bug will show up.

Since 0.9.9 and 0.9.10 are the same, why do we upgrade to 0.9.10 instead of 0.9.9? I don't see 0.9.10 as an official release in Helix

After trying in IntelliJ with source, there are some discrepancy between the Helix source code tags and the actual artifacts release.
Simply put, the folks in Helix made a mistake in the released artifact of 0.9.9 build, and the 0.9.10 is the actual one containing fixes.

close this one to clean up since https://github.com/apache/pinot/pull/8325/files#diff-5ed6b0dd4c47b74179789062c2413dda3b74f2640248037e159d2c3981c38b8c is in place already


Youtube : Filter out a channel by date

What website(s) would this filter apply to?
youtube.com
What content should be blocked?
Hey
The filter I'm thinking of would filter out the videos from a channel older than a certain timespan (older than 2 weeks, older than 1 year....)
The Hide feature is bugged on youtube(videos keep reappearing), this option would make up for it, without hiding the newest videos from a channel, and without hiding old videos when we subscribe to a new channel
What options should be offered?
For each channel, we should be able to customise a timespan
Did you write a custom uBlock filter already?
Sadly not, I tried to mirror this script : https://letsblock.it/filters/youtube-upcoming-videos but I don't know the variables names
Additional notes
No response

To add some exemple :
Let's say I subscribed to 3 channel A B and C
I'm up to date on channel A, so in my subscription tab, I would like to only see what A published in the last two weeks to be sure not to miss a thing
I'm catching up with B and am currently 1 year away from the present day, I'd like to see only what B published in let's say the last 54 weeks
And I haven't started watching C yet so nothing from C should be filtered out

Hello @AruthaC, thanks for using the project and suggesting improvements!
A similar idea has been discussed in https://github.com/letsblockit/letsblockit/issues/349, but implementing it through this project would be way too clunky and brittle: the YouTube web frontend does not expose the video age and I refuse to match on display strings, as a significant portion of our users are international.
I feel like your workflow would be better served by alternative frontends such as NewPipe than by customizing the official frontend.

Hey
Thanks for your answer
Sorry, I hadn't seen issue 349
I'll check NewPipe, thanks for the suggestion
Have a nice day :)


In graph options, update "Theme" to "Graph Theme"

Describe the bug
In graph options, update "Theme" to "Graph Theme"

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.

Hello, is this issue still available? I'd like to work on it if possible.

hey i want to work on this issue please acceptmy request

hey i am intrested in contributing to elemenate the bug
please assign this to me

@aditya237 Unfortunately, GitHub does not let us assign issues to community members, but you are the first to "dibs" this issue, so please go ahead! Let us know if you have any questions :)

sir i'm a first time developer .So sir please let me help me to know which file am I supposed to edit


make the announcements page more image oriented

the feature image should appear full and then after that the information about it and other details would be visible

I was planning of making titles instead of a list type presentation of the announcements, that can serve this

you can close this one maybe


task 350 - 351: WinoMT

tasks added to #187
dataset: WinoMT

@Mirzyaaliii  wondering if you looked into this PR as well?

This PR looks good to me.  @XudongOliverShen Minor spelling mistake in Readme "aggrees" to "agrees" and "disaggrees" to "disagrees".
One question: In both the anti and pro tasks the task definition is exactly the same. In the Readme I can see "anti means the gender disagrees with the cultural stereotype of the profession" and "pro means the gender agrees with the cultural stereotype of the profession". Should this information be included in the definitions to differentiate between the two tasks?

fixed. question is also answered in https://github.com/allenai/natural-instructions-expansion/pull/187#issuecomment-915647979

@kuntalkumarpal, could you check and let us know if your comments are addressed or if you have any other feedback?

@kuntalkumarpal, a gentle reminder.

Looks good to me.


extraData set to 0 after token transfer

I am packing some variables in the newly implemented "ExtraData" in 4.1.0 however, upon transferring a token the .extraData is set to 0 and not is carried over

ok it seems like i need to override _extraData function for it to work
if I need the .extraData to be carried as is i wrote my function as
    function _extraData(         address from,         address to,         uint24 previousExtraData     ) internal view override returns (uint24) {     return previousExtraData; }
does it look OK?

Yes. That’s how you do it.

Thank you


How do you set connections and variables programmatically?

I would like to set variables and connections programmatically so that they're automatically loaded into Airflow. I'm running 2.6.3 with the only change being adding the following to .env.localrunner.
AIRFLOW_CONN_SLACK_CONN='my-conn-type://login:password@host:port/schema?param1=val1&param2=val2'
AIRFLOW_VAR_DEPLOY_ENVIRONMENT=qa

The problem is that both the variable and the connection don't show up in the UI. I've tried rebuilding from scratch and restarting the container. Am I setting these in the wrong place?

It turns out I can access both the variable and the connection in my dag but they still don't show up in the UI - is this intentional?

You should do this in the startup script; startup.sh

I tried setting a connection in both .env.localrunner and exporting in startup.sh and the connection never shows up in the Airflow UI. I can use the connection in a dag, however.
I'll also mention that I set load_default_connections to False in airflow.cfg and I still see the default list populated.


Adapt OpenTelemetry native metrics protocol

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

e2e are failing, it seems missing some adjustments.

At least, starter/pom.xml is forgotten to change.


docs(SETUP): fix code formatting command

What kind of change does this PR introduce?
fix
currently, the document says to run yarn pretty to format the code but the pretty script is not present in package.json

Did you add tests for your changes?
NA

Does this PR introduce a breaking change?
NO

What needs to be documented once your changes are merged?
Nothing.

Thanks


[bug] thermostat /set (e.g., /zwave2mqtt/8/67/1/1/set) ignores some values

Version 4.0.4
Build/Run method

[x] Docker
[ ] PKG
[ ] Manually built (git clone - npm install - npm run build )

Zwave2Mqtt version: 4.0.4
Openzwave Version: 1.6.1280
Describe the bug
An incoming value via mqtt is ignored if it's a decimal with a lot of places.
To Reproduce
$ docker exec -it mosquitto mosquitto_pub -t zwave2mqtt/8/67/1/1/set -m 28.4444
Oct 05 10:13:22 nuc docker[29982]: 2020-10-05T17:13:22.926Z z2m:Mqtt Message received on zwave2mqtt/8/67/1/1/set

$ docker exec -it mosquitto mosquitto_pub -t zwave2mqtt/8/67/1/1/set -m 28.444
Oct 05 10:13:32 nuc docker[29982]: 2020-10-05T17:13:32.910Z z2m:Mqtt Message received on zwave2mqtt/8/67/1/1/set
Oct 05 10:13:34 nuc docker[29982]: 2020-10-05T17:13:34.581Z z2m:Zwave zwave node 8: changed: 67-1-1:Heating 1:28 -> 28

Expected behavior
Any value that node can parse as a number should be sent to ZWave.

further assessment, I don't think the issue is on the mqtt side, I think it's on the zwave side. Here's a comparison of setting 28.4444 vs 28.444 directly on the object via z2m UI:
Oct 05 10:17:14 nuc docker[29982]: 2020-10-05T17:17:14.556Z z2m:App Zwave api call: setValue [ 8, 67, 1, 1, '28.4444' ]
Oct 05 10:17:14 nuc docker[29982]: 2020-10-05T17:17:14.564Z z2m:Zwave Success zwave api call setValue

Oct 05 10:17:25 nuc docker[29982]: 2020-10-05T17:17:25.705Z z2m:App Zwave api call: setValue [ 8, 67, 1, 1, '28.444' ]
Oct 05 10:17:25 nuc docker[29982]: 2020-10-05T17:17:25.708Z z2m:Zwave Success zwave api call setValue
Oct 05 10:17:27 nuc docker[29982]: 2020-10-05T17:17:27.399Z z2m:Zwave zwave node 8: changed: 67-1-1:Heating 1:28 -> 28

@pdbogen I think the device doesn't allow that number, even if you set it from the ui then it writes 28.

Yeah, the problem is that 28.4444 doesn't trigger an actual call to zwave stack at all but 28.444 does. I'd be happy if they both set 28.
HomeAssistant is sending 28.444444444444443 as a result of Fahrenheit to Celsius conversion.

or... maybe the zwave stack itself is treating this differently. :thinking: it could be a bug in openzwave itself, perhaps?

@pdbogen From what I can see I don't see any error here, what you could try to do is create a parse receive function to remove some decimals: https://github.com/OpenZWave/Zwave2Mqtt#gateway-values-table
Something like:
return Number(value.toFixed(2));

@pdbogen Did you try with my suggestion?

I worked around it in my case by changing automations in question to publish mqtt directly with integer values instead of making HA climate changes.
I still intend to dig around and find out why the behavior differs for different numbers of decimal places, but I'm thinking maybe the issue is somewhere with ozw itself, or maybe even it's device-specific. I'll update this issue with any findings..

@pdbogen Ok closing this for now, feel free to comment if there are updates


WIP

todo: tests
Description
Annotated decoder
Checklist
Quality

[ ] Commit sequence makes sense and have useful messages, see ref.
[ ] New tests are added and existing tests are updated.
[ ] Self-reviewed the PR.

Maintenance

[ ] Linked an issue or added the PR to the current sprint of ouroboros-network project.
[ ] Added labels.
[ ] Updated changelog files.
[ ] The documentation has been properly updated, see ref.

Having a single runDecoderWithChannel and runDecoderWithLimit avoids some code duplication to ease maintenance, but to avoid leaking space for codec's that do not need annotation capability there should probably be a custom monoid instance for the bytes type which is strict in the accumulator and essentially a no-op.


Package and Data element creation issue

Hi @marcellourbani / Team,
Need your help to resolve the issue.
We are using ABAP remote FS in VS code to create a package and data elements in SAP ECC but ended with error Error 404:Not Found. Kindly refer the attached document for VS code and SAP error details in the attached.
VS code Error log.docx
Best Regards,
Narayana

Hello @info4Narayana
Looks like an ABAP version issue: your system is either too old or too new for that functionality. Afraid you'll have to do this in eclipse or sapgui
Hopefully most of the rest will work.
What ABAP version are you on?
Best regards,
Marcello

Hi Marcello,
Thanks a lot for swift response.
Please find the print screen for SAP ECC Version details.

@info4Narayana yep, it's aversion issue.
I think the eclipse API I use to do that is available since 7.5 or so. Maybe there's an older one who is, but I don't support it.
Please stick to eclipse/se80 for this. You can probably use vscode for the rest
Marcello


Update README.md

Reviewers: @michael-ciniawsky, @ertrzyiks
Add quotes
sorry, this PR format is too hard for me to read through, cuz I have only a couple of mintues

:warning: Pull Request - Issues :warning:

Commit: 5753ffc09fd48add460f06f7c7d0ac79ab5a6686
Commits must be in the following format: %{type}(%{scope}): %{description}

This message was auto-generated by https://gitcop.com

Coverage remained the same at 95.455% when pulling 5753ffc09fd48add460f06f7c7d0ac79ab5a6686 on vyorkin:patch-1 into 0c335d4946e1510f57a661d70766e788a32c44b5 on michael-ciniawsky:master.


Create the helper functions for memory operations and make the strategies use those helper functions

It seems the typos were already fixed in the most recent commit of the PR, PTAL :)


update README

@lucas-clemente: Please add your copyright sentence.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #472   +/-   ##
=======================================
  Coverage   90.57%   90.57%
=======================================
  Files          80       80
  Lines        5261     5261
=======================================
  Hits         4765     4765
  Misses        316      316
  Partials      180      180

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 c98a620...622e856. Read the comment docs.


How to run a single spec?

Is it possible to run a single spec?

No, it isn't. But I already received some similar feedback. I'm going to support this in a future version.

OK, thank you.
Is it possible to run a single spec file?

You can use -s opstion to run a single section like this:
$ h2spec -s 3.5

Oh, I see. Just like on the documentation. Thank you.

Hello folks,
v2 branch seems to be missing the   -s: Section number on which to run the test. (Example: -s 6.1 -s 6.2) altogether. Is it an intentional design decision or just a missing piece open for pull requests?  @summerwind would you prefer a new approach/syntax or a simple port from the previous version is fine?

@Karm Thank you for following up. Let's close.

Thank you!!!


[BUG] bin/doom install fails

Today I decided to try out emacs 27 HEAD because of performance issues. So I deleted ~/.emacs.d/.local and ran ./bin/doom install with the new emacs version. This lead to the following error:

➜ ckruse@Pug ~/.emacs.d (develop) ✗ % ./bin/doom install
Installing Doom Emacs!

Creating Code/emacs/doom.d/
✓ Created Code/emacs/doom.d/
! init.el already exists, skipping
! config.el already exists, skipping
! packages.el already exists, skipping

Autoload file error: autoloads.pkg.el -> (file-missing Cannot open load file No such file or directory helm-easymenu)
Bootstrapping straight.el...
Bootstrapping straight.el...done
Looking for gnu-elpa-mirror recipe → Cloning melpa...
Looking for emacsmirror-mirror recipe → Cloning gnu-elpa-mirror...
Looking for straight recipe → Cloning emacsmirror-mirror...
Building straight...
Cloning use-package...
Building use-package...
Building use-package → Building bind-key...
Building use-package...
Cloning emacs-async...
Building async...
Cloning gcmh...
Building gcmh...
Warning (emacs): Your package autoloads file is missing
There was an unexpected error:
Error in an autoloads file (doom-autoload-error)
Run `bin/doom refresh' to generate them
Run the command again with the -d (or --debug) option to enable debug
mode and, hopefully, generate a stack trace. If you decide to file a bug
report, please include it!
Emacs outputs to standard error, so you’ll need to redirect stderr to
stdout to pipe this to a file or clipboard!
e.g. doom -d install 2>&1 | clipboard-program
Error in Doom Emacs core: doom-autoload-error, "Run `bin/doom refresh' to generate them"

After running ./bin/doom refresh everything works as expected, even ./bin/doom install.
Steps to reproduce

delete ~/.emacs.d/.local
run ./bin/doom install

System information

emacs   version    27.0.50
        features   RSVG IMAGEMAGICK GLIB NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS JSON PDUMPER LCMS2 GMP
        build      Dec 06, 2019
        buildopts  (--disable-dependency-tracking --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-plus/HEAD-48373e7/share/info/emacs --prefix=/usr/local/Cellar/emacs-plus/HEAD-48373e7 --with-xml2 --without-dbus --with-gnutls --with-imagemagick --with-json --with-modules --with-rsvg --with-ns --disable-ns-self-contained)
        windowsys  ns
        daemonp    server-running
doom    version    2.0.9
        build      HEAD -> develop, origin/develop, origin/HEAD 318754c4d 2019-12-05 20:48:53 -0500
system  type       darwin
        config     x86_64-apple-darwin19.0.0
        shell      /usr/local/bin/zsh
        uname      Darwin 19.0.0 Darwin Kernel Version 19.0.0: Thu Oct 17 16:17:15 PDT 2019; root:xnu-6153.41.3~29/RELEASE_X86_64 x86_64
        path       (~/Code/termitel/termitool/frontend/node_modules/.bin/ ~/.rbenv/shims/ ~/.rbenv/shims/ /usr/local/bin/ /usr/bin/ /bin/ /usr/sbin/ /sbin/ /usr/local/MacGPG2/bin/ /usr/local/opt/go/libexec/bin/ ~/.fzf/bin/ /usr/local/opt/go/libexec/bin/ /usr/local/Cellar/emacs-plus/HEAD-48373e7/libexec/emacs/27.0.50/x86_64-apple-darwin19.0.0/)
config  envfile    envvar-file
        elc-files  0
        modules    (:completion company (helm +fuzzy) :ui doom doom-dashboard doom-quit hl-todo indent-guides modeline nav-flash ophints (popup +all +defaults) treemacs unicode vc-gutter window-select :editor file-templates fold (format +onsave) multiple-cursors rotate-text snippets :emacs (dired +icons) vc :tools ansible editorconfig eval flycheck (lookup +docsets) lsp macos magit :lang data erlang (elixir +lsp) emacs-lisp (javascript +lsp) markdown (org +attach +babel +capture +export +habit +present +protocol) (ruby +lsp) sh (web +lsp) :config (default +bindings +smartparens))
        packages   (n/a)
        elpa       (n/a)

As of cd158c5 this issue should be resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!


got this error message and then proxy is killed.
Have a nice day.
Edit : strange behavior seems like it dosent apply docker restart policy -> i use restart always but it don't work

shame on me forget to detach -> all the time using ansible bad habit forget basic things close terminal close container.


support two way data binding

[x] observ-struct
[x] observ-varhash
[ ] observ-array

cc @nrw

@Raynos can you please describe the problem? thanks

@Raynos this may provide some insight... not sure if it's too slow though... but thought it might help...
https://github.com/benjamine/jsondiffpatch

I think we fixed observ-array, cannot remember.

See this issue ( https://github.com/Raynos/observ-array/issues/9 ). cc @kuraga
I never verified whether all the edgecases were handled.
Also see https://github.com/nrw/observ-varhash/issues/1

Hm... It's difficult to understand, when notes about issues are posted in different components' repositories :smile:

@kuraga distributed conversations :) hard to find them.
It's also not fully documented what two way data binding means. it basically means that if you mutate either the parent or child in both the parent & child get updated.
var parentArr = hg.array();
var childArr = hg.array();

parentArr.push(childArr);

childArr.push(4);
parentArr.get(0).get(0) === 4;

parentArr.set([
  [5]
])
childArr.get(0) === 4;
parentArr.get(0).get(0) === 5;

I'm not really sure what I want the semantics to be in all the edge cases tbh... Whatever makes time travel work.
The whole point is to be able to serialize the entire state of the world at any time and bring it back at any other time. which means appState.set(bigComplexObject) should propogate the changes down correctly to all the places.

An ObservVarhash is a version of observ-struct that allows adding and removing keys. Mutation of an observable element in the hash will cause the ObservVarhash to emit a new changed plain javascript object.

@nrw does it mean than we can use observ-varhash as a replacement of observ-struct in mercury?

@kuraga observ-varhash does everything that observ-struct does, so we could use it as a replacement. I always use struct when i know my keys are static and i use varhash when i know they'll change. are you just looking to reduce dependencies?

I'm looking everything to understand every line of mercury :smile:

i see. :)
If you look at the source, varhash is just struct with a bunch of bookkeeping for adding and removing keys. You could just use varhash all the time, if you like.

I use struct to communicate "these are fixed keys"
I use varhash to communicate "there are N keys"
There is a different, a struct can be like struct({ n: string, m: number })
A varhash should be homogenous so each key should have the same shape varhash({ a: componentT, b: componentT })

With regards to the observable array issue... this problem only arises because the way hg.state is used is to create multiple root state atoms and then merge them. Instead, mercury could use/enforce a single root tree that uses cursors to get/set branches of a single tree. This avoids the entire class of complexity associated with making multiple observable states and then effectively having to merge them when they are set as properties for one another. Does that make sense?

I'm not really sure what I want the semantics to be in all the edge cases tbh... Whatever makes time travel work.

@Raynos or anyone, could you explain this a little further (what would/wouldnt make time travel work, what are examples of edge cases)?  im a bit new to all this, but may contribute as im noticing observ-array being a bottleneck

@cellvia last time I looked at this I observed that using varhash instead of array is better in general.
Most of the time you have an array of objects; you can just create a varhash indexed by id or


Coherance test

•	A S2 and S1 tests to know how many files are there in the API concerning the MPC files.  Deadline June.

This is done and all OK for S1.

To be done for S2.

Coherence tests are OK for S1 and S2.


ARROW-5694: Support list of Decimals in conversion to pandas

Codecov Report

Merging #4661 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4661      +/-   ##
==========================================
- Coverage   89.01%   88.94%   -0.07%
==========================================
  Files         814      713     -101
  Lines      103424    97032    -6392
  Branches     1420        0    -1420
==========================================
- Hits        92062    86305    -5757
+ Misses      11352    10727     -625
+ Partials       10        0      -10

Impacted Files
Coverage Δ

python/pyarrow/tests/test_pandas.py
95.02% <100%> (+0.01%)
:arrow_up:

cpp/src/arrow/python/arrow_to_pandas.cc
91.56% <100%> (ø)
:arrow_up:

cpp/src/arrow/csv/column-builder.cc
95.29% <0%> (-1.77%)
:arrow_down:

js/src/util/fn.ts

js/src/builder/index.ts

js/src/enum.ts

js/src/Arrow.node.ts

js/src/schema.ts

js/src/visitor/vectorloader.ts

js/src/io/whatwg/reader.ts

... and 97 more

Continue to review full report at Codecov.

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


ENH: Simplify multi-module compilation and wiring by auto detecting provides and requires

Add to Module:

autoProvides() as the types this module will provide. (Initially restricted to top level provided interfaces)
autoProvidesAspects() as the aspects this module will provide
autoRequires() as the types this module requires when wiring that are not provided locally but are provided by another module that is in the classpath at the time of compilation

Currently this all needs to be done manually via @InjectModule(provides=... ,  requires=...).  Conceptually with these autoProvides and autoRequires this can be determined at compile time.

Works well on the modular coffee maker example - works without any explicit requires/provides with the autoProvides and autoRequires doing the right thing. I think this is good.

Works well on the modular coffee maker example - works without any explicit requires/provides with the autoProvides and autoRequires doing the right thing. I think this is good.

got a link?

https://github.com/dinject/examples/tree/master/modular-coffee-app
On Fri, 20 Jan 2023 at 13:27, Josiah Noel @.***> wrote:

Works well on the modular coffee maker example - works without any
explicit requires/provides with the autoProvides and autoRequires doing the
right thing. I think this is good.
got a link?
—
Reply to this email directly, view it on GitHub
https://github.com/avaje/avaje-inject/issues/262#issuecomment-1397779270,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABTATLVTYT5DWH24VG4HN3WTHLYDANCNFSM6AAAAAAT2DHUO4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


NLog ShortDate

Hi all. Please help me.
My archive files is current:
log.file-2015-02-17.log
log.file-2015-02-16.log
log.file-2015-02-15.log

lof.file.Today.log

But for example in log.file-2015-02-16.log is store log info about  2015-02-15.
Can i change my config that in log.file-2015-02-16.log was contain information log with the same data(2015-02-16)?
My config:
  <targets>
    <target name="rotationByDate" xsi:type="File" layout="${verbose_inline}"
            fileName="${basedir}/logs/log.file-Today.log"
            archiveFileName ="${basedir}/logs/log.file-${shortdate}.log"
            archiveEvery="Day"
            archiveNumbering = "Rolling"
            maxArchiveFiles="3"/>
  </targets>

Cheers,
JinPachi.

This is default behavior and can't be configured (yet)


Ender Dragon Not Paying

*Detailed description of the issue:
As title, players with the Hunter job do not get paid despite it being defined in the config.
CONFIG SECTION (DELETE IF NOT RELEVANT):
```    Ender_Dragon:
      income: 1045.00
      points: 0
      experience: 100```

Jobs version:  <IP_ADDRESS>
Server Type (Spigot/Paper/etc): Paper/Purpur
Server Version (using /ver): 1.20.2

Do you have record about dragon payment in /jobs info Hunter?

Do you have record about dragon payment in /jobs info Hunter?

It does not @Zrips

hunter.txt
Yes, no hunter related errors.

@Zrips ?

@Zrips ?

@Zrips hello?


Use the search service to render the map at /map

If we can make this map created through a search request we can use the same code basis as the search.So that when the code changes for search the /map reflects the same changes

Currently /map is showing the list of followers/followings.
@mariobehling, do you have any preference to what needed to be showed on /map?
An alternative that I can think of is:

If user is logged in: Show the timeline in the home view as tweet in map
If not logged in: Just show 1000-2000 latest tweets in the global view.

@fuksi Do you know the Facebook activity feed on the top right? The idea is that we have a feed like that on the right. This feed can be swiped away by the user.
Content of the activity feed is: Tweets of people user follows (=following), own IoT devices, other data sources
The user will ultimately be able to control what gets into the feed.
At the moment it is not easily possible to get all tweets of people a user follows except for a long search queue. I have discussed this with @Orbiter a bit already. Let's move this to M3 and get his hinput here.

I missed the last point and did a quick one for demonstration anyway. It's still live for one day on:
http://gofullstack.me:3001/map (requires login to twitter)

This can be a foundation for future implementation during Milestone3

Ok, looks good, but in my screen it goes over the edge. I am worried we develop on top of something, that is not fixed.

[ ] This page should not be scrollable. The map should end on the bottom of the screen, just like Google Maps etc.

[ ] Regarding the activity feeds - yes, nice. If I scroll over the feed or click it, there must be some action on the wall, e.g. show the feed location on the wall. Otherwise there is not point in having a feed on the map.

[ ] Maybe let's try to move the layer control to the left. This way we have more space for the activity feed.

[ ] The bottom items "twitter" and "internet of things" should be rather integrated in an activity feed dropdown menu. At the moment, there is no "internet of things" implemented. So, just take out these two items, please.

@fuksi Let's move that to the end of M3 or depending if the server supports it to M3.5. Currently the feed content is not working as expected, but it looks very nice already!

please retrieve the follower list from loklak_server, documentation is online at http://loklak.org/api.html#account .. please scroll down to "Retrieval of Twitter User Account Data" for details

I have one blocker left in implementing this:

account.json does return list of the followers, but the result doesn't contain the lastest status from that followers => we'll need to still, use hello.js to get to twitter API for the new feeds, and of course there are always limtis
If i'm using hello.js for the feeds, I can't plot the feed on the map because there are no location information about the feeds. What I can possibly create (I'm still wondering about the connection) is a behavior which:

Clicking on a new feeds

Will pop up the info of the user accordingly (sorry for the bad example):

This implementation will change the /map view and its code completely.
@prasht63, @mariobehling I would like your input on this, or a go ahead :)

account.json does return list of the followers, but the result doesn't contain the latest status from that followers => we'll need to still, use hello.js to get to twitter API for the new feeds, and of course there are always limits

@Orbiter Why is not possible to use the data from loklak? What do you need with "latest status from that followers"? Which status?

The new feeds should contain statuses from people you're following( like the home view in twitter when you logged in).
Meanwhile, the new servlet account.json only retrieve the list of people you're following (& following you), along with user details, like this http://pastebin.com/t7GCVSDZ

@fuksi you can go ahead with it.

@fuksi So with "status" you mean "tweets". Now the tweets, where it is usually the account information. I missed that change. It does not make sense. The position of the tweet should - there needs to be the account info. Please change that. Regarding tweets on the map, I will open a separate issue. If we do not show tweets "statuses" then there is no need for the twitter API, right?

account.json does return list of the followers, but the result doesn't contain the latest status from that followers => we'll need to still, use hello.js to get to twitter API for the new feeds, and of course there are always limits

When I retrieve the user data from twitter using the twitter API, then there is actually the latest status embedded. I Then remove this status, because:

account.json must deal with mass-retrieval of user  and therefore does a caching of all that data. Because such data does not change so fast, a response from the cache is valid. Using the cache will allow us to provide mass-data at this interface without hitting the API limitation
leaving the user status inside the account data would mean that the status is not fresh when the result comes from the cache. I could put the status to the chache but it would be of no use
the latest status information does not belong to acount.json, it can be taken from search.json like search.json?q=from:<user>&count=1

I also don't understand why you need the latest status of all followers all at once. As far as I understand, you need the status only if a user is clicked. In that case you can fire up a search.json?q=from:<user>&count=1 to get that status.

True that I can get the lastest tweet with the search servlet, except that clicking is an instant behavior, while loading the result from the server take 2-5 secs (not including plotting the point on the map). Although I will give it a try to implement it this way:

When user logged in the main (home) view, get all user's followers/following already.
After that, have also a background process to get the latest tweets from all following. Save all in the global scope. Define an interval so contiuous requests are not blocked by the server.

This way we can prevent the unwanted delays in user experience.

@fuksi in this feature, the goal is to plot the followers/followings on the map. Here we do not need to plot tweets on the map. How it works: The position on the map is created through the reference location in the profile.
 In a second step we can add two tweets below this info, but this can also be a future feature. For now see the image below. This is already good as an example:

This is also related to https://github.com/loklak/loklak_webclient/issues/359

There is a security issue. After I log out, I still have access to the map with followers and all tweets. This feature could be still available in the search, but it should not be available on the user account map anymore.

This is solved with https://github.com/loklak/loklak_webclient/issues/424


Added Laravel 5.5+ Auto Discover for Service Providers

Inclusion of Composer extra section with Service Provider  details

Coverage remained the same at 83.815% when pulling aa19c6975294b60bf89ac241f9ce5dd89b7df5cd on davidpiesse:master into 53e0d18122bf8b21af3f54a56adb3661683705b3 on njbarrett:master.

Coverage remained the same at 83.815% when pulling aa19c6975294b60bf89ac241f9ce5dd89b7df5cd on davidpiesse:master into 53e0d18122bf8b21af3f54a56adb3661683705b3 on njbarrett:master.

Coverage remained the same at 83.815% when pulling aa19c6975294b60bf89ac241f9ce5dd89b7df5cd on davidpiesse:master into 53e0d18122bf8b21af3f54a56adb3661683705b3 on njbarrett:master.

Thanks for this addition.


failed to send alert data to Slack:Post : dial tcp <IP_ADDRESS>:443: i/o timeout

When ever i tried kapacitopr to post to my slack channel from my personal laptop it is working fine but when i tried to use kapacitor on staging machine , i am getting below error:
E! failed to send alert data to Slack: Post : dial tcp <IP_ADDRESS>:443: i/o timeout
Although , I am able to post to slack using curl post command easily.
I am using kapacitor 0.12.0 version in staging.

Check that your staging environment has network access to the IP <IP_ADDRESS> over port 443.

Hi Nathaniel,
We  are using proxy servers in our staging machines for any external calls .
And using those https proxy I can easily make call to slack channel by shooting a curl post request which hot successfully executed .
But when kapacitor tried to make post request it is showing the error which I already mentioned. So, I am not sure why kapacitor calls to slack channel should fail?
What do you think ,what could be problem here  and if you know alternate way to do that ,please let me know that as well.
Thanks
Varun

@nathanielc . Any updates ??

I'm facing the same issue:
When I execute curl command to post messages to my slack channel, it works fine (by setting https_proxy). But when I integrate with Kapacitor it won't work and gives the following error:
failed to send alert data to Slack: Post https://hooks.slack.com/services/######/######/####: dial tcp <IP_ADDRESS>:443: i/o timeout
Please let me know if something else is missing here.

Seems that Kapacitor doesn't support HTTP proxies well. #841 is a feature request to add proxy support for the various alert handlers and external integrations for Kapacitor.
Let's consolidate this issue there.


Add brew instructions to install the epinio CLI

Epinio can be now installed with Homebrew. Even if it's simple, we have to document it.

Installation from the epinio custom tap will be added after next Epinio's release.


Is it possible if we modify update location user in a minutes not based on latest location user?

so basically location would be update after user move right? so is it possible if i change the methode that i would to get location user after 5 minutes at a time for example

Location information is usually obtained through subscription. When submitting a subscription request, you can set the interval for sending messages. This way, your application will receive current location information at the intervals you specified, even if the user hasn't moved.  You don't need to call the getCurrentLocation API every 5 minutes to get the current location.

i tried to implement your suggestion into my program code by giving an interval of 5 minutes, but unfortunately getting the user's current location is still based on user movement rather than the time interval itself. Actually, this time interval runs well but does not execute the update of the user's current location. Can you provide an example of the program code of what you mean?

iOS or Android?
On Tue, Feb 27, 2024, 2:22 a.m. Wahyu Budi Utomo @.***>
wrote:

i tried to implement your suggestion into my program code by giving an
interval of 5 minutes, but unfortunately getting the user's current
location is still based on user movement rather than the time interval
itself. Actually, this time interval runs well but does not execute the
update of the user's current location. Can you provide an example of the
program code of what you mean?
—
Reply to this email directly, view it on GitHub
https://github.com/royer/simple_bg_location/issues/4#issuecomment-1966231254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3UUM3O2L4HJYV3N3O3NYDYVWXWPAVCNFSM6AAAAABDY3D3D2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRWGIZTCMRVGQ
.
You are receiving this because you commented.Message ID:
@.***>

i tried to implement your suggestion into my program code by giving an interval of 5 minutes, but unfortunately getting the user's current location is still based on user movement rather than the time interval itself. Actually, this time interval runs well but does not execute the update of the user's current location. Can you provide an example of the program code of what you mean?

OK. I check my source, I setMinUpdateDistanceMeters default 5 meter. so on android if move distance less 5 meter will not update.  because it will save power. you can set requestSettings.distanceFilter = 0 when call requestPositionUpdate to receive update if user not move.


[sei-cosmwasm] Refactor sudo finalize block structs to sei cosmwasm

reminder that we need to publish a new version for this


There is a missing command in crib-cheat_git.md

[x] add the missing command line after line 153 (Änderung eines bestimmten files)
[x] add an additional line break
[x] specify the description
[x] make pull request

The commit message is missing. You entered the file name crib-cheat_git.md as message.
A valid message would have been: Add missing command #76


Add a pytest-args option to ddev test

What does this PR do?
This adds a new --pytest-args to ddev test.
Motivation
This allows additional customization of the pytest call done by ddev test, in particular:

Pass a test name to run a single test
Pass --junit-xml to collect junit output.

It should work for appveyor if we run the tests this way:
ddev test $CHECK -pa "--junit-xml $CHECK-\$TOX_ENV_NAME.xml"


Dynamic Agent template: OpenAI key error

Bug Description
I currently want to expand the Dymanic Agent of langflow. However, I get the following error:
AuthenticationError: litellm.AuthenticationError: AuthenticationError: OpenAIException - Error code: 401 - {'error':
{'message': 'Incorrect API key provided: dummy. You can find your API key at
https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code':
'invalid_api_key'}}
Reproduction

Choose the Dynamic Agent template as a new project in langflow
Add your openAI KI
Run the flow

Expected behavior
There should not be an error and the flow should be working
Who can help?
@ogabrielluiz @nicoloboschi @zzzming @jo
Operating System
MacOS
Langflow Version
1.3.5
Python Version
3.11
Screenshot

Flow File
No response

Hey @L-Huber
What version are you using?
I just tried on the latest version 1.0.19.post2 and worked as expected.

@L-Huber
Can you share your flow?

Dynamic Agent (3) - not working.json
sure!

@L-Huber
Make sure all the OpenAI components have the API Key.
I tried to replicate it, but I didn't get the error.

However, if I forget to add an API Key to a component, the error appears.

@carlosrcoelho that is the funny thing: all previous steps until "Hierarchical Crew" work:

so the API key should be correct.

I have the same issue. I think it's an error with "Hierarchical Crew" component. I tried using Groq as the LLM, and there's an issue with that too.
Error building Component Hierarchical Crew:
litellm.BadRequestError: LLM Provider NOT provided. Pass in the LLM provider you are trying to call. You passed model=llama-3.2-1b-preview
Pass model as E.g. For 'Huggingface' inference endpoints pass in completion(model='huggingface/starcoder',..) Learn more: https://docs.litellm.ai/docs/providers

@SavvySupport Do you have any solution to this?

Hello,
Same error here with sequential crew and custom base url for the llm (to use with openrouter). All previous steps work fine, llm works with other types of components (like chains).

Same issue.

I'm experiencing the same issue. Has anyone found a solution? I believe it might be a bug.

Hey @L-Huber
Did you try using the new version 1.1.1? Test it out and let me know.

@carlosrcoelho I will try it out with the new version and let you know. I might actually be a version issue.

@L-Huber
Do you need any assistance with this case? If not, please let us know if this issue can be closed.

@carlosrcoelho I currently cannot even upgrade my langflow version. Is this is known issue?

@L-Huber
Sorry to hear that.
Please give a try using uv.
Doc: https://docs.langflow.org/get-started-installation


Constants.deviceId should be Constants.installationId

In analytics.js, the client ID is set as Constants.deviceId from Expo. But I don't think that's actually supported by Expo - it should be Constants.installationId. I believe deviceid is currently set to installationId, but it is undocumented. For future stability, deviceId should be changed to installationId.
Where clientId is set: https://github.com/ryanvanderpol/expo-analytics/blob/697565cb87e0d54ad6d492faa2c06786f7052f33/analytics.js#L34

@kevwang1 I use the library like this:
import { Analytics } from 'expo-analytics';
import Constants from 'expo-constants';
import { PixelRatio, Dimensions } from 'react-native';

const width = PixelRatio.getPixelSizeForLayoutSize(Dimensions.get('window').width);
const height = PixelRatio.getPixelSizeForLayoutSize(Dimensions.get('window').height);

const analytics = new Analytics('UA-123456-1', {
  ds: 'app',
  cid: Constants.installationId,
  sr: `${width}x${height}`,
});

Please give a look at Expo Analytics Google

@murshudov You've just copied this project without attribution, and made it worse.

This change is now live in 1.0.16


Import of getHashedTransactionID

I have to import it like:
import {BridgeType, ChainStatus, PartialBridgeTxn, RoutingHelper, TransactionType} from "@glitter-finance/sdk-core";
import {getHashedTransactionId} from "@glitter-finance/sdk-core/dist/lib/common/utils/hashedTransactionId";

@Elaixzar are you still facing such issues?


2024 HA can't load ims platform

Logger: homeassistant.setup
Source: setup.py:403
First occurred: 00:42:38 (1 occurrences)
Last logged: 00:42:38
Unable to prepare setup for platform 'ims.sensor': Platform not found (Exception importing custom_components.ims.sensor).

Logger: homeassistant.loader
Source: loader.py:842
First occurred: 00:42:38 (1 occurrences)
Last logged: 00:42:38
Unexpected exception importing platform custom_components.ims.sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 842, in get_platform
cache[full_name] = self._import_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 859, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/ims/sensor.py", line 325, in
class ImsSensor(ImsEntity, SensorEntity, ImsSensorEntityDescription):
TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases

Having the same issue:
Logger: homeassistant.loader
Source: loader.py:842
First occurred: 11:21:34 (1 occurrences)
Last logged: 11:21:34
Unexpected exception importing platform custom_components.ims.sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 842, in get_platform
cache[full_name] = self._import_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 859, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/ims/sensor.py", line 325, in
class ImsSensor(ImsEntity, SensorEntity, ImsSensorEntityDescription):
TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases

Same issue
Logger: homeassistant.loader
Source: loader.py:842
First occurred: 8:26:24 AM (1 occurrences)
Last logged: 8:26:24 AM

Unexpected exception importing platform custom_components.ims.sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 842, in get_platform
    cache[full_name] = self._import_platform(platform_name)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 859, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/ims/sensor.py", line 325, in <module>
    class ImsSensor(ImsEntity, SensorEntity, ImsSensorEntityDescription):
TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases

This entity is no longer being provided by the ims integration. If the entity is no longer in use, delete it in settings

is there any ETA for issue fix?

The issue is because of EntityDescription changes introduced in 2024.1. I'm abroad and on vacation at the moment, so it'll take a while.

@GuyKh Sure thing. Enjoy it!

Same issue for me.
I'll wait for a solution

I see there is a PR fixing this issue from @GuyKh, Who is in-charge of merging it so we can use it?

I am. It's not ready yet

fixed in new IMS version. thanks

fixed in 0.1.20. Thanks @GuyKh !


Intellisense is not woking

after a few minutes the Intellisense stops working, even after restarting the visual studio code

needs more info

Having the same issue here.

I don't know if I should create a new Issue or if I should piggy back on this one and add some infos so here we go:
Intellisense is not working in an Angular V7 project, the only intellisense I get is for selenium (ex: I'll add HttpClient and the auto import only proposes the selenium one)
I have no intellisense on any Angular module (ex: typing HttpClientMod... does not yield any options)
How to reproduce: simple ng new myapp with the latest cli version (7.0.3) and try adding any core module or use autoimport in a service using the httpclien.
Intellisense works when I do an implements and will show lifecycles, same for adding a lifecycle method in the component.
My typescript version is 3.1.3 on both my project and vscode.

@Lakston Create a new issue.
@BhagathB Please follow this guide to submit a proper issue report. This should include:

What are you trying to do?
What went wrong?
Steps to reproduce the problem


Error exporting journeys from classic deployments

Reported by Geza Simon on July 19, 2022.
$ frodo journey export  -m classic -i Example example alpha
Connected to ForgeRock Access Management 7.1.0 Build 4e72fe392c000b0a15027eb41267d01bfd2d2220 (2021-May-10 11:40)
Exporting journey...
⠴ Examplenode:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "AxiosError: Request failed with status code 404".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v18.5.0

Seeing the same in another classic deployment:
% frodo journey export -A ibm
Classic deployment detected.
Connected to ForgeRock Access Management 7.2.0 Build 64ef7ebc01ed3df1a1264d7b0400351bc101361f (2022-June-27 08:15)
Exporting all journeys to separate files...
[==--------------------------------------] 4% | 1/22 | Example[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v18.2.0

This appears to be a regression.


handle the merge of error hooks

The previous PR got merged without this commit

@yorinasub17 @rhoboat the pervious PR was missing the merge part that @yorinasub17 pointed out

Running tests on this PR now. This time. 😅  I'm learning!

Terraform fmt............................................................Passed
goimports................................................................Failed
- hook id: goimports
- files were modified by this hook

test/integration_test.go
test/integration_test.go
test/integration_test.go
test/integration_test.go
test/integration_test.go

I will push a fix

@smitthakkar96 Also please merge or rebase on master, which has the fix from #1980.

@rhoboat can you run the build. I also rebased and fix the pre-commit errors

Are you sure you rebased/merged with master? I don't see the included change to test/integration_test.go.

This is what I see when I do git log @rhoboat

I can even see the change maybe something up with github

Thanks @smitthakkar96 for your contributions and for sticking this through!

No problem! Thanks for quick and detailed feedback and helping to merge error_hooks :)
I was interested in creating the RFC as we were discussing in the previous PR regarding lexical ordering of hooks. Can I go ahead?


fix(input): teaxtarea label squish

Closes #4110
📝 Description

Textarea's inner-wrapper's h-full causes label to be squished
can be reproduced in storybook by setting minRows on Textarea
more prominent when minRows is set to a low value

⛳ Current behavior (updates)

🚀 New behavior

💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
useInput also affects Input but no visible changes are expected

Summary by CodeRabbit

Bug Fixes

Resolved an issue with the textarea label being squished, ensuring proper display and alignment.

New Features

Added a new class to the input component to prevent the label from shrinking when space is limited, enhancing visual consistency.

Thanks! @Peterl561


cronjobber-updatetz image - Does not exists

Trying to deploy cronjobber-updatetz gives error about non existing image, it seems url is not working anymore.
https://quay.io/repository/hiddeco/cronjobber-updatetz
  Warning  Failed     8m48s (x4 over 10m)  kubelet, ip-172-20-54-167.ec2.internal  Failed to pull image "quay.io/hiddeco/cronjobber-updatetz:0.1.1": rpc error: code = Unknown desc = Error response from daemon: Get https://quay.io/v2/: EOF

Quay.io is currently experiencing issues. See: https://status.quay.io

@hiddeco It seems there is still something wrong, qay is okay already, but image is not found. I cannot see 0.1.1 here https://quay.io/repository/hiddeco/cronjobber-updatetz?tag=0.1.1&tab=tags

Sorry about that, should be available now.

Great, works now, thanks!


[SPARK-9200][SQL] Don't implicitly cast non-atomic types to string type.

Merged build triggered.

Merged build started.

Test build #38325 has started for   PR 7636 at commit 3e67327.

Test build #38325 has finished for   PR 7636 at commit 3e67327.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #38334 has started for   PR 7636 at commit 3e67327.

I'm going to merge it since the test failure was due to a flaky test somewhere else.

Test build #38334 has finished for   PR 7636 at commit 3e67327.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.


Add CommonJS package manager support (closes #43)

I'm hesitant to leave the build in a broken state - any way you can get this to work?
Also will this break anything for non-webpack scenarios?

I've fixed the broken build, was missing global variables module and exports in .jshintrc.
But now it says something about the Coverage decreased, I'm not sure how to cover that, since it's not inside the module.
Environments that don't use CommonJS will not enter the if with the module.exports, and for those who do support it, it's only a convenience, people can still use the old way.

@rpocklin Can we bump a release for this one?

Done! v1.6.0 - thanks for bumping me.

Did you npm publish?
How does that work anyway? I mean how does a release tag get on the npm repository?

I wish it was automatic :P  it's published now.


Nothing exported

For some reason, UglifyJS is removing all the assignments to exports, and nothing is exported from the minified file.
requireing the minified file from Node.js results in an empty object.

The cause of this problem is that in order to export stuff in Node.js the code needs to assign fields to module.exports, while UglifyJS assigns global.exports, where global is obtained by returning this from a directly invoked global function.
There is probably only one solution: give up UglifyJS's default wrapper and create manual prologue and epilogue code.


fix:  Added 20% deviation in gasPricing.

Description:
This adds a 20% deviation in the gas price.  Gas prices on testnet vary during the business day.
Related issue(s):
Fixes #2010
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

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

Comparison is base (af2e2b6) 78.46% compared to head (4007339) 77.83%.
Report is 6 commits behind head on main.

:exclamation: Current head 4007339 differs from pull request most recent head d4deced. Consider uploading reports for the commit d4deced to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2033      +/-   ##
==========================================
- Coverage   78.46%   77.83%   -0.63%
==========================================
  Files          27       40      +13
  Lines        2442     3086     +644
  Branches      507      625     +118
==========================================
+ Hits         1916     2402     +486
- Misses        365      479     +114
- Partials      161      205      +44

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


what needs to be defined as a standard.

Hi, Trying to address my own concernz in #34 but also trying to answer questions I have been receiving regarding how OpenID4VP and other decentralized identity can be used to achieve the goals of this project.
I am trying to do two things:

separate what needs to be defined in a standard and what is implementation-specific
clarify where authentication is needed and when authorization is needed

In short, from what I have gathered, I think what needs to be defined in a standard is the following flow: "Data Space X authenticates itself to Data Space Y and requests access to certain resources. (Data Space X has a way to pre-discover out of band what it needs to present to Data Space Y to get authorization for each resource being requested.) Data Space Y makes authorization decision and returns Access Token to Data Space X. Data Space X takes that Access Token to Data Space Y's Server (can also be called wallet, ID Hub, etc.) and Data Space Y returns the resource."
and it this flow is happening twice as per diagram below: 1) Data Space B getting access to Data Space A to get information about Data Space A (part of B authenticating A); and 2) Data Space A getting access to Data Space B to get B's resources it wants (end goal)

Below are implementation details:

If Data Space A' Authorization Server alone doesn't know if Data Space B should be allowed access, B can use to authenticate itself in step 2 using whatever it received in step 1 from A itself.
In this model, Wallet (Server) need to be able to process the Access Token. By using VCs as Access Tokens, one might be able to simplify that.

If above diagram is true, I think defining, a new OAuth grant type might be sufficient where and entity authenticates itself using a sender-constrained Token (aka VC) (in steps 1 and 2) to receive an Access Token (in steps 4 and 6 respectfully). No need to Decentralized Identity Foundation ID Hub/Decentralized Web Nodes (DWN) spec. and probably no need for OID4VP either.
What I still don't understand is if Data Space A is sending a VC to authenticate itself in step 1, why does Data Space B needs more VCs about Data Space A from A's server...

Hi Kristina,
Thanks for your help and insight and for taking the time to create the diagram and writing down your thoughts on this issue. You raise some really good points and we see some areas where the specifications can be improved based on your feedback:

As you highlighted in Issue 33 (Issue 33 (https://github.com/eclipse-tractusx/identity-trust/issues/33)), text needs to be incorporated describing the use of a nonce for verifier requests to a client's Credential Service. This is a known issue and a PR is planned shortly.
There are plans to improve consistency in verbiage and hopefully this work will be done over the course of the next couple of months
It is valuable to reference parts of other specifications where appropriate. As there is not yet a finalized specification format it may be decide to replace those references with inline descriptions if it makes the text clearer
The diagram presented in Issue 36 (https://github.com/eclipse-tractusx/identity-trust/issues/36) does not accurately capture dataspace interactions. Thank you for adding this diagram to explain your thoughts and to help us understand your proposal. While it would be possible to redraw the diagram to reflect the mechanisms at play in a dataspace and thus make it more accurate, it will be necessary to first start by reviewing the DSP specifications and Catena-X use cases. These have been developed over the last three years and should be the basis of discussion. A great starting point are the DSP specifications (https://github.com/International-Data-Spaces-Association/ids-specification) because those outline the problems being addressed and the constraints of the current identity and trust protocol being defined.

It would be great to continue the discussion with the DSP in mind.

I don't understand why this issue was closed when it contains protocol feedback and a concrete proposal? It might not 100% fit the DSP ideas, but I 100% agree that there is value to try to better align with extrem standards/concepts.
A new OAuth grant type that uses either DIDs or some form of client attestation (some form of attestation that this system belongs to company or trust domain XYZ) would go a long way and solve at least the AS related parts of the system in a way to aligns with existing standards and can be re-used.

I'd like to preface the following by saying we appreciate feedback, and, of course, we want to align with existing standards when possible. We also need to make forward progress and deliver a solution in a reasonable amount of time. Fundamental to this is respecting the work that has already gone into these specifications (and the DSP specification). This involves taking the time to engage and understand those specifications and the use cases that have informed them. Asking questions is welcome and fine. Posting issues or solutions that make no effort to follow this process is not.
This issue was closed because it does not contain any actionable items. For example, #33 identifies a specific item in the specification (use of a nonce), which is still open. As both @PeterKoen-MSFT and I mentioned, the diagram does not accurately capture dataspace interactions and is not actionable.  For example:

"Dataspaces" do not communicate with one another
"Wallets" (the term in the specs is "Credential Service") do not manage data resources
The diagram confuses fundamental DSP architectural principles, namely the separation between the control plane and data plane
There is no understanding of the key DSP messaging patterns, including catalog request, contract negotiation, and data transfer request

If one "corrects" (re-writes) the diagram and:

Removes the "dataspace" actors which don't exist in DSP
Removes the "Dataspace B Wallet," which does not play a role in these interactions
Removes steps 6-8, which are misrepresented and are not part of the IATP interactions
Includes the proper actors as defined in the DSP specs or in the current specs here

The resulting presentation flow will only change by introducing a new series of calls to an authorization server to receive an access token for the Credential Service. It does nothing else.
Adding this extra step introduces a series of unnecessary complications:

The requesting dataspace participant agent will need to expose an authorization server and pass an authorization request object as part of its outbound request
The authorization server will need to correlate the request for the access token with the outbound request object, guard against replay attacks, and ensure that the generated access token corresponds to the access privileges needed for the outbound request. This couples the authorization server to the Credential Service rather than the Credential Service to the authorization server as is with the current design. The authorization server will now need to understand an additional grant type (or whatever the IATP defines, see the point below). This is not the case with the current design.
Significantly more remote calls will be introduced into the DSP interactions, particularly because they are based on bipartite message exchanges that must establish trust.
We will need to standardize the request for the access token, which we do not have to do in the current flow as it is not part of the identity and trust interaction. This will significantly increase the surface area of what needs to be standardized and greatly restrict implementation freedom, including two implementations that are currently making use of this.

There was a conscious effort not to expose the authorization server through IATP interactions for the above reasons. You're welcome to make an argument for why this should be done, but please invest time in understanding the DSP specifications and our use cases. Feel free to bring it to the working group as a formal proposal for discussion.
We currently have two working implementations of the presentation flow client: EDC and a proof-of-concept that uses Keycloak. It's therefore not unreasonable to assume the current design can be implemented with many off-the-shelf solutions.
Specifically regarding client attestation in Catena-X, I would recommend looking at its defined credentials (membership, dismantler, etc.) for more details.


Update _slice_collection to use toBands and select

Avoids the expensive call to toList unless the ImageCollection contains more than 5000 images and we are slicing past the 5000th image.
Fixes https://github.com/google/Xee/issues/88 where xee wasn't recognizing new bands added to existing images or renamed bands.

Thanks for the explanation! That makes sense to me. I see how this works
around the limits rather than the specifics.
On Sat, Nov 4, 2023 at 3:31 AM Rylan Boothman @.***>
wrote:

@.**** commented on this pull request.
In xee/ext.py
https://github.com/google/Xee/pull/91#discussion_r1381954762:

@@ -679,31 +679,19 @@ def _slice_collection(self, image_slice: slice) -> ee.Image:
# Get the right range of Images in the collection, either a single image or
# a range of images...
start, stop, stride = image_slice.indices(self.shape[0])

If the input images have IDs, just slice them. Otherwise, we need to do

an expensive toList() operation.

if self.store.image_ids:

 imgs = self.store.image_ids[start:stop:stride]

selectors = list(range(start, stop, stride))
col = self.store.image_collection.select(self.variable_name)
if self.shape[0] <= 5000:  # 5000 == max bands in an Image

 col_as_image = col.toBands()

 return col_as_image.select(selectors)

elif stop < 5000:  # 5000 == max bands in an Image

I don't think so. Even if the start/stride arguments would make the final
image have fewer than 5000 bands, I'm avoiding toList by converting to a
multiband image before slicing so I need to grab the first "stop" bands and
toBands will fail if it would return an image with more than 5000 bands.
—
Reply to this email directly, view it on GitHub
https://github.com/google/Xee/pull/91#discussion_r1381954762, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARXAB77ASVIM3FZUJRBHQLYCUL6BAVCNFSM6AAAAAA627ORW6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMJTGEYTMNRZGY
.
You are receiving this because you commented.Message ID:
@.***>

Discussion with Noel Gorelick on Earth Engine Developer forums about a more efficient way to slice an image collection: https://groups.google.com/g/google-earth-engine-developers/c/fSAo9Jn615U
The recommendation was to get a list of system:index, slice that list, then filter the image collection for the remaining system:index

What is the difference between a system:id and a system:index? We do
something similar to the suggestion with ids.
I have a hunch that the recommended approach won’t work for all cases, and
thus we need a toList() or toBand() alternative implementation.
On Tue, Nov 7, 2023 at 2:00 AM Rylan Boothman @.***>
wrote:

Discussion with Noel Gorelick on Earth Engine Developer forums about a
more efficient way to slice an image collection:
https://groups.google.com/g/google-earth-engine-developers/c/fSAo9Jn615U
The recommendation was to get a list of system:index, slice that list,
then filter the image collection for the remaining system:index
—
Reply to this email directly, view it on GitHub
https://github.com/google/Xee/pull/91#issuecomment-1795029027, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARXAB5N2KOGKFYXGGEUG5TYDD3R7AVCNFSM6AAAAAA627ORW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJVGAZDSMBSG4
.
You are receiving this because you commented.Message ID:
@.***>

We likely need someone more familiar with earth engine's internals than I am to confirm this, but I believe the main differences between system:id and system:index are:

system:id is a unique identifier of an object from one of the collections in earth engine's data catalogue while system:index is a unique identifier of an object within the specific context of one collection. This means you can use the system:id to load an image from the data catalogue, but you cannot use the system:index to do this because the system:index is specific to one collection that could be user created.
system:id behaves like a regular property (i.e. it can be modified and it isn't guaranteed to exist) while system:index behaves differently (it is automatically created if you create a collection of constant images and it cannot be modified by mapping over a collection)

I think the issue with how system:id was originally being used comes from here:
 col = ee.ImageCollection(imgs)

Because this reloads the raw images from the data catalogue and therefore throws out any modifications that a user made to the collection beforehand.
I'm not sure that its possible to have an image collection without the images all having a unique system:index, but I will add the alternative methods back in as a fail safe because I'm not 100% certain of that.
Relevant gis stackexchange answer: https://gis.stackexchange.com/a/390307
Earth engine script where I was messing around with system:id and system:index: https://code.earthengine.google.com/8eb85fdb77aeb1adca98fe5eaa36246c

@boothmanrylan Could you sync and resolve the conflicts? Then, I can approve and run the integration tests against your PR.

@naschmitz I resolved the conflicts.
The tests are passing for me locally, but failing here on the "Authenticate to Google Cloud" step which I'm not sure how to resolve. If there is something I need to change to get that to work please let me know.


BinderHub v0.1 on JOSS

Per https://github.com/jupyterhub/binder/issues/48 we decided that a good first step for citation is to submit a JOSS article for BinderHub. It seems to me that v0.1 is a good candidate for this.
Does anybody have any qualms with me putting together a JOSS proposal that we can review/submit after we release v0.1? That way we'll be able to give people something citable for Binder.
Relevant links

the jupyter guidelines for JOSS.
the JOSS author guidelines

I don't see the paper?

Closing for the same reasons as in https://github.com/jupyterhub/binderhub/pull/428