skeleton-navigation-es2016-vs broken

I tried to share my experience with any Aurelia-vs skeletons before and failed to "impress anyone" as the problems I stated before remain. As I also happen to care about the perception VS users get encountering Aurelia for the first time, I am writing this again, with more details, starting with my own understanding of what should take place when opening this solution with this Visual Studio:
.
Opening this solution, VS starts installing dependencies (running npm install behind the scenes). After few minutes, dependencies are not installed as shown below:

Note: a solution where the user is advised to use the windows console (CMD.EXE or equivalent) and run the npm install followed by jspm install is not an integrated solution and makes Aurelia look bad.
At minimum, folks that support this skeleton need to determine why is it that npm install fails, fix that, then verify that jspm install works as well and when the user hits the CTRL-5 button (run without debugging) VS should invoke the gulp watch behind the scenes.

@AshleyGrant @cmichaelgraham Can you two look into this asap please?

npm dependencies not installed seems to be tooling bug.
at least for us in vs 2015 update 2 it always show npm dependencies node as not installed.
Although everything seems to be working fine.
for jspm yes - you got to run install yourself

What do you think about a postinstall npm task running jspm install?

Yeah I've been seeing npm - not installed in other projects as well going back before VS 2015 Update 2.
As far as running jspm install goes there is a Gulp task called build-jspm that you can run.

While the final decision will be made by @EisenbergEffect, I strongly disagree with the the solution like:

As far as running jspm install goes there is a Gulp task called build-jspm that you can run.

Here are my arguments:

In many cases, people trying to run this skeleton app are coming from the Windows / Visual Studio world, interested to see what is this Aurelia thing. It is possible that they have little or no experience with "MEAN" development and even if they do, they may never encountered the concepts of "cross compilation" from ES2015 version of JavaScript to ECMA 5.1, or the role of jspm in app building. So, just like we currently have VS invoking npm install (unsuccessfully however) we should have make VS invoke jspm install upon successful execution of npm install. All that should take place in the context of opening the skeleton solution file.

Using the same reasoning (summarized as using VS interface to build and run the skeleton app, without having ever to know about gulp scripts) this skeleton app needs a lots of work in order to get there. After all, I remember well the world of using the notepad, and CLI based compiler and mbuild - all of which are now completely hidden by VS.

I stated this before and was not able to make any impact. So, if there is a consensus that the current state of the VS targeting skeletons is OK, I will stop complaining about it and let the current team responsible for these skeletons work as they see fit.
Please take my comments as "written with best intentions" and motivated by my care for Aurelia.

I agree that it's not really in good shape now. Most of my team doesn't use VS for front-end development so it's difficult for us to get the "right" solution in place. We'd love some contributions from the community to improve this.
Long term, we are working with Microsoft to incorporate a new project dialog that uses our CLI, however, that will likely take a bit longer due to the MS cycle and ours syncing up.

Long term, we are working with Microsoft to incorporate a new project dialog that uses our CLI, however, that will likely take a bit longer due to the MS cycle and ours syncing up.

Based on Rob's comment above, having a "good enough" solution that will "hold the water" until dotnet core CLI based skeleton generation becomes available is something that needs to be done soon.
In my opinion skeletons need to be at least temporary owned  Aurelia-UI-Toolkits team and start with the solid definition of what needs to be done. Once signed off by @EisenbergEffect , we would ask for volunteers from the community to implement that definition.
If someone has a better idea how to address this - please tell us all :-)

@adriatic After listening to your responses on the SideWaffle repo I talked with @EisenbergEffect about his plans for Aurelia and the CLI project. I also talked to him about the work Sayed has been doing with pecan-waffle. It uses PowerShell to create project templates much faster than what SideWaffle can do now. When Sayed tested this and he found that the project creation for an Aurelia template just took 3 minutes versus the 30+ minutes it takes to create the project now.
Per my previous chat with Rob I hope to have an extension running in the next week or so that will combine the ASP.NET templates with pecan-waffle. This will be our temporary solution until the CLI is released at which point I'll start working on integrating it into the extension.
However, it should be noted that while this will help make creating the project's easier on Visual Studio it won't help with people not knowing about jspm or that they need to run any of the other commands. Per the comments above it probably wouldn't run any additional commands via the postinstall npm task.

@RandomlyKnighted Would there be a way to make the generated project automatically run the initial jspm install commands? If not, is it possible to display a readme file when the project first opens to provide instructions?

Well Visual Studio automatically runs npm install when the project first opens. So if we used the postinstall task then that should run the jspm install command automatically.
If you're wanting to avoid that then we could do the README. In the SideWaffle project we've got a template or two that opens a webpage when installed. I can check and see how that was done and do the same thing in the Aurelia extension.

I think running the post install would probably be best for VS users. For TS users we probably also want to post install typings. If we can do those two things and also show the readme with a bit of explanation about what just happened and what to do next, that would probably go a long way.

For TS users, what would the command be that would need to be added to the postinstall task?

New VS skeletons (reviewed by Mads himself) are releasing soon, along with some updates to the readme files.


Support another output format for Atlassian tools

