Fix service loading for WebLogic <IP_ADDRESS>.0

The issue has to do with how ServiceLoaders behave on that particular version of WebLogic. I'm not sure how WebLogic can even mess with such low-level stuff...
When ServiceLoaders are asked to load a resource for the bootstrap class loader (which is represented with null), they actually use ClassLoader.getSystemClassLoader().loadClass(String) and rely on the parent delegation to load the classes from the bootstrap CL.
Our agent is actually present on both the system and the bootstrap CL (which is not ideal tbh) and on this specific WebLogic version, the boot delegation for ClassLoader.getSystemClassLoader().loadClass(String) does not seem to work for some reason. The workaround is to load the classes via java.lang.Class#forName(java.lang.String, boolean, java.lang.ClassLoader) which allows the ClassLoader to be null which then reliably loads the classes from the bootstrap classloader.

Codecov Report

Merging #649 into master will decrease coverage by 2.82%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #649      +/-   ##
============================================
- Coverage     64.77%   61.95%   -2.83%
  Complexity       68       68
============================================
  Files           195      191       -4
  Lines          8088     7338     -750
  Branches       1025      858     -167
============================================
- Hits           5239     4546     -693
+ Misses         2547     2511      -36
+ Partials        302      281      -21

Impacted Files
Coverage Δ
Complexity Δ

...m/agent/util/DependencyInjectingServiceLoader.java
77.77% <66.66%> (-0.41%)
0 <0> (ø)

...lastic/apm/agent/impl/ElasticApmTracerBuilder.java
73.21% <66.66%> (-5.8%)
0 <0> (ø)

...va/co/elastic/apm/agent/impl/payload/Language.java
66.66% <0%> (-33.34%)
0% <0%> (ø)

...co/elastic/apm/agent/impl/payload/RuntimeInfo.java
66.66% <0%> (-33.34%)
0% <0%> (ø)

...iguration/PrefixingConfigurationSourceWrapper.java
41.66% <0%> (-16.67%)
0% <0%> (ø)

...astic/apm/agent/report/ApmServerHealthChecker.java
43.58% <0%> (-12.78%)
0% <0%> (ø)

.../apm/agent/report/serialize/DslJsonSerializer.java
83.23% <0%> (-6.39%)
0% <0%> (ø)

...pm/agent/report/IntakeV2ReportingEventHandler.java
76.23% <0%> (-5.88%)
0% <0%> (ø)

...a/co/elastic/apm/agent/report/ReporterFactory.java
79.16% <0%> (-5.69%)
0% <0%> (ø)

...m/agent/jdbc/PreparedStatementInstrumentation.java
47.61% <0%> (-3.9%)
0% <0%> (ø)

... and 8 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 70cde50...b000a89. Read the comment docs.

TIL: it's possible to override the system class loader via the system property java.system.class.loader, which is what this WebLogic version does. See https://docs.oracle.com/javase/7/docs/api/java/lang/ClassLoader.html#getSystemClassLoader()


Faster CI

We could make the CI faster using something similar to Synthetify's approach https://github.com/Synthetify/synthetify-protocol/blob/master/.github/workflows/test.yml
Their CI runs in a few minutes because it's more parallel (GH actions offers up to 20 parallel jobs on the free plan) and because they do a bunch of caching.

@armaniferrante are you ok with CI migration to GitHub actions?

@armaniferrante are you ok with CI migration to GitHub actions?

Fine with me! Do we need to pay for anything? I've never used it before, admittedly.


Is latest build broken?

Hi, I am trying this library and when I add all the dependencies and refresh gradle project only requery android is available, is the build broken or is the problem somewhere in my app?
Thanks

Should be working fine. What environment are you using? try gradle assemble on the terminal and see if that works. If so the problem is likely an IDE setup issue, If you are using Intellij make sure you have JDK 8 and are using Intellij IDEA 15.

Also if you are just adding it as a gradle dependency can you paste the error that you get from gradle? Thanks.

I am using Android with Kotlin and Android Studio, the problem is that I am not able to import anything from the base requery artifact, it doesn't appear in the External libraries tab in the Studio, while the artifact does.
I have this in my dependencies, is it correct?
    compile 'io.requery:requery:1.0-SNAPSHOT'
    compile 'io.requery:requery-android:1.0-SNAPSHOT'
    apt 'io.requery:requery-processor:1.0-SNAPSHOT'

Gradle build normally works. Is there a problem with required Java version as I am still using the default 6 for Android development?

Those are the correct dependencies. Have you added http://oss.jfrog.org/artifactory/oss-snapshot-local to your repositories? Currently only snapshots are available, there will be also a release to bintray soon.
repositories {
    jcenter()
    maven { url 'http://oss.jfrog.org/artifactory/oss-snapshot-local' }
}

If that doesn't work, can you copy paste the error from gradle output? Thanks

I have added the repository, it builds fine, but classes from the main artifact are not available. Gradle output si here, although there are no errors.
:app:preBuild UP-TO-DATE
:app:preDebugBuild UP-TO-DATE
:app:checkDebugManifest
:app:preReleaseBuild UP-TO-DATE
:app:prepareComAndroidSupportAppcompatV72311Library UP-TO-DATE
:app:prepareComAndroidSupportDesign2310Library UP-TO-DATE
:app:prepareComAndroidSupportRecyclerviewV72310Library UP-TO-DATE
:app:prepareComAndroidSupportSupportV42311Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbinding030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingAppcompatV7030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingAppcompatV7Kotlin030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingDesign030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingDesignKotlin030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingKotlin030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingSupportV4030Library UP-TO-DATE
:app:prepareComJakewhartonRxbindingRxbindingSupportV4Kotlin030Library UP-TO-DATE
:app:prepareIoReactivexRxandroid110Library UP-TO-DATE
:app:prepareIoRequeryRequeryAndroid10SNAPSHOTLibrary UP-TO-DATE
:app:prepareDebugDependencies
:app:compileDebugAidl UP-TO-DATE
:app:compileDebugRenderscript UP-TO-DATE
:app:generateDebugBuildConfig UP-TO-DATE
:app:generateDebugAssets UP-TO-DATE
:app:mergeDebugAssets UP-TO-DATE
:app:generateDebugResValues UP-TO-DATE
:app:generateDebugResources UP-TO-DATE
:app:mergeDebugResources UP-TO-DATE
:app:processDebugManifest UP-TO-DATE
:app:processDebugResources UP-TO-DATE
:app:generateDebugSources UP-TO-DATE
:app:preDebugAndroidTestBuild UP-TO-DATE
:app:prepareDebugAndroidTestDependencies
:app:compileDebugAndroidTestAidl UP-TO-DATE
:app:processDebugAndroidTestManifest UP-TO-DATE
:app:compileDebugAndroidTestRenderscript UP-TO-DATE
:app:generateDebugAndroidTestBuildConfig UP-TO-DATE
:app:generateDebugAndroidTestAssets UP-TO-DATE
:app:mergeDebugAndroidTestAssets UP-TO-DATE
:app:generateDebugAndroidTestResValues UP-TO-DATE
:app:generateDebugAndroidTestResources UP-TO-DATE
:app:mergeDebugAndroidTestResources UP-TO-DATE
:app:processDebugAndroidTestResources UP-TO-DATE
:app:generateDebugAndroidTestSources UP-TO-DATE

BUILD SUCCESSFUL

Total time: 5.905 secs

I am sorry for this inconvience, it seem like it was just a bug in Android Studio, after restarting it, it works fine.

Sure no problem glad it's working!


Riot configuration link to customise HS

From riot-meta created by manuroe: vector-im/riot-meta#307
This link will open the app and set up the homeserver URL (and the identity server URL while it is still required).
doc:  https://docs.google.com/document/d/12lYrrjP9q5BXghJICYTo0gvHMewGrXGznoozMRLPpMw

Done in #3296


Add ability to set a custom client name

I noticed that PathPlanner would show up as something like DartClient_XXXXXXX, so I figured I would add a way for it to have a custom name. If no name is provided, then it defaults to DartClient_XXXXXXX.

This resolves https://github.com/mjansen4857/nt4/issues/11 correct?

Didn't even know this PR was here until now... oops.


Gradle precompileJte always runs even if nothing has changed

On my system - if I run the precompileJte task then it doesn't seem to take into account that the inputs and outputs have not changed.
From the gradle log:
> Task :app:compileKotlin UP-TO-DATE
> Task :app:compileJava NO-SOURCE
> Task :app:precompileJte
> Task :app:processResources UP-TO-DATE
> Task :app:classes
> Task :app:inspectClassesForKotlinIC UP-TO-DATE
> Task :app:jar UP-TO-DATE

I've had a quick look at your plugin code and it looks fine.
I've also changed the output directory to something that's not used/changed and the problem still occurs.
I'll try to have a better look when I have more time :)

Thanks for reporting and looking into it 👍


[palm-secure-backend] Missing code samples

The samples for the palm-secure-backend do not point to real locations. The links used are here and here.

What are the real locations?

Links:
https://github.com/google/generative-ai-docs/tree/main/demos/palm/web/quick-prompt
https://github.com/google/generative-ai-docs/tree/main/demos/palm/web/list-it


Cleanup extra spaces

Needs to be fixed in the scraper code
Fixes #65

Closing this in favor of #67


SqlScript/SqlScriptQuery: Added localization

Pull Request (PR) description

Changes to SqlScript

