Remove rgdal dependency

The package rgdal is currently listed as a package dependency for LTabundR, but that package is no longer maintained. Remove all rgdal functions/data and replace with sf tools.

This was actually fixed back in March but I did not have the time to verify. Now verified. All rgdal dependencies should be gone. Confirmed with :
packrat:::recursivePackageDependencies("LTabundR", ignore = "", lib.loc = .libPaths()[1]) %>% sort


No such file or directory: u'/usr/local/lib/python2.7/dist-packages/mkdocs_bootswatch/flatly/mkdocs_theme.yml'

I've installed mkdocs and bootswatch using pip:
$ sudo pip install mkdocs
$ sudo pip install mkdocs-bootswatch

Then i've configure the flatly theme in my mkdocs site configuration, and I end up with:
$ mkdocs -v build
DEBUG   -  Loading configuration file: /srv/admin-docs/site-markdown/mkdocs.yml
DEBUG   -  [Errno 2] No such file or directory: u'/usr/local/lib/python2.7/dist-packages/mkdocs_bootswatch/flatly/mkdocs_theme.yml'

And indeed, the file there does not exist.
Am I doing something wrong?

The problem disappears if I manually create the file /usr/local/lib/python2.7/dist-packages/mkdocs_bootswatch/flatly/mkdocs_theme.yml with the content:
static_templates: []

This was fixed in #36.


Feature request: Is it possible to get a global break on error flag?

As discussed 2016/11/18:

BreakOnError should be default.

It is reasonable that essentially all levels of the XML file can override this default behavior.  On a per configuration, per action and per command.

As discussed 2016/11/18:

BreakOnError should be default.

It is reasonable that essentially all levels of the XML file can override this default behavior.  On a per configuration, per action and per command.

By default, breakOnError is set to true for actions. There is a different issue that was interfering with the breakOnError working correctly. That should be fixed shortly.
Based on that fix, I think it is perfectly fine if we enable users to set it to false, currently only at an action level. In the future, we can do this for a command as well.

I agree!


Filter Grid Component

Having used the grid component filtering seems to be a missing function to most grid components. There isn't a built in filtering method and simply filtering the bound entities doesn't refresh the grid in the UI. Being able to filter the component will be a really useful addition.

@paulpitchford FYI

@paulpitchford expect this component before Aug 31, 2022.


Crash Report: Argument rangeInElement (133,635) endOffset must not exceed descriptor text range (13207, 13656) length (449).

Description
Stacktrace
Please paste the full stacktrace from the IDEA error popup

This has been fixed recently and will be available in version 0.6.1. Thanks for reporting!


Does anyone have a sample Swift Project using this??

hey @iPhoneSDKPro, thanks for the project interest. this is the swift port:
http://github.com/nextlevel


