Not able to make change background color to transparent

Not able to make change background color to transparent, from bottom sheet background color option is available

HAving the same issue, When assign Borders to the Container
 var value = await showTopModalSheet<
                                            String?>(
                                        context,
                                        Container(
                                          height: 100,
                                          decoration: BoxDecoration(
                                              color: hexStringToColor("151515"),
                                              borderRadius: BorderRadius.only(
                                                  topLeft: Radius.circular(20),
                                                  bottomLeft:
                                                      Radius.circular(20),
                                                  bottomRight:
                                                      Radius.circular(20),
                                                  topRight:
                                                      Radius.circular(20))),
                                        ));

there is background color white shown.

Temporary Solution:

Goto the showTopModalSheet and Change the  barrierColor: Colors.transparent and its fixed.


[website] Add more tags for icon search

Adding more tags to icons for better search + minor fixes to routing issue in icons page.

I'm not sure what happened but this isn't a clean commit so can you fix that up?


@react-aria/toast causing Dynamic require of "react" is not supported

🐛 Bug Report
When instantiating a new ToastQueue, or using the useToastState hook, I get a runtime error Dynamic require of "react" is not supported.
🤔 Expected Behavior
There should be no runtime error when using Toasts.
😯 Current Behavior

💁 Possible Solution
I tried to externalize react, which initially removed the Dynamic require error, but it led to having 2 versions of react in the DOM.

🔦 Context
We are trying to make use of @react-aria/toast to handle Toasts for our design system.
I feel as if this could be related to using esbuild/tsup? Here is our config if that may alude to anything:
export default defineConfig({
	outDir: "dist",
	format: ["esm", "cjs"],
	dts: true,
	platform: "browser",
	external: ["react"],
	clean: true,
	splitting: true,
	treeshake: true,
	entry: ["src/**/*.ts"],
	esbuildPlugins: [svgrPlugin(), dynamicImport()],
});

💻 Code Sample
This is enough to cause the runtime error.
const toastQueue = new ToastQueue<ReactNode>({
	maxVisibleToasts: 5,
});

Another way is:
const state = useToastState<ReactNode>({
    maxVisibleToasts: 5,
});

🌍 Your Environment

Software
Version(s)

react-aria
3.26.0

react-stately
3.24.0

Browser
Firefox Dev 116

Operating System
Windows 10

@react-aria/toast
3.0.0-beta.2

🧢 Your Company/Team
Lightspeed HQ

🕷 Tracking Issue (optional)
n/a

I don't think anything in our code is doing a dynamic require, but maybe related to the external: ["react"] in your config? Not really familiar with esbuild though.

Appreciate you getting back! I haven't had a chance to debug further :( Marking react as external here means that esbuild assumes the module is available at runtime. So react doesn't need to be bundled with our package. This is why without external, our tests resulted in 2 versions of React in the dom.
I'd be happy to try any ideas, otherwise I think its safe to call this a build tooling issue and you can close 🙂

Appreciate you getting back! I haven't had a chance to debug further :( Marking react as external here means that esbuild assumes the module is available at runtime. So react doesn't need to be bundled with our package. This is why without external, our tests resulted in 2 versions of React in the dom.
I'd be happy to try any ideas, otherwise I think its safe to call this a build tooling issue and you can close 🙂

@kysley  Did you ever figure out a solution for this?  It seems like these components are meant for building component libraries, but they do not seem to be compatible with bundling tools like esbuild or tsup.  I am finding it very difficult to build a reusable component.


test_rpc_misc fails

Appeared with #246
State diff does not match. Perhaps the tx fails because of invalid program (incompatible after updating to 0.10.0)?
@drknzz

It appears that the previously referenced here line is the problem, as without it the state diff is as it should be.
Once again, I do not know why it was there already in the first place when I started working on RPC, but probably just a bug.

Great! thanks for looking into this.


Fix for tests failing on Un*x due to "file:///"

Testing. Do not merge yet.

@dotnet-bot build Innerloop CentOS7.1 Debug Build and Test


tombstone

I searched through source code of mapd-core and found no word like "tombstone".
It may hint that mapd-core doesn't do any data (file) compaction for now. Is compaction in future milestone?
My point for such tombstone is similar to that in issue 86. If compaction is supported (thus is DELETE), by augmenting mapdql in similar appraoch, adding UPDATE function to mapd is possible without significant changes to mapd core.

Updates and deletes are definitely on the road-map. And yes, compaction too.


Replacing mouse events and location with multi-touch (TUIO)

I'm trying to replace the mouse input with TUIO based multi-touch (https://github.com/borismus/MagicTouch) for polyline creation. A touch controlled div will function as the location input, but unfortunately I can't get it working.
How would I replace _onMouseDown, _onMouseMove and _onMouseUp in leaflet.draw with a custom input?

Please see https://github.com/Leaflet/Leaflet.draw/issues/2 for touch support.


[Admin][TerritorialCouncil] Add Political committee members on Territorial council admin pages

[ch12237]
[ch12246]


[Feature] Show details of Nostr user you are paying rather than the website

Feature description
When zapping a nostr user on Nostr sites, it doesn't make sense that we are paying the website (we aren't). It is especially confusing when doing split payments.

Describe the solution
Show the name or profile of the person who we are zapping
Describe alternatives
No response
Additional context
No response
Are you working on this?
None

Since there is just an invoice being passed to webln.sendPayment() I wonder where you would get this information from?

@reneaaron hmm, yes I guess this is not possible, not without storing some reference inside the invoice, which we currently do not do, and all zap providers would need to do it too.

I don't think there is currently a way to achieve this. Feel free to re-open this issue if you ever have a genius idea.


[gauge] displaying field name instead of label

Gauge char displaying field name instead of label
Expected results
In the images attached, it should say Año (year), instead it says ano (asshole literally)
Actual results
what actually happens.
Screenshots
If applicable, add screenshots to help explain your problem.

Definition of the columns

Gauge char displaying the work word.

Environment
(please complete the following information):

superset version: 0.999.0dev
github image: '2db7d9c927ca'

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

😂 is it only happening to gauge? or other charts as well (?)

I've only seen it in gauge

Is this still an issue in 2.0.1? Hoping to validate this, as some changes were made to that chart.

Looks like this should've been closed by the linked PR.


[FEATURE REQUEST] Determine next epoch nonce without local DB

Hi!
I would like to know if its possible to update the nonce command to be able calculate the nonce of the next epoch without using the local DB. I'm currently using BlockFrost for an internal project but,  unfortunately its not possible to get the nonce of the next epoch in advance.
Is it possible to improve something on your side ? If its not relevant for this project, I'd be happy if you could tell me what data I need to retrieve to make this calculation?
Thanks @AndrewWestberg for the previous feature you've implemented it works :)
Thanks you!

You can find nonce for next epoch once available from https://api.koios.rest/#get-/epoch_info OR use CLI to capture protocol-state which will include next epoch's nonce

@rdlrt Thanks for your reply. As we are mainly using BlockFrost currently, we can't change and use another external services.I looked at the cardano-cli query protocol-state command. The field corresponding to the next epoch is this one ?
 "evolvingNonce": {
        "contents": "d7213c6d8fef09ea65c886efc0c4ad35f567951f10f5baef42658a6d1c034045",
        "tag": "Nonce"
  },

See reference here for deriving nonce using that output


Fixed #30646 -- Always close unusable connections

Ensure that unusable connections are always closed whether or not an
error related to the connection has occurred previously.
https://code.djangoproject.com/ticket/30646

Closing per ticket.


Terrain Project uses different level numbers on CLI than displayed in Fusion UI

Reproduction steps

Create a terrain resource and note its maximum level
** You can do this by temporarily adding it to a terrain project using the Fusion UI, then remove it afterward
Use genewterrainresource (or gemodifyterrainresource) to add your new resource and specify maximum level of 1 less than the maximum shown in step 1$  /opt/google/bin/genewterrainproject -o Projects/MyTerrainProject --maxlevel 29 Resources/Terrain/MyTerrainResource

Open Asset Manager in Fusion UI
Open the new terrain project that was created
Look at the override level indicated in the Terrain Project window (You may have to expand the window to see all of the text)

Expected: The terrain shown in the Terrain Project window displays the same override level than the one specified on the CLI
Actual: The terrain shown in the Terrain Project window displays an override level that is 5 levels lower than the one specified on the CLI

It seems that internally, Fusion uses different level ranges than the ones shown in the Fusion UI. I'm not sure why this is, but the number used in the CLI should match the one shown in the UI. Perhaps deprecating the --maxlevel parameter and adding a new one that does match in order to support legacy scripts, but also help eliminate confusion because of the discrepancy?


Back button will not send the app to the background

Version
Tell us which versions you are using:

react-native-router-flux 4.0.0-beta.40
react-native 0.55.4

Expected behaviour
If you are navigating inside the app, after pressing multiple times back button on android, the app should be sent to the background the same way when you press the menu button
Actual behaviour
After pressing back button the app will come back to the initial screen (correct), but then it stays on the screen no matter how many times you press back button.
Just before this library was updated I had an issue where the back button would kill the app, so I fixed it and it was working perfectly. After updating flux to 4.0.0-beta.40 I have this issue .
Keep in mind that the scene structure did not change at all from what it was in the previous version

Same issue, when in the initial page, the hardware back button has no effect at all.
You may implement the BackHandler yourself, if you really need it right now.
https://facebook.github.io/react-native/docs/backhandler

@matthewkwong2  I added backAndroidHandler in my route with the function
onBackPress = () => {
        if (Actions.state.index === 0) {
          return false
        }
        Actions.pop()
        return true
    }

And it works fine for now, but I did not needed to do that before the new update