I have recently seen that Atlassian tools like Jira want external links in the format as follows:
[link title|https://www.example.com]
This is not supported by any of the exiting output formats and hence a new one is needed.

Fixed in Release Titles Extension - 2.6.0 · glawrence/titles-extension


how to create a screen capture "image" of an element

i've read the docs but i cant find where i can export the result to a base64 image of what was captured 😢
also is it possible to capture a specific element ex.document.querySelector('[data-export]') ?

This library returns a new lightweight, self-contained HTML document in several formats, but NOT an image. More details on the readme page.

from the docs

This library can be used to:

Create a web page screen capture "image", and display the "snapshot" (e.g. by using an iframe).

As you can see it's not really an image - that is why we added quotes...
Regarding capturing a single element - that's a good idea - I'll add it as a new issue.
Thanks!

perfect, could u plz ping me when added ?

@ctf0 Sure. You can also just subscribe to issue #42


Add style parameters to admin boundary files

The styles used for admin boundaries are currently universal. We'd like to be able to allow each deployment to configure how boundaries are displayed. This includes increasing thickness for admin 1 boundary in relation to admin 2 boundaries, and so forth for each level.
See also https://github.com/WFP-VAM/prism-frontend/issues/169

See also  https://github.com/WFP-VAM/prism-frontend/issues/293


FR: Add filter_entry

Thanks for jwalk! I might have missed it, but if it isn't there already, I'd appreciate a filter_entry method similar to WalkDir proper.

I think you can do with with process_entries
?
It's a bit more involved (but offers more flexibility). You are given a mutable Vector of entries:

To filter an entry out remove it from the Vec
Reorder the Vec to change the order of how the entries will be enumerated
If you want to return a directory entry, but not descend into it's contents use entry.content_spec = None.


AttributeError: module 'fbchat' has no attribute 'Session'

Description of the problem
AttributeError: module 'fbchat' has no attribute 'Session'
Code to reproduce
import fbchat

# Log the user in
session = fbchat.Session.login("..., "...")

print("Own id: {}".format(session.user.id))

# Send a message to yourself
session.user.send_text("Hi me!")

# Log the user out
session.logout()

Traceback
Traceback (most recent call last):
  File "send.py", line 4, in <module>
    session = fbchat.Session.login("..., "...")
AttributeError: module 'fbchat' has no attribute 'Session'

Environment information

Python 3
fbchat version newest

Are you sure that the fbchat version is the newest? pip install fbchat installs an older version. In this case you should uninstall it, download the .zip and install that.

Are you sure that the fbchat version is the newest? pip install fbchat installs an older version. In this case you should uninstall it, download the .zip and install that.

Yes I probably have old version (installed by pip).
Thanks I will reinstall from source.

Can you say how to uninstall older version and install the newest version??

pip uninstall fbchat or pip3 uninstall fbchat should remove the old version.
Download the latest version with git clone https://github.com/carpedm20/fbchat
Go into the directory with cd fbchat and install it with pip3 install fbchat .  (make sure the dot is at the and).

pip install git+https://github.com/carpedm20/fbchat.git

I have the same issue but reinstalling directly from git didn't solve it. I still have the exact same error. What can I do now?

Make sure to download using git clone https://github.com/carpedm20/fbchat and install it from the directory instead of installing it directly from git with pip install git+https://github.com/carpedm20/fbchat.git.

I used git clone, it does the same thing. Have you tried it very recently? Just asking since your last reply was a few months ago.

Try pip3 install git+https://github.com/carpedm20/fbchat.git it works for me on windows.

Hello Sir. Can you help me to make a fb chat bot? I am facing some minor
problems but I can't solve these. I need a brief resource also. Please help
me. Thanks in advance.😊
On Sun, Aug 30, 2020, 12:07 AM smilexs4<EMAIL_ADDRESS>wrote:

Try pip3 install git+https://github.com/carpedm20/fbchat.git it works for
me on windows.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/carpedm20/fbchat/issues/566#issuecomment-683324088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APSQSXR3FYPMNBCFC3VIQC3SDE7X7ANCNFSM4M5JHCOA
.


Fix broken build and overview documentation links.

This change only affects to the readme file. It will not change anything on the binary.
Knative build documentation and Overview documentation links are broken. They must contain docs folder.

/ok-to-test
/lgtm
/approve

/lgtm


iso19115 to DCAT -> triple store

Semic produced an xslt to transform iso19115 to DCAT
https://github.com/SEMICeu/iso-19139-to-dcat-ap
this could be used as a starting point for loading iso19115 (as fetched from inspire portal) to SWR triple store.
Alternative option is to use pygeometa to import iso19115 and then export to DCAT
Actually did we agree to use (geo)DCAT(ap) as a standard to describe data resources? the standard is less optimal to describe non-dataset resources, such as reports, articles, videos, software.
For now we indicated to use the Dublin Core schema only.
Expected:
Schema to describe resources:

[ ] use dublin core
[x] use DCAT
[ ] other (wikidata, datacite, schema.org, iso19115)

iso -> rdf

[ ] semic xslt
[x] pygeometa
[ ] other (yarrrml, tarql, hale studio)

this development is upcoming, iso19139 has been loaded from various sources (bonares, inspire) into a central database. we envision pygeometa to transform iso19139 to mcf and mcf to dcat (jsonld) and rdflib to convert from jsonld to ttl


Unable to boot Windows

After enabling the feature "Virtual Machine Platform", I am unable to boot into Windows successfully. I keep getting a BSOD.
Am on Windows Version 20H2 (OS Build 19042.867)
Hotfix(s):                 7 Hotfix(s) Installed.
[01]: KB4601554
[02]: KB4562830
[03]: KB4577586
[04]: KB4580325
[05]: KB4589212
[06]: KB5001567
[07]: KB5000858
Network Card(s):           3 NIC(s) Installed.
[01]: Intel(R) Wi-Fi 6 AX200 160MHz
Connection Name: Wi-Fi
DHCP Enabled:    Yes
DHCP Server:     <IP_ADDRESS>
IP address(es)
[01]: <IP_ADDRESS>
[02]: fe80::a07f:2f10:f799:64e8
[02]: Realtek PCIe GbE Family Controller
Connection Name: Ethernet
Status:          Media disconnected
[03]: Bluetooth Device (Personal Area Network)
Connection Name: Bluetooth Network Connection
Status:          Media disconnected
Hyper-V Requirements:      VM Monitor Mode Extensions: Yes
Virtualization Enabled In Firmware: Yes
Second Level Address Translation: Yes
Data Execution Prevention Available: Yes

/dupe #5911


⚠ ESPN has degraded performance

In 478e09b, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1498 ms

Resolved: ESPN performance has improved in a1a1648.


[SPARK-35749][SPARK-35773][SQL] Parse unit list interval literals as tightest year-month/day-time interval types

What changes were proposed in this pull request?

This PR allow the parser to parse unit list interval literals like '3' day '10' hours '3' seconds or '8' years '3' months as YearMonthIntervalType or DayTimeIntervalType.
Why are the changes needed?

For ANSI compliance.
Does this PR introduce any user-facing change?

Yes. I noted the following things in the sql-migration-guide.md.

Unit list interval literals are parsed as YearMonthIntervaType or DayTimeIntervalType instead of CalendarIntervalType.
WEEK, MILLISECONS, MICROSECOND and NANOSECOND are not valid units for unit list interval literals.
Units of year-month and day-time cannot be mixed like 1 YEAR 2 MINUTES.

How was this patch tested?

New tests and modified tests.

Test build #139921 has started for PR 32949 at commit d17721e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44449/

cc: @MaxGekk
I have some concerns as follows. What do you think of them?

https://github.com/apache/spark/pull/32949#discussion_r653519077
https://github.com/apache/spark/pull/32949#discussion_r653522074
https://github.com/apache/spark/pull/32949#discussion_r653588113
https://github.com/apache/spark/pull/32949#discussion_r653595847
https://github.com/apache/spark/pull/32949#discussion_r653605039

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44449/

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

Test build #139921 has finished for PR 32949 at commit d17721e.

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

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

@sarutak Could you resolve conflicts, please.

Test build #140215 has started for PR 32949 at commit a284267.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44743/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44743/

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

Test build #140219 has started for PR 32949 at commit 5b37384.

Test build #140215 has finished for PR 32949 at commit a284267.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44747/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44747/

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

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

Test build #140219 has finished for PR 32949 at commit 5b37384.

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

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

@cloud-fan @AngersZhuuuu @Peng-Lei Since this PR can change behavior and impact on users apps, may I ask you to help in reviewing this PR, please.

@MaxGekk If you feel this change complicated, how about the following change in this PR?

Leave the behavior for multi units interval literals as it is like 1 year 2 months (still represented as CalendarIntervalType).
Leave the behavior for plural form units as it is like 3 years (still represented as CalendarIntervalType)
Change the behavior for single and singular form units like 3 year to comply with ANSI (represented as YearMonthIntervalType or DayTimeIntervalType).

One general comment about test: I think most tests do not want to test mixed fields of year-month and day-time interval. Can we update these tests to create valid intervals, and add some extra tests for the mixed-fields case?

@sarutak any progress?

@cloud-fan
This is in progress. SPARK-35983 blocks this to fix the issue pointed out here but I'll open PR for it today.

Test build #140711 has started for PR 32949 at commit 16d9af8.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45222/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45222/

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

Test build #140711 has finished for PR 32949 at commit 16d9af8.

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

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

Test build #140759 has started for PR 32949 at commit 10fc91e.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45270/

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

Test build #140759 has finished for PR 32949 at commit 10fc91e.

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

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

Test build #140769 has started for PR 32949 at commit 22c4050.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45281/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45281/

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

Can you add some test in interval.sql like interval '123231' day,interval '12342121.12313' second etc .. ?

Test build #140769 has finished for PR 32949 at commit 22c4050.

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

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

Test build #140833 has started for PR 32949 at commit 03d2522.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45344/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45344/

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

Test build #140833 has finished for PR 32949 at commit 03d2522.

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

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

Test build #140854 has started for PR 32949 at commit 94ade45.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45365/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45365/

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

Test build #140854 has finished for PR 32949 at commit 94ade45.

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

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

Test build #140863 has started for PR 32949 at commit fa40bcd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45374/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45374/

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

Test build #140863 has finished for PR 32949 at commit fa40bcd.

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

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

Test build #140939 has started for PR 32949 at commit 48f080f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45451/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45451/

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

Test build #140939 has finished for PR 32949 at commit 48f080f.

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

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

thanks, merging to master/3.2!


z=0 changed for yappCoordPCB?

For cutouts on front/back/left/right sides, the p(1) coordinate (z height in model coordinate) currently aligns with the underside of the PCB, though the coordinates documentation indicates it should align with the top surface of the PCB. It looks like this was changed when introducing support for multiple PCBs - references to pcbZ were replaced with getPCB_Z which doesn't include the board thickness.
Was this a deliberate change to the z=0 reference point? If it's a bug, I think replacing all 7 usages of getPCB_Z with getPCB_Za will restore the z=0 level to the top surface of the PCB. So far I've only testing with side plane cutouts, so I'm not sure if there are wider impacts of this Z->Za change.
The example Multiple_PCB_v30.scad includes pcbThickness() offsets for the p(1) components which is why those cutouts align to the board top surface.

Update released

Thank you!


Fix typo in prometheus readme

@k8s-bot ok to test

@tmc can you sign the linux foundation CLA by clicking the details link

yeah just went through that -- do I need to say something to re-trigger?

there it goes.

We are working through the false positives of CI in another PR. This LGTM.

LGTM. @viglesiasce I'm wondering if README changes require a bump in Chart.yaml (since the README is packaged in the chart?). Will the CI re-publish the current version?

I am thinking that for README only fixes we dont bump the version of the chart. Its not a functional change to anything so its not worth burning version numbers and have people confused that they dont see a change. I could be convinced otherwise but just feel its a bit heavy handed. The chart will in fact be republished with the merge. I'm not sure what happens if/when we start signing charts from this repo. Same question could be posed for NOTES.txt changes which due to templating I would be more inclined to bumping the version.

@viglesiasce yeah I agree with you, NOTES.txt should solicit a bump, but READMEs shouldn't. Good point, signing will be an issue, in which case we may need to bump for READMEs (perhaps we can start using a SemVer tag or something).
@tmc can you rebase on top of the latest changes so the CI can test these changes?

@prydonius I think CI runs off the merged branch.
@k8s-bot ok to test

Just realised it's just a README fix anyway :P


ion-js TypeError: Failed to convert parameter ... to Ion Binary: j_.Value.from is not a function

With latest esbuild release - deploying a cdk application that uses esbuild to bundle typescript before deplying lambda code for
a QLDB client service. Facing following error on execute transaction function from AWS qldb nodejs driver typescript:  j_.Value.from is not a function.
After tracing found:
It is coming from ion-js Value type.
Something changed between v0.17.18 and latest esbuild that cdk uses by default. Making compiling the ion-js typescript code not properly build the relevant Javascript object.
After locking the esbuild version to 0.17.18 the code compiled correctly.
Uncertain but suspect it has something to do with namespace, interface and class declarations for Value here: https://github.com/amazon-ion/ion-js/blob/master/src/dom/Value.ts

Can you please provide more context here? What is the typescript version as well as ion-js version being used? If you can also provide a minimal example of how ion-js is being used here that would be helpful.

Can you please provide more context here? What is the typescript version as well as ion-js version being used? If you can also provide a minimal example of how ion-js is being used here that would be helpful.

Versions: from package.json
"ion-js": "^4.3.0"
"typescript": "~5.1.3"
ion-js wasn't used directly but part of the amazon-qldb-nodejs-driver
the error is raised here from trying to dumpBinary
https://github.com/awslabs/amazon-qldb-driver-nodejs/blob/master/src/Transaction.ts#L141
This occurred when using latest esbuild version to deploy the cdk app that talks to qldb
but when locking esbuild to v0.17.18 it works fine
making me suspect a typescript build issue for the Value interface


feat: Variable DOCKER_GID

Problem
Default DOCKER_GID on my Ubuntu 20.04.5 LTS (Focal Fossa) system seems to differ from everyone else's
My value: 119
Other environments seems to work fine with 999
Approach

Roll forward pull-upstream.sh to include all installed plugins and repos
pull-upstream.sh should also execute git status to see current branch for each plugin
Allow the group for GOSU_USER to be set from host's environment variables (default=999)

How to Test

Run id to see your local user's groups - find the ID for the docker group here
Set DOCKER_GID envvar on host: export DOCKER_GID=119
Execute make dev - this should start Girder with 119 as the Docker Group ID
(Optional) add export DOCKER_GID=119 to your local ~/.bashrc or ~/.profile if your GID is not 999

Alternative fix for this went in as a part of https://github.com/whole-tale/deploy-dev/pull/58


Handle legacy core styles in ng generate

The relevant parts of this PR were submitted as part of #25734


[Cosmos DB] stack trace is incomplete in FetchError making it very hard to troubleshoot root cause (SQL API)

Steps to reproduce

Flood a cosmosdb container (SQL) with read requests so that rate limiting will occur
Log the errors that are thrown

Expected
Errors thrown by the SDK do contain a meaningful stack trace so that developers know which part of their application code is responsible for the error.
Actual
The FetchError thrown by the SDK contains an incomplete stack trace. The caller method is missing and therefore developers do not know what part of their application code is responsible for the error. This is very problematic for medium to complex applications.
Example:
error: [app] Unhandled exception: FetchError: request to https://myexample-westeurope.documents.azure.com:443/dbs/MasterDB/colls/Core/docs failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
    at ClientRequest.<anonymous> (C:\DEV\flourishapi\node_modules\node-fetch\lib\index.js:1455:11)
    at ClientRequest.clsBind (C:\DEV\flourishapi\node_modules\cls-hooked\context.js:172:17)
    at ClientRequest.emit (events.js:327:22)
    at ClientRequest.EventEmitter.emit (domain.js:482:12)
    at ClientRequest.emitted (C:\DEV\flourishapi\node_modules\emitter-listener\listener.js:134:21)
    at TLSSocket.socketErrorListener (_http_client.js:426:9)
    at TLSSocket.emit (events.js:315:20)
    at TLSSocket.EventEmitter.emit (domain.js:482:12)
    at emitErrorNT (internal/streams/destroy.js:92:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:60:3)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {"timestamp":"2020-09-14T05:41:03.181Z"}

Tested with Node v12.18.0 and "@azure/cosmos": "3.9.0".

hey @FlourishS, this stack trace is all internal to node and is coming from node-fetch rather than the SDK. The SDK handles errors to provide meaningful stack traces often but in this instance it seems the developer would be aware of where the application is causing the error since this is intentionally being rate limited. We can improve errors and messages so if there's a case where it's unclear why the unhandled error is throwing, we'll provide a better error there.
All that said - we intend to remove our dependency on node-fetch and that would alleviate these FetchErrors that are unhandled - these errors would then have to be handled by our implementation that replaces fetch

Hey @zfoster thank you very much for your explanation.
To make the case more tangible: Assume a production application with hundred different business entities that are persisted in cosmosdb. Outside of peak hours you find some of these FetchErrors in your app logs. Unfortunately you will not be able to trace down the problem due to this stack trace. Instead you will need to enable the highly problematic debug log in your production environment to log every single query and whether it failed or succeeded. I would expect these internal node errors to be wrapped by the sdk.
Looking forward to the new sdk release that you mentioned. Thanks.

@zfoster Do you have any issue tracking the move away from node-fetch which we can close this issue in favor of?

good question, we’ve been tracking an issue in the node-fetch repo, cc @southpolesteve. I think we don’t because they haven’t fully committed to breaking the browser flag and moving it to a new project.
Software Engineer
CosmosDB

From: Ramya Rao<EMAIL_ADDRESS>Sent: Tuesday, September 29, 2020 9:55:58 PM
To: Azure/azure-sdk-for-js<EMAIL_ADDRESS>Cc: Zach Foster<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-js] [Cosmos DB] stack trace is incomplete in FetchError making it very hard to troubleshoot root cause (SQL API) (#11229)
@zfosterhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzfoster&data=02|01|Zach.Foster%40microsoft.com|e79da936b170488041f308d864e3faae|72f988bf86f141af91ab2d7cd011db47|1|0|637370277613118692&sdata=WA2XUpGfWl6I45At%2BzqoLmiyXgQZ%2BK%2Bo6BZ0ZBX0h4Y%3D&reserved=0 Do you have any issue tracking the move away from node-fetch which we can close this issue in favor of?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-js%2Fissues%2F11229%23issuecomment-701112239&data=02|01|Zach.Foster%40microsoft.com|e79da936b170488041f308d864e3faae|72f988bf86f141af91ab2d7cd011db47|1|0|637370277613118692&sdata=f8kLyfH0PMz80AL%2B713hagIcYNXo3vPVn79ue%2Br97G8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAALHHQR7QUCLBH77A5V3O5TSIKF25ANCNFSM4RLFKNKA&data=02|01|Zach.Foster%40microsoft.com|e79da936b170488041f308d864e3faae|72f988bf86f141af91ab2d7cd011db47|1|0|637370277613128685&sdata=z%2BDmU9W6wO%2FFtct%2FL2JftbTupM9ZZBgq3EkhpTXTEKU%3D&reserved=0.

@zfoster Let's add it to the v4 issue and close this one

@zfoster Can this be closed if the needful update has been made to the v4 issue as mentioned by @southpolesteve above?

@zfoster Can this be closed if the needful update has been made to the v4 issue as mentioned by @southpolesteve above?

Added to the v4 RFC list


Fixes to taylor review

@NTaylorMullen  - this is the right PR - addresses your issues except a couple formatting cases

@DamianEdwards  @danroth27  this PR has all your requested fixes

Let's handle these changes in the original PR: https://github.com/aspnet/Docs/pull/419


Latent var only experiments

Thanks for sharing the great work!
I was wondering how to try the Latent var (LV) only in Table 3: Decoder ablations of PGP paper. Is there any configuration?
Thanks in advance,
Zhen

Sure, you can change the aggregator from PGP to Global Attention. You'll need to modify the training config file as follows:
Replace aggregator parameters with
# Aggregator parameters
aggregator_type: 'global_attention'
aggregator_args:
  target_agent_enc_size: 32
  context_enc_size: 32
  emb_size: 128
  num_heads: 32

Also remove the behavior cloning loss from the config
losses: ['min_ade_k']
loss_weights: [1]
loss_args:
  - k: 10

Thank you for your quick reply!


Documentation fixes

This PR contains:

[ ] New features
[ ] Changes to dev-tools e.g. CI config / github tooling
[X] Docs
[ ] Bug fixes
[ ] Code refactor

What is the current behavior? (You can also link to an open issue here)
The documentation contains several typos (misspellings, case issues, punctuation) as well as some issues with a few proper names. There is also a broken link that was fixed and one phrasing improvement.
What is the new behavior?
Fixes the identified issues.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Not to my knowledge. I also ran quarto preview . to make sure everything built without issues.
Other information:
I was reading the docs to start writing an eval with Inspect. It's a well-built framework and I hope to make more substantive contributions later. Thanks for open sourcing it!

Thank you!!! We don't yet have the process for taking external PRs into this repo worked out (but we are working on it!). Your edits are excellent and we will incorporate them as part of our next sync (development currently occurs on an internal repo and which is then synced frequently to this public repo).

Sounds good, thanks! I'll close this PR then.


User experience updates

Updated all of the tab names to be more clear on what they do
Updated settings collapsible menu names to be more clear
Changed wording and labels on some settings
Added tooltips for some settings
Updated autorotation window UI elements to be a bit more clear (see screenshot and 0cb6560)

Added an "about" tab (80c7ea8)

Screenshot of updated window with requested changes.


Need an example how to use it with otelhttp tracing

I don't understand how to implement tracing middleware (https://pkg.go.dev/go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp) into web server generated from spec. Can you provide an example please?

I did it
func AddTracingMiddleware(handler http.Handler) http.Handler {
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		otelhttp.NewHandler(handler, middleware.MatchedRouteFrom(r).PathPattern).ServeHTTP(w, r)
	})
}


Download file with long filename failing on eCryptfs

This is a clone of https://github.com/boto/botocore/issues/1916#issue-538043859 since the issue is still occuring.
I commented with https://github.com/boto/botocore/issues/1916#issuecomment-647422922, but since this didn't reopen the issue, I think the best thing to do is to reopen a new bug report.
Here are the exact commands I use:
$ aws s3 cp s3://bucket-name/dirname dirname --recursive
download failed: s3://bucket-name/dir-name/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.gz to dir-name/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.gz [Errno 36] File name too long: '/home/nav/dir-name/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.gz.8C2e65Fa'

If I'm understanding this correctly, this is because I'm using eCryptfs which requires filenames to be less than 140 characters (based on https://unix.stackexchange.com/a/32834/215614). So the fix would be checking the actual system's filename limit and creating the temp file name based on that, or just lowering the 255 limit to 140.

@TheNavigat - Thank you for your post. Currently we are using a max file length of 255. To make it to work with every file system we need to check for system's filename limit before creating temp file. Marking this as a bug.
From the article you linked it looked like if filenames are not encrypted, then you can safely write filenames of up to 255 characters and encrypt their contents, as the filenames written to the lower filesystem will simply match. In the meantime you can change you code to use 140 as max file length and make it to work.

I don't know if there's an easy way to change whether eCryptfs encrypts the filename on a currently running system.
However, I modified the code and lowered the limit to 140, and that worked indeed. Thanks a lot!

Checking in - I saw a related comment here: https://github.com/aws/aws-cli/issues/3514#issuecomment-562358218. 255 characters does seem to be the system default but I believe S3 itself has a 1024 character limit. Regardless - it seems like this specific issue related to a third-party service and a solution was found. Based on that I think this issue can be closed, but please let us know if you had any follow up questions.


[feature request] allow pointer accessor outside predicate for parsing

Currently i can even parse it. It would be usefull to implement your own "scope" via "patching".
I would like to use a patcher to patchh somethin like this:
.Test

and patch it to (if not used in a "build in function"!)
CurrentScope.Test

https://expr-lang.org/docs/patch
but i get directly an error cannot use pointer accessor outside closure or cannot use pointer accessor outside predicate without even a chance to access the ast tree.

So what would we the case for this?
Pointer accessor (#.Test or .Test) is reserved for builtins like all and map. To access your current env you can simply use Test (do leading dot).

@antonmedv
Basically simple to say: To build such closure functions myself.
For example, I have functions that describe an element.
element(“information”, closure)
The first time the expr is executed, only the information is drawn. The closure is not executed. With a certain interaction, the expr is executed again, but this time with closure and the context of the element.
If patching were possible here, I would try to give myself a small way to define callbacks that allow a “tree” as a parameter. Which I can then execute in the callback (callback is then closure).


[NSE-540] enable reuse subquery

What changes were proposed in this pull request?
This patch fixes the subquery reusing. The issue seems introduced by hashagg code change
Note this patch also removed the columnar subquery expression support.
Signed-off-by: Yuan Zhou<EMAIL_ADDRESS>How was this patch tested?
locally verified

close #540


Basic usage of Command Mode.

:print("Hello World") should output Hello World
Basically this need to be a lua shell I think.

Crude as it may be I think this issues can be close.


Theming - Spectrum Components

Update our current usage of React Spectrum components to use DH theming.
Epic: https://github.com/deephaven/web-client-ui/issues/1537

@bmingles should this ticket be closed?

Taken care of in other PRs such as #1680


MB-8945: Clean up MtoShipmentForm log pollution

Description
As part of the TRA epic to clear out console warnings/errors during front-end test runs, this pull request fulfills a ticket to resolve emitted messages from the test fixtures for MtoShipmentForm.
In this case, the emitting warnings indicated an inconsistency with how the formatSwaggerDate function was using the moment library to parse date strings in multiple possible formats (from both user input and database stored values). The easiest way to solve this issue was to adapt it to utilize a parseDate function that's in use elsewhere in our code base, which has moment check the input date string for multiple valid date formats.
Reviewer Notes
Ensure that all client and integration tests pass, and that the MtoShipmentForm.test.jsx file does not emit any console messages during testing. There should be no perceptible UI change to the user.
Setup
yarn test MtoShipmentForm

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Part of epic MB-8944.
Closes MB-8945.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-8945

  Generated by :no_entry_sign: dangerJS against 43f05998de4dc4109c0c5b8a91bfe2fb74f86d8b


Privatelink support

This pr adds initial support for private links, however due to the backwards compatible nature of the webdav api the dav/meta endpoint needs to return a path that is relative to the currently logged in users home. Currently shares are jailed into a subfolder inside the users home ... and we would need to reconstruct the path in the shares folder.
Maybe the full implementation should wait until we have a globally accessible endpoint.

bitrotted and superseded by https://github.com/cs3org/reva/pull/2741


Vstoshared

New sample that shows how to migrate from VSTO add-in to Office web add-in by using a shared code library so that you can maintain both codebases.

@davidchesnut FYI, I'm OOF until 12/23 -- so please request review from someone else if you need feedback before then. Thanks!


TextInput with ScrollView, touch events not firing

Hey there, I'm having an issue where after I input something into a TextInput, I have to press twice to press something else
The hierarchy for my code is
     <View>
        <ScrollView>
             <View>
                  <View>
                        <TextInput>

I have buttons in the same View as the TextInput, and I cannot click on any of them without first hiding the keyboard
I've seen the other closed issue, however that does not solve the issue.

Hi @ErkanSen14, how is your issue different from #4087?

I only saw the other closed solution and not this one; marcshilling and hufeng's solutions solve it for me. Thank you!

@facebook-github-bot duplicate #4087

@damusnet tells me this issue is a duplicate of #4087. Let's discuss there, closing this one.


React Kitchen Sink Demo page stopped working

I think with the update to 3.4.0, the react kitchen sink demo page stopped working:
https://framework7.io/react/kitchen-sink.html
Notice the phone iframe is just a blank white rectangle.
The js console says, ReferenceError: Can't find variable: EXTEND.  I suspect that's the issue.

Thanks, fixed


"WebGL: INVALID_OPERATION: useProgram: program not valid", source: https://cdnjs.cloudflare.com/ajax/libs/three.js/110/three.min.js (124) I/chromium( 4662): [INFO:CONSOLE(124)] "WebGL: INVALID_OPERATION: useProgram: program not valid", source: https://cdnjs.cloudflare.com/ajax/libs/three.js/110/three.min.js

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Use the webview plugin to display this website: "https://gerardjm018.github.io/animationproto/whitesleep.html"
The website is displayed but some of the material in the website is not displayed
There are also some invalid operation in the debug console when running the webview " "WebGL: INVALID_OPERATION: useProgram: program not valid", source: https://cdnjs.cloudflare.com/ajax/libs/three.js/110/three.min.js "

Expected results
all the material in the website is fully shown in the webview
Actual results
some of the material is missing
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

class Webview extends StatefulWidget {
  final String animation;
  final String room;
  const Webview(
      {Key? key,
      required this.animation,
      required this.room})
      : super(key: key);

  @override
  State<Webview> createState() => _WebviewState();
}

class _WebviewState extends State<Webview> {
  String _animation = 'Base';
  WebViewController? _controller;
  Map<String, String> animationMap = {
    'Base': 'https://www.youtube.com/',
    'Sleep': 'https://gerardjm018.github.io/animationproto/sleep.html',
    'Workout': 'https://gerardjm018.github.io/animationproto/walkAction.html',
    'Study': ''
  };

  Map<String, String> roomMap = {

  };

  @override
  void initState()  {
    super.initState();

    _controller = WebViewController()
  ..setJavaScriptMode(JavaScriptMode.unrestricted)
  ..setBackgroundColor(const Color(0x00000000))
  ..setNavigationDelegate(
    NavigationDelegate(
      onProgress: (int progress) {
        // Update loading bar.
      },
      onPageStarted: (String url) {},
      onPageFinished: (String url) {},
      onWebResourceError: (WebResourceError error) {},
      onNavigationRequest: (NavigationRequest request) {
        if (request.url.startsWith('https://www.youtube.com/')) {
          return NavigationDecision.prevent;
        }
        return NavigationDecision.navigate;
      },
    ),
  )

  ..loadRequest(Uri.parse(animationMap[_animation]!));
    print(widget.animation);
  }

  @override
  Widget build(BuildContext context) {
    return Row(children: [Container(child: WebViewWidget(controller: _controller!), height: 300,width: 300,),],
    );
  }
}

this is to call the webviewwidget
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.10.2, on Microsoft Windows [Version 10.0.22621.1992], locale en-ID)
    • Flutter version 3.10.2 on channel stable at C:\Users\****\Documents\flutter_dev\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9cd3d0d9ff (8 weeks ago), 2023-05-23 20:57:28 -0700
    • Engine revision 90fa3ae28f
    • Dart version 3.0.2
    • DevTools version 2.23.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\*****\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.6.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.6.33712.159
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] VS Code (version 1.80.1)
    • VS Code at C:\Users\*****\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.68.0

[√] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.22621.1992]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 114.0.5735.199
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 114.0.1823.67

