Documentation: Provide a dotenv example

This extends the documentation on dotenv and adds a reminder that sensitive keys may be output into the generated code.

Thank you @jonathanstanley ,
Can you please explain the note you added: The env values will be saved in the generated code output.  Be careful not to commit code with sensitive keys. ? We don't use the environment variables for the generated output, it's only for parsing the YAML config file.

Thank you @jonathanstanley ,
Can you please explain the note you added: The env values will be saved in the generated code output.  Be careful not to commit code with sensitive keys. ? We don't use the environment variables for the generated output, it's only for parsing the YAML config file.

Ah, yes good question.
In my case, it was a plugin that was reading the value and adding it to the output .ts to create a hard-coded fetcher with the APIKEY in the header.  Or if someone were to create a custom plugin, the possibility for those parsed dotenv values to find their way into the output might be overlooked.
It should probably be written as may be instead of will be.

Ah, yes good question.
In my case, it was a plugin that was reading the value and adding it to the output .ts to create a hard-coded fetcher with the APIKEY in the header.  Or if someone were to create a custom plugin, the possibility for those parsed dotenv values to find their way into the output might be overlooked.
It should probably be written as may be instead of will be.

Thanks @jonathanstanley !


j.get_post_details_v3(post_id, skip=0)

Issue
j.get_post_details_v3(post_id, skip=0) seems to have changed. Now it returns:

one list of dicts called 'details'
one list of dicts called replies
additional fields like these:
'next': <13 digits>,
'hasPrev': boolean,
'remaining': int,
'shareable': boolean,
'readonly': boolean,
'votable': boolean,
'reboostable': boolean,
'powers': list,
'banned': boolean,
'restrictions': list,
'restricted': boolean,
'muted': boolean,
'postblock': boolean,
'ads': list,
'verifiedStatus': 'not_set',
'canReceiveChatRequest': boolean
...

It seems that the skip parameter has no effect anymore, but the next parameter is too short to be a post_id. Not sure what it is.
Environment
If you're reporting a bug, please attach the output of the following commands:
$ pip show jodel_api
$ pip -V
$ python -V
$ python -c "import jodel_api; print(jodel_api.JodelAccount.version); print(jodel_api.JodelAccount.secret)"

Edit: next is probably an unix timestamp, so maybe that should be fed to the "after" parameter, like on other endpoints.

Could you please include the requests and responses you got there? Did you record communication from the app to the jodel server or did you use the api directly?

I used the API directly.


[Bug]: Runtime exception on /min-side navigation

Contact Details
No response
What happened?
On local dev server, when navigating to /min-side
Reproduction steps

Open dev server
Start page
"Min side"

What device are you seeing the problem on?
Desktop
What browsers are you seeing the problem on?
No response
Relevant log output
7 | export const getUserId = (user: User) => {
>  8 |   const val = user[authUserIdClaim];
     |                   ^
   9 |   if (typeof val !== "number") throw new Error(`User claim ${authUserIdClaim} is not a number`);
  10 |
  11 |   return val;
error - lib/user.ts (8:19) @ getUserId
error - TypeError: Cannot read properties of undefined (reading 'https://gieffektivt.no/user-id')

Nvm, this was on #760. Sorry, didn't see the correlation.


setuptools doesn't use venv on macOs

Hi, on macOs Sierra (got the issue before too),
setuptools appears to be producing commands with the wrong python env when it generates shebangs.
I use virtualenvs with python3 (brew install python3),
when I pip install commands, they end up using the /Cellar/ python instead of the venv.
Here's an example of a session showing the incorrect shebangs:
› mkvirtualenv -p python3 bugreport
Running virtualenv with interpreter /usr/local/bin/python3
Using base prefix '/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6'
New python executable in /Users/laurent/.virtualenvs/bugreport/bin/python3.6
Also creating executable in /Users/laurent/.virtualenvs/bugreport/bin/python
Installing setuptools, pip, wheel...done.
virtualenvwrapper.user_scripts creating /Users/laurent/.virtualenvs/bugreport/bin/predeactivate
virtualenvwrapper.user_scripts creating /Users/laurent/.virtualenvs/bugreport/bin/postdeactivate
virtualenvwrapper.user_scripts creating /Users/laurent/.virtualenvs/bugreport/bin/preactivate
virtualenvwrapper.user_scripts creating /Users/laurent/.virtualenvs/bugreport/bin/postactivate
virtualenvwrapper.user_scripts creating /Users/laurent/.virtualenvs/bugreport/bin/get_env_details
(bugreport)

# pip is correct
› which pip
/Users/laurent/.virtualenvs/bugreport/bin/pip
› head -n1 `which pip`
#!/Users/laurent/.virtualenvs/bugreport/bin/python3.6

# pip install django pytest ipython...

› which django-admin
/Users/laurent/.virtualenvs/bugreport/bin/django-admin
› head -n1 `which django-admin`
#!/usr/local/Cellar/python3/3.6.1/bin/python3.6

› which pytest
/Users/laurent/.virtualenvs/bugreport/bin/pytest
› head -n1 `which pytest`
#!/usr/local/Cellar/python3/3.6.1/bin/python3.6

› which ipython
/Users/laurent/.virtualenvs/bugreport/bin/ipython
› head -n1 `which ipython`
#!/usr/local/Cellar/python3/3.6.1/bin/python3.6

I'm pretty sure this issue doesn't involve Setuptools. When installing with pip, the she bangs are generated by pip and distlib and not Setuptools. I suggest searching the bug tracker there and filing a new report if you don't find one.

Whoops, it was me pointing @lsenta to setuptools (from pytest) because I thought it was that generating shebangs - sorry for that! 😆


Sorting by stars

Allow sorting by number of stars.

This should probably live in https://github.com/nuxt/modules/ ? 😋


Teams: Instructors should be able to provide feedback to empty submissions to students in text exercises

Describe the bug
It is not possible to provide feedback to empty submissions.
To Reproduce

Create a text exercise for teams
Open an empty submission
Try to provide feedback and submit
Verify you get error messages that it is not possible

Expected behavior
Should be possible also to provide feedback to empty submissions.
Screenshots
No response
Which version of Artemis are you seeing the problem on?
7.0.5
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
No response

See #1670


Move from develop to master branch

So here's what I'm thinking -- the need for a develop branch is pretty much gone. As I recall I created it to fork off the 0.2.x release from the original 0.1 branch, but clearly 0.2.x is pretty stable now and seems to be what everyone is using.
I am therefore proposing that we merge develop into master and delete the develop branch. All development work for the next release would continue on the master branch, which is I think what most people expect. So we'd have the following workflow:

bugfixes get committed to the the release branch, such as 0.2.x branch; those bugfixes can then be cherrypicked into master if needed.
development for the next major release will be on master. when ready for release candidates, master gets branched into a new release branch (such as 0.3.x) that will take bugfixes, then master is ready to accept new PRs and features for the next release (0.4.x, etc.).

Any feedback? I'll probably do this end of November, maybe sooner if positive feedback. I'd like to take a look thru the currently open tickets and PRs on the 0.3.x branch first and work on whatever can be done before the switch to avoid a little rebasing hassle.

As an active contributor lately @timthelion what do you think?

I support the idea. It would mean that more people would test my code ;) and it could encourage more users because the project would look more active.

I like your project a lot, I'm missing some time for now but I'm coming soon to test it all. Your workflow proposal is fine for me, although I'm looking at it for now.

Just did this check out #789 for my comment. Accepting patches on master for next couple of weeks then we'll branch off a 0.3.x release!


Moderndeck not showing up

after I install the flatpak for moderndeck I try and look it up in my application launcher it does not show up. I run arch 5.15.40-1

What desktop environment are you using?

I am using plasma-Kde

I think it has to do with the fish shell because it is now working but I am using a work around. my default was fish now I changed it to zsh but my terminal starts fish.


Options not working

This is my configuration
<vue-barcode
     :value="codigoBarras"
     width="1"
     height="60"
     :options="{
        lineColor: '#007bff',
        fontSize: '20',
        lineColor: '#990000',
        marginLeft: 30,
        marginTop: 50,
        background: '#dddddd'
     }"
>
      Não possível renderizar a imagem
</vue-barcode>

The style is not applied to the component. I'm using it on quasar framework.
And here is how it's rendered

Options should not be wrapped in 'options'

I've followed the instructions on the official's page. I've copied and pasted the same example that the page contains and the only thing you say me is Options should not be wrapped in 'options'.  That's quite unrespectful.


fix: handle mismatch flow version

Related Issues
Fixes https://github.com/descope/etc/issues/7797

@nirgur  I'm thinking about the OIDC scenario. the flow is starting with some query params. will they be used also where restarting the flow?

is this config turned on or off by default ?
i suggested to start off, suggest a specific customer to turn it on, see in logs if gets less version mismatch

@nirgur I'm thinking about the OIDC scenario. the flow is starting with some query params. will they be used also where restarting the flow?

We are reseting only the

@nirgur I'm thinking about the OIDC scenario. the flow is starting with some query params. will they be used also where restarting the flow?

We are resetting only the stepId & executionId so we should still have it in the context,
But it worth checking this scenario to make sure it's working, the question is how we can do it?
It's not easy to reproduce this error

is this config turned on or off by default ? i suggested to start off, suggest a specific customer to turn it on, see in logs if gets less version mismatch

Yes, it's on by default, do you think we should add another flag for it?
Just to be clear, the descoper will still get the error, the change is that now, in some cases, the flow will restart

these delicate client side logic tends to have some edge cases

We will add a flag for restarting the flow that will be disabled by default
I thought about calling it restart-on-error
Does anyone have a better name?


[Task]: Update mypy type checker in tox.ini

What needs to happen?
If we try to update mypy-protobuf in build-requirements.txt to a latest version and run myypy==0.782(this is what we use currently to run mypy) type checker on the stubs generated by mypy-protobuf, we will see some errors which are wrong.

Update mypy to the latest version since the current version we use (0.782) was released almost 2 years back.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue

Version 0.790 seems to solve this and raise 7 errors. The newer version 1.0.1 is throwing 128 errors on  the PR https://github.com/apache/beam/pull/24599
I will go ahead and update mypy to 0.790 to unblock https://github.com/apache/beam/pull/24599

Version 0.790 seems to throw minimal errors with Beam codebase.  mypy 1.0.1 throwing 128 errors indicates that Beam code base has lot more type hinting errors which were not identified by old versions of mypy.


Typescript 3.5 causes makeStyles problem

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When using typescript 3.5, code that worked in typescript 3.4 is now rejected:
const useStyles = makeStyles((theme: Theme) =>
	createStyles({
		root: {
			paddingTop: theme.spacing(0.5),
			paddingBottom: theme.spacing(0.5),
		},
	})
);

export default function MyComponent(props: any) {
	const { root } = useStyles(); // ERROR: Expected 1 arguments, got 0
	return
{props.children}
;
}

Expected Behavior 🤔
useStyles() should work with 0 arguments
The problem seems to come from a change in something that's commented on in makeStyles.d.ts:
 * If a style callback is given with `theme => stylesOfTheme` then typescript
 * infers `Props` to `any`.
 * If a static object is given with { ...members } then typescript infers `Props`
 * to `{}`.
 *
 * So we require no props in `useStyles` if `Props` in `makeStyles(styles)` is
 * inferred to either `any` or `{}`

In typescript 3.5, Props is inferred to object now.
Steps to Reproduce 🕹
I cannot reproduce this on codesandbox.io right now.  I suspect it's because I can't get the right typescript version there.  I can however reproduce this from a new empty create-react-app, using just the packages below and the code above.
Your Environment 🌎
    "@material-ui/core": "^4.0.1",
    "@material-ui/icons": "^4.0.1",
    "@material-ui/styles": "^4.0.1",
    "typescript": "^3.5.1"

3.5 announcement is scheduled for tomorrow (holiday for me). Will have a look on friday and see what changed.

3.5 was actually released today; I guess they changed.  No hurry of course, but 3.5.1 is the latest tag on npm right now.

But I hope you understand that the latest breaking release will always take some time to adjust to. I recommend you never upgrade to the latest typescript versions in production. Releases in typescript don't follow semantic versioning. Minor releases are almost always breaking.

I do understand.  I upgraded to typescript 3.5.1 on a separate branch of my app, saw this issue, and switched back to my main branch.  My app remains on 3.4.5 and can stay there for the foreseeable future.  I wrote it up right away because, might as well have the information available for anyone who is interesting in looking into it.

Just a little digging I've done. Issue seems to be with IsEmptyInterface, specifically the third clause
unknown extends T ? false : true

Simply testing this out with the default value of {}:
type Test = unknown extends {} ? false : true

Test is true in 3.4.5 and false in 3.5.1
Not sure what the right fix would be. I assume it's due to https://github.com/Microsoft/TypeScript/pull/30637 (Also the first listed "Breaking change" on https://devblogs.microsoft.com/typescript/announcing-typescript-3-5/)

I've got exactly the same issue. Workaround is to just pass in an empty object, so it becomes useStyles({})

Well it's a breaking change. We now have to figure out how to support to incompatible versions of typescript. I think that's where typeVersions comes in. I hope this doesn't require a complete fork but only one for how we determine if a given styles type requires props.

My app remains on 3.4.5

@nmain Since you're on that version of typescript you don't need to specify the theme type and you can omit createStyles. Doesn't solve the issue, but might clean up your app ;)

Actually I was wrong, it does solve the issue.
You didn't specify but you're importing from @material-ui/styles, when i import from @material-ui/core/styles or remove createStyles it works fine

I'm using the latest version (4.0.2) and<EMAIL_ADDRESS>and the issue seems to be still there. Am I the only one still affected?
A screenshot of the error:

Am I the only one still affected?

No, I'm also having the same error with these versions.

@HorusGoul @PieterBoeren Could you provide an example the reproduces the problem? The one from the OP doesn't cause issues. Also, since you're using a newer version of typescript you can omit createStyles and get around the issue

I use this syntax (which fails):
const useStyles = makeStyles(() => {

  return {
    root: {
      backgroundColor: 'red',
    }
  };
});

However, it works when using:
const useStyles2 = makeStyles({
  root: {
    backgroundColor: 'red',
  },
});

But I like the first syntax (which was working before typescript 3.5, because you can then add the theme object very quickly when needed:
makeStyles((theme: Theme) => ...)

Just to be clear: I don't use "createStyles" at all.

@merceyz I'm using the same syntax
const useStyles = makeStyles(theme => ({
  ...
});

Provide a codesandbox please, I can't reproduce with those snippets.
Tried to reproduce in the docs and a fresh project using create-react-app

Can't reproduce it in a CodeSandbox, but based on a standard app create via  create-react-app, adding
"strictPropertyInitialization": false,
"strictNullChecks": false

to the tsconfig breaks it. Removing the two lines (or setting them to "true") and it works...

Thank you, I can reproduce it as well now. Working on a fix

@PieterBoeren Your issue isn't the same as what this issue was for. See https://github.com/mui-org/material-ui/pull/16088#issuecomment-499475862
TL;DR: material-ui doesn't support "strictNullChecks": false

@merceyz Do you know what the use case is for checking
unknown extends T ? false : true

in IsEmptyInterface? That's what seems to differ based on whether strictNullChecks is true or false now. I don't really know enough about how Props is ever used with styles

Remove it and run the test. Should fail because it considers unknown to be an empty interface.

false if the given type is unknown

Yeah I just meant what's the practical use case for that. Obviously if this only supports strictNullChecks that's fine, but it's that check that's causing the problem with strictNullChecks turned off, so really just curious how you could trigger than in actual code or what it protects against.
Basically:

TS 3.4.5, strictNullChecks: true: unknown extends {} == false
TS 3.4.5, strictNullChecks: false: unknown extends {} == false
TS 3.5.1, strictNullChecks: true: unknown extends {} == false
TS 3.5.1, strictNullChecks: false: unknown extends {} == true

Again, just wondering, not saying anything is wrong.

Good question. I think my thinking just went (back in 3.3): "no inferrable means {} so check for that. Anything else is considered required." It's not about a specific case but more about principle of least astonishment with the given constraints.
There's a case to be made to also accept 0 arguments if the expected type is unknown because you can handle undefined in that case (not that you ever see it at runtime). So yeah we could expand StylesRequireProps to also evaluate to true if the type is unknown.
My problem currently is that the types and tests between core/styles and styles are fractured so I'm extra careful about changes. And if those changes only serve unsupported environments I have to defer that for now.
But once we resolve this we can definitely revisit that.

Any updates?

@sashankaryal The original issue was fixed in 4.0.2. If the issue still persists in later versions please open a separate issue and fill out the template.


Bugfix/emv

This is a small pile of changes required to make EMV tags be capable of being uniquely identified and parsed more correctly.  As far as I can tell, these don't interfere with existing tag encoding/decoding and probably should be applied to DER and CER if applicable.

Current coverage is 84.42%

Merging #5 into master will increase coverage by +0.01% as of c03ab2e

@@            master      #5   diff @@
======================================
  Files           21      21
  Stmts         2605    2607     +2
  Branches         0       0
  Methods          0       0
======================================
+ Hit           2199    2201     +2
  Partial          0       0
  Missed         406     406

Review entire Coverage Diff as of c03ab2e

Powered by Codecov. Updated on successful CI builds.

Could use a regression test for encoding univ.Integer(2, tag.TagSet((), tag.Tag(tag.tagClassContext, 0, 128))), so it doesn't come back in the future.

Actually, doesn't this patch break my example? What's the original failing case? The integer from example encodes as:

with patch: 9f000102
without: 9f81000102

According to X690-0207 <IP_ADDRESS>.2 a) "bit 8 of each octet shall be set to one unless it is the last octet of the identifier octets". The initial identifier octet doesn't seem to encode any bits on its own - the whole tag number needs to be base-128 encoded.

The caching bug is pretty nasty though - definitely should be fixed:
In [1]: from pyasn1.codec.ber import decoder
In [2]: decoder.decode(b'\x1f\x02\x01\x02')
Out[2]: (Integer(2, tagSet=TagSet((), Tag(tagClass=0, tagFormat=0, tagId=2))), b'')
In [3]: decoder.decode(b'\x1f\x02\x01\x02')
Out[3]: (Integer(258, tagSet=TagSet((), Tag(tagClass=0, tagFormat=0, tagId=2))), b'')

The caching bug should probably be a different pull request.  I don't have a problem making that happen if necessary.
As far as the extra bit is concerned I sent an email to pyasn1-users that contained this:

I've run into some problems decoding multiple extended tags starting with 9F (9F02, 9F03, 9F06, etc).   If I make the following changes, they decode perfectly.  The reason it's a problem is because the tagId's end up being < 31.  This makes me wonder if the tag number should be 0x80 after the SHL 7 instead of 0.  Otherwise it seems as though (for instance) 82 and 9F02 would decode/encode incorrectly.

I'm not an X690 guru by any means.  I've just been tasked with making EMV work.  To do so, I have to make the tags in emv-asn1 work with BER encoding, which the above change permits.  I've not yet run into any problems (I'm still working through a test suite against our processor) by using this set of patches.  It leaves the on-wire BER the way it seems it should be and just changes the internal (to pyasn1) representation to have the extra bits.  Of course, it also affects the way the ASN.1 grammar is written to be compatible, which I've noted in the above repo as well.
If this can be fixed some other way, please, let me know.

The tags 82 and 9F02 should be equivalent if accepted. But without some specific option like "yeah_I_really_want_to_break_the_standards=True", I don't think pyasn1 should ever encode 9F02 on its own. The caching fix should sort out the decoding side without issues.
But once decoding is fixed, what other issue did you see? The other patches break encoding currently - for example univ.Integer(2, tag.TagSet((), tag.Tag(tag.tagClassContext, 0, 128))) should be
(0x80 + 0x1F), (0x80 + 0x01), (0x00 + 0x00), 0x01, 0x02
   |      |      |       |       |      +- tag+=0*1
   |      |      |       |       +- no more tag bytes follow
   |      |      |       +- tag += 1*128
   |      |      +- more tag bytes follow
   |      +- tag is >31, spread across extra bytes
   +- context-specific, primitive