Added en-US localization (issue #624).
Added additional unit tests for code coverage.

Changes to SqlScriptQuery

Added en-US localization.
Added additional unit tests for code coverage.

This Pull Request (PR) fixes the following issues

Fixes #624

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #1324 into dev will decrease coverage by <1%.
The diff coverage is 84%.

@@         Coverage Diff          @@
##            dev   #1324   +/-   ##
====================================
- Coverage    98%     98%   -1%
====================================
  Files        36      36
  Lines      4836    4862   +26
====================================
+ Hits       4762    4784   +22
- Misses       74      78    +4

Closing and reopening to kick off the tests again.


searge llm node doesn't load on comfyui

IS there any solution, i tried to install llama ccp [cuda] via cmd but failed although

I have the same problem.
Check the version of Python in your ComfyUI_windows_portable\python_ebeded folder.  It should be 3.11.  If it is 3.12 (this is the latest python version installed with ComfyUI windows portable) it looks like we need python 3.11 to run these nodes,
But I have no idea how to downgrade Python from 3.12 to 3.11 in a windows-portable installation.

Ok... found a solution!
It's not a short one, but it is easy and it works!
The problem is ComfyUI, since v.0.2.4, started to use Python 3.12, while a lot of custom nodes were still on python 3.11.
Many LLM nodes, and Searge LLM nodes too, can work on python 3.11, but not on 3.12.
So you have to downgrade python version from 3.12 to 3.11.
I have no clue how to do it in the windows-portable version, as python works in an embedded folder.  Maybe it is a "venv" version, but since I have no idea how to work with python I found this workaround:
(IMPORTANT: Before starting, keep the old ComfyUI installation and create a new folder where to install this fresh installation with python 3.11. For example I created a folder called "ComfyUI_3.11")

You need to do a new fresh ComfyUI installation using the old v.0.2.3, the last ComfyUI version using python 3.11.  Here is the direct link to the download:  https://github.com/comfyanonymous/ComfyUI/releases/download/v0.2.4/ComfyUI_windows_portable_nvidia.7z
Once you download it and unzip it in the new folder (I used the "ComfyUI_3.11" folder), you should install ComfyUI Manager to make it easier to manage ComfyUI. Link to ComfyUI Manager:  https://github.com/ltdrdata/ComfyUI-Manager
Do not run ComfyUI yet!  Once installed COmfyUI Manager, you should update ComfyUI and all Python dependencies.  Use the .bat file in the "update" folder:  "update_comfyui_and_python_dependencies.bat"
Good, now you should have a fresh install of the latest version of ComfyUI (today, 26 Dec. 2024, is the v.0.3.9-26), with the latest Manager version and Python 3.11.9 with the latest dependencies like pytorch 2.5.1+cu124).
Now the long and boring part: you will need to install all your custom nodes and models.  Use the ComfyUI Manager to install the custom nodes, and once this is done, check all the models (checkpoints, unet, loras, etc.) you had in your old installation and copy all of them in the new installation.
You are done!  The new ComfyUI installation will run on Python 3.11 but it will be updated to the latest version of Comfy and of Python dependencies.  Searge LLM nodes works perfectly!  Enjoy!
My personal suggestion:  do a backup copy of the old ComfyUI folder and the new ComfyUI_3.11 folder for security.  And do a backup of your latest ComfyUI folder regularly, before every ComfyUI update, this way if something goes wrong, you will have a previous version that works for sure.

Hope this help.


Docs config updates

@pjbreaux
Fixes #66 - Add intersphinx mapping and autosectionlabel to docs configs
I added the configurations to docs/conf.py that a)  make this doc set reference-able from the other F5 doc sets and b) automatically create section headers based on the section title.
Note:
This PR needs to be merged in before the one I'm going to assign to you shortly in the heat repo.

@jputrino: Please rebase your changes on top of the latest kilo.

Done!

Ship it!


call_ltac should return goals instead of dyns

Right now call_ltac has type (A * list dyn), where A is the type (goal) in which the tactic is run, and the list contains the subgoals generated, as dyns. It should return a (A * list goal) instead, since subgoals might contain variables not in the same environment as the running environment.
The fix implies moving the type of goal from MCTactic to MetaCoq.

(See bugs.v)

This has been partially fixed in 0958fbbe06f684a82743dcb41153e77066b97f53

I will close it for now. It is not taking into account cintros'ed hypothesis (i.e., de Bruijn indices).


Gender fix and changing 'and' for 'but'

In portuguese verbs can have gender, so I have changed to 'instalados' that refer to 'estão' witch is a masculine verb.
In addition we changed 'e' to 'mas', because is a negation adverb, witch improoves cohesion.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Some grammar fixes.


Accrued allowance feature

Currently the application allocate the number of free time off days at the start of the year and employee is able to spend them at any point.
Few customers asked for feature when the allowance is accrued based on how far in the current year we are now.
"... We would need some notion of accrued time. Instead of having a fixed amount of time available at the beginning of the year, we officially accrue time based on an annual rate. If you had 12 vacation days per year, you would accrue 1 day per month (or more precisely, a certain amount per day). This is useful for 2 cases ..."
To implement this feature we do not need to do any DB changes other than adding new flag to the company level - that enables/disables it.
The business logic needs to be amended such a way that when requesting new leave system checks if user still within accrued number of available days.
That feature overlaps with some functionality discussed in #37

Got interest from one more person.

More interest from customers.

+1 Accrual should be in days per period or % per period.  Periods corresponding to pay cycles, monthly weekly bi-weekly

One more person is asking for this.

Accrues are defined by:

days to accrue on every period
schedule (month, fortnight, months) and date

I think in case of TOM it will be days based rather then hours based and more likely on month base. Also the precision is going to be up to .5 day.

One more person asked for the feature:

I would like to track accruals for PTO and Sick time.

Let David (email from 1 Aug) know when the feature is done.

One more person asked for the feature

I too am interested in this.  It would be SUPER!  Also if it could use hours instead of days.  If you guys need help, I'd be more than happy to poke around the code...

Thank you @darntough

One more person asked about this feature. So it needs to go into progress.

I would love to see that you could select the amount of hours your want to take as leave of absence as well. Our company gives employees 27,5 days off per year, but those can be used by the employees per hour.

+1

The feature is in master.
Now it is possible to set given department to have its employe with accrual holiday entitlements.


Expand numbers from BamStats

Percentage mapped reads is reporter per lane now. We care about the overall mapped reads per sample, however, so we want to make that conversion. This is what we see now:
    SIB909A8.141216_HB05PADXX_ACTTGA.lane1:
      BamStats:
        percentag_mapped_reads: 99.7589420977317

To make an accurate average we would need the total number of reads and total number of mapped reads. We can't infer that from only this number. Perhaps one "lane" is undetermined reads and then we need to know how many read (pairs) was in that file which we get the mapped percentage number from.
Perhaps also update the key: percentag_mapped_reads to percentage_mapped_reads ;)

Changed the typo.
Added raw total sequences and reads mapped to qcCollect alongside percentage_mapped_reads


Batch reorder for more complex systems

For LINAC mostly

This means a batch reordering system that does not put the batch with the highest k first, as this does not work as well with breeder or sub-critical systems.


release v3.2.0

Release checklist

[x] Ensure all milestoned issues/PRs are addressed, or reassign to a new milestone.
[x] Add the dont merge label to all PRs that are close to be ready to merge (or merge them if they are ready) in capa and capa-rules.
[x] Ensure the CI workflow succeeds in master.
[ ] Ensure that python scripts/lint.py rules/ --thorough succeeds (only missing examples  offenses are allowed in the nursery).
[x] Review changes

capa https://github.com/mandiant/capa/compare/<last-release>...master
capa-rules https://github.com/mandiant/capa-rules/compare/<last-release>...master

[x] Update CHANGELOG.md

Do not forget to add a nice introduction thanking contributors
Remember that we need a major release if we introduce breaking changes
Sections: see template below
Update Raw diffs links
Create placeholder for master (unreleased) section## master (unreleased)

### New Features

### Breaking Changes

### New Rules (0)

-

### Bug Fixes

### capa explorer IDA Pro plugin

### Development