With the upgrade - probably the app structure will have to be updated.
Could you provide a sample of your structure so we can work with it to see how proceed and maybe add a migration guide?

Closed because of inactivity

@aksonov @daviscabral I'm having this issue as well and I think I've narrowed it down:
onBackPress = () => navigationStore.pop();
https://github.com/RNRF/react-native-router-flux/blob/master/src/Router.js#L36
calls:
pop = ({ timeout, key, ...params } = {}) => {
    const res = filterParam(params);
    if (timeout) {
      setTimeout(() => this.pop(params), timeout);
    } else {
      this.dispatch(NavigationActions.back({ key }));
      if (res.refresh) {
        this.refresh(res.refresh);
      }
    }
    return true;
  };

https://github.com/RNRF/react-native-router-flux/blob/master/src/navigationStore.js#L952
This issue is that it needs to return false if the states are equal to exit the app, like it did with with 4.0.0 beta's:
return !isEqual(previous, getActiveState(this.state));
https://github.com/RNRF/react-native-router-flux/blob/4.0.0-beta.31/src/navigationStore.js#L1040

@aksonov @daviscabral
Please reopen this issue. It's not fixed in 4.0.5 yet
Here is repository to reproduce

Facing same problem.
Upgraded to 4.0.6 and my app won't go to background when I return false in backAndroidHandler's function like it did in the v4 beta versions.

@wmonecke
Hopefully you've resolved the problem already.
But if you haven't, you can do something like this.
  componentDidMount() {
    BackHandler.addEventListener('hardwareBackPress', this.onBackPress);
  }

  componentWillUnmount() {
    BackHandler.removeEventListener('hardwareBackPress', this.onBackPress);
  }

onBackPress = () => {
    if (Actions.state.index === 0) {
      return false
    }
    Actions.pop();
    return true
  }


(GH-2956) Remove build.config from Frosting template

Removes build.config from Frosting template. Cake version from the build.config file was not used and .NET SDK version is now used from global.json file which has been added to the template. One side effect of this approach is that we always download the .NET install script on Windows, but this is something which we already did on Linux/macOS.
Also cleaned up some stuff which was in the bootstrapper script, but not used.
Fixes #2956

When running the build.sh on WSL/Ubuntu I get the folllowing error:

But work for @gep13 on macOS.

@cake-build/cake-team Is anyone else in a position to take this change for a spin?  As @pascalberger mentioned, this is working for me on my Mac.

Isn't this taken care of in #2946?

I can't say since there's not much of information what exactly changed in the PR and a lot of files have changed. On a quick glance the file seems to be removed, which might solve it  (but I don't see how it is done after #2946)

Had another look at #2946 and agree that the way bootstrapping is done in #2946 will make this obsolete


Discord crashes

Everytime I open discord with betterdiscord, my discord crashes.

This is a plugin-specific issue.


[SPARK-25202] [SQL] Implements split with limit sql function

What changes were proposed in this pull request?
Adds support for the setting limit in the sql split function
How was this patch tested?

Updated unit tests
Tested using Scala spark shell

Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can you add tests in StringFunctionsSuite, too?

ok to test.

Test build #95237 has started for PR 22227 at commit ceb3f41.

@phegstrom Thanks for your contribution!
Btw, seems like your email address in your commits is not connected to your GitHub account. Could you connect the address to your account, or use other address connected to your account? Otherwise, your contribution will not be connected to you.

Test build #95237 has finished for PR 22227 at commit ceb3f41.

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

Merged build finished. Test FAILed.

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

Test build #95311 has started for PR 22227 at commit 4e10733.

Test build #95311 has finished for PR 22227 at commit 4e10733.

This patch fails from timeout after a configured wait of `400m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

@maropu @HyukjinKwon let me know what you think, took care of your comments

Test build #95388 has started for PR 22227 at commit e8c8c8c.

Test build #95388 has finished for PR 22227 at commit e8c8c8c.

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

Merged build finished. Test PASSed.

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

Test build #95470 has started for PR 22227 at commit 8e16328.

Test build #95473 has started for PR 22227 at commit ca23ea3.

Test build #95473 has finished for PR 22227 at commit ca23ea3.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #95476 has finished for PR 22227 at commit 96bc875.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #95486 has started for PR 22227 at commit 79599eb.

Test build #95486 has finished for PR 22227 at commit 79599eb.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Test build #95494 has started for PR 22227 at commit fa128db.

Will do @rxin. Question -- does this need to happen in nullSafeEval() in the StringSplit class? Unsure where the 0 -> -1 logic should live.

Test build #95494 has finished for PR 22227 at commit fa128db.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

You need to handle that in both codegen(doGenCode) and interpreter(nullSafeEval) path. Also, can you add tests to check if they have the same behaviour in the limit=0/limit=-1 cases? You'd be better to explicitly put comments in a proper position about the decision.

Test build #95511 has started for PR 22227 at commit a641106.

Test build #95511 has finished for PR 22227 at commit a641106.

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

Merged build finished. Test FAILed.

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

@maropu @rxin made those changes to fallback to -1 if limit = 0, let me know what you think.
@felixcheung updated R tests as well

Test build #95558 has started for PR 22227 at commit 7e4ba98.

Test build #95558 has finished for PR 22227 at commit 7e4ba98.

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

Merged build finished. Test FAILed.

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

Test build #95578 has started for PR 22227 at commit d17d2df.

Test build #95578 has finished for PR 22227 at commit d17d2df.

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

Merged build finished. Test FAILed.

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

Test build #95774 has started for PR 22227 at commit 64b0afc.

Test build #95774 has finished for PR 22227 at commit 64b0afc.

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

Merged build finished. Test FAILed.

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

hey @HyukjinKwon , the SparkR test now failing is
Duplicated \argument entries in documentation object 'column_string_functions':
  'pattern'

Functions with \usage entries need to have the appropriate \alias
entries, and all their arguments documented.
The \usage entries must correspond to syntactically valid R code.
See the chapter 'Writing R documentation files' in the 'Writing R

Can I not have @param pattern twice in the docs for the rdname column_string_functions?

Test build #95803 has started for PR 22227 at commit 4e84df0.

Test build #95803 has finished for PR 22227 at commit 4e84df0.

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

Merged build finished. Test PASSed.

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

@HyukjinKwon all tests now passing! Let me know if there is anything else to do for this PR. Thanks

Test build #95872 has started for PR 22227 at commit b12ee88.

Test build #95872 has finished for PR 22227 at commit b12ee88.

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

Merged build finished. Test FAILed.

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

hey @HyukjinKwon, I'm seeing two tests fail in the most recent run, but traces don't seem related to any of my changes -- not to mention the test name is (It is not a test it is a sbt.testing.SuiteSelector). Is this something I need to worry about?

retest this please

Test build #95907 has started for PR 22227 at commit b12ee88.

retest this please

Test build #95918 has started for PR 22227 at commit b12ee88.

Test build #95907 has finished for PR 22227 at commit b12ee88.

This patch fails from timeout after a configured wait of `400m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #95918 has finished for PR 22227 at commit b12ee88.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

hmm @HyukjinKwon any ideas what's happening?

retest this please

retest this please

Test build #95987 has started for PR 22227 at commit 69d2190.

Test build #95987 has finished for PR 22227 at commit 69d2190.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #96015 has started for PR 22227 at commit 69d2190.

Test build #96015 has finished for PR 22227 at commit 69d2190.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #96021 has started for PR 22227 at commit 69d2190.

Test build #96021 has finished for PR 22227 at commit 69d2190.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #96030 has started for PR 22227 at commit 69d2190.

Test build #96030 has finished for PR 22227 at commit 69d2190.

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

Merged build finished. Test PASSed.

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

Seems fine otherwise.

Test build #96075 has started for PR 22227 at commit b5994ad.

Test build #96075 has finished for PR 22227 at commit b5994ad.

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

Merged build finished. Test PASSed.

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

Test build #96112 has started for PR 22227 at commit 5c8f487.

Test build #96112 has finished for PR 22227 at commit 5c8f487.

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

Merged build finished. Test PASSed.

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

long thread, are we all good with this?

Can one of the admins verify this patch?

LGTM except for the R/python parts (I'm not familiar with these parts and I'll leave them to @felixcheung and @HyukjinKwon).

ok to test

Test build #96481 has started for PR 22227 at commit 5c8f487.

Test build #96481 has finished for PR 22227 at commit 5c8f487.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

@phegstrom, can you close and reopen this PR to retrigger the AppVeyor test above? Looks it's failed due to time limitation.

@HyukjinKwon are things passing?

retest this please

Test build #96966 has started for PR 22227 at commit 34ba74f.

Test build #96966 has finished for PR 22227 at commit 34ba74f.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #96981 has started for PR 22227 at commit 34ba74f.

Merged to master.


Adding 45 p5js firebase drawings

adds challenge files
updates contribution
includes thumbnail image
updates challenge details

I had to be a tad creative with the images. I hope they are satisfctory.
At least they might make you chuckle :)

Thank you!!  I personally love your image!  Here are some relatedChallenges to add:
"relatedChallenges": ["122-quick-draw","130-drawing-with-fourier-transform-and-epicycles"]

That drawing is incredible @borntofrappe! @jasontheillustrator you have got to take a look!

I'll wait to merge in case you want to add the related challenges suggested by @kfahn22!

Thanks for the kind words @shiffman 😄


Cannot copy a file from a UNC path when the filename contains both a hash and whitespace characters

