RFC: Remove Reliable Distributed Systems and put replacement in Extras

Problem:
Reliable Distributed Systems is an elective level course that is only partially available.
Duration:
Feb 29, 2020
Background:
OSSU promises the equivalent of an undergraduate degree in Computer Science. In order to evaluate our recommended courses, we use the Curriculum Guidelines for Undergraduate Programs in Computer Science (CS2013). More information can be found here.
CS2013 details Parallel and Distributed Computing as an essential knowledge area. But within this knowledge area, the topic of distributed computing is labeled an elective.
Separately, the current recommendations for distributed systems are Reliable Distributed Algorithms 1 & 2. Unfortunately, the assignments are unavailable. The first course has been offline for 2 years. The second course is still listed, but the virtual machine image used to distribute the programming environment is no longer available. The specification for the assignments in the second course are still available, it is possible that the programming environment could be created from scratch. Fortunately the lectures for both classes are available as a Youtube playlist.
There are two other courses that cover similar material. Both are free to audit (watch lectures) but require payment to view assignments:
Cloud Computing Concepts, Part 1 is the first of a 6 course specialization on Cloud Computing.
Distributed Programming in Java is the last of three courses in a specialization. Being the last in the specialization, the class may require students to take the earlier courses to follow the material.
Proposal:

Remove Reliable Distributed Systems from the main curriculum.
Add Cloud Computing to Extras/Courses.

Alternatives:
a) Simply move Reliable Distributed Algorithms to Extras/Courses.
b) Place Distributed Programming in Java in Extras/Courses. This choice would require some study to determine if this third course in the specialization can be completed without the previous two courses (Parallel Programming in Java, Concurrent Programming in Java). Students would be welcome to take these two other courses. Note that there is currently a free Udacity course with C++ assignments that covers the same topics as Parallel Programming in Java.