### Raw diffs
- [capa <release>...master](https://github.com/mandiant/capa/compare/<release>...master)
- [capa-rules <release>...master](https://github.com/mandiant/capa-rules/compare/<release>...master)

[x] Update capa/version.py
[x] Create a PR with the updated CHANGELOG.md and capa/version.py. Copy this checklist in the PR description.
[x] After PR review, merge the PR and create the release in GH using text from the CHANGELOG.md.
[x] Verify GH actions upload artifacts, publish to PyPI and create a tag in capa rules upon completion.
[ ] Spread the word
[ ] Update internal service

closes #905

Thanks for the approval!


NIFI-6694 Removed erroneous inclusion of the Hive 3 component on the …

…JDK 8 ja_JP locale build stage on Travis
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Merging based on https://issues.apache.org/jira/browse/NIFI-6694?focusedCommentId=16941510&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16941510


Upgrade pkl version in use to 0.26.1

Parameterise downloading the pkl binary version, and have the lookup be the version to make it easier to debug.
Cache directory prefix changes
Update the cache directory in use to be -2 suffixed, as per latest changes to Pkl in 0.26.0.
See: https://pkl-lang.org/main/current/release-notes/0.26.html#cache-directory-prefix-changes

Interesting. Looks like we need to build/test against a newer version of JDK for Pkl 0.26.1, as it requires JDK17+.


Move service extraction logic over to ArgumentExtractors

Presently, argument extractor do nothing other than handing off an incoming request over to a given service type, so the service itself can be constructed. This leaves very little for the extractor to do, and couples the service type itself with the parameters of the request. The task of extracting a service from a service should really be done by the argument extractor itself, allowing the service to mostly just act as a POJO, carrying properties and producing a right response based on those.
The task proposed above should simply be considered a refactoring effort, without any additional changes or compatibility issues. It allows extractors to take full control of dealing with the request and also allows to have better control over the construction of that service in one spot, rather than passing parameters and arguments from the config, over to the extractor, over the service, etc!
Relates to https://github.com/Jasig/cas/issues/654

Relevant to https://github.com/Jasig/cas/issues/998 and may be handled there.


Read Saint Kitts and Nevis' BUR1 from pdf

Description
Read Saint Kitts and Nevis BUR1 from PDF.
Checklist
Please confirm that this pull request has done the following:

[ ] Tests added
[ ] Documentation added (where applicable)
[ ] Changelog item added to changelog/

I think we need the agriculture trend table from pages 131/132, so we can obtain all PRIMAP-hist sectors for all gases through downscaling.

I think we need the agriculture trend table from pages 131/132, so we can obtain all PRIMAP-hist sectors for all gases through downscaling.

The data from the table on pages 131/132 is already read in with the overview table on pages 111/112/113.
We now have the sectors 3 / 3.A / 3.B / 3.C and 3.D  for 2008 to 2018 (with 3.D being NE for all years) and KYOTOGHG (AR6GWP100). Do you mean downscaling entities for the years 2008-2018? For example, create values for N2O and CH4 for 3.A for all the years based on the values of 2018?

@JGuetschow I updated the downscaling for most primap sectors. We now have the values by entity for 2008-2018  for the categories:
1, 1.A, 1.B, 3, 3.A, 3.B, 3.C, 3.D, 4
Still missing are the values for the categories 2, 2.A, 2.B, 2.C, 2.D, 2.E, 2.F, 2.G, 2.H. They are all zero and the downscaling currently does not work for that.
Also, we don't have the trend for 1.C, as there are no values for KYOTOGHG.


No css classes in header column in Production

Thead th cell has span inside it with role="button" and class="MUIDataTableHeadCell-toolButton-[mui-number-prefix]" in development mode. But in production there is no class="MUIDataTableHeadCell-toolButton-[mui-number-prefix]".
Expected Behavior
The class must be present in production mode as well as in Development one.
Current Behavior
The class is present just in development mode.
Steps to Reproduce (for bugs)

Add table to the page.
Inspect a document in development and production mode.
See that class is not present in Production.

Your Environment

Tech
Version

Material-UI
4.11.0

MUI-datatables
3.4.0

React
16.13.1

browser
Chrome

Attacments:
Dev:

Prod:

That's the result of webpack (local-css).

I believe @wdh2100 is right, and this isn't a mui-datatables thing, it's how JSS was setup to work. It shouldn't really impact anything though. I use overrides in my code with the original names and everything still works fine.

I believe @wdh2100 is right, and this isn't a mui-datatables thing, it's how JSS was setup to work. It shouldn't really impact anything though. I use overrides in my code with the original names and everything still works fine.

I use overrides as well. Everything works amazingly except that case with "MUIDataTableHeadCell-toolButton" class. It is not a custom one. This class comes from the library itself.

That's the result of webpack (local-css).
Well... maybe not.
Could you give me an CodeSandBox

I can see css class in SandBox.
https://codesandbox.io/s/youthful-lederberg-6cunm?file=/src/App.js

As a result of webpack, jss-xxx means local css.
https://github.com/gregnb/mui-datatables/blob/master/src/components/TableHeadCell.js
Here we are using a style override (local css-include jss) on the button.
Attacments:
Dev:

Prod:

As a result, it looks like this.
production mode : https://webpack.js.org/plugins/uglifyjs-webpack-plugin/

@BohdanYavorskyi are you saying your override for "MUIDataTableHeadCell-toolButton" is not working in production?
Try adding a StylesProvider to you app: (see the createGenerateClassName example at the top)
https://material-ui.com/styles/api/
It may be that some of your classnames are having collisions. I was able to override the classes fine in production mode.

@BohdanYavorskyi are you saying your override for "MUIDataTableHeadCell-toolButton" is not working in production?
Try adding a StylesProvider to you app: (see the createGenerateClassName example at the top)
https://material-ui.com/styles/api/
It may be that some of your classnames are having collisions. I was able to override the classes fine in production mode.

@patorjk yeah, I'm overriding this class. I use a default StylesProvider


Bug: Metrics calculation resulting in different totals with aliases enabled and disabled state for hyades

Description
In the current implementation of metrics calculation, when aliases are enabled for Snyk and OSV, the vulnerabilities count on the overview of project and also on the dashboard is different from (less than) the total audit vulnerabilities count for the same project.
When aliases are disabled, the counts are the same in both the places.
This seems to be a gap in the stored procedure used for the metrics calculation.
With alias disabled

With alias enabled

This is intended behavior when aliases are enabled. Metrics do not count the same vulnerability multiple times if they are aliasing each other.


Remove custom versions from dev-build

Fixes https://github.com/C2SM/spack-c2sm/issues/912

[x] Adapt instructions for using version suffixes with dev-build
[x] Clean-up conf.py
[x] Add Makefile to build HTML docs locally (and thus enable .rst previews in VS Code)

Unfortunately make html fails with
sphinx-build -b html -c ./ . _build/html
make: sphinx-build: Command not found
make: *** [Makefile:18: html] Error 127

That means that you don't have sphinx-build installed within your local environment.
You should run this beforehand:
pip install sphinx sphinx_rtd_theme sphinx-copybutton

That's good to know. Shall we make a note about this in the documentation or add it to the requirements.txt ?

I have updated the requirements.txt with the missing packages. Note that they get installed when running the GitHub actions, making the pre-build-command there obsolete.


Add 500ms delay when refreshing web UI after metadata modification

Motivation:
A user may get a dirty result after modifying metadata due to replication lag. Adding some delay before refreshing may help to make the situation less often.
Modifications:

Add 500ms delay before refreshing web UI.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #182   +/-   ##
=======================================
  Coverage   63.82%   63.82%
=======================================
  Files         275      275
  Lines        9510     9510
  Branches     1066     1066
=======================================
  Hits         6070     6070
  Misses       2813     2813
  Partials      627      627

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 1c4a7b6...a54b663. Read the comment docs.

Thanks, @hyangtack !


Update dev setup

Hello, i updated the dev setup and compat.lua, also fixed some files getting double '---@meta\n' during code generation

Hi Azaganoth, I should apologize for never even responding after a month. The pull request seemed too big with a lot of formatting changes, and I had trouble understanding it.

I didn't see where there were double ---@meta, only that the whole files got replaced.
Some math api shorthands are not the same, like acos returns degrees, while math.acos returns radians. You can reference this from compat.lua as found from when opening Wow.exe in a text editor

The project is messy though, so I can't say I'm proud of the current state.

Its alright u done a great job making it all work.

Some files are written with CRLF line breaks (\r\n) so when this is executed it adds double ---@meta\n. I just rewrote those files using LF (\n) line break.
I know the shorthands are not the same, but compat.lua is a type declaration file, so the internal function logic is irrelevant. With the parameters and return types being the same I dont see a problem with using the already defined declaration.


Contiuation DllCall issues

V1:
var := DllCall("Func"
     , "Str", "ABC" ; Comment
     , "Str, "123"   ; Comment
     , "Str", "DEF") ;Comment

hCtrl := DllCall("CreateWindowEx"
      , "Uint", 0x200        ; WS_EX_CLIENTEDGE
      , "str",  "HiEdit"     ; ClassName
      , "str",  ""		     ; WindowName
      , "Uint", WS_CLIPCHILDREN | WS_CHILD | WS_VISIBLE | hStyle
      , "int",  X            ; Left
      , "int",  Y            ; Top
      , "int",  W            ; Width
      , "int",  H            ; Height
      , "Uint", HParent      ; hWndParent
      , "Uint", MODULEID	 ; hMenu
      , "Uint", 0            ; hInstance
      , "Uint", 0, "Uint")

V2 (Converted):
var := DllCall("Func"
     , "Str", "ABC"
     , "Str, "123"
     , "Str", "DEF")

hCtrl := DllCall("CreateWindowEx"

V2 (Expected):
var := DllCall("Func"
     , "Str", "ABC" ; Comment
     , "Str, "123"   ; Comment
     , "Str", "DEF") ;Comment

hCtrl := DllCall("CreateWindowEx"
      , "Uint", 0x200        ; WS_EX_CLIENTEDGE
      , "str",  "HiEdit"     ; ClassName
      , "str",  ""		     ; WindowName
      , "Uint", WS_CLIPCHILDREN | WS_CHILD | WS_VISIBLE | hStyle
      , "int",  X            ; Left
      , "int",  Y            ; Top
      , "int",  W            ; Width
      , "int",  H            ; Height
      , "Uint", HParent      ; hWndParent
      , "Uint", MODULEID	 ; hMenu
      , "Uint", 0            ; hInstance
      , "Uint", 0, "Uint")

Two examples, second one came when trying to reproduce first
Need to look into it further
Example 1: Comments are stripped
When return variables are removed comments are still missing

, "str",  "" causes issues in the second example

Issues with large portions of code going missing are now fixed
Comments in continuations face 1 of 2 issues
var := "line 1" ; comment 1
    . "line 2" ; comment 2
; -> converts to ->
var := "line 1"
    . "line 2" ; comment 1

The converter treats above as one line (rather than 2, this is intended behaviour)
Fix would likely be
; Store comment in map
; line comment is on: comment

; restore once line conversion is done

Same issue with function continuations but they are stripped instead
Maybe same fix as above?


feat(adaptive-cards): custom card elements (Momentum UI)

Description of changes made

Momentum UI components
Changes in adaptivecard utils
No test coverage

Reference Link : https://docs.microsoft.com/en-us/adaptive-cards/sdk/rendering-cards/javascript/extensibility
Associated issue number/tracking link
SPARK-94094
reviewers
@InteractiveTimmy

Hello,
Based on the conversation we had, we have marked this pull request as "ready to merge" whenever you're ready. Feel free to reach out via the contributors' space in Webex Teams and we will proceed with a quick review of any changes [if there are any] and merge.

Hello,
Based on the conversation we had, we have marked this pull request as "ready to merge" whenever you're ready. Feel free to reach out via the contributors' space in Webex Teams and we will proceed with a quick review of any changes [if there are any] and merge.


Add Power Trip by Boom Kitty

Song Name
Power Trip
Artist Name
Boom Kitty
Source
Youtube
Youtube Link
l6OsF7RlQb4
Direct File Link
No response
Song ID
39
Start Offset [ms]
4
Extra
Created from Auto Nong.
Created for level: 81921474
Downloaded: No

I really want to listen to Power trip in gd but robtop hasn't added them into the actual game yet lll


Excerpt uses 400 words instead of 400 characters

I published my first blog post with the plugin active and the post itself was a 400 word wall of text (instead of 400 character excerpt) in the actual Fedi post.
Images showing the bug:

Actually I would see that as an interesting option : we could choose either to publish a certain length of text in characters (so it's always the same space taken), or to publish a certain amount of words to give a more or less accurate introduction to the blog post.
Would that be possible ?

Sure, I think so!

I had the idea to use shortcodes instead of %excerpt% to be able to use params like [ap_excerpt count_type="words" count="400"]. But how do I pass the post_ID to the shortcode? Any ideas @akirk @mediaformat ?

We should be using the setup_postdata() function so that in the shortcode get_the_ID() can be called.

can you send me a link to the "Gutenberg template blocks" documentation?

If you're using the classic editor, IIRC you can set the excerpt cutoff with the  tag. (I don't know if that's still how it's done with the new editor though, sorry)

Chiming in: While you are pondering this, in addition to breaking on words or characters, it would be also be useful to break on the first paragraph.
This lets me write an intro to my blog post, which may be longer or shorter than X words/characters, which is then posted. In practice, the intro paragraph should be short to prevent the wall of text, but I would be assured of getting the whole paragraph.

I tried setting the excerpt cutoff with the 'more' tag (it seems to have been stripped out of my previous comment, because I forgot to escape it and of course it was) but the cut didn't, well, make the cut. And neither did any of my paragraph breaks. (Is that because I'm using the classic editor? I don't have to put in 'p' tags for paragraph breaks to show up on the blog itself, so I didn't know they wouldn't come across to Mastodon.

I think it should work if you set the excerpt by hand to exactly what you want to see on mastodon, as a quick hack!?!

Can confirm, this works.

Ah thanks for that tip. I'd prefer it to respect the more tag (and not strip returns as it currently does) but manually specifying the excerpt is good enough for now. I wish I found this page earlier.

Fixed with: https://github.com/pfefferle/wordpress-activitypub/pull/247
Thanks @toolstack


feat: add rollupDepPlugin

Current dependencies on/for this PR:

main

PR #1   👈

PR #2

This stack of pull requests is managed by Graphite.


Django Integration - Some Random Questions

Hello guys,
First of all some greetings, this work looks absolutely great. I have a few questions for you.

How mmany models do we you think it is worth to manage to start feeling benefits using such a system?
I use Django and custom version of this package: https://github.com/fridiculous/django-estimators, what pros and cons would you see to make the switch (or not doing it) ?
Do you actually believe it is more efficient to save each model in a DB rather than some cloud storage such as AWS S3 and save the link to the model in the app database ?
Do you have any benchmark on how long it takes to load a model from ModelDB and save a model to ModelDB ?
Do you have views on making on pre-compiled container available on Docker Hub for instance? It would be so wonderful to quickly spin-up the last release and test it live.

Thanks a lot for your answers, I think some of the answers to this questions should appear somewhere in the Readme.md. It is absolutely a must know for someone who is interested in going live with this project.
Once again, thank you very much for your contribution to the ML community.
Best regards from France,
Jonathan D.

@DEKHTIARJonathan https://hub.docker.com/search/?isAutomated=0&isOfficial=0&page=1&pullCount=0&q=modeldb&starCount=0 should answer your fifth question about docker-hub support.

Oh nice @zitterbewegung, thanks for it ! I don't know why I didn't find them...
Anyway, I'm still very interested about the four other questions ;)


errors: example code for errors.As has extraneous code

The example code for errors.As (viewed at https://pkg.go.dev/errors?tab=doc#As) is the entire contents of the wrap_test.go file. It should be just func ExampleAs() { ... } I presume.
In talking this over with @FiloSottile, this may be a more systemic issue which can be improved in godoc. For example, we have the same issue with the chacha20poly1305.NewX example code. However, perhaps we should fix the example for the errors package for now either way.
While we're in there, perhaps we should check if there are any improvements to the documentation for errors as tracked in https://github.com/golang/go/issues/31716.
/cc @jba @mpvl

The package doc for testing says

The entire test file is presented as the example when it contains a single example function, at least one other function, type, variable, or constant declaration, and no test or benchmark functions.

(Emphasis mine.) The wrap_test.go does have a test function, so the whole file shouldn't be displayed as the example.
So I think this is a bug in the testing package.

@dmitshur, is this a godoc bug?

Is this a consequence of #38376 ?

Yes. @jba's analysis is correct. It's a regression in go/doc package in Go 1.14, filed #38409. Thanks for spotting and reporting.

Looks like this is now tracked in #31716 and #38409.


How to restrict some commands on a open session

How to restrict some crucial commands like "rm  -rf" , "init 0"
Is there any exiting options for command restrictions?

You may need to do this on a Selinux or permission level in the linux box, if you want to implement it across many boxes maybe you want to use Config tools like Ansible.

@ajohnsc Thanks for your response, Now we have restricted the commands on the front end layer, We did some changes in the JavaScript to record every user command and restricted them before send to the server. and its working as expected


Suggested fixes

Trying it out
You can test this beta of the gem directly from github by putting this in your Gemfile:
gem 'arduino_ci', git: 'https://github.com/ianfixes/arduino_ci.git', branch: '2020-10-16_suggestions'

Highlights from CHANGELOG.md
Added

arduino_ci_remote.rb CLI switch --skip-examples-compilation
CppLibrary.header_files to find header files
LibraryProperties to read metadata from Arduino libraries
CppLibrary.library_properties_path, CppLibrary.library_properties?, CppLibrary.library_properties to expose library properties of a Cpp library
CppLibrary.arduino_library_dependencies to list the dependent libraries specified by the library.properties file

Changed

CppLibrary functions returning C++ header or code files now respect the 1.0/1.5 library specification

Deprecated

arduino_ci_remote.rb CLI switch --skip-compilation

Fixed

Don't define ostream& operator<<(nullptr_t) if already defined by Apple
CppLibrary.in_tests_dir? no longer produces an error if there is no tests directory
The definition of the _SFR_IO8 macro no longer produces errors about rvalues

See CHANGELOG.md for more

Issues Fixed

Fixes #175
Fixes #168
Fixes #187

Reverting 8180efb as per this https://github.com/Arduino-CI/arduino_ci/issues/196#issuecomment-723476602


Training Data Clasification ML model stuck at "Finilizing models..."

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog):  16.14..0.2255902
Visual Studio Version: VS2022 Enterprise

Describe the bug

At step "Train" in the ".mbconfig" UI Editor
Training ML model stuck in the "Finilizing models..." process

To Reproduce
Steps to reproduce the behavior:

Go to Scenario
Pick "Data classification"
Chose environment Local CPU
For Data input choose SQL server then choose the table with the Output boolean column
Train for 60 seconds

Expected behavior
Expected to get trained model
Screenshots

Additional context
When training with other scenarios, for example, "Value prediction", after starting the training process, there are some outputs regarding training models and stuff. This one has no output, except "Training time in second: 60".

It takes really long time to finalize, like 15+ minutes.


All methods on models should use value receivers

Is your feature request related to a problem? Please describe.
Goland has been harassing me about the fact that some structs in the package model mix value and pointer receivers. This is not recommended by the Go documentation.
Describe the solution you'd like
It would make sense to convert the methods with pointer receivers to value receiving methods as the mutability provided by using pointers is achieved through the dao design pattern.
Describe alternatives you've considered
It would also be possible to convert the methods with value receivers to methods with pointer receivers, the docs suggest this would be more efficient but since most methods use value receivers and the commitment to the dao design pattern was made, I think using value receivers makes more sense.
Of course it could also be left as is.
Additional context
The Go docs talk about this in their FAQ here

I agree. However, some models have methods that mutate the model, so we should be careful not to break anything.

I had a second look at this an realised that dao is only used by the api package and not the model package. So I see no reason for methods on models to use value receivers at all. Aside from consistency, I think that copying rather large structs for often fairly simple methods is a bit inefficient.
I doubt this would make a significant difference in performance, but if there's no reason to use value receivers then it might as well be changed.


Add tmuxinator project hooks

Hello, this is my first contribution here. I found myself in need of more control around tmux session "events", and after looking at #476 I decided to provide this PR.
It adds 2 new "events" that can be acted upon in the project configuration file: on tmux session reattach and on tmux session stop.
A practical use case would be to run vagrant suspend and vagrant up on session detach and reattach.
Plus I duplicated the pre and post setting rewording it as on_start and on_detach, because so the code has been moved in the Tmuxinator::Hooks module and made sense with the new "hook" naming. As I don't know you policy about breaking changes, please let me know if and how you would like to address this.
Thank you for this little "gem" ;) it is really useful and has really helped me in managing my workflow.

Coverage increased (+0.04%) to 99.227% when pulling 4428337f5444d76af7a8810f9e0ba45e9b3c6198 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.04%) to 99.227% when pulling 4428337f5444d76af7a8810f9e0ba45e9b3c6198 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.04%) to 99.227% when pulling 4428337f5444d76af7a8810f9e0ba45e9b3c6198 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.04%) to 99.227% when pulling 4428337f5444d76af7a8810f9e0ba45e9b3c6198 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