[√] Network resources
    • All expected network resources are available.

@GERARDJM018
I tried your code sample running on Android emulator using latest master version and observed below:

I tried passing different animation values, but I only got to see the youtube link. Also, when I tried to access the html links directly in browser, it didn't show anything. I also didn't get the invalid_operation error in the console while accessing the app.
Can you also verify if using same links in native webview, do you get expected result or same behavior as here ?

oh yea sorry for the code mistake, I mean when passing "https://gerardjm018.github.io/animationproto/whitesleep.html" this URL on the webview widget.

^ this is the image of me opening the url

Can you also verify if using same links in native webview, do you get expected result or same behavior as here ?


go list "can't load package"

What version of Go are you using (go version)?
go version go1.7.1 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/kyl/Code/gocode"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.7.1/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.7.1/libexec/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/6f/k23pyfjx09l61t96dyyg3kyh0000gn/T/go-build752362407=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
~ > go list
What did you expect to see?
a list of installed go packages
What did you see instead?
can't load package: package .: no buildable Go source files in /Users/kyl

installed with brew. just uninstalled all go versions and reinstalled and it still happens.
 ~ > brew info go
go: stable 1.7.1 (bottled), HEAD
The Go programming language
https://golang.org
/usr/local/Cellar/go/1.7.1 (6,436 files, 250.6M) *
  Poured from bottle on 2016-09-08 at 12:05:47
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/go.rb

