Not able to use column names with upper-case letters, special characters

I created a keyspace in cqlsh:
CREATE KEYSPACE demo
  WITH REPLICATION = {
   'class' : 'SimpleStrategy',
   'replication_factor' : 1
  };

Create table with _id as the primary key:
CREATE TABLE bots (
   "_id"           text,
   name         text,
   PRIMARY KEY ("_id"));

Insert a new row into the table:
INSERT INTO bots ("_id", name) VALUES ('000000000000000000000000', 'test');

Send a curl command to filter by _id and you get a Filter type not supported error:
$ curl -H 'Content-Type: application/json' -H 'X-Cassandra-Token: Cassandra:Y2Fzc2FuZHJh:Y2Fzc2FuZHJh' -X POST -d '{"findOne":{"filter":{"_id":"000000000000000000000000"},"options":{}}}' http://<IP_ADDRESS>:8181/v1/demo/bots
{"errors":[{"message":"Server internal error: Filter type not supported, unable to resolve to a filtering strategy","errorCode":"SERVER_INTERNAL_ERROR"}]}
$

This only seems to happen with _id, replace _id with id and this works fine.

It looks like this issue isn't limited to just _id, but also means you can't query by properties with uppercase letters. For example, consider the following setup:
CREATE TABLE people (
   id           text,
   "firstName"  text,
   PRIMARY KEY (id));

CREATE INDEX ON people ("firstName");

INSERT INTO PEOPLE (id, "firstName") VALUES ('000000000000000000000000', 'John');

Making request that filters by firstName fails:
$ curl -H 'Content-Type: application/json' -H 'X-Cassandra-Token: Cassandra:Y2Fzc2FuZHJh:Y2Fzc2FuZHJh' -X POST -d '{"findOne":{"filter":{"firstName":"John"},"options":{}}}' http://<IP_ADDRESS>:8181/v1/demo/people
{"errors":[{"message":"Server failed: root cause: (java.lang.RuntimeException) io.stargate.sgv2.jsonapi.service.operation.filters.table.codecs.UnknownColumnException: No column found for table people with name firstname","errorCode":"SERVER_UNHANDLED_ERROR"},{"message":"Server failed: root cause: (io.stargate.sgv2.jsonapi.service.operation.filters.table.codecs.UnknownColumnException) No column found for table people with name firstname","errorCode":"SERVER_UNHANDLED_ERROR"}]}

Works fine if you replace firstName with first_name.

It is possible there are 2 different problems (special handling for _id due to Collection requiring different kind of filter construction, vs. quoting for non-lower-case column names) or just 1 (quoting). But I can create failing tests for quoting and see if solving that might solve _id case too.

@vkarpov15 I noticed that you didn't include specific curl command you used -- would it be possible to add that, just so I know I am reproducing exact issue (like find vs findOne)

It definitely appears that:

Creating tables with columns like _id or ID works fine (after fixes in PR)
Inserting rows with columns like _id or ID also works fine (ditto)
findOne()/find() fails with both
Failure reason appears different between _id and ID (or similar), based on error message

I may need to create a different issue.

Changed description here to match the general problem with unquoted column names being used; to be fixed by #1355

This issue, PR that fixes (#1355) is for general problem: filed #1357 for follow-up work wrt _id.

@vkarpov15 Fixed almost all issues, but _id itself still problematic for findOne/find for some reason. So #1357 is for solving that. But at least Id and ID etc should now work.


when use python setup.py, I encounter the following problem. I don't know why

Configuring...
Traceback (most recent call last):
File "setup.py", line 21, in
subprocess.Popen(['cmake','../opensfm/src'], cwd='cmake_build').wait()
File "D:\Anaconda2\lib\subprocess.py", line 390, in init
errread, errwrite)
File "D:\Anaconda2\lib\subprocess.py", line 640, in _execute_child
startupinfo)
WindowsError: [Error 2]

Looks like it is having problems running cmake. Is cmake installed on your system?

Thanks a lot for your reply, I think i don't have the cmake installed. I will try it. Thanks a lot.


[WIP] chore: do not print console secret unless we have a license

Fixes #833

LGTM, but I don't understand comment included in the configmap-and-deployment.yaml file.

It is confusing. So we were not originally printing this resource. specially when it was specified that it was not being created. So now we are always printing because of the possibility of including a license. To avoid this we remove the printing always and put it behind a conditional.


description can only write in one line? and how to insert image?

hello, can you tell me how to edit my article in multi-line ? and how to insert a image in my article since including description of *.yaml file.

Hello @blackmatch,
what exactly do you mean with "article"? If your refer to the project files under exampleSite/data/projects and it's description parameter your can use the normal markdown syntax to include images:
![](/path/to/image.png)

Drop your image in the static folder and create a relative link to the file like above.

can you tell me how to edit my article in multi-line

Are you trying to use a multi-line string within the YAML files?
Otherwise, please give more information about your problem(s).

Reopen this issue if you still have questions.


Add com.snowplowanalytics.mobile/diagnostic_error/jsonschema/1-0-0

This is the specification for a new event generated when an internal error occurs in a mobile tracker.
It can be used as diagnostic of the tracker and it can be helpful to spot in advance anomalies in the tracker's behaviour.
The diagnostic_error events can be collected by the customer and they are activated enabling a diagnostic feature in the mobile tracker settings.
It's related to the issue https://github.com/snowplow/snowplow-android-tracker/issues/343
https://github.com/snowplow/snowplow-android-tracker/pull/344

@chuwy It's my first time pushing a schema, so, please, let me know anything missing and/or any doc reference useful for this PR.

Thanks @colmsnowplow , fixed!

Hey @AlexBenny! The schema looks good, but wondering if you considered application_error as a more generic one?

Closing in favor of https://github.com/snowplow/iglu-central/pull/1029


v1.0.0 Assert.equal(double, int literal) behave different from v0.13.3

This assert used works in v0.13.3:
  Assert.equal(1003200.0, 1003200);

but in v1.0.0:
/.dub/packages/fluent-asserts-1.0.0/fluent-asserts/source/fluentasserts/core/base.d(506): 1.0032e+06 should equal 1003200. 1.0032e+06 is not equal to 1003200.
Diff:
1[+.]0032[+e+]0[-0][+6]

 Expected:1003200
   Actual:1.0032e+06

need to change it to:
  Assert.equal(1003200.0, 1003200.0);

to make it pass in v1.0.0.
I'm wondering in such case, should the int 1003200 be cast to double (in the fluent-asserts library) can then compare?

Same problem with (int, bool) literal pair, different behavior from v0.13.3:
Assert.equal(0, false);

/.dub/packages/fluent-asserts-1.0.0/fluent-asserts/source/fluentasserts/core/base.d(506): 0 should equal false. 0 is not equal to false.
Diff:
[-false][+0]

 Expected:false
   Actual:0

can fluentasserts promote the bool to int internally?

Find another related one, and is more serious: this assert failed in v1.0.0, but works fine in v0.13.3.
  double dn = 43.5001 / 4.35;
  Assert.greaterThan(dn, 10.0);  // 10.00002298850574712643

/.dub/packages/fluent-asserts-1.0.0/fluent-asserts/source/fluentasserts/core/base.d(506): 10 should be greater than 10. 10 is less than or equal to 10.

 Expected:greater than 10
   Actual:10

Why in this case, the double should not be promoted to int, but somehow promoted (from the message, it's comparing the ints instead of doubles?)


Fix a bug in the normalizer where roots with no dependencies were dro…

…pped
As reported in the comments of #51 .

👍

(merge when green)

LGTM too. Thanks!

Happy to help.  This tool is great :)


fix typo

[Current coverage][cc-pull] is 100%

Merging [#757][cc-pull] into [v2.x][cc-base-branch] will not change coverage

@@               v2.x       #757   diff @@
==========================================
  Files             4          4
  Lines           414        414
  Methods          80         80
  Messages          0          0
  Branches         99         99
==========================================
  Hits            414        414
  Misses            0          0
  Partials          0          0

Powered by Codecov. Last updated by [d47d0f9...00bd388][cc-compare]
[cc-base-branch]: https://codecov.io/gh/koajs/koa/branch/v2.x?src=pr
[cc-compare]: https://codecov.io/gh/koajs/koa/compare/d47d0f9619901051aeec4bc30ce1037668e2e1d5...00bd3889e305c7697f8060eb787f258b3555314b
[cc-pull]: https://codecov.io/gh/koajs/koa/pull/757?src=pr

Appears to be a duplicate of #756 ?

sry,I'm not familiar with github

@qingmingsang You made two copies of the same pull request, close one of them.


chore(rpc): build rpc as OnComponentsInitializedHook

Build RPC component as OnComponentsInitializedHook. Part of task to expose EthApi top level, for user to configure, like for different networks.
Exposes extension component builders as Option<OnComponentsInitializedHook<N>>s. The hook takes a mutable reference to the extension component container as param. This is how extension component builders stage their output for installation in the node, as well as how they read state from previously initialised components that they may need to build themselves. This has two main benefits:

Since extension components are supposed to be optional, it's better that component inter-dependencies are not modelled with rigid type transitions, e.g. Engine<Node> is the only way to get to Rpc<Engine<Node>>, unlike for core components.
This flexibility allows us to implement the extension component builders in several PRs. For this PR, engine is just staged from the scope of DefaultNodeLauncher::launch_node.

closing, needs design


chore(prettier): Add github action to verify prettier

FYI, this is going to fail, since some fixes are in the other PRs, but I wanted to get this up before I forgot.

@wrycu I don't know your thoughts here, but the fact that I missed a few files probably means we want a CI action verifying prettier.


Add layers support

SharpLayer etc

fixed by https://github.com/EkkoG/EGChainable/commit/666acd5e28a988ae49f068dadd9663cec661aac4


Bubble up control plane endpoint from hcp into hc

For consumers to watch if they see fit.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Ref https://issues.redhat.com/browse/HOSTEDCP-559
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm


Alter mkdir wrapper use mkdir_p to create intermediate directories

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[X] Have you written new tests for your changes? Here's an example.

Proposal - https://github.com/Homebrew/brew-evolution/pull/13
Relevant PR discussion https://github.com/Homebrew/homebrew-core/pull/4976#issuecomment-248258491 - https://github.com/Homebrew/brew/pull/1043

@MikeMcQuaid - Made a PR plus document as per the conversation we had earlier here.
Please note that I get a failure when running brew tests on latest master without my change applied (i.e. issue completely unrelated to this change).
No (additional) tests fail due to this change.

Failure I see FYI.
Looks like a path issue or perhaps something missing from Gemfile. Either way, seems OK on CI :-).

Why wouldn't you just use mkpath, out of curiosity?

@DomT4 is there a mkpath do?

@MikeMcQuaid No, like mkdir_p it doesn't natively take a block, but unlike mkdir we haven't extended on top of that.
We've been loosely using mkpath instead of mkdir_p for a while, so my temptation would be to extend mkpath rather than mkdir behaving too differently than it does natively, but it's not a blocking objection.
We tend to use both mkpath and mkdir_p like this:
path = buildpath/"test/test2"
path.mkpath
cd path do
  system "blah"
  system "make", "blah"
