PNG Screenshots

Description
This PR adds PNG screenshots using stb_image_write, and lets you take them with F12 instead of only with a replay script. To support that in a sane way, rawdraw is also updated for upstream changes that add defines for the key constants. And the emu extension interface is updated a bit to make working with modifier keys easier.
Test Instructions
Run the emulator, hit F12 and take some screenshots. They should be written out to screenshot-xxxx.png with xxxx being a seconds+milliseconds unix timestamp. Run the emulator with -r and take some screenshots. Replay the recording with -p. The screenshot-taking should be recorded and replayed properly.
Ticket Links

Readiness Checklist

[X] I have run make format to format the changes
[X] I have compiled the firmware and the changes have no warnings
[X] I have compiled the emulator and the changes have no warnings
[X] I have run make cppcheck and checked that cppcheck_result.txt has no warnings for the changes
[X] I have added doxygen comments to any code used by more than one Swadge mode. This includes /*! \file comments with Design Philosophy, Usage, and Example sections for new headers.
[X] I have run make docs and checked that doxy_warnings.txt has no warnings for the new code

Yeah, and you can have emu_ext.h include the header with the CNFG key codes
so extensions wouldn't have to add another include either.
On Wed, Apr 17, 2024 at 10:42 AM Dylan Whichard @.***>
wrote:

@.**** commented on this pull request.
In emulator/src/extensions/emu_ext.h
https://github.com/AEFeinstein/Super-2024-Swadge-FW/pull/211#discussion_r1568968344
:

@@ -94,6 +96,99 @@ typedef enum
EMU_SCROLL_RIGHT = 0x40,
} mouseButton_t;

+typedef enum
+{

EMU_MOD_NONE  = 0x00,
EMU_MOD_ALT   = 0x01,
EMU_MOD_CTRL  = 0x02,
EMU_MOD_SHIFT = 0x04,
EMU_MOD_SUPER = 0x08,
+} modKey_t;

+/**

@brief Normalized key-codes across all platforms for use by emulator extensions

These are just the CNFG keycodes renamed so extensions don't have to include rawdraw directly.

Eeh, how bad is including rawdraw directly? I look at this and see one
more thing to maintain (even if keyboards aren't likely to change any time
soon).
Oh yeah that's a good point. I did this with the single file rawdraw
initially where it kind of made sense, but now that we just have normal
header files it's not really any different.
—
Reply to this email directly, view it on GitHub
https://github.com/AEFeinstein/Super-2024-Swadge-FW/pull/211#discussion_r1568968344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABYODC67MNIQHJBH2BFLMLY52C6LAVCNFSM6AAAAABGKMX5HGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMBWGI2TEOJWGM
.
You are receiving this because your review was requested.Message ID:
@.***>

Alright, that should do it. Just waiting on the build now.


Have a common section for the configuration of Spine plugins in Gradle build script

At the moment there are at least Gradle plugins supplied by Spine tools, that are configured in build.gradle:

modelCompiler;
codestyleChecker.

We need to have a common spine "namespace" to make the declaration more clean and logical (similarly to the configuration of protobuf plugin — check out the build.gradle). E.g.
spine {

    modelCompiler {
        // ...
    }
    codestyleChecker {
        // ...
    }
}

IMPORTANT: the configuration set on the upper level (e.g. in the root /build.gradle) should still be overridable in the children modules (e.g. some property may be customized in /client/build.gradle).

Implemented as a Bootstrap plugin.


Add support for from_edge

closes #5
closes #20

@fnattino can you please review this PR? The linter errors have been discussed. The function merge_lines can be refactored too, but I am not sure if this will be done in the refactoring PR. Another issue is reported at #20.

Hi @SarahAlidoost really great work! I have left few suggestions, but amazing to see you got it working :).

Hi @fnattino thanks for reviewing and your suggestions. I addressed the comments. Please let me know if something needs further improvements. Please also check the additional test and my reply here. Another point, I am wondering if you have seen issue #20.

Hi @SarahAlidoost, see #21 for a fix to #20


Right click to close main menu or exit current tool

Closes #1014
Compiled version for tests
New:

right click to close main menu if no other tool active
right click to exit currently active tool

Improvements for existing code:

fixed variable, local parameter name collisions

the only caveat is when no tool is active but the user has selected a road

right-clicking at this point exists tool as oppose to closing the road info panel.

the only caveat is when no tool is active but the user has selected a road

Good point, it's driven by separate logic (cool spaghetti), I'll fix it 🙃

I've implemented fix for closing road selection panel on single right click, @chameleon-tbn your turn and I'll merge if you won't find anything else. Good addition to Stable 11.5

I''ll test today in füll modded game :)

I've tested now in a game with 135 Mods, 10k assets. All seems to work as expected - even if i really need to 'relearn' using the right click in the correct way :) 1x = closing actual activated UI, 2x = leave the actual selected button, 3x = completely close TM:PE UI... really handy... i like :) a go from me :)


Update laika-sbt to 0.19.4

About this PR
📦 Updates org.planet42:laika-sbt from 0.18.2 to 0.19.4
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.planet42", artifactId = "laika-sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.planet42", artifactId = "laika-sbt" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #176.


add statements

Add a statement option to the gem.
Statements are based off of invoices, but allow for a statement period rather than a due date. A statement also does not require a status. This exposes start_date and end_date attributes in addition to those attributes exposed by invoices.

I like it!


Fixes issue #82. Multiple tenants now use their own id to scope.

Fixes issue #82

Why is this test failing? I did not even change that line of code? :laughing:


new term request: "intermediate density lipoproteins"

New term: intermediate density lipoproteins
Synonym: IDL
IS-A: CHEBI:6495 - lipoprotein
MESH: D053419
ChEBI has HDL, LDL, VLDL, but is missing IDL.

Now done - see CHEBI:132933.
Kind regards,
Gareth


Overriding stringArgs in MapPartitionsInR

What changes were proposed in this pull request?
As discussed in https://github.com/apache/spark/pull/12836
we need to override stringArgs method in MapPartitionsInR in order to avoid too large strings generated by "stringArgs" method based on the input arguments.
In this case exclude some of the input arguments: serialized R objects.
How was this patch tested?
Existing test cases

@sun-rui , @liancheng, @shivaram

Test build #60318 has started for PR 13610 at commit 939dbd5.

Thanks @NarineK - Change looks pretty good to me.  Minor comment: Can you update the title to have [SPARKR][SQL][SPARK-###] at the beginning ?

LGTM pending Jenkins, and the PR title change. Thanks!

Thanks! Changed the title!

Test build #60318 has finished for PR 13610 at commit 939dbd5.

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

Merged build finished. Test PASSed.

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

Merging to master and branch-2.0. Thanks!


[FR] Add Support for PowerQuery syntax

SentinelOne now has a new query language called PowerQuery that has better performance and capabilities. Field names are different and syntax is slightly changed.

Closed as this was moved to its own repo


Invalid filenames in sdsio_manager on unix-type systems

Path operations in sdsio-server.py hardcode backslahes. This will only work on windows. In the unix-world this will create filenames similar to .\Accelerometer.0.sds, which is actually a hidden file.
See: https://github.com/ARM-software/SDS-Framework/blob/2db17db76acdd2fdf70ead0b721c2518a48879d2/utilities/SDSIO-Server/sdsio-server.py#L41
Since the os.path module is available already, I recommend using os.path.join to create safe file names.
e.g.:
fname = path.join(self.out_dir, f"{name}.{file_index}.sds")

This should be fixed now with commit 3191a3f.


Not working on Windows XP?

Hi,
the application doesn't work on my EEE PC running on windows XP. It says when I launch the executable : "not a valid Win32 application"
Is it normal? Am I missing something?
Thanks in advance!

Windows XP is an extremely old operating system, and even major software like Google Chrome is no longer being updated for it. Consider upgrading.

@Steevflores Electron is only supported by Win7 and later: https://github.com/electron/electron/blob/master/docs/tutorial/supported-platforms.md

Ok thanks. Well, I wanted to take that EEE PC on a road trip, and scan in real time. But I can't manage to install something newer than XP on this. Too bad :p


libusb-1.0 development package not found

Ubuntu 18.04.2 LTS
4.15.0-46-generic
x86_64 GNU/Linux
qt opensource-linux-x64-5.12.2
libusb is located at:
./usr/include/libusb-1.0/libusb.h
./home/thomasd/Qt5.12.2/5.12.2/Src/qtwebengine/src/3rdparty/chromium/third_party/libusb/src/libusb/libusb.h
After run the Project I get an error: libusb-1.0 development package not found

I fix the error, there was no pkg-config installed.


feat(#1141): new 'merge' option to merge YML without variables expansion

new 'merge' option and treatment
new 'expandVariables' for the Job constructor to flag if we should expand or not
new 'merge' test-case for validation

Closes #1141
Explanation
This introduces a new option named --merge (not sure about that, let me know if you have a better idea) which do basically the same job as the --preview one but without variables expansion. The result should be (quite) the same as the "Full configuration" tool of GitLab's internal CI editor: https://docs.gitlab.com/ee/ci/pipeline_editor/#view-full-configuration.
Dev implementation
I just added a new option called merge which follows the same logic as the preview one, adding a expandVariables bollean flag to switch between one or the other.
I also added correponding test-case to validate it on an environment block in a CI using a variable as 'name' that can not be guessed outside a git tag.
NOTE - I'm having a failure running the tests but I don't think it was due to my dev. Please see the final notes below.
Illustration
With the same YML as in the #1141 ticket:
# .gitlab-ci.yml
---

.anchored_vars: &anchored_vars
  MY_VAR: 'my var'

my_job:
  rules:
    - if: $CI_COMMIT_REF_NAME
      variables:
        <<: *anchored_vars
  script:
    - echo "$MY_VAR"
  environment:
    name: $MY_VAR

We now have a not-expanded result using the --merge options:
$ gitlab-ci-local --merge

---
stages:
  - .pre
  - build
  - test
  - deploy
  - .post
my_job:
  rules:
    - if: $CI_COMMIT_TAG
      variables:
        MY_VAR: my global var from .anchored_vars
    - when: never
  script:
    - echo $MY_VAR
  environment:
    name: $MY_VAR

Which is valid passing it to the ci/lint GitLab API validator:
$ jq --null-input --arg yaml "$(gitlab-ci-local --merge)" '.content=$yaml' \
  | curl "https://scm.clubmed.com/api/v4/projects/${GITLAB_PROJECT_ID}/ci/lint?include_merged_yaml=true" \
    --header "Authorization: Bearer ${GITLAB_TOKEN}" \
    --header 'Content-Type: application/json' \
    --data @- \
  | jq

{
  "valid": true,
  "errors": [],
  "warnings": [],
  "merged_yaml": "---\nstages:\n- \".pre\"\n- build\n- test\n- deploy\n- \".post\"\nmy_job:\n  rules:\n  - if: \"$CI_COMMIT_TAG\"\n    variables:\n      MY_VAR: my global var from .anchored_vars\n  - when: never\n  script:\n  - echo $MY_VAR\n  environment:\n    name: \"$MY_VAR\"\n",
  "includes": []
}

NOTES

I'm a newbie in TS dev so do not hesitate for any change request ;)
The name of the option can really be challenged as I'm not sure about "merge", which is actually what it does but has a special meaning in a VCS environment...
Running the test suite to validate my dev I faced a single failure but I don't think it was due to it (please let me know):

Summary of all failing tests
 FAIL  tests/test-cases/artifacts-shell/integration.artifacts-shell.test.ts (6.889 s)
  ● artifacts-shell <consume> --needs --shell-isolation

    expect(received).toEqual(expected) // deep equality

    Expected: 1
    Received: 2

      23 |         return l.match(regex) !== null;
      24 |     });
    > 25 |     expect(found.length).toEqual(1);
         |                          ^
      26 |     expect(await fs.pathExists("tests/test-cases/artifacts-shell/path/file1")).toEqual(true);
      27 | });
      28 |

      at tests/test-cases/artifacts-shell/integration.artifacts-shell.test.ts:25:26

Test Suites: 1 failed, 109 passed, 110 total
Tests:       1 failed, 260 passed, 261 total
Snapshots:   0 total
Time:        55.212 s, estimated 60 s

@firecow I'm working for a french tourism company (https://github.com/ClubMediterranee) and this feature and my two other fixes (#1138 and #1140 ) are quite urgent for us as we would love to use your tool in our internal CI development validation (we have a CE Gitlab factory and are on the way to increase our developments robustness).
I'm totally open for discussion, help, question etc.
And many thanks for your great work ! The gitlab-ci-local tool is just amazing ;)

other fixes (https://github.com/firecow/gitlab-ci-local/pull/1138 and https://github.com/firecow/gitlab-ci-local/pull/1140 ) are quite urgent for us as we would love to use your tool in our internal CI development validation

by the way, to speed things up,
while waiting for the review and for firecow to publish a new release,
you can create the executable from your forked branch
npm run build

Depending on your os:
# npm run --help
  pkg-linux
    pkg src/index.js --public -t node18-linux-x64 -o bin/linux/gitlab-ci-local && chmod +x bin/linux/gitlab-ci-local && gzip -c bin/linux/gitlab-ci-local > bin/linux.gz
  pkg-macos
    pkg src/index.js --public -t node18-macos-x64 -o bin/macos/gitlab-ci-local && gzip -c bin/macos/gitlab-ci-local > bin/macos.gz
  pkg-win
    pkg src/index.js --public -t node18-win-x64 -o bin/win/gitlab-ci-local && gzip -c bin/win/gitlab-ci-local.exe > bin/win.gz

npm run <pkg-linux/pkg-macos/pkg-win>

Hello @ANGkeith , thank you for the advice, that's exactly what I did ;)

@e-picas Let's just fix how --preview is working instead of adding a new option
I find it perfectly reasonable that --preview output should be able to pass https://gitlab.com/ci/lint, and I'm sure the other collaborators do too.

@e-picas I'll be publishing a new version tomorrow :+1:


How to setup the environment

Hi, I am stuck at the last phase of the setup.
I git cloned Molli to the directory, ran pip install ., but received an error at python -c "import somn", where it says
Traceback (most recent call last):   File "<string>", line 1, in <module>   File "/home/yyf/Lucid_Somnambulist/Lucid_Somnambulist/somn/__init__.py", line 1, in <module>     from . import build   File "/home/yyf/Lucid_Somnambulist/Lucid_Somnambulist/somn/build/__init__.py", line 10, in <module>     from .parsing import InputParser, cleanup_handles   File "/home/yyf/Lucid_Somnambulist/Lucid_Somnambulist/somn/build/parsing.py", line 1, in <module>     import molli as ml ModuleNotFoundError: No module named 'molli'
I believe it has something to do with the location the Molli package, which brings my question:
In Readme, it says "Finally, download and install this package by running the following command in the top-level directory (/path/to/package/Lucid_Somnambulist/):", I assume this actually means Lucid_Somnambulist/Lucid_Somnambulist , rather than the so called top Lucid_Somnambulist/, as the there is no env config file for the setup. So I installed Molli under Lucid_Somnambulist/Lucid_Somnambulist, followed the procedure mentioned above and got stuck there.
Any suggestion?
@ianrinehart @nhadler

Fixed, a few notes:

molli should not be installed at top top directory, but rather Lucid_Somnambulist/Lucid_Somnambulist
to install molli smoothly, run pip install git+https://github.com/SEDenmarkLab/molli_firstgen.git#egg=molli with reference to the top answer in the post

it's probably worth it to update the readme for clarification.
i will close this issue myself, but yet I'm still lost at where to actually start testing around....


POI example crash the app with error: Unsupported object <CPPointOfInterestTemplate:....

Hi,
I am trying to use POI template same as in examples but app crash:
import {
    CarPlay, PointOfInterestTemplate
} from 'react-native-carplay';

const template = new PointOfInterestTemplate({
            title: 'Example',
            items: [
              {
                id: 'test',
                location: { latitude: 64.011, longitude: -21.66 },
                title: 'Testing',
                subtitle: 'foobar',
              },
            ],
          });
CarPlay.setRootTemplate(template, false);

But error that I get is:

If this is not possible I tried something else:

CarMapView() {
        return (
          <View style={{ flex: 1}} >
            <MapView
                style={{flex: 1}}
                provider={PROVIDER_GOOGLE}
                showsUserLocation={true}
                followsUserLocation={true}>
                <View style={{backgroundColor: '#f00', padding: 30, marginLeft: 50}}>
                    <TouchableOpacity style={{backgroundColor: '#0ff'}} onPress={() => {CarPlay.pushTemplate(someOtherTemplate);}}>
                        <Text>ITEM A</Text>
                    </TouchableOpacity>
                </View>
            </MapView>
          </View>
        );
      }

const mapTemplate = new MapTemplate({
            component: this.CarMapView
          });

CarPlay.setRootTemplate(mapTemplate, false);

but this doesn't catch touch event on button over map. Any idea?

Regrettably this hasn't been implemented yet. In the code you can see.
# pragma PointOfInterest
-(void)pointOfInterestTemplate:(CPPointOfInterestTemplate *)pointOfInterestTemplate didChangeMapRegion:(MKCoordinateRegion)region {
    // noop
}

Supported in 2.3.0

issue is still there in 2.3.0, The method is in there, but the error still remains. The supported events needed "didChangeMapRegion" at the top of the class, but something else is going on that does not allow you to push the POI template to the screen.
The CPPointOfInterestTemplate is not in the list of Allowed classes . Tried present as well and it is also not supported, is there another way to push this template onto the stack?

Yeah we need a MKCoordinateRegion to json converter so we can support the event.
The original issue is resolved however, please create a new issue for missing events (or PR)

@vvusts Are you able use PointOfInterestTemplate now? Still I'm facing same issue. App getting crash (Closed unfortunately). Can you someone help me how to use this template?
Adding code sample below:
const template = new PointOfInterestTemplate({
            title: 'Example',
            items: [
                {
                    id: 'test',
                    location: { latitude: 64.011, longitude: -21.66 },
                    title: 'Testing',
                    subtitle: 'foobar',
                },
            ],
        });
        CarPlay.pushTemplate(template, false);

@birkir  Still facing the same issue , any fix

@vvusts  hey are u able to get the POI work ?


Not pinned or pinned indirectly

After retrieving the local pins (via ListAsync), trying to iterate through and remove (via Pin.RemoveAsync), I'm receiving an error message
"not pinned or pinned indirectly"
Doing a tad bit of research, there seemed to be some conversation about how to handle pinning with regard to direct, recursive, and indirect pins...I trust that this library handles iterative removal? Am I approaching this the wrong way?

This library, net-ipfs-core only defines the PIN API, it does not implement it.
I assume you are using net-ipfs-http-client; which is just a wrapper around the API and calls the IPFS daemon.  Which daemon are you using (go or js and version number)?
Can you show the code you are using and the stack trace.

I'm using the Go client. IPFS Desktop more directly.
IEnumerable<Cid> localPackages = await ListLocalPackagesAsync();
            foreach(Cid c in localPackages)
            {
                Trace.WriteLine("Removing "+c+" from local storage");
                try
                {
                    await ipfs.Pin.RemoveAsync(c);
                }catch(HttpRequestException ex)
                {
                    Trace.WriteLine("Unable to remove "+c+" from local storage");
                    Trace.WriteLine(ex.Message);
                }
            }
            IEnumerable<Cid> remainingPackages = await ipfs.Pin.ListAsync();

So, it looks like calling ListAsync, returns indirect pins first, which throw the error message, then once it reaches the direct pins is able to delete them, which then cascades to the indirect pins. Removing them "solving" the issue.
And yes, you're correct, ipfs-core is different than ipfs-http-client, I apologize for posting in the wrong project.

What is the code ListLocalPackagesAsync about?  Why not just use ipfs.Pin.ListAsync?


Feature Request: Start logging and saving immediately.

I can't use this app for the one use case that I need- to inspect the logs after a system crash. The reason is because to save you need to press Start and then Stop again. When the OS crashes I need to inspect the log and find out why but I can't because there is no opportunity to save the log after a crash.
I request you allow the user to press start logging and this will save the log results to a file right away.

Yes. This definitely is a concern.


improve a11y of buttons

Either make the buttons that only show upon hover also show upon focus on the template-instance-container containers (add tabindex="0" to the HTML?),
or make all buttons visible when tabbing is detected.

can't work on for now, and not required for specific use case i had in mind


Release Notes notification should have an option to "Do not show again"

Steps to Reproduce:

Update ADS

Expected: Release notes pop-up should have option to not show again

Closing older issues.  I don't think we show this pop-up anymore.


Notification if site breaks

This is just a little brainstorming:
At work I run scripts which take several hours or days. To get notified if these tasks finishes, I installed a telegram bot. This bot sends me a message with the results, if the tasks finished correctly or a error message if something goes wrong.
I was thinking of something similar for our site. If the site breaks or some error occures, we should get a fast notification. I found a way to add a discord webhook, which could send us such notifications:
https://10mohi6.medium.com/super-easy-python-discord-notifications-api-and-webhook-9c2d85ffced9

Discord Webhook works great!


PowerShell@2: filePath uses \ or /?

I was composing a new PowerShell@2 pipeline task and looked up if I should specify the filePath with forward slashes / or with backslashes \ as directory separators. The docs contain no instructions on this. There is one example that uses backslashes.
filePath: $(System.DefaultWorkingDirectory)\test2.ps1

So I assumed I should use backslashes. But, doing this gave me a pipeline error:

##[error]ENOENT: no such file or directory, stat '/azp/agent/_work/55/s/vm-customscripts-demo\pipeline-scripts\upload.ps1'

Changing the backslashes to forward slashes fixed the error for my environment.
I believe that the path separator for the filePath is platform-dependent: / for linux/macos pipeline agents, \ for windows agents.
Could you clarify this in the docs?

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

ID: 66590238-4299-1c66-31fe-ce6912c44968
Version Independent ID: 66590238-4299-1c66-31fe-ce6912c44968
Content: PowerShell@2 - PowerShell v2 task
Content Source: task-reference/powershell-v2.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


Complete the Note Change in source/schedules/schedules.rst

Task completed can be viewed on the Trello page.
Builds have been successful in my development branch using Travis-CI.

Old pull request, creating new one with better implementations


Requirejs and CBOR-js

CBOR and window.CBOR is always undefined even though I loaded it before using it and trying to import it with import * as CBOR from 'cbor-js'.

So want I did was copy the lib and change it so it was typed and a module.


Add addf command

close #17
close #21
close #26

Codecov Report

Merging #96 (6963261) into master (464a653) will increase coverage by 0.53%.
The diff coverage is 77.61%.

@@             Coverage Diff              @@
##             master      #96      +/-   ##
============================================
+ Coverage     72.15%   72.69%   +0.53%
- Complexity      399      454      +55
============================================
  Files            70       80      +10
  Lines          1232     1366     +134
  Branches        125      138      +13
============================================
+ Hits            889      993     +104
- Misses          311      340      +29
- Partials         32       33       +1

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
71.42% <0.00%> (-3.58%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
18.75% <0.00%> (-1.25%)
:arrow_down:

...c/main/java/seedu/address/ui/FacilityListCard.java
0.00% <0.00%> (ø)

.../main/java/seedu/address/ui/FacilityListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
98.03% <75.00%> (-1.97%)
:arrow_down:

...edu/address/model/facility/UniqueFacilityList.java
85.71% <85.71%> (ø)

...in/java/seedu/address/model/facility/Facility.java
86.20% <86.20%> (ø)

...address/logic/parser/AddFacilityCommandParser.java
92.30% <92.30%> (ø)

... and 9 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 464a653...6963261. Read the comment docs.


Deprecated router not working with beta 7

Ionic 2
Plunker link http://plnkr.co/edit/O6SMLbaH0iKujoA99pM2?p=preview
I also tried with latest router too. But its not working.

https://github.com/driftyco/ionic/blob/2.0/CHANGELOG.md#router
The Router component was disabled on beta 7.

@paulogr  I was trying with Deprecated one router-deprecated

Hey @dheran ! Sorry, but neither router is going to work as expected for now. With all the changes going on with the router on the Angular 2 side of things, we have decided to hold off on really implementing Angular router implementation until it gets more stable. While we would love to give our users a working implementation now, we have decided to hold off as it would most likely be pretty buggy. Were always shooting for bleeding edge, until its really broken (:  Thanks for using Ionic!


asset-compute worker test file structure is missing required folders

Expected Behaviour
When creating a project with Asset Compute Workers, the expected test file structure is this:
project/
  test/asset-compute/
        worker-1/
             <test-case>/
        worker-2/
             <test-case>/

(see example here: https://github.com/adobe/asset-compute-example-workers/tree/master/projects/worker-azure-ai/test/asset-compute)
The test files need to be attached to workers since they only apply to specific workers, not all the workers in the project. Also, our aio-cli-plugin-asset-compute expects this specific file structure to be able to find the tests.
Actual Behaviour
The new file structure is not the same, the test cases are not nested in the correct folders: asset-compute/<worker-name> which means the tests are not connected to specific workers. Also, the test and actions files are nested under src/dx-asset-compute-worker-1 instead of being in the root of the project.

So when you run aio app test, our plugin cannot find the test files:
[jamiedelbick:...gration-tests/build/project]$ aio app test
Using local project's @adobe/aio-cli-plugin-asset-compute version 1.5.0dx/asset-compute/worker/1/* (hook) - running tests...
No worker tests found in 'test/asset-compute/*'app:test results:
    √ dx/asset-compute/worker/1/* (hook)

Reproduce Scenario (including but not limited to)
Steps to Reproduce

install latests version of aio-cli (v8) and dependents
aio app init
(follow steps to choose asset compute worker)
run aio app test

Platform and Version
Using local project's @adobe/aio-cli-plugin-asset-compute version 1.5.0

  System:
    OS: macOS 11.2.3
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 20.04 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.20.1 - ~/.nvm/versions/node/v12.20.1/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 6.14.10 - ~/.nvm/versions/node/v12.20.1/bin/npm
  Virtualization:
    Docker: 20.10.7 - /usr/local/bin/docker
  npmGlobalPackages:
    @adobe/aio-cli: 8.0.0

  CLI plugins:
    core:
      @adobe/aio-cli 8.0.0
      @adobe/aio-cli-plugin-app 8.0.0
      @adobe/aio-cli-plugin-auth 2.4.1
      @adobe/aio-cli-plugin-config 2.3.0
      @adobe/aio-cli-plugin-console 3.4.0
      @oclif/plugin-autocomplete 0.3.0
      @oclif/plugin-help 2.2.3
      @oclif/plugin-not-found 1.2.4
      @oclif/plugin-plugins 1.10.1
      @oclif/plugin-warn-if-update-available 1.7.0
    user:
      @adobe/aio-cli-plugin-asset-compute 1.5.0
      @adobe/aio-cli-plugin-certificate 0.3.0 (*)
      @adobe/aio-cli-plugin-events 1.1.5 (*)
      @adobe/aio-cli-plugin-info 2.0.0 (*)
      @adobe/aio-cli-plugin-runtime 4.0.0 (*)
    link:

Sample Code that illustrates the problem
There are two issues:

asset compute worker tests cases should be nested in two crutial folders: test/asset-compute/<worker-name>
I believe this change was brought in through this PR: https://github.com/adobe/generator-aio-app/pull/148/files#diff-edddb1be7fc41fd5d024cfea1b932ed6cc4fc9344bf0bc66de503c0f0a7a8d55L80-L88

new aio file structure that makes the test folder nested inside src/dx-asset-compute-worker-1.

This is a critical bug because aio app test is broken for all Asset Compute Workers created with the most recent version of the aio cli

Verified that the tests don't have the prefix asset-compute/[worker-name] and will be putting out new unit tests and a patch (will update this issue with a link).
Since everything is an extension now however, all tests will be under the extension itself, and not the root.

I tested my fix by modifying this in the asset-compute plugin: const TEST_DIR = path.join("src", "dx-asset-compute-worker-1", "test", "asset-compute");
❯ aio app test
dx/asset-compute/worker/1/* (hook) - running tests...
Actions:
- worker

Running tests in src/dx-asset-compute-worker-1/test/asset-compute/worker

Error: ENOENT: no such file or directory, open 'manifest.yml'

app:test results:
    × dx/asset-compute/worker/1/* (hook)

For a new project, we don't generate a manifest.yml anymore so there may be further changes needed for your tests.
Each extension has its own ext.config.yaml in the root of the extension folder, this contains the extension's runtime manifest.

PR https://github.com/adobe/generator-aio-app/pull/158

@jdelbick this is actually deeper than only test files. @shazron posted a PR to fix the test asset-compute subfolder hieararchy, but this will not be enough, as the actually test folder or action folder could be anywhere, there is no guarantee anymore that those folder are located in the root.  This means all commands in the asset-compute plugin need to be updated to support latest changes.
The app structure is more flexible now, multiple extensions can be defined in the same root folder, and users can move configuration and code around. This means that asset-compute, just like aio app needs to support:

migrated legacy NUI apps
apps with NUI extensions generated in the src/dx-asset-compute-1/ subfolder after a fresh aio app init
apps with NUI extensions moved to another folder post init, as said users reorganize all their code and configuration, there is no guarantee that the NUI code will be stored in the root or even under src/dx-asset-compute-1/, it can be anywhere!

Thankfully, we implemented a config loader that abstracts those corner cases away and exposes the needed paths and configuration bits https://github.com/adobe/aio-cli-plugin-app/blob/master/src/lib/config-loader.js
Here is an example on how asset-compute commands could use the config-loader:
const loadConfig = require('@adobe/aio-cli-plugin-app/src/lib/config-loader')

const fullConfig = loadConfig()
const nuiConfig = fullConfig.all['dx/asset-compute/worker/1']

if (!nuiConfig) {
  throw new Error('this is not an asset compute extension')
}

const runtimeManifest = nuiConfig.manifest.full
const testsRoot = nuiConfig.tests.unit
const actionsRoot = nuiConfig.actions.src
const actionsDistRoot = nuiConfig.actions.dist
<and many more see https://github.com/adobe/aio-cli-plugin-app/blob/master/src/lib/config-loader.js#L425>

To make things easier for you, we can move the config-loader to a separate aio-lib-app module, so that you don't have to depend on the full app plugin.
cc @meryllblanchet @sarahxxu @purplecabbage

@moritzraho  @shazron Thank you so much! This is all very helpful.

To make things easier for you, we can move the config-loader to a separate aio-lib-app module, so that you don't have to depend on the full app plugin.

That would be helpful since we don't currently have aio-cli-plugin-app as a dependency in our plugin: https://github.com/adobe/aio-cli-plugin-asset-compute/blob/master/package.json#L8-L25 (we only have it as a dev dependency)

@moritzraho is it still guaranteed that the .env file will be in the root?

@moritzraho is it still guaranteed that the .env file will be in the root of the project?

For now yes 🙂

another question, @moritzraho is this extension path dx/asset-compute/worker/1 expected to change?
const nuiConfig = fullConfig.all['dx/asset-compute/worker/1']

This path has to be hardcoded in our plugin so if this changes, we won't be able to find the location of nui config


docs: Use flutter pub add to add flame dependency

Description

Use flutter pub add instead of munging the pubspec.yaml file for adding flame as a dependency

Checklist

[X] I have followed the Contributor Guide when preparing my PR.
[X] I have updated/added tests for ALL new/updated/fixed functionality.
[X] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[X] I have updated/added relevant examples in examples or docs.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[X] No, this PR is not a breaking change.

Related Issues

Replace or remove this text.

Lgtm! I don't understand why it complains that consts isn't needed in the dictionary anymore, but I guess you can just remove it so that it becomes green.

Done!


Change my own phone number

cc @tkellen
WAIT
I solved everything I told below, now I'm only finishing the PR to merge it.
I was missing an Ampersand parameter which was a { patch: true }, so it sent a patch with only the changes I've mad.

Old message:
I'm getting an error when I try to save.
Ampersand sends a PUT request to the API, which returns a 404 with the following response message:
Cannot PUT /v3/employees/32

It tries to connect to http://api.loc/v3/employees/32 (my own profile) which works if I browse directly.
I deployed this branch to the staging server, but you can see the Ajax results below.
Here is the RAW request header:
PUT /v3/employees/32 HTTP/1.1
Host: api.loc
Connection: keep-alive
Content-Length: 445
accept: application/vnd.api+json
Origin: http://new-people.loc
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.86 Safari/537.36
authorization: Bearer a5df5c8d54f5afbd5857ac9d0f55cd507b7b65a7
content-type: application/vnd.api+json
Referer: http://new-people.loc/profile/leo-balter
Accept-Encoding: gzip, deflate, sdch
Accept-Language: en-US,en;q=0.8,fr;q=0.6,pt;q=0.4

Here is the request payload (you might want to parse this json), you can see a different value on the phone number:
{"data":{"id":"32","type":"employees","attributes":{"date_end":null,"date_start":"2014-10-01T00:00:00.000Z","email":"leo@bocoup.com","email_personal":"leonardo.balter@gmail.com","first":"Leo","github_user":"leobalter","irc":"leobalter","last":"Balter","notes":"In case of emergency: call my cell.","phone":"1234 1234","slug":"leo-balter","twitter":"leobalter","website":null},"relationships":{"position":{"data":{"id":"1","type":"positions"}}}}}

And the response header:
HTTP/1.1 404 Not Found
Server: nginx/1.4.6 (Ubuntu)
Date: Thu, 19 Nov 2015 23:41:54 GMT
Content-Type: text/html; charset=utf-8
Transfer-Encoding: chunked
Connection: keep-alive
Vary: Accept-Encoding
X-Powered-By: Express
Access-Control-Allow-Origin: *
Vary: Origin
X-Content-Type-Options: nosniff
set-cookie: connect.sid=s%3ABXYGwxKh6osOEezcOAWZhu7BJbsZrplg.ReD2FIxhw1II0BIUmA8oV9f2ax%2FAsVixkuSdNHxJMTA; Path=/; HttpOnly
Content-Encoding: gzip

Sweeeeeeeeeet.
Saving works, but it doesn't reflect on the screen when you hit save (you have to reload).  Also, there should be an option to cancel the edit if you decide you don't want to save.
Bummed I'm gonna be out tomorrow. What do ya say we launch this for real consumption on Monday after our post-mortem?

Saving works, but it doesn't reflect on the screen when you hit save (you have to reload).

That's what I'm working on right now :)

there should be an option to cancel the edit

Sure, right after I solve the first problem

this for real consumption on Monday after our post-mortem?

I'm game for that. The style need some love but it's not that bad.

DONE!
It's working! WOT!
Updates value after save and cancel button is there as well.

Deploying to the staging server.

Other than my comments, :+1:

@bobholt I did some small improvements on e7de9a7 limited to the some propTypes that can't be set required.

great @leobalter!


Exclude temporary files from reloader.

Fixes #1333.

This looks correct, but I wish I could confirm it. I don't use Windows or win32com.

I'm also worried that this will break the reloader if the virtualenv was created in the temp directory.

That's true. I guess the best thing to do is to ask some people to try it?
I think that these issues are pretty specific to individual machines/setups (there's no general trend) so it might be best to have an environment variable which is a list of glob patterns for directories to exclude so that it can be optionally enabled if people start getting problems.

Yes, I think I might want to rework this to allow passing in a list of excluded paths, since it's already possible to pass a list of extra paths. I think there's a request for that somewhere.

I'm happy to alter the PR to do that.

Could have sworn there were issues in Werkzeug or Flask about it, but couldn't find any. Let's create a new PR for passing a list to exclude.

Ok – should I create an issue first?

It can still reference the one linked here, I'll add a comment about the solution I'm thinking of.

Sounds good!


Add sybase connector

@academy-codex gentle ping

Hey @tooptoop4. I am using sajdbc.  See my fork today.

where is that fork @academy-codex ?

@tooptoop4 You can see the PR I have made.

any update on this issue?

Any updates on this issue @academy-codex @tooptoop4 ?
This connector will be of great help.

https://github.com/prestosql/presto/pull/3105 is better than this


fix: change appsmith alias

Description

[!TIP]
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.

Fixes #Issue Number
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10202292996
Commit: 28cf925d879fc3abcf007f0f1f4beda85a39d2d6
Cypress dashboard.
Tags: @tag.All
Spec:
Thu, 01 Aug 2024 17:35:23 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

@KelvinOm why is there a need for this change? Also, you haven't updated the eslintrc file in this PR.

@KelvinOm why is there a need for this change? Also, you haven't updated the eslintrc file in this PR.

@ankitakinger Fixed the lint rule.
You can find the full context of this decision here.


Error setup

After run:
docker run -v /path/to/filebeat.yml:/filebeat.yml prima/filebeat:1
I got this:
Loading config file error: Failed to read /filebeat.yml: read /filebeat.yml: is a directory. Exiting.

Sorry, It's my mistake :D


[Feat]: Create Issue forms and make the issue options more visually appealing

Currently, there is no issue template for this repo, and no issue form which will help users in contributing.

I will add Issue Forms to the respective issues to make it easier for the contributor to raise it.

Here is an example of how a Bug Issue Form will look like 👇

and issue templates:

I want to work on this issue. So, kindly assign this issue to me ( GSSOC'24 ).


Unable to delete Dashboards via kibana

If I try and delete a dashboard that I have created, I get a 403 forbidden from kibana.
Remote Address:xxx:xxx:xxx:xxx:80
Request URL:http://traildash.xxxxxx.com/es/kibana-int/dashboard/TESTDASH
Request Method:DELETE
Status Code:403 Forbidden
workaround if ES port is accessible is to manually "DELETE :9200/kibana-init/dashboard/TESTDASH"

Hi
Just in case you are missing this bit:
ElasticSearch proxy ensures your logs are secure and read-only
No need to open direct access to your ElasticSearch instance
Helps to achieve PCI and HIPAA compliance in the cloud

@cuotos , I have opened the following issue : "panic: runtime error: index out of range"
have you experienced anything like it?
I am running the script out of docker and I just get that. I am not sure why is that for..
Thanks!!

My mistake. I had read that but had forgotten.
@rainmanh, No I haven't seen that error.

This is working as intended - I think it is better to error towards compliance, even if that means deleting dashboards takes an extra step.


Question: is it optimized for within range search?

Can I make queries like Where(x=>x.a>=9 && x.a<=10 && x.b>=5 && x.b<=6 && x.c>=3 && x.c<=12)? Will it use ordered indexes? Binary search?

Hi @AqlaSolutions - The primary goal of this library is to serve as the underlying engine of the Effort testing library. These kind of optimizations are not included in the query optimizer.


Illegal Access Error when clicking the Locked modal prompt (riddle not completed) on OS-X

Clicking on the lock modal prompt (riddle not completed) in a Story/Activity shows Illegal Access Error on MacOS-X (can click OK to continue using the app)

Fixed in version 0.7.4, had to postpone freeing the modal prompt after closing it, issuing  a message to the main thread message processing queue to do the freeing there


Add ci skip suffix to PR commits

There is no scenario of which I can think that one would want a CI run for the PR containing the upcoming changelog.
Instead, it often triggers other pipelines that run on the pull_request trigger without any need.

Good catch, though I'd say that this applies only to niche cases when non-admins are in charge for releases.
In almost all other cases, admins do releases and can still merge without required status checks to pass.
Seeing this in contrast to the amount of unneeded CI runs saved, I'd argue that this should be added and set as the default.

This would break CIs running on the changelog and changes from before / after hooks.

This would break CIs running on the changelog and changes from before / after hooks.

Why do we need hooks to run on pull_request?
I am not sure that my change proposal is fully understood: everything stays the same (and will still run on pushes to main, etc.), I just don't think we need any runs for existing PRs and their respective commits merging main, see e.g. https://ci.woodpecker-ci.org/repos/3780/pipeline/21066/1.
The release helper is not even triggered in this CI run.

Why do we need hooks to run on pull_request?

We alter code / files in the release PRs and that should be check. Take the helm repo as an example: https://github.com/woodpecker-ci/helm/blob/main/release-config.ts#L6

Yes, but when running in main and not within an open PR?

The changed code / config / changelog is part of the PR commits and should be checkable by something like prettier etc.

Yes, if this is set up and configured. I don't think it should trigger builds by default though as most people don't have something like this set up.
Instead, it will spin up many check runs that run by default on PR events, wasting a lot of resources that could be saved by a conservative default setting.
Right now the default is to run all pull_request workflows for every update of the changelog text. If I can't convince you to set this to true by default, would it be okay to add this option at all?


A Beautiful Restful Request and Respond Format - Lingjun Meng's Blog

http://www.menglingjun.com/2014/11/08/A-Beautiful-Restful-Request-and-Respond-Format/
Every failure is leading towards success.

test


Sorting broken for annotated tags

Hi, we are using this project to version a project with annotated tags, and the latest update broke the versioning. The recent change to sort by date for #22 changed the tags sorting to consider the committerdate of the tag. This field does not exist for annotated tags, as they are a separate object from the commit. Based on this thread https://stackoverflow.com/a/34919313 I would suggest changing to creatordate, which works for both annotated and lightweight tags. The previous sort condition version:refname also worked fine for us, so maybe consider having the sorting configurable?
Thanks

Hi.
Please check the v1.6.1 release.

Awesome, it works now, thank you!

I've also added a way to pass sorting column through the config dict.


drivers/spi: Fixed incorrect prompt.

The prompt for SPI1 IRQ priority stated SPI0 incorrectly.
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #7175   +/-   ##
=======================================
  Coverage   58.54%   58.54%
=======================================
  Files         464      464
  Lines       47400    47400
  Branches     8783     8783
=======================================
  Hits        27749    27749
  Misses      15820    15820
  Partials     3831     3831

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 f5f95ee...2b9574a. Read the comment docs.


poll: should drop IE8 support?

Should cssobj drop old IE support? (IE<=8), mainly the @media hook
That's only 2 benefit I can see to remove old IE @media hook:

Smaller size ( before 4.2K min.gz, after  3.9K min.gz

More maintainable code, the CSSOM plugin will be smaller, and more clearly.

The question:

do you use this feature now?

targeting IE>8, is that OK for you?

IE8 is not even officially supported anymore. Guess no one is supporting IE8 today.:P You can even drop 9+10 after April 17th~ 2017, as support runs out for those as well.

@rofrischmann Thank you for the reply. If you live in China, the question is really a question... 1/3 of the computer here still running Windows XP...
Alibaba Inc., Chinese biggest e-biz website, still support IE8 till this year.
If I can make some perf improvement, due to the dropping of IE8, I'm definitely drop it. But now I can see it's not a perf problem, but some bundle size.

The fact Microsoft no longer supports a browser version doesn't mean it is no longer used... IE below 8 has to be dropped, it is not usable on the modern Internet anyway. IE 8 has still some world usage (3 % IIRC), so supporting it is still useful, at least for sites targeting a large user base.
Honestly, I am not sure that the size reduction is worth it; ease of maintenance is important, but since the code is already there...
Perhaps next year it will be worth asking again... 😄

Version 1.x will still keep the @media IE8 hook, IE8 still in the support list.
Any update of IE 8 support, will updated here in the future.


check that asset.map is a function before call it

Some assets do not have .map() method.
Here https://github.com/kevlened/copy-webpack-plugin/blob/a59ba168a4d46bd5a1c82fddac6b6ee6452d7799/src/writeFile.js#L84-L91 plugin adds new assets that have only .source() and .size() methods. They cause errors when precessed with EncodingPlugin.

thanks for your contribution

published in version v0.2.1


Samsung Chromebook Plus - startxfce4 causes reboot

Please paste the output of the following command here: sudo edit-chroot -all
``name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20170315143304~master:95589555
release: xenial
architecture: arm64
xmethod: xorg
targets: x11,xfce,touch,keyboard,extension
host: version 9413.0.0 (Official Build) dev-channel kevin
kernel: Linux localhost 4.4.52-07972-gf27275748d2d #1 SMP PREEMPT Wed Mar 29 19:03:23 PDT 2017 aarch64 aarch64 aarch64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...`
Please describe your issue:
Previously working system reduced to black screen and reboot when X11 needed to run.
If known, describe the steps to reproduce the issue:
Previous to this Google update my chromebook plus was working with xenial and xfce. Following the update, issuing the command "sudo startxfce4" causes a non-recoverable black screen proceeded about 30 seconds later by a reboot. I have tried power washing, re-installing xenial, as well as using different distros and targets - trusty, precise and zesty: same result. "startcli" works suggesting X11 problem. I see other issues with X11 but no one has reported a reboot.

me too

The last chromeOS update also broke X11 for me.
Using the HP Chromebook 14, the arm64 one.
Here is the error log on a fresh chroot (trusty, x11 target only, i3 installed and xinit).
trusty_crouton_x11_log.txt
Something changed in the way chromeOS handles the screen and the chroots don't detect it anymore. I will test the xiwi target later.

I am running the same command on my samsung with intel, and having the same problem: CLI: sudo startxfce4 ;;; Then It tries to book, goes to black. SHIFT + ALT + FWD does nothing: which key is FWD, or BACK...?

I am still quite new at this so...if I am doing something wrong let me know, There is always another path.

chronos@localhost ~/Downloads $ sudo enter-chroot startxfce4
Entering /mnt/stateful_partition/crouton/chroots/precise...
Failed to start message bus: Failed to open "/etc/selinux/targeted/contexts/dbus_contexts": No such file or directory
WARNING: starting chroot system dbus daemon failed with code 1
/usr/bin/startxfce4: Starting X server
_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.
X.Org X Server 1.15.1
Release Date: 2014-04-13
X Protocol Version 11, Revision 0
Build Operating System: Linux 2.6.42-75-generic x86_64 Ubuntu
Current Operating System: Linux localhost 3.18.0-13753-gc65690b #1 SMP PREEMPT Tue Mar 28 23:37:52 PDT 2017 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 3584000 verity payload=PARTUUID=f37d1afa-50b8-c048-8794-67cbdb7bcd9b/PARTNROFF=1 hashtree=PARTUUID=f37d1afa-50b8-c048-8794-67cbdb7bcd9b/PARTNROFF=1 hashstart=3584000 alg=sha1 root_hexdigest=547b0c6304ac0f9e49ec5ff930b2e27351608671 salt=a89149d641b7f2698f2e1f6b88c7f1d533d72b9ce93dde674368aec128ac7c92" noinitrd vt.global_cursor_default=0 kern_guid=f37d1afa-50b8-c048-8794-67cbdb7bcd9b add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic
Build Date: 12 February 2015  03:37:52PM
xorg-server 2:1.15.1-0ubuntu2~precise5 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.30.2
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Wed Apr 12 22:48:42 2017
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
setversion 1.4 failed: Permission denied
Initializing built-in extension Generic Event Extension
Initializing built-in extension SHAPE
Initializing built-in extension MIT-SHM
Initializing built-in extension XInputExtension
Initializing built-in extension XTEST
Initializing built-in extension BIG-REQUESTS
Initializing built-in extension SYNC
Initializing built-in extension XKEYBOARD
Initializing built-in extension XC-MISC
Initializing built-in extension SECURITY
Initializing built-in extension XINERAMA
Initializing built-in extension XFIXES
Initializing built-in extension RENDER
Initializing built-in extension RANDR
Initializing built-in extension COMPOSITE
Initializing built-in extension DAMAGE
Initializing built-in extension MIT-SCREEN-SAVER
Initializing built-in extension DOUBLE-BUFFER
Initializing built-in extension RECORD
Initializing built-in extension DPMS
Initializing built-in extension X-Resource
Initializing built-in extension XVideo
Initializing built-in extension XVideo-MotionCompensation
Initializing built-in extension XFree86-VidModeExtension
Initializing built-in extension XFree86-DGA
Initializing built-in extension XFree86-DRI
Initializing built-in extension DRI2
Loading extension GLX
xf86EnableIOPorts: failed to set IOPL for I/O (Operation not permitted)
[intel_init_bufmgr:1069] Error initializing buffer manager.
setversion 1.4 failed: Permission denied
crouton: version 1-20161129162558~master:5f2f11f9
release: precise
architecture: amd64
xmethod: xorg
targets: xfce
host: version 9202.60.0 (Official Build) stable-channel celes
kernel: Linux localhost 3.18.0-13753-gc65690b #1 SMP PREEMPT Tue Mar 28 23:37:52 PDT 2017 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Failed to start message bus: Failed to open "/etc/selinux/targeted/contexts/dbus_contexts": No such file or directory
EOF in dbus-launch reading address from bus daemon
/etc/xdg/xfce4/xinitrc: 1: /etc/xdg/xfce4/xinitrc: ssh-agent: not found
Loading extension GLX
xf86EnableIOPorts: failed to set IOPL for I/O (Operation not permitted)
[intel_init_bufmgr:1069] Error initializing buffer manager.
setversion 1.4 failed: Permission denied
crouton: version 1-20161129162558~master:5f2f11f9
release: precise
architecture: amd64
xmethod: xorg
targets: xfce
host: version 9202.60.0 (Official Build) stable-channel celes
kernel: Linux localhost 3.18.0-13753-gc65690b #1 SMP PREEMPT Tue Mar 28 23:37:52 PDT 2017 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Failed to start message bus: Failed to open "/etc/selinux/targeted/contexts/dbus_contexts": No such file or directory
EOF in dbus-launch reading address from bus daemon
/etc/xdg/xfce4/xinitrc: 1: /etc/xdg/xfce4/xinitrc: ssh-agent: not found^C/usr/bin/xinit: connection to X server lost
waiting for X server to shut down xscreensaver: 22:50:14: SIGHUP received: restarting...
xscreensaver: 22:50:14: running as coderhyder/coderhyder (1000/1000)
Hangup
Hangup
Running exit commands...
(EE) Server terminated successfully (0). Closing log file.
xscreensaver: 22:50:14: Can't open display: :1
xscreensaver: 22:50:14: running as coderhyder/coderhyder (1000/1000)
xscreensaver: 22:50:14: Errors at startup are usually authorization problems.
But you're not logging in as root (good!) so something
else must be wrong.  Did you read the manual and the FAQ?
          http://www.jwz.org/xscreensaver/faq.html
          http://www.jwz.org/xscreensaver/man.html

/usr/bin/xinit: unexpected signal 2
Unmounting /mnt/stateful_partition/crouton/chroots/precise...
chronos@localhost ~/Downloads $ ^C
chronos@localhost ~/Downloads $

According to your screen grab you typed 'sudo enter-chroot startxfce4', not 'sudo startxfce4' at the shell prompt. I'm not sure if this is the same problem.

me too,  my old chroot, backups and a brand new one all halt and restart when x starts.
heres my facts.  chrome is stable nyan blaze  57.0.2987.146.  crouton was updated. all my chroots are trusty and xfce.  computer is an hp14?  its got a nvidia tegra chip. arm

"Fixed".
I was on dev channel.
Change to stable channel. Go into developer mode.
Re-installed everything. Now working. Welcome back Ubuntu!

I have this same issue and changing to stable channel didn't fix anything. Anybody got any other suggestions?

This shouldnt be closed as other people (including myself) are running into this same issue.

@battlemidget and others,
Please at least show us the output of: sudo edit-chroot -all
Then we'll know what releases and targets are involved.
Also, is this reboot happening only on the SCB+ and only with the xfce target?
-DennisL

I'm on a Samsung Chromebook Plus.  It's happening on xfce and lxde:
chronos@localhost / $ sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20170619181610~master:980f090f
release: xenial
architecture: arm64
xmethod: xorg
targets: lxde
host: version 9460.60.0 (Official Build) stable-channel kevin
kernel: Linux localhost 4.4.52-08254-ge4cd28825ca4 #1 SMP PREEMPT Mon Jun 5 16:59:00 PDT 2017 aarch64 aarch64 aarch64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Same problem with kde on CB+ on stable channel
sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20170619181610~master:980f090f
release: xenial
architecture: arm64
xmethod: xorg
targets: touch,kde
host: version 9460.73.0 (Official Build) stable-channel kevin
kernel: Linux localhost 4.4.52-08273-ge6c7193d61d7 #1 SMP PREEMPT Thu Jul 13 12:07:57 PDT 2017 aarch64 aarch64 aarch64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

@battlemidget, @crony1, @cd109876, and others,
This looks to be confined to the 'xorg' xmethod but, as an experiment, could one or more of you please add and test the 'xiwi' xmethod just to be sure?
Thanx,
-DennisL

Yes, xiwi works. I had to manually install xserver-xorg-core and xserver-xorg-video-dummy first or xiwi would fail to install.

@cd109876,
Thanx for trying 'xiwi' and reporting your results.
It's not a perfect solution but it may be a workaround until the issue can be fixed at least.
Unfortunately, I don't have a SCB+ so I can't troubleshoot it but maybe the devs or others can find a way.
Hope this helps,
-DennisL

Same issues as this thread.  The above comment helped immensely -- startxfce only works with xiwi, and xiwi required a manual install of the above two aforementioned packages.  Also on a Chromebook Plus.

I had the same problem when using the xorg target on my ARMv7 chromebook HP 14.
For me it works when using only target x11 (like in https://github.com/dnschneid/crouton/wiki/i3) but I can't change tty with ctrl+alt+f1. I hope it'll help you.

@rbutera Very new to this, how did you manually install the two packages?
Thanks.

I think I used the -f flag on apt-get.  I honestly cannot remember!


Refactor package version handling

Critical Changes
Changes
Issues Closed

We fixed an issue causing the github_release_notes task to fail on 2GP projects when Issues were closed in a Pull Request.

It'd be cool to get dunder methods added to support comparisons, e.g. .parse('<IP_ADDRESS>') > ...('<IP_ADDRESS>'), but doesn't need to be in this PR

I like this idea a lot - we could then potentially drop most of our uses of StrictVersion and LooseVersion, which also come with unexpected and subtle behavior differences.


Update grpc dependency to 1.25

@dazuma This PR has been fixed, PTAL. There was a showcase test that specified a Gapic error class that is removed in this update.


rkt: add a config helper command

Add a rkt config command that would print the actual configuration rkt run and other commands would use. The output would probably be JSON formatted like so:
{
  "stage0": [
    {
      "rktKind": "auth",
      "rktVersion": "v1",
      "domains": ["quay.io"],
      ...
    },
    {
      "rktKind": "auth",
      "rktVersion": "v1",
      domains: ["gcr.io"],
      ...
    },
    {
      "rktKind": "paths",
      "rktVersion": "v1",
      ...
    },
    ...
  ],
  "stage1": [
    {
      "rktKind": "network",
      "rktVersion": "v1",
      ...
    },
  ]
}

In my naive world-view I was naively assuming to be able to merely serialize the output of https://github.com/coreos/rkt/blob/master/rkt/rkt.go#L286 somehow.
But for i.e. rkt run I see quite a lot of subcommand dependent configuration: https://github.com/coreos/rkt/blob/master/rkt/run.go#L176-L237.
Does it make sense to have a rkt config run, rkt config run-prepared or is there any minimal common denominator for rkt config?
@krnowak @iaguis @alban @jonboulle can you give a blind man a bit of guidance?

@s-urbaniak What we would like here are the credentials of remote repositories. I think the serialization of https://github.com/coreos/rkt/blob/master/rkt/rkt.go#L286 would be enough. The other configuration you mentioned in rkt run are about specific images & pods and I don't think we need to serialize any of that in a rkt command.

The main purpose of the rkt config format is to print what is the effective configuration used by rkt run and others. This means seeing what is the result of all the overriding and whatnot. If it can help quayctl in getting credentials, then it's cool, but I really don't want to have a command that is usable only by happenstance to one application.
Also, serializing the Config struct is not acceptable - the config API is not frozen, we can change it in any way and at any time we want. I want the output of the rkt config command to be stable and the only way to ensure it is to print it in the similar format as described above - we are committed to the stability of our configuration formats, so let it serve us here too.

As a temporary compromise the initial work can actually focus only on printing the auth config, but in the long run I'd like to have all the configuration printed.

Got it, thanks @krnowak, btw by "serializing the Config struct" I did not mean to attach json tags to the original struct but introduce a serialize-only struct (current playground https://github.com/s-urbaniak/rkt/blob/config/rkt/config.go#L35-L49).
I'll adapt the my playground cfg struct to reflect your example given in the description and reiterate from there.

I was wondering if you could reuse authV1 and others for that.

I've been playing around with different output formats and implementations, my findings are being reflected in the following small subcommand spec document:
https://docs.google.com/document/d/1tOXU6PO-hSsdQOSq1U-ci7_cvxaH25-DvIfF2DZRvSU/edit#


feat(jest-transform): support transformers written in ESM

Summary
This allows people to write their transformers in ESM. This is mostly a refactor since import() is async, I've changed ScriptTransformer to load all transformers when it's created rather than on demand.
To not force all consumers to call load, I now expose more of a factory method which does the extra function call for the user.
This solves the use case behind #11081, but I think that issue might be useful for other async work.
~I'll need to update docs here, but I'll do that after landing docusaurus 2~ Just added a line, I think since we don't mess with versioned docs it should be fine

Test plan
E2E test added

Codecov Report

Merging #11163 (fc0f616) into master (3c679f1) will decrease coverage by 0.05%.
The diff coverage is 61.36%.

@@            Coverage Diff             @@
##           master   #11163      +/-   ##
==========================================
- Coverage   64.20%   64.14%   -0.06%
==========================================
  Files         307      307
  Lines       13364    13373       +9
  Branches     3257     3259       +2
==========================================
- Hits         8580     8578       -2
- Misses       4080     4089       +9
- Partials      704      706       +2

Impacted Files
Coverage Δ

packages/jest-core/src/runGlobalHook.ts
21.73% <0.00%> (ø)

packages/jest-reporters/src/CoverageWorker.ts
33.33% <ø> (ø)

packages/jest-runner/src/runTest.ts
1.98% <0.00%> (ø)

packages/jest-transform/src/ScriptTransformer.ts
63.63% <56.66%> (-2.74%)
:arrow_down:

packages/jest-core/src/TestScheduler.ts
68.18% <87.50%> (ø)

...ckages/jest-reporters/src/generateEmptyCoverage.ts
75.00% <100.00%> (+2.27%)
:arrow_up:

packages/jest-runtime/src/index.ts
55.59% <100.00%> (ø)

packages/expect/src/utils.ts
94.83% <0.00%> (-1.30%)
:arrow_down:

Continue to review full report at Codecov.

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

yeyyyyyyyyyyyy


Add node.js back to Quickstart

https://cockroachlabs.atlassian.net/browse/DOC-8192
Add node back to the Quickstart (using same language and code sample that was previously part of this page, not really new content)

cc @lnhsingh if you want to take a look


Tierra del Fuego, Argentina Parcels

urban parcels: http://catastro.aref.gob.ar/arcgis/rest/services/interna/TDF_catastro_ush/MapServer/8
rural parcels: http://catastro.aref.gob.ar/arcgis/rest/services/interna/TDF_catastro_ush/MapServer/9

Southern most addresses?

On Apr 25, 2017, at 07:15, Justin Meyers<EMAIL_ADDRESS>wrote:
urban parcels: http://catastro.aref.gob.ar/arcgis/rest/services/interna/TDF_catastro_ush/MapServer/8
rural parcels: http://catastro.aref.gob.ar/arcgis/rest/services/interna/TDF_catastro_ush/MapServer/9
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

they are down there mate!

The regexp for the field DIRECCION is going to be a little tricky, so I gave up writing it. In any case, I wanted to record the various cases it has to handle:

JUAN BAUTISTA ALBERDI 905
12 DE OCTUBRE Nº 532
COMODORO AUGUSTO LASSERRE S/Nº (house number should be S/Nº or maybe left blank)
RUTA NACIONAL Nº 3 Nº 760
RUTA NACIONAL Nº 3 S/Nº
CARLOS PELLEGRINI Nº 872-UF1 (number should probably be just 872)

These would be great candidates for @trescube’s work on unit tests inside sources.

See Czech countrywide for example usage.


Select not returning out vertices

OrientDB Version: 2.2.27 and 2.2.30
Java Version: 1.8.0_151
OS: Windows 10 Pro
Expected behavior
Select should return out vertices giving an source vertex id.
Actual behavior
Using SELECT to get the out vertices returns nothing, but TRAVERSE returns them.
Steps to reproduce
I deleted the whole OrientDB installation and all of the databases and installed version 2.2.30. I have created the database from scratch, no extras, I just registered two classes, Person and Card, both derived from V. I added two vertices, Person (Name=Mike, @rid=25:0) and Card (UID=123, @rid=34:0) and one IdentifiesWith edge going from Person to the Card. When I run this query, I get nothing:
select out('IdentifiesWith') from V where @rid == #25:0
but if I run this query, it finds the card:
traverse out('IdentifiesWith') from #25:0
From the online documentation the select clause should return the result. I can reproduce this problem  every time and I have a database that I can share if needed.
Thank you and best regards
Tomaz

Hi @tomazk8
I just tried it and it seems to work fine.
Could you please try the following:
select expand(out('IdentifiesWith')) from V where @Rid == #25:0

Thanks
Luigi

Hi Luigi
Sorry, I made a mistake above. The Id of the Person is #33:0, not #25:0. But also in this case the select I wrote previously returns nothing. Your query on the other hand (but with @Rid == #33:0) does return the Card vertex.
I've ran my original query also in a test .NET Core application (using a OrientDB .NET client) and there I get an object back with #rid = #-2:0 and one of the fields holds the results. So it actually returns an object with results, and not the the results directly... or something in this direction?
Best regards
Tomaz

Hi @tomazk8
Could you please post the results of
SELECT FROM V

and
SELECT FROM E

Thanks
Luigi

Hi @tomazk8
Yes, what happens here is that you get an object containing pointers to the out() vertices, ie. the RIDs of these vertices.
If you want their actual values, you have to do an expand():
select expand(out('IdentifiesWith')) from V where @Rid = #33:0

Thanks
Luigi

Hi Luigi
Ok, great. Thank you very much for your help.
Best regards
Tomaz


error in prior.plot()

When the prior.plot() is called, it gives the error below,
gamma_prior = GPy.priors.Gamma.from_EV(1.5, 0.7)
gamma_prior.plot()

AttributeError: 'Rectangle' object has no property 'normed'

This should be fixed by replacing "normed" with "density".

Thanks for the productive post @flydream0428 💪 Would you mind cutting a PR that fixes this?

@ekalosak Sure, I will have a go

Done, thank you.


Cannot Run

bash: ./gb: No such file or directory

You have to build the executable gb first by running make.
Let's close this non-issue please.


display usage options as a combobox

I wonder that autocomplete may be faster for dealing with editing 100s of rows :)


GraphQL middleware fixes

Current state
The Apollo server gets recreated for every request which is not needed. A weird side effect of that is that the request object never changes.
Changes introduced here
As we know the path it is not required to have other phases before this handler. The Apollo server gets created only once now (initially and via every HMR).
Checklist

[x] All commit messages adhere to the appropriate format in order to trigger a correct release
[x] All code is written in untranspiled ECMAScript (ES 2017) and is formatted using prettier

Hey @herschel666, I changed the commit message of the first comment to be of kind feat in order to trigger a minor release because the middleware phase has changed now.


AttributeError: 'VideoMenu' object has no attribute '_registered_menu_items' #400

HI
I have installed recently wagtailmenus to my wagtail project, currently I have another extra torchbox libraries, included wagtailvideos.
I just installed the wagtailvideo library and then after the basic setup I got the following error:
env_1 | AttributeError: 'VideoMenu' object has no attribute '_registered_menu_items'
env_1 | Internal Server Error: /admin/
env_1 | Traceback (most recent call last):
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/core/handlers/exception.py", line 34, in inner
env_1 | response = get_response(request)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/core/handlers/base.py", line 115, in _get_response
env_1 | response = self.process_exception_by_middleware(e, request)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/core/handlers/base.py", line 113, in _get_response
env_1 | response = wrapped_callback(request, *callback_args, **callback_kwargs)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/views/decorators/cache.py", line 44, in _wrapped_view_func
env_1 | response = view_func(request, *args, **kwargs)
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/urls/init.py", line 102, in wrapper
env_1 | return view_func(request, *args, **kwargs)
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/decorators.py", line 34, in decorated_view
env_1 | return view_func(request, *args, **kwargs)
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/views/home.py", line 119, in home
env_1 | 'user': request.user
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/shortcuts.py", line 36, in render
env_1 | content = loader.render_to_string(template_name, context, request, using=using)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/loader.py", line 62, in render_to_string
env_1 | return template.render(context, request)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/backends/django.py", line 61, in render
env_1 | return self.template.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 171, in render
env_1 | return self._render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/test/utils.py", line 96, in instrumented_test_render
env_1 | return self.nodelist.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 937, in render
env_1 | bit = node.render_annotated(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 904, in render_annotated
env_1 | return self.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/loader_tags.py", line 150, in render
env_1 | return compiled_parent._render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/test/utils.py", line 96, in instrumented_test_render
env_1 | return self.nodelist.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 937, in render
env_1 | bit = node.render_annotated(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 904, in render_annotated
env_1 | return self.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/loader_tags.py", line 150, in render
env_1 | return compiled_parent._render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/test/utils.py", line 96, in instrumented_test_render
env_1 | return self.nodelist.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 937, in render
env_1 | bit = node.render_annotated(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 904, in render_annotated
env_1 | return self.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/loader_tags.py", line 150, in render
env_1 | return compiled_parent._render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/test/utils.py", line 96, in instrumented_test_render
env_1 | return self.nodelist.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 937, in render
env_1 | bit = node.render_annotated(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 904, in render_annotated
env_1 | return self.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/loader_tags.py", line 62, in render
env_1 | result = block.nodelist.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 937, in render
env_1 | bit = node.render_annotated(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/base.py", line 904, in render_annotated
env_1 | return self.render(context)
env_1 | File "/usr/local/lib/python3.7/dist-packages/django/template/library.py", line 192, in render
env_1 | output = self.func(*resolved_args, **resolved_kwargs)
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/templatetags/wagtailadmin_tags.py", line 82, in main_nav_js
env_1 | return admin_menu.media['js']
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/menu.py", line 78, in media
env_1 | media += item.media
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/menu.py", line 106, in media
env_1 | return Media(js=[static('wagtailadmin/js/submenu.js')]) + self.menu.media
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/menu.py", line 77, in media
env_1 | for item in self.registered_menu_items:
env_1 | File "/usr/local/lib/python3.7/dist-packages/wagtail/admin/menu.py", line 64, in registered_menu_items
env_1 | if self._registered_menu_items is None:
env_1 | AttributeError: 'VideoMenu' object has no attribute '_registered_menu_items'
the error appears just after installing the library and open your project admin: localhots:8000/admin in my case.
this are the torchbox libraries I'm ussing:

torchbox libraries
wagtailvideos==2.10.2
wagtail-footnotes==0.6.0
wagtailmenus==3.0.2

this is the major parts of my setting file:

TEMPLATES = [
{
'BACKEND': 'django.template.backends.django.DjangoTemplates',
'DIRS': [
os.path.join(BASE_DIR, 'v2/templates/v2'),
os.path.join(BASE_DIR, 'api/templates/api'),
],
'APP_DIRS': True,
'OPTIONS': {
'context_processors': [
'django.template.context_processors.debug',
'django.template.context_processors.request',
'django.contrib.auth.context_processors.auth',
'django.contrib.messages.context_processors.messages',
'social_django.context_processors.backends',
'social_django.context_processors.login_redirect',
'wagtail.contrib.settings.context_processors.settings',
'wagtailmenus.context_processors.wagtailmenus',
],
'debug': False,
},
},
]

INSTALLED_APPS = [
# Django apps
'django.contrib.admin',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'django.contrib.sitemaps',

# Third party apps required by project and Wagtail
'modelcluster',
# Compressor is no longer required by wagtail
# but we still like offline compression of assets
'compressor',
'taggit',
'social_django',
# 'django_nose',

# Security
'csp',

# CORS
'corsheaders',

# Project apps that override wagtail functionality
....

# Wagtail apps
'wagtail.contrib.forms',
'wagtail.contrib.redirects',
'wagtail.embeds',
'wagtail.sites',
'wagtail.users',
'wagtail.snippets',
'wagtail.documents',
'wagtail.images',
'wagtail.search',
'wagtail.admin',
'wagtail.core',
'wagtail.contrib.sitemaps',
'wagtail.api',
'wagtail.contrib.settings',
'rest_framework',
# required for wagtailmenus
'wagtail.contrib.modeladmin',
# torchbox apps
'wagtailmenus',
'wagtailvideos',
'wagtail_footnotes',
...
'wagtail_webstories',
]

Thanks for logging this, can you try removing wagtailmenus  and see if the problem persists? I don't see it in the stack trace though so it might not be related. Also what version of Wagtail are you running?
Just as a side note, when logging bugs you can surround code in 3 backticks (```code goes here```) and it will format it properly.

Thank you @seb-b in fact it just happens after installing the wagtailmenus library, however int he traceback this is the reference to wagtailvideos:
env_1 | if self._registered_menu_items is None:
env_1 | AttributeError: 'VideoMenu' object has no attribute '_registered_menu_items'

infact VideoMenu is class inside the wagtailvideos library :
https://github.com/neon-jungle/wagtailvideos/blob/master/wagtailvideos/wagtail_hooks.py

Hi @leonardorhojas @seb-b this error isn't happening because of wagtailmenus. It's happening because you've added wagtail.contrib.modeladmin.
Essentially, the VideoMenu.__init__() is missing a call to super here: https://github.com/neon-jungle/wagtailvideos/blob/master/wagtailvideos/wagtail_hooks.py#L71
In order to trigger the following code in wagtail:
https://github.com/wagtail/wagtail/blob/main/wagtail/admin/menu.py#L66

@leonardorhojas Yes, just to clarify: The issue is with the code linked to above, which is within wagtailvideos. The error has nothing to do with wagtailmenus.


Output in JSON format

Add a .#json package that contains the output in JSON format, allowing flexibility for the structure and content we want to return.

This enables om ci run --json (https://omnix.page/om/ci.html):


[3pts] Recipes List

Should display a list of recipes created by the logged-in user as in the wireframe.
Should lead to recipe details.
Should allow the user to delete it  If the user is the owner of the recipe

This issue was closed by pull request https://github.com/Aquinasmoriasi/recipe-app/pull/21.


feat: add support for named exports

This PR adds support for named export in the plugin v2 implementation.
This is fundamental as a lot of guidelines recommend to use named exports for tree-shaking.
import * as style from ",.style.module.css";
Rather than
import style from ",.style.module.css";
It does not add support for tree-shaking itself, but it allows the above syntax to work for modules that use it.

@LucaColonnello v2.3.2 published with this feature, thanks. https://www.npmjs.com/package/esbuild-css-modules-plugin/v/2.3.2


Error when run test_electra.sh

When I run test_electra.sh. I face this error: Segmentation fault (core dumped)

Is there any to fix this error? Thanks!

I guess it may be caused by the different version of PyTorch and Transformers. I have no other idea about the segment fault error. I have tested all codes and checkpoints on my device before code release.

When I run test_electra.sh. I face this error: Segmentation fault (core dumped)  Is there any to fix this error? Thanks!

Hello, I have encountered the same issue. May I ask how you resolved it later? I would greatly appreciate it if you could reply.


fix typo peerDepdendencies => peerDependencies

Hello, this was an intentional misspelling that corrects an accidental misspelling in a very early version of the new plugin system.

Can you add a comment in the code then?

why don't you "replace all"?

At least one plugin was already published on a Github repo that I don't control. But fear not, I believe it's updated now and I will remove this chunk of code soon.


[Event Request] codeunit 6620 "Copy Document Mgt."- OnBeforeCopyPurchPostedDeferralsModify

Hi, would it be possible to get an event OnBeforeCopyPurchPostedDeferralsModify codeunit 6620 "Copy Document Mgt.", like this :

    local procedure CopyPurchPostedDeferrals(ToPurchaseLine: Record "Purchase Line"; DeferralDocType: Enum "Deferral Document Type"; FromDocType: Integer; FromDocNo: Code[20]; FromLineNo: Integer; ToDocType: Integer; ToDocNo: Code[20]; ToLineNo: Integer)
    var
    IsHandled: Boolean;
    begin
        ToPurchaseLine."Returns Deferral Start Date" :=
            CopyPostedDeferrals(
                DeferralDocType, FromDocType, FromDocNo, FromLineNo, ToDocType, ToDocNo, ToLineNo);
        IsHandled:= false;
        OnBeforeCopyPurchPostedDeferralsModify(ToPurchaseLine, IsHandled);
        if not IsHandled then
        ToPurchaseLine.Modify();
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeCopyPurchPostedDeferralsModify(var ToPurchaseLine: Record "Purchase Line"; var IsHandled: Boolean)
    begin
    end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


google drive link is not accessible

Thanks for your great work,the google drive link for your pretrained models is not accessible please provide an accessible link

Hello!
If possible, please share the models on google drive.
The previous provided links do not work.
Thank you!


Nested loop with iterator yields unexpected values

I would have expected the coroutine to yield the values 1, 2, 3, 4, 5, 6, 7, 8 in this order, but instead, it skips the 4 and terminates early. Here's a test case:
test("two nested loops with iterators should yield correct values") {
  val iteratorTest = coroutine { () =>
    val s = Seq(
      Seq(1,2), Seq(3,4), Seq(5,6), Seq(7,8)
    )
    val it1 = s.toIterator
    while (it1.hasNext) {
      val it2 = it1.next.iterator
      while (it2.hasNext) {
        yieldval(it2.next)
      }
    }
  }
  val c = call(iteratorTest())
  assert(c.resume)
  assert(c.value == 1)
  assert(c.resume)
  assert(c.value == 2)
  assert(c.resume)
  assert(c.value == 3)
  assert(c.resume)
  assert(c.value == 4)
  assert(c.resume)
  assert(c.value == 5)
  assert(c.resume)
  assert(c.value == 6)
  assert(c.resume)
  assert(c.value == 7)
  assert(c.resume)
  assert(c.value == 8)
  assert(!c.resume)
}

Apparently, this is not directly related to iterators. See below for a simpler test case without iterators.
When you replace the yieldval statement with the commented yieldval statement, the test passes.
test("A nested loop with a sequence kept in the outermost loop should yield correct values") {
  val test = coroutine { () =>
    val s: Seq[Seq[Int]] = Seq(
      Seq(1, 2),
      Seq(3, 4))
    var i = 0
    while (i < s.length) {
      val seq = s(i)
      var k = 0
      while (k < seq.length) {
        yieldval(seq(k))
        // yieldval(s(i)(k))
        k += 1
      }
      i += 1
    }
  }

  val c = call(test())

  assert(c.resume)
  assert(c.value == 1)

  assert(c.resume)
  assert(c.value == 2)

  assert(c.resume)
  assert(c.value == 3)

  assert(c.resume)
  assert(c.value == 4)
}

Thanks for reporting! That's almost certainly a bug - I will take a look into this.


npm run in vue-barista.app

This dependency was not found:

@/components/Printing in ./node_modules/cache-loader/dist/cjs.js??ref--13-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--1-0!./node_modules/vue-loader-v16/dist??ref--1-1!./src/App.vue?vue&type=script&lang=js

To install it, you can run: npm install --save @/components/Printing

and ppm run serve in due-order.app in http://localhost:8080 header is ok...but the message is:
"The Store is closed, come back soon"

sorry but through the tutorial https://workshop.serverlesscoffee.com/0-setup/1-self-hosted.html
and logged into the AWS console, the first step -Cloud formation- always gives me the following error:
So I follow the tutorial and in Cloud9 there is nothing created as the tutorial indicates.
However, the only thing I advanced was through the console and with the sam commands. and npm to put it on localhost.
Does it have something to do with user permissions to get the reference tutorial right?
Thank you very much and excuse my insistence


Cannot Detach policy on cluster destroy

I have issues
I'm submitting a...

[X] bug report
[ ] feature request
[ ] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
Cannot
If this is a bug, how to reproduce? Please include a code sample if relevant.
Create a cluster with policies and destroy...get the following error
* module.eks.module.eks.aws_iam_role.workers (destroy): 1 error(s) occurred:
* aws_iam_role.workers: DeleteConflict: Cannot delete entity, must detach all policies first.

What's the expected behavior?
Destroy the cluster and detach policies
Are you able to fix this problem and submit a PR? Link here if you have already.
https://github.com/terraform-aws-modules/terraform-aws-eks/pull/196
Environment details

Affected module version: 1.7.0
OS: Mac
Terraform version: 0.11.10

Any other relevant info

Resolved in https://github.com/terraform-aws-modules/terraform-aws-eks/pull/196


Updating Google's logo

Also GDG logo https://github.com/GDGSeattle/DevFest/blob/master/images/goog-dev-pic-50.svg

@cliveb are you sure this is the new logo? If we go to https://developers.google.com/groups/logo/ , the logo it is the same as the logo of the template I think

@iblancasa I guess both are ok, but these are the latest and Google I/O is using (I made svg) https://sites.google.com/site/nagdgresources/google-developer-logos

Ok! I will change it

I am not sure... because if someone want to take this template... he will change GDG Lviv logo by his GDG logo.
I am not sure if change GDG logo has sense

Right, the logo that @cliveb posted was for "Google Developers" and not for GDG.

Ah, Google Developers =/= GDG
so

GDG Seattle
sponsored by
< > Google Developers
Thanks clear now.

Looks nice! Thank you @iblancasa

@ozasadnyy you are welcome :+1:


Authentication possible?

Describe the problem
Can I put authentication in front of svelte press?
I want to add a login screen, then based upon the profile show pages, or hide pages.

Sveltepress preserves the full power of sveltekit. So you can directly use any features that can help you achieve authentication.
For example, a +page.server.js/+layout.server.js, or hooks.server.js


remove check for zero address when code size is enforced

enforceHasContractCode will revert if passed the zero address.

Thanks!


Filesystem provider parse_album errors out

What version of Music Assistant has the issue?
2022.7.0
The problem
filesystem provider errors out likely due to spaces in the filename (not in the parent folder name) as it seems
How to reproduce
create path structure as indicated in the below logs and sync
Relevant log output
2022-07-23 11:46:19 ERROR (MainThread) [music_assistant.music.file] Error processing /mnt/music/Kabah/La Vuelta Al Mundo/02 Florecitas.mp3 - not enough values to unpack (expected 2, got 1)
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 234, in sync_library
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 501, in _parse_track
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 738, in _parse_album
2022-07-23 11:46:20 ERROR (MainThread) [music_assistant.music.file] Error processing /mnt/music/Kabah/La Vuelta Al Mundo/03 El Complot.mp3 - not enough values to unpack (expected 2, got 1)
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 234, in sync_library
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 501, in _parse_track
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 738, in _parse_album

Additional information
music mounted via nfs from synology
What version of Home Assistant Core are your running
2022.7.6
What type of installation are you running?
Home Assistant Supervised
On what type of hardware are you running?
Linux

This is a real edgecase. So you have no ID3 tags in your files and then it falls back to parsing the artist and title from the filename except in that case it kind of expects a filename like this "artist - title.mp3"
I will fix the bad error so it ignores these files.
I strongly suggest you to either use proper ID3 tags on your files or rename them to have the common/standard filename of "artist - album".

Can we perhaps get a fallback parsing option for artist/album/title.mp3 pattern as this is the default iTunes library structure that a lot of people have?

I will leave it up to Marcel to answer your question but I would like to emphasise the actual answer is all of the tracks need to be properly tagged to get the most of MA and to minimise unexpected behaviour. Musicbrainz Picard is free and easy to use so I strongly recommend you use that to solve this issue.

The album and artist structure is used for additional metadata not to be used as primary source for the details. That's where ID3 tags are invented for.
Yes that structure is fully supported but still requires you to have proper ID3 tags, I'm not going to deviate from that.

Marked as enhancement to prevent the bad error

I've just released patch version 2022.7.1 which should fix this issue.
If you still experience any issues after updating to 2022.7.1 please let us know, thanks!

I have the same issue, unfortunately the latest update doesn't seem to solve the issue for me...
Trouble is my files do all have ID3 Tags and for some reason it is falling back on this process to gather the information.  My files do not have the name of the artist in the file name, just the track number then the track name (01 Welcome to the Jungle.flac) so will always fail when it gets to this point, my point is it should never even need to try and gather the data in this way for my files.
The error I see in the error log (single entry) changes every time I refresh the log as if it is reusing the same entry for each file it now ignores, error description is always the same but the track with the issue changes every time.  I have dip sample checked "erroneous" files and confirm they do all have valid ID3 tags.  At the moment it means Instead of over 14000 music files I have less than 1000

@antonyslater do not comment on an already closed issue please.
Please open your own issue if you have any issues. That will keep our minds sane, thank you!
That said, I did find the issue you reported above. Fixed in 2022.7.2 !

@antonyslater do not comment on an already closed issue please. Please open your own issue if you have any issues. That will keep our minds sane, thank you! That said, I did find the issue you reported above. Fixed in 2022.7.2 !

My apologies, it was only because it was the exact same issue I was seeing.  Will try 2022.7.2 and if I still have issues will log a fresh issue.


Subcutaneous glucose computation: Use plasma glucose concentration (G) instead of plasma glucose mass (Gp)

As per equation A17 in the paper:
Man, Chiara Dalla, et al. "The UVA/PADOVA type 1 diabetes simulator: new features." Journal of diabetes science and technology 8.1 (2014): 26-34.
dGs(t)/dt = - ksc  * Gs(t) +  ksc * G(t)
Hence, in t1dpatient.py, line195,
wouldn't the computation be:
dxdt[12] = (-params.ksc * x[12] + params.ksc * x[3]/params.Vg) instead of dxdt[12] = (-params.ksc * x[12] + params.ksc * x[3])?
Thanks!

Looks like it might be a typo in the original paper. The correct equations can be found in the following paper:
Messori, Mirko, et al. "Individualized model predictive control for the artificial pancreas: In silico evaluation of closed-loop glucose control." IEEE Control Systems Magazine 38.1 (2018): 86-104.

Looks like it might be a typo in the original paper. The correct equations can be found in the following paper:
Messori, Mirko, et al. "Individualized model predictive control for the artificial pancreas: In silico evaluation of closed-loop glucose control." IEEE Control Systems Magazine 38.1 (2018): 86-104.