That's not what go list does.
Run go help list for details.


SourceJar plugin needs to be applied after setting java toolchain version

Probably due to how a ClasspathExec task gets the java launcher convention during initialization, and that type of task is created when this plugin is applied. Not sure if it is really an issue, but I've decided to report it as such.
I've worked around it with my Plugin DSL setup by doing this:
plugins {
    // ...
    id 'org.moddingx.modgradle.sourcejar' version '[4,5)' apply false
    // ...
}

// setting version variable and other mundane stuff...

java.toolchain.languageVersion = JavaLanguageVersion.of(17)

apply plugin: 'org.moddingx.modgradle.sourcejar'

// everything else...

If you're curious, here's my settings.gradle setup:
pluginManagement {
    resolutionStrategy {
        eachPlugin {
            // ModGradle
            var plugin = requested.id.toString()
            if (plugin.startsWith('org.moddingx.modgradle.')) {
                useModule "org.moddingx:ModGradle:${requested.version}"
            }
        }
    }
    repositories {
        gradlePluginPortal()
        maven { url = 'https://maven.minecraftforge.net/' }
        maven { url = 'https://repo.spongepowered.org/maven' }
        maven { url = 'https://maven.parchmentmc.org' }
        maven { url = 'https://maven.moddingx.org' }
    }
}