end

I'd be in favour of perhaps only having the mkdir_p behaviour when in the block form.

@MikeMcQuaid I think having such inconsistency between block/non-block form would be a violation of the principle of least surprise.
I would strongly suggest it being the same, in whichever form we land upon.

To be fair, almost all of Homebrew's magic violates the principle of least surprise, hence why we see a bunch of PRs with people using things like cd path followed by make followed by cd buildpath etc even though our cd can take a block to handle a lot of that automatically.
That principle is why I wondered if it was less invasive to modify mkpath to take a block here rather than the vanilla mkdir to create multiple layers. That argument can swing either way though.

@DomT4 I understand what you're saying, but don't think that it is a useful parallel to this particular situation.
Having substantially different behaviour on a single method call when with/without a block is confusing.
I think that's distinct in meaningful way to having consistently behaving (but enhanced) functions.

@DomT4 Cool, agreed, feel free to make mkdir do mkdir_p (and it could call it instead of old_mkdir as well).

Having substantially different behaviour on a single method call when with/without a block is confusing.

Sure, that's a fair point 👍. Interested in your comments on the gap between surprising behaviour on tweaking it like this and tweaking mkpath to take a block instead though, but unless any other maintainers care I'm happy enough to leave it.

@DomT4 @MikeMcQuaid I think either solution is fine, as long as it behaves consistently.
The mkdir one appeals to me because it's more UNIXy than mkpath, and appears to have no side-effects.
But, that's a minor distinction and I would be equally happy with adding block support to mkdir_p and/or mkpath.
Happy to contribute it, either way :-).

Nobody else has rushed to agree with my thoughts, so I'm happy here whenever @MikeMcQuaid is.

Thanks for your contribution to Homebrew!  You rock!


Offer major version tag/branch alias

I was wrong. Well sort of. npm-bump rarely updates, but all my other actions update all the time. So I should just bite the bullet and set up major version alias automation and support that here too.

This is offered.


Npm start doesn't work the first time

Due to the current setup, npm start doesn't work at the very first try after cloning the repo.
Workaround
After cloning run npm install
Then run npm run build
Afterwards, npm start will work again

It is not an issue, you have to use npm install to get the node_modules. Afterwards you can run npm start

Unfortunately, some people may encounter an issue when trying to run the project after installing the node_modules. The attached screenshots depict the error.
The steps to reproduce the bug (from a fresh install):

git clone https://github.com/zero-to-mastery/WebBlocks.git
cd WebBlocks
npm install
npm start

We are looking into the issue, but the workaround highlighted above should solve the problem.

Due to the current setup, npm start doesn't work on the very first try after cloning the repo.
Workaround
After cloning run npm install Then run npm run build Afterwards, npm start will work again

Hi , It worked!
Thank you so much.
one thing I want to ask you is that whenever we stop the dev server then if we made changes then again we have to do npm build and then npm start.

The solution provided by @r4pt0s works for me.

I am again encountering the same problem, even after I followed the workaround provided by @r4pt0s. It only works when cloning our repo for the first time and everything is fresh and later if we change something then it again shows the above-described error.

i think the problem is related to webblocks.css

this something come after doing npm run build

i think the error is coming due 'stencil.config.ts', it throws an error: Cannot find name 'process'.

Thanks for sharing your thoughts. We are still investigating why this is happening now.
I just can give you a typical answer for now, it worked for me after setting this project up 😅.
I will look into it today and hopefully merge a fix today.

Thanks for sharing your thoughts. We are still investigating why this is happening now.
I just can give you a typical answer for now, it worked for me after setting this project up 😅.
I will look into it today and hopefully merge a fix today.

i installed @types/node and tried to import but it says :

this is changes that I made in stencil.config.ts:

when I tried in node, process.env.PWD it returns undefined. it must be a path issue i think.

From me and @IM-Deane , i think it is a path issue in stencil.config.ts.

Should be fixed now @Chetan-khachane
Please pull the latest changes from main


Upgrade to MRML 3.1.0

Great developer instructions! I was able to make this change with basically no knowledge of rust. Pulling in the fix for this issue, which allows using css files with mj-include.

@mgd020 done!


Test batched causallm inference

As per title.

Looks good, we should bump to Transformers 4.39 for the CI OOM


[TritonCTS] Review: Updated function to find clock nets

[PR for Review!!!]
This changes how clocks are loaded into TritonCTS.
First, we create a set and use dbSta's findClkNets function. After that, we insert all clocks from the user inputs (if there are any and if they aren't already in the set.)
After that, we create the Clock objects inside TritonCTS for the nets that have more than 2 sinks. This includes input and inout pins.
Another thing that was added to this PR was a unit test that test the new functionality. This was based on Cherry's dbSta find_clock unit test. Also, the reports for TritonCTS changed a little, to report cases where pins do not have shapes, the number of clocks, and others...
[Code in this PR was made with the help/additions of Matt Liberty, James Cherry, and Tom Spyrou]

The first commit part is just from when me and Mateus created the fork to code for PartClusManager. The current PR has no code for it, but I'll see if I can clear up the commit history to remove it. Sorry about that.
For the part on changing the OpenRoad call, I can change it to only use it when initializing the DbWrapper class.
parseClockNames is just a simple function that transforms an std::string to an std::vectorstd::string. It doesn't define any new names or creates any TritonCTS objects (Clock, Net...).

Closing this and, after I finish the fixes, I'll do another PR.

@MarceloDanigno A better work flow is to keep the original pull request open. When you push changes to your fork, the commits will automatically be added to the pull request. You can re-request a review whenever it's ready.
If the commit history is messy and you only want the final set of changes, you can select "squash and merge" as the merge option. This will squash all changes into a single commit before merging with OpenROAD (so for example, if you revert several changes, only the final state is captured).
If you need to ensure the history is clean and spread over multiple commits, that might be a reason to close and open a different pull request. But that should be a rare circumstance.

@rovinski Thanks for the information Austin! I wasn't aware of these options, so this is definitely useful. I'll keep it in mind for future PR's.


How to re-enable clipboard?

After using this program, it disabled windows clipboard history and sycning across devices, how do I re-enable it?

Give the successor to Bloatynosy, Winpilot, a shot.


zeebe-operate-helm and zeebe-tasklist-helm doesnot allow toleration and podSelector values in values.yaml

Hi,
I wanted to deploy all pods related to Zeebe in a different node pools in AKS. I figured it out that I can run all the pods in one node pool except the tasklist and operate pods.
Is there anyway to set these values for Tasklist and operate pods?
I have used below values.yaml file for installing the zeebe-full-helm chart.
helm upgrade --namespace zeebe --install --wait --timeout 10m0s zb zeebe/zeebe-full-helm -f values.yaml
`
global:
zeebe: "{{ .Release.Name }}-zeebe"
zeebe-cluster-helm:
elasticsearch:
tolerations:
- key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"
nodeSelector:
zeebe: isolatenodepool
gateway:
tolerations:
- key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"
nodeSelector:
zeebe: isolatenodepool
tolerations:

key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"
nodeSelector:
zeebe: isolatenodepool

cloudevents:
enabled: false
tasklist:
enabled: true
zeeqs:
enabled: false
ingress-nginx:
controller:
tolerations:
- key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"
nodeSelector:
zeebe: isolatenodepool
service:
ports:
http: 80
https: 443
annotations:
service.beta.kubernetes.io/azure-load-balancer-internal: "true"
zeebe-tasklist-helm:
ingress:
enabled: true
annotations:
ingress.kubernetes.io/ingress.class: nginx
ingress.kubernetes.io/rewrite-target: "/"
nginx.ingress.kubernetes.io/ssl-redirect: "false"
path: /
host: tasklist.zeebe.weud
tls:
enabled: false
secretName:
nodeSelector:
zeebe: isolatenodepool
tolerations:

key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"

zeebe-operate-helm:
ingress:
enabled: true
annotations:
ingress.kubernetes.io/ingress.class: nginx
ingress.kubernetes.io/rewrite-target: "/"
nginx.ingress.kubernetes.io/ssl-redirect: "false"
path: /
host: operate.zeebe.weud
tls:
enabled: false
secretName:
nodeSelector:
zeebe: isolatenodepool
tolerations:

key: "zeebe"
operator: "Equal"
value: "gpu"
effect: "NoSchedule"
`

result in AKS cluster:

Done via #257


Fix Amelia location in DS2

Fixes Amelias location in DS2, this should stop players wondering about, and getting lost. (Reported at least 2x daily)
Added Slayer Req for Fally Hard.


feat: Twerk crowdin.yml

twerked the crowdin.yml with appropriate changes

Thanks for you PR @Biki-das!
Can you please explain what you mean by "appropriate changes"?

Thanks for you PR @Biki-das! Can you please explain what you mean by "appropriate changes"?

by appropriate i meant the typical format of crowdin.yml .

@molant  why my pr is semantically incorrect? not able to figure out

The title needs to be something like fix: this feature: another thing chore: bla
The Crowdin file is working as we are using the defaults and it validates so I'm not sure this change is necessary.
@erickzhao what do you think?

The title needs to be something like fix: this feature: another thing chore: bla
The Crowdin file is working as we are using the defaults and it validates so I'm not sure this change is necessary. @erickzhao what do you think?

ok , the crowdin have this as their configuration file - https://support.crowdin.com/configuration-file/#writing-a-simple-configuration-file,  this might help.


Use builder based interface to generate Krnl loops

This PR updates code generation for various operators to use the builder based interfaces for generation of Krnl loops.
Signed-off-by: Ettore Tiotto<EMAIL_ADDRESS>

Jenkins Linux amd64 Build #4530 [push] Use builder based interf... started at 05:01

Jenkins Linux ppc64le Build #3667 [push] Use builder based interf... started at 06:04

Jenkins Linux s390x Build #4547 [push] Use builder based interf... started at 06:01

Jenkins Linux s390x Build #4547 [push] Use builder based interf... failed after 50 min

Jenkins Linux ppc64le Build #3667 [push] Use builder based interf... passed after 57 min

Jenkins Linux amd64 Build #4530 [push] Use builder based interf... passed after 1 hr 5 min


NJsonSchema v11 [WIP]

Closes https://github.com/RicoSuter/NJsonSchema/issues/1045

Is there something still needed for this PR? Do you need help with something?


Triggers Jasmine deprecation

Since Jasmine 3.7.0, using setup-polly-jest triggers the following deprecation:
DEPRECATION: An asynchronous before/it/after function was defined with the async keyword but also took a done callback. This is not supported and will stop working in the future. Either remove the done callback (recommended) or remove the async keyword.

It's probably because of:
https://github.com/gribnoysup/setup-polly-jest/blob/13db62ea1503b4ff61215f5a3c4d1bbd8de0d03d/src/jasmine.js#L96
Since the before and after proxies have the done callback and are also asynchronous.