But it's not correct anymore with the other patches you posted.

82 and [9F02(http://www.emvlab.org/emvtags/show/t9F02/) aren't equivalent.  Having them decode from BER into something different is important.
I ran the tests in the package and it didn't fail any of them, so no existing functionality is disturbed, or the coverage is lacking.

The following is a base64 encoded BER block that is decoded properly by our vendor.  It has both 82 and 9F02 inside it.  While I understand it may not be "to spec", unfortunately it's what the EMV council has spec'd.  If you use emv-asn1, and asn1ate that grammar, you can decode this.  Without it, you cannot.  I will look at adding a test for your above example (since it doesn't seem to be covered by the existing tests) and attempt to adjust my second two patches.
'TwegAAAAAxAQUAtWSVNBIENSRURJVIICXACEB6AAAAADEBCVBQIAAIAAmgMWAyGbAugAnAEAXyoCCEBfNAEBnwIGAAAAAAhYnwMGAAAAAAAAnwYHoAAAAAMQEJ8JAgCMnxAHBgEKA6AAAJ8RAQGfEg9DUkVESVRPIERFIFZJU0GfGgIIQJ8eCDgwMzMwMTI3nyEDFzMAnyYIEa1OFebMt5KfJwGAnzMD4PjInzQDHgMAnzUBIZ82AgADnzcE5kxstJ85AQWfQAXwAPCgAZ9BBAAAATSfUwFS'

@mmattice Could you please elaborate on "82 and 9F02 aren't equivalent"? From BER perspective they  seem equivalent, no? Although 9F02 looks strange as it employs large tag infrastructure for encoding small tags (<=30).
Would it be possible to ask someone from EMV to explain how 82 could be made different from 9F02 when decoding by a strict BER decoder?
BTW, I've added some more tests to the test suite.

One is the amount of the transaction, the other is an indicator of capabilities.
I would love to get in touch with EMVCo.  It's $2500 for a guaranteed response.

EMVCo will review your comment but is not obligated to provide you with a response. Only Subscriber Queries will receive direct EMVCo feedback. If you wish to be a subscriber, SUBSCRIBE NOW!

After finding https://github.com/Shopify/grizzly_ber and this lovely tidbit:

This works differently from OpenSSL::ASN1 in that it does not decode tags (EMV-style). This is important when processing a long tag like 9F02 (Amount Authorized in EMV). OpenSSL will write it as the TLV-DER tag 82. GrizzlyBer maintains the original TLV-BER tag 9F02.

I'm closing this PR.  Thanks for entertaining my learning/discovery process.

@mmattice If you want pyasn1 encoding/decoding tags in a way EMV does, you could customize pyasn1 encoder/decoder code in your application and pass you code to pyasn1 codecs.
That should be fairly easy to do for encoder (overload IntegerEncoder.encodeTag()), to decode colliding tags differently (82 vs 9F02) we could move tag decoding code from Decoder.call() to a separate method (e.g. Decoder.decodeTag?) to make it easily overloadable/customizable.
Would that be a useful change, what do you think?

They're not actually following X.680 afaict.  Tags 9F05 and 9F06 for instance aren't Null and OID respectively.  Also, they're using reserved types like on 9F0F that have no meaning yet. I'm waiting for a response from EMVCo presently on the issue.  Although, I'm also not holding my breath.
Even if they miraculously decide they're going to fix their use of BER, I think I'm going to be stuck with this problem for a while yet anyway.  I've got a whole (internal only because it's generated code) asn1ate'd asn1spec I'm using to get the EMV tags turned into identifiable objects in my code.  Having pyasn1 decode things in a way that's not representable in a proper asn1 grammar (even a kludged one), would make that difficult without hacking the crap out of the pyasn1 internals.
Because what they're doing isn't BER, I'm probably going to dupe the BER encoder/decoder pair into an EMV namespace for my kludge so that it doesn't affect other pyasn1 using things on the system.
I've already got a private repo for a package to encode/decode the Ingenico EMV tag format (which is text based -- 'T82:02:h5800\x1cT9F02:06:h000000008000\x1c' for example).  When I asked them directly if they were going to switch to a more standard format like BER, they said they weren't because they had a major customer that wanted the format they now use.  I wonder if that major customer didn't run into this same problem and side-stepped the problem this way.


Update shell.js to fix broken build after Meteor 2.3 + node 14

See https://github.com/wojtkowiak/meteor-desktop/issues/303
Projects on Meteor 2.3+ will see this error when building on local
internal/fs/utils.js:811
  throw new ERR_INVALID_ARG_TYPE(
  ^

TypeError [ERR_INVALID_ARG_TYPE] [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined
    at Object.writeFileSync (fs.js:1521:5)
    at Module.<anonymous> (/home/evan/.meteor/packages/meteor-tool/.2.6.0.vf20yq.9k39d++os.linux.x86_64+web.browser+web.browser.legacy+web.cordova/mt-os.linux.x86_64/tools/fs/tools/fs/files.ts:1493:23)
    at packages/meteor-desktop-bundler/bundler.js:869:48
    at runAsync (/home/evan/devel/coygo/webapp/node_modules/@babel/core/lib/transformation/index.js:40:10)
    at /home/evan/devel/coygo/webapp/node_modules/@babel/core/lib/transform.js:34:34
    at processTicksAndRejections (internal/process/task_queues.js:77:11) {
  code: 'ERR_INVALID_ARG_TYPE'
}

@KoenLav can we get this published to npm so we can release fixes like this? See #5


Only render ~20 most recent messages

(Only load the remaining messages if the customer scrolls up)

Hello, I am interested to pick this issue, but could you specify on which file those messages are rendered?

hey @pandurijal! this might be a bit tricky to tackle, but if you're willing to give it a shot the code lives in the https://github.com/papercups-io/chat-window repo in this component: https://github.com/papercups-io/chat-window/blob/master/components/ChatWindow.tsx#L691
here are some instructions for getting setup for development: https://github.com/papercups-io/chat-widget#development
but let me know if you need any help or have any questions!


Timer Not Counting Down Correctly: Example Shown. Removed Pokemon Return To Map After Refresh Or Reload.

Pokemon count down as normal but on reaching Zero or Expired they stay on the map and have this type of data attached. Have removed them individually but they return after map is refreshed or reloaded. Thank You .. Great Work Though, Love It :8ball:
Ponyta - #77 (Rare) - FIRE
Disappears at 22:56:22 574h08m18s)
Location: -32.523135,<PHONE_NUMBER>
Exclude   Notify   Remove   Get directions

Same issue here

Sams here

I find the shared despawn timer interesting. When you account for timezones, all of these timeres show exactly the same despawn timmer. Mine show "Despawn at 09:56:22 (UTC-06:00 Central Time)"

Same same. Interesting to note, for every instance of a timed-out pokemon on my map, there is a second pokemon directly underneath the timed out one that is unselectable, but there nonetheless.

Duplicate of #883 which is a duplicate of #881 which is a duplicate of #880. Issue is being worked on here: https://github.com/PokemonGoMap/PokemonGo-Map Specifically here: https://github.com/PokemonGoMap/PokemonGo-Map/pull/572


Working with Typescript.

I've got a project that I'm converting to typescript, I've run into an issue with toaster not popping when it's converted. checking my logs it looks like the toaster object exists (with a pop method) but it just doesn't do anything, no console errors, no pop. Any suggestions?
class myClass {
public toaster;
constructor(toaster, ...){
this.toaster = toaster;
}
...
public createToasterAlert(alert:string) {
console.log("toasting..." + alert);
console.log(this.toaster);
this.toaster.pop({
type: 'info',
body: 'info-toast-directive',
bodyOutputType: 'directive',
directiveData: {body: " " + alert + ""}
});
}

Nothing necessarily looks amiss.
Are you able to create plunker?  I'll be happy to take a look if you can.

Sorry about that, I am not able to make a plunker of it, it's work based code.  Looks like we got it working, though they may have switched to another plugin.

Not a problem.  If it ends up still being a problem, feel free to update this issue and I'll be happy to take a look.


Add conversation flow for Custom Mistral 7B Bot

Managed to get the conversation flow started. Now we have to change the minstral_model_rag.py so that when the conversation flow ends, that it connects to the model:

Now managed to allow the user to continue asking questions to the model´s query engine after the conversation flow had finished:

Modified the code to maintain a history of states that the user has visited. Whenever "solved" is entered, the system should use this history to determine the last state and revert to it, allowing the conversation to naturally flow from that point onwards again after your question has been solved.

Modified the code to maintain a history of states that the user has visited. Whenever "solved" is entered, the system should use this history to determine the last state and revert to it, allowing the conversation to naturally flow from that point onwards again after your question has been solved.

After testing, conversation flow working fine, and generating pdf.
Issue Closed.


[Unexpected Behavior] Editing detailed stats for the first time has wrong default settings highlighted

What you're using
Default theme:
Options -> Profiles -> pick a new profile -> Edit -> Edit Detailed Stats
Saving at this state (i.e. pressing exit) will actually turn these settings off.

Expected behavior

Oh, yeah, I see what is happening, I'll head on to fix this.

Fixed on 4.9.9.


Clarifying roles required to setup Azure DevOps

What are you changing in this pull request and why?
Updating the Azure DevOps guide to include which personas are needed for each section
Checklist

[ ] Review the Content style guide so my content adheres to these guidelines.
[ ] For docs versioning, review how to version a whole page and version a block of content.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding or removing pages (delete if not applicable):

[ ] Add/remove page in website/sidebars.js
[ ] Provide a unique filename for new pages
[ ] Add an entry for deleted pages in website/vercel.json
[ ] Run link testing locally with npm run build to update the links that point to deleted pages

Thanks @matthewshaver !
I have added 2 comments

Thanks @b-per I addressed the comments, hopefully to your satisfaction 😄


Update to nodejs 20

Can you update this to nodejs 20 in your actions files? https://github.com/HatsuneMiku3939/direnv-action/pull/249
Node16 is getting deprecated from github action runners....
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

closed by #249


Added README with screenshots

Issue #12

Thanks! I have done a bit of manual merging of the parts I like best from the different pull requests.


Typing fixes

The cause comes from the way files are handled around Deno and Node.js. See affected lines: https://github.com/insyri/tpy/blob/ac0ee895e2ca8e9225eed8f334dfe18d083912e9/src/types/guild.d.ts#L9-L10
In Deno, this is perfectly fine, in Node.js, the imports are not found. It would be required to instead reference the index of the v8 instead of the direct guild file, like so:
import { RESTPostAPIGuildsJSONBody } from "discord-api-types/rest/v8";
import { APIUnavailableGuild } from "discord-api-types/payloads/v8";

Inspect both of these inside an IDE and intellisense will prove it so. Also, Deno has the mod.ts extension rather than the index.ts, so we would have to add a script to edit that.
Pseudo Solution
The transpiled Node version of these imports look like this:
import { RESTPostAPIGuildsJSONBody } from "discord-api-types/rest/v8/guild";
import { APIUnavailableGuild } from "discord-api-types/payloads/v8/guild";

The directive to transform this would be to reference the parent folder of the guild module.
- import { RESTPostAPIGuildsJSONBody } from "discord-api-types/rest/v8/guild";
+ import { RESTPostAPIGuildsJSONBody } from "discord-api-types/rest/v8";
- import { APIUnavailableGuild } from "discord-api-types/payloads/v8/guild";
+ import { APIUnavailableGuild } from "discord-api-types/payloads/v8";

Probably a build tag like build:ref-parent-module with a regex that matches an import, within that, find of last occurrence of / and delete until ".


Ps to 3.0 structure

Move Pype photoshop to host folder
Contains https://github.com/pypeclub/pype/pull/1014

based on my testing all is looking good. @kalisp Happy to merge when you confirm it's running on your end too.


How to disconnect git

After disconnecting git from the azure function app through the deployment center, as a Contributor, I am unable to add functions manually. What is the proper way to revert or cancel this continuous deployment setting? This should be documented here.

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

ID: 5a34424e-6a23-5517-5755-c9282643876f
Version Independent ID: 1c375649-94d6-863a-203b-cb5ed5e4044e
Content: Continuous deployment for Azure Functions
Content Source: articles/azure-functions/functions-continuous-deployment.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @norman-v I have assigned this to content author for further review and update as appropriate.

After an app has been marked as not-editable in the portal, the ability to edit it again would be a feature request.
In either case, it's always a best practice to do your development locally. #please-close


Music loops when I am doing other tasks

OS:
Linux Mint 18.2
Issue Descriptions:
Music will loop when I open other apps, and seems to get stuck until the OS settles.
Steps to Reproduce:
Clicking on anything that needs to open from the taskbar, or opening up something from the launch menu will reproduce it.
There is another issue which causes the wrong album cover to be displayed when hovering over the volume button on the taskbar. IDK if the 2 are related.

This may be just a memory issue. Computer specs?

As the reporter has not responded to the request for additional information this issue will be closed.  If further information can be provided at some point in the future, comment on this issue with the requested information and if appropriate the issue will be reopened.


Raspberry Pi Screen

Is this the correct Raspberry Pi screen to buy?
[https://www.amazon.com/TFT-LCD-Raspberry-Controller-Interface/dp/B08KZXSJW2/ref=sr_1_12?dchild=1&keywords=XPT2046&qid=1603199760&sr=8-12]
The link in the Shopping List doesnt really point to anything

Yes, this looks good. The important thing is to have the right keywords in the product description which there are.

@maestro32 Please close this.


build blueprint-compiler from source

Changes in this PR

Building blueprint-compiler from source
removing the patch

@fsobolev do you think that the patch worth maintaining?

do you think that the patch worth maintaining?

What was the purpose of the patch in the first place? Wasn't it because the icon didn't show without it?

What was the purpose of the patch in the first place? Wasn't it because the icon didn't show without it?

The patch is there to show the symbolic icon in the gnome activities icon.

What was the purpose of the patch in the first place? Wasn't it because the icon didn't show without it?

The patch is there to show the symbolic icon in the gnome activities icon.

So why would we get rid of that...

What was the purpose of the patch in the first place? Wasn't it because the icon didn't show without it?

The patch is there to show the symbolic icon in the gnome activities icon.

So why would we get rid of that...

This much change just for a symbolic icon! It's not like the app doesn't show any icon, it just shows the black and white version of the app's original icon

This much change just for a symbolic icon!

I don't mind maintaining it to show the correct icon. It's not like the desktop file changes often where we need to fix the patch. Just when we move things around, but I update the patch.
Please restore it back, I'll maintain it dw

Could you do the same PR for Tagger please?

Sure.. just a moment, working on snapcraft


Fixes: Netbox 4.0 breaking change

Renames device.device_role to device.role. This fixes adds logic to handle this change across Netbox versions.

@bortok Have you got a moment to review this one?

@tardoe sorry for the delay, I'm traveling this month. The reason I didn't merge it right away is the need to bump netbox versions in the CI environment to test with 4.0.

Hi @tardoe, I'm working on this. Could you please cancel this PR and make a new one with dev branch as a target? Thank you and sorry for the delay.


Unable to assign devices

We were trying to assign a device to a test customer but unable to assign it from the Colocrossing Portal Addon and at the product detail page: http://prntscr.com/8k96kx
There is no device to assign there + Unassigned Services link is not clickable.
We are using WHMCS 6.0.2

I have not seen anyone have that issue yet and I am unable to replicate it. I will need access to the admin side of your WHMCS to further investigate. Open a ticket in our portal system at portal.colocrossing.com so we may further assist you.
Regards,
Matt Andre


Apply validation state colors to form group help blocks

This will apply AdminLTE's custom form validation state colors to the help text (.help-block) within a form group. Prior to this change, help text would use Bootstrap's default colors.
This change also includes a few examples of help text added to pages/forms/general.html. Here is an snippet of a form group with some help text:

   Input with
                    error
  <input type="text" class="form-control" id="inputError" placeholder="Enter ...">
  Help block with error

See Bootstrap's documentation for more information about help text.

Great addition! Thanks!

No problem. Thanks for your great work on this theme!

When do you plan on putting out a new release?
I pull AdminLTE into my project using npm right now, which I think depends on releases. If you're not planning to soon, no worries, I'll just copy or clone the project into my repo. Thanks again!


Improve analysis feature validation webhooks

Details
In our current CRDs we have the idea of an ObjectReference where Analysis has an obj ref to AnalysisDefinition and this has a bunch of obj ref for each AnalysisValueTemplate to use. These references are used inside client.get functions to retrieve the corresponding k8s resource.
This works fine if the user first creates an AnalysisValueTemplate then refers to it in a AnalysisDefinition and finally declares an Analysis CRD, but if done from the other side user may struggle with invalid names errors which could be frustrating.
A solution would be to validate the names and namespace field in a similar fashion of what k8s does for metadata.name and metadata.namespace : e.g., should be forbidden, no uppercase etc.  see docs here and official docs here.
Acceptance Criteria

[ ] introduce or fix existing webhooks to check the validity of ObjectReference field
[ ] add kuttl tests
[ ] unit tests

DoD

[ ] user cannot apply CRDs with badly formatted ObjectReferences

we decided to keep the behavior aligned with what K8s does, i.e., do not validate until the resource is really needed


Replace '0.0.0' version with 'latest'

Replaced bad-looking '0.0.0' version with awesome 'latest' in Saleor's package.json. Pure cosmetics.

Turns out that NPM displays tiny warning, in which it complains about bad syntax, so we should keep track on Saleor's versions and update package.json after each release.
At least users would know which Saleor's version they are using :)

What's the use case for having a meaningful version string in package.json? This file never makes it to a production environment. Changing this version will also trigger a change in package-lock.json the next time you run npm install so it can result in this change popping up in unrelated pull requests.

Also, the versioning format that we use it's not obvious and loosely follows semver (according to NPM docs, it should). I wouldn't change it at all.


Minor cleanup

Change #1: List -> Collection
I don't think order matters and using collection allows people to pass both Sets and Lists
Change #2: kotlindoc
The method isn't limited to StandaloneContext so I reworded the comment

Ok. Why don't you update the other starter functions? (in StandaloneContext and android)

This is only a patch against the Koin class

The order is important, for override rules. It's name the same to use module A & module A' or A' & A ... Last overriden definition is not the same :/


Fat Headline not visible when Headlines are touching in org mode.

I'm not sure if this is intentional or not, but when multiple Headlines are touching because they are collapsed or because they are simply written next to each other, the fat headline disappears at the point of contact. Is there a way to stop this from happening? Thanks
Screenshots below:

That is because what you see are not the actual headlines anymore. That's just :help 'foldtext'
Foldtext is not really in the buffer, so there is also no way to target it with treesitter. This won't be supported.


SSL certificate error

When running the appimage version, after pressing the download button the following error is outputted, and nothing is downloaded.  The python version works fine however.
Exception in thread Thread-1 (start):
Traceback (most recent call last):
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 1348, in do_open
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 1282, in request
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 1328, in _send_request
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 1277, in endheaders
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 1037, in _send_output
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 975, in send
File "/tmp/.mount_ESOLinhiD7PY/http/client.py", line 1454, in connect
File "/tmp/.mount_ESOLinhiD7PY/ssl.py", line 512, in wrap_socket
File "/tmp/.mount_ESOLinhiD7PY/ssl.py", line 1070, in _create
File "/tmp/.mount_ESOLinhiD7PY/ssl.py", line 1341, in do_handshake
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/tmp/.mount_ESOLinhiD7PY/threading.py", line 1009, in _bootstrap_inner
File "/tmp/.mount_ESOLinhiD7PY/threading.py", line 946, in run
File "/tmp/.mount_ESOLinhiD7PY/addondownloader.py", line 40, in start
File "/tmp/.mount_ESOLinhiD7PY/addondownloader.py", line 69, in download
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 216, in urlopen
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 519, in open
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 536, in _open
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 496, in _call_chain
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 1391, in https_open
File "/tmp/.mount_ESOLinhiD7PY/urllib/request.py", line 1351, in do_open
urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)>

Notes:

Problem seems to happen only when running the appimage, not when running directly from python

Is appimage being blocked somehow from downloading?

Some updates
The issue seems to happen when the appimage cant find the TSL certificate on the machine if the path differs from the one in Fedora 36
certifi.where() doesnt work
Need to ask Nuitka devs whats going on

There's a test for fixing this by disabling the SSL cert lookup in https://github.com/Akselmo/ESOLinuxAddonManager/releases/tag/1.2.3

The certification location is not dynamic: when compiled with nuitka it gets the cert location and its always the one used by the build machine.

Should be fixed now: https://github.com/Akselmo/ESOLinuxAddonManager/releases/tag/1.2.5
If it doesn't work for you, try this instead: https://github.com/Akselmo/ESOLinuxAddonManager/releases/tag/1.2.3-no-certs


Code review of the API for justforfunc

The episode is already out!
https://www.youtube.com/watch?v=nhElL62BSn0

👍 Thanks so much @campoy!  Very educational.

@campoy Thanks for this great video!


CircleCI reports config errors

https://circleci.com/gh/istio/istio/165022
Your config file has errors and may not run correctly:
  2 schema violations found
    required key [jobs] not found
    required key [version] not found

@rshriram feel free to reassign as appropriate

this seems transient? can it be closed?


add backdrop ducks.jpg

tell me if i'm missing something this is my first time.

picture should be 960x720 (you might need to crop part of it)

thanks.

updated the size.

this is your picture right?

yes im a dev and photographer...

alright, adding stuff to this repo doesnt add it directly into the backdrop picker but i can do that

wait for the future is that in the main repo and how do i do that?

its in the penguinmod.github.io repo, like this:
https://github.com/PenguinMod/penguinmod.github.io/commit/89e6e41458b5182746936555153e52bd86f824ae#diff-f5212a6f4293788a48500e688df2f42ae1d6a4b92fc16f21d039e297f67b5878

dang im really bad at javascript but ill try next time


1074 | Number of Submatrices That Sum to Target | CPP

`Number of Submatrices That Sum to Target -
What will change -
Type of Issue -

[x] Adding New Code

Programming Language

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign this to me. I have an efficient solution


Attach license to the project

Closes #263.

Code Samples should be Creative Commons CC0 1.0 Universal (CC0) License, by which all copyright and related rights are waived.

Code Samples should be Creative Commons CC0 1.0 Universal (CC0) License, by which all copyright and related rights are waived.

I added a note about that to the license page.

Can you have a look? Thanks.

Can you have a look? Thanks.


feat(payment): INT-4605 Zip: Display special text for PaymentSubmitBu…

…tton
What? INT-4605
Display Continue with Zip instead of Place Order
Why?
It's more accurate since the shopper is redirected to Zip's page.
Testing / Proof
Before:

After:

@bigcommerce/apex-integrations  @bigcommerce/checkout

💚 Tested
QA Details: I checked that the text continue with zip displays correctly.


Create test data for PRE07

In scope
A test data for a whole moth must be created, and it must reflect the following changes:

BRS-001 Change of supplier @khatozen (02/01/2022)

BRS-002 End of supply @khatozen (04/01/2022)

BRS-003 incorrect change of supplier (back in time) @khatozen (06/01/2022)

BRS-004 create a metering point (status new) @khatozen (08/01/2022)

BRS-006 update master data (change time resolution) @MadsBloendAndersen (10/01/2022)

BRS-007 close down a metering point (status Closed down) @MadsBloendAndersen (12/01/2022)

BRS-012 Change of settlement method @MadsBloendAndersen (14/01/2022)

BRS-013 disconnect and connect a metering point @MadsBloendAndersen (16/01/2022)

Indsend timeseries for Januar 2022 @MadsBloendAndersen

Indsend timeseries for Marts 2022 + Oktober 2022 @MadsBloendAndersen

Out of scope
Change in Charges:
Change of master data for charges do not affects the aggregation, except stop charge.
Update the price I DKK for hourly, daily or monthly prices
It is not allowed to change the time resolution for a charge.
Change in Charge links:

Create a new charge link on a metering point
Stop a charge link on a metering point
Update charge links (quantity for Fee/ subscription)
Charge links exist, the charge is not stopped, but the price is missing.

A test data for a whole moth must be created, and it must reflect the following changes:

BRS-001 Change of supplier
BRS-002 End of supply
BRS-003 incorrect change of supplier (back in time)
BRS-004 create a metering point (status new)
BRS-006 update master data (change time resolution)
BRS-007 close down a metering point (status Closed down)
BRS-012 Change of settlement method
BRS-013 disconnect and connect a metering point

Change in Charges:
Change of master data for charges do not affects the aggregation, except stop charge.
Update the price I DKK for hourly, daily or monthly prices
It is not allowed to change the time resolution for a charge.
Change in Charge links:

Create a new charge link on a metering point
Stop a charge link on a metering point
Update charge links (quantity for Fee/ subscription)
Charge links exist, the charge is not stopped, but the price is missing.

Test data suite here:
https://energinet.sharepoint.com/:x:/r/sites/Joules/Delte dokumenter/General/PRE07 data/MeteringPointMasterData_804_PRE07 (without profiled) V1.xlsx?d=w214c5fbdc38d4c2ba3c905c3e0384611&csf=1&web=1


Mark v2alpha Builder annotation as internal.

This uses the associates attribute of kt_jvm_library to combine multiple targets in the same Kotlin package into a single Kotlin module. This allows us to treat Kotlin's internal visibility more like Java's default/package-private visibility, without having to use a single Bazel target.

This change is

Obsoleted by kt_jvm_proto_library.


Allow services to report config files as invalid

Any linter service can invalidate a config, as Hound proper can, by
enqueuing the InvalidConfigFileJob
https://trello.com/c/42U5K0Bs

A few questions/suggestions, but looks good overall.


take advances of t.TempDir for unit tests

Go provides two nice functions to generate temporary folder and files, it would be nice to use them to avoid having this folder in the repository https://github.com/kubefirst/kubefirst/tree/main/test/artifacts/init

I went a little further in the topic, there is actually a even better approach for tests, that is testing.TempDir, that self clean up after the test case finishes, I'll follow that for the tests in the /pkg folder that is not integration test for the commands.

Something like https://github.com/spf13/afero is probably a better idea.


Add Time, Status Indexes to JobResult

Closes: #DNE
What's Changed

Added indexes to JobResult to speed up list and homepage performance.

TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[NA] Attached Screenshots, Payload Example
[NA] Unit, Integration Tests
[NA] Documentation Updates (when adding/changing features)
[NA] Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

Needs blackening, otherwise looks good. Thanks!

🤦


I can't run the server

When installing the dependencies I get this error message

The machine I am using is one based on the ARM architecture.

What's not working? looks alright to me.

My bad this is my output after installing all dependencies

Seems the same as this issue #9. It seems that Linux doesn't allow opening port 80.
A simple fix is by running npx next dev, it'll run the dev server with default port 3000. Or by changing the dev script to run the default port by removing -p 80 here in the package.json. https://github.com/ccrsxx/twitter-clone/blob/f0e162e02930180db5c05a3df229c4180c891536/package.json#L6

now its working, thank you


Icons in file explorer disappeared after killing tsserver

VSCode Version:1.32.0
OS Version:win10

Steps to Reproduce:

Kill tsserver

Does this issue occur when all extensions are disabled?: No

Since there is no suggestions while I'm typeing, I thought there must be something comsuming my cpu and not responding. So I killed tsserver. After restarting, icons in file explorer and on opened files bars doesn't appear.

@blurHY You closed and reopened the VSCode window?
So you are saying there was a icon theme configured before, but after reload it was no longer set. Did it revert to the seti theme, or the no-icon theme?
What was that icon theme?
Can you still see the icon theme in the list of available themes?

I deleted all my configuration files and reinstalled vscode.


tests: Filter useless warnings

Filter out:

DeprecationWarnings for updater module when we are on purpose
testing deprecated methods from updater
SubjectAltNameWarning for connections to our test server

These warnings are visible with e.g.
python3 test_updater.py
The large change in test_download.py is just indentation into with-block.
Signed-off-by: Jussi Kukkonen<EMAIL_ADDRESS>
Please verify and check that the pull request fulfills the following
requirements:

[x] The code follows the Code Style Guidelines
[x] Tests have been added for the bug fix or new feature
[x] Docs have been added for the bug fix or new feature

I just realised there is a way to do the filtering with single line instead of two at every callsite... I'll modify this a bit

apologies force pushing after review @MVrachev. The changes are:

function is now more descriptively called utils.ignore_deprecation_warnings(module) instead of utils.filter_deprecation_warnings()
the function is a contextmanager now so handles the actual filtering

Now it's even better.
Tried it locally and it works like a charm.
LGTM!


Example 1 fails on Fedora CoreOS

Thanks for the great write up.
I am using Fedora Core OS but found step 20( connecting remotely) doesn't work.  It looks like Firewalld is blocking the remote connection, disabling firewalld allows this to work.
What is the correct was to get podman/firewalld to correctly allow the socket ports to be accessible externally?

Thanks, for bringing up this topic. I opened a PR https://github.com/eriksjolund/podman-traefik-socket-activation/pull/12
for clarification.
Some more explanation:
In Example 1,  podman is running as rootless podman so it does not have enough permissions to modify the firewall on the host.
The systemd user manager (/usr/lib/systemd/systemdsystemd --user) creates the listening socket and then executes podman run ....
Both of these programs are running as the the user test.
For example
$ ps axuw | grep "systemd --user" | grep test
test       1128  0.0  0.3  31672 14312 ?        Ss   Nov20   0:00 /usr/lib/systemd/systemd --user

Instead of disabling firewalld you could also open up specific ports.
For example the command  firewall-cmd takes the argument --add-port=80/tcp
For more details, check out
https://docs.fedoraproject.org/en-US/quick-docs/firewalld/


Close icon for modified tabs is blue on hover

Title pretty much sums it up, attached is a screenshot. Personally I think the icon should be white/grey like everywhere else.

No, this is correct. It reflects the status of the file. Though I understand the contrast against the background could use some love.

Ok, I was just wondering about the blue color, because it's not used anywhere else as far as I can tell. Probably use the accent color instead (in my case "teal")? That might also increase the contrast a little...

Oh, right… I hardly noticed that. Thanks.


App doesn't reconnect on lost connection

When I suspend and later resume my computer it loses connection to the MQTT server and doesn't attempt to reconnect. I need to restart the app manually.
I see this error:
ERROR   [client]   Connect comms goroutine - error triggered read tcp <IP_ADDRESS>:46248-><IP_ADDRESS>:1883: read: connection reset by peer

I can reproduce the error message putting my computer to sleep, but it reconnects in some way. Perhaps I need to leave it asleep longer. Were there any more messages? How long did you leave it after resuming? There should be a 30-second keep-alive, perhaps we can try reducing that.

It happens every time I put the computer to sleep for a longer time (ie. a few hours).  It doesn't reconnect, neither within 30 seconds nor when I wait 30 minutes. There are no other messages in the log.

Okay, thanks for the info. I'll see what I can do, started a new job last week so time is short.

Sorry for not taking more of a look at this - I expect it might just be a matter of checking for a connection occasionally, and reconnecting if it goes down. If anyone wants to take a stab at it and make a PR, I'd be happy to see it.
Frankly, I haven't set this software up again since moving for my job, though I will in future. Until it bothers me, it'll probably keep being pushed down the line 😢

May be a workaround. If you can implement an exit code we can use systemd to do the magic (simple restart).

Looking into this a bit more, I can see a couple of possibilities. It might be failing on "initial connection", which would be related to SetConnectRetry (default false). Or it might be failing because it is currently writing, which would be related to SetWriteTimeout (default no timeout).
I'm going to deploy these settings to machines for a few days and see if it makes a noticeable difference.

How did this go @phallows ?

It seems to be working quite well, but I've got an issue with an unrelated part of my process that makes it hard to verify. I think its probably worth a PR so that others can test, I'll submit that as soon as I can


tests(templates): remove listen http2 (DO NOT MERGE)

Summary
fork from #12327, try to remove listen http2.
Checklist

[ ] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

It will be merged back to #12327.


Fix flaky test testGetPropertyNames.

Fix the flaky test testGetPropertyNames due to the non-deterministic iteration order of HashSet.
This flaky test is found by using Nondex, a tool used to find flaky tests in Java projects. The original assertion on Line#69 may fail because the iteration order of HashSet is non-deterministic, and the elements in the array converted from somePropertyNames may not be the same as the array result. By converting result to a HashSet, the new assertion assertEquals will only check the elements are equal in both sets so that the test will always pass.

I have read the CLA Document and I hereby sign the CLA

Codecov Report

Merging #4082 (1189252) into master (0e14333) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4082      +/-   ##
============================================
- Coverage     52.45%   52.39%   -0.06%
+ Complexity     2612     2609       -3
============================================
  Files           484      484
  Lines         15203    15203
  Branches       1572     1572
============================================
- Hits           7974     7966       -8
- Misses         6675     6683       +8
  Partials        554      554

Impacted Files
Coverage Δ

...work/apollo/biz/message/DatabaseMessageSender.java
50.00% <0.00%> (-14.59%)
:arrow_down:

.../apollo/internals/RemoteConfigLongPollService.java
77.71% <0.00%> (-0.61%)
:arrow_down:

Continue to review full report at Codecov.

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


fix: search box: icon button default htmltype should be button path should have a default

SUMMARY:
The search box icon button didn't have any defaults, and the icon button was causing page refreshes due to a missing htmlType, and if customized - the icon path may be undefined.
The InfoBar button types were incorrectly using NeutralButton, this change updates to use SystemUIButton.
Also cleans up some type annotations.
JIRA TASK (Eightfold Employees Only):
ENG-2959
ENG-23859
CHANGE TYPE:

[X] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[X] I have added unittests for this change

TEST PLAN:
Run yarn and yarn storybook. Verify the changes to InfoBar and SearchBox stories or use Codesandbox CI to test.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@38b7c0a). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #356   +/-   ##
=======================================
  Coverage        ?   89.43%
=======================================
  Files           ?      481
  Lines           ?    12992
  Branches        ?     4372
=======================================
  Hits            ?    11620
  Misses          ?     1347
  Partials        ?       25

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remove multiroom broadcast and improve malformed input safety

Small comment on console log vs error, otherwise :+1:


.version file pointing at wrong url

KSP-AVC can't check for new versions because the url isn't pointing to a valid .version file. Current link is
https://raw.githubusercontent.com/TriggerAu/ksptips/master/ksptips/ksptips.version
Change it to
https://raw.githubusercontent.com/TriggerAu/KSPTips/master/PluginBuilder/KSPTips.version

Forgot to bulid the version file - done now


Add MCU data table to README

TAKE INSPO FROM THIS REPO AND ADD A SIMILAR TABLE LIKE THE ONE BELOW
https://github.com/cpq/bare-metal-programming-guide?tab=readme-ov-file
https://github.com/nhivp/Awesome-Embedded?tab=readme-ov-file

hey so I must add the samethings that are in the image above?

yeah so in the readme file, you need to create a table similar to the one above by finding the information in the documents in the repository

so about finding the information in docs you mean the link that is in the description of the projects . or I must use the repositories that you have put above or there were just for reference ?.

so about finding the information in docs you mean the link that is in the description of the projects . or I must use the repositories that you have put above or there were just for reference ?.

You will find necessary information in the files in this repository. The links in this description are just for reference


Don't allocate Task when running a users tag body.

We should change the TagHelper infrastructure code gen to call into a new GetAndSetChildContent (or a method equivalent) that executes the child content and sets the result on the current TagHelperOutput. What happens today is:
if (!__tagHelperExecutionContext.Output.IsContentModified)
{
	__tagHelperExecutionContext.Output.Content = await __tagHelperExecutionContext.Output.GetChildContentAsync();
}

Where instead we could do:
if (!__tagHelperExecutionContext.Output.IsContentModified)
{
	await __tagHelperExecutionContext.Output.GetAndSetChildContent();
}

This would avoid allocating the Task<TagHelperContent> when calling into GetChildContentAsync. Turns out it represents a decent size chunk of the TagHelper execution infrastructure:

Self assigned, this was part of our hackathon. PR is already out.

4212b7e7130a77e199d3a81d6717dda8f14f0967
https://github.com/aspnet/Mvc/commit/56bbb215ec9975adcd2e4f73b9326250b71b691b


branch not indexed

I have this configuration:
{
    "$schema": "../schemas/v2/index.json",
    "repos": [
        {
            "type": "gitlab",
            "url": "https://gitlab.****.com",
            "token": "glpat-*****",
            "all": true,
            "revisions": [
                    "main",
                    "dev"
            ]
        }
    ]
}

But when I search, I can see only results from the main branch, also in the repositories page it shows only one branch per repository. Shouldn't it index the "dev" branch also?

It looks like your config is slightly off - revisions is of type object that has the property branches, which is where you want to specify your branches. E.g.,:
{
    "$schema": "../schemas/v2/index.json",
    "repos": [
        {
            "type": "gitlab",
            "url": "https://gitlab.****.com",
            "token": "glpat-*****",
            "all": true,
            "revisions": {
                "branches": [
                    "main",
                    "dev"
                ]
            }
        }
    ]
}

An enhancement we should make is to raise a warning / error when a configuration is incorrect.
Going to close this issue, feel free to re-open if this didn't fix your issue.


Escape special characters from string passed to RegExp

Fixes #132

Done. Let me know if there's any other changes needed.

:shipit: thanks!


kvm: Add missing libraries to stage1 image

Due to last pxe.img upgrade, few libraries are missing inside
stage1-kvm.aci image. This patch is resolving this issue.
Note: this issue impacts on entering feature for kvm flavor

LGTM

LGTM


gigasecond: use times (not dates) for inputs and outputs

A duration of a gigasecond should be measured in seconds, not
days.
The gigasecond problem has been implemented in a number of languages,
and this issue has been generated for each of these language tracks.
This may already be fixed in this track, if so, please make a note of it
and close the issue.
There has been some discussion about whether or not gigaseconds should
take daylight savings time into account, and the conclusion was "no", since
not all locations observe daylight savings time.

I can confirm the issue exists in xpython. I'm working on a fix now.


Change text to C++11 at least as soon as C++11 feature included in source text.

remove obsolete keywords
fix variable types like size_t, time_t.

fixed


New in-inplace mul!, new tests for it and *(ZZ2Matrix, ZZ2Vector)

@matthias314  Okay, I tried the rebasing, hopefully everything is correct now,

Sure, matrix--matrix also sounds good, just haven't gotten around to it. Perhaps merge this now, and leave the matrix-matrix case to a separate commit? (Would it be helpful to open an issue for it?)

I've added mul! for matrix-matrix multiplication and also the 5-parameter variants.


SDP-897: Remove Stellar logo from Page Header

It was decided to not use the Stellar logo in the product, since this will be hosted by others and if we were to use the Stellar logo, it could seem like we’re co-responsible for their use case, which could raise a legal problem for SDF.

stellar-disbursement-platform-frontend-preview is available here:http://sdp-frontend-pr19.previews.kube001.services.stellar-ops.com/

stellar-disbursement-platform-frontend-preview is available here:http://sdp-frontend-pr19.previews.kube001.services.stellar-ops.com/


mua: fix default message-id

Context: https://github.com/swoosh/swoosh/pull/945#issuecomment-2532876675

This PR introduces an error
** (KeyError) key :from not found in: %Mail.Message{headers: %{}, body: nil, parts: [], multipart: true}
    (swoosh 1.17.4) lib/swoosh/adapters/mua.ex:226: Swoosh.Adapters.Mua.put_headers/2
    (swoosh 1.17.4) lib/swoosh/adapters/mua.ex:190: Swoosh.Adapters.Mua.render/1
    (swoosh 1.17.4) lib/swoosh/adapters/mua.ex:163: Swoosh.Adapters.Mua.deliver/2

👋 @teamon
Thank you for the report! https://github.com/swoosh/swoosh/pull/982 should fix it. I'll also try opening a PR to add Swoosh.Adapters.Mua-specific tests to catch problems like this earlier.


dev ux: simplify import paths of frequently used items in scripts and demos

I propose making common scripts/demos less verbose (i.e. developer onboarding), we can make the most commonly imported objects auto-imported at the top level of the package.
before
from mflux.config.model_config import ModelConfig
from mflux.config.config import Config
from mflux.flux.flux import Flux1

after
from mflux import Flux1, Config, ModelConfig

considerations:

any package/module re-organizations in the future can be transparent to the user
this is soft change, any pre-existing code that does the full path import will still work
imports within the package implementation should still use full path imports following explicit is better than implicit value, as code at that level is meant for maintainers
added a basic test that would exercise the imports via generate.py

also taking opportunity to address the seemingly unnecessary (probably debug artifact from the past):
os.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
is src/mflux and was being inserted as the first element of sys.path, which probably fixed an import path error in the past. I think this is a no-op removal.

I like it. Will rebase this on top of latest main make similar changes to the new controlnet file


envoy access log clarification

We are using istio-1.8.3 and which is using envoy 1.16.3-dev.  From envoy access log, we would like to clarify what is  UPSTREAM_LOCAL_ADDRESS, DOWNSTREAM_REMOTE_ADDRESS & DOWNSTREAM_LOCAL_ADDRESS.
access log definition
[%START_TIME%] "%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH)% %PROTOCOL%" %RESPONSE_CODE% %RESPONSE_FLAGS% %RESPONSE_CODE_DETAILS% %CONNECTION_TERMINATION_DETAILS% "%UPSTREAM_TRANSPORT_FAILURE_REA
SON%" %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% "%REQ(X-FORWARDED-FOR)%" "%REQ(USER-AGENT)%" "%REQ(X-REQUEST-ID)%" "%REQ(:AUTHORITY)%" "%UPSTREAM_HOST%" %UPSTREAM_CLUSTER% %UPSTREAM_LOCAL_ADDRESS% %DOWNSTREAM_LOCAL
_ADDRESS% %DOWNSTREAM_REMOTE_ADDRESS% %REQUESTED_SERVER_NAME% %ROUTE_NAME%\
access log
[2021-05-01T18:38:31.133Z] "- - -" 0 "-" 143 2096 6005 - "-" "-" "-" "-" "<IP_ADDRESS>:55443" outbound|55443||winsolst01 <IP_ADDRESS>:46192 <IP_ADDRESS>:55443 <IP_ADDRESS>:44295 - -
extraction from the log
%UPSTREAM_LOCAL_ADDRESS% %DOWNSTREAM_LOCAL_ADDRESS% %DOWNSTREAM_REMOTE_ADDRESS% %REQUESTED_SERVER_NAME% %ROUTE_NAME%\n"
<IP_ADDRESS>:46192      <IP_ADDRESS>:55443         <IP_ADDRESS>:44295         -                        -
<IP_ADDRESS> is the pod address.  It is the downstream address
<IP_ADDRESS>:55443  is the address that the process inside the pod connecting to.  it is the upstream address.
%UPSTREAM_LOCAL_ADDRESS%
Local address of the upstream connection. If the address is an IP address it includes both address and port.
%DOWNSTREAM_REMOTE_ADDRESS%
Remote address of the downstream connection. If the address is an IP address it includes both address and port.
%DOWNSTREAM_LOCAL_ADDRESS%
Local address of the downstream connection. If the address is an IP address it includes both address and port. If the original connection was redirected by iptables REDIRECT, this represents the original destination address restored by the Original Destination Filter using SO_ORIGINAL_DST socket option. If the original connection was redirected by iptables TPROXY, and the listener’s transparent option was set to true, this represents the original destination address and port.
Thanks.

The overall description is awesome!

%DOWNSTREAM_LOCAL_ADDRESS%
Local address of the downstream connection. If the address is an IP address it includes both address and port. If the original connection was redirected by iptables REDIRECT, this represents the original destination address restored by the Original Destination Filter using SO_ORIGINAL_DST socket option. If the original connection was redirected by iptables TPROXY, and the listener’s transparent option was set to true, this represents the original destination address and port.

Thank you for clarifying it. There are several positive and negative methods to recover the original dst.

ProxyProtocolFilter
The Downstream local address is the ip in the proxy protocol payload.
x-envoy-original-host
Not impact DOWNSTREAM_LOCAL_ADDRESS


Pyrouge takes so much time to give the results

Hey, I run the package with the instructions on github/website, but it takes so much to give the results for Rouge scores (~20/25 minutes!) .
This is my code, which is actually a simple copy from the documentation:
from pyrouge import Rouge155
r = Rouge155()
r.system_dir = 'rad_decoder_sys/'
r.model_dir = 'rad_decoder_ref/'
r.system_filename_pattern = 'rademb-preds.(\d+).txt'
r.model_filename_pattern = 'rademb-gold.#ID#.txt'

output = r.convert_and_evaluate()
print(output)

When I try to run the script above, I end up with these logs:

2018-12-07 17:48:23,520 [MainThread  ] [INFO ]  Writing summaries.
2018-12-07 17:48:23,522 [MainThread  ] [INFO ]  Processing summaries. Saving system files to /var/folders/gz/4hxz5p9d235bxgbmb4phkk_00000gp/T/tmpre5wi1ih/system and model files to /var/folders/gz/4hxz5p9d235bxgbmb4phkk_00000gp/T/tmpre5wi1ih/model.
2018-12-07 17:48:23,522 [MainThread  ] [INFO ]  Processing files in rad_decoder_sys/.
2018-12-07 17:48:23,527 [MainThread  ] [INFO ]  Processing rademb-p.3614.txt.
2018-12-07 17:48:23,528 [MainThread  ] [INFO ]  Processing rademb-p.1003.txt.
...
2018-12-07 17:48:29,831 [MainThread  ] [INFO ]  Saved processed files to /var/folders/gz/4hxz5p9d235bxgbmb4phkk_00000gp/T/tmpre5wi1ih/model.
2018-12-07 17:48:57,667 [MainThread  ] [INFO ]  Written ROUGE configuration to /var/folders/gz/4hxz5p9d235bxgbmb4phkk_00000gp/T/tmp4z2m9wl7/rouge_conf.xml
2018-12-07 17:48:57,667 [MainThread  ] [INFO ]  Running ROUGE with command /Users/ss4164/PycharmProjects/rouge_calculator/official_rouge/RELEASE-1.5.5/ROUGE-1.5.5.pl -e /Users/ss4164/PycharmProjects/rouge_calculator/official_rouge/RELEASE-1.5.5/data -c 95 -2 -1 -U -r 1000 -n 4 -w 1.2 -a -m /var/folders/gz/4hxz5p9d235bxgbmb4phkk_00000gp/T/tmp4z2m9wl7/rouge_conf.xml

I tried with both PyCharm and terminal, but that didn't help.
Anyone having the same issue?
Thanks!

Running pyrouge takes that long for me also.
You could try https://github.com/pltrdy/rouge if your model/gold summaries have the same length.
Otherwise, as far as I know, we have to run the ROUGE perl script.
Does anyone have other solutions(model/gold summaries have different lengths, without using the ROUGE perl script)

Not an issue afaik. For about 12k instances, I experience similar evaluation times.
If you want something faster (albeit results aren't 100% identical), I recommend py-rouge


Rename CBL0033J to CBL0003J

Signed-off-by: Hartanto Ario Widjaya<EMAIL_ADDRESS>This would fix #198. It appears #153 is not merged into the mainframe system, thus resulting in confusion with the PDF content.
Personally, I would prefer that the new CBL0003.cbl be renamed to something else or adopt the CBL0033 naming (8af11a09e2218c03901ebf877b40fbceb73e02b6) to prevent confusion with existing tutorials and PDF which are already on circulation (such as Coursera, Pluralsight and the IBM Learning Platform).

Thanks for contributing @tanto259. If @paulnewt just added that CBL0033J (https://github.com/openmainframeproject/cobol-programming-course/blob/master/COBOL Programming Course %231 - Getting Started/Labs/jcl/CBL0033J.jcl) member on the mainframe, would that resolve this issue? Tagging @klausmelcher who contributed #153 as well.

Thanks for contributing @tanto259. If @paulnewt just added that CBL0033J (https://github.com/openmainframeproject/cobol-programming-course/blob/master/COBOL Programming Course %231 - Getting Started/Labs/jcl/CBL0033J.jcl) member on the mainframe, would that resolve this issue? Tagging @klausmelcher who contributed #153 as well.

Makes sense to me. We need to enable the system to support new content and old content simultaneously to accommodate all users. I'll wait for @paulnewt's comment before approving so he is aware of the needed update.

Makes sense to me. We need to enable the system to support new content and old content simultaneously to accommodate all users. I'll wait for @paulnewt's comment before approving so he is aware of the needed update.

ZOS.PUBLIC.CBL member CBL0003 now exists.  ZOS.PUBLIC.JCL member CBL0033J now exists.  Member CBL0003J and CBL0033J compile newly added source CBL0003

ZOS.PUBLIC.CBL member CBL0003 now exists.  ZOS.PUBLIC.JCL member CBL0033J now exists.  Member CBL0003J and CBL0033J compile newly added source CBL0003

Thanks for the upload @paulnewt !
But I think there's some issues here:

Originally, before #153, CBL0003J is a JCL which is intentionally designed to have a bug, in which it executes CBL0001 but with an incorrect DD name
Klaus then submitted #153, in which he added a COBOL program which he called CBL0003 and the corresponding JCL CBL0003J, which is designed to explain subroutine calls. Since CBL0003J already existed previously, he renamed the old one (which is buggy) to CBL0033J.
However, this causes some issues since the Lab files on both Coursera and IBM Learning Platform still references the buggy JCL as CBL0003J instead of its new name, CBL0033J. Thus, in order to maintain compatibility with prior docs out there, I open this pull request.

Basically what this pull request did is bring back the renamed JCL to CBL0003J and changes Klaus' COBOL and JCL to adopt the CBL0033 naming. But the update basically removed the buggy JCL and put an exact copy of CBL0001 as CBL0003, so Klaus' code is still not on the system.

Thanks for the upload @paulnewt !
But I think there's some issues here:

Originally, before #153, CBL0003J is a JCL which is intentionally designed to have a bug, in which it executes CBL0001 but with an incorrect DD name
Klaus then submitted #153, in which he added a COBOL program which he called CBL0003 and the corresponding JCL CBL0003J, which is designed to explain subroutine calls. Since CBL0003J already existed previously, he renamed the old one (which is buggy) to CBL0033J.
However, this causes some issues since the Lab files on both Coursera and IBM Learning Platform still references the buggy JCL as CBL0003J instead of its new name, CBL0033J. Thus, in order to maintain compatibility with prior docs out there, I open this pull request.

Basically what this pull request did is bring back the renamed JCL to CBL0003J and changes Klaus' COBOL and JCL to adopt the CBL0033 naming. But the update basically removed the buggy JCL and put an exact copy of CBL0001 as CBL0003, so Klaus' code is still not on the system.

@tanto259 @paulnewt the confusion comes from CBL0003J.jcl executing CBL0001.cobol and CBL0033J.jcl executing CBL0003.cobol. We don't want to change the former because of backwards compatibility on existing tutorials in circulation that are somewhat out of date. But we can change the latter without much impact.
Going forward we should avoid renaming existing programs for backwards compatibility reasons.
@tanto259 can you review what is on the system and work with @paulnewt if an update is needed. The PR looks good to merge once we verify the system matches.
To summarize:
CBL0003J compiles CBL0001
CBL0033J compiles CBL0033
CBL0003 should not exist to avoid confusion

Sure. @paulnewt, here are the changes needed to the system:

rename CBL0003.cbl to CBL0033.cbl in ZOS.PUBLIC.CBL
move CBL0003J and CBL0033J from ZOS.PUBLIC.ADHOC.JCL to ZOS.PUBLIC.JCL and overwrite the old one there

Those should correspond to the changes made in this pull request. At the end of the day, CBL0003.cbl should no longer exist on the system.

@MikeBauerCA I have talked to Paul, and he has merged the required changes to the system.
ZOS.PUBLIC.CBL(CBL0033) now exists, CBL0003 is no longer there, and ZOS.PUBLIC.JCL(CBL0003J and CBL0033J) updated.

@tanto259 thanks, if you can resolve the merge conflicts in this PR, I will accept and merge.


Unable to connect to database

the application breaks down when a sql request is made as the database object ($conn) is not being instantiated.

Is the database imported correctly?


Without filter not functioning correctly with hours between

The without filter does now work correctly with an hours between filter parameter. The statement that is failing is as follows:
define ClaimWithQualifiyingPOSWithoutEncounter: ClaimWithQualifyingPOS C without InpatientEncounters E such that hours between C.billablePeriod."start" and E.period."start"  <= 24
The context for this logic is available in the attached doducment:
mmi_cql-issue_without_filter_not_working.txt

Hi @jscherer-motivemi. Is this an issue against one of the execution engines?  If so, which one?

Hi @cmoesel. I am not sure that I know. We use the CQL java engine version 1.2 currently. Let me know if you need additional information.

OK.  In that case I'd recommend you file these against that specific project here:
https://github.com/DBCG/cql_engine
(FYI @brynrhodes)


Get text decompression to process .BIN files & output a single .BIN

Currently, you can build the BrightisCompressor in the Scripts folder to use what we already have.

FUN_80020170 is the decompress function in SCPS_101.05 ingame.
We are fairly sure that the game may be using different tables for other [smaller] files in the game.

i.e. OVR.BIN may be using a standard Shift-JIS table w/ command codes, but ONMOVR.BIN may be using something modified.

The goal for this is to have:

Process the entirety of one of the .BIN files in one pass
Output the decompressed file as one .BIN file
Be able to process any/all files in the game that contains text that will need to be translated. [i.e. OVR.BIN, ONMOVR.BIN, etc.]

Onepiecefreak has built the BrightistFiler tool here, which accomplishes this well & allows the file to be compressed back after being split into sectors/blobs,
Closing successfully for now.


Strongly signed

Hi,
I am having an issue registering the Nuget package into the GAC. I know it's not ideal, but that's how they are doing things here. I am getting an error indicating that the DLL is not strongly signed. Is this something that will change in the future?
Tim

See https://github.com/AutoMapper/AutoMapper.Extensions.Microsoft.DependencyInjection/issues/13


Use str.format, raise exception in name function

Hey,
this change tackles the eminent error as can be seen in #31 (doesn't resolve the actual issue).

name method now raises an exception when no data could be retrieved
Unfavorable line wrapping fixed
string building in combination with the logger was wrong. I've replaced everything with the better str.format method e.g.

OSError: [Errno Could not read data from Mi Flora sensor %s] C4:7C:8D:60:DB:A5

Sorry to bother you. Any issues with this PR I should be aware of? Cheers ;)

logging should not use string.format
lines should be no longer than 79 characters.

logging should not use string.format

Okay, the bug still needs to be fixed somehow...

lines should be no longer than 79 characters

A rather archaic length. Nowadays 120 or 160 is reasonable. Anyhow that's up to you.

Please create a new pull request for the fix
About line length and a lot of other style guidelines, please check the PEP8 Python style gudielines:
https://www.python.org/dev/peps/pep-0008/#maximum-line-length

Done (#35)
I'm aware of PEP8. As I said, it's up to you.


Practice 4 - pure_pursuit_follower - code comments

Comments

As a reminder there are some comments in the #2 that have not been updated yet

I the path_callback

if len(msg.waypoints) !=0:
https://github.com/Zillaynain/autoware_mini_practice/blob/8304c8d0cc018d758cd8c7153778f83ce0af80ba/practice_4/nodes/control/pure_pursuit_follower.py#L57-L59
But in else only one of them
https://github.com/Zillaynain/autoware_mini_practice/blob/8304c8d0cc018d758cd8c7153778f83ce0af80ba/practice_4/nodes/control/pure_pursuit_follower.py#L60-L61

First thing in current_pose_callback should be assigning self.path_linestring and self.distance_to_velocity_interpolator to local variables using threading.lock() and then check if they are not None and only then proceed to use them in the callback
https://github.com/Zillaynain/autoware_mini_practice/blob/8304c8d0cc018d758cd8c7153778f83ce0af80ba/practice_4/nodes/control/pure_pursuit_follower.py#L64-L67

Why do you have unused and almost copy of the current_pose_callback called **a**current_pose_callback in your code? Delete any unnecessary code!
https://github.com/Zillaynain/autoware_mini_practice/blob/8304c8d0cc018d758cd8c7153778f83ce0af80ba/practice_4/nodes/control/pure_pursuit_follower.py#L88-L111

all the comments incorporated.

OK

OK

You are setting self.path_linestring already in if and else, same with self.distance_to_velocity_interpolator. In if and else you should set only local variable and global variable should be set under self.lock.

https://github.com/Zillaynain/autoware_mini_practice/blob/06c5047b05955dbe2bd7b2d8819646841408d540/practice_4/nodes/control/pure_pursuit_follower.py#L57-L65

In the beginning of the current_pose_callback you should set path_linestring value from global variable using self.lock.

Please check

ok


CALL... YIELD clause ignores WHERE condition

Describe the bug
CALL ... YIELD clause ignores the WHERE condition. WHERE clauses used proceeding a CALL ... YIELD subquery will be ignored.
How are you accessing AGE (Command line, driver, etc.)?
psql commandline
What is the command that caused the error?
SELECT * FROM cypher('test', $$ CALL sqrt(1) YIELD sqrt WHERE sqrt = 2 RETURN sqrt$$) as (a agtype);

Result:
  a
-----
 1.0
(1 row)

Expected behavior
Return value should be null; sqrt(1) != WHERE condition of 2.
Environment (please complete the following information):

Version: latest version of master branch

This is the offending line -
2184         if (is_ag_node(clause, cypher_call))
I changed it to -
2184         if (is_ag_node(clause->self, cypher_call))
It looks like just a typo that got overlooked.


Change the layout in index.astro.

Hi ,developer ,I want to change  the layout of index.astro as above , I just a beginner of frontend learner ,  can give any tips about it ? I try  many idea but  can't implement.

It's been a while since you opened this.
So, I'm not sure this has been solved yourself.
In case I have to create such layout, I would use Flexbox of CSS. (in this case TailwindCSS)
Code

SVG

    Lorem ipsum dolor sit amet consectetur adipisicing elit. Eligendi
    voluptates unde reiciendis labore. Alias nemo maiores assumenda
    voluptatibus sunt molestias eum, enim id dolorem autem quia incidunt
    deleniti a voluptate?

Result:
Mobile

Tablet & Web

Closing this issue for now.
If you have any problems, feel free to open a new discussion.
Thanks for your understanding.


Autoform 5 breaking changes

Autoform v5 was released today and has lots of breaking changes.  We have some work to do before it's ok to upgrade.  I just started working on that, but I figured I'd throw this issue up so people know what's going on when Telescope automatically updates Autoform and then falls over and dies on them.  :)
Until it's sorted out, everyone should throw a version constraint in their .meteor/packages file to prevent the upgrade.
Specifically, this needs to be changed to this<EMAIL_ADDRESS>

We should coordinate on this. Maybe ping me on Slack to let me know what you're working on exactly?


Inhanve footer design and navbar designBUG:

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Footer design and navbar design is not looking good we can add some good design  and hover buttons
Record

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

See my issue

please assign this task to me, i have good experience in and did some good projects in HTML,CSS & JS.

I'm proficient in design and have a strong grasp of CSS. I've created clones of Zomato and Netflix. Please consider assigning this task to me.

I would like to contribute on this project. Please assign it to me. @Sameena10

Hey.I can help you with this please assign me this task under GSSoC'24  as a contributor.
Record

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