[After #289] Meson/LCOV: Apply a workaround to increase lcov of 'service-db.cc

This patch applies a simple workaround to increase the line coverage of the 'service-db.cc' file.
Signed-off-by: Wook Song<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #306. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Remove NoneType in results preprocessing

None types in results for preprocessing will result in an error when trying to return. Filtering out the None types beforehand appears to fix this, and preprocessing - alongside #17 - continues to actually write now.

Don't merge yet. This has knock on effects when training due to how chunking is done.

When I remove NoneType in results preprocessing, I have another error that the training trunk shape is not 100000. How to resolve this error?


The Docs Bounty is Over

Preview is available here:http://developers-pr893.previews.kube001.services.stellar-ops.com


Configurable TAP probe, refactor judge resources to shared red team

resources for red team type activities as a shared lib

Model-as-a-judge logic can apply to many techniques

refactor TAP probe to use Configurable

updated probe to accept generic model config
set default to cpu for hf model in tap generation
update util function to allow any valid generator
warn when generator is not known supported config
remove unused max_token params

Verification
List the steps needed to make sure this thing works
config.file.yaml:
plugins:
  generators:
    huggingface:
      hf_args:
        device: cpu
  probes:
    tap:
      attack_model_config:
        max_tokens: 500
        hf_args:
          device: cpu
      evaluator_model_type: nim
      evaluator_model_name: meta/llama3-8b-instruct
      evaluator_model_config:
        uri: http://<IP_ADDRESS>:8000/v1
        api_key: fake

[ ] garak -m huggingface -n meta-llama/Llama-2-7b-chat-hf --config config.file.yaml --probes tap.TAP
[ ] Run the tests and ensure they pass python -m pytest tests/

This is just foundation that will aid code reuse in the detector implementation, the PR enables the TAP probe to accept model configuration in it's existing usage of LLMaaJ.

lgtm


[BUG] - Sprint List - Sprint Estimate not showing

Exactly as in the title.
The sprint.get_sprints() function does not return any 'estimate' column.

... but it does return the 'tasks' column. One quick Jinja fix later and it's taken care of!

Estimate summary now shows up!


Enforce use of variables to define colours

The ColorKeyword linter ensures against color: purple, preferring color: #800080.
However, the documentation for this linter suggests that...

A better approach is to use global variables like $color-text-body.

Can this be enforced using existing linters? If not, could I propose a new linter, ColorVariable, requiring variable definitions ($color-text-body etc.) in place of colours (eg. #333) themselves?
The reasons for preferring variables over direct colour definitions are already outlined in the ColorKeyword linter docs. In addition to the points mentioned, forcing the use of variables can ensure only colours from the project's colour palette are used.
Some additional considerations:

The new linter could supersede the existing ColorKeyword linter.
The existing ColorKeyword linter must avoid reporting variables.
Any 'config' or 'common' files defining the colour variables should specify // scss-lint:disable ColorVariable (this should be documented, as using it would be compulsory to avoid linting the colour definitions themselves).
A more involved solution could additionally check the definition of the referenced variable within other files (i.e. to ensure the specified variable indeed referenced a valid colour).

Hey @olliebennett, thanks for suggesting ways we can improve scss-lint.
I think there's value in something like what you describe, but done in a different way.
First of all, the ColorKeyword linter still has value for developers who wish to standardize on always using hex codes for colors (the documentation perhaps doesn't capture this reason effectively, but it does exist). This removes the need to address the first two points in your list.
I think what you're actually looking for is the ability to report a lint for any color that is not referenced as part of a variable definition. In this case, I would call it ColorReference.
Thus, we would not report lints for references where a color was being stored in a variable:
$color-text-body: #333;

...but we would report lints for references to explicit colors
p {
  color: #333;
  @include my-color-adjuster(#ccc, 50);
  ... etc ...
}

I think this gets to the core of what you are looking for. Let me know if that's not the case.

Thanks @sds. I agree that ColorReference is more succinct, and yes - your example indeed provides what I am looking for.
I can attempt to create a pull request for this if you don't have the bandwidth to tackle it?

A pull request would be appreciated. Otherwise I'll get to it when I can.
Thanks!

My coworker Andy and I just submitted a PR for this called UseVariablesForProperties that lets you specify the props for which you want to use variables. This way, you can enforce variables for 'color', 'font', etc...

I'd love to see that in

Implemented in #349.

Great to see work happening on this. While I think the new VariableForProperties linter is very useful, it doesn't quite meet the requirements of this issue because one can still use non-standard colours in their code.
color: #EEE; // now successfully linted
border: 1px solid #EEE; // still not linted

Naturally, I don't want to specify a variable for all borders (and configuring lint of border-color has no effect on the above). Perhaps we still need a color-specific linter to achieve this? Or could/should the new VariableforProperties linter be extended to support this?
The same would apply to fonts, where one could require the font-family to be a pre-configured, but not worry about font size.
Not sure if the above has been considered? Again, thanks to @sds and @mikeholm for getting it this far!

Hey @olliebennett,
Thanks for pointing out this discrepancy. You are right that this particular issue as filed has not quite been addressed yet. I was a bit trigger-happy with closing it.
The VariableForProperty linter is still valuable on its own, but we still need a linter that enforces using color literals only in variable declarations.

I've added the ColorVariable linter in e72cb23. Thanks for your input, everyone.


Assignment-2 completed

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Submitting Assignment_2 of ASC
What did you learn from the changes you have made?
Learned linear regressing. Splitting, fitting, and assessing the model accuracy.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Not at this time
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
I was unable to split the training and test set properly
How were these changes tested?
These changes were tested in vs code
A reference to a related issue in your repository (if applicable)
na
Checklist

[ ] I can confirm that my changes are working as intended

A2: Incomplete
Q1 part (i) you must specify the number of columns
Q1 part (ii) you must explain what it is in short? (1-2 sentences at max). Not only state it is float64
Q1 part (iii) The question is asking to list the top 5 rows in which HP (horsepower) is largest
Q1 part(iv) The dataset has more than 5 predictors, please take a closer look
Q2 part (i) what about cylinders vs mpg, what is the association for that plot ?
Q2 part (ii) More specifically it is a line of best fit, that is looking to minimize the squared distance from the fitted regression line and the plot of points
Q3 did not complete step 2 correctly. try refitting the model in a different way
Q4 np.mean is not the correct function to call here. RMSE is a single numeric value that you should have for your output

Hi,
Thank you for your comments! I tried X_train, X_test, y_train, and y_test but could not generate an equal number of rows on both train sets. Other than that, all the comments were addressed.

A2: Incomplete
Q1 part (iii) I swould suggest trying .nlargest() to list the top 5 rows with highest Horsepower
Q2 Part (i) Solution is Missing
Q3 please try the following line of code "numeric_predictors = mpg_train.select_dtypes(include=[float, int]).drop(columns=['mpg'])". Reflect on what this line of code is attempting to achieve, then fit your linear model from there.
Q4 Changes still need to be made.

Hi Melokazz,
Q2. I am still unable to fix the Q3 as you mentioned, it is giving the unequal number of x_train and y_train. I couldn't find you in slack. It would be great if we could talk.


Changing README file

Explanation About the Issue

I have tried to install frappe charts on my machine locally
I encountered this issue  npm-run-all is not a recognizable command
So I added a line of code to resolve this issue

Regards,
    I hope this little contribution of mine would help newcomers to setup frappe chart locally without any heckle.

Thanks for being my
First PR :fire:


Add finder for download's dir

Hi I was wondering if it was possible to add a finder to find the directory where you want to download? We do not always know the right way to download, and I think that would be very useful.
Thank you for your reply.

I don't think aria2c provides an ls-like interface on RPC server, thus impossible. https://aria2.github.io/manual/en/html/aria2c.html#methods
This project really depends on the interfaces exposed by aria2 (imagine webui-aria2 and aria2 on seperated servers and aria2 server only exposes port 6800 for RPC, you cannot even know where is available to put)
If you are just confused by your own file system, try my docker image for fully packed aria2 and webui-aria2 and use -v /host/path/to/downloads:/downloads in docker parameters to make everything clear and tidy.


Fixed: ListParty doesn't show middleName (OFBIZ-11681)

(OFBIZ-11681)
added: funtion to retrieve middleName from Person record and append it to the result

Implemented with PR #122


Screenshot

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

(triage) Hi @Matt2D, do you have plans for this PR?

Yep! Sorry about the delay I have a pair PR in flutter/engine that is taking some time cleaning up. If anything needs to happen to this PR feel free to do so


iOS 14 Accuracy Authorization support

Up!🤟

Please, provide code for request and get status

there can be option for accuracyAuthorization (.fullAccuracy) check with location permission

ref: https://github.com/richardrodz/AccuracyAuthorization

For anyone looking for a quick fix, I have a fork that has the option of explicitly requesting preciseLocationAlwaysAndWhenInUse and preciseLocationWhenInUse. This was just a super quick fix for our internal purposes, but if the approach is seen as a good general solution, I will gladly put up a PR. It simply checks CLLocationManager().accuracyAuthorization to determine what level of accuracy was granted. There may be a better approach, or deeper integration with the new API for asking for temporary full accuracy.

@Strobocop I updated app to new version, maybe you can make PR with your code?
I will add images and other stuff.

Closing because no activity.


Enable automated index updates

We have update functions for indexes we use to track remote sources, such as crossref and unpaywall. They can be run any time, but should also be set to run every day. We haven't completed that yet due to previous issues we had with the live system causing an unknown error. The dev system can run the updates for the ones where we share the same dev and live index. So we need to finish this by adding a schedule to dev that doesn't go to live, and runs the necessary updates every day.

This will ship as soon as permissions & deposit do.

Previous comment remains true.

I've tested this on dev and live for permissions affiliations (because it is small and simple) and it works. Tomorrow I'll check that it runs as expected on its own then I'll turn on other permissions and then bigger ones like crossref, oadoi etc. I'll keep this issue open for this week, but there's nothing left to actually do unless unexpected problems occur.


Total Failure Following Module update

If you update modules to the latest in Azure this breaks the scriping as it fails to understand
Start-AzureRmAutomationRunbook -automationAccountName $automationAccountName -Name $runbookName -ResourceGroupName $aroResourceGroupName -Parameters $params
Only solution is to delete all, and re-deploy with a new auto account without the updated modules.

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

ID: 225c9d05-83dd-b006-0025-3753f5ab25bf
Version Independent ID: 9eecef0c-b1cb-1136-faf7-542214492096
Content: Start/Stop VMs during off-hours solution
Content Source: articles/automation/automation-solution-vm-management.md
Service: automation
Sub-service: process-automation
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@LeeWeedon Thanks for reaching out. Sorry to hear about the trouble you are having.  Unfortunately this is a known issue. A note had been added to the top of the article earlier this week to point out that the latest Azure modules do not work with the Start Stop VM Solution.  When there is an update to this we will make sure to add it to the Updates RSS feed on Azure.com [Azure updates](https://azure.microsoft.com/en-us/updates/?product=automation

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


TODO: Document and Schema Version Controls and Logging of changes for Audit

If is it possible could you also add in the TODO List the following two points

Document and Schema Version Controls (history of changes)
Logging of changes for Audit (who, when, and what changed in the document)

You have a good project with best practice for learning Deno from scratch

Yes @YuriyTigiev This looks a very good practice, I'll add in TODO.

I have done the first step - update the function createUser.
Deno driver doesn't support multi-document and I'm not sure if this implementation is optimal for my case.
  public static async createUser(
    options: CreateUserStructure,
  ): Promise<ObjectId | Error> {
    const { name, email, password, role, isDisabled } = options;
    const hashedPassword = await HashHelper.encrypt(password);
    const createdAt = new Date();

    const user: ObjectId = await User.insertOne(
      { name, email, password: hashedPassword, role, isDisabled, createdAt, doc_version: 1},
    );

    if(user){
      const user_history: ObjectId = await UserHistory.insertOne(
        {_id:user,  name, email, password: hashedPassword, role, isDisabled, createdAt, doc_version: 1},
      );
    }
    else {
      log.error("Could not create user");
      return throwError({
        status: Status.BadRequest,
        name: "BadRequest",
        path: "user",
        param: "user",
        message: `Could not create user`,
        type: "BadRequest",
      });
    }
    return user;
  }

@YuriyTigiev Looks good to me. Would you mind opening a pull request?

Looks good to me. Would you mind opening a pull request?

Pull or push?
You don't have permissions to push to 'vicky-gonsalves/deno_rest' on GitHub. Would you like to create a fork and push to it instead?
I can send you changes (3 files) by email.  I

Hi @YuriyTigiev ,
Let's follow the best practice here so that in future other devs can also see the changes.
Fork master branch into your git account -> make changes-> push commits to your master branch-> open Pull request to this master branch.
Once you open the pull request, I ll review the changes and merge into master branch

I will do it
but now  deno cache --reload --unstable --lock-write --lock=lock.json ./deps.ts returns an error
error: Import<EMAIL_ADDRESS>failed: 404 Not Found

@YuriyTigiev It seems there is some issue with the djwt library. Meanwhile, you can use repository version directly untill developers solve the issue.
https://raw.githubusercontent.com/timonson/djwt/master/mod.ts

Now I'm getting  another error
import * as yup from "https://cdn.skypack.dev/yup";

export {
  AES,
  encode,
} from "https://deno.land/x/god_crypto/mod.ts";
export {
  Application,
  Router,
  send,
  Context,
  Status,
  isHttpError,
  helpers,
} from "https://deno.land/x/oak/mod.ts";
export type {
  RouterContext,
} from "https://deno.land/x/oak/mod.ts";
export { config as dotEnv } from "https://deno.land/x/dotenv/mod.ts";
export {
  handlers,
  getLogger,
  setup,
} from "https://deno.land/std/log/mod.ts";
export { MongoClient, ObjectId } from "https://deno.land/x/mongo/mod.ts";
export { oakCors } from "https://raw.githubusercontent.com/tajpouria/cors/master/mod.ts"; // Currently using fork version to support deno 1.4.0
export { validateJwt } from "https://raw.githubusercontent.com/timonson/djwt/master/mod.ts";
export type {
  Jose,
  Payload,
} from "https://raw.githubusercontent.com/timonson/djwt/master/mod.ts";
export {
  makeJwt,
  setExpiration,
} from "https://raw.githubusercontent.com/timonson/djwt/master/mod.ts";
export { yup };

Check file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts
error: TS2305 [ERROR]: Module<EMAIL_ADDRESS>has no exported member 'validateJwt'.
export { validateJwt } from<EMAIL_ADDRESS>         ~~~~~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:27:10

TS2305 [ERROR]: Module<EMAIL_ADDRESS>has no exported member 'Jose'.
  Jose,
  ~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:29:3

TS2305 [ERROR]: Module<EMAIL_ADDRESS>has no exported member 'makeJwt'.
  makeJwt,
  ~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:33:3

TS2305 [ERROR]: Module<EMAIL_ADDRESS>has no exported member 'setExpiration'.
  setExpiration,
  ~~~~~~~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:34:3

Found 4 errors.

Try deleting lock.json file and reloading the deps. It may solve the issue

The same result

From the error stack, I don't know why but for some reason in your dependencies it still looking for<EMAIL_ADDRESS>It should be pointing to https://raw.githubusercontent.com/timonson/djwt/master/mod.ts

Check file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts
error: TS2305 [ERROR]: Module '"https://raw.githubusercontent.com/timonson/djwt/master/mod.ts"' has no exported member 'validateJwt'.
export { validateJwt } from "https://raw.githubusercontent.com/timonson/djwt/master/mod.ts";
         ~~~~~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:27:10

TS2305 [ERROR]: Module '"https://raw.githubusercontent.com/timonson/djwt/master/mod.ts"' has no exported member 'Jose'.
  Jose,
  ~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:29:3

TS2305 [ERROR]: Module '"https://raw.githubusercontent.com/timonson/djwt/master/mod.ts"' has no exported member 'makeJwt'.
  makeJwt,
  ~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:33:3

TS2305 [ERROR]: Module '"https://raw.githubusercontent.com/timonson/djwt/master/mod.ts"' has no exported member 'setExpiration'.
  setExpiration,
  ~~~~~~~~~~~~~
    at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:34:3

Found 4 errors.
PS C:\Users\yuriy\OneDrive\Documents\GitHub\deno_rest>

@YuriyTigiev It looks like<EMAIL_ADDRESS>ia back to life. Could you possibly revwrt back and check its working?

I think we should wait for 1-2 days.
Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>error: Import<EMAIL_ADDRESS>failed: 404 Not Found
at file:///C:/Users/yuriy/OneDrive/Documents/GitHub/deno_rest/deps.ts:28:0
PS C:\Users\yuriy\OneDrive\Documents\GitHub\deno_rest>

Yes I believe they are working on it. Let's wait.

https://github.com/timonson/djwt/issues/38#issuecomment-723205372

@YuriyTigiev Thanks for commenting. I am fixing those breaking changes for djwt

40114edf9b72a2090ec83e46bb56ee26bc723d64
This fixes the issue of djwt
@YuriyTigiev Please check this commit.
I have also added support for Deno v1.5.1 and fixed some lint and code format in following commit
da3f980c46c7c6fd300d63657f593f88ee1bfe27

If is it possible could you also add in the TODO List the following two points

Document and Schema Version Controls (history of changes)
Logging of changes for Audit (who, when, and what changed in the document)

You have a good project with best practice for learning Deno from scratch

#9

I have checked. Everything works


test installation in SUSE kiwi, bind to <IP_ADDRESS> and disable detection of systemd

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

We need install this rpm in SUSE customized OS where PID1 is not equals to systemd, just for a test.

When I install this rpm in SUSE, system unit could not be bring up because of missing linux group.


Mean Sea Level

The sea level changes are, I think, all relative to mean sea level, rather than being relative to the local land surface. This is important because in some areas the coast is sinking or being up-lifted at rates which are comparable (e.g. East Anglia is sinking by about 30cm per century). To avoid confusion, I think the text should say explicitly that coastal subsidence and/or uplifting is not included in these figures.

Implemented in the spreadsheet and will be included in the next update.
"Projected changes in the mean sea level index are calculated relative to the mean of the mean sea level during the historical portion of this dataset which covers the period from 1977-2005. Storm surges caused by atmospheric forcing, and coastal subsidence and/or uplifting are not taken into account in the calculation of this index."

Updated the description with the following text:
Tidal dynamics, sea level change data from climate models, and large-scale long-term changes to land height due to post-glacial rebound are taken into account in the calculation of this index.

https://github.com/cedadev/c3s_434_ecde_page_text/commit/68ab06166f979ff5ef6ea14edc9f973a8a7eea6f
@sharppaul this is now ready to be published.


Add cache-from and cache-to options

I have a workflow that gets triggered whenever I create a tag, however, the created cache only works for that tag, and not other tags or any other run at all, see this SO post for what I mean: https://stackoverflow.com/questions/74113734/share-gha-docker-cache-among-branches

The article is about docker action and I am not sure how it relates to this action.
From my experience, the cache mechanism seems to be working regardless of the tag.
https://github.com/kaz-utashiro/greple-jq/actions
If you can show me the history of the action that is actually not working, I may be able to provide more details.

Here's my workflow https://github.com/Xpl0itU/WiiUDownloader/blob/main/.github/workflows/macos.yml, inter-tags caching isn't working for me: https://github.com/Xpl0itU/WiiUDownloader/actions/runs/8741312805/job/23987045220

I found a statement in https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#restrictions-for-accessing-a-cache.

Workflow runs also cannot restore caches created for different tag names.

So it seems to be an intended restriction of actions/cache.
There are some discussions.
https://github.com/actions/cache/issues/1342


Fix testing farm result mapping

Map testing_farm_result to pull_request.
Fixes #504

recheck

For the third time..;(

Only my failing internet connection has saved your PR from being rage-merged by me :-)

recheck


IsFirstRowAsColumnNames issue in Mono 2.10.12

When reading a simple excel sheet, if IsFirstRowAsColumnNames is set to true, it will throw an error if the first entry in the second row is a number.  I am using this on Unity Mac with Mono 2.10.12 and the latest from develop branch for .net 2.0.
I've 2 excel files to help demonstrate the issue: Succeeds.xlsx (http://s000.tinyupload.com/?file_id=64684690629063216723) where there is a a string in each column can be read successfully.
The second file, Fails.xlsx (http://s000.tinyupload.com/?file_id=00124105141439481298) where the columns are filled with numbers (besides the column name) cannot be read in, and throws the below error. Both files can be read successfully if IsFirstRowAsColumnNames is false. I've also tested them as .xls and the results are the same.
The error it throws is:
InvalidCastException: Cannot cast from source type to destination type.
System.Data.Common.DoubleDataContainer.DoCopyValue (System.Data.Common.DataContainer from, Int32 from_index, Int32 to_index)

This is the code that I am using to read the data in:
// Read the excel file depending on it's type
if (xlsRegex.IsMatch(filePath))
{
    // Reading from a binary Excel file ('97-2003 format; *.xls)
    excelReader = ExcelReaderFactory.CreateBinaryReader(stream);
}
else if (xlsxRegex.IsMatch(filePath))
{
    // Reading from a OpenXml Excel file (2007 format; *.xlsx)
    excelReader = ExcelReaderFactory.CreateOpenXmlReader(stream);
}
else
{
    Debug.LogError("Unexpected files type: " + filePath);
    return null;
}

// First row are columns names
excelReader.IsFirstRowAsColumnNames = true;

// DataSet - The result of each spreadsheet will be created in the result.Tables
DataSet result = excelReader.AsDataSet();

I've tried out the files in the TestApp on a Windows machine and they are both read in successfully, so perhaps this is a Mono only issue?

This is the same as #30, which demonstrates the problem clearly. Although it sounds like a problem in Mono - should verify.

With the latest release IsFirstRowAsColumnNames property is actually missing. I don't know if its intentional   But it provides another way to resolve this in ReadMe 👍


Fix Google Analytics implementation

Currently Analytics is recording data but unfortunately I can't get the tag manager to fire on certain events.
That makes it difficult to optimize our campaigns for conversions.

Fixed!


How to access Receipt additional details from verify method

Hi Guys
Some drivers like Behpardakht or AsanPardakht have additional detail with createReceipt on return of verify method like cardNumber.
But how to access this additional info at callback page after calling the verify method ?

This can be used by calling object child:
$receipt->cardNumber


Only generate workload identity signing key if not already done

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
This PR prevents recreating the gardener-apiserver pod on each make gardener-up.
cc @ialidzhikov @vpnachev
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm

/retest


Azurekid/multiple data sources/updated techniques

Required items, please complete
Change(s):

added missing mitre techniques

Reason for Change(s):

missing mitre techniques

Version Updated:

Yes

Testing Completed:

Yes, using Pester

Checked that the validations are passing and have addressed any issues that are present:

Yes

@aprakash13 I noticed a merge conflict and fixed this
As a result, I needed to update the version number of some detections to pass the test.

@azurekid, there are some data connectors removed /deleted, is there any reason for that? as it is affecting the solutions.

@azurekid, there are some data connectors removed /deleted, is there any reason for that? as it is affecting the solutions.

Hi and thanks for looking in to this.
I didn't add the removal of data connectors in my Pull Request when looking at my initial branch and commits.
I think these are removed on a merge origin/master but where not part of my updates.

@azurekid , will you please raise a pr for the changes that were not needed and got committed in this?

@azurekid , will you please raise a pr for the changes that were not needed and got committed in this?

Hi, I just did, but also noticed that @aprakash13 has reverted all changes.
How would you like me to proceed? I had all ready created a PR that was reverting the deleted files #5508


Fixed error message around "PHP Deprecated:  fwrite(): Passing null to ..." - done

In newer PHP (> v8), it has strict parameter checks for fwrite() function. This results in the following error message if there is an unrecognisable instruction on the user .tag file. Raising an issue to proactively fix this error message.
PHP Deprecated:  fwrite(): Passing null to parameter #2 ($data) of type string is deprecated in tagui_parse.php on line 240

Impact - before the change, the error message for invalid step will show up with another cryptic PHP error message. after this change, the expected error message for the wrong instruction will show up and that's all.

Closing since this change has made its way into the latest packaged release.


npm run start:local (serve-ssr) no longer working

Actual Behavior
The run script npm run start:local that maps to ng run intershop-pwa:serve-ssr:local that is supposed to run a development PWA server with Server Side Rendering is no longer working as expected. Instead of rendering any page opening the PWA results in undefined:7:223720: missing '}' (the number might vary).
When starting the PWA with npm run start or ng serve the PWA pages render as expected.
Expected Behavior
Normally rendering any PWA page when running npm run start:local.
Steps to Reproduce the Bug
Steps to reproduce the behavior:

checkout the current develop branch
npm install
npm run start:local
open http://localhost:4200

Notes
The problem was introduced with the "deps: update to Angular 12 and Webpack 5" commit https://github.com/intershop/intershop-pwa/commit/c3c80edba22e2a1284ffabe90acef031864f377b.

Build defaults have changed in Angular 12. By default a production build is now executed. To fix this, local must disable some options.


ionic lib : This command has been deprecate

Ionic version: 1.3.1
I'm submitting a  bug report
Current behavior:
This command has been deprecate.  All resources are currently available in NPM a
nd we recommend that you use NPM to manage these.
More information is available here: https://github.com/driftyco/ionic-cli/wiki/M
igrating-to-NPM-from-bower

Local Ionic version: 1.3.1  (C:\Users\Cebit\myApp06\www\lib\ionic\bower.json)
Invalid version: undefined

Expected behavior:
I want to update ionic lib, because of this issue. of course I'm not sure updating the ionic lib will solve the issue.
Ionic info:
Cordova CLI: 5.4.0
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic Framework Version: 1.3.1
Ionic CLI Version: 2.0.0
Ionic App Lib Version: 2.0.0-beta.20
OS: Windows 7 SP1
Node Version: v4.5.0

Any idea?

This issue was moved to driftyco/ionic-cli#1771


Added settings to improve performance on IE 11 by allowing prevention of DOM object removal

I've added two extra settings aimed at improving performance. This has a substantial effect on modern heavy AngularJS apps on IE 11. In my performance tests, I was seeing 600ms overhead due to the offsetWidth calculations when the component was added/removed from the DOM.
The new settings will prevent DOM manipulation and will only hide the progress when it is done, so it is ready to go for the next call on SPAs.

@rstacruz can you pls review this PR?

@celeronpm Thank you for this PR, It improved my app, I'm debugging a performance issue in my app, and looks like this solved the issue!

This force repaint/reflow is introduced in https://github.com/rstacruz/nprogress/commit/031c5dc81d2b3d11ecb0e13082695d2b835d469d#diff-a320ff3f25e8b7cc682b156f26a26c91b47309d8783eadaa4934ca5114adae44R50 and affecting all browsers other than IE11.


Check other government department libraries and design systems for navigation component/patterns

What
Review design systems from other departments across government for components and patterns related to navigation, and add them to our research collection.
Why
This'll give us a view of what other departments may already have done on the topic of navigation to inform our work.
Done when

[x] Digital Land Design System
[x] Digital Scotland Design System
[x] DWP Design System
[x] GOV.WALES components
[x] HM Land Registry Design System
[x] HMRC Design Patterns
[x] Home Office Design System
[x] Ministry of Justice Pattern Library
[x] MOD.UK Design System
[x] NHS Design System
[x] NICE Design System
[x] Office for National Statistics Design System
[x] Transport for West Midlands Design System
[x] Hackney Design System
[x] Croydon design and content guidelines

Added a look at GOV.UK Prototype Component and GOV.UK Publishing Components

All investigation have been added to document


Create check_videos.js Cypress Test

Create the cypress test to check video tags to ensure videos are playable. This is in regards to https://github.com/corona-warn-app/cwa-website/issues/2590
NOTE: This is not ready to merge and is a work in progress. As suggested by @dsarkar and @MikeMcC399, I am opening this PR to facilitate collaboration. The current test script is able to locate videos and confirm the video is paused initially and start the video, but I am having difficulty properly checking that the video is actually playable/playing. Hoping that someone else may have some suggestions or want to take a stab at it while I try to work through the kinks on my side.

I suggest to convert this PR to draft as described here: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-stage-of-a-pull-request
@dsarkar could also add the do-not-merge label.

@Ein-Tim Great suggestions! I wasn't aware of the draft PR feature. I don't seem to be able to do this from the mobile version, but I will look to make the swap when I am back on my computer.
@dsarkar I think the suggested label by @Ein-Tim would make sense at this time.

@ahodzic2

This PR is related to #2590.

Unfortunately this test does not seem to address #2590 effectively. The test looks in /de/faq/results/ and /en/faq/results/ where there is just one video in each language version.
https://www.coronawarn.app/en/faq/results/#in_a_nutshell contains a link to https://www.coronawarn.app/assets/video/erklaerfilm_en.mp4
(similarly for the German page)
If the link to the video file is incorrect or the video file erklaerfilm_en.mp4 is missing, the test still passes. The test should fail.

I didn't specifically say so in the issue #2590, but to be effective any check for video files should also be looking for videos on other pages of the website. The blogs, the science page and the sign language page (Germany only) also contain links to video, so the test should also cover these locations.

Briefly looking at the code, I wonder why there are two log file names:
cypress/logs/broken_videos_result.txt and
cypress/logs/broken_links_result.txt
Does it make sense to have two different file names?

Hello @ahodzic2 and thanks for your contribution! We did review your PR and ran into some issues. Brian adapted your code and enhanced it a little bit. You can find the changes in his PR draft: https://github.com/corona-warn-app/cwa-website/pull/2931
You can take a look at what's changed from your original code and update your PR accordingly if you like. I would then merge your PR to our ongoing cypress-workflow branch that is planned to go live next week.

@svengabr Sounds good! Was hoping someone with a little more experience could take this and adapt it as needed, so that is perfect.
I'll go ahead and update this PR accordingly.
Thanks!

@ahodzic2 it appears that there are many changes unrelated to this issue are introduces in your latest PR. Could you please update your branch from master? I just made sure our cypress-workflow is already up-to-date

There is also another PR still up that solves the same issue: https://github.com/corona-warn-app/cwa-website/pull/2931/files

The sources were messed up and I was unable to figure it out. I closed your PR and re-opened a new one that is now able to merge https://github.com/corona-warn-app/cwa-website/pull/2950


scorecard - allow initContainer image to be configurable by user

Feature Request
https://github.com/operator-framework/operator-sdk/blob/master/internal/scorecard/testpod.go#L36
in scorecard today,  it uses a hard-coded image name for the initContainer when doing the untar.
For some environments, accessing dockerhub is not allowed, ideally we would have a means
to override the initContainer image to be used with something else (e.g. ubi-minimal).
Describe the solution you'd like.
I'd like to see in the config.yaml file and/or the command line a configurable initContainer image
name so that users can override the default value if they wish.
@scoheb
/language go

This is a dupe of #4886. Closing this is favor of that issue.


Handle error code 10 on Android

This PR handles error code 10 on Android by checking EN permission when accessing EN methods. It also extracts all exception in the app into a utils for convenience.
Ref https://github.com/cds-snc/covid-shield-mobile/issues/593
How to test?
Test in all cases: EN is OFF, Bluetooth is OFF, EN and Bluetooth are OFF.
Expected log

Error: performExposureStatusUpdate {"error": {"error": [Error: API_NOT_ENABLED], "message": "API_NOT_ENABLED"}} is called after performExposureStatusUpdate.
finalize is called at the end with: lastChecked.period should be the same before and after updating. last.Checked.timestamp should be different.

 {}
[Thu Jul 23 2020 15:26:07.183]  LOG      Saving exposure configuration to secure storage. {}
[Thu Jul 23 2020 15:26:07.191]  LOG      retrieveDiagnosisKeys {"period": 0, "url": "http://<IP_ADDRESS>:8001/retrieve/302/00000/f3068242ab71ef941656bd16e81856eb034819cf52af520899513dad50bb1c72"}
[Thu Jul 23 2020 15:26:07.229]  LOG      performExposureStatusUpdate {"exposureConfiguration": {"attenuationLevelValues": [0, 0, 2, 2, 2, 2, 2, 2], "attenuationWeight": 50, "daysSinceLastExposureLevelValues": [0, 1, 1, 1, 1, 1, 1, 1], "daysSinceLastExposureWeight": 50, "durationLevelValues": [0, 0, 0, 0, 5, 5, 5, 5], "durationWeight": 50, "minimumRiskScore": 1, "transmissionRiskLevelValues": [1, 1, 1, 1, 1, 1, 1, 1], "transmissionRiskWeight": 50}, "keysFileUrls": ["/data/user/0/ca.gc.hcsc.canada.stopcovid/app_c2acf5e3-43d0-4c49-b2d9-7f67dd53f4c2/keys.zip"], "lastCheckedPeriod": 18466}
[Thu Jul 23 2020 15:26:07.354]  LOG      Error: performExposureStatusUpdate {"error": {"error": [Error: API_NOT_ENABLED], "message": "API_NOT_ENABLED"}}
[Thu Jul 23 2020 15:26:07.477]  LOG      finalize {"currentExposureStatus": {"lastChecked": {"period": 0, "timestamp":<PHONE_NUMBER>901}, "type": "monitoring"}, "previousExposureStatus": {"type": "monitoring"}}
[Thu Jul 23 2020 15:26:07.484]  LOG      updatedExposureStatusInBackground {"exposureStatus": {"lastChecked": {"period": 0, "timestamp":<PHONE_NUMBER>901}, "type": "monitoring"}}

If you already passed onboarding screen and enable EN framework:

Terminate the app.
Go to Settings, disable EN framework.
Open the app.
Observe js log.

If you are still in onboarding screen:

In ExposureNotificationServiceProvider, call updateExposureStatusInBackground immediately.
Open the app.
Observe js log.
Go through onboarding flow and DO NOT allow to turn on EN.
At home screen, observe js log.
Click Enable COVID Alert.
Allow to turn on EN.
Observe js log goes through as normal without Error: performExposureStatusUpdate. You might see it hangs at performExposureStatusUpdate because you don't have exposed TEKs or you are in exposed state.

@henrytao-me
I'm still testing but wondering what comes back from the Framework in this case

We'll handle https://github.com/cds-snc/covid-alert-app/pull/864#issuecomment-663541852 in a separate PR.


Replace lib/data/puppeteer-core with puppeteer-core dependency

lib/data/puppeteer-core folder has 35.6 MB as size and it can be replaced and maintained as puppeteer-core dependency

https://github.com/rebrowser/rebrowser-patches
The relevant puppeteer-core file was patched with this update, and Runtime usage was disabled. Since the patch doesn't work correctly on every operating system and version, I wanted to include the pre-patched version directly in the library. When downloaded via npm, it catches most of the issues in the test file.
https://github.com/zfcsoftware/puppeteer-real-browser/blob/ecfcef6946d68b75e0e7d98dc21797a3c016b12c/test/cjs/test.js#L27

You can maybe use a library like patch-package where with "postinstall": "patch-package" in scripts you can apply all patches

https://www.npmjs.com/package/puppeteer-core-patch
Updated.


Current session top nav bar

Description
This PR contains the code for Top Nav Bar in Current Session Tab. By default, when the user goes into the current session tab, it will show the Menu tab. The user can tap on any tabs to go to the corresponding screen.
How to test
Pull this branch locally and go to the current session tab.

Notes
Issue tracking
https://trello.com/c/LCeAQaMR

🙌 good stuff!
I noticed that the style for summary page is a little bit off, probably because I used absolute values for positioning (CSS...). Do you mind making these adjustment and see how it looks on your simulator?

change height in GuestList.js to 45%
in CurrentSessionSummary.js

for container, remove marginTop
for title, change marginTop to 5% and height to 10%

Also should we rename CurrentSessionSummary.js to CurrentSessionSummary.screen.js for consistency?
Thank you! (totally fine with changing those myself after this is merged!)

Hi @tm2838 , I have made the changes as you suggested. In Android, it looks fine, but on iPhone simulators, it looks a bit off.


Validate k8s v1.12 for windows,

K8s v1.12 will be made the default Kubernetes version -https://github.com/rancher/rancher/issues/16262 , need to certify k8s v1.12 for windows.

Closing as we've decided to not go with 1.12 as a default version due to stability issues.


execution-timeout 300 >> Not working as expected

Description
I have set the execution time argument to 300 seconds to speed up the analysis. I expected the maximum analysis time to be 5 minutes, but it exceeds 20 minutes for some analysed contracts. I tried different values for this argument (e.g., 10s, 120s, 300s, the default: 86400), but it does not work as expected. Could you please explain the effect of this parameter and why the analysis does not stop when the allotted time expires?
For example:
$ myth analyze ./17f2e87d6f62e815dc918d768df6804f.sol --execution-timeout 300

Expected behavior
The analysis will be stopped and the report will be generated after a maximum of 5 minutes.
Environment

Mythril version: several versions including 0.4.24 and 0.4.28
Solidity compiler and version: Nothing specific, this happens with all versions up to version 8. The example provided was run on version 0.4.11.
Python version: 3.10 and also occurred with other versions like 3.12
OS and Version: Ubuntu 22.04.4

Thanks for helping!

Hi @shikahJS, can you send me the file with the source code to reproduce this error?
The z3 solver's timeout is not perfect in some cases.

Hi @norhh , this is the source code, please check it.
17f2e87d6f62e815dc918d768df6804f.sol.zip
Thanks.

Hi @shikahJS, there are multiple contracts in your given file. Hence, when you set a timeout, it is 5 minutes per contract, which is why it takes more than 20 minutes.

Hi @norhh, This makes sense, I thought the time was allocated to the whole myth command regardless of how many contracts were in the source code. However, I am still a bit confused about this argument. Sometimes when I stick to the default value, i.e. 24h, the analysis is much faster compared to setting the value to 5m, and I get the same analysis result. Kindly check the attached contract.

Source code: a6c230ff97ae1906c6f9c50c18250437.sol.zip

The analysis time with default settings was: 2.529884031s

The analysis time when the execution-time = 300s was: 28.348898807s

Analysis result: {"error": null, "issues": [], "success": true}

I wonder if reducing the execution-time argument will have a negative impact, as also my PC freezes a lot with the 5m setting compared to the default setting.
I am looking for the best trade-off between time and accuracy, as I am currently using Mythril to build a labeled dataset with thousand records.
I appreciate your prompt response.
Thank you.

Larger timeouts are usually better for such large contracts. I believe the current default timeout is 3600 (although it is large, it is not big enough). A good timeout depends on your computation budget. Usually larger is better (finding the ideal tradeoff itself is a hard task and is contract dependent).
This phenomenon mentioned in your post happens because for longer timeouts (>300 seconds), solver calls are made at every branch, which is not the case for shorted timeout calls. In case of shorter timeout execution, the solver call happens at the end of a transaction, this causes the slowdown.

Thank you for clarifying that.


Separate proxy support

This is what I had to do in order to get it working with a separate configurable proxy  docker container with docker compose
This seems to be related to this issue: https://stackoverflow.com/questions/30554702/cant-connect-to-flask-web-service-connection-refused
I'm not particularly tied to how I re-implemented get_origin_host. It was a quick fix to get things working and I'm happy to make any changes there.

closing in favor of #7


Revert "[HTML5] Better engine config parsing."

This reverts commit 2f509f1b12c33234a0d8f0e254c727fd92e57720.
Breaks closure compiler builds.
And adds a warning for future readers.
Fixes #57592 . Pretty sure it affects the exported version too (classical) too.

Thanks!

Cherry-picked for 3.5.

Cherry-picked for 3.4.3.


NODE_PATH usage breaks with relative imports in .less files.

Original comment
I'm looking at finding a work-around now.

Thanks for creating an issue and  PR. I just replied to the PR, please see that for more details.


generics Instantiated as null : {} causes other type errors to disappear

Examples:
class C<R>
{
  value : R;
  constructor(v? : R) { this.value = v }
}

// OK, no type error
var a : C<string> = new C("x");

// OK, type error
var b : C<string> = new C(1);
var c : C<string> = true ? new C("x") : new C(1);
var d : C<string> = new C(1) || new C<string>(null);
function e() : C<string> { if (false) { return new C(null) } else { return new C(1) } }

// Bad, no type error
var f : C<string> = true ? new C(null) : new C(1);
var g : C<string> = true ? new C(1) : new C(null);
var h : C<string> = (() => { if (false) { return new C(null) } else { return new C(1) } })();
var i : C<string> = new C(1) || new C(null);

Here, any time new C(1) is used it should have the type C<number> and not unify with C<string>. It seems like R not being known in new C(null) prevents these other type errors from occurring.

Basically what is happening here is that you're getting C<any> from new C(null) and C<number> from new C(1). Between the two, C<any> "wins out" and is assignable to C<string>.
Surprisingly, --noImplicitAny doesn't report a problem with this. Wondering if @JsonFreeman or @danquirk know whether this is intentional or not.

Thanks for the clarification. It seems then that in e.g. x = p ? a : b  there are only checks that a and b unify and that a+b unify with x. I think it would make sense to check that a and b unify with x individually. I was surprised that the typing differs between if statements and ternary expressions.

@DanielRosenwasser What surprises me is that the behavior of new C(null) is different from e.g. [null] or { x : null }. Those are inferred to Array<any> and { x : any }, but ternary expressions like above fail to type check for them. What makes a user defined class different?

Hey guys, I just remembered - this seems to be a duplicate of #531, but a symptom of #1436. What we proposed doing is widening type arguments to any at specific points instead of when inferring them.

I think this is a duplicate of #1436, as @DanielRosenwasser said. I am still in favor of not widening when we infer a type argument. As @ahejlsberg points out, the cost is that we have to have a general algorithm for walking a type and performing widening, and the algorithm has to be resilient to circularities in the type. I don't think this is unreasonable. We do this for all the other type walks.


[wishlist] Server-side reflog

There should be an option to enable server-side reflog for all or specific repositories. This is very helpful when somebody accidentally push -f
https://stackoverflow.com/questions/48769443/git-reflog-in-bare-repository-with-core-logallrefupdates-set-true-returns-nothin

why not just set this value in the gitea user's .gitconfig?

why not just set this value in the gitea user's .gitconfig?

Why not just user bare repositories, command line git and regular SSH access instead of this Gitea thing? :-)
Of course you can set this value per repository, as long as you are the server administrator.
It would be nice to let repository owners set this with Gitea web interface; also possibly make it default per server, per user, per organization etc.

This has been implemented now via config option. Issue can likely be closed.


Add streaming support in OllamaChatGenerator

Is your feature request related to a problem? Please describe.
OllamaChatGenerator doesn't support streaming
Describe the solution you'd like
There's already support for OllamaGenerator and we should implement the streaming support for OllamaChatGenerator
Describe alternatives you've considered
N/A
Additional context
N/A

Closed by https://github.com/deepset-ai/haystack-core-integrations/pull/757


Refreshing the project

Seeing as there's unexpected support and interest in this library (for the caching of data i guess) and the github api has upgraded entirely(?) to v3, I will look and see if I can make some more things to manipulate (Gists, Issues and more).

the work has started with automatic model and migration generation, also the retrieval&push logic is only in Fetchers module.


Container Remote Development: $SSH_AUTH_SOCK not respected while attaching a session

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

VS Code Version: 1.63.2
OS Version: Arch Linux with Kernel 5.16.2-arch1-1 and Openbox

Steps to Reproduce:

Starting ssh-agent and manipulating ~/.bash_profile (or ~/.bashrc) according to the official doc, i.e.

Starting an ssh-agent via ssh-agent and
adding following lines to the bash startup scripts if [ -z "$SSH_AUTH_SOCK" ]; then
   # Check for a currently running instance of the agent
   RUNNING_AGENT="`ps -ax | grep 'ssh-agent -s' | grep -v grep | wc -l | tr -d '[:space:]'`"
   if [ "$RUNNING_AGENT" = "0" ]; then
        # Launch a new instance of the agent
        ssh-agent -s &> $HOME/.ssh/ssh-agent
   fi
   eval `cat $HOME/.ssh/ssh-agent`
 fi

I checked, that every new spawned terminal (as well as the integrated bash terminal in VS Code) has the variable $SSH_AUTH_SOCK set accordingly, which is the case.
I ensured that the VS Code Setting Inherit Env is set to true in the general settings (as this already caused troubles, c.f. https://github.com/microsoft/vscode-remote-release/issues/2926)
Then, when I try to attach to a container (in the Docker extension, right click on a container and Attach Visual Studio Code), i see in the log files (F1 -> Remote Container Developers: Show All Logs):  ....
 [2022-02-04T10:11:22.656Z] Start: Launching Remote-Containers helper.
 [2022-02-04T10:11:22.657Z] ssh-agent: SSH_AUTH_SOCK not set on host.
 [2022-02-04T10:11:22.657Z] Start: Run: gpgconf --list-dir agent-extra-socket
  ...
  and thus, the inside the container, I don't have access to the agent, and git etc doesn't work.

I tried it as well with the VS Code insiders -- with no success.
What I also found out:
As a workaround, I tried to manually mount the ssh-agent socket into the container, by adding the following portion to my docker-compose.yml
myapp:
   ...
   volumes:
      - $SSH_AUTH_SOCK:/root/.ssh/ssh_auth_sock

When I start this compose file via "Right click on file -> Compose Up" in VS Code's file explorer, then I get the following error indicating that the env var $SSH_AUTH_SOCK is not present when I let VS Code start docker-compose by itself:
> Executing task: docker-compose -f "docker/devel/docker-compose.yml" up -d --build <

source /home/juergen/miniconda3/bin/activate
WARN[0000] The "SSH_AUTH_SOCK" variable is not set. Defaulting to a blank string.
1 error(s) decoding:

* error decoding 'Volumes[2]': invalid spec: :/root/.ssh/ssh_auth_sock: empty section between colons
The terminal process "/bin/bash '-c', 'docker-compose -f "docker/devel/docker-compose.yml" up -d --build'" failed to launch (exit code: 15).

But when I up the containers via VS Code's terminal and
docker-compose -f docker/devel/docker-compose.yml up

then it works.
Maybe someone who knows, how docker-compose gets executed by VS Code, has a clue, why $SSH_AUTH_SOCK is not available when I start docker-compose by VS Code.
Interestingly, on MacOS, Ubuntu and Windows we managed to get it running without problems. Somehow I suspect that it's some specific problems with Arch linux default settings in ~/.bash_rc etc. But I ensured, that each new bash session inherits SSH_AUTH_SOCK, but this isn't translated to the container.

This could be due to .bash_profile or .bashrc not finishing within 10 seconds. E.g., we have had similar reports when you start zsh or fish from .bashrc (this makes it not finish at all).
To further track this down, could you:

Run F1 > Preferences: Configure Runtime Arguments.
Add "log-level": "trace" to the opened argv.json.
Close all windows and restart VS Code.
Run F1 > Output: Focus on Output View.
Switch to the Log (Main) channel.
Search for the entries containing "shellenv" (Cmd + F works), remove any private or secret information and append the entries here.

Afterwards remove the "log-level": "trace" entry to stop filling the logs with trace output.


Vs2017 support

Adds support for VS 2017 and an icon

Just published this! Thanks to @rprouse and @jcansdale for their help!


Provide CI testing for camel-linkedin

https://fabric8-ci.fusesource.com/view/wildfly-camel/job/wildfly-camel/lastCompletedBuild/testReport/org.wildfly.camel.test.linkedin/

Wont fix (for now)....
Seems there's some logic on the LinkedIn side which detects that logins are occurring from multiple locations (E.g locally and on the build environment) and subsequently denies access to the APIs. Which means we'd need to maintain multiple developer accounts in order for this to work properly.
For now I'm happy to periodically run these tests manually whenever we do major component upgrades.
I may revisit this in future. It should be possible to record and replay the API responses. There are some challenges with HTTPS in Betamax and also the alternative mock framework Hoverfly (which uses Jersey). If I can resolve these (or find an alternative) then we could go down the mock response route.


Hitbox not showing up

Found an error where the hitbox isn't being created. It isn't being called in createSprite even though the code is in the class.

After rechecking the code I found that there was an extra "," under the class Sprite() where hitbox was declared. However this also doesn't fix the issue

I found the typo, under createSprite() where we actually create the hitbox, instead of getting the height from the Sprite() class I was calling this.height. It should have been this.hitbox.height. The hitbox is now showing.


pull request for e2e test at 12/27/2016 01:15:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Replace AlignMeasures with ConstrainedReschedule for pulse alignment

Summary
This PR introduces new pass ConstrainedReschedule (name should be discussed) which is a drop-in-replacement of AlignMeasures  pass. New pass takes both pulse and acquire alignment constraints thus it guarantees your circuit is executable on recent IBM Quantum backend with non-"1" pulse alignment constraint.
Details and comments
ConstrainedReschedule is implemented as AnalysisPass,  which updates node_start_time dictionary in the property set. This is follow-up of #7709 and necessary for #7760 . Note that this pass doesn't regenerate DAG instance, thus performance of scheduling chain will be improved. The pass ordering of preset pass managers has been also updated.
old, at #7709:
scheduling (analysis) -> padding (transform) -> alignment (transform)
new:
scheduling (analysis) -> alignment (analysis) -> padding (transform)
TODO

[x] write release note
[ ] update unittests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

233 of 259 (89.96%)  changed or added relevant lines in 18 files are covered.
144 unchanged lines in 14 files lost coverage.
Overall coverage increased (+0.03%) to 83.489%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/scheduling/alignments/pulse_gate_validation.py
21
22
95.45%

qiskit/transpiler/basepasses.py
2
4
50.0%

qiskit/transpiler/preset_passmanagers/level0.py
18
20
90.0%

qiskit/transpiler/preset_passmanagers/level1.py
18
20
90.0%

qiskit/transpiler/preset_passmanagers/level2.py
18
20
90.0%

qiskit/transpiler/preset_passmanagers/level3.py
18
20
90.0%

qiskit/transpiler/passes/scheduling/alignments/align_measures.py
7
10
70.0%

qiskit/transpiler/passes/scheduling/alignments/reschedule.py
74
78
94.87%

qiskit/transpiler/passes/scheduling/alignments/check_durations.py
16
24
66.67%

Files with Coverage Reduction
New Missed Lines
%

qiskit/circuit/gate.py
3
96.3%

qiskit/transpiler/target.py
3
98.06%

qiskit/algorithms/minimum_eigen_solvers/qaoa.py
4
89.19%

qiskit/circuit/instruction.py
4
95.34%

qiskit/circuit/library/standard_gates/x.py
6
98.21%

qiskit/circuit/library/arithmetic/weighted_adder.py
7
94.93%

qiskit/init.py
8
68.66%

qiskit/transpiler/passes/optimization/consolidate_blocks.py
8
89.66%

qiskit/circuit/library/arithmetic/piecewise_linear_pauli_rotations.py
10
89.32%

qiskit/circuit/library/arithmetic/polynomial_pauli_rotations.py
11
87.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
52460

Relevant Lines:
62835

💛  - Coveralls

Now unittests are updated and the PR is ready for final review. I noticed reschedule pass also needs classical I/O latency information to compute correct node start/end time of instructions acting on classical registers. These are already covered by existing test case in test.python.transpiler.test_instruction_alignment.
This makes me think adding latencies in the property set is better approach, rather than setting values in the constructor of every pass. Constructor values tend to have poor documentation (because developer just copies value without doc, or sometimes forget to update all docs synchronously, etc...) and to make pass API unstable since we didn't decide spec for these values. Instead, I introduced SetIOLatency analysis pass, that just copies values to property set, and one must call this pass before running scheduling passes (otherwise latencies default to zero). The IO latency information is also copied to output circuit to be used by timeline drawer.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: nkanazawa1989:x: Naoki KanazawaNaoki Kanazawa seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Sorry seems like I messed up CLA. I think I've fixed the wrong commit. The last commit is to move paddings to own module as what I've done for other two modules, since we have merged #7760 .


smbmap: init at unstable-2023-03-29

Description of changes
SMB enumeration tool
https://github.com/ShawnDEvans/smbmap
Related to https://github.com/NixOS/nixpkgs/issues/81418

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

2 packages built:

smbmap
smbmap.dist


fix: always remove message from memory spooler

Issue #, if available:
Given spooler storageType "Disk", and addMessage fails for some reason, we add the message to Memory spooler as a fallback. When we call removeMessageById it will fail with NullPointerException (It tries to remove message from DiskSpooler with the given ID) as message was never added to persistent spooler but added to Memory and due to this, the message will actually never be removed from the memory spooler , which is bad.
Description of changes:

Always remove message from Memory Spooler.
Removed a check in persistentQueueSync which is never possible now that we moved persistentQueueSync back to spooler initialization. Earlier the check was added as we were calling queue sync from Disk Spooler implementation, in that case the condition would have occurred when we restarted the disk Spooler component.
Why is this change necessary:

How was this change tested:

[ ] Updated or added new unit tests.
[ ] Updated or added new integration tests.
[ ] Updated or added new end-to-end tests.
[ ] If my code makes a remote network call, it was tested with a proxy.

Any additional information or context required to review the change:
Documentation Checklist:

[ ] Updated the README if applicable.

Compatibility Checklist:

[ ] I confirm that the change is backwards compatible.
[ ] Any modification or deletion of public interfaces does not impact other plugin components.
[ ] For external library version updates, I have reviewed its change logs and Nucleus does not consume
any deprecated method or type.

Refer to Compatibility Guidelines for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Fix your PR title. It must conform to conventional commit format.

Where is the NPE? That needs to be fixed and these changes don't really look like they're fixing the root cause.

To be clear, there would be SQLException due to NPE would be from DiskSpooler Remove method, we catch that SQLException. The removeMessageById from spool wouldn't throw anything. And we should be ok with this scenario, as its just the case where we fallback to adding to Memory.

Why is there an NPE in disk spooler? That needs to be fixed

Why is there an NPE in disk spooler? That needs to be fixed

Never mind, my mistake. I was using a modified exampleSpooler component in my UAT run from which I had the error. The modified disk spooler had some extra logs and was trying to print message from DiskSpooler from within the remove method. Long story short, no NPE.


[Request] MDI Layout

Hello again!,
Is possible to implement Template inject of window layout?
Thank you!

Hi...this is the last thing I need to implement to get to version 1.0...so right now no, there's a bit of the API missing...

P.S...do you mean to set the initial layout (size, position etc) of tool windows?

Yes! Size, position, titlebar template, ecc.
Thank you very much for great library
Tommaso


Question: Create a list of all Elements and their Valuekeys

I am currently facing the problem that i can't find a way to create a list of all elements and their Valuekeys, of a apk. Is there anyway that can be resolved? Using the driver.find_elements(by=AppiumBy.CLASS_NAME,value='android.widget') leaves me with an Error message. So there has to be a way to do it with the Finder.
Any help would be appreciated!

You can switch to NATIVE_APP to using selector of Android element

Sorry, i think i can't quite follow, where can i switch to NATIVE_APP?

https://github.com/appium/python-client/blob/d5b84b899e4abad6afd914d0e5342476cb955de2/appium/webdriver/switch_to.py#L38 should help for the switching context.


test(steps): increase coverage of nontrivial steps

Oop, I think this is a duplicate of #35. Closing.

actually, since you split this out into two separate issues, lets keep this one and close #35.

all steps are covered by unit tests, if new one were found, we could reopen this.


fix workaround for bsc#1181283

What does this PR change?
The bug only affects SLE15SP3, meaning @includedir is only added in SLE15SP3
By having commented #includedir, I introduced a bug for previous
products based on SLE15SP2 or older, which was that there was no includedir
so sudo was not working on those products.
Let's comment the one that starts with @ so it only affects SP3.

Thank you @jordimassaguerpla .

Thank you @jordimassaguerpla .


Cant get Harmful Park to boot (Freezes at the 'Loading screen' which is the first image)

Harmful Park hangs at the vary first screen, the "Loading screen" will just stay frozen on the TV when booting off a Cheat Cart using
Tonyhax 1.2.1i international on a
SCPH-7501 with the game
Harmful Park (JAP)
SLPS-00498 its
Triggered at loading screen/first screen game will hang & never boot into the title, using a blank
verbatium CD-R burnt @1x with the newest version of
My cheat cart is a Datel 128k Pro Action Replay flashed w/tonyhax 1.2.1 international BUT
Note:
The Slot2 tonyhax v1.4.5 works great with Harmful park on a CD-R backup with the same "SCPH-7501/BIOS 4.1" console, no freeze up at any point game plays perfect I tried 4 times no issues what so ever

Hmm, SCPH5501 + Tonyhax International v1.2.1 ROM flashed to GameShark Pro v3.0 cart is working correctly for this game. I will test with my SCPH-7000 next which is the closes I have to your console.

SCPH-7500 + Tonyhax International v1.2.1 ROM flashed to GameShark Pro v3.0 cart is also working correctly for me.

Yeap it was my cart I tried a 512k GSpro3.2 its working now... So weird every other game works perfect what is going on after I cleaned them all wiped the ROM's & re-flashed every issue seemed to go away. Just one game wont work? weirdo city

Yeap it was my cart I tried a 512k GSpro3.2 its working now... So weird every other game works perfect what is going on after I cleaned them all wiped the ROM's & re-flashed every issue seemed to go away. Just one game wont work? weirdo city

Hmm I only have GameShark Pro v3.0s which are 512KB. Essentially any cart should work though I haven't experienced what you have.

Alex has resolved this and we think it was possiblely FreePSXBoot was left in slot 2 some games will not boot via cart with unirom or tonyhax in slot2 I just checked international 1.2.6 same outcome freeze at boot screen
So:
DO NOT LEAVE FreePSXBoot IN WHILE USING CHEAT CART -ROM-
this goes for ANY exploit I've tried not just International


Feature Request:  Allow config file to be split up (importing)

I ran into this issue playing with the github module as well as writing custom modules.
It would be nice to be able import other config files.  While letting the config be more modular, more importantly it lets us keep sensitive stuff out of source control (like API keys)

[module/battery]
{import battery.conf}

or
[module/github]
token = {import secrets.githubtoken}

[module/github]
token = ${file:/absolute/path/to/token}

Welp, I meant to pots this a few days ago, and when I finally do, it turns out that not only you already had the feature, but it was just documented this morning in the wiki as well


Stretchly no splash or settings screen

I'm using latest version: Stretchly.Setup.1.6.0.exe   113 MB downloaded as Admin today   2021 05 03
Description
Stretchly installed and used for approximately one year. Then stopped functioning. App shows up on Windows search but nothing happens on click, double click.  Locate .exe file in folder and click on that - nothing happens.
Operating system:  Windows 10 updated  2021 05 03
Steps to Reproduce
[First Step]  Windows search for app. Stretchly App logo as app appears normally.
[Second Step]  Click on App
[and so on...]
Expected behavior:  A splash or settings screen, something to set frequency and durations
Actual behavior: No result
Reproduces how often: Always
Additional Information
Stretchly continued to appear as App when searched in Windows but non-responsive as above,
Updated Windows, no change.
Uninstalled Stretchly with the uninstall.exe file successfully.
Windows restart. No change.
Downloaded as above and reinstalled. App and .exe file appear but remain no-responsive as above.
Windows restart. No change.
Uninstalled Stretchly again.
Restarted Windows 10 as Admin.
Reinstalled Stretchly from Github as above.
App on Windows Search appears normally. exe file present
App / .exe file still non responsive
Stretchly mini-splash appears "10 seconds to Mini-Break"
Break screen appears with motion instructions, lasts 10 seconds.
SINCE there is no App splash or settings screen it is not possible to tell when this mini-break or anything else is set,
frequency or duration. HOW to get App to normal function, open from App and see settings?
I used the App for about one year and was able to change frequency and durations, etc. but not now.
No other computer user, no repairs, no changes other than Windows 10 updates.
It has now given me a popup for a Long Break in 30 seconds, then 5 min break screen, and postpones normally.
I would really like to know how to open a settings screen as on original install about 2 years ago.
<EMAIL_ADDRESS>

ThinkPad P71
Device name	LAPTOP-RL19VKR6
Processor	Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz   2.90 GHz
Installed RAM	64.0 GB (63.9 GB usable)
Device ID	7CC5E657-D6F0-4917-9D0F-AEBD27C7292D
Product ID	00330-50823-44433-AAOEM
System type	64-bit operating system, x64-based processor
Pen and touch	No pen or touch input is available for this display
Edition	Windows 10 Pro
Version	20H2
Installed on	21/04/03
OS build	19042.928
Serial number	PF10BFFX
Experience	Windows Feature Experience Pack 120.2212.551.0

Hi @stevenmonrad , the menu for Stretchly appears in tray area. Here's how I can get there on win 10:

click on ^

click on stretchly icon

Then you can click on different things :)


Support read only attributes

As described in issue #54 we also have read only attributes. This attempts to handle the issue. I have added org.reflect as dependency, as it allows a bit better reflection handling that the vanilla reflection instruments.
I have run and extended the tests against a local postgresql instance, as it appears to be designed by the TransactionalTestBase.
I would recommend to have a look into Testcontainers that would ease the setup time for other contributors and would allow integration to services like Travis CI.

First of all, Thanks for your contribution!
Feel free to add you to the List of authors in the pom.xml! 👍
Yes, setting up the unit test environment for this project is a mess, I will improve it and as a start user and password credentials will be configurable, so it is not hardcoded anymore. Docker and Testcontainers would make things easier for collaboration. I will look into it.

Oh! org.reflect makes it really a LOT easier. :-) Well done.


React 18 support

Hi! We recently upgraded our react version to react 18, which this package currently does not support.
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^17" from<EMAIL_ADDRESS>npm ERR! node_modules/react-seomatic
npm ERR<EMAIL_ADDRESS>from the root project
Do you have any plan to upgrade react version? Would be great, as it's really useful 👍

I’ve bumped the version to include React 18 as a peer. Give it a try now and please close if it all works.

Awesome! It did not work though. It seems there's missing a "|" (pipe) in the peerDependencies before v 18.


Unable to Download Models Due to Malformed Manifests

I'm running Ollama 0.1.20 in WSL2/Ubuntu.  In the past I was able to download new models fine but now when I try to download them I get something similar to the following error messages and am prevented from downloading:
pulling manifest
Error: pull model manifest: Get "https://registry.ollama.ai/v2/library/codellama/manifests/latest": malformed HTTP response "\x00\x00\x1e\x04\x00\x00\x00\x00\x00\x00\x05\x00\x10\x00\x00\x00\x03\x00\x00\x00\xfa\x00\x06\x00\x10\x01@\x00\x01\x00\x00\x10\x00\x00\x04\x00\x10\x00\x00"

I tried deleting Ollama and reinstalling and the issue persists (I'm not sure if this is the right URL but accessing https://registry.ollama.ai/v2/library/codellama/manifests/latest also gives me MANIFEST_INVALID error when I access it from my browser

Closing issue.  I've figured out the problem, I'd set HTTPS_PROXY in my environment variables and that was causing issues.  Commenting out that line makes everything work as expected


QA: Bumps Dependencies and Improves Static Analysis

Q
A

QA
yes

Description

Drops PHP 7.3
Drops Service Manager v2
Updates latest dev dependencies
Adds template types for plugin manager
Fixes newly surfaced type errors
Drops Interop\Container
Moves Service Manager from dev deps to required deps

Is something wrong with the laminas-ci action 1.20 release? - Everything is passing locally

Yup: new got version broke everything 😩
We need to patch it to add https://stackoverflow.com/questions/71849415/i-cannot-add-the-parent-directory-to-safe-directory-in-git/71904131#71904131 to it.

FWIW, raised https://github.com/laminas/laminas-continuous-integration-action/pull/95, need a review :)


Still an issue related to "Segmentation fault in multi-threaded code"

The user who have reported the issue #304  tried the new version and it fails:
Backtrace (demangled):
[PID=22896][TID=2040][0/7] __restore_rt
[PID=22896][TID=2040][1/7] std::_Hashtable<unsigned long, std::pair<unsigned long const, omnitrace::(anonymous namespace)::cid_data>, std::allocator<std::pair<unsigned long const, omnitrace::(anonymous namespace)::cid_data>>, std::__detail::_Select1st, std::equal_to, std::hash, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true>>::_M_find_node(unsigned long, unsigned long const&, unsigned long) const [clone .isra.848] +0x14 [PID=22896][TID=2040][2/7] omnitrace::hip_activity_callback(char const*, char const*, void*) +0x1321 [PID=22896][TID=2040][3/7] roctracer::MemoryPool::reader_fun(void*) +0x6a [PID=22896][TID=2040][4/7] omnitrace::component::pthread_create_gotcha::wrapper::operator()() const +0x144 [PID=22896][TID=2040][5/7] omnitrace::component::pthread_create_gotcha::wrapper::wrap(void*) +0xa2 [PID=22896][TID=2040][6/7] start_thread +0xdc
Backtrace (lineinfo):
[PID=22896][TID=2040][0/6]
[/lib64/libpthread.so.0:?] __restore_rt [PID=22896][TID=2040][1/6]
[/home/omnitrace/source/lib/omnitrace/library/roctracer.cpp:944] omnitrace::hip_activity_callback(char const*, char const*, void*)
[/usr/include/c++/7/bits/unordered_map.h:920] find
[/usr/include/c++/7/bits/hashtable.h:1426] find [PID=22896][TID=2040][2/6]
[??:?] roctracer::MemoryPool::reader_fun(void*)
[PID=22896][TID=2040][3/6]
[/home/omnitrace/source/lib/omnitrace/library/components/pthread_create_gotcha.cpp:276] omnitrace::component::pthread_create_gotcha::wrapper::operator()() const [PID=22896][TID=2040][4/6]
[/home/omnitrace/source/lib/omnitrace/library/components/pthread_create_gotcha.cpp:308] omnitrace::component::pthread_create_gotcha::wrapper::wrap(void*)
[PID=22896][TID=2040][5/6]
[/lib64/libpthread.so.0:?] start_thread

Tell them to disable critical-tracing support.

That error is a data race for some critical trace data. Critical tracing has always been alpha and development for supporting it has effectively been abandoned in favor of causal profiling. Realistically, it should be removed at this point.
I think the option is OMNITRACE_CRITICAL_TRACE=OFF

This will be fixed by OMNITRACE_CRITICAL_TRACE=OFF. Critical tracing will be removed soon as it was incomplete due to it being superseded by causal profiling.


What is the classification of streams in the left sidebar?

The sidebar on the left lets me "pin" streams. But there seems to be another distinction Zulip is making, since the list of streems is separated into three categories:

pinned
unpinned
???

I have yet to figure out what that last category is about. I thought it might have to do with activity, but I have sometimes looked for streams with recent discussions in them and ended up finding them in that bottom list. So I have no mental model for what that list is, or how to control which streams end up there. I mostly know that when I am sure I am subscribed to a stream, and cannot find it, it probably was moved to the bottom thing. IOW, this third category is sometimes a source of frustration, but so far has not been useful to me.
This is less of a bug report than a feedback of the form "users cannot tell what your UI is doing". :)

I have now found https://rust-lang.zulipchat.com/help/manage-inactive-streams, so it seems my initial guess about activity was right. But then this seems to be buggy, since some streams in there had more recent activity than other streams that are not demoted. Maybe that's another instance of https://github.com/zulip/zulip/issues/17903? This is in the same large high-traffic instance, where Zulip seems to often miss some recent activity.

Thanks for the report! You've found the right help center article for this. The issue is related to #17903; streams are demoted if they have 0 messages in the browser's local cache, discussed there. This ends up being pretty useful in organizations that have streams that don't make sense to archive but haven't been used for weeks/months, since they don't clutter the UI.
When we're able to fix the browser's cache to have a set of messages defined by a rule like "All messages in the last 5000 messages or the last 14 days, whichever is longer" that we feel good about and can explain to users, I expect that to naturally fix this problem. We're actively working towards that goal; but it's one of those projects that's a chain of 10 different complex refactorings, and it's hard to guess how long that will take.
I'm going to close this issue because from an implementation perspective, it's a duplicate of #17903, namely that the browser cache that powers both of these features doesn't work how it should.


[WIP] feat: Persistent Relay cache

The type of this PR is: Feature
Resolves CX-2402

Description
This is an attempt to implement the "stale-while-revalidate " caching pattern with @wora/relay-store in combination with Relay's new "store-and-network" fetch policy.

PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[ ] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

New persistent Relay cache (behind feature flag) - ole

iOS user-facing changes

Android user-facing changes

Dev changes

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against bf57f114c9cfa4fbdb6dff24b6683f4a8bebb6b3

@olerichter00 do you think we should keep this open or should we close for now?

@olerichter00 do you think we should keep this open or should we close for now?

@gkartalis we can close this for now and reopen in case I pick the work up again 👍


Fix typo in Connection.php

avaiable -> available

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
-

Thanks.

Thanks.


chore(ci): specify environment using provided field

#1460  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @krzema12 and the rest of your teammates on  Graphite


plugins folder issues

I am trying to use the plugins folder to deploy a custom sensor and can't get it to work in the plugins folder. If I copy the sensor folder/file to the dags folder I can at least import the custom sensor and use it as a workaround
The first issue I encountered was the aws-mwaa-local-running/plugins folder had root ownership so I couldn't put plugin folders/files in the folder. I used chown to change the ownership and was able to drop folders/files in.
Files I drop in compile successfully (a pycache folder is created), but they do not show up in the plugins page in the Airflow.

closing - it looks like this may be a misunderstanding about how plugins need to be written to show up in the plugins folder.


fix(deps): add dnf repositories for build dependencies

Describe the changes and your approach

Verification Checklist

[ ] I performed a self-review of my changes and approve them.
[ ] I checked of the automated test results for any irregularities.
[ ] I manually tested the code changes on my local environment.
[ ] I considered changes to the README or test docs. (Check if not applicable.)

Amazon Linux and RHEL are still failing, but Rocky and Oracle are passing now by adding the epel release package.
Amazon Linux doesn't support the epel release, so need to create a separate issue to potentially throw and error if compile is run on Amazon Linux.
RHEL is failing for a separate reason, but troubleshooting is difficult without an enterprise license.


fix bug with inlining isArray where src isn't RegOpnd

In such a case, we need to load into a RegOpnd, but we were doing this in the wrong location.

:shipit:


Tautomer canonicalizer invariant violation

Bug description:
Canonicalizer has a runtime error on a specific structure O=[N+](O-])/C=C/c1ccccn1`. Error cannot be caught by try except. Error seems to be related to the stereochemistry as it does not occur when deleted from SMILES string. Error is related to latest versions of rdkit as it does not happen in rdkit==2023.3.3. Initial error happened on Google VM with python 3.11.5 and conda environment within installed rdkit==2023.9.4 but it may not be relevant as error reproduces in Google Colab using pip install.
Code to reproduce:
from rdkit import Chem
from rdkit.Chem.MolStandardize import rdMolStandardize

try:
  mol = Chem.MolFromSmiles('O=[N+]([O-])/C=C/c1ccccn1')
  enumerator = rdMolStandardize.TautomerEnumerator()
  enumerator.Canonicalize(mol)
except Exception as err:
  print(err)

Also look at Google colab here.
Expected behavior:
Should not crash as molecule is valid and ideally should be able to be caught by try except to avoid interrupting workflow.
Configuration:

RDKit version: 2023.9.4
OS: [e.g. Ubuntu 20.04]
Python version (likely not relevant): 3.11.5
Are you using conda? Yes - miniconda
If you are using conda, which channel did you install the rdkit from? conda-forge

Confirmed.
Thanks for the detailed bug report @fioruggiu !

Another example is an oxime. Note, there is no error if SetRemoveBondStereo(True), or if the oxime has no bond stereo to begin with.
$ docker run --rm -v $(pwd):/app python:3.11-slim sh -c 'pip install -q rdkit; python -c "import rdkit; import rdkit.Chem; from rdkit.Chem.MolStandardize import rdMolStandardize; enumerator = rdMolStandardize.TautomerEnumerator(); enumerator.SetRemoveBondStereo(False); canon = enumerator.Canonicalize(rdkit.Chem.MolFromSmiles(\"O/N=C/c1ccccn1\"))"'
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv

[notice] A new release of pip is available: 23.2.1 -> 23.3.2
[notice] To update, run: pip install --upgrade pip
[20:33:25]

****
Invariant Violation
could not find atom2
Violation occurred on line 228 in file /project/build/temp.linux-x86_64-cpython-311/rdkit/Code/GraphMol/Canon.cpp
Failed Expression: firstFromAtom2
----------
Stacktrace:
----------
****

Traceback (most recent call last):
  File "<string>", line 1, in <module>
RuntimeError: Invariant Violation
        could not find atom2
        Violation occurred on line 228 in file Code/GraphMol/Canon.cpp
        Failed Expression: firstFromAtom2
        RDKIT: 2023.09.4
        BOOST: 1_78

Bisected to https://github.com/rdkit/rdkit/pull/6643
In debugging the issue, I saw the canonicalizer is generating a molecule that couldn't be converted to smiles. The graph looked like

I'm afraid that the molecule has "directions" still on the double bond, like a (for lack of a better representation) c/C=C/=[N+](-[O-])O,

Here's a minimal reproducible of this that does not require tautomer canonicalization:
In [2]: m = Chem.MolFromSmiles('CC=C=CC')

In [3]: m.GetBondWithIdx(2).SetStereoAtoms(1,4)

In [4]: m.GetBondWithIdx(2).SetStereo(Chem.BondStereo.STEREOCIS)

In [5]: Chem.MolToSmiles(m)
[09:32:30]

****
Invariant Violation
could not find atom1
Violation occurred on line 222 in file /localhome/glandrum/RDKit_git/Code/GraphMol/Canon.cpp
Failed Expression: firstFromAtom1
----------


Mod loads up (seen in mod menu) but doesn't show up in the inventory UI

Expected Behaviour
Mod to show a tab on the 2nd page of inventory UI
Actual Behaviour
It doesn't show in the 2nd page of inventory UI
Reproduction Steps
I have other mods that take up the 2nd page like Adorn, Automobility (This one might be causing the problem, but not sure), McCaw's Furniture, Immersive Aircrafts and Handcrafted. MTR was showing up when some of the above mods were not there, but ones the above mods are installed, it doesn't come up
Environment
Minecraft Version 1.20.1 (fabric loader version 0.15.1
Total 38 mods are installed but none affected this mod's behaviour before, only when some more mods were added to the 2nd page, this mod didn't show up in the inventory UI.

1.20.1 has the items in the Vanilla Tabs. And its not recomended yet

This should be fixed in 4.0.0. Please reopen this bug if you still encounter this issue with 4.0.0, thanks!


Ability to send traces to AWS X-ray

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
It would be good to have the istio envoy out of the box send traces to X-ray. We can run a X-ray agent daemon set / Distro for OpenTelemetry to collect the traces and send it to X-ray.
The current version of envoy used by Istio doesn't have support for X-ray is what I read. Correct me if I'm wrong.
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

@saaish PTAL https://aws-otel.github.io/docs/getting-started/x-ray
gernerally envoy support to send trace to OTEL in zipkin protocol

@zirain should we own a write-up of OTel tracing integrations via the collector and Telemetry API?

a simple example can be found here, plan to add a doc in istio.io, but I'm poor with writing in English.

a simple example can be found here, plan to add a doc in istio.io, but I'm poor with writing in English.

No worries on the English side. I'm happy to help get it into shape.

linked to https://github.com/istio/istio.io/pull/10977

I would like to use Istio to send traces to X-ray. I tried to follow the comments in this thread, but I couldn't get the results I wanted.
There are a few things I don't understand:

Can I use Telemetry CR to configure Istio if I used Helm for the installation?
If I enable Istio to send accesslog to the OTel collector is it the same as sending traces?
Is there any way to send traces from istio-proxy and have the "Service Map" and "traces" section populated on x-ray?

Hi @matcasx did you make any progress on the Istio to AWS X-Ray integration via ADOT? Right now there seems to be very limited info on sending Istio traces via OpenTelemetry to AWS X-Ray with ADOT. Is there a recommended approach suggested by Istio team for this?

I see two ways of integrating with X-Ray, but none of them is ideal.

Using Istio/Envoy's native support of zipkin and otel-tracing with Otel Collector

AWS X-Ray trace id is looks like "1-63441c4a-abcdef012345678912345678". The first token is the version (always 1), the second one is the current timestamp and the final one is a random string. By removing the hyphens and version it becomes a 32-alphanumeric id that could match up with both otel-tracing and zipkin. The problem is that X-Ray validates the timestamp part, which should be within the last 30 days and, really, it should be the current timestamp to properly use the search tools in AWS X-Ray.
So for it to work the tracing headers must be generated externally to istio, so istio can propagate it. Additionally, envoy's spans do not conform with otel semantic conventions so it requires additional transformation.
A configuration for that looks like that:
Istio Config
    extensionProviders:
    # either
      - name: zipkin
        zipkin:
          service: amazon-xray-otel-collector.amazon-xray.svc.cluster.local
          port: 9411
      - name: otel-tracing
        opentelemetry:
          port: 4317
          service: amazon-xray-otel-collector.amazon-xray.svc.cluster.local

And it requires an otel collector pipeline that receives otlp/zipkin and exports to x-ray.
 processors:
    attributes/fix_status_code_type:
      actions:
        - key: http.status_code
          action: convert
          converted_type: int

    span/set_status_ok:
      include:
        match_type: regexp
        attributes:
          - key: http.status_code
            value: "^(2|3).*"
      status:
        code: "Ok"

    span/set_status_error:
      exclude:
        match_type: regexp
        attributes:
          - key: http.status_code
            value: "^(2|3).*"
      status:
        code: "Error"
services:
    pipelines:
      traces:
        receivers:
          - otlp
        processors:
          - span/set_status_ok
          - span/set_status_error
          - attributes/fix_status_code_type
          - batch
        exporters:
          - awsxray

Using Envoy Filter to setup Envoy's native X-Ray support

Envoy's X-Ray extension provides a better way to integrate with X-Ray, but it can currently only be configured via Envoy Filter in Istio. It creates a problem that the node name cannot be configured per workload, so it is a hardcoded value and all the traces appear under a single node in X-Ray console. Presumably Aws App Mesh configures this extension with the correct value of "segment_name".
The filter looks like:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: add-xray-tracing
spec:
  configPatches:
    - applyTo: NETWORK_FILTER
      match:
        context: ANY
        listener:
          filterChain:
            filter:
              name: "envoy.filters.network.http_connection_manager"
      patch:
        operation: MERGE
        value:
          typed_config:
            "@type": "type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager"
            tracing:
              http:
                name: envoy.tracers.xray
              provider:
                name: "envoy.tracers.xray"
                typed_config:
                  "@type": type.googleapis.com/envoy.config.trace.v3.XRayConfig
                  segment_name: "Mesh" # This should be statically set and can't be configured per workload!
                  "@type": "type.googleapis.com/envoy.config.trace.v3.XRayConfig"
                  segment_name: Envoy
                  daemon_endpoint:
                    protocol: UDP
                    address: <IP_ADDRESS>
                    port_value: 2000

It would be great if we could have this Envoy's X-Ray extension to be configured via Istio!


flower to stone

flower change to stone
ss

Stone is translated as follows:
125:        blocksToWeb.put("STONE", 6);
136:        blocksToWeb.put("COBBLESTONE", 11);

397:            case 6: material = Material.STONE; break;
402:            case 11: material = Material.COBBLESTONE;

not sure how flowers are getting translated to stone, the default for untranslated materials is DIAMOND_ORE.

Tested all the flowers with WebSandboxMC v1.0.3, relogged and the white flower (22) and blue flower (23) placed from the web client change to stone:

There are more flower types in NetCraft than in Minecraft... I'll translate to red rose and yellow flower for now.