Support for Sakai LMS - https://www.sakailms.org/

Hi all,
First of all, thank you very much and congratulations for the big success with Safe Exam Browser, it's a super cool project and a great piece of software that serves the academic mission.
I've checked the website and it states that has support for Moodle and Ilias, I'm wondering what needs to be done to announce support for the Sakai LMS https://www.sakailms.org/.
The integration has already been done and we're currently testing it, seems to be working really fine, some Spanish Universities wanted support for SEB and we contributed the integration for Sakai, it's documented publicly here:
https://sakaiproject.atlassian.net/browse/S2U-21
So that's my question, what steps do we need to announce the integration in the SEB website, if it's worth to document that is also available for Sakai 22 and newer versions.
Thank you very much for the guidance!
Miguel

Hi Miguel,
Thanks for your kind words about SEB. It's great that you provided an SEB integration for Sakai! It looks really good looking at the pages you linked.
We are discussing internally how exactly and where on the website we can announce the integration and mention compatibility with Sakai. I'll let you know as soon as I get this confirmed.
Daniel

Thanks @danschlet , count on me as well to release the SEB integration information in the Sakai LMS website.
Miguel


added responsive navbar

Description
Issue #279 fixed and added new navigation bar in home.html code with few enhancements (not major changes). Added "blue" hover effect because the page is related to water.
Type of change
Navigation Bar added
Checklist:

[ ] My code follows the style guidelines of this project - Yes
[ ] I have performed a self-review of my own code - Yes
[ ] My changes generate no new warnings - Yes

Reviewer: Vinit Shahdeo

This functionality is required to be added to the updated code in the feature/frontend branch and not the master branch. You can just copy the required changes you have done to src/home.html in the master branch, and paste it inside the same file in the feature/frontend branch.
Also, consider pulling the latest changes from upstream before pasting in the latter branch.


Turn off react-hooks problems in stories files

Closes: https://github.com/storybookjs/storybook/issues/21115
What Changed
I chatted with @valentinpalkovic and we concluded that instead of writing an automigration adding code that in neither case looks "quite right", we considered maybe adding an exception to the config exposed by the plugin, which turns off the problems reported from react-hooks eslint plugin.
Checklist
Check the ones applicable to your change:

[ ] Ran yarn update-all
[ ] Tests are updated
[ ] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] maintenance
[ ] documentation
[ ] patch
[x] minor
[ ] major

LGTM, I am still curious what @yannbf  thinks about this!

Can one plugin turn off a rule from another plugin? And are we sure hooks always keep working? Or only if they use storybook hooks?

Can one plugin turn off a rule from another plugin?

@kasperpeulen I think this answers your question:
https://github.com/storybookjs/eslint-plugin-storybook/pull/149/files#diff-ded734a061dba2176af9593a93a97d7e478a8aa10774724c78ade7977e783832R13
This very plugin is already doing that.

And are we sure hooks always keep working?

This does not impact runtime of hooks.
There's no difference in react's hooks vs custom hooks for the eslint plugin, you can't disable 1, and not the other.

To resolve this issue, please use the official storybook eslint plugin. the latest version should resolve this issue, see: https://github.com/storybookjs/eslint-plugin-storybook/releases/tag/v0.7.0


Change monitor log

Ops... Yes, I did not add a new key (logpipeline) in the template.

How is your configuration file?


Fix refreshFunction() continuously called on touch/click when no data changed

Also you can stop the refreshFunction if you pass hasMore prop

Also you can stop the refreshFunction if you pass hasMore prop

Actually with regards to resetting state pullToRefreshThresholdBreached, it should always be reset(set to false) right after refreshFunction is being fired, possibly inside:
if (this.state.pullToRefreshThresholdBreached) {
  this.props.refreshFunction && this.props.refreshFunction();
}

I agree, the flag should be made false right after calling it. 100%

Great, i will update the pull request then!

@ankeetmaini, i made a new pull request in #190


Phase 2 ideas

Phase 2 attack ideas
forme: swarm like bees.

goes from high in the map to swoosh close to the player to try and hit him.
changes its formation into some sort of torus that shoots a lazer.

a small amount of cars get separated from the main swarm, and they materialize as anoying minions that try to deal damage. You need to demolish them in their resting phase or something.

Another idea: the boss form is some sort of mecha sphere.

Transforms into some sort of rotating gun, and shoots a lazer that follows you;
Splits in half and focuses with rendering on specific boostpads to recharge its fuel. The player has to take all the focused boostpads before it does to prevent it from recharging;
The fuel gauge is displayed with colored circles above it. When the fuel is too low, the boss drops on the ground;
It generates very slow electric balls that chase after where you're going;
When it's on the ground, it spins like giglypuff, aims you. Player needs to dodge that. If the player does, the boss hits the wall, and the player can demo a couple of red cars. It's slowly rebuilding itself. After that, it goes straight back up in the air;
In the "impossible" and all the harder difficulties, the boss never goes on the ground. You have to demo it while it's in the air;


[ISSUE #2241] reinforcement hook implementation to support abort processing

FEATURE REQUEST
Currently the broker hooks resides in mqtrace package, which doesn't describe hook's general usage. We need to refactor them to a different subpackage
We need to make hooks be able to interrupt the sending/consume process, by return early with error messages.

Plz pass the CI

Coverage decreased (-0.06%) to 51.138% when pulling 6ef4580f464ea1d5eed3f2a9f1028e72a672c0ff on imaffe:affe-0812-chepick into a093f8a638f1a1329b0c7567bf38e8590fa668f8 on apache:develop.


Added comments back and added aliases

@src-code @thierryk

CLA is valid!

Coverage remained the same at 100.0% when pulling 51df51332401a11f385518ea0ae938a46440c654 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling 51df51332401a11f385518ea0ae938a46440c654 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

LGTM, but @thierryk should weigh in before we merge this

Coverage remained the same at 100.0% when pulling 915ac47446b848741c0ea483b2aef1d5ec8b0586 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling f9d3c5ec6a67572c1ce423ab3602da53a99e2559 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling f9d3c5ec6a67572c1ce423ab3602da53a99e2559 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling 659938750e687d524374978f8d0492cd8cf1900d on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling 659938750e687d524374978f8d0492cd8cf1900d on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling 02f9ddb68d43810e727d1453d0e66aa4661c4633 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Coverage remained the same at 100.0% when pulling 02f9ddb68d43810e727d1453d0e66aa4661c4633 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.

Sorry @renatoi, you need to revert the [specificity thing] (https://github.com/yahoo/atomizer/commit/02f9ddb68d43810e727d1453d0e66aa4661c4633) change. My bad...
After that, LGTM, :shipit:

Coverage remained the same at 100.0% when pulling 53c45c781b79bb40e2fabd16043f29e436e29970 on aliases into 3aacadb165e185f6fa3e3a067f4d76982ae5bd90 on master.


Add a test to validate the Communication Matrix matches 'ss'

This test validates the ports in a generated communication matrix match a snapshot of the node's listening ports (created with the Linux ss utility)

/lgtm
/approve


Method that returns all group members || query that returns all the rooms a user belong .

Hey and merry xmas .
I am looking for this : https://docs.rocket.chat/api/rest-api/methods/groups/members
I want to implement a method that removes all the rooms a users belongs to .
Can you help me with either a query that returns all the groups a user belongs to ( and a query for channels).
or a method that returns all the usernames of a group , i am trying to use .getUsernames but it always returns null ( i guess it's not fetching the data from the server)
Thank you for your time .

I am trying to implement this but no luck so far , getting
The parameter "roomId" or "roomName" is required [error-room-param-not-provided]"
public User[] getMembers(Group group) throws IOException {
    RocketChatClientResponse res = this.callBuilder.buildCall(RocketChatRestApiV1.GroupsMembers,null,group.getId());
    if (!res.isSuccessful())
        throw new IOException("The call to get the members of the Group was unsuccessful: \"" + res.getError() + "\"");

    if (!res.hasGroups())
        throw new IOException("Failed to get the members of the group");

    return res.getUsers();
}

}


High-Availability does not appear to share state when logging in

It appears that the onpremise node is expecting some kind of state, perhaps a cookie, when logging in. Speculation: when running two pods in a round-robin load balancer, it is possible to login to the pod that didn't originate the token.
Is there some kind of HA setting I'm missing?
I have S3 configured and working.

https://structurizr.com/help/on-premises/faq doesn't provide any additional information.

I imagine the problem is the round-robin load balancer. Short term, you'll either need to configure sticky sessions instead, or setup session replication/clustering in Apache Tomcat (assuming you're using that). Longer term, we could add a way to configure session state to be stored outside the web server (e.g. in Redis), but that's not available at the moment.

I've added support for storing HTTP sessions in Redis, which is available in the latest version of the on-premises installation, and this should provide a way to share session information across pods when using a round-robin load balancer. See https://structurizr.com/share/18571/documentation#http-sessions for more details.

Excellent work! Which release is this in?
It really says a lot about your group when you can accomplish this kind of improvement. Kudos, really!

Thanks, you're welcome. 2860 onwards -> https://structurizr.com/help/changelog

I integrated this change, but it appears that using a Redis password is mandatory. This makes local development more difficult as you have to then wire in a config into the container with a password (no easy env var). Is it possible to remove that restriction?

Just leaving the password empty should work:
structurizr.redis.password=

Ah I tried structurizr.redis.password=. I'll try again in a bit.

Confirmed working. Awesome! Thanks!


Week 3

I accidentally merged my changes from week 3 into the main branch before you could review them. So I am creating a new pull request.

This just has a change to the CSS and not any Week 3 changes. Did you push the right branch?


Massively speed up relabelling while addressing issues in #71 and #72

Don't start traversing from / if it can be avoided.
Use faster -execdir {} + to avoid running out of file descriptors.
Use -ignore_readdir_race to avoid errors traversing /proc.
Use -prune to stop needlessly traversing directories that will have restorecon applied recursively anyway.

The FC085 error that Travis has raised was already there. The suggestion is to use converge_by but I don't know how to use that when the idempotence is dependent on the outcome of the block.

I'll fix the other errors.
Can you see if it still converges correctly without FC085/converge_by please?

If I don't respond grab me on slack 👍

Did a kitchen test run with centos-6.8, centos-7.4, fedora-25, all good.


Scope tasks to journal in site-admin flow manager

References: https://www.pivotaltracker.com/story/show/86118656
Journal-specific flows should still scope papers/cards to the journal whether or not a site admin is looking at them.

AP + CT

Looks good, happy to see a corresponding test. Before my :+1: though, the failing test needs to be fixed.

@mikem - Good call.  We weren't providing a default flow for a test that required one.  Fixed it.  Will merge after it spins green.


add default sentry config url

@alexgul4enko please create hotfix PRs to master branch

already in master. please merge master to next-generation


Adding support for push notifications to all devices by one request

The changes proposed in this request is for supporting the new Topic push notification provided by Google, which allows the developers to send push notifications to all Android devices with one request instead of sending them in batches including multiple registration IDs.

Fully supporting the "topic" concept by allowing setting the 'to' and 'condition' parameter seems a better solution
From Firebase documentation:

About 'to' parameter: "The value must be a registration token, notification key, or topic. Do not set this field when sending to multiple topics. See condition."
About 'registration_ids' parameter: "Use this parameter only for multicast messaging, not for single recipients. Multicast messages (sending to more than 1 registration tokens) are allowed using HTTP JSON format only."

If only one device token is added to the message then message formatting (->toJson()) should not return 'registration_ids' key with only that one token but a 'to' key
In order to support /topics/global topic it could be a constant in the class to be used in ->setTo()


feat: initial progress reporting

Initial version of progress reporting via tracing spans, closes #389.
Reporting is done in 4 flavors:

standard mode: fancy reporting with spinners
simple mode: simple messages printed
json: not implmented
off: the layer is completely disabled

The tasks start when a span is first entered, and end when the span is closed (dropped), the reason for this is async code where a span is repeatedly entered whenever the futures are polled.
In standard mode the same spinner is reused for all nested spans, and the ongoing task is only done when the outermost span is closed.

standard mode example
https://user-images.githubusercontent.com/25967296/190286758-cbd82564-7f60-4fe5-8bc9-4b0d8b9cabd7.mp4

The agent setup ended up being too quick when I actually got to try it out so I had to add artificial network delays for the vid above...
I also introduced a mirrord-common library for stuff like this that are used from both the cli and the layer.
Todo

I cannot seem to pass env vars to the layer from the cli, so by default the layer has standard reporting enabled as well.
We only check if a span is closed, if it's closed, success is printed, even if the task at hand failed. I'm not sure how to handle this, I'd say it's probably not a huge issue as the error is immediately printed, but a false success message is still ugly or probably even confusing.
Each time I write a tracing layer I need to rediscover how tracing works, so the code is a bit messy and requires a cleanup.

@tamasfe Hey, wanted to check if you were planning to get back to this one anytime soon?

We're in no rush to get this merged, so if you're planning to get back to it sometimes in the near future we'll leave it to you. I think we do like the current approach.

Closing in favor of #537.


Terminal messages

Several messages are printed when opening the application from a terminal,  such as: "2017-11-23 15:40:18.658 Origami Helper[1796:13431] Couldn't set selectedTextBackgroundColor from default ()".
Try to find the cause for these messages. Different platform may generate different messages.

https://bugs.chromium.org/p/chromium/issues/detail?id=641509


Inject and mocks

Hi,
Let's start with praise, I really love your DI design :), not based on tags, using constructors, allows to use private structs - great!.
I was hoping to use it in a project for managing my real deps and mocks, however I have some doubts how could I use it to solve my issue.
Right now my main looks like this (of course it really has +20+ more deps;)):
serviceA := pkgA.New()
serviceB := pkbB.New(serviceA)
serviceB.DoStuff()