Hey @alanpoulain! Thanks for the report and sorry that it took awhile to get to it, the issue should be fixed in the latest release

Nice, thank you for your work!


Large blank area on mobile - forceHeight:false, Mobile, Relative Mode

Hello,
I have read issue 58 and 61 and I have a version of the bug in issue 61.
When viewed on an iPad or iPhone there is a large blank area below the content even though forceHeight is set to false.
The large blank area appears to be the HTML tag from inspecting it in safari.
I have double checked that Twitter Bootstrap is not messing up the HTML tag.
Here is my code:
https://github.com/wkjiv/Mastline-Front-End
This is over my head, any help would be greatly appreciated.

Hi @Prinzhorn
I think I'm getting an instance of this issue. I have set forceHeight: true. Desktop is fine and some android devices are fine but iOS seems to add the extra space at the foot of the page.
I tried to hack it with height: auto !important; for screens <1023 but this isn't behaving either.
Site is deployed at https://www.techwillsaveus.com/microbit/ (but still to be finished/fixed) Any help would be MUCH appreciated.
Loving skrollr, it has really helped me out. Thank you :)


add packaging test suite

In order to avoid changes that could prevent people from installing a working copy of molecule we need to create a test suite that is packaging molecule and test that it can successfully be install in all know use-cases.
Testing matrix should contain a combinations covering at least:

Distros: CentOS-7, Fedoa-28, Fedora-rawhide, Ubuntu-latest, MacOS-latest
Method: wheel, from source, editable (pip -e)
Type (non venv): as root, as normal user, as user package (pip install --user)
Python: system python, pyenv installed
Virtualenvs: isolated and with use-system-packages (maybe even considering case where a version of molecule is already installed on system, and we have a virtualenv with system site packages and we want to install the current version of molecule in it) -- this should work.

For start, it should be enough to test that

installation suceeds
pip check does not report conflicts
user is not required to perform extra installation steps (like installing new repositories  or packages that are not shipped by supported distros) -- shorly installing molecule should not require user to bring their system in an unsuported status.

python -m molecule --version does succeed

Because there are lots of combinations to test I am inclined to believe that it may prove useful to use pytest/xdist/html in order to run this operations.
I will be trying to make POC implementing such testing because I think this could be useful not only for molecule but for any python tool that gets shipped via pypa, python packaging being known to be a permanent source of confusions among users.

I empathise with the sentiment here I don't think we have the volunteer capacity to implement and maintain what you're proposing. But I will wait to see your POC (thank you for offering to take this on) and see from there. As far as I can tell, this is why we do pre-releases and I encourage other co-maintainers to do manual testing where possible. Packaging issues are a Python ecosystem structural issue and we can't hope to safeguard against all the cases within this project.

@decentral1se Yep, keep this open and I will raise a POC change about in the next weeks. I plan to do this on a smaller project as this issue has nothing to do with molecule, it could apply to any python package.
PS. I am watching the packaging thread on https://discuss.python.org/c/packaging and this is stresfull enough even with a passive role.

Oh awesome, I will try to observe the discussions there too.

This was implemented long time ago but I forgot to close it, we have tox packaging environment for that.


Sorting the guest devices child devices by name

Improvement
This PR adds the order statement to GuestDevice relationship  with :child_devices. The main goal is to provide a better readability for the user. For example in the network devices page:
Actual result:

Expected result:

@miq-bot assign @agrare

Once the relationship between Network Devices and its ports is changing (by this PR: https://github.com/ManageIQ/manageiq/pull/17268) this approach will not work anymore, so I'm closing this PR


[ChatLLaMA] Training process halted

Hi, it seemed like the training process was fine unitl it hit iteration 71? ran the line: python artifacts/main.py artifacts/config/config.yaml --type ALL

Hi @MuffinC, thanks for reaching out, we are sorry for the inconvenience.
This problem should have been fixed in the previous release and we haven't encountered it since.
Did you use the last available version of the pip or of the source code?

Hi Pier, yes i am using the latest version of pip and i've just pulled from the repo while changing my python version to 10. Still encountering the error


SAK-29369 - Unpublished, softly deleted sites dont appear in the list of Softly Deleted sites in Worksite setup

Unpublished, softly deleted sites dont appear in the list of Softly Deleted sites in Worksite setup

+1, good catch @mcarropellicer

I verified this bug on a local (10.3) instance, but when I applied the PR changes, I received the following stack trace and the list of softly deleted sites was still empty in the UI:
2015-06-23 17:29:23,504  WARN http-nio-8447-exec-2 org.sakaiproject.db.impl.BasicSqlService - Sql.dbRead: sql: select SAKAI_SITE.SITE_ID from SAKAI_SITE,SAKAI_SITE_USER where SAKAI_SITE.SITE_ID = SAKAI_SITE_USER.SITE_ID and SAKAI_SITE_USER.USER_ID = ? and SAKAI_SITE.IS_USER = '0' and SAKAI_SITE.IS_SPECIAL = '0' and SAKAI_SITE.IS_SOFTLY_DELETED = '1' and SAKAI_SITE_USER.PERMISSION <= -1 order by SAKAI_SITE.TITLE ASC,SAKAI_SITE.TITLE limit 1 offset 0 java.sql.SQLException: No value specified for parameter 1         at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:1074)         at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:988)         at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:974)         at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:919)
I'll paste the entire stack trace on the JIRA

Thanks @bjones86

Verified the fix does work on a trunk instance; seems KNL-1358 is required.


Implement Close() for rethinkdb

Describe the feature
Implement Close() for rethinkdb
Release Note

RELEASE NOTE:

/assign


Diagram Editor doesn't load when workspace path contain space in Windows 11

Description
Diagram Editor doesn't load when workspace path contain space in windows
Steps to Reproduce

Create a workspace that has a space in the path (eg: C:\Users\WSO2\Sample Directory\8.2.0\workspace)
Create Hello World Sample
Try to load HelloWorld.xml
You should see the following error

Affected Component
IntegrationStudio
Version
8.2.0
Environment Details (with versions)
Windows 11
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

fixed with https://github.com/wso2/integration-studio/pull/1216


Desync on MP with slow connection