plugins {
    id 'org.gradle.toolchains.foojay-resolver-convention' version '0.5.0'
}

Gradle 8.1.1
ForgeGradle 6.0.11
ModGradle 4.0.4

This seems to be a ForgeGradle problem. ForgeGradle accesses the toolchain version directly in their task constructors instead of using a provider. The ClasspathExec task works fine and allows the toolchain version to be changed after it has been constructed.
See:
https://github.com/MinecraftForge/ForgeGradle/blob/FG_6.0/src/common/java/net/minecraftforge/gradle/common/tasks/JarExec.java#L83-L86
https://github.com/MinecraftForge/ForgeGradle/blob/FG_6.0/src/common/java/net/minecraftforge/gradle/common/tasks/ExtractRangeMap.java#L30-L37


commit translated contents

Docs Build status updates of commit 0a1f833:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 0a1f833:
:warning: Validation status: warnings

File
Status
Preview URL
Details

DocsLegal/index.md
:warning:Warning

Details

DocsLegal/GDPR.md
:white_check_mark:Succeeded

DocsLegal/TOC.md
:white_check_mark:Succeeded

DocsLegal/termsofuse.md
:white_check_mark:Succeeded

DocsLegal/index.md

Line 16: [Warning-InvalidFileLink] Invalid file link:(~/information-protection/software-license-terms).

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


Capital letter not allowed in resource names starting in 5.25.0

Describe what happened
Starting in 5.25.0 we are seeing the following error when spinning up resources with capital letters in the name:
cloudflare:index/d1Database:D1Database resource 'MyDatabaseDatabase' has a problem: Invalid Attribute Value Match. Attribute name must contain only lowercase alphanumeric characters, got: MyDatabase. Examine values at 'MyDatabaseDatabase.name'.
we already have resources with names that are in this format so this is a breaking change. i'm assuming this is coming from the terraform provider but i'll look closer
Sample program
n/a
Log output
No response
Affected Resource(s)
No response
Output of pulumi about
n/a
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

sorry turns out cloudflare made this change in their api and it's bubbling down - closing


Replay demo rosbag failed and Dreamview restart failed

Question 1:
I downloaded demo_2.0.bag successfully. But when I execute "rosbag play -l ./docs/demo_guide/demo_2.0.bag" if failed as following message.
Traceback (most recent call last):
File "/usr/bin/rosbag", line 34, in
import rosbag
File "/usr/lib/python2.7/dist-packages/rosbag/init.py", line 33, in
from .bag import Bag, Compression, ROSBagException, ROSBagFormatException, ROSBagUnindexedException
File "/usr/lib/python2.7/dist-packages/rosbag/bag.py", line 58, in
import genmsg
ImportError: No module named genmsg
Could you let me know how can i fix this error?
Question 2:
I started Dreamview by http://localhost:8888 successfully. But after restarting ubuntu, the page said "Firefox cannot establish connection to localhost:8888". Could you let me know how can i fix this error?

I have a similair issue, the /tmp/supervisord.log file tells me that both monitor and dreamview exit because of a SIGABRT signal. Not sure where to go from here

Found the logs for dreamview and monitor, it was trying to access .ros, which was owned by root and not my user,
chown -R user ~/.
fixed the problem

@h0wser Do you mean
chown -R user ~/.
can fix the problem of launching localhost:8888? I tried this but the Firefox still cannot open localhost:8888.
Besides, there is not /tmp/supervisord.log such a file in my folder.

@lyxzhl are you still experiencing the error? Closing this issue currently, but feel free to open it should you have any more questions.


Correct Payload Size Calculation and Elf Header Corruption Fix

Fixed the calculation for fitting the payload on last page of RX segment using one mentioned in Issue #8
ELF header offsets are adjusted only if the section is really expanded, fixing Issue #7

Thanks!


Text Content Removed from Children (2.6.1)

Version
2.6.1
Test Case
Here is an example. If you right click you'll see the menu pop up but empty. If you inspect the DOM, you'll notice that the child elements are rendered but with no text content.
Steps to reproduce
Render normal children (not using <MenuItem>) within a <ContextMenu>.
Expected Behavior
Children should remain intact.
Actual Behavior
Child elements are being modified and somehow their text content is being removed. I tried 2.6.0 and 2.5.2 which both work fine so this must have been introduced in the latest version.

@codeart1st I think your last change 1a21fba69c9dab71b63292abebc23113e55b464c is causing this.

No problem I will look into this.

https://github.com/vkbansal/react-contextmenu/blob/master/src/AbstractMenu.js#L105 raw strings are no object types of ReactElementSymbol and it wasn't a good idea to skip those childs and nullify them.

@codeart1st no problem, thanks for jumping on this so quickly! Please ping me when a patch is released.

Published v2.6.2

Awesome -- updated, tested and confirmed.


[refactor]: main.go into handler.go

As of now, the all handler functions and routes are present in main.go file itself which isn't ideal. It should be brokendown into small code portion to allow better scaling in future such as handler.go, utils.go, etc..

this issue is resolved via #5


Add :hasChild(el) pseudo-selector to support :has(el) selections that search in direct children only, see #1116

Added a feature to make :has(el) pseudo selector be able to search only in direct children, as requested in issue #1116.
For example:
doc.select("div:has(p)") //returns divs that are containing p elements anywhere in their sub-hierarchy.
New Feature:
doc.select("div:hasChild(p)") //returns divs that are containing p elements only in their direct children.
I have added the necessary comment for this new pseudo-selector in Selector.java so the documentation can be updated and added the necessary test case.

Hello @jhy ,
Hope you are doing well.
Could you please check out this PR when you have the time.
Thank you in advance.

Thanks @haruntuncay, this looks good. We went with #1214 to simplify the selector query syntax.


Add tags to licenses.json

I would like to suggest adding the tags to the licenses.json file.  This would make the SPDX License Generator Tool implementation simpler and more efficient.  It may also help other consumers of the FSF API by not requiring them to read the individual JSON files for some of the more interesting information stored in the tags.
The resultant licenses.json file size is < 22 KB.
The file licenses.json prior to this change is 14 KB.
I would consider the additional size a reasonable trade-off for easier access to the additional information.

Supersceded by the just-merged #6, which adds everything to a new licenses-full.json.


Custom base model & merge adapter issue

Feature Request
I fine-tune mistral successfully now i want to further fine-tune and i m not able to select my own model as base model and can you please add --merge-adapter drop down so we user have option to choose for

Motivation
For continues fine-tune
Additional Context
Thank you!

training parameters shouldn't be empty and has merge_adapters. you can just set it to true.

@abhishekkrthakur  that's not issue that i clear while screenshot issue is i already have trained model i need to finetune that means i want to select my own model as base model

can you please add --merge-adapter drop down so we user have option to choose for
is this not one of the issues?

so im saying is that one of your issues is already a non issue. its available in training params and im asking why is your training params empty?

oh got it thanks and other issue ?

i used "merge_adapter": true, seems it stuck at  it is not moving since 1 hour

if you use merge adapter, you need to make sure you have a large enough gpu

i m using google colab pro with

Use A100 GPU

ability to add custom models has been implemented. to use custom models pass comma separated custom models in AUTOTRAIN_CUSTOM_MODELS environment variable.
e.g. export AUTOTRAIN_CUSTOM_MODELS="abhishek/asjfahsdf, abhishek/4572436"

closing the issue as issues have been addressed.

Use A100 GPU

not working like no error no progress nothing

found the merge adapter issue and fixed it. also, all AutoTrain Finetuned LLMs are now deployable to Hugging Face Inference Endpoints WITHOUT merging adapter weights.

Thanks a lot
really appreciate your work

config.json: 100% 638/638 [00:00<00:00, 3.36MB/s]
Downloading shards: 100% 8/8 [00:00<00:00, 16.25it/s]
Loading checkpoint shards:  38% 3/8 [00:27<00:45,  9.16s/it]Traceback (most recent call last):
File "/usr/local/bin/accelerate", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/accelerate_cli.py", line 47, in main
args.func(args)
File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/launch.py", line 1017, in launch_command
simple_launcher(args)
File "/usr/local/lib/python3.10/dist-packages/accelerate/commands/launch.py", line 637, in simple_launcher
raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '-m', 'autotrain.trainers.clm', '--training_config', 'quip_v2/training_params.json']' died with <Signals.SIGKILL: 9>.

now i m getting this before i got success not got this error

that's out of memory error.

btw, if you dont have a big gpu, you can also merge after training the model. you can either use the space: https://huggingface.co/spaces/autotrain-projects/llm-merge-adapter or just copy paste its code.

i tried that as well not working

⚠ WARNING | 2023-12-21 15:29:10 | main:train:471 - Failed to merge adapter weights: local variable 'active_adapters' referenced before assignment
⚠ WARNING | 2023-12-21 15:29:10 | main:train:472 - Skipping adapter merge. Only adapter weights will be saved.
🚀 INFO   | 2023-12-21 15:29:10 | main:train:481 - Pushing model to hub...

whats your base model? i can try it locally to reproduce

HuggingFaceH4/zephyr-7b-beta