Which version of the AzCopy was used?
10.1.2
Note: The version is visible when running AzCopy without any argument
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
.\azcopy cp "\testserver\testshare\file with space and #in-the-name.zip" "https://somestorage.blob.core.windows.net/somecontainer/file with space and #in-the-name.zip?st=someSASToken"
What problem was encountered?
Cannot copy the file to the blob storage and gives a fail to copy in the results
How can we reproduce the problem in the simplest way?
Try to copy a file with azcopy from a windows file share to a blob storage container WHERE the source file name contains both hash (#) and whitespace ( ) character(s)
Have you found a mitigation/solution?
Remove all # charcters OR remove all whitespace characters from the source file

Thanks @hentie. We've logged this for investigation

This one works in the latest release (10.3.0)! Enjoy!


Artifacts?

Where are the artifacts?

Hmm good point, I didn't want to have them under version control as before, but its definitely something I can add to the CI.


1.8.7 release

Bug fixes. Target date: 2019-09-24.

Done: https://pypi.org/project/Fiona/1.8.7/#files.
There are a couple changes for the binary wheels:

GDAL version is 2.4.2
Raster format drivers have been eliminated
Shared libraries symbols have been stripped

The wheels should be lighter and faster in some cases.


file naming and organisation in io/params/parsing/io_parse_settings

I think that we should be more precise in the naming of the toml files for opening/parsing the input files of the different modules, specially if these are all in the same folder.
I say that because the different tools have several outputs, and it is not the same output table that will be used if we want to run the DDA peptidoform, DDA precursor ion or DIA module. It is at least the case for Proline, and I think for i2MassChroQ. I also see that this is the case for FragPipe that already has a fragpipe.toml and fragpipe_DIA.toml.
I think that it would be better to be very systematic: fragpipe_DDA_ion.toml, fragpipe_DDA_peptidoform.toml, fragpipe_DIA_ion.toml...
Maybe there is an even better way to handle this.
I know that in some cases, the toml will be exactly the same for several modules. But wouldn't it be cleaner to still have several toml? The idea is to make it simpler for people who want to add tools. Of course this is very open to discussion.

Hi Marie,
In trying to implement the diaPASEF ion quant module that we'll use the newly generated diaPASEF for, I quickly found that indeed, the current way the repo is structured is not very friendly towards adding new modules, especially modules that are very similar but need slight changes. So in the PR I'm drafting for this new module I have (probably gone a bit overboard) and restructured quite a bit. Once I'm done I will let you know, and then we should discuss it a bit more. I might draft the new diaPASEF module again in another PR without the restructuring, so we can take a longer time to discuss the refactoring outside the implementation of the diaPASEF module.

I love this! Thank you so much.
Please let me know if I can help in any way.

Organization is restructured after refactoring (#452). Each module now has its own folder of tomls. There are improvements to be made, because the tomls themselves contain a lot of information that can be generalized, but this can be made into a new issue.


visidata: 2.6 -> 2.6.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for visidata is: "Interactive terminal multitool for tabular data"
meta.homepage for visidata is: "http://visidata.org/"
meta.changelog for visidata is: "https://github.com/saulpw/visidata/blob/v2.6.1/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1/bin/vd passed the binary check.

/nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1/bin/visidata passed the binary check.

/nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1/bin/.vd-wrapped passed the binary check.

/nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1/bin/.visidata-wrapped passed the binary check.

4 of 4 passed binary check by having a zero exit code.

4 of 4 passed binary check by having the new version present in output.

found 2.6.1 with grep in /nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1

found 2.6.1 in filename of file in /nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1

directory tree listing: https://gist.github.com/12adc2f461b5526eece07e06ed7c483d

du listing: https://gist.github.com/d311feab52a6baaf65ddc13838affe8a

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
visidata

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A visidata https://github.com/r-ryantm/nixpkgs/archive/dde4e813d329c6db19dcc4c988927629739ce115.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1
ls -la /nix/store/fxzfsqan36cyyiawjbzw8xhcj38r6y62-visidata-2.6.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

visidata

Maintainer pings
cc @7c6f434c for testing.

Result of nixpkgs-review pr 140074 run on x86_64-linux 1

1 package built:

visidata


Unsupported platform: FreeBSD #722

It's pretty simple. For BSD, you need to use gmake for the gnu makefile.
Thanks.

@darkoverlordofdata here's a semi related question, but you might be able to help answer it. Can you easily run premake on freebsd? I've seen it on freshports and other places so it seems like it's not a stretch. I'd be curious if you can (without much effort) build and run the latest premake 5 alpha there.
https://premake.github.io/download.html#v5

This was resolved in 0.3.0 with a bsd specific make file, see projects/make.bsd


Add initialValue? to QuickPickOptions

It's sometimes necessary to get a pre-filled QuickPick. In my usecase, I want to
write an extension to navigate directories, and to do this, I need the QuickPick
to be pre-filled with the current directory.
I couldn't test this, because I couldn't figure out for the life of me how to load up an extension in the dev vscode. I know about ~/.vscode-oss-dev, but I keep running into problems with the vscode.ts.d (I had to hand-edit this) and engines.vscode settings. Surely, there's a less painful way to co-develop vscode and extensions!

Hi @artagnon, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Coverage decreased (-0.002%) to 61.011% when pulling aa168b1d289e1c51819df99d9b8f35a7e6b8342e on artagnon:quickpick.initialvalue into 1de4d832ccb061c13cf70eaf0221fbaa86abfbdd on Microsoft:master.

@artagnon, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Sorry, this fell the radar. We are now finishing up with the September release. I'll tackle this first thing in Oct.

Anything I can do? This is a pretty straightforward feature; only, isn't it hard to test?

I am sorry, this keeps slipping because the quick open area is a little dated and needs some general polish/revisiting. Because things are a little messy in the past every new feature did brake something . That's why I haven't merged this yet but I am wait for being able to do a larger cleanup.

Our cpptools extensions also wants to set an initial filter value for quick picks too. When is the ETA for this?


Refactor e2e InstallOSM to to support helm install too

https://github.com/openservicemesh/osm/pull/1889
helm install APIs were added to e2e, however it happens to separate apis to specifically just test the install.
It'd be a benefit in the long run that installing through helm was just an option for the test suite and used the available
APIs, so all tests could potentially run with helm installation instead of CLI installation just with a toggle/switch change.

Tests                      [X]

Use cases:
Running the entire suite with helm install/CLI agnostically, using the same test APIs.

There's little value in running the entire suite with and without Helm, as the CLI internally leverages the Helm API. We have a test that installs and uninstalls using Helm, so I hope this suffices.


Missing /e "Reason" examples and parameter options.

Please provide more information on how to properly use the "/e" command switch to provide a Reason.  The Reason Options as well as an example usage would be greatly appreciated.

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

ID: 2770335e-d15c-6389-36b9-d687b7794b92
Version Independent ID: 319c885a-1c12-595f-8018-c563945afe93
Content: shutdown
Content Source: WindowsServerDocs/administration/windows-commands/shutdown.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: alalve

Hey JourneyInAntiquity,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made adding an example on how to use /e parameter. Note that the reason will be tracked into the shutdown event tracker. Changes will be live after the next business day.

To restart the remote computer myremoteserver in 5 minutes due to a security update and include this reason in the shutdown event tracker, type:
shutdown /r /t 300 /e /c "Security Update" /m \\myremoteserver

We will now be closing this thread. Thank you and have a most pleasant day.
#please-close


Alternity-RPG: Add toogle worn and calculated total mass to the equipment list.

Changes / Comments
Added a checkbox to toggle wearing equipment and added a calculated total worn mass to the bottom of the equipment list.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


/admin Page Auth Key not working

After a fresh install i need to input an Auth Key on the admin page.

I generated one and pasted it in the compose-file.
cheshire-cat-core:     image: ghcr.io/cheshire-cat-ai/core:latest     container_name: cheshire_cat_core     #depends_on:     #  - cheshire-cat-vector-memory     #  - embedder     #  - ollama     environment:       - PYTHONUNBUFFERED=1       - WATCHFILES_FORCE_POLLING=true       - CORE_HOST=${CORE_HOST:-localhost}       - CORE_PORT=${CORE_PORT:-1865}       - QDRANT_HOST=${QDRANT_HOST:-cheshire_cat_vector_memory}       - QDRANT_PORT=${QDRANT_PORT:-6333}       - CORE_USE_SECURE_PROTOCOLS=${CORE_USE_SECURE_PROTOCOLS:-}       - LOG_LEVEL=${LOG_LEVEL:-WARNING}       - API_KEY=${API_KEY:-9da0f457-cecd-459e-********}       - DEBUG=${DEBUG:-true}       - SAVE_MEMORY_SNAPSHOTS=${SAVE_MEMORY_SNAPSHOTS:-false}     ports:       - ${CORE_PORT:-1865}:80     volumes:       - cheshire_cat_core_static:/app/cat/static       - cheshire_cat_core_public:/app/cat/public       - cheshire_cat_core_plugins:/app/cat/plugins     restart: unless-stopped
also tried without any API_KEY Variable in compose-file also didn't worked and i got the attempt to input an auth key on the /admin page

Matt is correct, this looks like an issue with Cheshire Cat, please open the issue over there and they should be able to help you out.


PinchGesture & Image out of border

This is what happens when i zoom in and move the image to one of the corners then i zoom out.
It doesn't act like PanGesture taking the image back to the closest corner if its out of border.

This only happens on the emulator, i think.


Stats loop to much giving point of next level

Fixing fake stats

As you say, the function getElementPrice is wrong.
We should fix this function. #241

Hello just to let you know the stats and the costresource are 2 real different errors.
Example:
Costresource function ask you to pay the price for the level you already have instead of the next level
The stat resource add you another extra level on the one you already have
To fix it, i implented this pull request, and i changed on building and research simly $leveltobuild+1 in function show, function addelementtoqueue

I think, if we real problem is described here: #241
If we adjust the values, This should be fine without changing the logic in includes/classes/class.statbuilder.php

After fixing #241, is this issue still valid?

For sure, lets understand your code
For level 1 to a level inferior or egual to what you have build, then it run an extra for
So you have level 20, it will run till level 21, as there is an extra loop for if its equal to yours thanks to the $Level++
simply need to remove that small = symbol
for($Level = 1; $Level <= $USER[$resource[$Techno]]; $Level++)
{
    $TechPoints	+= $Units * pow($pricelist[$Techno]['factor'], $Level);
}

If you remove that = symbol
Then it will make a last loop if the level is at 19 to go to 20. and then will stop. so it will get what you realy have,
hope you see what i mean

Hi!
I asked a friend who is good in math. We optimized the function. Here is our current result:
$TechPoints = $Units * $pricelist[$Techno]['factor'] * ( 2 * (pow($pricelist[$Techno]['factor'], $USER[$resource[$Techno]]) - $pricelist[$Techno]['factor']) + 1);

Means:
Points = (All resources / 1000) * Factor * ( 2 * ( Factor ^ Level ) - Factor) + 1)

There is a chance to test this function?

for sure, i will test i after my IRL work and give you an update around 23h CET

Thanks
Hilarious001 <notifications<EMAIL_ADDRESS>schrieb am Mi., 17. Aug. 2016, 12:09:
for sure, i will test i after my IRL work and give you an update around 23h CET
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/jkroepke/2Moons/pull/240#issuecomment-240369733, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABfQCwbvDlV7g5hqtx36ixxDtcSDdGWRks5qgt11gaJpZM4JZPax.


Submit an infographic to Wikidata Visualisation Challenge if Time

https://se.wikimedia.org/wiki/Projekt:Öppna_data_2015/Visualisering/Tävling
What
Data from, or about, Wikidata must be included in the visualization.
Visualizations that combine this with other Open Data sources are encouraged by the jury.
It is allowed to reuse resources produced before the competition starts, i.e. you do not have to start from scratch.

May 4, 09.00 - June 1, 23:59 (CEST)

oh dear

oh dear indeed!
On Sun, May 17, 2015 at 12:15 AM, fsoong<EMAIL_ADDRESS>wrote:

oh dear
—
Reply to this email directly or view it on GitHub
https://github.com/notconfusing/WIGI/issues/19#issuecomment-102665120.

This should be fun! But looking at the time...

@Komchi I assigned this to myself, but if you want to take a shot at designing and then @fsoong can give you input I think that's a good use of time.
@Masssly ''It is allowed to reuse resources produced before the competition starts, i.e. you do not have to start from scratch."
So maybe a good way to do this is to take one or more of the graphics that has already been made in the original paper, and then use the same data, but redesign the style and arrange a few together so that it tells a more compelling story. I think the best ones would be %-female by date of birth, and the world map. But I give you creative leeway.

Sounds like a good plan. I'm planning to share with Komchi some online
tutorials for information design basics and designing landing pages in
photoshop. Infographic tutorials tend to be more print-based, which would
be less translatable to our main data viz / dashboard project :)
On Mon, May 18, 2015 at 3:17 PM, Max Klein<EMAIL_ADDRESS>wrote:

@Komchi https://github.com/Komchi I assigned this to myself, but if you
want to take a shot at designing and then @fsoong
https://github.com/fsoong can give you input I think that's a good use
of time.
@Masssly https://github.com/Masssly ''It is allowed to reuse resources
produced before the competition starts, i.e. you do not have to start from
scratch."
So maybe a good way to do this is to take one or more of the graphics that
has already been made in the original paper, and then use the same data,
but redesign the style and arrange a few together so that it tells a more
compelling story. I think the best ones would be %-female by date of birth,
and the world map. But I give you creative leeway.
—
Reply to this email directly or view it on GitHub
https://github.com/notconfusing/WIGI/issues/19#issuecomment-103231145.

--
Frances Soong
Visual UX Designer
(908) 635-1534
francessoong.com http://www.francessoong.com/
LinkedIn https://www.linkedin.com/in/francessoong | Twitter
https://www.linkedin.com/in/francessoong

Yeah I would love to give it a try.
On Tue, May 19, 2015 at 3:47 AM, Max Klein<EMAIL_ADDRESS>wrote:

@Komchi https://github.com/Komchi I assigned this to myself, but if you
want to take a shot at designing and then @fsoong
https://github.com/fsoong can give you input I think that's a good use
of time.
@Masssly https://github.com/Masssly ''It is allowed to reuse resources
produced before the competition starts, i.e. you do not have to start from
scratch."
So maybe a good way to do this is to take one or more of the graphics that
has already been made in the original paper, and then use the same data,
but redesign the style and arrange a few together so that it tells a more
compelling story. I think the best ones would be %-female by date of birth,
and the world map. But I give you creative leeway.
—
Reply to this email directly or view it on GitHub
https://github.com/notconfusing/WIGI/issues/19#issuecomment-103231145.

@notconfusing I am looking at how we can arouse and keep the interest of the jury for a while. The "world map" and "female percentage of biographies by culture" seem more related than the "world map" and "%-female by date of birth".
Do you have a reason why you chose the latter?
:thought_balloon: Also, I haven't graphed but I can imagine how kindling " female percentage of biographies" by "culture" by "date (era) of birth" will be.

I think they're all good candidates if they got a face lift and were weaved
into an info graphic.
On 19 May 2015 08:58, "Mohammed Sadat Abdulai"<EMAIL_ADDRESS>wrote:

@notconfusing https://github.com/notconfusing I am looking at how we
can arouse and keep the interest of the jury for a while. The "world map"
and "female percentage of biographies by culture" seem more related than
the "world map" and "%-female by date of birth".
Do you have a reason why you chose the latter?
[image: :thought_balloon:] Also, I haven't graphed but I can imagine how
kindling " female percentage of biographies" by "culture" by "date (era) of
birth" will be.
—
Reply to this email directly or view it on GitHub
https://github.com/notconfusing/WIGI/issues/19#issuecomment-103563067.