@endorama ^ it looks like there's some extra whitespace in the spec.  Annoying, I know, but valuable for the maintainers. In the future you can run the rubocop checks locally with bundle exec rake rubocop

I'm sorry, I did that but that one has slipped trough my checks. Everything should be fine now.

Coverage increased (+0.04%) to 99.227% when pulling 63dbaad9b3615ac6204e8cf3f59701eb869513ac on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

@endorama After our earlier exchange, I found a few minutes to pull this down and experiment.
A few thoughts:

If others agree that the "hooks" approach is a better solution than sticking event methods in Project (I do), I'd propose aliasing the existing methods to their corresponding Hook methods, adding a deprecation warning and removing them in a future release.
sample.yml is missing an on_start entry
Should the README's Custom attachment and post commands section be updated to cover the new options and deprecations (if that's the approach we decide to take)?
I think hook_on_stop may need to be moved before tmux_kill_session_command. On my system (Debian 8.6, Bash 4.3.30, Tmux 1.9, Ruby 2.3.1p112) on_stop commands are ignored unless I reverse these two method calls. Is this working for you?
Document (and potentially fix) behavior of on_reattach_hook; it appears to only be executed when reattaching via tmuxinator start ... and not tmux attach -t ...

sample.yml is missing an on_start entry

I'll fix this