Hi.
Bit hard to explain/debug, but sometimes people with a somewhat slower connection to my server are getting desyncs. From their desync reports i
could figure out that the rng generator was giving different results. (See desync-report-2019-03-13_12-49-45.zip)
It seems to be in the firstMazeRowMangrove variable where they differ sometimes.
This was tested on a 0.17 install, so this might be the issue (since the PR for 0.17 isn't accepted yet).
We also installed some other mods, but this is the only difference i found in the script.dat that would/could generate these issues.
I've been trying to fix / reproduce it myself, but cannot seem to do that with the lack of knowledge i have.
Anyhow in this case it seems that the 7th iteration of the chunk was giving issues.
The server is a linux headless, the client on windows (all cases), but not all windows clients seem to get this result. Most/
At least it comes from the Cmwc.deriveNew function it seems.
And as far as i can see, only the 7th chunk was giving issues. There are no other mods that hook into the chunk generation, we have stdlib installed, but that doesn't seem to influence it.
And its also that its not immidiatly desyncing, but after a second or 10 - 15
If you need more information, please let me know.
P.s. this is with the 0.17 PR included.

Would you be able to try running a game with the maze block size at the default one chunk (32x32)?
I have a theory that chunks are being generated in a different order for different players. That shouldn't affect the default block size, because RibbonMaze was designed to make sure that the order of chunk generation doesn't matter -- in fact that's the job of Cmwc.deriveNew(), a new deterministic pseudorandom number generator is derived for each chunk in a strict order, regardless of when the chunk is generated. This was mainly to make sure mapgen seeds have reliable results, but there was an element of paranoia around desyncs of exactly the nature you've raised.
Except that the option to make the block size bigger was added into the mod later, and that relaxed some of the strict logic. Cmwc.deriveNew() is called per "block" not per "chunk" for mangroves and resources -- so there are four chunks in each of the blocks with the settings you've got sharing a single RNG. Therefore, if factorio doesn't emit chunk generation events in the same order on each machine, then the resources and mangroves will differ, and boom -- desync.
The nightmare is now real.
If this is the case, and we can prove it, then it is presumably a bug in 0.17; it's hard for mods to be deterministic if the events generated by the core are not. If it's really the responsibility of my mod to be immune to different player's machines emitting chunk generation events in different orders, then it's going to impact memory usage and save size, because I'll need to derive more random number generators to defend against it. But one step at a time.

Been away for a little while, but even with the 32 chunk size it's giving desyncs.
And tested with only the maze runner mod, so it might seem you are right, that because of the lagg/delay it generates the chunk differently.
And it seems thats only with slower connections.
The desnyc happened as soon as the one on the slower connection generated a new chunk.
Hope you can figure it out and that its not a 0.17 core bug
desync-report-2019-03-22_22-06-54.zip

Very much appreciating your help figuring this out!
I was anticipating no desyncs on block size 32, but I've figured out that the fish generation is using the wrong RNG, which could result in different chunk generation if the chunks are generated in a different order on two machines regardless of block size. I've released a fix in release 0.7.1.
Would you be able to test again at block size 32? I think if the problem is now solved at block size 32, that really tells us something very interesting.

I've tried it with 0.7.1
Just update the mod with the old save game.
Same map, 32 block size and only ribbon maze as mod installed.
I didn't have problems, the tester who had desyncs before immidiatly desyncs (within 18 ticks of joining)
So it didn't seem to help (enough)
5853.646 Info ServerMultiplayerManager.cpp:837: updateTick(10478) received stateChanged peerID(3) oldState(TryingToCatchUp) newState(WaitingForCommandToStartSendingTickClosures)
5853.646 Info GameActionHandler.cpp:4019: UpdateTick (10478) processed PlayerJoinGame peerID(3) playerIndex(1) mode(connect)
5853.748 Info ServerMultiplayerManager.cpp:837: updateTick(10484) received stateChanged peerID(3) oldState(WaitingForCommandToStartSendingTickClosures) newState(InGame)
2019-03-26 19:26:22 [JOIN] XXXXX joined the game
5870.931 Info ServerMultiplayerManager.cpp:837: updateTick(11515) received stateChanged peerID(3) oldState(InGame) newState(DesyncedWaitingForMap)
5870.931 Info ServerMultiplayerManager.cpp:917: updateTick(11515) received playerDesynced peer (3)

desync-report-2019-03-26_20-26-51.zip

Did the tester have success connecting to your game with Ribbon Maze in 0.16?
I can't think of any reason remaining, why the firstMazeRowMangroveRng would differ between two systems, unless maybe the chunk is being generated twice on one machine and once on the other. And if that's happening, I think that's a bug in the core.

He was able to connect without any problem on 0.16, never had a desync.
And in the last one the 20th version of the {Q={...}} was giving errors. Seems different every time. However the first few was in within the first 10 rng numbers, now its in the 20's...
And with this one its the second to last thats different. I would expect more to be different, but it seems it's only one. Might be a core bug though :(
At least my knowledge stops with just minor debugging... where it's actually comming from... no idea. I hope you can figure it out. Maybe the Factorio devs can help you out.

Have been doing a few extra tests from my mac with connection and slowness:
Have been trying to connect through multiple vpn's throughout the world. But none gave me desyncs. So seems to be a windows only issues.
If you need some additional information please let me know.

This is believed to be fixed in 0.7.4 as I was able to reproduce the issue with a seed of "1" generating different maps on Windows and Linux. Problem was with Lua having different arithmetic behaviour on those platforms. This has been worked around.
Closing unless desync reports are received for new multiplayer maps.


打开文件夹直接闪退 Open folders directly flicker

VSCode Version: Code 1.19.0 (816be6780ca8bd0ab80314e11478c48c70d09383, 2017-12-14T12:06:43.492Z)
OS Version: Windows_NT x64 10.0.16299
Extensions:

Extension
Author (truncated)
Version

vscode-caniuse
aka
0.5.3

vscode-eclipse-keybindings
alp
0.9.6

markdown-preview-github-styles
bie
0.1.0

translation
dus
0.0.4

code-runner
for
0.8.5

live-html-previewer
hdg
0.3.0

beautify
Hoo
1.1.1

python
ms-
0.9.1

vscode-jsp
pth
0.0.2

vscode-java-debug
vsc
0.4.0

vscode-java-pack
vsc
0.2.0

(1 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

@parker-wang can you share more details what happens?


Cannot detect mixed languages, only shows the language of last part.

import pyidaungsu as pds
language= pds.detect ("ထမင်းစားပြီးပြီလား၊ ထမင္းစားၿပီးၿပီလား၊ ၢ်သိၣ်လိတၢ်ဖးလံာ် ကွဲးလံာ်အိၣ်လၢ မ့ရ့ၣ်အစုပူၤလီၤ.")
print (language)

Shan

In this case, only Shan is shown. If MultiLang or MixedLang tag is shown, it will be better.
pds.detect ("Hi its me")

spanish
pds.detect ("Hi its me.")
eng

For the second part, only period determines if the lang is Spanish or English.

@thawzi the model is experimental. it needs some more work to be able to use this reliably. thanks for addressing the issue.


New HTML Media Capture .JSON  file

Support data from my tests and this O'Reilly book
HTML Media Capture is 56 on the features suggestion list with 12 votes https://github.com/Fyrd/caniuse/issues/1115

Thanks!


build: Update common-jvm dep to 0.92.0

The update deprecates some Spanner function calls, so this change also addresses those.

This change is


Can't use finger to draw. What is wrong in my avatar?

"Your avatar rig's left and/or right index finger's distal bone is unmapped!"

Bit of a late response lmao. If you still have issues, we can help you at discord.vrlabs.dev
Basically your avatar only has two finger bones in your index finger, not three. you need to add the third one, or to edit the script to take the second one.


Non-group entry methods don't respect envelope group dependence

Original issue: https://charm.cs.illinois.edu/redmine/issues/1565

No body.

Original date: 2017-12-29 17:46:38

Per discussion with Eric B, assigning to Nitin, based on recent work around CkEntryOptions handling

Original date: 2018-01-17 23:33:09

Fix: ~~https://charm.cs.illinois.edu/gerrit/#/c/3528/~~ https://github.com/UIUC-PPL/charm/commit/d24d191d92a48c9cab38b84e7565e794f588949d


STORM-2303 [storm-opentsdb] Fix list invariant issue for JDK 7

use bounded wildcard type to fix invariant issue

I'll create pull request for 1.x separately to trigger Travis CI build for 1.x branch.

+1


Remove traverse dependency

This dependency pulls in over 50 transitive dependencies, totaling 16 MB from npm. Let's just reimplement the relevant logic ourselves.
I've tested this down to Node 0.10, the oldest version that runs on my machine.

Can you find a smaller alternative? I used it to simplify I'm not particularly interested in inlining.

Just opened #8 as an alternative to this.

Great!

Whoops meant to merge the other one...


Visual refresh for experimental Image Viewer

Provide a fresh coat of paint for the ImageViewer sample :)
Use Kotlin color scheme, use gradient backgrounds
Use animations for transitions between screens and images
New square-based layout for pictures (List-view hidden behind a code toggle)
Provide mock human readable names for images in repository
Converted state data classes into Compose-aware state-holders
Use delegate syntax where possible
Commonize state management (image gallery state is no longer created in platform-specific code)
Use SharedFlow to provide "external events" (like keyboard shortcuts) to the view model
Move towards unidirectional data flow (avoid passing Mutable States down the hierarchy where possible, provide callback-mechanisms where useful)
Split composables into their logical components (rather than having combined e.g. TopContent elements)
Fix side effects running during composition (they should only run in a LaunchedEffect

cc @igordmn @dima-avdeev-jb

Once #2740 is available we can also get rid of the Experimental annotations for components that use the Material3 stuff under the hood.


Adds a tilt-to-wake icon.

This PR provides simple tilt-to-wake icons used by the asteroid-settings PR(https://github.com/AsteroidOS/asteroid-settings/pull/23).
Please let me know what you think of these icons. Suggestions are highly appreciated :)

I like the concept and the overall icon idea. However, I think the watch strap looks a bit unpolished at the bottom.

Thanks :)
This is just the ios-watch and ios-sync icons merged and rotated.
I agree with the look of the strap. I can try to reduce the size of the strap all together and increasing the size of the watch. That might help make the icon better looking and less unpolished :)

ok, I have been messing around a bit, what do you think of these versions:

I think they look a lot better!

Great! I will adjust the PR. Which one do you like more, the bottom one?

@MagneFire I like the top one better since there is less going on. simpler is better imho

Alright, I agree with that :)
I have updated the PR.
Thanks for your feedback :smile:

I agree with everything that has been said above :)
They look great like this. Could you just make them go through a svg minifier ?

Great! :+1:
I have minified the svgs

Mh are you sure? I still see comments, inkscape tags and \n

Snap, I just used an online svg minimizer ;)
Ill look into the inkscape one.

Alright, how about know?

Mhh, it still looks like it could be made shorter but don't worry about it, that will do ;)
This is certainly not what will take the most disk space or processing power on the watch anyway ahah.


Does token refresh work when the access token has expired?

Hi, I begin to use devise-api and I found it a great gem, thanks.
However, I do not understand how the refresh token works.
My configuration is like suggested, but I do not see any reference for refresh_token expiration: api.refresh_token.expires_in seems be the than api.access_token.expires_in to me.
# ==> Configuration for :api
  config.api.configure do |api|

      # Access Token
      # access tokens. Access tokens are the credentials representing the authorization given to an application
      # api.access_token.expires_in = 2.hours
      api.access_token.expires_in = 5.seconds
      api.access_token.expires_in_infinite = ->(_resource_owner) { false }
      api.access_token.generator = ->(_resource_owner) { Devise.friendly_token(60) }

      # Refresh Token
      # Unlike access tokens, refresh tokens have a longer lifespan
      api.refresh_token.enabled = true
      api.refresh_token.expires_in = 1.week
      api.refresh_token.generator = ->(_resource_owner) { Devise.friendly_token(60) }
      api.refresh_token.expires_in_infinite = ->(_resource_owner) { false }

      # Authorization
      api.authorization.key = 'Authorization'
      api.authorization.scheme = 'Bearer'
      api.authorization.location = :both # :header or :params or :both
      api.authorization.params_key = 'access_token'

      # Base classes
      api.base_token_model = 'Devise::Api::Token'
      api.base_controller = '::DeviseController'

      # After successful callbacks
      api.after_successful_sign_in = ->(_resource_owner, _token, _request) { }
      api.after_successful_sign_up = ->(_resource_owner, _token, _request) { }
      api.after_successful_refresh = ->(_resource_owner, _token, _request) { }
      api.after_successful_revoke = ->(_resource_owner, _token, _request) { }

      # Before callbacks
      api.before_sign_in = ->(_params, _request, _resource_class) { }
      api.before_sign_up = ->(_params, _request, _resource_class) { }
      api.before_refresh = ->(_params, _request, _resource_class) { }
      # api.before_revoke = ->(_params, _request, _resource_class) { }
    end

Thus, to test it, I decrese the access_token expiration to 5 seconds, and it got the record at database in the table devise_api_tokens
// users/tokens/sign_in
{
    "refresh_token": "arvMhr-aTiBPqv53N1B3n21B_KsJbnXCdZTDM3E2HxWoVoHihpwy-kMRyzNp",
    "expires_in": 5,
    "token_type": "Bearer",
    "resource_owner": {
        "id": 1,
        "email"<EMAIL_ADDRESS>        "created_at": "2023-03-29T07:27:40.783Z",
        "updated_at": "2023-03-29T07:28:31.340Z"
    },
    "access_token": "q-H4PeyB7PnVAusBsUbs2c1_y8kPb9xPdzUWxzGkTAZ9wcxrEtfTAzB8B9QH"
}

However, when I go to refresh this token , I got "token has expired":
curl --location --request POST 'http://localhost:3000/api/v1/users/tokens/refresh' \
--header 'Authorization: Bearer arvMhr-aTiBPqv53N1B3n21B_KsJbnXCdZTDM3E2HxWoVoHihpwy-kMRyzNp'

// users/tokens/refresh
{
    "error": "expired_token",
    "error_description": [
        "Token has expired"
    ]
}

Finnaly, It seems to be get the same reference of token expiration than access token. But on your code I found this reference :

And now , I am lost. Could someone help me, please?

Hi @jessefilho , you are right. You can fork the repository and then you can send a PR for closing this issue

ty.
work in progress @nejdetkadir


Update README according to https://github.com/nf-core/tools/issues/2186

PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint)
- [ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
- [ ] Usage Documentation in docs/usage.md is updated.
- [ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[x] README.md is updated (including new tool citations and authors/contributors).

@nf-core-bot fix linting

Are we still waiting on other things being added for this to be merged @grst @FriederikeHanssen ?

Yep what will be removed needs to be added to the website somewhere: https://nfcore.slack.com/archives/C01QPMKBYNR/p1678782050033109
I wanted to do it last week, but the AWS thing hold me up. Won't have time this week, sorry.

Ok, this may have to wait until the next release then which shouldn't be too long after the Hackathon. Want to get a release out in the next couple of days.


Stuck at apple logo

Hi and thanks for everything.
I have done exactly as you say but the installer just stuck at apple logo.
Thank you for this repo very much

I was able to change the plist file so it show the error
This is what I got

Thanks

Hi 👋
So that I have a better understanding of what's going on, I have a few questions:

Have you tried resetting NVRAM?
Is there compiled aml files in the ACPI folder?
Have all of the expected kexts downloaded correctly (occasionally GitHub's API can bug out and not download the couple that are fetched from there)
Did you enable the Intel WiFi option or leave it disabled?

Thanks

Hi, Thanks for answering.

Yes.
Yes
Yes, double-checked.
I had, Yes.
Catalina latest.

i have the same exact issue. Try to solve by simply googling the errors to no avail.

Yes.
Yes
Yes
Yes on wifi
Catalina 10.15.6, latest
I will try to upload new EFI without the WIFI fix to see if that will help.
Thanks!

Just tried without the WIFI fix. The same issue, no dice. Thanks for your help!

Also Alex, if you figure out how to get Scull Canyon to work with Big Sur, please let me know. As far as I know, Apple changed a lot of thing in BS, but it would be awesome if you can figure it out. Thanks for all of your hard work!

This issue generally tends to arise when the correct kexts are not present.
The kexts this machine requires are:

Lilu
VirtualSMC
WhateverGreen
AppleALC
CPUFriend
NVMeFix
IntelMausiEthernet
USBInjectAll
IntelBluetoothFirmware
(optional) itlwm

I just created a new USB from the latest master and was able to boot both the Catalina installer and my installation successfully, no problems

Thanks Alex. I will look and make sure i have the right kext present. Yes, I am using your script to download them. Currently using xcode 11.6 and CLT 12 beta 3. I am running Big Sur on my Mac Pro now.
Thanks for your efforts. I will do this in the next couple hours and report back.
BTW: any plans on getting Big Sur going on the Skull Canyon?

Looks like I am missing:

IntelBluetoothFirmware
(optional) itlwm

Have HXCI in addition. Could that cause the problem? I too recreated the USB just last night and that's what i got from the script.
Thanks!

IntelBluetoothFirmware missing would cause it to fail to boot.
itlwm is optional depending on whether you said yes or no, the script will toggle its enabled state within the plist.
I know that kext is downloaded from GitHub, maybe the API timed out somewhere...
I'll have a look and see if I can make that more reliable/retry if failed
RE Big Sur Beta: Last time I tried that (around 10 days ago with the latest OpenCore at the time) it failed to boot the new KernelCollection. Forcing use of prelinkedkernel did get past that stage with a USB installer but still wouldn't boot, kept getting messages about launchd getting killed in the log. If it's an issue of kexts not being compatible yet, there's not much I can do until the developers release updated versions.

Thanks for the quick response!! I will try to create the USB again now and if the BT kext is still missing, i will add it manually.
Thanks for the Big Sur update. Keep us posted, it's hard to go back to Catalina after you get used to the new UI!
I appreciate your work!

So, the script is failing to download the BTF kext which would explain it missing. Also, it's missing in the Dortania kext library, I had to download it straight from here https://github.com/OpenIntelWireless/IntelBluetoothFirmware/releases/tag/1.1.1

Is that the right one? Because the install is still stuck on Apple logo :(.
I uploaded my final EFI here https://drive.google.com/drive/folders/1RK6O2k4wveUF8TxUgvMY2oulGm7G1Vwo?usp=sharing
Any chance you can take a look and see if there are any other issues? Last week, I ran the config.plist by the Sanity Checker linked on Dortania and it came with several "issues", but I figured it was due to this build being specific to this NUC.
Thanks. Please let me know.

Thanks for the log
I know what caused the problem, I couldn't assume that jq would be installed to parse the download URL. It should now be fixed in 8a7cbb7
If you run the script again it should now work. It wasn't booting because itlwm.kext failed to download, and it was set as Enabled in your config.plist.

 I will try that as soon as Catalina works :). Does Thunderbolt and Ethernet work on BS?

Ethernet works
Thunderbolt sort of works...devices connected on boot should be functional, but hotplug/sleep breaks Thunderbolt completely
I don't have any TB3 devices to test with so I haven't really tried to get that working yet

I have T3 and T2 devices (with T3 to T2 adapter). I will try and report back. I will be using the NUC headless to control T2 enclosure for media serving (actually as slave back up server) and will disable sleep, so it should work - hopefully :).

So, successfully installed Catalina. Wi-Fi works, too. Odd thing. after install, the internal SSD's EFI folder is empty and therefore the NUC doesn't see it as bootable drive. I mounted the install USB EFI and moved it to the internal SSD's EFI. That way the NUC boots to the OC options menu and automatically loads Catalina. I am new to all of this, but would have assumed that the EFI would automatically migrate to the target internal drive during install. But I may be doing something wrong. Regardless, Catalina works great otherwise, so success :). Thank you!!!

Yeah, you have to manually copy it over.
I don't think Apple uses the EFI System Partition for anything even for real Macs, except for occasional firmware updates?
Good to hear you've got it working. It sounds like this has been resolved.

yes, definitely resolved. I am onto testing Thunderbolt now. Assuming the details are under PCI vs. Thunderbolt.... OR, do I need another kext?

Thunderbolt discussion is being tracked in #6

Hey guys, sorry to comment on a closed issue but I'm having a similar issue and haven't been able to resolve it.
Specs:
Intel NUC6i7KYK
32GB RAM
1x 1TB WD Black NVME SSD
Have you tried resetting NVRAM?
Not seeing the option to in the opencore bootloader menu?
Is there compiled aml files in the ACPI folder?
Yes
Have all of the expected kexts downloaded correctly (occasionally GitHub's API can bug out and not download the couple that are fetched from there)
Yes, all mentioned above are present in the kext folder.
Did you enable the Intel WiFi option or leave it disabled?
Enabled.
Are you trying to run Catalina or Big Sur Beta?
Tried Catalina 10.15.7 & Big Sur Beta 11, both produce the same result of being stuck on the Apple Logo.
Possible Issues:

Using a USB3.0 "Pocket SSD" drive, don't currently have a 2.0 one big enough
Using a Mojave VM & Xcode 11.3.1 to build

Other than those 2 things, I believe everything else has been pretty vanilla in terms of following the instructions here. Any thoughts?

Hi @TurtIeSocks
Can you boot in verbose mode (add -v to boot-args or press Cmd/Windows + V on loading the OpenCore boot selector) and post a picture of what it gets stuck on?
Thanks 😃
Alex

Sure! This is the result. Thank you for the reply :)

No worries
Halting on critical error here means there's a kext that is required by OpenCore, but is missing from the folder.
I have found that if you build the repo on Big Sur it can fail to compile the IntelMausiEthernet kext because of a pre-release toolchain issue, so that could be missing.
Check the post above for the kexts that are required for the system to boot.
Thanks
Alex

Looks like all of the above mentioned kexts are present. Building on Mojave.

@TurtIeSocks
Check your config.plist. Maybe there’s something missing or malformed line.

Looks like all of the above mentioned kexts are present. Building on Mojave.

Rename AirportItlwm.kext to itlwm.kext for now
I wasn't aware it started downloading that kext instead of the original one...I'll get that changed

Ah, good catch. Renaming alone didn't appear to fix it, but removing the kext entirely has gotten me past the Apple logo :) thanks again!

Everything went smoothly for Catalina. Can confirm that the Big Sur installation also hung after trying to rename the itlwm.kext file. Works with excluding the kext entirely though.


Construct UrdfBotPawn and UrdfRobotComponent

Constructing UrdfBotPawn and UrdfRobotComponent.
A few known issued in this PR at current stage:

asymetric revolute/prismatic joint. By default UPhysicsConstraintComponent must have symmetric joint limit, i.e. the range to move in positive and negative direction is the same.
multi visual node for individual link. I havent find a clean way to weld multiple UStaticMeshComponent and ensure the physical behavior is as expected.  This requires more investigation more than expected and will not be included in this PR.
visual link ignore material color and only uses texture filename as unreal material uasset reference path.
config absolute path in urdfbot.yaml properly. Need suggestions to decide the design here.

Hi Renhan, here are my thoughts on your points above.

It seems It is fine for now if UPhysicsConstraintComponent can only support symmetric joint limits. We don't need to support all possible URDF files. When we're setting the joint limits in the Unreal constraint objects, we can simply assert that the limits specified in the UrdfJointDesc are symmetric. As a side note, it is good that we're setting joint limits like this, even if they're not supported by RBDL.
Likewise, when we're setting Unreal meshes, we can simply use the first entry of UrdfLinkDesc::visual_descs_ and ignore the rest. If we ignore some aspect of the URDF file like this, we should print a warning message.
Eventually, we should include some custom attributes in our parser, e.g., we could have an optional spear_unreal_asset_path attribute on the material node or something. In general, this seems like a cleaner solution than overriding the intended meaning of existing attributes. It is conceivable that users (and maybe even our future selves) would want to use the same URDF file in multiple renderers or simulators. Having renderer-or-simulator-specific values in their own custom attributes and nodes (instead of overriding standard ones) would make it easier to use the same URDF file in multiple simulators.
Perhaps a cleaner solution would be for spear.get_config() to set a URDFBOT.URDFBOT_PAWN.URDF_DIR config value directly in the code (e.g., after reading all the default_config.*.yaml files, but before reading the user-specified files, so the user can override if desired). This would be the first instance of setting a config parameter value directly in our core Python code, rather than through a default_config.*.yaml file, although our example code already does this in several places. I feel this approach is justified because spear.get_config() is already responsible for preparing a usable config object, and it already knows about the python/spear/config folder, so it would make sense for it to also know about the python/spear/urdf folder. We could then interpret URDFBOT.URDFBOT_PAWN.URDF_FILE as being relative to URDFBOT.URDFBOT_PAWN.URDF_DIR. By default, if the user wants to load one of our usual URDF files, they could just specify the filename only, e.g., URDFBOT.URDFBOT_PAWN.URDF_FILE="locobot.xml" in their user_config.yaml. On the other hand, if a user wanted to load their own URDF file, they could do that too by setting URDFBOT.URDFBOT_PAWN.URDF_DIR to be empty.

I'm not ready to give up on constructing a hierarchy of Unreal components to match the hierarchy in the URDF file. I think we could use NewObject instead of CreateDefaultSubobject. We already use NewObject in various places in our code. I believe NewObject has limitations, such as the components not being visible in the editor or available to blueprints, But so far, those limitations have not been overly burdensome for us, and I think NewObject would be a good fit for dynamically creating the hierarchy of link and joint components.
Here is a discussion about CreateDefaultSubobject vs NewObject. This discussion mentions that NewObject is well-suited for creating components dynamically:
https://forums.unrealengine.com/t/newobject-or-createdefaultsubobject/335095
Here is a sketch of the design I had in mind.
//
// UrdfBotPawn.cpp
//

UUrdfRobotComponent* robot_component_ = nullptr; // defined in header

AUrdfBotPawn::AUrdfBotPawn(const FObjectInitializer& object_initializer) : AActor(object_initializer)
{
    UrdfRobotDesc robot_desc = UrdfParser::parse(Config::get<std::string>("URDFBOT.URDFBOT_PAWN.URDF_FILE_PATH"));

    robot_component_ = CreateDefaultSubobject<UUrdfRobotComponent>(Unreal::toFName("AUrdfBotPawn::robot_component_" + robot_desc.name_));
    robot_component_->initialize(&robot_desc);

    RootComponent = robot_component_;
}

//
// UrdfRobotComponent.cpp
//

UUrdfLinkComponent* root_link_component_ = nullptr; // defined in header

UUrdfRobotComponent::initialize(UrdfRobotDesc* robot_desc)
{
    ASSERT(robot_desc);

    UrdfLinkDesc* root_link_desc = robot_desc->root_link_desc_;
    ASSERT(root_link_desc);
    root_link_component_ = NewObject<UUrdfLinkComponent>(this, Unreal::toFName("UUrdfRobotComponent::root_link_component_" + root_link_desc_->name_));
    root_link_component_->initialize(root_link_desc);
    root_link_component_->SetupAttachment(this);
}

//
// UrdfLinkComponent.cpp
//

std::vector<UUrdfLinkComponent*> child_link_components_; // defined in header
std::vector<UUrdfLinkComponent*> child_joint_components_; // defined in header

UUrdfLinkComponent::initialize(UrdfLinkDesc* link_desc)
{
    ASSERT(link_desc);
    ASSERT(link_desc_->child_link_descs_.size() == link_desc_->child_joint_descs_.size());

    // initialize inertial, material, collision properties here...

    int num_child_links = link_desc_->child_link_descs_.size();

    for (int i = 0; i < num_child_links; i++) {
        UrdfLinkDesc* child_link_desc = link_desc_->child_link_descs_.at(i);
        ASSERT(child_link_desc);
        UUrdfLinkComponent* child_link_component = NewObject<UUrdfLinkComponent>(this, Unreal::toFName("UUrdfLinkComponent::child_link_components_" + child_link_desc->name_));
        child_link_component->initialize(child_link_desc);
        child_link_component->SetupAttachment(this);
        child_link_components_.push_back(child_link_component);

        UrdfJointDesc* child_joint_desc = link_desc_->child_joint_descs_.at(i);
        ASSERT(child_joint_desc);
        UUrdfLinkComponent* child_joint_component = NewObject<UUrdfLinkComponent>(this, Unreal::toFName("UUrdfLinkComponent::child_joint_components_" + child_joint_desc->name_));
        child_joint_component->initialize(child_joint_desc, this, child_link_component);
        child_joint_component->SetupAttachment(this);
        child_joint_components_.push_back(child_joint_component);
    }
}

//
// UrdfJointComponent.cpp
//

UUrdfJointComponent::initialize(UrdfJointDesc* joint_desc, UrdfLinkComponent* parent_link_component, UrdfLinkComponent* child_link_component)
{
    ASSERT(joint_desc);
    ASSERT(parent_link_component);
    ASSERT(child_link_component);

    // initialize joint properties here...
}

I like your flat implementation, where the UUrdfRobotComponent owns (and is directly responsible for creating) all subcomponents.
I'm including an updated sketch here that follows your proposed design, but is slightly cleaner. There are a few things to notice about this sketch.

I have completely separate functions for initializing a component and recursively creating subcomponents. Note that initializeComponent(...) never does anything recursive, and createChildComponents(...) never does any detailed initialization of a subcomponent. Separating these distinct responsibilities into separate functions makes the code easier to read.
I assume that each UrdfJointDesc pointer is stored with the child UrdfLinkDesc, rather than parent. This is a slightly different data layout than what we have currently, but it makes the code in UUrdfRobotComponent easier to read. With this modification in place, UUrdfRobotComponent can call child_link_component->initializeComponent(child_link_desc), which is slightly semantically cleaner than calling child_link_component->initializeComponent(child_link_desc, joint_desc). This modification also implies that the for loop that iterates over children can use a range-based syntax, which is easier to read than the classical c-style for-loop in the current PR.
I re-arranged the recursive code so I only need a single for loop that iterates over children. The current PR has two for loops that do the same thing.

//
// UrdfBotPawn.cpp
//

UUrdfRobotComponent* robot_component_ = nullptr; // defined in header

AUrdfBotPawn::AUrdfBotPawn(const FObjectInitializer& object_initializer) : AActor(object_initializer)
{
    UrdfRobotDesc robot_desc = UrdfParser::parse(Config::get<std::string>("URDFBOT.URDFBOT_PAWN.URDF_FILE_PATH"));

    robot_component_ = CreateDefaultSubobject<UUrdfRobotComponent>(Unreal::toFName("AUrdfBotPawn::robot_component_::" + robot_desc.name_));
    robot_component_->initializeComponent(&robot_desc);
    robot_component_->createChildComponents(&robot_desc);

    RootComponent = robot_component_;
}

//
// UrdfRobotComponent.cpp
//

 // public
std::map<std::string, UUrdfLinkComponent*> link_components_;
std::map<std::string, UUrdfJointComponent*> joint_components_;

// public
UUrdfRobotComponent::initializeComponent(UrdfRobotDesc* robot_desc)
{
    ASSERT(robot_desc);

    // initialize name
}

// public
UUrdfRobotComponent::createChildComponents(UrdfRobotDesc* robot_desc)
{
    ASSERT(robot_desc);

    UrdfLinkDesc* root_link_desc = robot_desc->root_link_desc_;
    ASSERT(root_link_desc);

    UUrdfLinkComponent* root_link_component = NewObject<UUrdfLinkComponent>(this);
    root_link_component->initializeComponent(root_link_desc);
    root_link_component->SetupAttachment(this);
    link_components_[root_link_desc->name_] = root_link_component;

    createChildComponents(root_link_desc, root_link_component);
}

// private
UUrdfRobotComponent::createChildComponents(UrdfLinkDesc* parent_link_desc, UUrdfLinkComponent* parent_link_component)
{
    ASSERT(parent_link_desc);
    ASSERT(parent_link_component);

    for (auto& child_link_desc : parent_link_desc->child_link_descs_) {
        ASSERT(child_link_desc);
        UUrdfLinkComponent* child_link_component = NewObject<UUrdfLinkComponent>(this);
        child_link_component->initializeComponent(child_link_desc);
        child_link_component->SetupAttachment(parent_link_component);
        link_components_[child_link_desc->name_] = child_link_component;

        UrdfJointDesc* child_joint_desc = child_link_desc->joint_desc;
        ASSERT(child_joint_desc);

        UUrdfJointComponent* child_joint_component = NewObject<UUrdfJointComponent>(this);
        child_joint_component->initializeComponent(child_joint_desc, parent_link_component, child_link_component);
        child_joint_component->SetupAttachment(parent_link_component);

        joint_components_[child_joint_desc->name_] = child_joint_component;

        createChildComponents(child_link_desc, child_link_component);
    }
}

//
// UrdfLinkComponent.cpp
//

// public
UUrdfLinkComponent::initializeComponent(UrdfLinkDesc* link_desc)
{
    ASSERT(link_desc);

    // initialize inertial, material, collision properties...
}

//
// UrdfJointComponent.cpp
//

// public
UUrdfJointComponent::initializeComponent(UrdfJointDesc* joint_desc, UrdfLinkComponent* parent_link_component, UrdfLinkComponent* child_link_component)
{
    ASSERT(joint_desc);
    ASSERT(parent_link_component);
    ASSERT(child_link_component);

    // initialize joint properties...
}


LDAP support in golang

It is pretty lame to see no built-in support for LDAP v3. LDAP is a most widely used protocol for authentication and identity management. Having a core package with full ASN/DER/BER support will make it very easy for everyone. Mozilla C LDAP SDK and Java JNDI could be good reference points for this implementation. I would love to see LDAP included in roadmap for golang (along with ASN/BER/DER which will open the doors for server side implementation as well).

See https://golang.org/doc/faq#x_in_std

I believe X is referring to X.yyy specs(ITU). LDAP v3 is an RFC spec. X'specs were for directory support(X.500-X.522). Even with X' golang does support X.509. So the argument is not valid. Also, LDAP is structured, uses ASN.1 for structured data representation. LDAP is core to system authentication  and authorization. Not supporting LDAP will restrict adoption of golang for security applications.

No, I wrote that FAQ entry. X is a placeholder to mean anything.
We are not adding LDAP to the standard library. See https://godoc.org/?q=ldap
I'm sorry if you find this lame.


Ruby warning after migrating from s3 action to aws_s3 plugin

I recently migrated from s3 action to aws_s3 plugin as suggested by the deprecation message of the action. Since then every time I run fastlane I receive the following warnings:
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-plugin-aws_s3-0.2.0/lib/fastlane/plugin/aws_s3/actions/aws_s3_action.rb:9: warning: already initialized constant Fastlane::Actions::SharedValues::S3_IPA_OUTPUT_PATH
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-2.9.0/fastlane/lib/fastlane/actions/s3.rb:9: warning: previous definition of S3_IPA_OUTPUT_PATH was here
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-plugin-aws_s3-0.2.0/lib/fastlane/plugin/aws_s3/actions/aws_s3_action.rb:10: warning: already initialized constant Fastlane::Actions::SharedValues::S3_DSYM_OUTPUT_PATH
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-2.9.0/fastlane/lib/fastlane/actions/s3.rb:10: warning: previous definition of S3_DSYM_OUTPUT_PATH was here
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-plugin-aws_s3-0.2.0/lib/fastlane/plugin/aws_s3/actions/aws_s3_action.rb:11: warning: already initialized constant Fastlane::Actions::SharedValues::S3_PLIST_OUTPUT_PATH
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-2.9.0/fastlane/lib/fastlane/actions/s3.rb:11: warning: previous definition of S3_PLIST_OUTPUT_PATH was here
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-plugin-aws_s3-0.2.0/lib/fastlane/plugin/aws_s3/actions/aws_s3_action.rb:12: warning: already initialized constant Fastlane::Actions::SharedValues::S3_HTML_OUTPUT_PATH
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-2.9.0/fastlane/lib/fastlane/actions/s3.rb:12: warning: previous definition of S3_HTML_OUTPUT_PATH was here
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-plugin-aws_s3-0.2.0/lib/fastlane/plugin/aws_s3/actions/aws_s3_action.rb:13: warning: already initialized constant Fastlane::Actions::SharedValues::S3_VERSION_OUTPUT_PATH
/usr/local/lib/ruby/gems/2.4.0/gems/fastlane-2.9.0/fastlane/lib/fastlane/actions/s3.rb:13: warning: previous definition of S3_VERSION_OUTPUT_PATH was here

Is there any way to suppress these warnings?

@mrtj If there isn't a way to suppress these warnings I will open a PR into the the main fastlane project and make a way 😊  But I agree, this is a bit annoying 😉

@mrtj This is fixed on master in fastlane. It should be included in the next release 🚀
https://github.com/fastlane/fastlane/pull/7976


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/source-wg-leads
/assign knative-sandbox/source-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 1ae8a49...eb4c06c:
  > eb4c06c Add CloudEvent SQL validation (# 6032)
  > eb62db9 Fix broken links (# 6030)
  > 7bf054f Add k8s trace attribute to PingSource (# 5928)
  > b8d1cfb rename pingsource component (# 6003)
bumping knative.dev/serving 719ab3a...0309174:
  > 0309174 upgrade to latest dependencies (# 12481)
  > 76bdf86 Fix broken links (# 12472)
  > 26b9a40 Move liveness checks to user container port (# 12479)
  > 7dba334 Update net-contour nightly (# 12478)
  > ef58256 Update net-kourier nightly (# 12477)
  > 15c37e7 upgrade to latest dependencies (# 12476)

/lgtm
/approve


Whiteboard collaberation with TogetherJS

Would it be possible to add TogetherJS support to allow multi user collaberation including audio chat!
TogetherJs is a free framework that allows multiple users to collaborate and is developed by Mozilla, meaning that it will probably be well supported in the future.
Furthermore, everything is client side, meaning you don't need to do any backend coding.
It's also used in JSFiddle!
There is also a fair tutorial in how to use TogetherJs for a drawing application: https://hacks.mozilla.org/2013/10/introducing-togetherjs/
with a full example here:
https://togetherjs.com/examples/drawing/
http://jsfiddle.net/5f8FL/1/
which would hopefully help!

Mozilla has a history of abandoning things that are not their core competency. (Firefox OS, Persona, Thunderbird being the most recent examples.) I'd much rather have this be a plugin than part of the core library. The real challenge is fixing the API to support this kind of thing more easily.

I think that improving the plugin API part is perhaps the most important thing, because I would imagine that many users want a drawing plugin that is collaborative.
Therefore I believe it's important to improve the literally canvas API so it can support this kind of thing. This would mean that users can make their drawing app collaborative regardless on what plugin they are using (i.e. not just TogetherJs).
I understand that you might not want to make TogetherJs a part of the core code, but the API of "literally canvas" should be good enough such that this can be easily done, which at the moment, it's clearly not!

Dupe of #351


Report Verification System to Prevent Spam

Introduce a verification system where submitted reports need to be verified by other users or through image analysis to prevent spam reports. This will maintain the integrity of the data submitted to the authorities.

Kindly assign me this issue

@Ricky2054 use testing mob no :-<PHONE_NUMBER>
Otp :- 000000
To login


Case Sensitivity in PRQL - Pros and Cons

I've been woolgathering about the value of case-sensitivity in PRQL. So I thought I would write down my random thoughts, to see what I think, and pass that along to you.
Assumptions:

PRQL will always require a compiler to generate the SQL. It can enforce any rules
I think SQL is case-insensitive. I know its keywords are; most of my experience is with SQLite, where I believe table and column names are also case-insensitive.
I work with lots of languages, and always have trouble remembering where case-sensitivity matters. I always stumble along for the first 15 minutes while I refresh my memory.
I don't use SQL professionally - I don't know whether its lack of case-sensitivity is a problem that needs to be rectified.

Pros to case-sensitivity:

It's consistent to require all table names, columns, variables to be the same case
PRQL requires its keywords (from, select, etc) to be lower case
The PRQL compiler can enforce all the constraints. That avoids a serious user trap of allowing variables "Abc" and "abc" to represent different values

Cons to case-sensitivity: (Arguments why it's OK not to be case-sensitive)

I think in terms of hurdles for developers. Learning something new is hard; learning arbitrary rules, especially if they "don't make a difference" is harder.
Who do we expect to be the "user base" for PRQL? If it's just software developers, then they'll be able to use whatever rules we devise. If they're grizzled SQL pro's who don't care about upper/lower case, not so much
The underlying SQL "engine" doesn't care - it simply executes what we output.
PRQL emits uppercase SQL keywords. Reading that out could be confusing.

Conclusion: In my mind, this argues for non-case-sensitivity: let variables "Abc" and "abc" be the same. The compiler can still check that toLower(variable-name) matches. The underlying/back end SQL engine won't care. It avoids "yet another" hassle in producing usable PRQL code. If we like, the PRQL compiler/VScode extension can provide hints that "Abc" and "abc" don't match, but will still produce correct code.
All this said, I am completely prepared to believe that there are compelling reasons for case-sensitivity. Thanks for listening.

I think this decision was made before I joined the project, but I also stand behind it. But it's a good thing that we have a discussion about it.
The trend in other programming languages is going toward case sensitive keywords and identifier, with basically all major languages being case sensitive (C, C++, Java, Rust, Go, JavaScript, Python). This is not a real justification, but just an observation.
Primary reason I dislike insensitivity is that it is really hard to implement for all languages. In slovenian, there are a few extra letters (č, š and ž) with their uppercase variants (Č, Š and Ž). I don't think that we should delve into supporting slovenian and every other letter collation, but I also think that if caj ~ CAJ then it should also hold that čaj ~ ČAJ. That would be much harder to explain to a non technical user than insensitivity.
Also, I like having a "normalized" code style as opposed to different conventions that are used by different companies or communities. Rust does this really well with rustfmt and Python with black formatter. I know that we could still auto format or display lints about non-conventional casing, but I think we can afford to inforce case sensitivity from the start.
And as you pointed out, we can enforce this rule.

I think this decision was made before I joined the project, but I also stand behind it. But it's a good thing that we have a discussion about it.

Good. I'm content with this decision. One of the reasons I write these things out is to document our discussion of the issues, and our conclusions. I will close this. Thanks, all.

One of the reasons I write these things out is to document a discussion of the issues, and our conclusions.

Great, thanks @richb-hanover , very much agreed


⚠ Fox Sports has degraded performance

In 0e361b1, Fox Sports (https://foxsports.com) experienced degraded performance:

HTTP code: 200
Response time: 881 ms

Resolved: Fox Sports performance has improved in 11d44e2.


Support resource naming based on DependentUpon source by convention

Related issues:
https://github.com/dotnet/winforms/issues/638
https://github.com/dotnet/project-system/issues/4807
https://github.com/dotnet/corefx/issues/34190
In SDK projects and new project system, we have eliminated the need for DependentUpon to make the appropriate file nesting in the IDE tree. However, there is a place where the build actually uses DependentUpon to:

Locate a source file, and parse (!) it to get first class name and namespace
Generate .resources accordingly.

Let first me say that this whole design is terrible and it is absolutely insane that we have code to find the class and namespace name in msbuild:
https://github.com/microsoft/msbuild/blob/f309171ed821eb66cb635403e1fbff41ea22ae9c/src/Tasks/CSharpParserUtilities.cs#L15-L20
But people and features in VS rely on this and it has been a consistent source of feedback in moving to .NET Core 3.0.  So I give up and propose:

Add a boolean property that, when true, causes us to look for the source file by convention when DependentUpon metadata is not found.

Set that property by default for .NET Core App 3.0 TFM.

cc @rainersigwald @davkean @livarcocc

Proposed property name: EmbeddedResourceUseDependentUponConvention
Proposed new argument to CreateManifestResourceName: UseDependentUponConvention
The new code would go after this if it didn't return anything (so explicit in-the-project metadata should continue to win):
https://github.com/microsoft/msbuild/blob/07d3c25a1461dfba3fcc1cc9b28cb8accd4e32b9/src/Tasks/CreateManifestResourceName.cs#L143

Sounds good.

@BenVillalobos can you take a look at this once you're feeling better?

Let first me say that this whole design is terrible and it is absolutely insane that we have code to find the class and namespace name in msbuild

This was for compatibility with the pre-MSBuild behavior, FWIW. The developer writing this parser wasn't happy about it either!

That's worth a lot. Fascinating.

Just hit this while porting EF6 to .NET Core. We drop two files into the project:

Migrations\201907170000000_MigrationName.cs // Contains a class named MigrationName
Migrations\201907170000000_MigrationName.resx

The resource gets embedded as RootNamespace.Migrations.201907170000000_MigrationName.resources instead of just RootNamespace.Migrations.MigrationName.resources causing ResourceManager to throw.
The work around, of course, is to explicitly set the item's DependentUpon metadata:
<EmbeddedResource Update="Migrations\201907170000000_MigrationName.resx">
  <DependentUpon>201907170000000_MigrationName.cs</DependentUpon>
</EmbeddedResource>

cc @ajcvickers @divega @Pilchie

Set that property by default for .NET Core App 3.0 TFM.

@nguerrera should this apply to .NET Standard 2.1 TFM as well?

Yes. Good catch.

Shouldn't this be turned on almost always with new format?

I'm hesitant to make a breaking change here.

Maybe turned off for known existing TFMs but on by default so that it is picked up by .NET Core 3.0 and up, .NET Standard 2.1 and up, new Mono versions, etc?

@BenVillalobos This is fixed, right? It still doesn't appear to be working in 3.0-preview9/16.3-preview3. https://github.com/aspnet/EntityFramework6/issues/1225

@bricelam It's fixed . . . unless the resx file is in a folder: #4695. Your workaround in the linked bug is good.

Now that the rules for manifest resource name selection have changed, refer to this dotnet docs issue for a detailed explanation on how this selection works now.


Error in the menu in the UWP app

Hi team,
I moved this from here: https://github.com/MicrosoftDocs/xamarin-docs.es-es/issues/82
The comment below was machine-translated
@wjvelasquez commented 2 days ago — with docs.microsoft.com
The app works as expected, although it has some errors.
The problem is that in the UWP app, which I do not understand the reason to ignore by you, the menu does not look good. The menu does not display the text unless it is expanded in all 3 points.
In my opinion. I think they should put more effort into UWP and not leave it out in their examples.

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

ID: 60587f77-05de-89b9-8a08-c7e7221d27af
Version Independent ID: 4080b687-b353-f833-4bb7-a7a3f9993338
Content: Perform navigation in a multi-page Xamarin.Forms application - Xamarin
Content Source: docs/get-started/quickstarts/multi-page.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Thanks for this feedback.
For product issues like UWP display problems, please raise an issue on the Xamarin.Forms open-source repo.


Browser integration test failure: provideCellStatusBarItems called on metadata change

Please see https://github.com/microsoft/vscode/runs/4550160925?check_suite_focus=true#step:16:563
  222 passing (3m)
  67 pending
  1 failing
  1) statusbar
       provideCellStatusBarItems called on metadata change:
     Error: asPromise TIMEOUT reached
      at Timeout._onTimeout (extensions/vscode-api-tests/src/utils.ts:131:11)
      at listOnTimeout (internal/timers.js:554:17)
      at processTimers (internal/timers.js:497:7)

/duplicate of https://github.com/microsoft/vscode/issues/126371#issuecomment-994185142


Fixes swiftlint 0.25.0 warnings

This PR contains fixes to Swiftlint 0.25.0 warnings. Though I feel like Presenter fix is relative, it would be good if we still keep it less viable to warnings.

Thanks @RoshanNindrai!


Fix coverage reporting

Collect coverage only on Ubuntu and one python version in order to fix failing coverage collection.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@68f602b). Click here to learn what that means.
The diff coverage is n/a.


Run/debugger not shown as on video

Issue Type: Bug
1 Launch VS Code
2 select a js file to run/debug
3 Select the debugger tab
==> The Run/Debugger buttons are not visible as in YouTube videos.  I just see "R..."
Screen shot
Screen Shot 2021-02-27 at 9.26.41 PM.png
I'm new to Java Script and have not coded anything since the 90's
I need help
VS Code version: Code 1.53.2 (622cb03f7e070a9670c94bae1a45d78d7181fbd4, 2021-02-11T11:45:54.515Z)
OS version: Darwin x64 20.3.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2500)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.54GB free)

Process Argv
--crash-reporter-id dd9896a9-49e2-4a58-b174-1bb0739bccfa

Screen Reader
yes

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

code-runner
for
0.11.2

vscode-docker
ms-
1.10.0

remote-containers
ms-
0.158.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
dockersubsetcf:30264245

Please paste your screenshot here. With most browsers you should be able to drag-drop a file onto the comment field.

Ok, I pasted it to the email before e but I guest the field is just text.  I am just trying to find an environment that lets me run Java Script..  I am doing this to work with Adobe Acrobat.  I also know C, C++ , and a few Scripting languages and shell scripts.
VS  Code seems like the perfect browser plus!
Ed

On Feb 28, 2021, at 12:08 AM, John Murray<EMAIL_ADDRESS>wrote:
Please paste your screenshot here. With most browsers you should be able to drag-drop a file onto the comment field.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/117843#issuecomment-787414156, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS7JUZA2TVOBG2TAIJQQ2IDTBH2YPANCNFSM4YKWVQKA.

/gifPlease

@ehackett00 instead of pasting the name of your screenshot ("Screen Shot 2021-02-27 at 9.26.41 PM.png") into the comments fields here on github, please paste the image file from the macOS Finder into the comment. The file will be automatically uploaded to github and a link is added in the comment.

Hello,
In the screen shot I saw an “R…”.  What is that?
Ed

On Feb 28, 2021, at 10:29 PM, VS Code Issue Triage Bot<EMAIL_ADDRESS>wrote:
Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev https://gifcap.dev/ helpful as a browser-based gif recording tool.

@ehackett00 if there is not enough space to show a name, then the name is shortened and an ellipsis ("...") is appended.
Typically this means that you'll have to make some view or panel larger in order to show the full name.
Since you cannot provide a screenshot, it is difficult for me to guess what you are seeing.

Hello,
Here is the screen shot for the third time!
Ed

On Mar 1, 2021, at 9:19 AM, Andre Weinand<EMAIL_ADDRESS>wrote:
@ehackett00 https://github.com/ehackett00 if there is not enough space to show a name, then the name is shortened and an ellipsis ("...") is appended.
Typically this means that you'll have to make some view or panel larger in order to show the full name.
Since you cannot provide a screenshot, it is difficult for me to guess what you are seeing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/117843#issuecomment-788122344, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS7JUZEFAGN6O6XOZ2HRNPDTBPEDBANCNFSM4YKWVQKA.

You said:

I am just trying to find an environment that lets me run Java Script..  I am doing this to work with Adobe Acrobat.

I'm not sure whether VS Code is a good fit for developing JavaScript for Adobe Acrobat.
I could not find a VS Code extension for JavaScript and Adobe Acrobat, which means that you would have to use the standard JavaScript support of VS Code which targets either Node.js or web browsers. So specific Adobe Acrobat functionality would not be available from within VS Code.
Is "JavaScript for Adobe Acrobat" still being used?
Related information like https://acrobatusers.com/tutorials/javascript_console/ appears to be rather old...


feat: add IVFFlat index functionality to PGVector

Hello, thanks for this extremely useful project.
pgvector now support 2 index types HNSW and IVFFlat,

I noticed that there is a pull request to support HNSW index to PGVector, may I ask when it would be available since we need to do ANN search and it is encouraged to move to PGVector?
Is there any plan to support IVFFlat index?

I'll try to review next week, but having another reviewer from the community will be helpful


nginx: formating the config file

Motivation for this change
Formating the config file with awk to readable format.
Used this script - https://github.com/nixcloud/nixcloud-webservices/blob/master/modules/web/webserver/lib/nginx_check_config.nix
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test nginx nginx-sso

Thanks!

@Izorkin Thanks, sometimes it's nice to have a more readable config. Most often it doesn't matter... But when you need it it's very handy.


⚠ Bofrost.at has degraded performance

In 3798bde, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1723 ms

Resolved: Bofrost.at performance has improved in ad6e380.


fixed many typos and errors

Fixed typos and grammatical errors on .Rmd chapter files.
Added notes with info for Windows users and other relevant updates (e.g., DigitalOcean new interface).

Hi Sean,
I finished reading your book yesterday, and as a way to give back, today I went through your .Rmd chapter files, fixing all typos and grammatical errors I could find. I also added notes with supplemental info for Windows users. I hope you find my contributions helpful, and thank you for all you do with this repository.
Best,
Jefferson

Thanks so much @decarvaj. Also please see: https://github.com/seankross/the-unix-workbench/blob/master/CONTRIBUTING.md#adding-your-name-to-the-guestbook


loading pretrained model

I downloaded the otter weights: OTTER-MPT7B-Init, where should I put this downloaded file? (now I put it in /Otter/pipeline/demos/interactive/)
And, when I try to run the demo (file path: /Otter/pipeline/demos/interactive/otter_image.ipynb),  the following line reports error:
model = OtterForConditionalGeneration.from_pretrained("luodian/OTTER-MPT7B-Init", device_map="sequential")
error message:
OSError: We couldn't connect to 'https://huggingface.co/' to load this file, couldn't find it in the cached files and it looks like mosaicml/mpt-7b-instruct is not the path to a directory containing a file named config.json. Checkout your internet connection or see how to run the library in offline mode at 'https://huggingface.co/docs/transformers/installation#offline-mode'.
I wonder how can I use the downloaded weights? And can the weights: OTTER-MPT7B-Init be loaded by module OtterForConditionalGeneration? Or I should load it by code: from otter_ai.models.flamingo.modeling_flamingo import FlamingoForConditionalGeneration?

model (model configuration, components, etc.),

Thank you for your contributions!

Or I should load it by code: from otter_ai.models.flamingo.modeling_flamingo import FlamingoForConditionalGeneration?

Yes, you should load it via Flamingo, that's better way. However, you can still load it via OtterForConditionalGeneration since we've handled the logic of loading different models.
As for the connection problem, may you need a VPN for it?

I have already downloaded the weights, how to use this weights instead of diresctly downloading it from huggingface?

Or I should load it by code: from otter_ai.models.flamingo.modeling_flamingo import FlamingoForConditionalGeneration?

Yes, you should load it via Flamingo, that's better way. However, you can still load it via OtterForConditionalGeneration since we've handled the logic of loading different models.
As for the connection problem, may you need a VPN for it?

I have already downloaded the weights, how to use this weights instead of diresctly downloading it from huggingface?

或者我应该通过代码加载它： from otter_ai.models.flamingo.modeling_flamingo import FlamingoForConditionalGeneration？

是的，你应该通过 Flamingo 加载它，这是更好的方法。但是，您仍然可以通过 OtterForConditionalGeneration 加载它，因为我们已经处理了加载不同模型的逻辑。
至于连接问题，需要VPN吗？

我已经下载了权重，如何使用这个权重而不是直接从huggingface下载？

I put the downloaded weights under huggingface path and the weights seem to be loaded successfully, while I have encounter another problem, when I load the weights by : if args.model_name.lower() == "otter":             model = OtterForConditionalGeneration.from_pretrained(                 args.pretrained_model_name_or_path,                 **kwargs,             )
reports an error of :
Traceback (most recent call last):   File "/data/data/project/LLMs/Otter/pipeline/train/instruction_following.py", line 536, in <module>     main()   File "/data/data/project/LLMs/Otter/pipeline/train/instruction_following.py", line 332, in main     model = OtterForConditionalGeneration.from_pretrained(   File "/data/ldata/anaconda3/envs/otter/lib/python3.9/site-packages/transformers/modeling_utils.py", line 3236, in from_pretrained     model = cls(config, *model_args, **model_kwargs)   File "/data/data/anaconda3/envs/otter/lib/python3.9/site-packages/deepspeed/runtime/zero/partition_parameters.py", line 503, in wrapper     f(module, *args, **kwargs)   File "/data/data/project/LLMs/Otter/src/otter_ai/models/otter/modeling_otter.py", line 751, in __init__     text_tokenizer = AutoTokenizer.from_pretrained("luodian/OTTER-MPT7B-Init")   File "/data/data/anaconda3/envs/otter/lib/python3.9/site-packages/transformers/models/auto/tokenization_auto.py", line 733, in from_pretrained     config = AutoConfig.from_pretrained(   File "/data/data/anaconda3/envs/otter/lib/python3.9/site-packages/transformers/models/auto/configuration_auto.py", line 1064, in from_pretrained     config_class = CONFIG_MAPPING[config_dict["model_type"]]   File "/data/data/anaconda3/envs/otter/lib/python3.9/site-packages/transformers/models/auto/configuration_auto.py", line 761, in __getitem__     raise KeyError(key) KeyError: 'otter'.
Is it because otter can't be used as model_name? I'm confused.

Can you check your transformers version via pip show transformers.
You may need to follow our fixed version.
transformers is always updating, and would cause many unexpected errors.

您可以通过查看您的变形金刚版本吗pip show transformers？
您可能需要遵循我们的固定版本。
变压器总是在更新，并且会导致许多意想不到的错误。

pip show transformers
Name: transformers
Version: 4.35.1
Summary: State-of-the-art Machine Learning for JAX, PyTorch and TensorFlow
Home-page: https://github.com/huggingface/transformers
Author: The Hugging Face team (past and future) with the help of all our contributors (https://github.com/huggingface/transformers/graphs/contributors)
Author-email<EMAIL_ADDRESS>License: Apache 2.0 License
Location: /data/data/anaconda3/envs/otter/lib/python3.9/site-packages
Requires: filelock, huggingface-hub, numpy, packaging, pyyaml, regex, requests, safetensors, tokenizers, tqdm
Required-by: peft, trl

您可以通过查看您的变形金刚版本吗pip show transformers？
您可能需要遵循我们的固定版本。
变压器总是在更新，并且会导致许多意想不到的错误。

It seems the version is ok.
Does the config file need to be modified?
Is this import statement correct for the weights I have already downloaded?
model = OtterForConditionalGeneration.from_pretrained("luodian/OTTER-MPT7B-Init", device_map="auto")
(I have changed  the line of tokenizer in class OtterForConditionalGeneration:     if "llama" not in config.text_config._name_or_path:             if config.text_config.architectures[0] == "MPTForCausalLM":                 # text_tokenizer = AutoTokenizer.from_pretrained("mosaicml/mpt-7b-instruct")                 text_tokenizer = AutoTokenizer.from_pretrained("luodian/OTTER-MPT7B-Init"))
What other code changes are needed?
Can you provide the full import model code?

您可以通过查看您的变形金刚版本吗pip show transformers？
您可能需要遵循我们的固定版本。
变压器总是在更新，并且会导致许多意想不到的错误。

My version of transformer is the same as the one you provided.
I've solved this problem, by re-downloading the model weights from huggingface's mirror site. Thank you for your reply.


Jira native fixes & more test coverage

Looks good to me at this stage.

Latest commit addresses review feedback, makes the Jira extension depend on camel-quarkus-support-spring, adds another test case and support for running the tests with OAuth creds.