i just trained zephyr beta and was able to merge it just fine. Not sure why you are getting the error. make sure of transformer and peft versions:
peft==0.7.1
transformers==4.36.1
However, just now i looked at kam414/quip-v1 and it does seem to have the merged weights 🤔

if you have manually added config.json and pytorch-model-*.bin files, please remove them before merging.

this merge i did via https://huggingface.co/spaces/autotrain-projects/llm-merge-adapter
and may be google colab pro will not work in my case

https://huggingface.co/spaces/autotrain-projects/llm-merge-adapter did you duplicate this and attach a gpu to it?

nope i just run this

please follow the instructions my friend :)
you can also do it in colab pro. try this:
pip install peft==0.7.1
pip install transformers==4.36.1

import torch
from peft import PeftModel
from transformers import AutoModelForCausalLM, AutoTokenizer

def merge_adapter(base_model_path, target_model_path, adapter_path):
    logger.info("Loading adapter...")
    model = AutoModelForCausalLM.from_pretrained(
        base_model_path,
        torch_dtype=torch.float16,
        low_cpu_mem_usage=True,
        trust_remote_code=True,
    )

    model = PeftModel.from_pretrained(model, adapter_path)

    tokenizer = AutoTokenizer.from_pretrained(
        base_model_path,
        trust_remote_code=True,
    )
    model = model.merge_and_unload()

    logger.info("Saving target model...")
    model.save_pretrained(target_model_path)
    tokenizer.save_pretrained(target_model_path)

Anyways, I cloned your model to my profile, duplicated the llm-merge-adapter space with a GPU and was able to successfully merge the model. Here it is: https://huggingface.co/abhishek/quip-v1/tree/main
It took 2mins.

@abhishekkrthakur  can you transfer this repo to me
thanks in advance 🙂

please use: https://huggingface.co/spaces/huggingface-projects/repo_duplicator
no need to duplicate this space ;)

WOOW this is awesome
thanks a lot

@abhishekkrthakur  one last question and please sorry to bother you.
when i again finetune kam414/quip-v1 from kam414/quip-v2 then it's kam414/quip-v2 base model still shows HuggingFaceH4/zephyr-7b-beta
https://huggingface.co/kam414/quip_v2/blob/main/adapter_config.json
so it means no matter how many time i train the model base model will be always HuggingFaceH4/zephyr-7b-beta is this Correct me if i m wrong
if yes then everytime i finetune i need to merge with HuggingFaceH4/zephyr-7b-beta ?

it will show the same name because in v1's config.json, its named as such.
you'll need to manually change the name but keep the same type for the model. ill try it myself then i can give you more info.


Has capability test

This PR adds some tests for the hasCapability method. These were things that I was curious about when I was looking at how to validate permissions via UCAN. There is still one question I had, about the originator field. I made a test that currently fails -- https://github.com/nichoth/ts-ucan/blob/f2fde61b72adcd0a434e5ce1ba91cfe1d834fba1/tests/attenuation.test.ts#L296 -- the question is how is originator used? It doesn't seem to affect the return value of hasCapability. I'm not sure if this is not implemented yet or if my test is wrong in some way.
Thanks. Hopefully this is helpful and not creating needless work 😬

Thanks, that's an amazing contribution.

the question is how is originator used? It doesn't seem to affect the return value of hasCapability. I'm not sure if this is not implemented yet or if my test is wrong in some way.

Yes, this is a bug. Whoops! hasCapability should check the originator to make sure it matches what's passed in and also check that the time bounds work out. All of that instead of running delegateCapabilityInfo... My bad.
Your contribution is amazing. You've created a failing test case. We'll use that to implement a fix for this.