I think hook_on_stop may need to be moved before tmux_kill_session_command. On my system (Debian 8.6, Bash 4.3.30, Tmux 1.9, Ruby 2.3.1p112) on_stop commands are ignored unless I reverse these two method calls. Is this working for you?

It is. I'm currently using my fork to start and stop a vagrant machine when issuing tmuxinator start project and tmuxinator stop project

Document (and potentially fix) behavior of on_reattach_hook. From what I can tell, it appears to only be executed when reattaching via tmuxinator start ... and not tmux attach -t ....

I would say this is by design. AFAIU for executing on tmux attach the proper tmux hook should be used. What is instead happening is that the hook_on_reattach command is written in the template file if tmux has a session named as the project name.
I'm starting to think that the naming hook is really misleading, in particular for on_attach and on_reattach ( which are more tmux concepts than tmuxinator ones ). Should I rename them in a more project-related way? For example:

on_reattach could be on_project_restart
on_attach could be on_project_first_start
on_start could be on_before_project_start
on_stop could be on_project_stop

This renaming would also be future proof when proper tmux hooks are implemented. Sorry for all this confusion.

Should I rename them in a more project-related way?

:+1:

Coverage increased (+0.04%) to 99.229% when pulling cbfc51095ee8631fa1035878ff0c193784db4321 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Sorry for the delay.

hooks remaned to hook_on_project_*
extracted the common code for hooks
moved project hooks to Tmuxinator::Hooks::Project to make the scope clearer
updated relevant specs

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

Coverage increased (+0.05%) to 99.24% when pulling fe7889c29d77090fbdef0252c0cd76f8f0dabbf0 on endorama:add-hooks into 6df10d1852ab4f427a703af4a8546a1fd4c25836 on tmuxinator:master.

@endorama Sorry for the delay. I'll have to find some time to revisit this and do a proper review. Did you give any additional consideration to adding a deprecation warning to the existing pre/post "hooks"?
In the meantime, does anyone else have any thoughts or feedback? Is this something people think is worth pursuing? If so, are these hooks deserving of a README entry?
/cc @J3RN @adamstrickland @Soliah

I'll review this on the weekend.

@Soliah thanks!
@ethagnawl IMHO, pre and post could be deprecated, as I think these "hooks" are more explicit and perform the same action, but I'd leave this decision to you as you see fit.

Ok PR looks, good. Definitely think the pre and post hooks should be deprecated. Additionally maybe lets cut a pre release version for testing in the wild.

Hello and sorry for the delay! I deprecated the pre and post options.

Hello @Soliah, any news about this? Just to confirm I'm being using this patch since February.

:tada: :tada:
Thanks a lot!!! :smile:

Hello and sorry for the delay! I deprecated the pre and post options.

The pre option could be set per-window. As I understand from the above, the hooks or pre_window option do not function like that. Is there an alternative for pre in this case?


Getting CaseClauseError in Tds.Protocol.ping

Firing up a new phoenix app and receiving the following every few seconds in the console.