Hi All, I spoke with with Komchi over video chat this past Wednesday and suggested he work with Max with daily syncs or 3-5 times a week check-ins. Like over email as he had done with me and others for the initial project. I shared with him quite a few resources, but am editing my list (I'm wary of there being too many resources out there and want to make sure I pick out good ones for good reasons.) I will post these later today. For now everyone should review what an infographic is on Wikipedia at the very least: https://en.wikipedia.org/wiki/Infographic. Note it's part of a series for "Data Visualization" that you can link to using the righthand navigation. The WIGI project is an "interactive data visualization."
I told Komchi, he and Max should plan two review sessions for Masssly and I to attend and give feedback. The first in-progress review sometime this Sunday and a final review next Friday. That way you have plenty of time to take in our feedback and preparing the final submissions before June 1st.

Okay, to start with what is an infographic and how to make a good one is to know there are many, many bad ones. I start with blog posts by Column Five, one of most well-known agencies creating the prettiest infographics you see going viral (including almost all the ones used on http://good.is).

"What Makes a Good Infographic?" http://www.columnfivemedia.com/makes-good-infographic
"Crafting an Infographic Narrative" http://www.columnfivemedia.com/crafting-an-infographic-narrative
"The Most Important Piece of an Infographic" http://www.columnfivemedia.com/important-piece-infographic
A pretty video summing why infographics are valuable: https://vimeo.com/29684853
Any of the other blog posts are recommended too. I do not recommend following most of the guides you find by immediately Googling. Since Column Five is an agency selling infographics, they don't provided as much information on how to make them, but many people who create infographics are even worse—selling you ads.

Next, Information Design 101— the foundation of infographics and data visualization

From Mapping Complex Information: https://sheilapontis.wordpress.com/2010/09/23/information-design-basics-and-essentials/

But you can read and learn all the basics without being able to produce anything of that level. To achieve that requires practice. Here are some using Photoshop, but I can dig up some tutorials for Sketch if need be.

The Only Prerequisite is Good Taste https://www.youtube.com/watch?v=E1oZhEIrer4
Design a Gratitude Log Landing page using Photoshop http://webdesign.tutsplus.com/tutorials/design-an-elegant-gratitude-log-landing-page-with-photoshop--cms-22787 (Easy exercise anyone can learn, very general not trend-specific)
6 Beginner “Safety First” Color Guidelines for the Web http://webdesign.tutsplus.com/tutorials/6-beginner-safety-first-color-guidelines-for-the-web--cms-21462 (Very fast guide geared for people without a natural knack or interest in color)

And other Photoshop tips form me:

Design similar pages using layers in Photoshop (have the background be one layer, stack unique elements from each page on top)
NEVER transform text (squish, skew, distort, or scale). Its disrespectful to the type designers intent and designs and you will confuse yourself when you try to copy that text later.

This is more for Komchi. Skillshare classes are of varying quality and are not all taught by experts, but are all fairly in-depth. Also some classes you learn how to critique and have your work critiqued.

Try the excercises from this Skillshare class, Visual Narratives 101 http://www.skillshare.com/classes/design/The-Art-of-the-Story-Creating-Visual-Narratives/1893972902
Apply for this scholarship too! http://www.skillshare.com/scholarships

More Advanced:

A Guide to Creating Dashboards People Love to Use http://static1.squarespace.com/static/52f42657e4b0b3416ff6b831/t/5310292ce4b08d35a87c9426/1393568044420/Guide_to_Dashboard_Design.pdf
(Hits a lot of stuff I’m teaching myself now: info from Tufte, Few, common functions, etc. and skims at skills I refined in college: fundamentals of color, type, grid, form, etc.)
Google’s Web Fundamentals https://developers.google.com/web/fundamentals/?hl=en
(Are geared more towards responsive design sensibilities, a lot of cool non-profit and data-viz tools still follow desktop web-only patterns as I suspect we will too.)

https://commons.wikimedia.org/wiki/File:WIGI_Infographic.png

To others: please do not share on social media or publicly just yet! There are a few touches I'd like to work on still and add with Max. If you email to friends or family make sure they know it's a submitted draft :)

@fsoong Can you do something about the fonts? See https://groups.google.com/forum/#!topic/wigi-project/HXiXSfCtaHE

Convert the picture to .svg so that it doesn't lose quality and its size can be adjusted. :)

I don't think I need to make the fonts larger or convert the picture to .svg. SVG is still not standardized format and renders different across different browsers. I've had infographics become unreadable in SVG If anything, I want to explore making the text darker in certain areas.
It'd help me though if you guys could use MS paint to draw on the graphic and point at which areas seem too small or hard to read. Please post your comments in this new issue below. This issue is closed.
Polish Infographic for Social Media #22
https://github.com/notconfusing/WIGI/issues/22


Some company backends should be grouped

Associated PR: https://github.com/hlissner/doom-emacs/pull/4476
Some company backends can completely shadow other backends following them. For example, having company-backends as (company-dabbrev company-yasnippet company-ispell) will make company-yasnippet and company-ispell completely useless, because company-dabbrev will always answer: "Yes! I will try to find completion candidates" (even if later on it comes up with an empty list of candidates). This makes the backends following it unreachable.
To solve this issue, these backends should be grouped, which results in merging the completion results.
To identify the backends having this behavior, we can check their docs or by trying them.
company-yasnippet has this behavior:
https://github.com/company-mode/company-mode/blob/0edc87d423b83c707e6c708f421d96f4d198803d/company-yasnippet.el#L134
(it gives recommendations on grouping)
For more info about grouping backends, see:
https://github.com/company-mode/company-mode/blob/0edc87d423b83c707e6c708f421d96f4d198803d/company.el#L452
As an example, the current doom configuration results in the following for org-mode:
(company-capf company-dabbrev company-yasnippet company-ispell)
which results in company-yasnippet and company-ispell being meaningless and unreachable.
The associated PR solves the issue for text-mode and its derived org-mode. It results in having company-backends set to (company-capf (:separate company-dabbrev company-yasnippet company-ispell)) (doom sync might be needed for the changes to take effect)
(prog-mode and other lang modes might need also a review)

For prog-mode this is an issue as well w.r.t ordering of completions.
For example in C++ with current settings, typing inc mixes LSP completions and snippet completions:

If instead you (setq company-backends '((:separate company-yasnippet company-capf))), we get snippets as first priority and then capf (lsp) completions second. Having the ordering is essentially required if you have have company tng mode enabled.

For prog-mode this is an issue as well w.r.t ordering of completions.
For example in C++ with current settings, typing inc mixes LSP completions and snippet completions:

If instead you (setq company-backends '((:separate company-yasnippet company-capf))), we get snippets as first priority and then capf (lsp) completions second. Having the ordering is essentially required if you have have company tng mode enabled.

I have the following for prog modes:
(set-company-backend! 'prog-mode
  '(:separate company-capf company-yasnippet company-dabbrev company-ispell))

However,  I noticed that lsp-mode overrides my config and prepends company-capf to company-backends, which results in shadowing the other backends. To avoid this issue we can remove the lsp added entry using lsp-after-open-hook:
(add-hook! lsp-after-open
  (setq company-backends (remove 'company-capf company-backends)))

I have the following for prog modes:
(set-company-backend! 'prog-mode
  '(:separate company-capf company-yasnippet company-dabbrev company-ispell))

However,  I noticed that lsp-mode overrides my config and prepends company-capf to company-backends, which results in shadowing the other backends. To avoid this issue we can remove the lsp added entry using lsp-after-open-hook:
(add-hook! lsp-after-open
  (setq company-backends (remove 'company-capf company-backends)))

doom also change the backend for dap-ui-repl - by setting it to company-capf.
workflow for dap-debug for python file.

set breakpoint (dap-breakpoint-add)
run dap-debug
run dap-ui-repl
list company backend (company-diag)

company-backends: (company-capf)

Used backend: company-capf
Value of c-a-p-f: (comint-completion-at-point t)
Major mode: dap-ui-repl-mode
Prefix: nil
Completions: none

This breaks the completion in the dap-ui-repl.
dap-mode sets the backend to (setq-local company-backends '(dap-ui-repl-company)).
We need to keep this;
How do I prevent doom from overwriting company-backends for this repl?
Some additional info:
M-x debug-on-variable-change for company-backends and running dap-ui-repl result in the the following two backtraces:

https://pastebin.ubuntu.com/p/g87kqdTjr9/
https://pastebin.ubuntu.com/p/r4xGhf37ry/

Disable the debug with M-x cancel-debug-watch
debug--implement-debug-watch(company-backends (company-capf) set #<buffer *dap-ui-repl*>)
  set(company-backends (company-capf))
  (or (memq major-mode '(fundamental-mode special-mode)) buffer-read-only (doom-temp-buffer-p (or (buffer-base-buffer) (current-buffer))) (set (make-local-variable 'company-backends) (+company--backends)))
  +company-init-backends-h()

See lsp-mode gitter for initial help by @yyoncho.


Add home directory to project clone container

What does this PR do?
Add a home directory (/home/user) to the project clone container to support tools that expect to have a home directory available.
What issues does this PR fix or reference?
E.g. while testing private repo access, ssh-agent expects keys to be mounted to /.ssh.
Is it tested? How?

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path to trigger)

[ ] v7-devworkspaces-operator-e2e: DevWorkspace e2e test
[ ] v7-devworkspace-happy-path: DevWorkspace e2e test

/test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path


Manage Django, Boto, Python-cloudfiles and Django-storages as external libraries

See #6 and #86.
This pull request is based on #81 codebase, so #81 may be merged before this one:

the application is packaged as an egg (#81)
Django, Boto and Python-cloudfiles are external dependencies (see #86 and dependencies-external branch)
Django-storages is an external dependency (see #6 and dependencies-external-storages branch

I froze version numbers in a restrictive way, to match the version that was included in fileconveyor. Maybe external librairies can be upgraded. But it should be another task.
I couldn't test every storage: I did my best to avoid import errors, but I cannot guarantee that transporters and storages actually work. So feedback is welcome.

Tested with:
 pip install -e git+file:///Users/wimleers/Work/fileconveyor#egg=fileconveyor

… I was impressed that this actually works. I based this command off of your explanation in #81, and it really just works! Awesome!
I'm super excited to commit this in a few seconds… Thank you so much for your awesome contributions! :)


"New > Java Class" wizard cannot create a class for Japanese identifiers.

Originally reported from https://github.com/redhat-developer/vscode-java/issues/3289
When I open the "New > Java Class" wizard and enter a Japanese class name, I receive the message "Invalid Java qualified name." and I am unable to create a Java class.
Environment

Operating System: Windows 11
JDK version: 17
Visual Studio Code version: 1.79.2
Java extension version: v0.25.14

Steps To Reproduce

Select any package, right-click, and choose "New > Java Class."
Enter Japanese characters as the class name. For example, "ほげ"

Current Result
It displays "Invalid Java qualified name."
Expected Result
The class is generated.
Additional Informations
In Java, we can use characters from languages such as Japanese and others in identifiers.
https://docs.oracle.com/javase/specs/jls/se17/html/jls-3.html#jls-3.8

Root cause is here: https://github.com/microsoft/vscode-java-dependency/blob/2ba3227264d9c0967d409bebac9bbf9fb07e9d67/src/utility.ts#L92


Investigate file uploading to multiple hosts

Sometimes we upload a blank/empty file. Something to do with IO objects.
EDIT: because uploading from one IO object in parallel.

https://github.com/Fizzadar/pyinfra/commit/ec119439044c4bb4f8eac7f94f4c3659aff2b752  & https://github.com/Fizzadar/pyinfra/commit/359ef5a6fc39520d892af0966086506a5c900f14 got half way to fixing this, but still need to handle template StringIO objects which may still be read in parallel - test if that works different to a normal file object (appears to!).

Fixed @ https://github.com/Fizzadar/pyinfra/commit/c3d4b895537ecf6cf14fa570b9643955610953ea. Files loaded per-host, StringIO objects are also per-host.


ModuleNotFoundError: No module named 'kaleido.scopes'; 'kaleido' is not a package

I've tried installing with both pip and conda and keep running into this same error
$ which python3
~/opt/anaconda3/bin/python3

$ cat ~/opt/anaconda3/lib/python3.8/site-packages/kaleido/_version.py
__version__ = "0.2.1"

$ python3
Python 3.8.8 (default, Apr 13 2021, 12:59:45)
[Clang 10.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from kaleido.scopes.plotly import PlotlyScope
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/cpsievert/github/plotly/kaleido.py", line 1, in <module>
    from kaleido.scopes.plotly import PlotlyScope
ModuleNotFoundError: No module named 'kaleido.scopes'; 'kaleido' is not a package

$ ▶ conda list
# packages in environment at /Users/cpsievert/opt/anaconda3:
#
# Name                    Version                   Build  Channel
...
kaleido                   0.2.1                    pypi_0    pypi
...

File "/Users/cpsievert/github/plotly/kaleido.py"
Looks like it's trying to import that file as the Kaleido package - can you rename it to something that's not a package name?

Whoops, nice catch, thanks!


Renamed SASL to sasl according to protocol definition for SASL plain …

Renamed SASL to sasl according to protocol definition for SASL authentication. Now using Buffer object to create the UTF-8 byte encoding

Thanks for sending this. I guess we can drop the utf8 dependency altogether then?

Thanks, tests are failing since utf8 library is still imported in GremlinClient. Could you please amend your last commit to reflect this change?
I think there's a way for me to commit on a fork branch, unsure how. I need to dig in Github doc.
Once this passes I'll merge and publish a new version on npm.

Sure, I have no problem taking care of that !
Voilà !
Patrick


feat(client_config): add Plugin Directory to bacula-fd config

Adds the ability to specify a Plugin Directory for the bacula-fd.

#151 broke this, so I merged it manually into master.
@xaque208 I assumed it was okay for you and wanted to avoid @buchstabensalat the pain of rebasing himself :wink:.

Yep, all good.  Thanks @smortex.


Change until behaviour - addresses #69

https://github.com/digital-asset/contingent-claims/issues/69

I am closing this as it's outdated by now. Will open a Draft PR based on the latest version of master.


[ISSUE-222][BUG] GetBlacklist/GetBlacklistResponse should replace null value with empty list

What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

ping @waitinfuture


Provide more detail on cross-platform code coverage in .NET Core

Coverlet is typically used with a solution like ReportGenerator to generate a human-readable code coverage report. This detail is missing. We should also provide a basic YAML snippet in this section to better guide customers using Linux or macOS.

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

ID: 483dcb1c-1a89-2b20-4cf2-abd3ae88d9ed
Version Independent ID: dc4722ed-5a4b-26ec-e7d5-a149bf6e79a7
Content: Building .NET Core projects with Azure Pipelines and TFS - Azure Pipelines & TFS
Content Source: docs/pipelines/languages/dotnet-core.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

@PBoraMSFT If it helps get this work done, I have a YAML file demonstrating how I accomplished this: https://github.com/scottaddie/ViewComponentsDemo/blob/master/azure-pipelines.build.yml

@PBoraMSFT can you look into this?

@PBoraMSFT  - it looks like we have the YAML snippets, but did we want to add the reference to known solutions for generating human readable reports with Coverlet?


Fix #1691

This PR changes the way of looking up the key from the json object based on #1694.
closes #1691

Codecov Report

Merging #2761 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2761      +/-   ##
==========================================
+ Coverage    82.9%   82.91%   +<.01%
==========================================
  Files          60       60
  Lines        4791     4793       +2
==========================================
+ Hits         3972     3974       +2
  Misses        819      819

Impacted Files
Coverage Δ

src/data.convert.js
88.73% <100%> (+0.16%)
: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 ed0e523...e1d8309. Read the comment docs.


Improve current architecture

Current architecture needs to be refactored and improved.

Split logic Image processing between resize and face detection.
Separating the Image Entity class into 2 classes: InputImage, OutputImage
Improve folders and classes naming.

Closed by PR #89


Forms definition

Providing a forms array in the options it would make sense to only show the provided features.
Additionally Custom elements should be shown as well.

If a forms dictionary is provided by settings use this one and do not use the standard one.

closed by #83


PHP 7.4 support

Using typed properties:
<?php
class User {
    public int $id;
    public string $name;
}
?>

Results in:
Try to avoid public class variables. Use Getter to access the variable. It supports the principle of privacy.

Yes, is there a change?

If there is a new concept in PHP for public class members, please let me know and argument with sources. Thanks.


Added FoalTS framework to fullstack frameworks

Thank you for contributing to the Node.js Toolbox catalog! 🙏
Note: Node.js Toolbox is specifically for backend libraries. If a library works both in the browser AND in Node.js feel free to add it. Libraries that work ONLY in the browser are out of scope for now.
Before submitting your pull request, please make sure:

[x] To use the exact package name as it appears on NPM (and npm install <package-name>)
[x] The packages are listed in alphabetical order
[x] The package doesn't already exist in a different category
[x] Description doesn't have a dot (.) at the end
[x] To format with Prettier before committing (npm run prettier)

@maikofelix47 Thanks again for your contribution! ❤
Change is live: https://nodejstoolbox.com/categories/full-stack-frameworks

Thank you @Maximization . I have become a regular user of the tool so i will try and improve it any way i can.

@maikofelix47 Fantastic! Let me know if there's any features/statistics missing that could be added to make it easier to find & compare libraries. I have some ideas in mind but curious to hear your suggestions.

@Maximization What i feel would make things better are

Search functionality
Filter by category
Weekly download statistics and graph showing the trend
Order libraries by stars or downloads.

@Maximization Thanks. For 1. Sometimes you may know a library but are not sure which category it belongs to. So instead of having to go through all the categories,it would be easier to search for the library name and the displayed results can indicate the library as well as the category it belongs to. On clicking the search result it can take them to the category page just like they clicked on the category since we don't have a library details page.
For item 2, with time the list of categories may grow longer and one may only be interested in 1 or 2 categories. Filter allows one to filter out all the other categories.

Awesome! The first point is very clear and I've added it to the roadmap. For a growing list of categories I had the idea of grouping categories into sections such as Code Quality, Data Persistence, Developer Tools, Testing, etc. That'll make it easier to browse.
I don't fully understand what you mean by filtering out categories, how does the page look like and what's the user interaction? Thanks for taking the time to explain this btw, appreciate it!

@Maximization Thanks for responding. The categories page is okay as it is. My suggestion was to cater to future cases where the list grew longer. If you are planning to group them as you've mentioned then it should sort it out.


corretto-<IP_ADDRESS>.1 support

Fixes Proemion/ts2-psp-merge#41442

@Proemion/devops Can you, please, merge it? Or maybe give us write access to this repo?


getting permission denied in ModSecurity error_log

I have installed modsecurity OWASP rules in my CENTOS 6.6 WHM 11.50.0 (build 29) and getting following errors in /usr/local/apache/logs/error_log. These are separate three errors I am getting.
ModSecurity: collection_store: Failed to access DBM file "/var/cpanel/secdatadir/ip": Permission denied
ModSecurity: Rule processing failed.
ModSecurity: Geo Lookup: Failed to lock proc mutex: Permission denied

Still no solutions to this yet?

what are the permissions on that file? this seems like an OS configuration issue not a ModSecurity issue.

(root)-(jobs:0)-(/var/cpanel/secdatadir)-> ls -lh
total 1.5M
drwxrwx--T  3 root   nobody 4.0K Aug 24 07:18 ./
drwx--x--x 92 root   root    12K Oct  9 21:37 ../
-rw-r-----  1 nobody nobody    0 Feb 11  2015 global.dir
-rw-r-----  1 nobody nobody    0 Feb 11  2015 global.pag
-rw-r-----  1 nobody nobody 4.0K Oct  7 10:19 ip.dir
-rw-r-----  1 nobody nobody  15M Oct  9 21:38 ip.pag
-rw-r-----  1 nobody nobody    0 Aug 24 07:18 user.dir
-rw-r-----  1 nobody nobody    0 Aug 24 07:18 user.pag

Can you try chowning those to the same person who owns your webserver (likely apache)? also lets try changing the permissions to 777. Check to see if that helps!

nope it is still giving same error. Any other way? FYI it's a cpanel.

Also having this same issue with a Wordpress Brute Force rule I've wrote. Any solution would be great.

@ConnorLinfoot  adding one rule should not cause this issue.... @pkhadka56  i'm unsure of what effects cpanel could have on it, coule you should me the ls -l of both the apache executable and the secdatadir?

@csanders-git I'm using cPanel also, if that helps at all.

@pkhadka56 if you are using cPanel with apache chroot jailed vhosts and mod_ruid2 then you will need to change /var/cpanel/secdatadir in the modsec2.cpanel.conf file to /var/log/secdatadir, then make that directory and chmod 1733 it, and copy the *.dir and .pag files into it with ugo+rw set.
This is because the apache 'nobody' process needs to read those files, and the users need to write to those files, which chroot environments cannot do to the /var/cpanel/secdatadir directory, but can for /var/log/
Hope that helps.

Thanks for the answer @udha but no luck.
inside /var/log
drwx-wx-wt   2 root     nobody 4.0K Oct 14 00:56 secdatadir/
inside secdatadir
drwx-wx-wt  2 root   nobody 4.0K Oct 14 00:56 ./
drwxr-xr-x. 8 root   root   4.0K Oct 14 00:53 ../
-rw-rw-rw-  1 nobody nobody    0 Jul 30 07:36 global.dir
-rw-rw-rw-  1 nobody nobody    0 Jul 30 07:36 global.pag
-rw-rw-rw-  1 nobody nobody 6.4M Sep 12 16:26 ip.dir

@udha new cpanel will force it to default again. is there a configuration to keep SecDataDir at "/var/log/secdatadir" ?

@hrace009 do following and see if it helps:
$ sudo vi /etc/apache2/conf.d/modsec_rule_01.conf
paste following
SecDataDir "/var/log/secdatadir"

save and exit
# sudo cp -R /var/cpanel/secdatadir /var/log/
# sudo chmod 1733 /var/log/secdatadir
# sudo chown -R nobody:nobody /var/log/secdatadir
# sudo chmod ugo+rw ip.*

Restart httpd and see if it helps.

@shunkica do you have some feedback about the security risk now that some time has passed?


Publish 7.5-20220603004608+0000

Publish 7.5-20220603004608+0000

@bot-gradle test and merge

OK, I've already triggered a build for you.


Fix setting the changed weapon locally flag on server

To my understanding, this flag has to be set on the client instead of the server.

That seems logical based on its comment and where it's used. Thank you!


I use the graph-index with garphite-api , and found error as below.

the graph data has datapoints less than 1d , and graph-index default draw graph -1d , -7d , -1mon ,-1y
The four type graphs don't display , and got the error.
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python2.7/dist-packages/graphite_api/app.py", line 455, in render
    image = doImageRender(request_options['graphClass'], graph_options)
  File "/usr/local/lib/python2.7/dist-packages/graphite_api/app.py", line 591, in doImageRender
    img = graphClass(**graphOptions)
  File "/usr/local/lib/python2.7/dist-packages/graphite_api/render/glyph.py", line 383, in __init__
    self.drawGraph(**params)
  File "/usr/local/lib/python2.7/dist-packages/graphite_api/render/glyph.py", line 895, in drawGraph
    self.setupYAxis()
  File "/usr/local/lib/python2.7/dist-packages/graphite_api/render/glyph.py", line 1426, in setupYAxis
    self.getExtents(label)['width'] for label in self.yLabels])
ValueError: max() arg is an empty sequence

Can you provide the full graph URL?

http://<IP_ADDRESS>:1999/render/?target=scale(servers.some_switch_mydomain_org.if_errors-XGigabitEthernet0_0_15.rx,8)&target=scale(servers.some_switch_mydomain_org.if_errors-XGigabitEthernet0_0_15.tx,8)&hideLegend=False&from=-1d&width=600&title=some_switch_mydomain_org if_errors-XGigabitEthernet0_0_15 - day&height=400

IF I add yMin & YMax params , then it works


investigate caching

Caching Issue

Post caching of Rust build was tasking a significant amount
Gradle and cargo_home are caching empty folders causing Gradle and deps to download for every action

Investigation

Github post job caching entails

Compressing all folders/files specified in cache path
Uploading compressed cache
Downloading target cache, compressed size indicates ~1.9GB which means post cache had to compressed chunk of files (comprising KBs of files) down to 1.9GB leading to delay in time of target post caching. We can further delete folders in the target folder.

Deleting the /target/debug/examples folder which consists of Ockam built examples more here https://doc.rust-lang.org/cargo/reference/cargo-targets.html#examples
Deleting the /target/debug/incremental folder more here
Deleting these folders further reduces target folder from 8.4G to 2.5G and takes approximately 2m 13s to compress and upload cache compared to recent 13m to cache.

For Gradle, we were caching from the $HOME (~) directory which is specified as /github/home, an invalid path even on docker. Hardcoding the path fixes this issues and gets caching working both on Gradle and cargo directory.

Note: I appended a cache- string to the keys so that caching of new paths dont clash with the old ones. We can always revert after.

Rebased and landed in #2473


Mocha Supertest Express: app.address is not a function even if I exported correctly app.

Good morning.
I'm still considering quite a newbe in Node.Js since very often I get into problems that with a bit of searching I figure out how to solve. Except this.
This is my ./src/app.js file:
const express = require('express');
const bodyParser = require('body-parser');
const ENV = require('./env');
const request = require('supertest');
const app = express();

app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: false }));

app.get('/', (req, res) => {
    res.json({
        service: 'user-service',
    });
});

// This works correctly
const req = request(app);
req
    .get('/')
    .expect('Content-Type', /json/)
    .expect(JSON.stringify({
        service: 'user-service'
    }))
    .expect(200)
    .end(function (err, res) {
        if (err) throw err;
    });

// I tired in both ways
module.exports = Object.assign({}, app);
// module.exports = app;

This is my ./index.js file:
const app = require('./src/app');
const ENV = require('./src/env');

app.listen(3000, () => {
    console.log(`Server started on port ${ENV.port}`);
});

module.exports = app;

This is my ./test/test.js file:
const request = require('supertest');
const app = request('../src/app');
/*
  using request('http://localhost:3000') works correctly
  but i know it's not the best way to setup a testing environment
*/
//const req = request('http://localhost:3000');
const req = request(app);

describe('Integration test exaple', function() {
    it('get /', function(done) {
        req
        .get('/')
        .expect('Content-Type', /json/)
        .expect(JSON.stringify({ service: 'user-service' }))
        .expect(200, done);
    });
});

I tried to require app both from app.js and index.js but the result it's the same:
>> mocha ./test/test

  Integration test exaple
    1) get /

  0 passing (9ms)
  1 failing

  1) Integration test exaple
       get /:
     TypeError: app.address is not a function
      at Test.serverAddress (node_modules\supertest\lib\test.js:55:18)
      at new Test (node_modules\supertest\lib\test.js:36:12)
      at Object.obj.(anonymous function) [as get] (node_modules\supertest\index.js:25:14)
      at Context.<anonymous> (test\integration.test.js:12:10)

Thanks in advance for any kind of help!

@Danver97 Exporting ´Object.assign({}, app)´ would never work because it will convert your app (which is a function) into an object. Are you sure it isn't working by just exporting the app instance?
Besides that, you can alternatively export the result of app.listen from your index file to export the Http server instance, which should work as expected on your integration test
Hope it helps.

@jonathansamines
It's pretty curious: I checked again and using module.exports = app doesn't work, BUT in another project it's working fine...
Also using the result of app.listen doesn't work.
I can't understand why...

@Danver97 Could you provide an isolated running test case for this? I could help figuring out what is wrong.

@jonathansamines
Sorry, what do you mean?
This is my code right now:
./src/app.js
const express = require('express');
const jwt = require('jsonwebtoken');

const ENV = require('./env');
const usrManager = require('../modules/userManager');
const User = require('../models/user');

const app = express();

app.get('/', (req, res) => {
    res.json({
        service: 'user-service',
    });
});

module.exports = app;

./test/integration.test.js
const request = require('supertest');
const jwt = require('jsonwebtoken');
const ENV = require('../src/env');
const app = request('../src/app');
const req = request(app);

describe('Integration test exaple', function() {
    const user = {
        username: 'pippo',
        email<EMAIL_ADDRESS>        password: 'pippopassword',
        name: 'pippo',
    }

    it('get /', function(done) {
        req
        .get('/')
        .set('Accept', 'application/json')
        .expect('Content-Type', /json/)
        .expect(JSON.stringify({ service: 'user-service' }))
        .expect(200, done);
    });
});

@jonathansamines
Here you are: https://github.com/Danver97/user-service

@Danver97 Not sure how I didn't notice that before but the problem is that you are using the function request to try to require your app in your tests. You should use require instead.
See: https://github.com/Danver97/user-service/blob/master/test/integration.test.js#L4

@jonathansamines Oh god. I can't believe it. How could I do something like that?
Thank you so much for your time. Really appreciated!

@Danver97 anyway thanks for asking this, did the same mistake--


Recover the textual information for SCCS v1716-v1780

@bibiko @xrotwang @SimonGreenhill  - are you able to extract data from a dBase database? See below.
"A disk included with this volume holds zipped files for the CONAN data base and a copy of a program needed to unzip the files. The CONAN date base requires dBASE IV (or later) to run and the files must be unzipped to a hard drive.  The numeric codes in ST81.DAT and ST82.DAT duplicate to codes in CONAN, but you cannot recover the textual information in coding decisions for specific cases without running the dBASE program."
"The CONAN materials on the disk should all be copied to directory on
your hard disk. If you have a directory dedicated to dBASE files you might
want to copy it to that directory.  Unzip the compressed files by issuing
the commands: "pkunzip conan" and "pkunzip concodes".  To use the CONAN
materials start dBASE and open the CONAN database."
Here's a link to the zipped 'conan' and 'concodes' files:
https://www.dropbox.com/s/654xp89abk8tpm0/CONAN.ZIP?dl=0
https://www.dropbox.com/s/qy624qlmwjqgvr6/CONCODES.ZIP?dl=0
Text file describing the files (conan includes 67 codes for a 101 society subsample of the SCCS):
https://www.dropbox.com/s/7o7v4scv7h7fjj6/Conan.txt?dl=0
And the MAPTAB files (from which I've already extracted the variables and codes - they do not contain textual info):
https://www.dropbox.com/s/kxtgm6wbechcbns/STDS81_82_CONAN_MAPTAB.zip?dl=0

DBF files can be read by using LibreOffice/OpenOffice quite easily if one knows the text encoding (but this shouldn't be the problem here)

Yes, extracting the data is not too much of a problem. What exactly do you want to do with it, though? Is this going to be a separate dataset, or should it replace/supersede/correct the SCCS data for the corresponding societies and variables?

@kirbykat This is what csvstat tells me for the DBF file converted to csv:
$ csvstat conan.csv
  1. VAR_NO
	<type 'unicode'>
	Nulls: False
	Unique values: 71
	5 most frequent values:
		56:	101
		54:	101
		42:	101
		48:	101
		43:	101
	Max length: 13
  2. VAR_VALUE
	<type 'unicode'>
	Nulls: True
	Unique values: 165
	5 most frequent values:
		99:	1218
		1:	972
		3:	801
		2:	725
		88:	583
	Max length: 7
  3. MODFD_VAL
	<type 'int'>
	Nulls: False
	Min: 0
	Max: 99999
	Sum: 777253
	Mean:<PHONE_NUMBER>49
	Median: 10
	Standard Deviation: 1282.68835766
	Unique values: 164
	5 most frequent values:
		99:	1218
		1:	972
		3:	801
		2:	725
		88:	583
  4. SOURCE_S
	<type 'unicode'>
	Nulls: True
	Unique values: 952
	5 most frequent values:
		A:Osgood 1951:	70
		A:Holmberg 1969:	70
		A: Elmendorf, W. 1960:	69
		A: Maybury-Lewis, D. 1967:	68
		A:Goldman 1963; B:Goldman 1948:	68
	Max length: 60
  5. TEXT_COM
	<type 'unicode'>
	Nulls: True
	Unique values: 5366
	5 most frequent values:
		[Variable 30=10 (No political office)]
 Orokaiv:	8
		[var 30 = 10; there is no political authority beyond the family-
unit]
chter un:	7
		A 47: Apart from this generalized and mainly symbolic
identification of members with the camp community, the group is
rarely mobilized as a corporate unit in conflict situations with
other groups. The prevailing fear and suspicion keeps groups at a
reasonable distance from each other, and violent conflicts occur
only rarely. Fights with sticks and stones may occasionally
develop over pastures between shepherds from different camps, and
when these take place sufficiently close to camp the whole group
may join in the fray. I never ovserved such a case in the field;
the Darbar camp had been involved in one the previous year.
d as a c:	5
		D 291: But now, at any rate, they are as peaceful as it is
possible to be.
re said:	4
		D 291: But now, at any rate, they are as peaceful as it is
possible to be.
hed his:	4
	Max length: 5045
  6. CASE_NAME
	<type 'unicode'>
	Nulls: True
	Unique values: 104
	5 most frequent values:
		Thonga:	70
		Yurok:	70
		Kung Bushmen:	70
		Yanomamo:	70
		Koreans:	70
	Max length: 15
  7. SCCS_NO
	<type 'int'>
	Nulls: True
	Min: 2
	Max: 186
	Sum: 654968
	Mean: 92.6535577875
	Median: 92
	Standard Deviation: 53.3568721142
	Unique values: 101
	5 most frequent values:
		133:	70
		132:	70
		130:	70
		136:	70
		135:	70
  8. CODER_NAME
	<type 'unicode'>
	Nulls: True
	Unique values: 10
	5 most frequent values:
		ba:	1260
		UR:	1259
		ra:	1229
		sa:	1190
		kl:	1050
	Max length: 4

Row count: 7071

@xrotwang It won't replace existing variables - codes (they should be the same), but I'd like to extract the Comments for each society-variable-code combination.
These look like comments:
A47: Apart from this generalized and mainly symbolic
identification of members with the camp community, the group is
rarely mobilized as a corporate unit in conflict situations with
other groups. The prevailing fear and suspicion keeps groups at a
reasonable distance from each other, and violent conflicts occur
only rarely. Fights with sticks and stones may occasionally
develop over pastures between shepherds from different camps, and
when these take place sufficiently close to camp the whole group
may join in the fray. I never ovserved such a case in the field;
the Darbar camp had been involved in one the previous year.
D 291: But now, at any rate, they are as peaceful as it is �
possible to be.

Should they be added as comment in SCCS/data.csv? This could presumably be done programmatically.

Yes, that would be wonderful.

It would also be ideal to figure out the relationship between SOURCE_S and SOURCE_NO, and to include this info in "EthnoReferences" (presumably these are the first authors and page numbers from which cultural data are taken)

Ok. That seems doable - it should be possible to

extract sources (and match them to sources we already have)
extract page numbers from the comment fields (and replace the single-letter source keys in the comments with bibtex keys)
add these comments to data.csv

Do you think comments are the best place for page numbers? I know we are trying to minimize changes to the schema, but perhaps we should pair each "source" and "ethnoReferences" field with a "page number" field. I think this would save having to extract this information from a comments field in the future. (And here, the comments field will sometimes contain quite a long comment from the TEXT_COM field.)

I think page numbers should be part of a reference, i.e. belong into the Source field. We are basically re-engineering the CLDF spec here, see https://github.com/glottobank/cldf/blob/master/core.md#data-files
In any case we can leverage some of the machinery I built for it, e.g. a csv reader/writer which can deal with secondary separators within cells of csv files. This makes markup like the one we'll use for the Source field less error-prone.

Great, so something like this:  meier2015[3-12], where meier2015 is the bibtex tag. Sounds good.

exactly

@kirbykat Is there a reason for mapping the CONAN code numbers to other numbers, e.g.

SCCS1719,4,seldom (occurrence uncommon) (original code 31),seldom

It doesn't seem that codes across variables have any shared semantics, so I think it would be less error-prone to just keep the CONAN numbers. In fact, in the example, the CONAN numbers may even add the information that cases 31 and 32 are special cases of code 30?

@xrotwang I would lean towards going with the original CONAN code numbers. In my experience, information has often been left out in 're-coding' initiatives, so your hunch that 31 and 32 are special cases of 30 could very well be correct (I still have not delved into the original CONAN data myself...)

@kirbykat I'm back to extracting the SCCS data from the CONAN files. I have 6 cases where the coded value isn't listed in the codebook. 4 of these specify a value of 10 for variable 27 (in CONAN terms), This variable is described in the codebook as
27 - Extent of burden caused by tribute payments or taxation

	99	=	missing data
	88	=	no tribute or taxation
	20	=	sporadic taxation or request for tribute
	21	=	the exactions are reported not to be burdensome
	22	=	the exactions are reported to be burdensome
	30	=	regular taxation or request for tribute
	31	=	the exactions are reported not to be burdensome
	32	=	the exactions are reported to be burdensome

The current description of the corresponding variable in the SCCS dataset does not seem to match this description:
SCCS1737,0,no tribute or taxation (original code 88),no tribute or taxation
SCCS1737,1,sporadic taxation or request for tribute (original code 10),sporadic taxation or request for tribute
SCCS1737,2,the exactions are reported not to be burdensome (original code 20),the exactions are reported not to be burdensome
SCCS1737,3,the exactions are reported to be burdensome (original code 21),the exactions are reported to be burdensome
SCCS1737,4,regular taxation or request for tribute (original code 30),regular taxation or request of tribute
SCCS1737,5,the exactions are reported not to be burdensome (original code 31),the exactions are reported not to be burdensome_duplicated_5
SCCS1737,6,the exactions are reported to be burdensome (original code 32),the exactions are reported to be burdensome_duplicated_6

Note that the description given here for original code 10 is the one for 20 in the codebook.
This also seems to be a case, where the numbers in the codebook clearly are meant to convey a relation between the codes:

21 means non-burdensome sporadic taxation
31 means non-burdensome regular taxation

Technically such variables are not categorical, because the codes do not form a partition of the design space. So either we should turn them into proper categorical variables (i.e. have codes sporadic (no info on burden), sporadic, burdensome, sporadic, non-burdensome, ...), or not label them as such.

@kirbykat here are the TEXT_COM fields of the 4 offending codings (which may provide an idea what code 10 should be):

A 74: Clans retained a hierarchical composition, but the ranking of lineage groups within a clan was marked by the titles given to the leaders of each, the concomitant was tribute in cattle.

[Although no tribute seems to have been paid to foreign authorities, people had to pay taxes to their own king:]
C 97: The simple peasant looked at the once magnificent barays and other water reservoirs which his forefathers had died in building, heard in silence the promises of the king's tax collectors that they would be made good, then quietly and secretly folded his tent and stole away.

C 218: They [the Ainu] were compelled to pay taxes.

A 5: It is reported that in the eighteenth century the Chinese appointed headmen or chiefs from among the Nivkh to act as middlemen between the Chinese administration and the people. Apparently, these agents carried no authority or weight whatsoever once the Chinese military detachment and tax collector were out of sight.

@xrotwang - I need to take a look at the detailed descriptions, and will try to do this later today. In other datasets, double-digit codes were often used to present two pieces of info "under" the same variable I'd (first digit of code signifies one thing, second something else). Later versions of datasets often split these into two separate variables. Are there some variables that appear in our SCCS version of the CONAN dataset but not in the original Dbf files? If so, the double digit original may have been split in the version currently on the DPLACE site. In some cases I think this splitting is preferable, but i have so far assessed these on a case-by-case basis.

AFAICT the 65 variables from CONAN appear in order and with the same variable description (although changed code descriptions) in our version of SCCS.

@kirbykat as a first step - I think - we should re-instate the original CONAN code numbers in our SCCS dataset.

@xrotwang that sounds good to me. However, are you ok with replacing the original codes for missing data with "NA"s, to keep things consistent across D-PLACE? I noticed the missing data code was 99 in the example you gave.

Yes, 99 for missing data is consistent across variables in CONAN and can be consistently replaced with NA.

@kirbykat Another question: You said not aggregating data is one of the selling points of D-PLACE. So considering we also allow continuous variables, shouldn't CONAN's variable 46, Size of local community, be kept with the original, non-binned values, rather than with the codes
SCCS1756,1,between 10 and 50,between 10 and 50
SCCS1756,2,between 51 and 100,between 51 and 100
SCCS1756,3,between 101 and 150,between 101 and 150
SCCS1756,4,between 151 and 200,between 151 and 200
SCCS1756,5,between 201 and 250,between 201 and 250
SCCS1756,6,between 251 and 300,between 251 and 300
SCCS1756,7,between 301 and 500,between 301and 500
SCCS1756,8,between 501 and 1000,between 501 and 1000
SCCS1756,9,greater than 1001,greater than 1001

@xrotwang Yes! Definitely. And cool that unbinned data are available!


Automate adding of distribution label to Jenkins slaves.

Instead of manually having to specify it in hiera, this can be derived
from the available facts.
This will make the process of upgrading these slaves to trusty simpler.

:+1:


Use name prop as component name for stateless components

Now it's able to mock arrow-style components aka stateless components aka lambdas

I will add tests this day

Current coverage is 87.44% (diff: 100%)

Merging #1795 into master will not change coverage

@@             master      #1795   diff @@
==========================================
  Files            38         38
  Lines          1211       1211
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1059       1059
  Misses          152        152
  Partials          0          0

Powered by Codecov. Last update f5aef40...339b088

Thanks! This is fine.