@nichoth Any chance you can rebase this on main? It should only be two merge conflicts, because the path of ./emailCapabilities has changed to ./capabilities/email, and I changed emailCaps = Array.from( to be multiline.
Would love to get this in so we can fix hasCapability 🙏

I just fixed the type checking with the updates, however there is one type I wasn't sure about -- https://github.com/nichoth/ts-ucan/blob/hasCapability-test/tests/attenuation.test.ts#L234
It is telling me
Property 'email' does not exist on type 'Capability'.ts(2339)

@nichoth this PR is almost good to go. I proposed some changes to this PR in another PR (see the mention above this comment :point_up_2: ).
Once we've merged that PR, we can merge this PR :tada:

Looks good! 👏


Fix contextual removal for events with only one listener

This fixes this issue:
var e = new EventEmitter()
  , context = { foo: 'bar' };

function foo() {}
e.on('foo', foo, context);

e.removeListener('foo', foo, { baz: 'quux' });

// e.listeners('foo').length === 0 but the context is different

Coverage remained the same at 100.0% when pulling 319353da488220d43427eb8d6368d4fccd0d745d on fix-remove-by-context into 0b185f254d1ee020206abf9b0bd01c8d19c99397 on master.

Another issue not handled here is this:
var e = new EventEmitter()
  , context = null;

function foo() {}
e.on('foo', foo, context);
e.on('foo', foo, { foo: 'bar' });

e.removeListener('foo', foo, context);

e.listeners('foo').length === 1 // false, because when context is falsy all listeners are removed

@lpinca so is that issue mentioned above fixed or still open?

As you pointed out the context can't be a falsy value, so ignore that comment.


New sphinx builder from 0.5.0 is compatible with readthedocs extension.

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/sphinx/cmd/build.py", line 284, in build_main
    app.build(args.force_all, filenames)
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/sphinx/application.py", line 337, in build
    self.builder.build_update()
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 294, in build_update
    self.build(to_build,
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 358, in build
    self.write(docnames, list(updated_docnames), method)
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 524, in write
    self.prepare_writing(docnames)
  File "/home/docs/checkouts/readthedocs.org/user_builds/subprojecttoctree-example-subproject/envs/latest/lib/python3.8/site-packages/subprojecttoctree/builders.py", line 10, in prepare_writing
    last_master_entry = self.env.last_master_entry
AttributeError: 'BuildEnvironment' object has no attribute 'last_master_entry'

@DriesSchaumont I'm still getting this error, both locally and on readthedocs, using sphinx-subprojecttoctree v5.4 and sphinx v7.2.6. I can't see any reference to last_master_entry anywhere in the Sphinx BuildEnvironment class.

Update I've worked out that this happened because the only item in the master toctree was a reference to the sub-project. You may want to guard against this (admittedly unlikely) scenario.

Hi @Iain-S, thanks for letting me know! I will create a seperate issue to track this.


Add space

I was going through django documents and I found a minor error regarding a space on this address. [https://docs.djangoproject.com/en/4.1/ref/contrib/messages/#displaying-messages ]
Check it out!

Thanks, but these changes are incorrect. It's fine as it is.


Course

What does this PR do?
Add resource(s)
For resources
YouTube videos , website courses ,
books , competitive programming platform can be used
Description
This pull request is created for resources for data structure and algorithms . This help people without any prior knowledge of coding learn these topic easily .
Why is this valuable (or not)?
Yes , it is valuable
How do we know it's really free?
Because resources provided free of cost by different platform across the world are used.
For book lists, is it a book? For course lists, is it a course? etc.
Book and course both
If accepted please mention "Hacktoberfest accepted"
Checklist:

[✓ ] Read our contributing guidelines.
[ ✓] Search for duplicates.
[✓ ] Include author(s) and platform where appropriate.
[✓ ] Put lists in alphabetical order, correct spacing.
[✓ ] Add needed indications (PDF, access notes, under construction).
[ ✓] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

This time it doesn't contain any link then why it is tagged spam .

@sohansai

Refer to the guidelines in docs/CONTRIBUTING.md for instructions on how to contribute. If you're new to GitHub or uncertain about the process, you can check this link

I had already gone through the contribution guidelines and then created this pull request . That is why inam asking because there is not in the pull request to be tagged ' spam '.

@sohansai

It seems there might be some confusion.
Your pull request appears to contain previously committed changes and a large number of commits. The 462 commits you're attempting to merge have already been added to the repository. It's crucial to understand GitHub's contribution process.
I've tagged it as spam, as duplicating commits could clutter the repository.
I kindly recommend familiarizing yourself with GitHub practices before contributing further. The pull request has been tagged as spam to prevent unintended duplications.

@eshellman please look into this matter

It appears you forked the "cookie-cutters" branch from the "free-programming-books" repo and intend to merge it. Please consider forking the main branch from the repo and contributing from there.

It appears you're trying to merge commits into EbookFoundation:cookie-cutter. Instead, merge them into the main branch.


feat: implement parse.map for mapping RIS tags to human-friendly names

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


BUG: Three dots menu is showing repetitive content as in navbar and  background is visible through menu

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hello @arghadipmanna101 , The navbar is not working properly. On hovering over three dots, a menu opens but background is visible through menu. Also, the menu shows repetitive elements as in navbar.

Kindly assign me this issue under GSSoC'24, I'll do my best!
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hello @arghadipmanna101 , I have made a PR on this issue. Kindly review it.


Plugin is not working completely / not printing anything just keeping Loading Preview

Version information-
PLugin Version: 0.8.0
iOS: iOS14
Xcode: v12
Device info (Checked in devices)

iPhone XR
iPad Pro

Bug-
Printer keeps loading and not proceed completely for printing.

After some further debugging it came that it's working for PDF only.
While printing other file types, Here's error I am getting - failed to find PDF header: '%PDF' not found
@katzer - Is any short workaround to fix this for all file types? Earlier in previous version of plugin it was working for printing all types of files, such as Docs/CSV/Excel files etc..

Hello Anuj ,
I have not faced #266 . But to fix my crash in https://github.com/katzer/cordova-plugin-printer/issues/264 , i used html2pdf to convert the html to pdf in base 64 format , and then i passed this base 64 format to the printer plugin.
Can you try this too?

Thanks @kirtipriya . But My case is something different. I want to print all types of files which were able to be printed in plugin's version 0.7.3 including - docs,sheet etc. which seems not supported in latest version of plugin.

@Anuj-Raghuvanshi ,
I am also facing this issue but i fixed,
when i was using Version information-
PLugin Version: 0.7.3
iOS: iOS14
Xcode: v12
I faced app-crash when i was click on Cancel.?
Solution: I upated PLugin Version: 0.8.0 app crash is fixed.
After that i faced continuously loading-preview issue?
Solution:
cordova.plugins.printer.print('data:application/pdf;base64,' + data);
Replace like this it will work fine cordova.plugins.printer.print('base64://' + data);


Fixes a bug in protobuf_json's deserialization feature

Fixes a bug in protobuf_json's deserialization feature
参考:Fixes a bug in gson's deserialization feature #5099

Codecov Report

Merging #5098 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5098      +/-   ##
============================================
- Coverage     64.06%   64.03%   -0.04%
- Complexity      452      453       +1
============================================
  Files           769      769
  Lines         33261    33261
  Branches       5249     5249
============================================
- Hits          21309    21299      -10
- Misses         9520     9528       +8
- Partials       2432     2434       +2

Impacted Files
Coverage Δ
Complexity Δ

.../dubbo/rpc/protocol/dubbo/DecodeableRpcResult.java
66.17% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../apache/dubbo/qos/protocol/QosProtocolWrapper.java
65.85% <0%> (-17.08%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (-6.53%)
0% <0%> (ø)

.../rpc/protocol/dubbo/LazyConnectExchangeClient.java
56.47% <0%> (-2.36%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

...pache/dubbo/remoting/transport/AbstractServer.java
47.91% <0%> (-1.05%)
0% <0%> (ø)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
61.87% <0%> (-0.63%)
29% <0%> (ø)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
90.65% <0%> (+0.93%)
17% <0%> (+1%)
:arrow_up:

.../dubbo/monitor/support/AbstractMonitorFactory.java
81.08% <0%> (+2.7%)
0% <0%> (ø)
:arrow_down:

...onfig/spring/extension/SpringExtensionFactory.java
85.36% <0%> (+4.87%)
0% <0%> (ø)
:arrow_down:

... and 1 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 79c6166...9d4d1e5. Read the comment docs.

Thanks, but this has been fixed.


feature: Ingest MBTA GTFS to S3 and DynamoDB

This PR adds a new task to the ingestor to bring static GTFS into our systems, in two different ways:

As .sqlite3 database files in the tm-gtfs bucket
As entries in the TripCounts DynamoDB table

Most of the heavy lifting is done by our new mbta-gtfs-sqlite library. The code here is responsible for querying the sqlite files it generates and counting the total number of trips for each route for each day.
Test plan:
Works when deployed and spot-checks indicate the data matches the Covid Recovery Dash

What data should be available? I was looking at the red line on dynamo and there's an 8 day span from 12/27/17 to 1/3/18
then nothing until 2019, but all of 2019 has data it seems.

We probably want the table definitions in the config, like this
https://github.com/transitmatters/data-ingestion/blob/29bb24748601250c54ff75813f6f23683ef8d82f/ingestor/.chalice/dynamo_tables.json

The byHour tables are by service date right? not actual date?

The byHour object is by service date right? not actual date?

The byHour field on a TripCounts table entry holds an array of size 24 with the total trips in count broken down by each hour. It does correspond to the actual date in that entry's date field.


Theme Creator with filters

Solution/Idea
It will be good to add custom filters to a image like hue, saturation, and more like photoshop so we can create theme colors easly.
I got this idea from color blindness filter

Original Theme

After some filters got Mint Colors

If anyone know any method to extract different colors from a theme like Nord to Mint, Strawberry please share.
Alternatives
Changing a set of hex code to a different hex code so it results in different theme color.

Can't Figma, Gimps, Paint.NET do this for free?


Fix SetLightDark fpr Relative Strings

There was one more Abslout Path. Replacing this also with OriginalString solved the problems on Issue #174 for me.
Sorry I never before created a PullRequest. Now the fix fpr SetLightDark and the FloatingStyle for the PasswordBox are together in this Request. Tried to revert it but that created only more Commits =(
Next time I will branch for each feature =)

Good job, thank you.


Added nix-darwin module

This has been working for me. I'm sure this could be refactored to reduce duplication between it and the nixos module, but I haven't bothered.

Thanks for being patient :-)
I think the code look pretty good as-is. I just added a comment about escaping but otherwise I'm fine with merging this.
I think it would be nice to have a corresponding news entry, though. You could just the one I wrote for the NixOS module as a model and just change where necessary.

@ElvishJerricco Please have a look at the comments I added. It would be very nice to be able to merge this :-)

@rycee I just pushed a change the avoids the need for escaping altogether. I'll add a news entry before you merge this.
The only other thing I might worry about is that none of the services in home manager should be expected to work; i.e. this is pretty much exclusively for the home.file options. It'd be good if we could outright disable services in the nix-darwin module.

Looks good to me! If you add a news entry I'll be happy to merge.
About the modules not being supported I understand the issue. I've been thinking that it would be good to have a platforms field in the module to say which platforms the module supports. But this would need to be an addition in Nixpkgs. When I have time I will start such a discussion there.

A platforms field in the module system sounds like a great idea!

@ElvishJerricco If you have the chance to add a news entry I would love to have this merged.

@rycee Done, and I also fixed home.files symlinks being owned by root.


feat(FTL-8017): added windows platform to test stage for github actions

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


Fixed #939: [wallet] cursor pointer for go back link

What type of PR is this?
bug
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #939
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

please double check all places

please double check all places

I had to rechecked. And I'm not sure what do you mean for all tag places?

please check all the other  tag and  node places and see they have cursor pointer

please check all the other tag and node places and see they have cursor pointer

resolved


Remove alternate FMS & CRTM versions from unified-dev

Editing this issue to cover both FMS and CRTM.
FMS and CRTM currently have alternate versions in unified-dev, which, among other things, can trigger concretization bugs. It would be lovely to get to a point where we can see unify: true.
Relatedly:
The following code appears in fms/package.py:
    with when("@release-jcsda"):
        git = "https://github.com/JCSDA/fms.git"

The with when() construct does not function as a conditional statement, so this should be corrected as soon as possible.

@AlexanderRichert-NOAA I don't think we'll be able to avoid multiple CRTM versions for a very long time (until everyone uses CRTM v3 - GFS v18 for the UFS?). For FMS, I am hopeful that once https://github.com/NOAA-GFDL/FMS/pull/1225 is merged, we have the opportunity to remove the different FMS versions and work with the latest FMS release in all our environments.

The FMS part of this issue is blocked by https://github.com/NOAA-GFDL/FMS/pull/1225

Pushing the can down the road to spack-stack-1.7.0 ...

@BenjaminTJohnson We added you here to check what the timeline/prospect of moving to crtm v3 everywhere is. If that's not feasible within the next year or so I suggest with close this issue after the FMS unification is done (hopefully soon!)

@climbfuji do you mean to ask when we're deprecating v2.x versions and only supporting v3.x?

For spack-stack, we need to know when we can stop installing crtm@v2 with each release - probably not for a while because some operational implementations will use crtm@v2?

@climbfuji yes, my best guess would be 2-3 years before we stop supporting v2.4.0 operationally.

We can now remove fms@release-jcsda from the templates and the JEDI packages, and have our package match spack develop. @srherbener Can you take care of this, please?

The fms divergence is now history ... all that remains is crtm.

For spack-stack-1.9.0, we still need different versions of CRTM. I will move this to the following release of spack-stack, with some luck everbody will be on the same version of CRTM 3.x.y by then.


Can keyd turns mouse movement into key presses?

Hi again and thank you for reading this issue.
I wonder if there's currently a way for keyd to translate mouse movement into keyboard presses, since sudo keyd -m can detect mouse movement.
Thank you very much!

Not at the moment. This is planned for a future release. In the meantime you might be interested in my other project, warpd.

Gotcha. Thank you for the response.


Set up Docker Desktop with Kubernetes

Subsumed into https://app.zenhub.com/workspaces/unity-analytics-6230deb25d28ca001efd6d6f/issues/unity-sds/unity-analytics/10


This is a test to make sure the new pipeline is the only one that runs

Please do not merge this PR, it is just a test to confirm that only the new pipeline runs on PRs

Summary

Cool, works fine... closing.


fail to vagrant up with  error: unable to create symlink test/unit/test_locale/eo/LC_MESSAGES/swift.mo (Protocol error)

hi,
when trying a clean vagrant up with latest master it fails while cloning swift. It seems to be a failure to create symlinks, possibly because of vboxsf not allowing symlinks?
==> default: * execute[git swift] action run
==> default:
==> default:
==> default:
==> default: ================================================================================
==> default:
==> default: Error executing action `run` on resource 'execute[git swift]'
==> default:     ================================================================================
==> default:
==> default:     Mixlib::ShellOut::ShellCommandFailed
==> default:     ------------------------------------
==> default:     Expected process to exit with [0], but received '128'
==> default:     ---- Begin output of git clone -b master git://github.com/openstack/swift.git ----
==> default:     STDOUT:
==> default:     STDERR: Cloning into 'swift'...
==> default:     error: unable to create symlink test/unit/test_locale/eo/LC_MESSAGES/swift.mo (Protocol error)
==> default:     fatal: unable to checkout working tree
==> default:     warning: Clone succeeded, but checkout failed.
==> default:     You can inspect what was checked out with 'git status'
==> default:     and retry the checkout with 'git checkout -f HEAD'
==> default:     ---- End output of git clone -b master git://github.com/openstack/swift.git ----
==> default:     Ran git clone -b master git://github.com/openstack/swift.git returned 128
==> default:
==> default:     Resource Declaration:
==> default:     ---------------------
==> default:     # In /tmp/vagrant-chef/c34c6bd56bb7e58c7ba4937358ce33a6/cookbooks/swift/recipes/source.rb
==> default:
==> default:      33: execute "git swift" do
==> default:      34:   cwd "/vagrant"
==> default:      35:   command "git clone -b #{node['swift_repo_branch']} #{node['swift_repo']}"
==> default:      36:   creates "/vagrant/swift"
==> default:      37:   action :run
==> default:      38: end
==> default:      39:
==> default:
==> default:     Compiled Resource:
==> default:     ------------------
==> default:     # Declared in /tmp/vagrant-chef/c34c6bd56bb7e58c7ba4937358ce33a6/cookbooks/swift/recipes/source.rb:33:in `from_file'
==> default:
==> default:     execute("git swift") do
==> default:       action [:run]
==> default:       retries 0
==> default:       retry_delay 2
==> default:       default_guard_interpreter :execute
==> default:       command "git clone -b master git://github.com/openstack/swift.git"
==> default:       backup 5
==> default:       creates "/vagrant/swift"
==> default:       cwd "/vagrant"
==> default:       returns 0
==> default:       declared_type :execute
==> default:       cookbook_name :swift
==> default:       recipe_name "source"
==> default:     end
==> default:
==> default:
==> default: [2015-11-17T14:35:19+00:00] INFO: Running queued delayed notifications before re-raising exception
==> default:
==> default: Running handlers:
==> default: [2015-11-17T14:35:19+00:00] ERROR: Running exception handlers
==> default: Running handlers complete
==> default:
==> default: [2015-11-17T14:35:19+00:00] ERROR: Exception handlers complete
==> default: Chef Client failed. 29 resources updated in 05 minutes 57 seconds
==> default: [2015-11-17T14:35:19+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
==> default: [2015-11-17T14:35:19+00:00] ERROR: execute[git swift] (swift::source line 33) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '128'
==> default: ---- Begin output of git clone -b master git://github.com/openstack/swift.git ----
==> default: STDOUT:
==> default: STDERR: Cloning into 'swift'...
==> default: error: unable to create symlink test/unit/test_locale/eo/LC_MESSAGES/swift.mo (Protocol error)
==> default: fatal: unable to checkout working tree
==> default: warning: Clone succeeded, but checkout failed.
==> default: You can inspect what was checked out with 'git status'
==> default: and retry the checkout with 'git checkout -f HEAD'
==> default: ---- End output of git clone -b master git://github.com/openstack/swift.git ----
==> default: Ran git clone -b master git://github.com/openstack/swift.git returned 128
==> default: [2015-11-17T14:35:19+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
Chef never successfully completed! Any errors should be visible in the
output above. Please fix your recipes so that they properly complete.

the quick and lame workaround by the way is to checkout swift-specs outside of /vagrant to get things going
diff --git a/cookbooks/swift/recipes/source.rb b/cookbooks/swift/recipes/source.rb
index a16511c..96e76f2 100644
--- a/cookbooks/swift/recipes/source.rb
+++ b/cookbooks/swift/recipes/source.rb
@@ -38,9 +38,9 @@ execute "git swift" do
 end

 execute "git swift-specs" do
-  cwd "/vagrant"
+  cwd "/tmp"
   command "git clone -b #{node['swift_specs_repo_branch']} #{node['swift_specs_repo']}"
-  creates "/vagrant/swift-specs"
+  creates "/tmp/swift-specs"
   action :run
 end

@@ -79,7 +79,7 @@ execute "python-swift-install" do
 end

 execute "swift-specs-install" do
-  cwd "/vagrant/swift-specs"
+  cwd "/tmp/swift-specs"
   command "pip install -r requirements.txt"
   action :run
 end

... testing

Hrmm... WOMM
$ VBoxManage --version
4.2.30r100415
$ vagrant --version
Vagrant 1.6.5

Did you make any changes to your localrc?

I'm assuming that's localrc-template in the repo root? no changes there.
I'm running more recent vbox/vagrant versions though on a debian jessie host. I'll try downgrading vbox and see if that changes things...
$ VBoxManage --version
5.0.4_Debianr102546
$ vagrant --version
Vagrant 1.7.4
$ uname -a
Linux i7 4.2.0-0.bpo.1-amd64 #1 SMP Debian 4.2.5-1~bpo8+1 (2015-11-02) x86_64 GNU/Linux

nah it's cool - let me upgrade

Hrmm... still working for me
$ VBoxManage --version
5.0.10r104061
$ vagrant --version
Vagrant 1.7.4

I guess it's probably something to do with me running on a mac and you're on linux?!
It's weird for me that in your error message the clone command failing seemed to be on the swift repo, but in your suggested fix (patch/diff) you only had to modify swift-specs (!?).
It's pretty useful to clone the repo in /vagrant because then you can edit the source from your host - it's one of the killer features of using vagrant (virtualbox shared file system).  But I guess if it's not working for you that's pretty terrible.  Maybe a cleaner hack would be to add a localrc option for SOURCE_ROOT which defaults to /vagrant - but you can set to /src or some other path on the guest os if that works better for you?  I know some folks do all their editing/commits on the guest vm anyway - and in that case it's probably better to have the code on the vm's filesystem instead of the vboxsf junk.  Plus having the source root configurable might open the door to doing something with other network attached filesystems or something else that people might find generally useful.
Let me know if you think something like a configurable source root would work for you - and if you'd be willing to take a stab at it?

very possible it is osx vs linux indeed! yup, SOURCE_ROOT variable seems like a viable option, I'll take a stab at it :+1:


Add a configurable minumum Boost amount

Currently, I can send a large amount of small boosts which would cause a mess to scroll through as well the notification sound to be played an obnoxious amount of times. A 1000 boosts each 1 sat is not very expensive. If we had a configurable minimum amount the user could make the price of boost and/or notification too expensive to be spamy. I think this would be configurable via environmental variables and the configuration file. Configuration via the web ui should also be considered.

A lot of these admin-type features might call for some sort of settings dashboard in the long term.

Yeah I hope to not have to do that for a while.

Yeah I hope to not have to do that for a while.

Hopefully between a few of us just dipping our toes in right now we can offload all of these frond end/UX concerns so you don't have to do everything every time <3
I hope I don't sound like I'm featurebegging in these posts, just trying to brainstorm stuff. Anything I suggest isn't with the expectation that someone else comes along and does it. I just need to play with some stuff and work up to it.
You have given us so much, Dave! We wanna give back.

Not at all Spencer. All good brother. 🙏
On Jan 26, 2022, 5:07 PM -0600, Spencer @.***>, wrote:

Yeah I hope to not have to do that for a while.
Hopefully between a few of us just dipping our toes in right now we can offload all of these frond end/UX concerns so you don't have to do everything every time <3
I hope I don't sound like I'm featurebegging in these posts, just trying to brainstorm stuff. Anything I suggest isn't with the expectation that someone else comes along and does it. I just need to play with some stuff and work up to it.
You have given us so much, Dave! We wanna give back.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

Yeah I hope to not have to do that for a while.

Then lets not and implement the settings without changes to the UI.
I am not sure what configuration on umbrel looks like but people on Raspiblitz are willing to ssh in and edit the configuration file and restart the service.


Access Controls from Menu Item

Hi there.
Amazing work on the new Access controls features!
I'm wondering if there is any way to utilize the Show/Hide buttons methods right from the Menu Item call?
MenuSection::make('File Manager')->path('/nova-file-manager')
I'm using this primarily as a read-only file browser/download for one of my projects.
Thank you!

Hi !
Unfortunately this is out of the scope of the package, I would recommend checking out Laravel Nova's documentation on how to handle this use case.
All the best


[BUG] Protocol headers shifted to the right

Describe the bug
After inserting a protocol through the upload protocol tool, the header is shifted one row to the right.
In the picture: The header peptide digestion should span the three parameters instead of the sample name

To Reproduce

Create new excel spreadsheet and load swate manifest.xml as add in
Create new annotation table in swate
Click on cell Source Name
Load a protocol using the protocol insert function in swate

Expected behavior
The header should be spanned above the three parameters
OS and framework information (please complete the following information):

Excel online
Swate 0.3.0

Additional context
The CustomXML is correct, the problem seems to occur when rendering the information

This is a breaking bug, i will immediatly start working on it!
For now you can circumvent the bug by adding the table in EXACTLY row B.

I have fixed the bug and will move it to production. Will update this issue as soon as it is live.

Fixed and should be live!


Name of the database when exception is logged

We have multiple database hosted on the server and when there is an exception, the log message don’t have name of the database that generated the exception. Can we add the database name as part of the exception logging

:+1:

I think is related to #2468

@fabriziofortino yes, I remember this old issue. It's pretty the same.

The #2468 is the same. It is very critical to manage multiple database in the same server


Adds Chinese translation of v1.0.0-rc.2

Hi all,
I added a Chinese version of the the TOML v1.0.0-rc.2 , based on the current latest one (cn/1.0.0-rc.1).
I have seen some discussions on the review process of translation. The previous Chinese versions were written by @LongTengDao, @chenrui333 and @ZhangHanDong, and could you review my pull request, thanks so much.
I have also translated some documents of the Rust language, such as Rust By Example. For the Chinese translation and review of TOML project, maybe I can participate in the future, thank you.

@aaranxu 辛苦，大致看了下，没有啥问题。

lgtm，辛苦了

Thanks for this @aaranxu and for the reviews @chenrui333 @ZhangHanDong! Much appreciated! ^>^