[error] GenServer #PID<0.318.0> terminating
** (CaseClauseError) no case clause matching: {:ok, %Tds.Result{columns: ["msg"], command: nil, num_rows: 1, rows: [["pong"]]}, %Tds.Protocol{env: %{trans: <<0>>}, itcp: nil, opts: [password: :REDACTED, idle_timeout: 5000, name: Home.Repo.Pool, otp_app: :home, repo: Home.Repo, timeout: 15000, pool_timeout: 5000, adapter: Tds.Ecto, database: "redacted", username: "redacted", hostname: "redacted", pool_size: 10, pool: DBConnection.Poolboy], pak_data: "", pak_header: "", query: nil, result: %Tds.Result{columns: ["msg"], command: nil, num_rows: 1, rows: [["pong"]]}, sock: {:gen_tcp, #Port<0.8983>}, state: :ready, tail: "", transaction: nil, usock: nil}}
(tds) lib/tds/protocol.ex:62: Tds.Protocol.ping/1
(db_connection) lib/db_connection/connection.ex:358: DBConnection.Connection.handle_info/2
(connection) lib/connection.ex:810: Connection.handle_async/3
(stdlib) gen_server.erl:601: :gen_server.try_dispatch/4
(stdlib) gen_server.erl:667: :gen_server.handle_msg/5
(stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
Last message: :timeout
State: {Tds.Protocol, %Tds.Protocol{env: %{trans: <<0>>}, itcp: nil, opts: [password: :REDACTED, idle_timeout: 5000, name: Home.Repo.Pool, otp_app: :home, repo: Home.Repo, timeout: 15000, pool_timeout: 5000, adapter: Tds.Ecto, database: "redacted", username: "redacted", hostname: "redacted", pool_size: 10, pool: DBConnection.Poolboy], pak_data: "", pak_header: "", query: nil, result: %Tds.Result{columns: nil, command: nil, num_rows: 0, rows: []}, sock: {:gen_tcp, #Port<0.8983>}, state: :executing, tail: "", transaction: nil, usock: nil}}

The fix is merged, but the Hex release 1.0.4 was apparently built from the latest Github tag 1.0.2, which doesn't include the fix. Can someone please make a proper release?

1.0.5 is released in hex now. It should include pr #41


If profile is not ambigous, default to it.

If there is single profile specified, default to it and do not require
--profile command line option.

Codecov Report

Merging #215 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
- Coverage   66.02%   66.01%   -0.01%
==========================================
  Files         500      500
  Lines       39527    39531       +4
  Branches     4031     4033       +2
==========================================
  Hits        26097    26097
- Misses      12440    12443       +3
- Partials      990      991       +1

Impacted Files
Coverage Δ

lib/python/treadmill/context.py
78.18% <0%> (-2.57%)
:arrow_down:

lib/python/treadmill/dnsutils.py
60% <0%> (+1.33%)
:arrow_up:

lib/python/treadmill/dnsctx.py
71.6% <0%> (+2.46%)
:arrow_up:

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 7018429...46fb05f. Read the comment docs.


Refactor: Extract default-language-modal to be a partial.

We are doing this refactor for the sake of keeping our template consistent with the indentation policy and maintaining its readability at the same time. Primarily since 4 space indents may become too much clumsy some times we should factor out some units as separate templates(partials) to ensure that templates are understandable and maintainable.
https://github.com/zulip/zulip/pull/4512#issuecomment-297263400

Automated message from Dropbox CLA bot
@adnrs96, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @adnrs96!


Qdt filter drop down goes blank after scrolling down and up once

Contents from the Qdt Filter drop down goes blank if a user scrolls down and back up. The items do not come back even if I type something in search. Have to refresh the page to get back the content from the dropdown. Looks like a bug. Let me know if you need more information.

Try the latest 2.0.1. I believe its fixed


commitMode: true still includes extra lines refering to pull request #0

The the following job sets commitMode to true but still contains unneeded references to nonexisting PRs.
Job
jobs:
  create-release:
    name: Create release
    runs-on: ubuntu-latest
    outputs:
      upload_url: ${{ steps.create-release.outputs.upload_url }}
    steps:
      - name: Build Changelog
        id: build-changelog
        uses: mikepenz/release-changelog-builder-action@v4
        with:
          commitMode: true
      - name: Create Release
        id: create-release
        uses: softprops/action-gh-release@v1
        with:
          body: ${{steps.build-changelog.outputs.changelog}}

Output
## 📦 Uncategorized

- Add coloring by prefix using conf:byPrefix.
   - PR: #0
- added travis file
   - PR: #0
- Travis update
   - PR: #0
- Travis deploy setup
   - PR: #0
- Trying to figure out right file location
   - PR: #0
- hardcoded path for test
   - PR: #0
- Skip cleanup
   - PR: #0
- Travis on tags
   - PR: #0
- Initial Dockerfile, runs but no config yet
   - PR: #0
- Merge pull request #29 from zazukoians/master
   - PR: #0
- added new comment for issue 32
   - PR: #0
- Merge pull request #27 from IMISE/feature-colorbyprefix
   - PR: #0
- Build docker image from source.
   - PR: #0
- Fix whitespace.
   - PR: #0
- Update the jena dependency.
   - PR: #0
- Merge pull request #39 from attadanta/better-docker-build
   - PR: #0
- Merge pull request #40 from attadanta/update-jena
   - PR: #0
- Support environment variables with the 'LodLive' prefix to override single config values in conf.ttl.
   - PR: #0
- Support .env file for configuration. Bump up Java version to 1.8 to support the Java dotenv library.
   - PR: #0
- Use SLF4J Logging instead of System.out.println() and System.err.println().
   - PR: #0
- Merge pull request #50 from hitontology/feature-logging
   - PR: #0
- Upgrade Tomcat from version 7 to 9.
   - PR: #0
- Use serial garbage collector to reduce memory consumption.
   - PR: #0
- Merge pull request #47 from KonradHoeffner/feature-environment-variables
   - PR: #0
- Merge pull request #55 from KonradHoeffner/optimize-docker-memory
   - PR: #0
- Upgrade Jena from version 2.13.0 to 4.4.0. Upgrade Java to version 11.
   - PR: #0
- Use Docker layer cache for Maven dependencies to speed up repeated builds with the same pom.xml
   - PR: #0
- fix conf:homeUrl default value
   - PR: #0
- Added translations for Spanish and Asturian
   - PR: #0
- Merge pull request #68 from herminiogg/master
   - PR: #0
- Merge pull request #61 from KonradHoeffner/patch-1
   - PR: #0
- Merge pull request #48 from KonradHoeffner/feature-dotenv
   - PR: #0
- Merge pull request #51 from KonradHoeffner/upgrade-docker-tomcat-9
   - PR: #0
- Merge pull request #60 from KonradHoeffner/docker-dependency-layer-cache
   - PR: #0
- Merge branch 'master' into upgrade-jena-4.4.0
   - PR: #0
- Merge pull request #58 from KonradHoeffner/upgrade-jena-4.4.0
   - PR: #0
- fix dotenv, resolve #69
   - PR: #0
- Add GitHub Container Registry publish action.
   - PR: #0
- Specify syntax version 1 in Dockerfile.
   - PR: #0
- update to actions/checkout@v4
   - PR: #0
- automatically generate release with release notes on appropriate tags
   - PR: #0

Solution
When commitMode is active, all lines containing    - PR: #0 should not be generated.

Thank you for creating the ticket. This currently works as expected, as this follows the default template (in line with the default behavior using pull requests)
You can see the default template defined here:
https://github.com/mikepenz/release-changelog-builder-action/blob/develop/src/configuration.ts#L74
Configure the action to use a different PR template to match your requirements.

      - name: Build Changelog
        id: build-changelog
        uses: mikepenz/release-changelog-builder-action@v4
        with:
          commitMode: true
          configurationJson: |
            {
              "pr_template": "- #{{TITLE}}",
            }

Thank you for the quick help and the awesome GitHub action, I will do that!


Mutation would throw error if no argument is provided?

The following would prompt error.  (GraphQLError: Syntax Error GraphQL request (2:28) Expected $, found ))
const toggle = gql`
  mutation toggle() {
    toggle() @client
  }
`;

However, the following is OK.
const toggle = gql`
  mutation toggle($makeup: Number) {
    toggle(_: $makeup) @client
  }
`;

For local state, mutation doesn't nessaessily have arguments, use cases include toogling the drawer, triggering animation, and,  for my use case, trigger the window.open('https://authenication-server.com', '_self') to redirect to authentication server.

not aware i can do this.
const toggle = gql`
  mutation toggle{
    toggle @client
  }
`;


400 Client Error in Azure Cloud Shell Connector

Environment
Windows build number: Microsoft Windows [version 10.0.18362.10005]
Windows Terminal version (if applicable): 0.3.2171.0
Steps to reproduce
1- Open a new CloudShell tab and log in
2- Execute some azure cli commands for about 10 min
Expected behavior
Azure CLI commands continue to execute without any problem
Actual behavior
After some time, the azure cli commands will failed with the following error :
Error occurred in request., HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Executig several times the same commands (az keyvault secret show --vault-name $kvName -n $secretName --query id -o tsvfor instance) won't change anything. However the azure cloud shell is opened in the browser (shell.azure.com) the same command will work and executing azure cli commands will start working again in the azure cloud shell of windows terminal.

This error always appears  when doing a command linked to azure AD like az ad user list . It works fine on shell.azure.com but not in WindowsTerminal.

Giving this a +1 - Adding / retrieving secrets is a common operation when running deploying scripts and this issue makes it near impossible to use Cloud Shell Connector for anything serious.

Using Terminal 0.7.3451.0 - Open a Cloud Shell Connector and connect to an active Azure subscription
Make sure the subscription has a vault called "SecureSecretVault" and it contains a secret called "TheSecret"
Run the following: az keyvault secret show --name "TheSecret" --vault-name "SecureSecretVault"

OBSERVE:
Error occurred in request., HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
EXPECTED:
The secret value is displayed

Alright, finally figured this one out. It's a chunk of work to support it, but it looks like AZCs has a "control" channel that it can use to request additional authentication/do other exchanges with the thing that's hosting it (typically the cloud shell javascript.)
One of those requests is for "token". :smile:

Do you propose any workarounds for #6038 besides the browser shell in the Azure Portal (which isn't really viable)?  I'm unable to do anything remote since the session cannot be loaded.  Thanks!

Hi,
Are there any plans to address this issue? Is there a documentation on the "control channel" one could use to contribute the fix?


KAFKA-5096: Log invalid user configs and use defaults

Kafka Streams do not allow users to modify some consumer configurations.
If the user modifies this property, currently an exception is thrown.
The following changes were made in this patch:

Defined a new array 'NON_CONFIGURABLE_CONSUMER_CONFIGS' to hold the names
of the configuration parameters that is not allowed to be modified. Currently, this
contains just 1 parameter - enable_auto_commit. When the 'exactly once'
feature is implemented ( KAFKA-4923), more parameters can be added to this
array.
Defined a new method 'checkIfUnexpectedUserSpecifiedConsumerConfig' to
check if user overwrote the values of any of the non configurable configuration
parameters. If so, then log a warning message and reset the default values
Updated the javadoc to include the configuration parameters that cannot be
modified by users.
Updated the corresponding tests in StreamsConfigTest.java to reflect the changes
made in StreamsConfig.java

retest this please


Ensure RunConfig parameters are writable by Viper

Signed-off-by: David Cassany<EMAIL_ADDRESS>

At a time I was surprised that Viper/Cobra worked while making few elements private and trying set them from yaml or params or env vars. Now I surprised it did not complaint at all if it was not working 😅


v1.0.0 Roadmap

With v1.0.0 we are releasing our first version that comes with breaking changes:
Breaking Changes:

[ ] #115: Only omit payload if undefined

I would also like to introduce Flowtype with this version as it is getting popular and already being used in many React application as well as in create-react-app.
cc @yangmillstheory

Sounds good.
I remember you had a great idea to clean up the documentation too. Just wanted to make sure that's not forgotten (not sure which epic it goes in).

Oh, yeah, documentation cleanup/improvement would also be nice. I will add it.

Because #115 seems to be more relevant than expected, we can release that as only change in v1.0.0 and everything else after to speed up the release.
cc @yangmillstheory

That sounds good. Are you going to take #115?

Yes @yangmillstheory

@timche can I take care of #47 in v1.0.0?

That would be huge breaking change. Could be sth for v2.

What about #18?
I'm not sure if this would also be breaking actually, since I think Redux throws errors on missing default state.
#18 just ensures the errors are thrown earlier.

Yes, this is actually breaking, since we are making this required, even it was throwing an error before.

OK. Please see #127 😄 .

Can we add type-checking of payloadCreator and metaCreator to this?
See the first example in https://github.com/acdlite/redux-actions/pull/128#issuecomment-244896176.

I've ported this roadmap issue over to https://github.com/acdlite/redux-actions/projects/1.


Maintenance of the recipe_ocean_bgc.yml

Is your feature request related to a problem? Please describe.
In the course of updating the WOA cmorizer with the latest data version in #1812, it was attempted to use that newer version of the WOA data in the recipe_ocean_bgc.ylm. This led to several issues that would be better cured in separate PRs. This issue aims at keeping track of the problems found when updating the recipe:

[x] multi-model-statistics applied despite using a single dataset --> commented
[x] Deprecated write_plots  and write_netcdf impact the recipe
[x] Title is missing
[x] Diagnostic fails with --> solved in #2540

  File "/mnt/lustre02/work/bd0854/b309141/repos/ESMValTool/esmvaltool/diag_scripts/ocean/diagnostic_model_vs_obs.py", line 551, in <module>
    main(config)
  File "/mnt/lustre02/work/bd0854/b309141/repos/ESMValTool/esmvaltool/diag_scripts/ocean/diagnostic_model_vs_obs.py", line 545, in main
    make_model_vs_obs_plots(cfg, metadatas, filename, obs_filename)
  File "/mnt/lustre02/work/bd0854/b309141/repos/ESMValTool/esmvaltool/diag_scripts/ocean/diagnostic_model_vs_obs.py", line 250, in make_model_vs_obs_plots
    provenance_record = _prepare_provenance_record(
  File "/mnt/lustre02/work/bd0854/b309141/repos/ESMValTool/esmvaltool/diag_scripts/ocean/diagnostic_model_vs_obs.py", line 492, in _prepare_provenance_record
    val = cfg[key]
KeyError: 'realms'

[ ] The cmorized data of the latest WOA data (WOA18) follow CMIP6 standards while the rest of the recipe is based on CMIP5 datasets. This leads to inconsistencies when using the volcello data needed for CMIP6-like data based on CMIP5 standards. (see this comment)

It would be great to have technical and scientific reviews of such changes, in particular for the last point.
Feel free to edit if something is missing.

If it may be of some help title and write options can be recovered from this commit

This recipe could be also extended by including the comparison of carbonate chemistry (e.g., dissic,talk) with the GLODAP observations introduced in #2185

I might guess this is still in there because is referrend in the ESMValTool v2 paper as an example of available ocean diagnostic tools ...
It can be revised, with some update too, and keep it as a demonstrator for ocean bgc diagnostics.

With ESMValGroup/ESMValCore#1497 and #2540 this recipe runs again. Is the last bullet point still relevant? Please close if not.

I'm not sure I understood the initial problem. @remi-kazeroni as author of the description here and @valeriupredoi as the author of the comment referenced by @remi-kazeroni in said description, could you guys please have a look and decide whether this is still a problem?

I'm not sure I understood the initial problem. @remi-kazeroni as author of the description here and @valeriupredoi as the author of the comment referenced by @remi-kazeroni in said description, could you guys please have a look and decide whether this is still a problem?

I think the problem no longer exists. If I remember correctly, we tested this recipe at a time where the automatic data download was not available and the recipe was crashing because of that. But now that the multi-model statistics over one dataset has been removed and missing data can be downloaded automatically, everything works fine. See for instance this run for v2.5.0rc3.
I'm closing this issue. Feel free to reopen if needed.


exec in wrapper script if we aren't in the repl

we had an internal system that didn't work when someone used the _wrapper script because we hadn't exec'ed.
This fixes it. Really, users should only use the wrapper.sh for the repl, but this is probably not clear, so this error is easy to make.
Alternatively, we could actually just not generate this file for non-repl targets, which I also think would work although it would be a break for people who have assumed it exists. This seems the most conservative change.
actually, the standard launcher calls this wrapper, so I think we do want this PR exactly as is.

So I'm not sure I completely understand the problem (haven't used the repl
rule that much) but if we can avoid introducing another conditional by only
generating this output for the repl rule I think it's a win.
What are the downsides?
On Fri, 13 Oct 2017 at 2:08 P. Oscar Boykin<EMAIL_ADDRESS>wrote:

@johnynek https://github.com/johnynek requested your review on:
bazelbuild/rules_scala#306
https://github.com/bazelbuild/rules_scala/pull/306 exec in wrapper
script if we aren't in the repl.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/306#event-1291321668, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABUIF4-oVXbnzw7GEEpu890tROJMAEIhks5srpvqgaJpZM4P3wQ_
.

Actually the java launching script requires the wrapper. I don’t want to refactor the java launching script we have now and I don’t see the gain immediately.
The problem we have is that we have tools that kill jobs, but without the exec we don’t kill the underlying process.

the wrapper script is something in what we are pulling from bazel:
https://github.com/bazelbuild/bazel/blob/0.4.5/src/main/java/com/google/devtools/build/lib/bazel/rules/java/java_stub_template.txt

I agree about not going into a big refactor.
I think I have some knowledge gap about "exec".
I read in SO that:
"If the command exists and is executable, it replaces the current shell. That means that if exec appears in a script, the instructions following the exec call won't ever be executed (unless exec is itself in a subshell). exec never returns."
If exec never returns isn't that a problem?
This might be nonsense so forgive me if I'm talking about an unrelated issue.
Btw,
Can you extract this if into a private method with a meaningful name? I won't remember this in 3 months

the java script from bazel already calls exec. The problem is, it is calling exec on a script that does not call exec. So, we are breaking the intent and functionality of exec by our wrapper script.
The wrapper script needs to not exec on the repl, since the repl needs to fixup the stty settings. But in the case of other binaries, we want exec (which is how all the java processes work in bazel generally).
I don't know what you want exactly. Can you tell me where you want more methods? At the point of creating the wrapper script, I added the exec in a case it is sure to be safe (where there is nothing happening but a call to another program).
I will need your help to make it clearer, because it is not unclear to me.
(Also note, this is blocking us and causing us to resort to hacks of using sed on the inner scripts generated to manually insert the exec after the fact, which is obviously a bad state to be in. I'd like to get this fixed before we cargo cult this bad sed pattern all over our code and we try very hard to stay on master).

I meant another method in the bzl file to encapsulate the details of the if and the exec/empty-string
I think it's unclear but it might be due to this being tacked on the Bazel java wrapped.
If any of this makes sense and you have an idea that's great, otherwise merge it as is.

Since you say it is okay, I'm going to merge and let us file an issue. I don't really see that the code is cleaner by moving these few lines into a separate function as it is only called once. If you feel more strongly, you are welcome to make an edit that improves things which maybe I don't see right now.


v 55.4 custom config in docker-  issue or docs/usability issue

Potential issue with the custom config feature with docker crate container. I didnt get any success neither with -des config or editing existing crate.yml file within container.
My environment Virtualbox, Ubuntu Server 1604 and Crate docker image 55.4. So Im in a Ubuntu env.
Tried to follow docs to set custom cluster name and node name. This happened. Maybe I misunderstood docs or its a usability issue.
a) crate -Des.cluster.name=testcluster
I used 'docker exec -it -u crate "containername" sh' to get into the container.
tried first as root but then elastic throws an error due to set default security level, changed to log into container as user 'crate'
I went to dir /crate/bin and tried to run "crate -Des.cluster.name=testcluster"
Then a second crate db instance was launched inside the container at published port 4201.
I just wanted to change clustername, not launch a new crate.
b) Tried to edit the crate.yml directly in the container.  Its content was just three diff path infos compared to host-installations crate.ymls. I added inside the container 'cluster.name = "testcluster"'. Restarted the container but it closed down immediately.
My final solution which worked fine was to use the crate.yml file from crate host-installation which includes all options etc. Made changes inside this one and copied this file to the crate.yml inside the container. Restarted and it worked just ok.
Read that support of manual editing of crate.yml not supported after v55 but I either didnt understood docs or its a issue in the container implementation of custom config.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/37760102-v-55-4-custom-config-in-docker-issue-or-docs-usability-issue?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github).

There is no need to change into the container and modify existing files. NEVER do this!
The are 2 ways of specifying Crate settings:

using the -Des. command line arguments

$ docker run crate:0.55.4 crate -Des.cluster.name=my_cluster_name -Des.node.name=node_1 ...

mounting the crate.yml configuration file as described here: https://github.com/crate/docker-crate#use-custom-crate-configuration

$ docker run --volume /custom/path/to/crate.yml:/crate/config/crate.yml crate:0.55.4 crate

Usage instructions for the Crate docker container can be found on DockerHub.

ok, thanks for answer.


MINOR: fix zookeeper_migration_test.py

#13390 remove the remote_kraft and the replacement is isolated_kraft
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I think we should be backported to 3.5 too

see #13630


Add Michman to Hosting section

Would be a great honour if Michman gets into the hosting section of this list.

Hi @kondratovbr, I was curious how long Michman has been around? It looks promising, but this is the first I have heard of it.

We are going to let this bake a while. Happy to look down the road though and this might be a service I try out because it looks neat.


Adds support for an OS agnostic web browser

As per @bencoman suggestion - pulls in a web browser package that should work on all OS's (and ultimately might chop the legs of OSProcess)

Needs testing on windows via the Exercism menu

@macta I tried testing this on my Windows 10 machine. I had no problems installing our dev repo in the image.
I had problems installing the ExternalWebBrowser from these instructions. I got the following error:
Could not resolve: StateSpecs-Specs [StateSpecs-Specs] in C:\<long file path to>\pharo-local\package-cache C:\<long file path to>\pharo-local\iceberg\dionisiydk\StateSpecs (Libgit)
I'll try to debug this but I don't have a deep understanding of how packages are installed on Pharo, so this might be beyond my current skill set.
Looks like testing this on Windows will be stalled for a while.

@samwson it’s alteady been added to the baseline so installing our project should have brought it in automatically and then that menu item will just use it.

@samWson, The following worked for me in a fresh Pharo 60540 image on Windows 10.
Metacello new
	baseline: 'ExternalWebBrowser';
	repository: 'github://seandenigris/Pharo-Web-Browser:master/repository';
	onConflict: [ :ex | ex allow ];
	load.
ExternalWebBrowser new open: 'http://google.com'

...long file path to...

@samWson, Your problem may be that long file path.
Try doing the same from an image located in C:\Temp.
Long filenames are one of the irritating constraints of Windows that should have a better workaround.

Guys - does the normal baseline just load it for you automatically (this branch has added it as a pre-req) ?

I'm weak with Metacello/Baseline stuff. @seandenigris @estebanlm could you advise...
Diff
[Full] (https://github.com/exercism/pharo/blob/a6290d4f412d12af86d7e7b0136bb4b4ef3d0526/dev/src/BaselineOfExercism/BaselineOfExercism.class.st)

I think @gregcline confirmed the baseline I created worked?


Added docker container for quick review of an application.

Can you confirm that the docker instance still works with https://github.com/vincentcox/StaCoAn/tree/develop?
I am cleaning up the structure. Currently I am unable to run docker because I am running VMWare to test the different builds. Unfortunately, hyper-V does not like VMWare so I cannot test it on my device.

I update the docker setup with a drag and drop interface. It's now pushed to  develop.
Again, I have no access to docker due vmware, so it would be a huge favour if you could test this again.
Do you mind if I put you in a "special contributor's" tab in the README? If you like I link your GitHub or twitter account. Your docker idea and code gave this project something extra so I would like to express my appreciation.

Hi ! I just took a look at it.
The docker now is much much smaller, but here: https://github.com/vincentcox/StaCoAn/blob/develop/src/helpers/server.py#L442
You have hardcoded a port (8080) for the report to be opened, but by default, the web server is running in port 8000, so when you click on it, you get an error, as the server is not running in that port.
Am I missing something here?
Feel free to add me wherever you want. Glad I helped a little !

Hi there,
You were correct!
Somehow my latest commit didn’t get trough.
Now it should work, can you try again?
I will add your twitter handle in the next commit! 🙌

On 26 Feb 2018, at 01:14, Borja Berastegui<EMAIL_ADDRESS>wrote:
Hi ! I just took a look at it.
The docker now is much much smaller, but here: https://github.com/vincentcox/StaCoAn/blob/develop/src/helpers/server.py#L442
You have hardcoded a port (8080) for the report to be opened, but by default, the web server is running in port 8000, so when you click on it, you get an error, as the server is not running in that port.
Am I missing something here?
Feel free to add me wherever you want. Glad I helped a little !
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Isn't supposed to run the web server if I start it like this instead of giving this error?
# python /StaCoAn/src/stacoan.py --enable-server
[INFO] Decompiling app...
[ERROR] No mobile app detected, exiting! Hgnnnhh

Damn, will look at it this evening. Thanks for testing it!

Just to double check, did you changed https://github.com/vincentcox/StaCoAn/blob/develop/docker/Dockerfile#L19 to RUN git clone https://github.com/vincentcox/StaCoAn/ -b develop?

Yup:
`root@4eefa16aef59:/StaCoAn/src# git branch

develop`

The HTML I end up with is like this:
Open report!

https://github.com/vincentcox/StaCoAn/commit/b2099e38401e1d53ebd45d975a1026a7b3e9340e
-> This fixes the path error you have.
About your comment over the port; there are 2 ports used:

8000 for drag and drop server
8080 for the report server

https://github.com/vincentcox/StaCoAn/tree/develop#top-contributors There you go! 😃

@vincentcox  Then, you might want to EXPOSE 8080 in the Dockerfile


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-06-05T14:13:49+00:00

/retest


Halloween theme, candy corn (Can close issue?- See issue #28)

The game seriously lacks Halloween UX. The challenge is to create your own assets for the candies and add incorporate them in the game.

Okay! I'm also new. I thought it would be worth a shot if the attitude was that we could have basically an unlimited amount of art assets. But of course I'll adhere to the new colorstandard.

I will submit a PR for this, but in case anyone needs it sooner, here is the color palette.
Orange #FF550F
Yellow #F2DD20
Green #72BC4B
Purple #50338E
Black #000
White #FFF

Hi! I have read your comments (@Alexandria @j-kincaid) and guess I can mention you directly here as well. Glad to make PR with them if/when any of these aren't totally useless. Here are the seven candies I did without adherence to colour palette in 70x70 pixel art. I have tried to make common candies and being able to match up with the current ones so the shape is kinda similar, but that went so-so. I have commented the names here and sometimes changes I can think of to make it the right colours?
 Brain candy / This might not work very well.
 Candy Corn/ As per suggestion but could do smaller color correction
 Chocolate Candy / Maybe it's fine with this small variant?
 Candy Teeth / How does zombie teeth sound?
 Chocolate in Pumpkin Foil / Smaller color correction
 Skull Candy/ I can see it in either purple, yellow or green color also.
 Gummy Worm / Smaller color correction or any fun color
@lizwait, forgive me, the contrasts aren't very good. ☺

@silverwavetidaldragon I can work with these, with some changes for accessibility! I will test different background colors for the game board for visibility considerations. If you go forward with a PR we can all review them.


AyusGup: Loading Spinner Component added

I have created a loader component that is taking width and height as props that is rendered wherever required.
Fixes #23

please merge my pr under hacktoberfest label

Hi @AyusGup
Here's a Holopin Badge that you can collect as a gift for contributing to Lumix!
https://holopin.io/collect/clo2tgips49930glf7mhyunyy


Presentation -- enable vertical navigation

When expanding a bullet, often part of the content is not visible. Reveal.js has vertical navigation but it works for defined slides. In the case of Roam Presentation, it needs to work more like page down/up.

duplicate of https://github.com/dvargas92495/roam-js-extensions/issues/585?

Indeed but might be interpreted differently, so if the two descriptions are merged into the same enhancement to clarify the scope (might be two different parameters to enable diff behaviour -- step/slide per bullet vs page down), I'd be happy to support it.


run Linux tests on Darwin as well

I made sure they run fine. This change helps us run the testsuite in Nixpkgs.

Thanks for the PR, and I am quiete sad to have tp turn it down.
But the same reasoning as on PR 86 applies here as well.
Enabling tests for more platforms means we must also be ready to handle issues for these platforms and ensure they work in the future.
I already have my hands full handeling Windows and Linux and am not capable/willing to handle OS's I am not familiar with (and don't have running locally...).
This is why darwin and BSD are listed as "enabled, but not supported" and the testfolder is still named "linux" and not "unix"

This is why darwin and BSD are listed as "enabled, but not supported" and the testfolder is still named "linux" and not "unix"

That could remain the case. It's just useful for e.g. Nixpkgs to be able to test correctness of this package on Linux as well as Darwin.
I assume you'd be happy if we reported an issue or filed a PR if the testsuite ever failed on Darwin?

Yes, please open issues and PRs if you find any problems. If they are solvable on the linuxside of the code I will be happy to fix.

just to clarify a little more:
currently the library has completely separate code for Windows and Linux, both for the implementation and the tests. This hold a lot of duplicate code in is quite annoying to maintain. This also means that adding support for a new OS requires copieing and modifying all that code again, which is especally hard when I don't have a local dev-system for that OS. Thats why I am currently not doing anything in that regard.
I know this is a controversial point, but for me enabling the tests for a OS means I obligate myself to that OS and should have it working. For this reasons I don't accept this PR at the moment.

please keeps this open. This is still a valid PR and I want to make it easy for people to find this and read why this is currently on hold 😄


print status to stderr and add mode to just print urls to stdout

it lets me do this for parallel downloading:
~/src/js/himawari.js/tmp 🐈  himawari --zoom=5 --urls | xargs nugget -c
Creating undefined in /Users/max/src/js/himawari.js/tmp ...
Downloading 050000_2_5.png
[=======================================>     ] 91.1% of 510.54 kB (11.02 kB/s)
Downloading 050000_2_8.png
[=================>                           ] 41.8% of 527.14 kB (4.86 kB/s)
Downloading 050000_2_9.png
[=======================================>     ] 89.5% of 500.58 kB (11.46 kB/s)
Downloading 050000_2_11.png
[===========================================> ] 97.9% of 506.16 kB (7.44 kB/s)
Downloading 050000_2_12.png
[===========================================> ] 98.1% of 560.04 kB (19.62 kB/s)
Downloading 050000_2_14.png
[===================>                         ] 46.3% of 504.01 kB (3.5 kB/s)
Downloading 050000_2_15.png
[>                                            ] 0.0% of 520.6 kB (0 B/s)
Downloading 050000_3_4.png
[===============>                             ] 37.4% of 637 kB (5.6 kB/s)
Downloading 050000_3_5.png
[=================================>           ] 76.6% of 641.74 kB (16.22 kB/s)
Downloading 050000_3_10.png
[================================>            ] 74.9% of 512.47 kB (11.89 kB/s)
Downloading 050000_3_12.png
[===============>                             ] 37.6% of 533.32 kB (540.88 B/s)
Downloading 050000_3_15.png
[========================>                    ] 55.8% of 543.05 kB (12.65 kB/s)
Downloading 050000_3_17.png
[========================================>    ] 91.6% of 278.81 kB (10.13 kB/s)
Downloading 050000_4_2.png
[============================>                ] 66.2% of 448.84 kB (37.15 kB/s)
Downloading 050000_4_3.png
[===============================>             ] 71.2% of 489.58 kB (12.63 kB/s)
Downloading 050000_4_5.png
[====================>                        ] 48.2% of 592.29 kB (1.42 kB/s)
Downloading 050000_4_6.png
[==============================>              ] 70.5% of 522.01 kB (7.97 kB/s)
Downloading 050000_4_7.png
[================>                            ] 39.5% of 602.71 kB (1.53 kB/s)

... and 381 more

Combined Speed: 175.63 kB/s

(doesnt integrate with the mosaic step but i might send another pr for that in the future)

Thank you so much for your willingness to contribute! Couple questions, though.

Why log those informative pieces to stderr?
Async.js is used heavily in this project. There are some weird things going on under the hood of himawari.js that seemingly disable its ability to parallelize downloads, but using something like 'async.map' might yield better results. Would you want to try that instead?

@jakiestfu I think switching from stdout to stderr for those info messages makes it so that piping works more predictably (nothing is piped except the URLs). @maxogden Might I suggest something like --urls-only so that only errors are piped to stderr, but your use case it also supported?
Perhaps another (maybe even better) alternative is to create a standalone module that handles fetching and returning the URLs that support the needs of himawari.js and @maxogden's experiments.

I'm going to take a crack at making a himawari-urls module that fits the needs of this module and @maxogden's use case based off of @jakiestfu's work. Will let you know when it's published.

Okay I made the thing: https://github.com/ngoldman/himawari-urls
var himawariURLs = require('himawari-urls')

himawariURLs({
  date: 'latest',
  infrared: true,
  zoom: 1
}, function (err, tiles) {
  if (err) console.error(err)
  tiles.map(function (t) { console.log(t) })
})

// { name: '0_0.png',
//  url: 'http://himawari8-dl.nict.go.jp/himawari8/img/INFRARED_FULL/1d/550/2016/02/10/192000_0_0.png',
//  x: 0,
//  y: 0 }

@maxogden himawari.js now supports parallelized downloads with the -p flag (himawari-bg is parallel by default), and himawari-urls supports your use case.
However I don't think it's a bad idea to support the above use case. @jakiestfu would you accept a PR that only prints urls to stdout then exits and silences all other non-error output?

@ngoldman All I'd have to do is provide the URL in the chunk callback and you can push those to an array. I could consider doing a "dry" feature/flag where no downloads are actually executed too.
Thoughts?

@jakiestfu I think that would work. So the desired outcome would be something like:
var himawari = require('himawari')

himawari({
  dry: true,
  chunk: function (info) {
    console.log(info.url)
  }
})

$ himawari --zoom 2 --dry
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_0_0.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_0_1.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_0_2.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_0_3.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_1_0.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_1_1.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_1_2.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_1_3.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_2_0.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_2_1.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_2_2.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_2_3.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_3_0.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_3_1.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_3_2.png
http://himawari8-dl.nict.go.jp/himawari8/img/D531106/4d/550/2016/02/17/222000_3_3.png

right?

Version 1.3.2 now supports urls flag for only printing URLs :)


(PUP-4602) Add rcng service provider

Here's a new provider for service functionality through rcng, while it shares some traits with the bsd provider it differs enough to warrant a new one due to the different paths and content of files that need to be enabled.

CLA signed by all contributors.

Hi @jasperla.  Thanks for the contribution.  Besides the minor comments above, this looks good to me.
Note to reviewers: AppVeyor failure is the result of the spurious rdoc spec.

Hi @jasperla.  Have you had a chance to review @joshcooper's feedback?  Thanks!

Hi, I've been too busy recently to handle this. I should be able to incorporate the feedback later this week or latest early next week.

Hi @peterhuene , @joshcooper 's feedback has been incorporated now.

@peterhuene Could you take another look at this now that it's updated?

@jasperla This looks good, other than my above comment above about the mode '0644' being specified in multiple places and is perhaps a candidate for extracting into a self.default_mode into the base BSD service provider method like lib/puppet/provider/sshkey/parsed.rb has.  Thoughts on extracting that out or are we happy with having the mode specified in multiple places?

@peterhuene I'm fine with having it specified in multiple locations; it feels like a piece of data too trivial for it's own method.

Good enough for me then. :+1:

@jasperla Thanks again for the contribution!  This has been merged to master and will appear in Puppet 4.3.0.