Now, what I would like to achieve is to replace real instances with the mocks in my tests, but defining all the deps, in every test pretty quickly gets really annoying as I already have 20+ services. So I was thinking about creating dependency graph and then using mocks instead of real objects.
    graph := inject.NewGraph()

    var (
        serviceA pkgA.InterfaceA
        serviceB pkgB.InterfaceB
    )

    graph.Define(&serviceA, inject.NewProvider(pkgA.New))
    graph.Define(&serviceB, inject.NewProvider(pkgB.New, &serviceA))

    graph.Resolve(&serviceA)
    serviceA.DoStuff()

Great, this looks cool!... but doesn't solve anything as how can I replace real objects with mocks?
I had three ideas

Assign a mock to serviceB var after deps being resolved
    graph := inject.NewGraph()

    var (
        serviceA pkgA.InterfaceA
        serviceB pkgB.InterfaceB
    )

    graph.Define(&serviceA, inject.NewProvider(pkgA.New))
    graph.Define(&serviceB, inject.NewProvider(pkgB.New, &serviceA))

    graph.Resolve(&serviceA)
    serviceB = &pkgMockB.New()

    serviceA.DoStuff()

but this can't work as we replace pointer... i tried to replace value but since this is an interface it seems to be impossible to replace in a interface one type (real struct) with another type (mock struct).
http://stackoverflow.com/a/29545033/916440
Before I continue with my second idea, I still think graph could allow replacing object (maybe extend api?) after graph.Resolve()

Second idea was to do the same but before Resolve(). So I was thinking that maybe library checks if serviceB is nil and only then it creates new instance. But it doesn't do that.
    graph := inject.NewGraph()

    var (
        serviceA pkgA.InterfaceA
        serviceB pkgB.InterfaceB
    )

    serviceB = &pkgMockB.New() // already created so graph could skip creating new instance
    graph.Define(&serviceA, inject.NewProvider(pkgA.New))
    graph.Define(&serviceB, inject.NewProvider(pkgB.New, &serviceA))

    graph.Resolve(&serviceA)

    serviceA.DoStuff()

Last idea is the most obvious one - I mean, it must work :) however it's less efficient/flexible.
Simply to put I need to add code that provides different set of providers for tests and different set of providers for main.
func buildGrap(providerServiceA, providerServiceB *graph.Provider) *graph.Graph {
    graph := inject.NewGraph()

    var (
        serviceA pkgA.InterfaceA
        serviceB pkgB.InterfaceB
    )

    graph.Define(&serviceA, providerServiceA)
    graph.Define(&serviceB, providerServiceB)

    graph.Resolve(&serviceA)

    return graph
}

This should work. The only issue is that every time I create test I will need to provide all providers. Probably adding something like registry for providers could be a good idea. With registry I could create all the mocks by default and then allow in tests to replace the ones that I want.
Any thoughts about this? How you would solve the issue with real code vs using mocks in tests and avoiding the need of recreating mocks in every single test?

Option 3 is what I had in mind, but I understand the desire to be more DRY.
The graph isn't really designed to be mutable other than adding. Sounds like you want a graph.Redefine method or something. You'd still need to have the pointer key tho in order to update the value in the graph. And you'd have to do it before graph.Resolve.
Would you like to make a PR to add that?

Correction: if you just do another graph.Define with the same pointer key before graph.Resolve it will replace the old definition.
So all you need to do for tests is abstract the graph definitions and let the test update the graph before resolving.


Remove RestUtils

Is your feature request related to a problem? Please describe.
The RestUtils class uses a very old Jersey client: com.sun.jersey:jersey-client:1.19.3 (Oct 2016). Because its part of the core it can run into conflicts when using current libs like 'org.glassfish.jersey.core:jersey-client:2.33'
Describe the solution you'd like
We should remove the RestUtils class. Unfortunately its still used in other components and needs to replaced:

Testerra platform module
Testerra selenoid connector