I think replacing RDS with the free Udacity course, but keeping it in the main curriculum is a good idea.
I was very much interested in RDS (it's distributed computing in Scala, I am planning on taking the Functional Programming in Scala Specialization eventually, maybe go into a career in parallel/distributed programming) so it would be sad if this topic is removed from the main curriculum. Of course I can just do my own thing and stray from the main curriculum, but that will diminish the feeling of accomplishment, and as someone who just started Advanced CS, I can say that the motivation comes from finishing the main curriculum courses one by one and making progress.
I did not even know that Extras existed; I don't think people will pay too much attention to things that are in Extras. It looks like a dumping ground of way too many courses; students would likely be confused as to which ones are worth doing. Of course if there is no good solution currently, then dumping things into Extras can be a temporary solution.
If the main curriculum is too bloated and needs to be trimmed then removing things (like the Software Construction courses) or moving things to Extras might be OK, but removing RDS from the main curriculum with no replacement feels like throwing out that particular topic altogether (there doesn't seem to be anything else that explicitly covers distributed/parallel programming).

Close of the Comment Period
Findings:

The topic of distributed programming is valued.
The extras section is not a clear resource for students.

Response:

I certainly appreciate student enthusiasm for a topic, but don’t think that alone can justify a course’s place in the core curriculum. One can imagine that at many universities there are well loved electives that students cherish, yet these courses remain elective. Our curricular guidelines state this is the proper place for Distributed Programming. Students should note that the related topic of parallel computing will remain in the core curriculum.
The Extras page can use some attention. I welcome suggestions or efforts to better organize the page.

Conclusion
Reliable Distributed Algorithms 1 & 2 will be removed from the curriculum. Cloud Computing Concepts, Part 1 will be added to the Extras page. The Udacity course Intro to Parallel Computing will remain in the required curriculum.

@spamegg1 It sounds like you have the beginning of an RFC to replace Hack the Kernel with Three Easy Pieces. I'd be very interested in such an RFC!


Added support for versionned JSON

Mongoid::CachedJson now supports JSON versionning with json_fields_for. The obvious use-case is to return different JSON versions from API v1 and API v2. For example, version 2 of Widget splits name into first and last.
class Widget
  include Mongoid::CachedJson

  field :first
  field :last

  json_fields_for :v1 \
    :name => { :definition => lambda { |instance| "#{instance.first} #{instance.last}" } }

  json_fields_for :v2 \
    :first => { },
    :last => { }

end

JSON fields declared with json_fields instead of json_fields_for are used for all versions.

I like the structure above for declaring json_fields for different versions; I wonder whether others would prefer a single json_fields call where each definition could specify an (optional) :versions argument, a la
json_fields \
  :name => { :versions => [ :v1 ], :definition => lambda { |instance| "#{instance.first} #{instance.last}" } }
  :first => { :versions => [ :v2 ] },
  :last => { :versions => [ :v2 ] }

I suspect a majority would prefer what you already have, but might be worth a quick poll.

You're right, it's possible people will want a declaration where you say whether a field applies to this or that version. That would be pretty nightmarish to manage imo, because when you add :v3, you have to go back to all JSONs and add it into each declaration? Unless of course versions is a range, but that has some of its own disadvantages.
@aaw - thoughts?

This looks great, and is a very useful feature. I would probably add a json_fields declaration in the JsonFoobar scaffold (or add a new one), so as to make sure that json_fields-declared fields still show up in all as_json versions even when multiple json_fields_for calls exist in the same class.

This looks good, though, I'd say it's merge-able whenever you want.

I like Frank's idea - I think in most cases, the intersection between the field definitions in two different versions of an API will be large, and it's going to get unmanageable once you have several json_fields_for declarations in the same class.
We should support ranges of versions, but it doesn't need to be much more syntax than you have already - the common case is that you've added a new field in a new version, the not-so-common case is that you've removed a field in a new version, so maybe something like:
json_fields \
  :name => { :version => :v1, :definition => lambda { |instance| "#{instance.first} #{instance.last}" } },
  :first => { :version => :v2 },
  :middle => { },
  :last => { :version => :v2 , :deprecated_at => :v3 }

Which would mean that :name is in version 1 and above, :middle is in every version, :first is in version 2 and above, and :last was in version 2 and version 3 but not version 4 or above.

I am going to close this, reopened https://github.com/dblock/mongoid-cached-json/pull/3 with the inline version. I took most of the suggestions except the retiring part - I think it's pretty dangerous to assume that the version is something sortable and incrementable - I might want the :red version or the :blue version.


Feature: Support Pydantic field type List[Vector())

SDK
Python
Description
I'm trying out LanceDB with Colbert style multi-vectors (actually ColPali). I'm using Pydantic models. I thought a List[Vector(dim=128)] would be the best approach but that doesn't seem to be supported by the schema conversion logic for table creation.
I think a List[List[float]] is my workaround - not tried it yet. Some might argue it's not a workaround. Given there's a special type for vector fields I would argue it is a workaround.
It would be great to have first-class support for multi-vectors.
Trying to create a table from a Pydantic model that has a field of type List[Vector()] isn't supported at present (throws an error). A field directly defined as Vector() is supported.
From what i can tell this is because _py_type_to_arrow_type() doesn't handle Vector(). So when the list type handling case recurses child types there's isn't a case to handle Vector(). Vector as a top-level type is handled in _pydantic_to_arrow_type() which is called first on each field. Then it calls _py_type_to_arrow_type() for a subset of fields.
Thanks for considering this.

@janaka yes, we are thinking to add the support to have this 1:N relationship in each row https://github.com/lancedb/lance/issues/2951

Looks like the Vector() nullable option is in the latest release which depends on a pre-release of Lance which isn't published?


mock_lambda moto.awslambda.exceptions.InvalidRoleFormat

Using moto v2.3.0
Installed in pipenv with pipenv install moto
I am trying to unit test a piece of code that calls lambda_client.invoke() by using @mock_lambda
but I am getting errors around iam roles.
Code I'm testing:
  def invoke_lambda():
      lambda_client = boto3.client('lambda')
      request = {'message': 'Hello World!'}
      lambda_response = lambda_client.invoke(
          FunctionName='hello-world-lambda',
          InvocationType='Event',
          Payload=json.dumps(request)
      )
      response = lambda_response['Payload'].read()
      print(response)
      return response

Test code:
  def lambda_event():
      code = '''
          def lambda_handler(event, context):
              return event
          '''
      zip_output = io.BytesIO()
      zip_file = zipfile.ZipFile(zip_output, 'w', zipfile.ZIP_DEFLATED)
      zip_file.writestr('lambda_function.py', code)
      zip_file.close()
      zip_output.seek(0)
      return zip_output.read()

    @mock_lambda
    @responses.activate
    def test_code(self):
        conn = boto3.client('lambda', 'us-west-2')
        conn.create_function(
            FunctionName='functionToBeTested',
            Runtime='python3.8',
            Role='test-iam-role',
            Handler='lambda_function.lambda_handler',
            Code={
                'ZipFile': self.lambda_event(),
            },
            Publish=True,
            Timeout=30,
            MemorySize=128
        )

Then when I try to run the test with pipenv run pytest, I get
E           moto.awslambda.exceptions.InvalidRoleFormat: An error occurred (ValidationException) when calling the None operation: 1 validation error detected: Value 'arn:aws:iam::accountid:role/role_name' at 'role' failed to satisfy constraint: Member must satisfy regular expression pattern: arn:(aws[a-zA-Z-]*)?:iam::(\d{12}):role/?[a-zA-Z_0-9+=,.@\-_/]+

I have also tried to change the iamrole from 'test-iam-role' to something that passes this validation ('arn:aws:iam::000000000000:role/role_name') but then I get:
E               moto.awslambda.exceptions.CrossAccountNotAllowed: An error occurred (AccessDeniedException) when calling the None operation: Cross-account pass role is not allowed.

What am I doing wrong?
Thanks in advance!

You need to make sure your IAM Role is also present inmoto.   See the following code from our Lambda tests for an example on how to do this:  Make sure IAM Role is available for lambda:CreateFunction

That worked! Thank you! Is that documented somewhere? I didn't see it.
Also, is there some way to assert that a mock lambda was called with the correct event input?

I don't believe there is documentation for this specific feature, @bronzeson. But in general we try to mimick AWS' behaviour as closely as possible, and try to throw the same exceptions as AWS if the input behaviour is invalid.
Invoking a Lambda-function starts a Docker-container, where the Lambda-code is then executed. This is a black box, similar to AWS, so it is not possible to assert the correct input directly. The output and execution logs of the Lambda are available via CloudWatchLogs, just as in AWS, so you can use that to verify indirectly whether the execution was successful.
Note that, if you use the decorators, it is not possible to connect to other services from within a Docker. There is some documentation on that here: http://docs.getmoto.org/en/latest/docs/services/lambda.html

Thanks again this was super helpful. Feel free to close!


is it possible to omit the "custom" button on left side of CameraKitGalleryView ?

We dont need that custom button on left side of the view - is it possible to add a prop to show/hide it ?

doh. just omit the prop for customButtonStyle and it wont display.


Issue with PWA

Hello @VirtuBox
I am trying to setup PWA for our multisites and tested these plugins:
https://wordpress.org/plugins/pwa-for-wp/
https://wordpress.org/plugins/super-progressive-web-apps/
https://wordpress.org/plugins/progressive-wp/
But the videos on our site will does not play on iPhone for every above plugins, and it show this issue as the screenshot:
https://prnt.sc/o1n31s
Also, after access the video page on iPhone Safri,  our multisites and the server will be caused down, and I need restart nginx to make the sites rework.
The authors of those plugins do not think the issue is from plugins, but is related to the server configuration, since there is no this kind of issue on their case.
I do not know how to track the issue and adjust the configuration to fix the issue.
It would be great appreciated if I could have your suggestion and instructions.
Thanks in advance.
Alex

Hello @VirtuBox
Did you have a chance to have a check the compatibility with PWA please?
Thanks

Hello @alexlii1971,
no sorry I haven't found the time to take a look on this issue.
You will probably find informations in /var/log/nginx/yourdomain.tld.error.log as well as in /var/log/nginx/error.log.

Hello @VirtuBox
Thanks, I am trying.
For now, here is a problem:
This is log directory:
http://prntscr.com/p4tz20
http://prntscr.com/p4tzjd
http://prntscr.com/p4tzqn
When I try:
ee log reset --fpm
it show:
[Errno 2] No such file or directory: '/var/log/php5/slow.log'
is there any mis configuration please?
Thanks

Hello @alexlii1971,
easyengine v3 is now pretty outdated, I recommend you to migrate to WordOps (EEv3 fork) as I'm currently contributing to this project.


Add a copy button to copy sent prompts to clipboard

Integrate a copy-to-clipboard button in chat prompt to facilitate easy copying of prompt messages. Include a title for the prompt message area to enhance clarity, prevent button and prompt message overlap, and also maintain stylistic consistency with response messages.
Close #769

Thank you for providing one of the most required feature. But the title occupies too much space and is not pretty.
Could you change the button to a hangover button next to the prompt bubble?

Not my best thing, additional help needed 😅


Add turbo colormap to Napari

🚀 Feature
A group at Google has created a rainbow colormap that shares some of the desirable properties of inferno/viridis/etc., but is slightly better for visualizing very dim and very bright features. You can read about it more here. I think this would be a very useful addition to Napari (which, btw is awesome, thank you!)

@henrypinkard yes, we were talking about that the other day. We'd love to have it in there. Would you like to make a PR adding it? We can guide you through the process if you get stuck

In theory, yes, but realistically I would likely triage it for at least a few months as I have no urgent need for it

No worries, if someone else wants to start working on it in the meantime they can post here, otherwise we'll get to it when we get to it. It should be pretty easy to add here https://github.com/napari/napari/tree/master/napari/util/colormaps


(Do Not Merge) updating docker scripts

This PR is only for feedback in updating the Heron Docker build scripts.
@kramasamy In the process of trying to diagnose other issues, I noticed the docker build scripts are not working as of today.  I made a few changes and was able to successfully build a Heron Ubuntu 14.04 image and place it in a zip file.  I think that I could easily make these changes to the other image types if you think these changes are ok to put in.
Let me know your thoughts.

To verify these changes execute
$ ./docker/scripts/build-artifacts.sh ubuntu14.04 0.19.0 ~/local-heron-release/
$ ./docker/scripts/build-docker.sh ubuntu14.04 0.19.0 ~/local-heron-release/

This build failed because of an integration test.  I don't see a way to kick off the build anymore.  Seems the button was removed.

looking over, I found in error in my docker config.
 ---> 578c3e61a98c
Step 2/19 : RUN apt-get -y update && apt-get -y install     python     unzip     software-properties-common     supervisor     curl &&     apt-get clean all
 ---> Using cache
 ---> b7292a142da0
Step 3/19 : RUN add-apt-repository ppa:openjdk-r/ppa && apt-get -y update &&     apt-get -y install openjdk-8-jdk && apt-get clean all
 ---> Using cache
 ---> dbdc059747ff
Step 4/19 : ENV JAVA_HOME /usr/lib/jvm/java-8-openjdk-amd64
 ---> Using cache
 ---> 3632db9a55cb
Step 5/19 : RUN update-ca-certificates -f
 ---> Using cache
 ---> a8870c88f73e
Step 6/19 : ADD artifacts /heron
 ---> Using cache
 ---> 8c3fb38e66ee
Step 7/19 : WORKDIR /heron
 ---> Using cache
 ---> 01f1def45d52
Step 8/19 : RUN /heron/heron-client-install.sh && /heron/heron-tools-install.sh
 ---> Running in a341c5438a1a
/bin/sh: 1: /heron/heron-client-install.sh: not found
The command '/bin/sh -c /heron/heron-client-install.sh && /heron/heron-tools-install.sh' returned a non-zero code: 127

I'll look into this more this weekend.

Looks good to me - let us merge it once the CI passes.

@kramasamy I went ahead and updated the other  docker images as well.  I think we are good to go.

@kramasamy After doing further investigation.  This PR does fix the broken docker references to the install scripts, but it does not fix the other os specific issues that need to be addressed.


Possible to start service via ADB / Shell

Hello and happy New Year! I am trying to start the service using a script so it's possible to to use GPSd inside my chroot without interacting with device directly. When running am start-foreground-service --user 0 -n io.github.tiagoshibata.gpsdclient/.GpsdForwarderService the app immediately produces an ANR. Do you know of a way I can start the service using adb shell or command line?
Here is a snippet from logcat
01-02 20:23:55.183 E/AndroidRuntime(11063): java.lang.RuntimeException: Unable to start service io.github.tiagoshibata.gpsdclient.GpsdForwarderService@78802ee with Intent { cmp=io.github.tiagoshibata.gpsdclient/.GpsdForwarderService }: java.lang.RuntimeException: GpsdClientService requires parameters io.github.tiagoshibata.GPSD_SERVER_ADDRESS and io.github.tiagoshibata.GPSD_SERVER_PORT

I tested with Android 7, and the complete command as root user is:
am startservice --user 0 \
    -e io.github.tiagoshibata.GPSD_SERVER_ADDRESS <IP_ADDRESS> \
    --ei io.github.tiagoshibata.GPSD_SERVER_PORT 6000 \
    io.github.tiagoshibata.gpsdclient/.GpsdForwarderService

The command for newer Androids (IIRC 9 and newer) should use am start-foreground-service --user 0 -n instead (I haven't tested this command yet):
am start-foreground-service --user 0 -n \
    -e io.github.tiagoshibata.GPSD_SERVER_ADDRESS <IP_ADDRESS> \
    --ei io.github.tiagoshibata.GPSD_SERVER_PORT 6000 \
    io.github.tiagoshibata.gpsdclient/.GpsdForwarderService

The -e flag specifies io.github.tiagoshibata.GPSD_SERVER_ADDRESS as string, while --ei specifies that io.github.tiagoshibata.GPSD_SERVER_PORT is an integer.
Thanks for your app :-)

Unfortunately, it is not possible to start the foreground service on Android 11 as is, without root. Fortunately I was able to decompile the .apk and export the service by modifying the manifest. It's also not possible to start it without user interaction of some kind..  i.e. device needs to be unlocked for application to come to front and service to start.
Thank you for all the input, it did not go unoticed!
Here is a small script I have devised for use in Termux with the modified apk.
#!/data/data/com.termux/files/usr/bin/bash

if [[ ${1} == stop ]]; then

  adb shell am stop-service --user 0 io.github.tiagoshibata.gpsdclient/.GpsdForwarderService

fi

ADRESS=${2}

PORT=${3}

if [[ -z ${ADDRESS} ]]; then

  ADDRESS="<IP_ADDRESS>"

fi

if [[ -z ${PORT} ]]; then

  PORT="5000"

fi

if [[ ${1} == start ]]; then

  SCREEN_STATE=$(adb shell dumpsys power |grep "Display Power:" |cut -d '=' -f2)

  until [ ${SCREEN_STATE} = "ON" ]; do

    sleep 1
    SCREEN_STATE=$(adb shell dumpsys power |grep "Display Power:" |cut -d '=' -f2)

  done

  adb shell settings put secure location_mode 3

  adb shell am start --user 0 io.github.tiagoshibata.gpsdclient/.MainActivity && adb shell am start-foreground-service --user 0 --activity-single-top -D -e io.github.tiagoshibata.GPSD_SERVER_ADDRESS ${ADDRESS} --ei io.github.tiagoshibata.GPSD_SERVER_PORT ${PORT} -n io.github.tiagoshibata.gpsdclient/.GpsdForwarderService && adb shell input keyevent 4

fi

exit 0`


Adding DoNotClearSuperAgent.

This will allow user to prevent from clearing up the SuperAgent objects every time when they do GET/PUT/PATCH/POST etc.  Default value for DoNotClearSuperAgent is false just to make it similar to how it works before.

Outdated.


feat: Add Travelling Salesman Problem(Naive Approach)

Description of Change

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

Made some changes.
@ayaankhan98 Please check if it is appropriate now. Thanks!

Thankyou @Panquesito7 , I'll make the necessary changes.
Thanks! :)

I think all the requested changes have been covered.
Please review @Panquesito7

Addressed all the requested changes. Please review. Thanks!

@Panquesito7: Here's the output, goes in:

Is it shows correct?

@Panquesito7: Here's the output, goes in:

Is it shows correct?

That looks correct for me. @Mayank17M, does that look correct?

Yep, that's correct :)

Thanks for reviewing @amino19! 😄


ListView is deprecated

There is no more information that I can provide to explain the error, because it's pretty simple: ListView is deprecated on React-Native and this plugin isn't useable anymore.
Anyone has any solution to it?

.

@brh55 any solution??

I have forked one of brh55 branches, flat-list-parent, and modernised the component lifecycle and merged in the master branch.
This was a quick and dirty exercise and I've not even looked at the unit tests. However this is now working swimmingly with the latest version of React.
It can be found here, hopefully this will assist in keeping this awesome component moving forwards.
git://github.com/eddyoc/react-native-masonry#flat-list-modernize

still not fixed ?

still not fixed ?

git://github.com/eddyoc/react-native-masonry

i've installed your fork and I'm still recieving the ListView error?

Guys you can use ListView
in react-native-masonry/components/Masonry
import { View, ListView, Image, Text, Dimensions } from 'react-native';
change with
import { View, Image, Text, Dimensions } from 'react-native';
import ListView from "deprecated-react-native-listview";

And also you need to add this deprecated library via yarn or npm


Incompatibility issues with werkzeug 1.0

On February 6th a new version of Werkzeug was released: https://pypi.org/project/Werkzeug/#history
It introduces a breaking API change which results in a crash.
Expected Behavior
Proper version of Werkzeug should be installed.

FYI: I encounter same problem, and I fixed it in my fork, and you can install it in this way.

Thank you rayluo!

FYI: I encounter same problem, and I fixed it in my fork, and you can install it in this way.

Thank you!

Fixed since 0.3.2.


Add plugin system

with pre and post parsing hooks, add queryByPath utility, full path and url to pages, metadata from Notion

fixes #20 and #6


Allow Darkcoin as a payment method

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/2280715-allow-darkcoin-as-a-payment-method?utm_campaign=plugin&utm_content=tracker%2F47&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F47&utm_medium=issues&utm_source=github).

Thanks for filing this Issue @ccverg, I am totally for adding this!

Due to the low transactional volume from BTC, we don't have any plans to support other (and more obscure) cryptos at this point.


sample_encode: Added encoding in qpfile mode

sample_encode reads file with information for each frame (display order, QP,
frame type) and encodes stream in encoded order using this readed information.

proposed changes made

@pgribov , please address Miroslav's comments

@mgonchar

@dmitryermilov

@onabiull


US 04.05.01

As a meal planner, I want to add ingredients I have picked up to my ingredient storage, reminding me to complete details like location, actual amount, and unit.

The unit means that kg, grams, litres
The amount means the number of the ingredient


Name is Completely Wrong

This article is supposed to be about C++ development. Why in the world does it include the Basic Android Application template that creates a project in JAVA? Furthermore, why in the world is that bug identified as "Closed - Not Bug" at https://developercommunity.visualstudio.com/t/c-android-project-template-creates-in-fact-a-java/8149?
Makes absolutely zero sense.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 3558cc94-e230-8596-f2bb-19dde4230493
Version Independent ID: 726e26ac-298a-e650-0267-a20846899722
Content: Cross-platform mobile development with C++
Content Source: docs/cross-platform/visual-cpp-for-cross-platform-mobile-development.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@smolbach
I can't speak to why the VS C++ cross-platform templates let you create Java projects, but they do, so we document the fact. A product issue should be taken up with Developer Community, since the docs team can't fix those.


Modal Not looking right

from 640px -> 768px

tyyy

droping my assignment
@hassnian  wanna take this? iirc ModalBody is yours


Update

Hi!
How would one update PowerShell through command line? Can it be done through cURL?
Thanks.

Depends on the OS, follow the instructions per OS in README.md.  If Windows, there is a installer proposal in the works and open for feedback


Use wrappedLineExtent in moveVisually

This adds to #4539, so please review that first.

Trying to put the last two patches in this on top of 7ad2ce62 produces relatively complicated-looking conflicts. Could you rebase them for me, since you're less likely to screw something up?
Is cb2ebf1 intended as a relatively pleasant alternative to an infinite loop for when there are bugs in moveVisually, or are there other situations that might trigger it?

Rebased. a7f80181 is intended for issues in moveVisually, yes. Those probably could come up with wrong bidi orders, too, though.

Thanks! Merged.


Rainbow Brush - Find a way to give Retain to the description of cards which always change their own descriptions

Wish. No energy, no motivation.

Rainbow Brush does not give Retain anymore.


Weird hidden characters after build

I'm using the build on SalesForce Cloud market as a static resource, I wasn't and error from the application when trying to save the result js files, I've isolated the problem by removing the line by line and I've found weird characters that are not allowed in a Html context. one of the examples of what I'm talking about is this '[' if you try to delete the first quote the cursor doesn't move the first time as there is a hidden char between the visible ones, another problem I've found was Ft=["","",""].
Let me know if you need more info

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34571885-weird-hidden-characters-after-build?utm_campaign=plugin&utm_content=tracker%2F32095848&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F32095848&utm_medium=issues&utm_source=github).

@mushdigital20 can you share a version of the generated app that exhibits the problem?

closing


YuMiArm close_gripper() function has multiple missing parameters

force and width are checked but not passed in to the constructor

Fixed by 70e5073 and f9a0add


Design document to restore cQube

Preparing the design documentation to restore the cQube

@htvenkatesh  please review the document.

Task Completed


Missing data rights and commercial usage legal statement --> e.g missing end user agreement

Expected Behavior
Commercial entities utilizing the OT network or protocol require legal documents that define the data ownership and general liabilities or warranties of the protocol and network.
E.g. include end-user agreement.
Actual Behavior
No document that clarifies data ownership or general legal liabilities and warranties when utilizing the OT network as commercial organization
No end user agreement.
Specifications
http://www.clock.org/~fair/opinion/software-liability.html

This issue is being closed as inactive due to the date of the last activity on it. If you believe this is still a problem, please create a new issue and confirm that it is reproducible in the current ot-node release version.
We are working towards closing open issues that meet specific criteria and ask you to create a new one for those that that are truly bugs in current release. We'll be monitoring those issues so that they are properly managed.
Thank you,
OriginTrail Team


Add --invoice-macaroon-file option

This PR adds --invoice-macaroon-file which can be used to create a NWC connection that only supports get_info, make_invoice, get_balance, lookup_invoice.
I know this code is probably shit since it's my first time writing a non-trivial amount of rust code but I wanted to let you decide if you're interested in this enough to merge or refine it. It's not backwards compatible though since I renamed --macaroon-file to --admin-macaroon-file.
--admin-macaroon-file and --invoice-macaroon-file should also probably use exclusive OR. It currently favors --invoice-macaroon-file. I also only tested get_info and make_invoice.
We're using this at SN for local dev so we can attach NWC send+recv, see https://github.com/stackernews/stacker.news/pull/1442.
Feel free to close this if not interested, just wanted to offer giving this back :)

This is great! Will definitely take a look through

Cool, thanks for the review! I'm learning how to write if/else in rust lol. Intriguing language.


scrollable button group list

hi, do you have any plan update this package that allow user can scroll  button group list ?

@YouSour I think this package is not maintained anymore, isn't it @Frezyx ?

@Frezyx i'm not sure , but i see source code just updated

Hello @YouSour !
Are you try to use GroupButton wrapping in ListView widget ?

@Frezyx yeah , what i want it scrollable 😄

Hello @YouSour ✋🏻
I release 3.0.0 version of package. In this version you can use different grouping types.
This functionality can help you with your problem.
Check extended example here

@Frezyx ok, thank you so much, you can close this issue , bro,  i 'll try it on my project , any problem i will report to u 😄


🛠 Tooling: Add a lint rule enforcing a space after }

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
I sometimes see code like that has a } not followed by a blank line and personally find it hard to read:
export function apple() {
	return "apple";
}
export function bananaOrCherry(condition: boolean) {
	if (condition) {
		return "banana";
	}
	console.log("cherry");
	return;
}

Normally I don't mess with lint rules that touch formatting. But Prettier won't touch this case, and it's irksome to me. Let's set the ESLint configuration in this template to require a newline after }.
export function apple() {
	return "apple";
}

export function bananaOrCherry(condition: boolean) {
	if (condition) {
		return "banana";
	}

	console.log("cherry");
	return;
}

Additional Info
No response

Iam a beginner and trying to find the solution for this issue and  found out that To configure ESLint to require a newline after }, you can add the object-curly-spacing rule and set the always option to true. Here's an example configuration in a .eslintrcfile.
{
"rules": {
"object-curly-spacing": ["Warning", "always"]
}
}
we can give a warning message so that the issue is solved is it correct?

@kou333 hello and welcome!

object-curly-spacing rule

Not quite. The object-curly-spacing rule applies to spacing inside object brackets. It might be configured to suggest something like:
- const value = {happy: true};
+ const value = { happy: true };

I'm looking for something that applies to a newline after a closing bracket. See the top post in this issue for what that would look like

we can give a warning message so that the issue is solved is it correct?

When we do find a rule, I'll want it to be configured as "error", not "warn" (see the ESLint Configure Rules docs). In my experience, warnings tend to get ignored over time. Only blocking errors really prevent people from checking in violating code.

Aside: your posts will be easier to read if you format them nicely with GitHub Flavored Markdown. See this GFM cheatsheet for a nice quick reference. Cheers!


What are the other valid setting values?

Based on the options in the Azure Portal, there should be 5 options for performanceScenario (4 test types and none), but the only one I can find documented anywhere is the default, basic. The other properties are similarly lacking in documentation of their valid options. We are basically left to guess at the non-default values. Can documentation of these properties be expanded to at least include a list of valid values?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 98ede797-d606-bbd4-e274-89da03f63721
Version Independent ID: 02c78d95-f74c-60c1-f5aa-0bf49ee41fd3
Content: Azure Performance Diagnostics VM Extension for Windows
Content Source: articles/virtual-machines/troubleshooting/performance-diagnostics-vm-extension.md
Service: virtual-machines-windows
GitHub Login: @genlin
Microsoft Alias: genli

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

#please-close This article moved, so I am closing this issue (I'm unsure if it was fixed). The new version should be here:
https://docs.microsoft.com/troubleshoot/azure/virtual-machines/performance-diagnostics-vm-extension
Investigating/working on this in AB#1958451


Zexuan

Fixes #(issue number)

Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Deploy your changes to github page or netlify and mention deployment link:
Add the deployment link here.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have mentioned related issue number in PR description.
[ ] I have added screenshots and related demo links of my changes..
[ ] The title of my pull request is a short description of the requested changes.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

@ZeTheDeveloper why are you spamming?

Hi QAXIMAAZARSHAD, the reason that you saw so many pull requests is that I
accidentally created pull requests into your repository instead of my
master branch. Sorry for the inconvenience and it won't happen again.
On Mon, 5 Sep 2022, 11:52 Qazi Maaz Arshad, @.***>
wrote:

@ZeTheDeveloper https://github.com/ZeTheDeveloper why are you spamming?
—
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/pull/535#issuecomment-1236515641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXQLK7TCKLCZWSRWA6EI3BLV4VU63ANCNFSM6AAAAAAQEMRMJY
.
You are receiving this because you were mentioned.Message ID:
@.***>

By the way, I am trying some Github Actions in my forked repository, does
the builds send emails to you as well? If so, I am so sorry for the
inconvenience.
On Mon, Sep 5, 2022 at 12:18 PM ZE XUAN LEE @.***> wrote:

Hi QAXIMAAZARSHAD, the reason that you saw so many pull requests is that I
accidentally created pull requests into your repository instead of my
master branch. Sorry for the inconvenience and it won't happen again.
On Mon, 5 Sep 2022, 11:52 Qazi Maaz Arshad, @.***>
wrote:

@ZeTheDeveloper https://github.com/ZeTheDeveloper why are you spamming?
—
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/pull/535#issuecomment-1236515641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXQLK7TCKLCZWSRWA6EI3BLV4VU63ANCNFSM6AAAAAAQEMRMJY
.
You are receiving this because you were mentioned.Message ID:
@.***>

By the way, I am trying some Github Actions in my forked repository, does the builds send emails to you as well? If so, I am so sorry for the inconvenience.
…
On Mon, Sep 5, 2022 at 12:18 PM ZE XUAN LEE @.> wrote: Hi QAXIMAAZARSHAD, the reason that you saw so many pull requests is that I accidentally created pull requests into your repository instead of my master branch. Sorry for the inconvenience and it won't happen again. On Mon, 5 Sep 2022, 11:52 Qazi Maaz Arshad, @.> wrote: > @ZeTheDeveloper https://github.com/ZeTheDeveloper why are you spamming? > > — > Reply to this email directly, view it on GitHub > <#535 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AXQLK7TCKLCZWSRWA6EI3BLV4VU63ANCNFSM6AAAAAAQEMRMJY > . > You are receiving this because you were mentioned.Message ID: > @.***> >

Nope,
I will get emails only when you make changes in my repository.

okay noted, thank you for the clarification. I'll make sure the accidental
creation of pull requests will not be happening again.
On Mon, Sep 5, 2022 at 12:33 PM Qazi Maaz Arshad @.***>
wrote:

By the way, I am trying some Github Actions in my forked repository, does
the builds send emails to you as well? If so, I am so sorry for the
inconvenience.
… <#m_7659928112774943453_>
On Mon, Sep 5, 2022 at 12:18 PM ZE XUAN LEE @.> wrote: Hi
QAXIMAAZARSHAD, the reason that you saw so many pull requests is that I
accidentally created pull requests into your repository instead of my
master branch. Sorry for the inconvenience and it won't happen again. On
Mon, 5 Sep 2022, 11:52 Qazi Maaz Arshad, @.> wrote: > @ZeTheDeveloper
https://github.com/ZeTheDeveloper https://github.com/ZeTheDeveloper why
are you spamming? > > — > Reply to this email directly, view it on GitHub >
<#535 (comment)
https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/pull/535#issuecomment-1236515641>,

or unsubscribe >
https://github.com/notifications/unsubscribe-auth/AXQLK7TCKLCZWSRWA6EI3BLV4VU63ANCNFSM6AAAAAAQEMRMJY
. > You are receiving this because you were mentioned.Message ID: > @.***>

Nope,
I will get emails only when you make changes in my repository.
—
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/pull/535#issuecomment-1236530857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXQLK7XXGY7ULNMKXQKBMLLV4VZ3NANCNFSM6AAAAAAQEMRMJY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add flex/bison and other yosys deps as explicit dependencies

Fixes https://github.com/google/heir/issues/172
Fixes https://github.com/google/heir/issues/118

@asraa This slightly adapts your PR, and it will also take some tweaking internally with the copybara rules. Will do that after pull_ready.

Migrating to an internal change, since the yosys dependency changes conflict with internal copybara transforms.


Use scopes where appropriate

If using application default credentials via the GOOGLE_APPLICATION_CREDENTIALS environment variable, the credentials need to be scoped explicitly. We should do that.

This is now done for the GCS wrapper, in #58. We don't need to do it for gRPC.


Form component - Heap out of memory

Anyone getting this error during build time?
I'm currently using
zod: 3.21.4
typescript: 5.1.6

I'm currently having this issue now. It's only happened after installing shadcn/ui. Hard to track down because before tsc can print out the memory details the whole process crashes.

I figured this out pretty quickly after posting this message. The reason we were having this issue is because we have multiple packages in our monorepo using Zod and shadcn/ui has a different version of Zod.
Since we're using pnpm we used the overrides feature and now the OOM error is gone.
"pnpm": {
    "overrides": {
      "csstype": "3.1.2",
      "vite": "5.0.12",
      "zod": "3.22.2"
    },

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Document how image thumbnails are generated

We should document how thumbnail generation works, when it happens, how the closest matching size is determined, and so on.
Related: #3320

Hi, I'd like to contribute to this project.

We'll have a section for that in the new docs: https://docs.getsaleor.com/docs/next/customization/thumbnails/


net_listening always returns True

What is wrong?
net_listening  "Returns true if client is actively listening for network connections." by spec, however trinity always returns True. The RPC Server should use the event bus to determine whether the server (or whatever runs it) is running and accepting connections.

Part of #26


feat: add ddev_version_constraint for add-ons

The Issue

https://github.com/ddev/ddev-xhgui/pull/31

The current approach when you need to add a version constraint to an add-on involves using ddev debug capabilities, which is not obvious.
This isn't good because capabilities aren't added for each DDEV release and you can't add a version constraint for some minor release.
How This PR Solves The Issue
Adds ddev_version_constraint to install.yaml for add-ons that allows you to check the semver constraint.
TODO:

[ ] update https://github.com/ddev/ddev-addon-template/blob/8d7dac3113074fbf6e6b37696a24d837fa323056/install.yaml#L24-L26.
[ ] update https://ddev.com/blog/advanced-add-on-contributor-training/

Manual Testing Instructions
git clone https://github.com/ddev/ddev-phpmyadmin
echo "ddev_version_constraint: '>= v1.23.4'" >> ddev-phpmyadmin/install.yaml
ddev get ddev-phpmyadmin # should show error
sed -i 's/v1.23.4/v1.23.3/' ddev-phpmyadmin/install.yaml # lower the constraint
ddev get ddev-phpmyadmin # should be installed

Can pull when you're ready. There are a couple of checkmarks still listed.


where is cd mAP & main.py?

I don't find the mAP directory and main.py . Where are they? I want to evaluate the test results.

Hi, in the './experiments' directory.


Re-design the Covid Section with proper Responsiveness

CURRENT BEHAVIOUR: The covid Page needs to be updated with improved responsiveness.
EXPECTED BEHAVIOUR: Re-design the whole page from scratch with proper responsiveness.

@Tess-Vanta Take this one.

@Tess-Vanta Take this one.

Yes I will take this up


[rancher-dns] [rancher-net] l2-flat network should update DNS for all containers on managed network

Rancher versions:
rancher/server: V1.1.0
rancher/agent: v1.2.7
Infrastructure Stack versions:
healthcheck: v0.3.3
l2-flat: v0.13.4
network-services: v0.7.18
scheduler: v0.8.2
Docker version: (docker version,docker info preferred)
Client:
Version:      17.06.1-ce
API version:  1.30
Go version:   go1.8.3
Git commit:   874a737
Built:        Thu Aug 17 22:48:34 2017
OS/Arch:      linux/amd64
Server:
Version:      17.06.1-ce
API version:  1.30 (minimum version 1.12)
Go version:   go1.8.3
Git commit:   874a737
Built:        Thu Aug 17 22:54:55 2017
OS/Arch:      linux/amd64
Experimental: false
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
NAME="RancherOS"
VERSION=v1.1.0
ID=rancheros
ID_LIKE=
VERSION_ID=v1.1.0
PRETTY_NAME="RancherOS v1.1.0"
HOME_URL="http://rancher.com/rancher-os/"
SUPPORT_URL="https://forums.rancher.com/c/rancher-os"
BUG_REPORT_URL="https://github.com/rancher/os/issues"
BUILD_ID=
4.9.45-rancher
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Hyper-V
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
1 Rancher Host Node w/ Internal DB + 2 Agent Nodes
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Create a Stack+Service and DO NOT expose any ports. Since the managed networking is l2-flat, even with no exposed ports (which just forwards from the host IP to the container), the container is still accessible from external devices. This is desired for situations in which each host has multiple services required by external applications running on overlapping ports - thus different IPs for each container is desired so that the services can operate on the same port.
Results:
The problem is that DNS updates are only triggered if the service has a port exposed, and even then it doesn't update the DNS entry to the IP on the managed network, it updates to the host's IP. I can understand why this is desirable in other network overlays, but it is not desirable in the l2-flat network configuration I had in mind.
I do see TXT entries added to the target DNS server for each container, but no A records for them unless a port is mapped and even then its for the IP of the host, not the container.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Begin testing OSG 3.5 out of testing (SOFTWARE-3781)

I've tagged most of the expected packages for the release into 3.5 testing so we should start seeing meaningful test results. Notable missing packages since they're not yet RFT:

condor 8.8
osg-configure 3.0.0
xcache 1.1

I checked Koji and all the packages you mentioned got tagged.


fixed stealth to override the newPage on the context not on browser i…

…tself. Fixed KV test same as in #feature/session-pool.

Just merge master and build docs and I'll merge it.


Update heading and navbar font to Quicksand

As a side note, should we increase the font size for the navbar title?

At the moment it is 16px, I think I prefer it as it is in terms of how it is balanced across the rest of the page 🤔
I had a quick look at other OSS packages:

Torch: 20px
TF: 14px
NumPy: 16px

I think Torch only gets away with such large font for their navbar because their navbar is super thick

At the moment it is 16px, I think I prefer it as it is in terms of how it is balanced across the rest of the page 🤔
I had a quick look at other OSS packages:

Torch: 20px
TF: 14px
NumPy: 16px

I think Torch only gets away with such large font for their navbar because their navbar is super thick

Ah, good call on the OSS comparison. I agree this should be fine! 👌


It won't install on latest node v15.11.0 + npm v7.6.0

Make sure these boxes are checked( - [x] ) before submitting an issue.

[y] Running latest npm and node? -  node: 'v15.11.0', npm: '7.6.0'
[y] The node_modules folder has right permissions (Run ls -ld $(npm root -g)/crontab-ui) - drwxrwxr-x
[y] Read issues.md
[y] Gone through existing open and closed issues.

~$ npm install crontab-ui
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: 'latest' },
npm WARN EBADENGINE   current: { node: 'v15.11.0', npm: '7.6.0' }
npm WARN EBADENGINE }

And it simply does not install or run.
It however does create and copy over file under ~/.npm-global/lib/node_modules/crontab-ui
Please help or fix.

As far as I can try, the version of {node: 'v15.8.0', npm: '7.5.1' } for some reason still works.

I think this is just a warning right. Does it not work?
Anyway I will change the min version to a lower value. Thanks!


[REQUEST-FEATURE] Where is OFFSET parameter in SugarRecord::find method

Android SQLite database supports OFFSET keyword [1:- here, 2:- here]. There's no OFFSET parameter available in SugarRecord::find method.
Currently is there any way if I want to use OFFSET?

I believe this is  currently possible by writing a raw SQL query
Example:
        Users.findWithQuery(Users.class, "SELECT * FROM USERS LIMIT ? OFFSET ?","10","0");

@kigen Thanks.


GitHub "master" becomes "main"

I realized that GitHub recently switched from master to main branch names for new repos.
That led to the following for me: Did not find a README[.md|.rst] file at https://raw.githubusercontent.com/iomega/ms2query/master/ while it should actually have looked in the main folder.
[Repo I ran it for was: https://github.com/iomega/ms2query]

Ok, just saw that I could run howfairis https://github.com/iomega/ms2query --branch main, but if main becomes the new standard on GitHub than it would still be nice to be included automatically I guess.

Agreed, this should be added

maybe we can first ask the github api what the default branch is for the repo if the user didn't specify it

PR #92

PR #92

PR was merged, closing


Keeps saying Disable Windows Defender

i have used every single version all of them say disable windows defender, even tho i did why does it say this please help!

WHICH MEANS UR INFO'S ALREADY GONE


[Fix-5726] When we used the UI page, we found some problems such as parameter validation, parameter update shows success but actually work

Purpose of the pull request

fix the issue: #5726
enhance the logic in #5696 , it's my fault that I failed to submit the latest code about 'createWarning.vue' due to the network error,  and I'm very sorry.

Brief change log

When editing an existing data source, the data source type should be changed.
When editing an existing data source, the name of the existing data source can be changed as expected.
When trying to create an token, if the expiration time is empty,It should be prevented at the front end
Non-Admin user profile page could display tenant information  as expected.
When trying to create an alarm instance, if the alarm plugin is empty, It should be prevented at the front end (#5696  case)

Verify this pull request
Manually verified the change by testing locally.

munual test works well like this:

update the name of existing dataouse can work as expected:

When trying to create an token, if the expiration time is empty,It should be prevented at the front end

Non-Admin user profile page does not display tenant information

Codecov Report

Merging #5727 (afe90c6) into dev (93f1d4d) will increase coverage by 0.02%.
The diff coverage is 38.30%.

@@             Coverage Diff              @@
##                dev    #5727      +/-   ##
============================================
+ Coverage     45.32%   45.35%   +0.02%
- Complexity     3678     3686       +8
============================================
  Files           607      607
  Lines         24797    24857      +60
  Branches       2803     2825      +22
============================================
+ Hits          11240    11274      +34
- Misses        12486    12499      +13
- Partials       1071     1084      +13

Impacted Files
Coverage Δ

...phinscheduler/alert/plugin/AlertPluginManager.java
80.55% <0.00%> (ø)

.../api/controller/AlertPluginInstanceController.java
7.69% <ø> (ø)

...eduler/api/service/impl/DataSourceServiceImpl.java
44.76% <0.00%> (ø)

...inscheduler/common/task/shell/ShellParameters.java
11.11% <0.00%> (ø)

...he/dolphinscheduler/common/utils/VarPoolUtils.java
53.84% <0.00%> (ø)

...ler/remote/command/TaskExecuteResponseCommand.java
0.00% <ø> (ø)

...server/master/processor/TaskResponseProcessor.java
0.00% <ø> (ø)

...rver/master/processor/queue/TaskResponseEvent.java
100.00% <ø> (ø)

...uler/server/master/registry/ServerNodeManager.java
7.01% <0.00%> (ø)

...heduler/server/master/runner/MasterExecThread.java
0.00% <0.00%> (ø)

... and 35 more

Continue to review full report at Codecov.

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

The modification involves  front-end and back-end.
@CalvinKirs  can you help to review the back-end code?
@zhuangchong can you help to review the font-end change?
Thanks!


Separate library build from app build

The makefiles are currently set up to build all of the libraries as part of building each app (using the app's own build directory for the results, etc).  This means that, for example, if you build all of the examples, you end up recompiling the same libraries 9 times.
The makefiles should be set up so that the libraries are compiled into their own destination directory, which only needs to be built once, and then is just linked in by any application build which needs it.
(Eventually, we should probably also add a top-level makefile with build/install targets for those people who just want to download and build esp-open-rtos and then just reference the prebuilt libs as a normal build dependency from their own (external) project directories/makefiles.)

That is a good point. One issue to be solved is how to treat different configurations of the libraries as done in https://github.com/kanflo/esp-open-rtos/blob/master/examples/terminal/FreeRTOSConfig.h
Here I needed a FreeRTOS feature not normally used and thanks to the e-o-r make system it was easy to reconfigure the FreeRTOS build for this app.

Also a good point..  Maybe we should keep both ways of doing it and let the user switch between them with a config variable in the app makefile..
It seems to me we might want even more granularity, though..  For example, just because you're building the FreeRTOS core with an extra feature doesn't necessarily mean you should have to recompile lwIP, axTLS, etc..

The "everything builds together" behaviour was originally a design decision based on my personal preferences when doing microcontroller stuff.
You make some really good points though, especially about integrating with third party build systems. I don't usually use IDEs but I'm guessing that integrating esp-open-rtos into Eclipse, for example, would be a total pain.
FWIW it is quite possible to have third party programs that build totally out of tree (we have private SuperHouse firmwares that are implemented like this). Limited to sticking with the esp-open-rtos Makefile system, of course. I should add a wiki page explaining how to do this (it's really just changing the path where you include common.mk from).
Another minor note is that setting "CROSS=ccache xtensa-lx106-elf-" will dramatically speed up building all examples at once, because most of them have the same library-visible headers. Although this is totally a crutch.
Here's what I think the pros/cons of the currrent approach are:
Pros

Build examples or new programs after a single git clone.
Easily make changes to library-level component configuration or compiler settings and have them propagate to all effected components.
Fairly easy to override settings without needing to dig through the Makefile to do it (may need more documentation to make this clear to people other than me).

Cons

Locked in to one fairly obscure and non-standard build system, with lots of non-recursive make gibberish in it.
Unnecessary recompiling of files when top-level config headers may or may not have significantly changed.
Most config changes at the "library" level will probably fail at link time (ie missing functions) even if they didn't cause the client code to recompile.
Changing the Makefile causes everything to rebuild, which is a pain if you're not actually changing the compiler command line.

... I think there definitely is a better approach that keeps most of the Pros while losing a lot of the Cons. Very open to suggestions/patches!
At first thought, one of the significant changes for "split libraries" is probably a big reorganisation of the directory layout, for a single "include" directory similar to how Espressif (and most C-based projects) do it. Which is perfectly fine, it just means some headers which are currently only stored in git submodules will need to be copied into the tree.

I don't think there's any reason we can't keep all those pros while making the build design a bit more modular..
I think it's important to recognize that this is going to have to be an evolutionary process.  As a first step, what I'd suggest is just changing the makefiles to have a different "build dir for application code" (relative to app dir) vs "build dir for SDK/OS libs" (default relative to esp-open-rtos root, and thus shared by different apps).  That way the lib build process will still automatically be pulled into the application build if necessary, but once they're built once, successive app builds, even of different applications, don't need to rebuild them (unless the app makefile overrides the lib build dir location (for the custom-config-of-FreeRTOS situation, etc)).
I had been thinking about suggesting a reorganization of the whole project directory tree anyway (it would make a lot more sense to me to move most of the Espressif SDK stuff into its own subdirectory, for example, and maybe move core/include up to be the top-level /include, etc), but haven't had a chance yet to put together a real proposal for that..
As far as the header locations, etc, what we probably ultimately want is a "build the OS" target, which builds all the libs and then puts them into "libs/" and also copies any needed public headers out of their individual module locations in to a common "include/" dir.  It would then become trivially easy to also make a 'build install' target, that just copies the contents of libs/ and include/ to an appropriate system location (i.e. '/usr/local/esp-open-rtos' or some such)

This all sounds good to me. "Building" the header files to an include directory is an easy solution.
If you want to leave this with me then I'm happy to have a go at it, put something up for review?

Sounds good to me (you're more familiar with the makefiles anyway) :)

Quoting @foogod from this mailing list thread:

I do like the idea of having /include eventually become "the one true
include directory for app builds", which would mean eventually it would
also contain a bunch of FreeRTOS headers, and relevant stuff from all the
other modules (presumably copied there during the library build)..  This
would require changes to the build process which I think are currently a
bit beyond me, though..  (Likewise, I think it'd be good to eventually move
/lib somewhere else and turn /lib into a "library build output directory"
for compiled libraries, so that if one wanted to build an app from
"outside" our build environment, all they need to do is... snip

+1 for this.
I haven't forgotten about this issue, but there are blocker issues I want to close first so that the Makefile's overall complexity can go down rather than up (it's already too complex):

#58 Assemble binary SDK source files in place
#38 Linker script needs simplification

Once these are done it should be pretty easy to have a dedicated library build as the default (with the option of per-compile library rebuilding if you want to tweak settings for flexibility, resources, size, whatever.)

As an entirely unsolicited opinion, I also prefer the approach of just building everything in one go and in one place. It keep everything conceptually simple and easier to modify and reuse, especially in cases where you want to make big changes.

Also, I'd like to be able to use the SDK by just putting it all in a subdirectory of my project, and adding a list of its .c files to my own Makefile, and include paths to my CFLAGS, and be done with it. This seems somewhat easier if the SDK is already doing pretty much the same.

As an entirely unsolicited opinion, I also prefer the approach of just building everything in one go and in one place. It keep everything conceptually simple and easier to modify and reuse, especially in cases where you want to make big changes.

Unsolicited opinions are fine if you're interested in using esp-open-rtos. :)

Also, I'd like to be able to use the SDK by just putting it all in a subdirectory of my project, and adding a list of its .c files to my own Makefile, and include paths to my CFLAGS, and be done with it. This seems somewhat easier if the SDK is already doing pretty much the same.

This might get unwieldy, I think. There are a lot of source files, including C and assembler source, and the exact list changes fairly regularly. Some internal "components" are built with private header search paths and macro definitions that are required to build them but not for any code linking to them. At best compiling everything with the same CFLAGS would be a very long list of CFLAGS, including a lot of cruft. At worse there will be namespace conflicts and it won't compile at all.
How would it suit to be able to call out from your makefile to a target inside esp-open-rtos to "update libraries if needed"? And for your own code the makefile just links to the built esp-open-rtos libraries and has the correct include path for the header files?
(In case it's not clear, you can currently use the existing makefile system to build-out-of-tree or to place the esp-open-rtos directory as a subdirectory of your program. I have projects that use it like that. And you can also override CFLAGS in your Makefile for the "library" components, before you include common.mk.
You do have to follow the conventions that the other esp-open-rtos Makefiles use though - for how you list source files for the build, etc. Which is a part that we want to fix down the track.)

Unsolicited opinions are fine if you're interested in using esp-open-rtos. :)

Seems I am. Usually I try to stay as close to the source as possible for these things and avoid third-party systems, but I've now tried working with both SDKs from Espressif, and they are just too terrible. The whole time I just wanted to rip everything out and rewrite everything.
I'm going to be trying to use I2S next, so I might contribute register headers for that and SLC, unless you already have those nearly done.

This might get unwieldy, I think. There are a lot of source files, including C and assembler source, and the exact list changes fairly regularly. Some internal "components" are built with private header search paths and macro definitions that are required to build them but not for any code linking to them. At best compiling everything with the same CFLAGS would be a very long list of CFLAGS, including a lot of cruft. At worse there will be namespace conflicts and it won't compile at all.

It's OK, I won't be upgrading constantly. I already got this set up now, actually, and I didn't seem to need any macros at all, but I'm not using any of the extras yet. The list of files isn't too unwieldy (except I did cheat with $(wildcard) in a few places).
The biggest pain is the gigantic list of include paths, and preprocessing the linker script and SDK libraries. I see you've fixed the linker script, which will make things much smoother, and I ended up just doing the libraries in a separate script, and checking the results into my repo.

How would it suit to be able to call out from your makefile to a target inside esp-open-rtos to "update libraries if needed"? And for your own code the makefile just links to the built esp-open-rtos libraries and has the correct include path for the header files?

That could be a compromise if things get out of hand. Especially if it put all libraries and headers in one place.

(In case it's not clear, you can currently use the existing makefile system to build-out-of-tree or to place the esp-open-rtos directory as a subdirectory of your program. I have projects that use it like that. And you can also override CFLAGS in your Makefile for the "library" components, before you include common.mk.

You do have to follow the conventions that the other esp-open-rtos Makefiles use though - for how you list source files for the build, etc. Which is a part that we want to fix down the track.)

It's a bit too magical. I try to avoid magic in my build system at all costs, I want it simple and easy to follow along when something breaks or needs changing, so I prefer my own. Makes it easier for others to replace the build system too if they want.
(My philosophy is that if you allow too much cleverness in the build system, you start relying on it, and understandability of your system suffers, which leads to poorer code.)

I'm going to be trying to use I2S next, so I might contribute register headers for that and SLC, unless you already have those nearly done.

I'm pretty sure nobody else has gotten around to tackling those yet, so please do feel free! :)

The biggest pain is the gigantic list of include paths, and preprocessing the linker script and SDK libraries

We're actually working on fixing all of those issues already.. The SDK lib preprocessing should go away once we switch over to rebuilding them all from C / assembler source, which is currently a work in progress..  Also, once we get the reverse-engineered version of libmain into the mainline source, we should be able to remove a bunch more of the complexity/preprocessing in the linker scripts, as we'll have a lot more control over what's actually inside the libraries (most of the current complexity is just to workaround limitations in the binary-only Espressif libs).  I had already proposed a start on a scheme to work toward consolidating and better organizing the include directories (https://groups.google.com/d/msg/esp-open-rtos/MC6zJyIwZjM/NmVsrkUOBgAJ) but just haven't had time to really do any of the work yet..
However, fixing the include directory situation is actually part of the reason I want to try to separate the esp-open-rtos build from the application build process, because currently the headers you potentially need to include from an app are strewn across a bunch of different submodules (many of which are not ours).  My current plan is for part of the esp-open-rtos build to copy all the include files that are needed by apps into one common include directory as part of the "build" of the OS libs, so that apps don't have to include tons of different dirs when they compile their own stuff, but in order to do this the esp-open-rtos build process needs to happen (at least somewhat) separately from the app build process.

How would it suit to be able to call out from your makefile to a target inside esp-open-rtos to "update libraries if needed"? And for your own code the makefile just links to the built esp-open-rtos libraries and has the correct include path for the header files?

That could be a compromise if things get out of hand. Especially if it put all libraries and headers in one place.

That is the plan, really.  What I would like to aim for is:

You can stick the esp-open-rtos sources anywhere relative to your app build directory and it doesn't care
You don't need to use esp-open-rtos's Makefiles/etc to build your app at all (but you can if you want to, at least for simple things, so it's easy for folks to get up and running quickly)
You can either manually build esp-open-rtos separately, and then just reference its libs/includes from your project, or you can have you Makefiles (or whatever) call esp-open-rtos' to automatically build/update the OS libs as part of your build.
esp-open-rtos' Makefiles will build everything into one 'lib' dir and one 'include' dir (you can either use its default or tell it where to put them).  Then all you need to do when building apps is add those dirs to the include path and libpath (and link in the libs your app uses, of course).

Regarding "magical" build scripts:  We all agree that the current ones are a bit too magic, and have plans to try to clean some of that up (but it's a question of priorities and manpower, really).  On the other hand, it is possible to go too far the other direction.  Making things too simple can result in a project that's impossible to maintain as it grows and doesn't allow the flexibility to support multiple users' use-cases.  It's important to find a balance.
This is another reason, however, why I believe it's good to compartmentalize and separate the build processes for different parts of things, so that you don't have to overcomplicate your app build scripts with all of the stuff necessary to build esp-open-rtos correctly, and esp-open-rtos doesn't need to overcomplicate its system to support building all kinds of different apps in addition to the core OS libs.  Each piece should pick one thing, do it right, and do it cleanly instead of trying to have a one-solution-for-everything monster build system that nobody really understands.

FWIW I agree with everything in @foogod's post. :)

That all sounds quite reasonable. You do seem to have your priorities quite straight with this project, which makes it a lot more appealing to use.

I have tried to use esp-open-rtos with Eclipse CDT and as good as it is (I did the same for Espressif RTOS SDK and it was a nightmare to get it working right) one of the things that could be improved is resolving circular dependencies between the libraries. For some reason Eclipse does not provide a way to link a group of libraries (--start-group / --end-group), which leads to some nasty hacks like putting list of the libraries into a compiler command template. Maybe combining some of binaries into one (using ar, for example), would help.
Another thing I have noticed is that https://github.com/pfalcon/esp-open-sdk already ships the lwip library which sometimes leads to conflicting includes. I was under the impression that many people who use esp-open-rtos also use esp-open-sdk; what would be the best way to avoid lwip conflicts?

Hello resetnow,
Would you mind to share how you integrate esp-open-rtos under eclipse?
I have done basic integration of Espressif NONOS SDK, but have read from you and others here that is a nightmare.
I really will appreciate your response,

@mhanuel26 you can either add your esp-open-rtos project as a “Makefile project with existing code” but if you want to use CDT builder (which eliminates the need to edit makefiles manually) I have created a gist here. It would still be a pain to set up, you could edit .project file by hand though and then reuse it.
Sorry for the comment spam.


Fix fetch doc archives

We were tripping over accessing the owner/repo fields on Repository:
[ INFO ] Ingesting (id: 38D534BF-4BE1-498E-B973-E7E885FC3A4E) ... [component: ingest] (App/Commands/Ingest.swift:91)
[ DEBUG ] Ingesting [38D534BF-4BE1-498E-B973-E7E885FC3A4E] [component: ingest] (App/Commands/Ingest.swift:119)
[ DEBUG ] Ingesting from Github ... [component: ingest] (App/Commands/Ingest.swift:122)
[ DEBUG ] updateStatus ops: 1 [component: ingest] (App/Commands/Common.swift:57)
[ DEBUG ] Ingesting from S3 ... [component: ingest] (App/Commands/Ingest.swift:130)
[ DEBUG ] ingestFromS3 version candidates: 4 [component: ingest] (App/Commands/Ingest.swift:296)
[ DEBUG ] ingestFromS3 versions with doc targets: 3 [component: ingest] (App/Commands/Ingest.swift:301)
FluentKit/Field.swift:23: Fatal error: Cannot access field before it is initialized or fetched: owner

The problem is that the Github ingestion pass saving the model apparently invalidates the relation to Repository. The fix is to refetch the two fields we need.
I've changed the entire ingestFromS3 function back to just using [Package.ID] as its input to avoid any future temptation to access fields on Joined<Package, Repository>.

I've deployed from this branch and we're now correctly discovering doc archives in S3:

Impact on ingest duration is minimal:

(which is to be expected)


CalendarDate accepts datetime.datetime values

This snippet works without any error:
import datetime

from param import Parameterized, CalendarDate

class A(Parameterized):
    cd = CalendarDate(datetime.datetime.now())

I expected this Parameter type to accept dates (e.g. datetime.date.today()) only. If this is a bug, it may be caused by the fact that datetime.datetime objects inherit from datetime.date (see this SO answer).

This is actually a known fact: https://github.com/holoviz/param/pull/348#issuecomment-517037116

It's known, but not necessarily desirable.  Given how long CalendarDate has accepted datetimes, it may not be practical to change that now, but maybe there could be a flag date_only to enforce that it is strictly a date and not a datetime.

Trying to see how CalendarDate is used out there:

A GH search returns just a few hits
panel uses it to define parameters (e.g. value) of the DatePicker widget. Construction this widget with a datetime.datetime parameter raises an error down in Bokeh: ValueError: failed to validate DatePicker(id='1060', ...).value: Expected a date value, got a datetime.datetime. So making the change directly would apparently not affect panel and its users.

Fine by me! Unless anyone else objects, I'm happy to start enforcing accepting dates only.


method call performance

Appears method calls are significantly slower than function calls, perhaps relates to JIT or JS function inlining. Tried running method_call repeatedly to warmup JS engine and to place cls.update in a var cls_update to avoid attr lookup overhead with only small improvement.
test_call.py:
from time import time

class Cls:
    def __init__(self):
        self.count = 0
    def update(self):
        self.count += 1

cls = Cls()

def update(cls):
    cls.count += 1

def function_call():
    for i in range(1000000):
        update(cls)

def method_call():
    for i in range(1000000):
        cls.update()

t = time()
function_call()
tf = time() - t
print('function call: {}s'.format(round(tf,3)))

t = time()
method_call()
tf = time() - t
print('method call: {}s'.format(round(tf,3)))

transcrypt -n test_call.py
console:
function call: 0.003s
method call: 0.242s

Environment:
Linux
Python 3.9.6
Transcrypt 3.9.0
Chrome/Firefox

Suppose the issue is in the get function in transcrypt runtime. Tried some code refactoring experiments, uncertain the impact on compatibility but obtained nearly a 5-fold increase in performance with the the following change in the get function:
original:
return function () {
    var args = [] .slice.apply (arguments);
    return func.apply (null, [aThis.__proxy__ ? aThis.__proxy__ : aThis] .concat (args));
};

revised:
return function () {
    var args = [aThis.__proxy__ ? aThis.__proxy__ : aThis];
    args.push.apply(args, arguments);
    return func.apply (null, args);
};

transcrypt -n test_call.py
console:
function call: 0.003s
method call: 0.052s

Refactoring of __get__ function in transcrypt runtime gained significant performance improvement.
Revision of __get__ function:
method handling with fcall pragma
value: function () {
    var args = [aThis];
    args.push.apply (args, arguments);
    return func.apply (null, args);
},

method handling
return function () {
    var args = [aThis.__proxy__ ? aThis.__proxy__ : aThis];
    args.push.apply (args, arguments);
    return func.apply (null, args);
};

Testing:
test_method_call.py
from time import time

class Cls:
    def __init__(self):
        self.count = 0
    def update_count(self):
        self.count += 1

cls = Cls()

def method_call():
    for i in range(1000000):
        cls.update_count()

t = time()
method_call()
tf = time() - t
print('method call: {}s'.format(round(tf,3)))

transcrypt -n test_method_call.py
Performance of revised vs original __get__:
original __get__ (chrome)
method call: 0.252s
revised __get__ (chrome)
method call: 0.050s
original __get__ (firefox)
method call: 0.458s
revised __get__ (firefox)
method call: 0.117s
Performance of revised vs original __get__ using __pragma__ ('fcall'):
original __get__ (chrome) fcall
method call: 0.219s
method call after 3*1M loops: 0.205s
revised __get__ (chrome) fcall
method call: 0.014s
method call after 3*1M loops: 0.006s
original __get__ (firefox) fcall
method call: 0.419s
method call after 3*1M loops: 0.413s
revised __get__ (firefox) fcall
method call: 0.078s
method call after 3*1M loops: 0.075s
Revised __get__ performance was increased 5x for chrome and 4x for firefox. Using fcall pragma, with original __get__ there is approx. 10-20% performance improvement, whereas with revised __get__ there is sum performance improvement of 18x for chrome and 6x for firefox. Chrome achieved 42x performance after warmup of fcall bound function that is close to performance of function calls.
Similar revision can be applied to class method and static method handling. Have done limited testing of __get__ revision, not certain of compatibility impact.


[WIP] Issue 10441 ToDo Improvement

Partially fixes: #10441
Testing Plan:

Added extra field for taking user input for task description.
Task name renders bold.
Added Index numbers to keep track of the order of creation of tasks.

GIFs or Screenshots:

Issue

When I add a task, the task name field resets but the description field stays the same containing the text.

I tried testing this and I came across an issue.

The description field does not get cleared.

@vinitS101 Yeah, I've mentioned that in the PR itself as an issue I'm facing. Can you guide me if you know what is causing that?

Check static\js\todo_widget.js, you will find this:
elem.find(".add-task").val('').focus();
You need to do something similar for .add-desc.

@vinitS101 @timabbott I guess everything is fixed now. :thinking:

LGTM.

@showell can you give this a quick look and merge if you're happy with it?
I agree with the original poster of #10441 that ultimately what we actually want to do is support GitHub's TODO list format in our markdown syntax, but I'm fine with merging this tweak to the existing feature.

Merged!  I am not super familiar with the todo widget, so I've never noticed its strange numbering issues.  (We generally don't want zero-based indexes for user-facing things, and the indexes should sort.)  But I think those quirks preceded your change.
Sorry it look so long to review this!  Thanks for working on it.


Error with xp_cmdshell

Hi,
After install new version 21.1.5 I get this error, before it works fine.
Error occurred during SQL query execution
Reason:
SQL Error [229] [S0005]: The EXECUTE permission was denied on the object 'xp_cmdshell', database 'mssqlsystemresource', schema 'sys'.
when running:
exec xp_cmdshell 'net use'
thank you
System information:

Windows 10
21.1.5

@resquej Hello, can you be more specific, when this happens? Also please attach a log file.

there is no update on the issue for a long time.
If it is still actual for you - feel free to reopen it


agni_tf_tools: 0.1.2-1 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository agni_tf_tools to 0.1.2-1:

upstream repository: https://github.com/ubi-agni/agni_tf_tools.git
release repository: https://github.com/ubi-agni-gbp/agni_tf_tools-release.git
distro file: lunar/distribution.yaml
bloom version: 0.8.0
previous version for package: 0.1.0-1

agni_tf_tools
* fix compiler warnings
* cmake: cleanup, correctly announce exported lib
* Contributors: Robert Haschke

Relies on #21477

Closing and reopening to trigger Travis.


(403) Forbidden while opening "Download the notebook instead"

It is not possible to open the "Download the notebook instead" page section.
Error
The remote server returned an error: (403) Forbidden.
Time 11:35 Seattle
2022-8-6

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8ab4ef99-c53f-2840-4e43-d09537805324
Version Independent ID: bfb0d3ad-e94e-0b5e-ed37-1dd654ecac14
Content: Boston Safety Data - Azure Open Datasets
Content Source: articles/open-datasets/dataset-boston-safety.md
Service: open-datasets
GitHub Login: @samuel100
Microsoft Alias: samkemp

@samuel100 Thanks for the feedback! We are currently investigating and will update you shortly.

See also #83040, #86417

This still a problem


12 - Add version number to footer

What this PR does / why we need it:
This PR adds the version number to the footer using the js-dataverse module.
Which issue(s) this PR closes:

Closes #12

Special notes for your reviewer:
Version number is only going to appear if you are logged in in a local dataverse installation. See this PR for more information on how to test this.
Apart from the version number I fixed some issues with the login/logout
Suggestions on how to test this:
To test this you need a Dataverse local installation connected to the frontend. See this PR for more information on how to test this.
For testing the mocked js-module api calls:
npm run storybook
If you go to the Layout page in storybook you can see the version number because the api call has been mocked.
Same with the cypress component tests and vitest tests, api calls are mocked.
Github Action for the e2e tests is not working because we don't have a way to test this, check this opened issue #65  for more information
⚠ We first need to merge feature/53-login-logout branch
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
Now we can obtain the dataverse version number from the js-dataverse-client and show it at the frontend footer.
Additional documentation:

Github Action for the e2e tests is not working because we don't have a way to test this, check this opened issue #65 for more information

Just a few minutes ago @poikilotherm got a Dataverse container running in GitHub Actions to file upload. Check this out: https://github.com/IQSS/dataverse-sample-data/compare/master...container-test
This means it's probably possible to get the e2e test working!

This looks great. I can see how #65 will fix the e2e tests in a local dev environment, will that fix the e2e test that runs in the github action? Or is that considered a separate issue from this PR?

I'm also interested in understanding why the e2e test is failing.
In addition, I'm wondering why the "Hello Dataverse" story doesn't have a footer at all. Should it? I took a quick swing at adding it (locally) but I must be doing it wrong because the version isn't there and it's weirdly centered:

This is what I tried:
$ git diff
diff --git a/src/sections/hello-dataverse/HelloDataverse.tsx b/src/sections/hello-dataverse/HelloDataverse.tsx
index 52091d7..593cefd 100644
--- a/src/sections/hello-dataverse/HelloDataverse.tsx
+++ b/src/sections/hello-dataverse/HelloDataverse.tsx
@@ -1,6 +1,7 @@
 import logo from '../ui/logo.svg'
 import styles from './HelloDataverse.module.scss'
 import { Trans, useTranslation } from 'react-i18next'
+import { FooterFactory } from '../layout/footer/FooterFactory'

 export function HelloDataverse() {
   const { t } = useTranslation('helloDataverse')
@@ -19,6 +20,7 @@ export function HelloDataverse() {
         rel="noopener noreferrer">
         {t('linkText')}
       </a>
+      {FooterFactory.create()}
     </section>
   )
 }

I noticed that the footer is not weirdly centered and correctly shows the version in the "Logged In" layout story:

@pdurbin I don't think the Hello Dataverse page was meant to have the footer, it was just a test page made for the original project setup.  The Footer is meant to be part of the Layout component, so maybe there are styles in that component that affect the Footer placement.

Actually, I see the footer now in Storybook. Maybe it was just hidden by the "Controls, Actions" thing at the bottom. I had to scroll down.
No version though. Is the version supposed to be there in the footer in Storybook? The page does use WithLayout.

I do see the version in the footer at https://dev1.dataverse.org/spa/ where I have the following PR deployed:

#67

I'm not sure why because I don't see any commits about a version in the footer in that PR.
It looks like this:

The more important thing is the failing e2e test though, I'd say. 😄

In addition, I'm wondering why the "Hello Dataverse" story doesn't have a footer at all. Should it? I took a quick swing at adding it (locally) but I must be doing it wrong because the version isn't there and it's weirdly centered:

The footer is there but there is scrolling, there shouldn't be a scroll, I fixed the height so now the footer it's always at the bottom without scrolling. Unless of course, that the body content is larger than the viewport.

@philippconzett @ekraffmiller
I did some changes with the help of Guillermo so now the e2e tests for the version number are working both locally and in the Github action
The tests for the login/logout are failing because we do not pass the user credentials. I was looking at the js-dataverse integration tests and there are no tests for this, there is a todo // TODO: Add more test cases once the integration test environment is established. So I guess is fine, we still cannot add the user credentials to the e2e tests
I suggest that we either skip the e2e test for login/logout and add a TODO, as was done in the js-dataverse repo, or let it fail.

I think it's fine to skip the e2e tests and add a TODO, because the container environment is almost ready. Once this is merged (https://github.com/IQSS/dataverse-frontend/pull/87) will we be able to add the e2e test?

@MellyGray you were right! "Base automatically changed from feature/53-login-logout to develop." I had no idea GitHub did this. Interesting!


help users with zsh

zsh is installed by default on macs and this is a bad experience as I just used what worked on bash + on the docs:
$ driftctl scan --from tfstate://**/*.tfstate
zsh: no matches found: tfstate://**/*.tfstate
$ bash
bash-3.2$ driftctl scan --from tfstate://**/*.tfstate
Scanned states (3)
Found resources not covered by IaC:
  aws_internet_gateway:
    - igw-01f1b9d38973e31ce
[...]

On zsh we should use something like driftctl scan --from 'tfstate://**/*.tfstate'
Let's update the docs!
It works well with bash too:
bash-3.2$ driftctl scan --from 'tfstate://**/*.tfstate'
Scanned states (3)
Found resources not covered by IaC:
  aws_internet_gateway:
    - igw-01f1b9d38973e31ce
[...]

Why not documenting using the quotes be the default?

very helpful post regarding the zsh, thanks


failed to load

Very cool add-on, thanks for making it

Using latest arcdps and 1.2 release (I made sure to download a new copy of arcdps)

To be fair I'm on Linux (I made sure to chmod it like I had to with arcdps) ~but my guildie has a typical windows 10 machine and is getting the same error~ guildie on windows 10 actually has it working, when they said couldn't find the file they meant on GitHub xD
I'm looking forward to being able to use it

This is usually what happens when you don't have the visual studio runtime installed. Can you try installing it?
https://support.microsoft.com/en-us/topic/the-latest-supported-visual-c-downloads-2647da03-1eea-4433-9aff-95f26a218cc0
Visual Studio 2015, 2017 and 2019
I really should add a troubleshooting section to the readme

https://github.com/datatobridge/arcdps-uploader#usage
If the extension fails to load, install the latest x64 Visual C++ redistributable

I was just about to post about that. I'll give it a go

I tried using winetricks to install vcrun2019, it showed up in the add/remove tool list, running again showed the repair window
It didn't work
Removed with the wine add/remove tool
Tried again with rclick in lutris, run EXE in wineprefix
ran https://aka.ms/vs/16/release/vc_redist.x64.exe
Install showed up in add/remove tool list again

Still failed
Ran the EXE again just to check and it showed the already installed repair window
did the repair and still failed

Is there a way to get more logging from arcdps?

Nope, this is the first arc dps addon I've tried. No other dll chainloading stuff. Using DXVK but that's not anywhere near the gw2 dir & being handled by wine

log is pretty empty
arcdps.log
Mar 25 21:57:48 arcdps 20210324.210141-352-x64
Mar 25 21:57:48 client: gw2 0x140000000-0x142580000 (0x3CFED3BD)
Mar 25 21:57:48 client: arcdps 0x8D90000-0x8EB2000 (0x0674D21C)
Mar 25 21:57:50 warning: failed to load "C:\Program Files\Guild Wars 2\bin64\arcdps_healing_stats.dll", loadlibrary getlasterror: 193
Mar 25 21:57:50 config: loading from "C:\Program Files\Guild Wars 2\addons\arcdps\arcdps.ini"
Mar 25 21:57:50 client: ready

λ file ~/dloads/d3d9_uploader.dll
d3d9_uploader.dll: PE32+ executable (DLL) (console) x86-64, for MS Windows
λ file /home/jk/documents/games/gw2/drive_c/Program\ Files/Guild\ Wars\ 2/bin64/arcdps_healing_stats.dll
/home/user/documents/games/gw2/drive_c/Program Files/Guild Wars 2/bin64/arcdps_healing_stats.dll: HTML document, ASCII text, with very long lines, with no line terminators
λ cat /home/user/documents/games/gw2/drive_c/Program\ Files/Guild\ Wars\ 2/bin64/arcdps_healing_stats.dll
You are being <a href="https://github-releases.githubusercontent.com/336379174/3d1fad00-8cfe-11eb-9eff-f169d3182a34?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20210325%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20210325T202935Z&X-Amz-Expires=300&X-Amz-Signature=081a1ae8295c122f8974f4f95ac5e7e3e6408a51a48b46498512ef6e3a77877e&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=336379174&response-content-disposition=attachment%3B%20filename%3Darcdps_healing_stats.dll&response-content-type=application%2Foctet-stream">redirected.

I'd downloaded the redirect page :facepalm:
We're in business! No idea if I needed the vcrun2019 install in the end but probably a good idea

Glad you got it solved :)


[Settings] Keyboard Navigation in Navigation pane and UIA pattern for tree items

ℹ Computer information

PowerToys version: 0.31.2
PowerToy Utility:
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 20H2

📝 Provide detailed reproduction steps (if any)

Using keyboard navigation user is unable to switch to different navigation tree item

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

This works for me in 0.41? For you as well?


tensorflow 2.0 giving tf.print message while using rejection_resample

I am using rejection_resample from tensorflow v2.0, this is giving me the following message. While this is no big deal, it is kind of annoying. I don't think this is an issue on my end, I didn't use any tf.print. It will be great to turn this off. Thank you.

Instructions for updating:
Use tf.print instead of tf.Print. Note that tf.print returns a no-output operator that directly prints the output. Outside of defuns or eager mode, this operator will not be executed unless it is directly specified in session.run or used as a control dependency for other operators. This is only a concern in graph mode. Below is an example of how to ensure tf.print executes in graph mode:
    sess = tf.Session()
    with sess.as_default():
        tensor = tf.range(10)
        print_op = tf.print(tensor)
        with tf.control_dependencies([print_op]):
          out = tf.add(tensor, tensor)
        sess.run(out)
    ```
Additionally, to use tf.print in python 2.7, users must make sure to import
the following:

  `from __future__ import print_function`

@celinew1221 I ran the code provided in TF GPU 2.0.0-alpha0. Have not received any error.
. Please check and let us know.

The code provided was in the error message itself. I am using rejection sampling function, and that produces this message. But I managed to get rid of it with the following code:
tflogger = tf.get_logger()
tflogger.setLevel('ERROR')


Latest release breaks backwards compatibility on ternary operator and resourceGroup() function

Bicep version
Bicep CLI version 0.31.34 (ec82b47d63)
Describe the bug
Previous to the latest release (0.31.34), it was possible to get a resourceGroup using a ternary operator and use it for a scope declaration, like so:
param myRgName string = ''
param myRsrcName string

resource myRsrc 'sometype' existing = {
  scope: empty(myRgName) ? resourceGroup() : resourceGroup(myRgName)
  name: myRsrcName
}

With 0.31.34, this instead produces an error:
Error BCP036: The property "scope" expected a value of type "resourceGroup" but the provided value is of type "{ id: string, location: string, managedBy: string, name: string, properties: properties, tags: Tags, type: string }". [https://aka.ms/bicep/core-diagnostics#BCP036]

I have found a workaround, which is to use the ternary inside the resourceGroup function call, like so:
param myRgName string = ''
param myRsrcName string

resource myRsrc 'sometype' existing = {
  scope: resourceGroup(empty(myRgName) ? resourceGroup().name : myRgName)
  name: myRsrcName
}

However, we have quite a few modules that use the ternary with the two separate resourceGroup() calls, and this backwards compatibility issue is going to break all of them, and thus cause a lot of problems for the users of our module library.

@alex-frankel most likely duplicate to https://github.com/Azure/bicep/issues/15517

@alex-frankel most likely duplicate to #15517

Oops, you're right.  I apologize for the duplicate.  I was searching for ternary and scope, not conditional and scope, so the other thread didn't come up.


🚀 Feature: Side Menu Bar Should be Slidable.

🔖 Feature description
The Side Menu Bar Should be  Slidable, the user can reduce its size by using a mouse.
🎤 Why is this feature needed ?
The Side Menu Bar Should be  Slideable, the user can reduce its size by using a mouse. The Text Behind the Menu Bar is Overalpping
✌ How do you aim to achieve this?
The Side Menu Bar Should be Slidable.
🔄 Additional Information

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Are you willing to submit PR?
Yes I am willing to submit a PR!

I'd love to fix this issue. can anyone assign me ?

Apologies @adcs1, there was an overlap.
This issue is unnecesary. The same issue is already being fixed here https://github.com/arc53/DocsGPT/issues/527.