Ok, now we have an issue of incompatibility:
Using Testerra 1.7 and Selenoid connector 1.4 with Maven the old com.sun.jersey lib get in conflict with jersey2 using in Selenoid connector:
NoSuchMethodError: javax.ws.rs.core.MultivaluedMap.addAll(Ljava/lang/Object;[Ljava/lang/Object;)
If you are not using RestUtils, you can fix this by the following workaround.
Add an exclusion for both Maven dependencies for driver-ui-desktop and report-ng:
<exclusions>
    <exclusion>
        <groupId>com.sun.jersey</groupId>
        <artifactId>jersey-client</artifactId>
    </exclusion>
</exclusions>


ndb.PickleProperty UnicodeDecodeError: 'ascii' codec can't decode byte with NDB_USE_CROSS_COMPATIBLE_PICKLE_PROTOCOL

Expected Behavior
No decode issues
Actual Behavior
Decode issues
Steps to Reproduce the Problem

set NDB_USE_CROSS_COMPATIBLE_PICKLE_PROTOCOL:True in app.yaml
Have a ndb.PickleProperty in python27
Access the ndb.PickleProperty in python312
File "/layers/google.python.pip/pip/lib/python3.12/site-packages/google/appengine/ext/ndb/model.py", line 1913, in _from_base_type UnicodeDecodeError: 'ascii' codec can't decode byte 0xe6 in position 1: ordinal not in range(128)

Specifications

Version: latest
Platform: python312

Cross-compatible pickle protocol is for a different problem from the one you are talking about. You probably need to wrap the property access in a getter function which will catch the error, then try to dig out the underlying stored value, and unpickle it 'manually' using a different protocol like 'latin1' - it depends on what you're storing in the property, which you didn't specify.

"NDB_USE_CROSS_COMPATIBLE_PICKLE_PROTOCOL" seems pretty self explanatory to me - it should work out of the box. I have no idea what problem it's intended to solve other than the name as it's not documented anywhere.
Still it's a good suggestion, better than waiting for a star to fall, but I could not find an easy way to access the base value at a quick glance without triggering the unpickle - maybe turning ndb.PickleProperty()'s to ndb.BlobProperty()'s and then unpickleing them with 'latin1' and re-pickleing them with Python3 default pickle is an option - yet it's quite a challenging undertaking as I'd need to manually block all access to my app and risk corrupting data

My Python is rusty but maybe dynamically overriding the section here in code:
https://chromium.googlesource.com/external/googleappengine/python/+/3b4e135a03ddf2132df0c0c8d3f7cdc9eaa8c34b/google/appengine/ext/ndb/model.py
class PickleProperty(BlobProperty):
--
class PickleProperty(BlobProperty):
  """A Property whose value is any picklable Python object."""
  def _to_base_type(self, value):
    return pickle.dumps(value, pickle.HIGHEST_PROTOCOL)
  def _from_base_type(self, value):
    return pickle.loads(value)

with
class PickleProperty(BlobProperty):
--
class PickleProperty(BlobProperty):
  """A Property whose value is any picklable Python object."""
  def _to_base_type(self, value):
    return pickle.dumps(value, pickle.HIGHEST_PROTOCOL)
  def _from_base_type(self, value):
    try: return pickle.loads(value)
    except: return pickle.loads(value,encoding='latin1')

is an easy fix
Is it unreasonable to expect App Engine team to do this themselves when NDB_USE_CROSS_COMPATIBLE_PICKLE_PROTOCOL flag is set?

This ended up being the patch that was refused here https://issuetracker.google.com/issues/322622693
def _from_base_type(self, value):
	try:
		return pickle.loads(value)
	except:
		return pickle.loads(value,encoding='latin1')
ndb.model.PickleProperty._from_base_type=_from_base_type

ah, yeah we're doing something similar just on a case-by-case basis where we know we have datetime objects in the pickle property.. it seems like the "except" should actually specify UnicodeDecode error there though ^
About what NDB_USE_CROSS_COMPATIBLE_PICKLE_PROTOCOL is for, this is all a bit confusing so I could be wrong about this, but my understanding was that it was so you could store a pickled object in a Python 3 instance and still have it be readable by a still-running Python 2 instance, because it will be pickle protocol 2 which Python 2 can still parse (it can't parse the newer, default pickle protocol used by Python 3). Which if I'm correct, this probably matters less now that Google won't let you deploy anything with Python 2 any more.
But a UnicodeDecodeError in Python 3 will still happen with datetime objects pickled in Python 2 in any protocol version without using the 'latin1' encoding, but that's not a bug with this library, it's just a "fact of life" when unpickling Python2-pickled objects in Python 3 (I think :D )


Integrate current activities with eachother

I have tied the login, list, and add/view/edit activities together. You can now login (account names are hardcoded for now) as well as view/add/edit items (synced with DB). Also included is the new version of the add/edit/view activity. That probably should have been it's own PR.

@bpaul2 your UI changes are here too.


Problem with Google APIs

I have installed candy_check and added the following to my Gemfile:
gem 'candy_check'
I have then run 'bundle install, with no problem.
But when I run my rails server in development, I keep getting this error:
`require': cannot load such file -- google/api_client (LoadError)
I have tried as many solutions as I could find on Stack Overflow and elsewhere, such as adding the following the my Gemfile:
gem 'google-api-client', require: 'google/apis'
gem 'candy_check'
But none of these many solutions has worked. Can you help me out and tell me how I can get candy_check to work, as it otherwise looks perfect for my purposes. I have pasted the full error message, below:
$ rails s -p 5000 -b <IP_ADDRESS>
Traceback (most recent call last):
29: from bin/rails:4:in <main>' 	28: from bin/rails:4:in require'
27: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/commands.rb:18:in <top (required)>' 	26: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/command.rb:46:in invoke'
25: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/command/base.rb:65:in perform' 	24: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/thor-1.0.1/lib/thor.rb:392:in dispatch'
23: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/thor-1.0.1/lib/thor/invocation.rb:127:in invoke_command' 	22: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/thor-1.0.1/lib/thor/command.rb:27:in run'
21: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/commands/server/server_command.rb:142:in perform' 	20: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/commands/server/server_command.rb:142:in tap'
19: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/commands/server/server_command.rb:145:in block in perform' 	18: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/railties-5.2.3/lib/rails/commands/server/server_command.rb:145:in require'
17: from /Users/homeuser/develop/ruby/json/json_fin_ruby/config/application.rb:16:in <top (required)>' 	16: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler.rb:114:in require'
15: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:65:in require' 	14: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:65:in each'
13: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:76:in block in require' 	12: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:76:in each'
11: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:81:in block (2 levels) in require' 	10: from /Users/homeuser/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/bundler/runtime.rb:81:in require'
9: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/candy_check-0.0.1/lib/candy_check.rb:4:in <top (required)>' 	 8: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in require'
7: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:257:in load_dependency' 	 6: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in block in require'
5: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in require' 	 4: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/candy_check-0.0.1/lib/candy_check/play_store.rb:1:in <top (required)>'
3: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in require' 	 2: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:257:in load_dependency'
1: from /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in block in require' /Users/homeuser/.rvm/gems/ruby-2.6.3/gems/activesupport-5.2.3/lib/active_support/dependencies.rb:291:in require': cannot load such file -- google/api_client (LoadError)

Same ✋

Same ✋

Same

Released as 0.4.0 but is also part of todays release 0.5.0


Don't fail for "options.organization" in certificate variables

In kubo-deployment's base manifest they are assuming that options.organization is a valid attribute. This breaks for bosh deploy --vars-store. Could we allow it as a noop attribute?
$ bosh deploy --vars-store ...

Evaluating manifest:
  Getting all variables from variable definitions sections:
    Generating variable 'tls-kubelet':
      Failed to generate certificate, parameters are invalid:
        Unsupported certificate parameter 'organization'

Current work around is to provide an operator file to remove all options.organization attributes from known variables.

Implemented upstream in https://github.com/cloudfoundry/config-server/pull/7

Hello, any update on this issue? Do we expect to be solved because of the change in the config-server?
thx!

Hi again! any update?
The reason why I am asking again is that In CFCR team we are going to enforce authorization between apiserver and kubelet. This authorization depends on a cert and specifically on the value of the  organization field.  Therefore --var-store probably will stop working.
Thanks!

This came up again for me due to https://github.com/cloudfoundry-incubator/cfdev (which does not include a config-server).

I have a PR on config-serverthat adds support for organization field: https://github.com/cloudfoundry/config-server/pull/12

What's the status of this? Just ran into the issue myself trying to deploy CFCR.


[Event Request] Application “Tests-Graph”, Codeunit 134625 "Graph Subscription Tests"

[Event Request] Application “Tests-Graph”, Codeunit 134625 "Graph Subscription Tests"

Could you please insert the OnTestInitialize Event at the beginning of the procedure Initialize?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[Suggestion] A Way to Return State of Interior

I was making a script that lets me open the safes via commands, and I thought that being able to get the current state of the safe door would make it easier to change its state without the user needing to specify open or shut.
Something like
local function fo4(i)
	FinanceOffice = exports['bob74_ipl']:GetFinanceOffice4Object()
	if i[1] == "gunsafe" then
		local state = FinanceOffice.SafeState("left")
		if state then
			FinanceOffice.Safe.Close("left")
		else
			FinanceOffice.Safe.Open("left")
		end
	end
end

/fo4 gunsafe

I know this is nearly 4 years later, but cool ideas not sure if applicable to Bob ipl though as a function?


PGVector implementation

Updated my .env to resemble this
DISCORD_TOKEN=
LLM_HOST=
LLM_PORT=
LLM_TEMP=
DB_USER=
DB_PASSWORD=
DB_NAME=
DB_HOST=
DB_PORT=
Information found for DB is on Oori-PGvector on 1pass

Error that were running into:
DETAIL:  Could not open extension control file "/usr/share/postgresql/15/extension/vector.control": No such file or directory.
HINT:  The extension must first be installed on the system where PostgreSQL is running.

Yesterday we were successfully able to connect to the db with an image that we thought included PGvector but this error implies otherwise.  Will need to find out what's really going on here...

ok, getting the issues with installing the vector extension seemingly no matter where you run it from:

restarting the postgres container seems to have fixed things; Kai thinks that this is because we have just failed to do so since adjusting the image we're pulling to one that has pgvector installed.

alright, got a super rudementary return from a table that we created. no vector functionality yet, but we're happy with it for now (https://github.com/OoriData/OgbujiPT/commit/54052212fa9d4e3c826999942ad5d01180b4913b). We should investigate sorting out list weirdness with pgvector-python, and once we have a robust example program, we should take some of the more boilerplate functionality and make it into methods of the PGv class in embedding_helper.py


Lifecycle Logging

I added two abstract classes, LoggerActivity and LoggerFragment, that have logging methods which will add the class instance's hash code to the log data, and the ability to automatically log the class instance's lifecycle callbacks. I find it's really helpful to be able to log and track the lifecycle state of your activity and fragment instances during development, especially when it comes to testing/debugging state management issues. I made updates to Logger, Printer, and LoggerPrinter to support this functionality.
I added some code in LogAssert and LoggerTest to allow some basic testing of the new functions, and fixed the tests for the json and xml logging that weren't working with the output from ShadowLog.
Finally I added a sample app that showcases the library's existing functions and examples of the LoggerActivity and LoggerFragment in action.

Logger scope is quite generic, lifecycle is a subset of the development. Unfortunately cannot increase the scope due to maintenance cost.


6.2.0-preview.2 for debian 9 is missing symbolic links to libcrypto.so and libssl.so

Steps to reproduce

Expected behavior

Actual behavior

Environment data

> $PSVersionTable

Related #7598


Powder Snow

Using a powder snow thats stored in your dank nul replaces your  dank will an empty bucket. How do you change the ID of a dank?

How do you change the ID of a dank?

Click on the right side and type the new id.

Using a powder snow thats stored in your dank nul replaces your dank will an empty bucket.

This is a vanilla blunder that is patchable but out of scope for this mod so I will be adding it to the blacklist.


Crash on inline_code

A markdown statement including inline codes like
Crash on `inline_code` statements

make script crash:
$ mdl make
Making '/Users/hadi/Codes/temp/lecture/lecture.md'...
Error producing PDF.
! Undefined control sequence.
l.4 server \passthrough

Done

It seems something related to pandoc.
One way to solve it is adding
\newcommand{\passthrough}[1]{\lstset{mathescape=false}#1\lstset{mathescape=true}}

to template. [+]


Run TagBot and CompatHelper at random times?

Right now, the default TagBot and CompatHelper templates will run every day at 00:00 UTC.
Instead, what if we made those templates run at a random hour and minute?
I.e. instead of this:
cron: 0 0 * * *

What if we had this?
cron: $(rand(0:59) $(rand(0:23)) * * *

@christopher-dG

This way, each time you run generate(my_template, "MyPackageName"), it picks a different hour and minute for TagBot, CompatHelper, etc.
So then, in the ecosystem, we can get packages running at different times, instead of all at the same time.

Sorry this slipped under my radar for so long.
Personally I don't really like this idea. I much prefer things to be consistent and predictable, and I feel like people are going to wonder what the significance of the numbers are if they see weird ones. It's not like we're bringing GitHub down every hour/day either.
And if you want that for yourself, you can always pass your own cron keyword to those plugins.


Heading text not aligned properly

The title on the main page "We are Programmers, Algo Phantoms is an Open Source organization focusses on Data Structure and Algorithms" is not centered aligned.
I wish to work on this issue under [GSSoC'21].

i want to contribute as a GSSOC participant


Fix an example of python task

When I tried digdag at first, I failed my init project due to python's version.
$ digdag init mydigdag
$ cd mydigdag
$ digdag run mydigdag.dig

The original python print method doesn't work on python3.
print(format) method works on python2 and python 3.
Python 2.7.11 (default, Jan 13 2016, 19:51:48)
>>> print("Step3 of session {0}".format('1'))
Step3 of session 1
>>> print "Step3 of session %s" % '1'
Step3 of session 1

Python 3.5.1 (default, Jan 13 2016, 19:58:59)
>>> print("Step3 of session {0}".format('1'))
Step3 of session 1
>>> print "Step3 of session %s" % '1'
  File "<stdin>", line 1
    print "Step3 of session %s" % '1'
                              ^
SyntaxError: Missing parentheses in call to 'print'

Also, I think it would be better to show recommended python version.

LGTM.
But I prefer
print("Step3 of session {time}".format(time=session_time))

style rather than using magic number {0}. It's just a taste.

👍
Thank you.

Thanks!


Spring(0) returns undefined

Hi!
I'm facing a challenge while trying to use react-motion. I'm working on a photography grid and would like to use this package to have a zoom-in transition.
My div has an absolute position and when you click the top and left values should animate toward 0. Unfortunately spring(0) returns undefined.
Any idea how to solve that issue?
Here's a part of my component:
class Picture extends React.Component {
  render() {
    const {
      originalSize,
      absoluteCoordinate,
      imageURI,
      viewport } = this.props.thumbnailData

    const initialValues = {
      width: originalSize.width,
      height: originalSize.height,
      top: absoluteCoordinate.top,
      left: absoluteCoordinate.left
    }

    console.log(initialValues.left)

    const finalValues = {
      width: spring(viewport.width),
      height: spring(viewport.height),
      top: spring(0),
      lett: spring(0)
    }

    console.log(finalValues.left)

    return (

        <Motion defaultStyle={initialValues} style={finalValues}>
          {
            (extrapolatingStyle) =>
              <div className="picture-zoomed"
                style={{
                  width: extrapolatingStyle.width,
                  height: extrapolatingStyle.height,
                  top: extrapolatingStyle.top,
                  left: extrapolatingStyle.left,
                  backgroundImage: `url(${imageURI})`
                }}>

          }
        </Motion>

    )
  }
}

Found a typo in my code and now it's working 😓


problem when using LAPACKE_dpotrf with dynamic array

Hello,
I'm trying to use the LAPACK Cholesky decomposition function. When I use static array as follows it works fine:
double a[9] = { 1.0, .6, .3, .6, 1., .5, .3, .5, 1 };
int info = LAPACKE_dpotrf(LAPACK_ROW_MAJOR, 'L', 3, a, 3);
However, when I use dynamic array
int n=59;
double * a = new double[n*n];
//assign values to a
int info = LAPACKE_dpotrf(LAPACK_ROW_MAJOR, 'L', n, a, n);
It's complaining about the 5th parameter, which is the LDA. The result stored in a doesn't make sense.
I wonder if anyone had this issue.
Thanks in advance.

What should be cholesky factorisation of uninitialised memory (typically all zeroes)?

I assume the "//assign values to a" is just a placeholder for brevity, but perhaps it would help to print the contents of the array before the dpotrf call to make sure that the allocation and assignment worked (could be a memory management error much earlier in the code that leads to undefined behaviour here) . Perhaps it would also help to know if the same error occurs when the program is linked against the standard netlib lapack.
(Also what is it complaining about - usually the routine will abort without producing a result in such cases, so what is stored in "a" would probably still correspond to the input.)

Thanks for the reply.
Yes,  the "//assign values to a" is just a placeholder for brevity. I printed out the array, and the allocation and assignment works fine. The error is: "Wrong parameter 5 in LAPACKE_dpotrf_work". and info is -5.
Yes, after running the function, "a" is still the same as the input (the lower triangle is not overwritten by the Cholesky L matrix), which means operation is aborted.
This piece of code is embedded in some class. If I try to do something similar in a standalone main() function, I don't have this issue. Guess there's some memory management error, but I don't know what it is.

If you are running this code on some form of Linux you could try using valgrind to check if memory management goes wrong somewhere earlier. As noted above you could also try with plain netlib lapack/lapacke in case it is some other function in OpenBLAS that has unintended side effects. Which version of OpenBLAS and what CPU and operating system is this all about by the way?

I'm using the binary of: OpenBLAS-v0.2.14-Win64-int64
CPU: Intel(R) Xeon(R) CPU E5-1650
Am I able to download netlib lapack/lapacke from the same sourceforge page as openBLAS? (sourceforge is forbidden in my workplace).
Thanks.

0.2.14 is quite old, two years actually, there should be prebuilt binaries of at least 0.2.19 available (though I am not sure if these include a Win64-int64 version). Reference netlib version is on netlib.org,
which links to http://icl.cs.utk.edu/lapack-for-windows/lapack/ for the windows binaries.
Pity about sourceforge being blocked, but I know there was some dodgy behaviour with adware/malware getting bundled with unsuspecting projects under their previous owner.

Hi Martin,
I switched to the netlib lapacke you referred to, and it works! So it might be a OpenBLAS issue.
BTW, I tried 0.2.19 Win32, and it has the same problem.
Thanks a lot.

Thanks for checking. This will be hard to track down if the problem is caused by some other OpenBLAS function that got called sometime before LAPACKE_dpotrf(_work) - would you happen to know which
function that could be ?

There are some mentions of dpotrf in netlib release notes during last years. Probably later LAPACK than from 2014 will have problem fixed.

Martin, I'm not using any other OpenBLAS function before LAPACKE_dpotrf.
brada4, OpenBLAS-v0.2.19-Win64-int32 was released in 2016. So it should include LAPACK fixes from 2014?

0.2.19 at least had netlib 3.6.0 already (0.2.20/0.2.21 has 3.7.0, but there are no windows binaries yet) so it would have to be a recent fix. (On the other hand, seeing that no other lapack or blas function gets called before the dpotrf does make it likely that the problem is in the dpotrf code itself. I notice that OpenBLAS seems to supplant the potrf source it inherits from netlib with an own, parallelized version - assuming that one gets used on Windows at all.)

Neither lapacke_dpotrf.c nor lapacke_dpotrf_work.c changed significantly in the past two years, and at the time the _work function gets called obviously none of the actual potrf code has run. So perhaps this is some more general ABI problem calling the provided (probably mingw-compiled) library from a (possibly) MSVC-compiled main program. (Needs input from someone more familiar with the windows builds I guess)

This could be a problem with mingw-w64: see: https://github.com/xianyi/OpenBLAS/issues/1270#issuecomment-321372302 and https://github.com/scipy/scipy/pull/7616#issuecomment-321371519

When using netlib lapacke as suggested by Martin (which works), I used the same copy of mingw64 dll files.
I even renamed libgfortran-3.dll as libgfortran_64-3.dll since I don't have libgfortran_64-3.dll. It didn't cause problem though.

@yhuo2017, can you run gcc -v on the command line and show the output here? It would be helpful to know the used mingw-w64 version you are using.


Compiling c++

I have a c++ code using tesseract api, all the functions link well but a single function doesn't:
undefined reference totesseract::TessBaseAPI::SetImage(Pix*)'`
the code uses other tesseract base api functions like getUTF8text and linker doesn't complain.

Hi @KarimTarabishy , can you produce some information like:

Tesseract version (commit hash)
Operating system
GCC configuration for building your c++ code

Best regards,
Le Duc. Nam

Please move this question to our forum if you still have this issue.

@zdenop, please close this issue.


Add section on Hermit rebranding

Thanks, @phil-opp :)

Thanks for submitting, I like the new logo!
Maybe you could add a sentence or two to introduce the Hermit project, for those who don't know it yet? You could also link to https://rust-osdev.com/showcase/rusty-hermit/, if it's still up-to-date.

Thanks for the feedback. I added a few sentences.
I'll also rework the showcase on Monday. :)

Thanks a lot!


Update operator pinning plugin - other metadata replacements

Take non-pullspec metadata from _fetch_operator_csv_modifications and use it to add/append/update/replace data in the OperatorManifest
Signed-off-by: Ben Alkov<EMAIL_ADDRESS>
CLOUDBLD-4179

Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

@ben-alkov if I'm reading the code correctly, I see that it assumes the "append" and "update" values are dictionaries where each key has a value of a dictionary with primitive values. I would rather see the code traverse the dictionaries until a primitive or list of primitive values are reached. Then only those values would be updated.
For example, if we had the following modifications file (made up example). I wouldn't want OSBS to overwrite the annotations dictionary with {"description": "Some new description"}. I would just want it to modify the description key.
{
   "update": {
       "metadata": {
           "name": "app.v1.0.1-01610399900-patched",
           "annotations": {
                 "description": "Some new description"
           }
       },
   }
}

The downside of not doing this is Freshmaker would have to pass in the whole annotations object, and thus have to read the CSV file on its end which is undesirable.

@ben-alkov if I'm reading the code correctly, I see that it assumes the "append" and "update" values are dictionaries where each key has a value of a dictionary with primitive values. I would rather see the code traverse the dictionaries until a non-dictionary is reached. Then only those values would be updated.
For example, if we had the following modifications file (made up example). I wouldn't want OSBS to overwrite the annotations dictionary with {"description": "Some new description"}. I would just want it to modify the description key.
{
   "update": {
       "metadata": {
           "name": "app.v1.0.1-01610399900-patched",
           "annotations": {
                 "description": "Some new description"
           }
       },
   }
}

The downside of not doing this is Freshmaker would have to pass in the whole annotations dictionary, and thus have to read the CSV file on its end which is undesirable.

Do we want to allow modifications to everything here? There won't be any restrictions what can be modified?

@ben-alkov if I'm reading the code correctly, I see that it assumes the "append" and "update" values are dictionaries where each key has a value of a dictionary with primitive values. I would rather see the code traverse the dictionaries until a non-dictionary is reached. Then only those values would be updated.
For example, if we had the following modifications file (made up example). I wouldn't want OSBS to overwrite the annotations dictionary with {"description": "Some new description"}. I would just want it to modify the description key.
{
   "update": {
       "metadata": {
           "name": "app.v1.0.1-01610399900-patched",
           "annotations": {
                 "description": "Some new description"
           }
       },
   }
}

The downside of not doing this is Freshmaker would have to pass in the whole annotations dictionary, and thus have to read the CSV file on its end which is undesirable.

Do we want to allow modifications to everything here? There won't be any restrictions what can be modified?

In my opinion, it'd be ideal for the CSV modification feature to be limited to Freshmaker, but I don't know if that adds too much complexity to OSBS.
I also don't see the harm in not restricting this as long as OSBS flags the CSV as having been modified in the Brew build metadata.
@lcarva, what are your thoughts?

@ben-alkov if I'm reading the code correctly, I see that it assumes the "append" and "update" values are dictionaries where each key has a value of a dictionary with primitive values. I would rather see the code traverse the dictionaries until a non-dictionary is reached. Then only those values would be updated.
For example, if we had the following modifications file (made up example). I wouldn't want OSBS to overwrite the annotations dictionary with {"description": "Some new description"}. I would just want it to modify the description key.
{
   "update": {
       "metadata": {
           "name": "app.v1.0.1-01610399900-patched",
           "annotations": {
                 "description": "Some new description"
           }
       },
   }
}

The downside of not doing this is Freshmaker would have to pass in the whole annotations dictionary, and thus have to read the CSV file on its end which is undesirable.

Do we want to allow modifications to everything here? There won't be any restrictions what can be modified?

In my opinion, it'd be ideal for the CSV modification feature to be limited to Freshmaker, but I don't know if that adds too much complexity to OSBS.
I also don't see the harm in not restricting this as long as OSBS flags the CSV as having been modified in the Brew build metadata.
@lcarva, what are your thoughts?

I believe we talked about restricting this to just isolated builds which is not necessarily limited to just Freshmaker.

@MartinBasti; Needs more tests, but PTAL?

@MartinBasti; PTAL @ tests - am I headed in the right direction? Anything I should do differently?

@MartinBasti; While writing more unit tests, I discovered that the 'append' code won't work if the list in question isn't directly under a top-level key (e.g. 'spec').
e.g. trying to append to 'spec': {'install': {'spec': {'deployments': [] fails when it hits the first non-list...

Should work in the same way as update (recursive lookup for attributes)

@MartinBasti; PTAL. Unit tests are broken due to schema validation (LOL), but I tested the 'append' code by hand and it should work as intended.

Why implementation differs between update and append, when there is only difference how the final element is handled?

Why implementation differs between update and append, when there is only difference how the final element is handled?
Update will create missing objects, but append won't. I see this as an inconsistency.

Append does create missing objects

https://github.com/containerbuildsystem/osbs-docs/blob/master/docs/admins.rst#server-side-configuration-for-atomic-reactor
Please add also documentation about new config option ^

Comments addressed here https://github.com/containerbuildsystem/atomic-reactor/pull/1599


use of image with a map source

Is it be possible to use this with a map source?
i.e.
<img src="ChangeView.png" alt="" width="545" height="706" usemap="#ChangeView"/>
<map name="ChangeView">
<area shape="rect" coords="50,72,249,185" href="#" alt="">
<area shape="rect" coords="295,74,495,185" href="#">
<area shape="rect" coords="295,226,494,339" href="#">
<area shape="rect" coords="51,227,249,338" href="#">
<area shape="rect" coords="49,377,251,491" href="#">
<area shape="rect" coords="295,378,496,492" href="#">
<area shape="rect" coords="52,531,248,642" href="#">
<area shape="rect" coords="295,531,495,642" href="#">
</map>

No, OSD does not support image maps. You can use a regular image with the simple image tile source:
http://openseadragon.github.io/examples/tilesource-image/
...but you'll have to handle the clicks yourself. Something like this:
var areas = [
  {
    bounds: new OpenSeadragon.Rect(50, 72, 249 - 50, 185 - 72),
    url: 'http://example.com'
  },
  {
    bounds: new OpenSeadragon.Rect(295, 74, 495 - 295, 185 - 74),
    url: 'http://openseadragon.github.io'
  }
];

var viewer = OpenSeadragon({
  tileSources: {
    tileSource: {
      type: 'image',
      url: 'ChangeView.png'
    },
    width: 545 // it'll set the height automatically based on the aspect ratio
  },
  // Additional options as needed; see http://openseadragon.github.io/docs/
});

viewer.addHandler('canvas-click', function(event) {
  var viewportPoint = event.pointFromPixel(event.position);
  areas.forEach(function(area) {
    if (area.bounds.containsPoint(viewportPoint)) {
      window.location.href = area.url;
    }
  });
});

many thanks for your reply. I think I can make this work


Feature: Remove the vertical margin between compact rows

What feature or improvement do you think would benefit Files?
I would love it if the margin between items was removed when in compact mode. The current row height is great in compact mode, but the selection highlight looks weird, especially when multiple items are selected.
Requirements

The height of a compact row should not change
The padding should be removed between compact rows
The hover/selection background height should be increased to make it the full height of the row

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.1265
Comments
Here is a comparison of Files vs Explorer:

Even explorer has alignment issues... 😆
Here is how compact vs normal mode looks on OneDrive.com which also uses the new WinUI design:
Normal:

Compact:

Original comment: https://github.com/files-community/Files/issues/11372#issuecomment-1436108821
Thanks for this amazing app. It's so much better than the built in File Explorer! ^_^

All four corners of the row should have rounded corners
If the row above is also selected, the top corners should no longer be rounded
If the row below is also selected, the bottom corners should no longer be rounded

This is out of scope, we can change the margin but the rounded corners is too much.

@Josh65-2201 That does look like they are asking for the same change, yes.


Colors for the built-in terminal.

I really love the team! The biggest issue I have with it, is the lack of style in the built-in terminal. It looks like it's using the default colors like the built-in themes.

@Jamerrone thanks for checking out the theme! I'll take a second look at the terminal and see if there are any alternatives that I can tweak.

My biggest issue is related to the contrast of the colors, for example:

This should be fixed:
Light

Dark


License Missing

I might have missed it, but I didn't see a license for this proj.

Please see the bottom of README


Updated packages to latest

Nothing to see here, just updating to latest
Tested by running locally and building out all workers

Closing in favour of Jess's pr which does many things


Still can't play the game.

Describe the bug
When I try to merge the files, the only options I get are "Stop" and "Replace". I'm also given the "do not have permission to open this" message when I try to use it. And when I'm trying to open huniepop, it gives me the same steam message it always does which is "Mac OS no longer supports 32-bit apps". Please help and thank you very much.
To Reproduce
Didn't really do anything in the game files other than trying to add the contents but it didn't really work.
Expected behavior
I expected for the game to be able to open with it.
System information (please fill out as many questions as you can):

Game version: [e.g. HuniePop 1.2.0 Humble Bundle, GOG, physical release, etc.]
System OS: macOS Big Sur 11.1
System model: 13-inch Macbook Pro 2020 Model
System CPU: Intel Core i5
System GPU: Intel Iris Plus Graphics 1536 MB
Display: 13"

It is known to fail with the Steam version. This is clearly stated in the readme.

It is known to fail with the Steam version. This is clearly stated in the readme.


Make app directory configurable via env variable

Given this type of structure, where cra lives in client
├── client
│   ├── README.md
│   ├── public
│   ├── src
│   ├── test
├── docker-compose.yml
├── package.json
├── server
│   ├── src
│   └── test
└── shared
    └── src
    └── test

Running APP_DIRECTORY=./client yarn start would make the app start, build and be tested with no issues
Obvisouly this is a POC and will need documentation and tests, but before going that far I'd like to check the opinion on the approach
Thanks

Hi @lordoffreaks, it's always best to start an issue before creating a PR - as we like to discuss this sort of thing (so that time isn't wasted by you/other contributors).
I think this makes sense, but also want to point out that there are likely going to be other issues with this approach. I don't see any tests, and I assume TypeScript support would fail in this case.
Have you also considered a monorepo?

@mrmckeb Thanks a lot for the guidance and sorry for following the wrong procedure.
I'll close the PR and open an issue.
Thanks a lot for the monorepo suggestion, that was the solution I finally went with


upgrade opentracing api to 0.33.0

Pull Request Test Coverage Report for Build 140

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 27.184%

Totals

Change from base Build 138:
0.0%

Covered Lines:
168

Relevant Lines:
618

💛  - Coveralls


"Display error details" link hard to read in Dark theme

"Display error details" link is legible in Light mode ...

... but not legible in Dark mode (should check legibility for both visited and unvisited link states) ...

Hi @theresa-lee, would you be up for taking this one?

I believe this was already fixed in #290 :)

Right, I think I was looking at the alternative UI which was not synced with master or merged into master at the time. Closing.


cannot unpack non-iterable bool object

api stop working and show this error
`[]
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/requests/models.py", line 910, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python3.7/json/init.py", line 348, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.7/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.7/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/content/TiktokApi/api.py", line 92, in getUserFeed
resp = res.json()
File "/usr/local/lib/python3.7/dist-packages/requests/models.py", line 917, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: [Errno Expecting value]
    # i removed the html code because hes is big!!!!!

TypeError                                 Traceback (most recent call last)
 in ()
41
42 while True:
---> 43     userFeed, cookie = Api.getUserFeed(username=user_name, max_cursor=max_cursor)
44     dl = Download(cookie=cookie, path=folder1)
45     for post in userFeed['itemListData']:
TypeError: cannot unpack non-iterable bool object`
i removed the html code from the error report because he is big!!!!!
this error happen with getMusicFeed and with getUserFeed

tiktok changed the API so currently the APIs are not working, I will update the API

thanks for your concern, waiting for update

updated


Try out Angular wrapper for Flickity

https://github.com/benjamincharity/angular-flickity

It's GPLv3


[firebase_auth_web] authentication status not persistent

Describe the bug
According to firebase auth doc, the default behavior of firebase auth persistency is LOCAL meaning it should persistent across refresh. However, that's not the case with firebase_auth_web.
Setting the persistency manually did help either:
https://github.com/tianhaoz95/iwfp/blob/faaabf9788fc393888a66a7b864e582fca4b8952/iwfpapp/web/index.html#L12-L26
To Reproduce
Steps to reproduce the behavior:

setup firebase_auth for web app
sign in
refresh the browser
user is signed out

Expected behavior
Users should still be signed in after refreshing the tab.
Additional context
Fun fact: it persists during development, but not when the app is compiled.

I have found the solution:
Instead of this:
await _firebaseAuth.currentUser();
use this:
await _firebaseAuth.onAuthStateChanged.first;

Hi @tianhaoz95
does the proposed solution works for you?
If doesn't can you please provide
your flutter doctor -v ,
your flutter run --verbose
and your pubspec.yaml
or if possible a reproducible minimal code sample.
Thank you

@iapicca yes it worked for me. You can probably mark as fixed if that is the suggested approach.

This is a bug if the implementations of the platform interfaces have different results.
Just checked and the JavaScript SDK treats it as a property: https://github.com/firebase/firebase-js-sdk/blob/master/packages/auth/src/auth.js#L928
I think we should update the getCurrentUser function to track the initialization and use the fix above to wait for the first authChanged response whilst the library is still initializing.

lgtm

@iapicca what is left in this issue? Do we want to use it as a tracking bug for the JS SDK fix?

Hi @tianhaoz95
I think you are right, thanks for letting me know
@slightfoot is it ok for you if I close this?

This fix cannot be put in the JS SDK. Assign this ticket to me.

any updates on this issue ?

Hey 👋
The rework of the firebase_auth plugin as part of the FlutterFire roadmap was published over a week ago with a ton of fixes and new features. Please could you try the new version and see if this is still an issue for you? If it is then please submit a new up to date GitHub issue.
We also added support on web for configuring persistence, see: https://firebase.flutter.dev/docs/auth/usage#persisting-authentication-state
For help migrating to the new plugins please see the new migration guide: https://firebase.flutter.dev/docs/migration


Broken markdown in docs/west_seattle/proposal.md

https://github.com/dabreegster/abstreet/blob/master/docs/west_seattle/proposal.md claims to be a markdown file, but has a failing iframe code.
It may be deficiency of github markdown and unfixable.

It shows up here: https://dabreegster.github.io/abstreet/west_seattle/proposal.html
I searched around for a way to embed videos in Markdown, and the iframe approach is the only I found. I might try to reorganize most of the docs around the .io page eventually.


pm open,type:2 -> endless loop

Library: 2.6.0
esp8266 @ 2.4.0
As soon as the wifi drops a pm open,type:2 (dont know the cause) the whole sketch breaks down.
client.loop(); seems to be processed as expected but takes so long it seems it results in a timeout every loop.
        if (_client->available()) {
			DEBUG_MSG("D: loop() | avail = %d | ", _client->available());

            uint8_t llen;
            uint16_t len = readPacket(&llen);

			DEBUG_MSG("D: loop() | llen = %d | len = %d | ", llen, len);

returns 2 bytes available .. and readPacket indicategs 2 bytes read.
I modified the timeouts

// MQTT_KEEPALIVE : keepAlive interval in Seconds
#ifndef MQTT_KEEPALIVE
#define MQTT_KEEPALIVE 8
#endif
// MQTT_SOCKET_TIMEOUT: socket timeout interval in Seconds
#ifndef MQTT_SOCKET_TIMEOUT
#define MQTT_SOCKET_TIMEOUT 8
#endif
But this seems to have no influence

For now I can't reproduce it .. ;_( My general assumption is that when the network connectivity is low and the esp8266 received a chunk of packets .. after processing all packets and the last packet is below a certain threashold and thus no valid mqtt packet the client might get stuck.


Lots of blank pages when exporting with any format using export.

Describe the bug
Lots of blank pages when exporting with any format (pptx,md or png ) using export.

{
  "name": "slidev",
  "type": "module",
  "private": true,
  "scripts": {
    "build": "slidev build",
    "dev": "slidev --remote --bind --open",
    "export": "slidev export"
  },
  "dependencies": {
    "@slidev/cli": "^0.49.4",
    "@slidev/theme-default": "latest",
    "@slidev/theme-seriph": "^0.25.0",
    "mokkapps": "^1.3.0",
    "slidev-theme-hep": "^0.1.1",
    "slidev-theme-light-icons": "^1.0.2",
    "slidev-theme-mokkapps": "^1.3.2",
    "vue": "^3.4.21"
  },
  "devDependencies": {
    "playwright-chromium": "^1.42.1"
  },
  "slidev": {
    "colorSchema": "light"
  }
}

Desktop (please complete the following information):

OS: Mac BigSur 11.6, M1 MacBook Pro
Browser: Chrome 125.0.6422.112
Slidev version: 0.49.4

Please provide a minimal reproduction.
Thanks.

Why reproduction is required

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required


Popup response for submission failed and successful is very similar

Steps to reproduce:

Put limit order IOC for price under the current market price so that it does not get hit
Approved in wallet with enough assets to fulfill order
At first glance, even though the submission failed, which is also in the text, the popup screen is very similar with colour and (including check V) with an order that is successful
I have included two images, one that is successful and one that failed.

this happened because the transaction status on IOC failure was default. it is a bug and I will address this in a new PR


Can't connect to remote jupyter kernels in newer vscode versions

Type: Bug
I seem to have difficulties connecting to remote kernels on the newest version of vscode for whatever reason. It looks to be able to connect to the remote server just fine, if I alter the token for example I get connection refused, but once I connect the list of kernels is just empty. The problems started to appear right after beginning to actively use conda environments on the remote server. That might have something to do with it. I can work around the issue by reverting to an earlier version of vscode, in this case 1.75 (Though I haven't tried others) and connecting via the older interface "Specify Jupyter server for connection" rather than the newer "connect to existing kernel" (or whatever it is called).
Strangely, as I've updated to 1.79 again I can access kernels I connected to in 1.75 just fine, just not any new ones. I now have the old "Specify Jupyter server for connection" option in my palette, but that doesn't seem to detect any new remote kernels either.
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz (8 x 2419)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.73GB (4.74GB free)

Process Argv
--crash-reporter-id 20cf60f7-e771-4643-af42-27d89f197a7c

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

vscode-autohotkey-plus
cwe
2.7.1

copilot
Git
1.94.220

language-cython
guy
0.0.7

better-cpp-syntax
jef
1.17.2

vscode-docker
ms-
1.25.2

isort
ms-
2022.8.0

python
ms-
2023.10.1

vscode-pylance
ms-
2023.6.40

jupyter
ms-
2023.5.1101742258

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.295.0

remote-wsl
ms-
0.79.5

cmake-tools
ms-
1.14.34

cpptools
ms-
1.16.3

cpptools-extension-pack
ms-
1.3.0

makefile-tools
ms-
0.7.0

cmake
twx
0.0.17

(4 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30776702
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyonecf:30548226
vscccc:30610679
2e4cg342:30602488
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmv:30756943
fixshowwlkth:30771522
pythongtdpath:30769146
i26e3531:30780429
pythonnosmt12:30779714
pythonidxpt:30772539
pythonnoceb:30776495

Thank you for filing this issue and sorry you are running into this.
Please could you share the logs from the Jupyter extension when you run into this problem

Please replicate the issue
Then please could you copy and upload the logs from the Jupyter output panel, you can get there output using the command Jupyter: Show Output. Please remember to copy and upload all of the output.

Hello, thank you for the response.
I seem to be unable to replicate the issue, unfortunately. Despite it having reoccurred numerous times previously. I tried reinstalling 1.78 and connecting to the remote kernel from there. This worked and upon updating to 1.79 I was still able to connect to my remote kernel. Neither restarting my laptop nor reinstalling vscode causes the issue to reoccur.
If this (understandably) isn't enough to go off of to find out what the problem was, could we perhaps close this issue and allow me to reopen it if the problem pops up again?

Thanks for getting back, as suggested I will close this
But if it does occur again please do share the logs


Added passthru functionality

There are now some configuration options under System / Configuration / Developer / Email Logger which make it possible to enable/disable the module and enable/disable passthrough.


CI legs need to to be updated to latest

Fedora29 -> 30 update can be a problem ATM. Faced an issue regarding set locale to utf-8.
cc @MichaelSimons

Fedora29 -> 30 update can be a problem ATM. Faced an issue regarding set locale to utf-8.

https://github.com/dotnet/source-build/issues/1339#issuecomment-548879934 helped for Fedora locale set error.

For additional context, this was a Change in Fedora 30. The consequences is that the language features that various locale features require don't work out of the box for non-C locales.
There's two fixes for this, as far as I can tell:

Use a C locale. C/C.UTF-8/POSIX work out of the box
Install the language pack for the non-C locale.  That's what glibc-langpack-en does.

Install the language pack for the non-C locale.  That's what glibc-langpack-en does.

#212 to install glibc-language-pack-en


关于获取jobConfig

getJobConfig(userName, jobName, next) {
const hdfs = new Hdfs(launcherConfig.webhdfsUri);
hdfs.readFile(
/Container/${userName}/${jobName}/JobConfig.json,
null,
(error, result) => {
if (!error) {
next(null, JSON.parse(result.content));
} else {
next(error);
}
}
);
}
关于这段代码，请问能否可以同步执行，或者有return返回呢？

Node.js is designed as an asynchronous event driven JavaScript runtime. Almost all I/O operations (including network I/O) are run asynchronously. Making it runs synchronously will block the main process and prevent it from serving other requests.
May I know what you are looking for so I can give you more advice to achieve them?

I want to get for a global variable, out of the function ,I can use the result.

@Gerhut   I want to get for a global variable, out of the function ,I can use the result.

How about storing the result in global just after the function callbacks? Like
getJobConfig(userName, jobName, (error, result) => {
  if (error == null) global.jobConfig = result
})

Or even using a Promise if the result status may be needed before it returns.
global.jobConfigPromise = new Promise((resolve, reject) => {
  getJobConfig(userName, jobName, (error, result) => {
    if (error) {
      reject(error)
    } else {
      resolve(result)
    }
  })
})

@Gerhut This method seems not to work.  the second method ，I do not understand

no further info. close the issue.


Make the UI Customizable

I'm currently using this amazing plugin in my brand new app wich uses neumorphic design. Since this plugin is not customizable the storymaker page looks absolutely different like another different app. I'm planing to make a PR for that. Please let me know if you agree with this orientation

Hi @Consticonstantin. PR is welcome.


Contact Form and MailPoet Subscription support

Ideally some magic should happen so the form is submitted to the live wordpress instance for both.
Main problems being probably captchas and email (un) subscription / management.
Maybe some page could be set to be an iframe to the live one?

Unfortunately, this is out of scope for this plugin. This would have to be implemented by Contact Form 7 or MailPoet.


Native module import problem

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version: 6.0.0-beta.47
Electron Version:: v6.1.7
Operating System: macOS 10.15.2

Expected Behavior
Load the native module.
Actual Behavior
The native module is in the correct path:
/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/native_modules//lib/node_printer.node
but the app crash with this output.
App threw an error during load Error: Cannot find module '/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/native_modules//lib/node_printer.node'     at webpackEmptyContext (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:8956:10)     at Object../node_modules/printer/lib/printer.js (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:9002:86)     at __webpack_require__ (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:21:30)     at Module../src/main.js (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:10138:65)     at __webpack_require__ (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:21:30)     at /Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:85:18     at Object.<anonymous> (/Users/antoniomgatto/projects/electron-printer-test/.webpack/main/index.js:88:10)     at Module._compile (internal/modules/cjs/loader.js:786:30)     at Object.Module._extensions..js (internal/modules/cjs/loader.js:798:10)     at Module.load (internal/modules/cjs/loader.js:645:32) webpack built b17b24ad1f3cf056d7d3 in 1107ms
To Reproduce
https://github.com/antoniomgatto/electron-forge-test
Steps to reproduce the error:
yarn
yarn start

Additional Information
I created this project with the command below:
N/A

Any updates on this @antoniomgatto ?

Duplicate of #1250


Create composer.json for instalation with composer

The installation is much simpler with Composer and https://github.com/RobLoach/component-installer
After merged you can test it:
composer config repositories.zenorocha-clipboardjs vcs "https://github.com/zenorocha/clipboard.js/"
composer require zenorocha/clipboardjs:dev-master

Before merged you can test using my git fork.
Composer creates a "components" folder that contains the "clipboardjs" whith all files spec in "files" of composer.json.
You can then register the "package" at https://packagist.org In this way the installation would be simpler:
composer require zenorocha/clipboardjs:dev-master

@zenorocha  is there anything holding this PR back?

If you do not work with composer this PR does not add anything.
If you use composer to have the dependencies of your project updated this PR will be of great help.

@zenorocha is there anything holding this PR back?

Anything holding it back, I just wasn't able to properly test it.


5/16 PM Publishing

Docs Build status updates of commit 43b825a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msbuild-api-docs/xml/Microsoft.Build.Evaluation/Project.xml
:warning:Warning
View
Details

msbuild-api-docs/xml/Microsoft.Build.Execution/BuildRequestDataFlags.xml
:warning:Warning
View
Details

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectElementLink.xml
:warning:Warning
View
Details

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectMetadataLink.xml
:warning:Warning
View
Details

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectRootElementLink.xml
:warning:Warning
View
Details

msbuild-api-docs/api/msbuild-api-docs/toc.yml
:warning:Warning
View
Details

msbuild-api-docs/xml/Microsoft.Build.Framework.Profiler/ProfiledLocation.xml
:bulb:Suggestion
View
Details

msbuild-api-docs/xml/Microsoft.Build.Framework.Profiler/ProfilerResult.xml
:bulb:Suggestion
View
Details

msbuild-api-docs/xml/Microsoft.Build.Framework.XamlTypes/BaseProperty.xml
:bulb:Suggestion
View
Details

msbuild-api-docs/xml/Microsoft.Build.Framework/RegisteredTaskObjectLifetime.xml
:bulb:Suggestion
View
Details

msbuild-api-docs/xml/Microsoft.Build.Framework/SdkReference.xml
:bulb:Suggestion
View
Details

msbuild-api-docs/xml/Microsoft.Build.Utilities/TargetDotNetFrameworkVersion.xml
:bulb:Suggestion
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

CODE-OF-CONDUCT.md
:white_check_mark:Succeeded

README.md
:white_check_mark:Succeeded

msbuild-api-docs/docfx.json
:white_check_mark:Succeeded

msbuild-api-docs/snippets/.editorconfig
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItem/Overview/AddNewItem.csproj
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItem/Overview/Program.cs
:white_check_mark:Succeeded
View

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItem/Overview/Properties/AssemblyInfo.cs
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItemGroup/GetEnumerator/ListItemAndPropertiesCS.csproj
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItemGroup/GetEnumerator/Program.cs
:white_check_mark:Succeeded
View

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildItemGroup/GetEnumerator/Properties/AssemblyInfo.cs
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildProperty/Overview/AddNewProperty.csproj
:white_check_mark:Succeeded

msbuild-api-docs/snippets/csharp/Microsoft.Build.BuildEngine/BuildProperty/Overview/Program.cs
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.

msbuild-api-docs/xml/Microsoft.Build.Evaluation/Project.xml

Line 0, Column 0: [Warning: xref-not-found] Cross reference not found: 'Microsoft.Build.Evaluation.ProjectItem.ChangeItemType(System.String)'.

msbuild-api-docs/xml/Microsoft.Build.Execution/BuildRequestDataFlags.xml

Line 0, Column 0: [Warning: xref-not-found] Cross reference not found: 'Microsoft.Build.Evaluation.ProjectRootElementCacheBase'.
Line 0, Column 0: [Suggestion: ECMA2Yaml_Enum_NoRemarks] Please note: <remarks> node on Enum fields will be ignored.

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectElementLink.xml

Line 0, Column 0: [Warning: xref-not-found] Cross reference not found: 'Microsoft.Build.Construction.ProjectElement.ExpressedAsAttribute'.

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectMetadataLink.xml

Line 0, Column 0: [Warning: xref-not-found] Cross reference not found: 'Microsoft.Build.Evaluation.ProjectMetadata.EvaluatedValueEscaped'.

msbuild-api-docs/xml/Microsoft.Build.ObjectModelRemoting/ProjectRootElementLink.xml

Line 0, Column 0: [Warning: xref-not-found] Cross reference not found: 'Microsoft.Build.Construction.ProjectRootElement.MarkDirty(System.String,System.String)'.

msbuild-api-docs/api/msbuild-api-docs/toc.yml

Line 10956, Column 12: [Warning: missing-attribute] Missing required attribute: 'name'.

msbuild-api-docs/xml/Microsoft.Build.Framework.Profiler/ProfiledLocation.xml

Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:Equals(System.Object) for uid: Microsoft.Build.Framework.Profiler.ProfiledLocation.Equals(System.Object).
Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:GetHashCode for uid: Microsoft.Build.Framework.Profiler.ProfiledLocation.GetHashCode.
Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:ToString for uid: Microsoft.Build.Framework.Profiler.ProfiledLocation.ToString.

msbuild-api-docs/xml/Microsoft.Build.Framework.Profiler/ProfilerResult.xml

Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:Equals(System.Object) for uid: Microsoft.Build.Framework.Profiler.ProfilerResult.Equals(System.Object).
Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:GetHashCode for uid: Microsoft.Build.Framework.Profiler.ProfilerResult.GetHashCode.

msbuild-api-docs/xml/Microsoft.Build.Framework.XamlTypes/BaseProperty.xml

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

msbuild-api-docs/xml/Microsoft.Build.Framework/RegisteredTaskObjectLifetime.xml

Line 0, Column 0: [Suggestion: ECMA2Yaml_Enum_NoRemarks] Please note: <remarks> node on Enum fields will be ignored.

msbuild-api-docs/xml/Microsoft.Build.Framework/SdkReference.xml

Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:Equals(System.Object) for uid: Microsoft.Build.Framework.SdkReference.Equals(System.Object).
Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:GetHashCode for uid: Microsoft.Build.Framework.SdkReference.GetHashCode.
Line 0, Column 0: [Suggestion: ECMA2Yaml_Inheritdoc_NoFoundParent] Found no member can be inherited by key:ToString for uid: Microsoft.Build.Framework.SdkReference.ToString.

msbuild-api-docs/xml/Microsoft.Build.Utilities/TargetDotNetFrameworkVersion.xml

Line 0, Column 0: [Suggestion: ECMA2Yaml_Enum_NoRemarks] Please note: <remarks> node on Enum fields will be ignored.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Last tag always has \n in the description

Hi,
the unwrap: true option fails to strip the last new line before */, so an empty description on the last tag in the comment always becomes '/' (not sure why) and a non-empty description always has a newline at the end.
In my code, I have to do this to remove spaces/newlines from before the end of the comment, before passing the comment to doctrine.
 comment = comment.replace(/[\s\n]*\*\/[\s\n]*$/, '*/');

Thanks

Related https://github.com/eslint/doctrine/issues/107

@nzakas I think this can be closed too as was related to #107 and that is closed.

@Madd0g what version are you using!

@nzakas - 0.6.4 was the last version I used.

@Madd0g Can you please try 0.7.0?

@gyandeeps - sorry for the delayed reply - I upgraded to 0.7.0, I still see a \n in the end of the last tag.


DbClient - metric wrong format hangs application

Environment Details

Helidon Version: 2.0.0-M3
Helidon SE

The following code:
DbClient.builder()
                .config(config.get("db"))
                .addInterceptor(DbCounter.builder()
                                        // %1$s - statement name, %2$s - statement type
                                        .nameFormat("db.counter.%1$s.%2$s.%3$s")
                                        .build())
                .build();

will pass, and when a statement is called, it just hangs indefinitely

This actually didn't occur in my test. I got
java.util.concurrent.CompletionException: java.util.MissingFormatArgumentException: Format specifier '%3$s'
on the server and returned to the client with no hang.
I used the jdbc example and curl http://localhost:8079/db

unable to recreate issue and so closing...


Fixed #26596 -- Added Jinja2 {{ csrf_input }} documentation

I'm not sure this completely addresses the ticket, in particular the points raised in https://code.djangoproject.com/ticket/26596#comment:6.

OK, I'll fix the formatting errors and think about how to completely address the ticket.

OK, I fixed the formatting. I think this explains the API for Jinja while the other documentation addresses the Django template language.
So do you think I should add something about how the examples are mostly for the Django template language?

I discussed this with @MarkusH and this is what we came up with.

I also added some syntax highlighting where it wasn't before

I made that change @carljm

After discussion with @carljm we decided to delete the paragraph following the one I added. We think people using Jinja2 templates should use the new example for CSRF protection.

So we don't care about template engines besides Jinja2 or something? This still doesn't seem to address Aymeric's requests in https://code.djangoproject.com/ticket/26596#comment:6 as far as I understand.

@timgraham You're right, this doesn't address the second half of Aymeric's comment in the ticket. That was intentional, because third party back ends will need quite a bit more detailed info to actually understand how to get access to the token in Python in order to provide CSRF functionality, and the details of how they provide it will depend very much on how their template language works, and they'll probably be best served by just looking at the Jinja2 backend' implementation. I guess we could add a sentence like "Third party template engines should provide access to the CSRF token via whatever mechanism makes most sense for their template language." But I don't really see much value in that, and I don't think it helps much to conflate documentation for two very different audiences (CSRF users and third party template backend authors) in one place in the docs.
Note that this PR doesn't remove any information that would be useful to any user of a non DTL, non Jinja2 templating language. The previous advice here was pretty useless AFAICT -- getting the token in JS is not relevant to providing it via a server side template.
I didn't suggest including mention of {{ csrf_token }} here because I don't know of a significant use case for it, so it seems like noise in the docs. {{ csrf_input }}is what almost everyone should use with Jinja2, and parallels the functionality documented for DTL.
I'd be curious to hear how @aymericaugustin would propose improving or expanding this patch.

The use case for {{ csrf_token }} is <body data-csrf-token="{{ csrf_token }}">; then some JavaScript can pull the CSRF token from there and include it in AJAX requests even if the CSRF cookie is HTTP-only.
It's a fairly narrow use case and there's a risk that the token gets out of sync; often it's more convenient:

to pull the token from the cookie (if it isn't http-only), or
to load it via an AJAX request e.g. by including it in the response to a login request.

I added the +django and +jinja. Is there consensus on what else needs to be done?

I'm happy with this PR as-is: I think it provides the most useful information without confusing things, and is clearly an improvement over the existing docs. I'd also be fine with adding a sentence in the Jinja2 section that says something like "If you need just the CSRF token alone (not wrapped in a hidden form input) in your template, you can access it via {{ csrf_token }}."
I think the idea of documentation for third-party template backend authors about how to automatically provide the CSRF token in the template context for their users is really a separate issue that doesn't need to be conflated with this ticket.

merged in 261738990e07a039b39aab0664eea71f226cd381 and 9c53facc45908bc0593de194a60bc75e5d34a48e, thanks!


How can I create sp_metadata.xml?

I can't find any documentation on creating sp_metadata.xml. Is there a tool for generating this?

Generating that is a responsibility of the service provider implementation. The example service provider implementation should provide the URLs for doing that. Follow that link back to djangosaml2 urls file and there should be a metadata view. Visit that url when running your service provider to download the XML file. You'll then place that XML file in your IDP metadata config directory. You'll also want to visit the IDP metadata URL and download the IDP XML file and place it in the SP's metadata config directory.

Going to close this issue, as there is nothing to do. As Adam mentioned, this is something which the packages will provide for you, and you get it simply by visiting the necessary url. The example setup contains an idp implemented with djangosaml2idp, and an sp implemented with djangosaml2, which both have a /metadata/ url which returns the xml file you're looking for. Some further information can also be found in the djangosaml2 documentation


API: Invalid signature

I'm submitting a ...
[X ] bug report
[ ] question about the decisions made in the repository
EUR-ADA Kraken
livebot trading
API: Invalid signature
Hi all, thanks for your support. I've spent like 12 hours non stop today to make a strategy that worked well for me while in backtest... now i was ready to test it in a live tradingbot and...
I always get the "API: Invalid signature" error for Kraken. I set up and activated the key. I put all the parameters and enabled the trading bot at the bottom of the config.js file. I have enabled everything... I have already red all the posts about it without any luck. Can someone please help me out?

I figured out the problem myself. I had just to disable the papertrading option in the config.js


feat(asset): compute "Now" in a way that can be saved

Pull Request
🤨 Rationale
Task: link
👩💻 Implementation
Added a hardcoded value: ${__now:date}, similar to other grafana variables. This will allow us to save the dashboard, and on reload new date will be computed.
🧪 Testing
Added tests+

✅ Checklist

[x] This PR has a title that follows the commit message format.

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

v3.43.0
GitHub release

Your semantic-release bot :package::rocket:


update to go 1.18 (continue testing against 1.17)

Go 1.16 reached end of life, so update to the current version of Go, but also
run CI on the previous version (which is still supported).
We should probably also decide wether or not we want the Dockerfiles to pin to
a specific minor version; this makes the releases more deterministic.

Codecov Report

Merging #3643 (0cca942) into main (7a6b9e3) will decrease coverage by 0.03%.
The diff coverage is n/a.

:exclamation: Current head 0cca942 differs from pull request most recent head 58fe15b. Consider uploading reports for the commit 58fe15b to get more accurate results

@@            Coverage Diff             @@
##             main    #3643      +/-   ##
==========================================
- Coverage   56.62%   56.58%   -0.04%
==========================================
  Files         103      103
  Lines        7520     7520
==========================================
- Hits         4258     4255       -3
- Misses       2593     2596       +3
  Partials      669      669

Impacted Files
Coverage Δ

...distribution/distribution/registry/handlers/app.go
45.91% <0.00%> (-0.56%)
:arrow_down:

Continue to review full report at Codecov.

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

We'll need to update the list of "required" checks again after this (as their name is based on the go version)

Argh... this one ..... (no, it's not a breaking change 😂)
#17 26.98 cmd/digest/main.go:1: : error obtaining VCS status: exec: "git": executable file not found in $PATH
#17 26.98 	Use -buildvcs=false to disable VCS stamping. (typecheck)

we seem to be missing git in the lint.Dockerfile

Whoop! All green now!


Link handling

Internal links
URLs with a known host have the origin part removed to make them internal.

*.hlx.page
*.hlx.live
mymomeymatters.alexforbes.co.za
mymomeymatters.alexforbes.com

External links
All other links should always open in a new window.

Done


Track third party JS library vulnerabilities

As an extension of the FR in https://github.com/HTTPArchive/httparchive/issues/77, the JS library data can be joined with vulnerability data. See https://github.com/snyk/vulnerabilitydb and https://github.com/RetireJS/retire.js.
Some questions this can answer:

are popular websites more secure?
what does the vulnerability distribution look like? across different severity levels?
what are the most prolific libraries that have vulnerabilities?
which websites share common vulnerabilities?

And by extension:

are any CMS platforms generating insecure websites?

Ideally, this data would be used by white hats to understand the security landscape on the web and help developers make more secure websites. There would be some kind of process for proactively alerting site owners about their vulnerabilities, or at least making it easy for them to get an audit of their site.
/cc @tlauinger @guypod @tkadlec @igrigorik @ebidel https://github.com/GoogleChrome/lighthouse/issues/1948

How/where does retire.js source vulnerability data? Or, put differently, what are the pros and cons of using snyk DB approach vs retire.js?

Retire.js has its own DB that is manually put together by volunteers from a range of public sources. Actually, there are two - one for client-side JS libraries, and another for Node packages. Snyk only has Node packages.
When we started our research project, we looked into getting package metadata from the NPM repository, but we found that some client-side JS libraries either aren't in NPM at all, or NPM doesn't have older versions since many libraries already existed before NPM was a thing. In conclusion, I think vulnerability data about Node packages can be a useful addition, but I wouldn't use it as the sole source of client-side vulnerabilities.

Snyk only has Node packages.

@guypod @tkadlec I do see JQuery and other client-side libs in your db. Where/how are these lists of vulnerabilities sourced?

Snyk does capture many client side libraries too, not just Node packages. These packages exist in npm too, which as you know is used regularly in front end dev.
Our tools test for those libraries only when fetched from npm, as we don't currently have the code to inventory libraries from the web page itself. Doing so accurately is it's own challenge, and we have some ideas we can share there but we haven't completed it.
If you're already building the inventorying into lighthouse or webpagetest, our DB should be a comprehensive source of these vulnerabilities, and give users good metadata and fix instructions for the issues they find. Will be happy to test out pages if you spot any gaps with this statement!
Using multiple sources is not a bad idea, though I don't think retire.js is a DB. It's a detection tool, referring to other DBs as the sources, so wouldn't that be redundant with the inventorying tech you build here?

On the subject of integration, LibraryDetector uses long-form strings like "Google Maps" to identify the detected libraries, while Snyk identifies from GH repo or npm name. I'm not even sure if the Maps API is open source and Snykable. I see https://snyk.io/test/github/googlemaps/google-maps-services-js but that's the nodejs library.
So we'd probably also need to map LibraryDetector names to Snyk-compatible repos/packages if available.